fix: gate Error_*.txt snapshots behind GALAXY_BEDROCK_DIAGNOSTICS env var
Previously, Bedrock error snapshots were unconditionally written to disk on any API or stream failure. Now they are only written when GALAXY_BEDROCK_DIAGNOSTICS=1 is set, consistent with the rest of the diagnostic logging. Bumps version to 1.2.1.
This commit is contained in:
Generated
+1
-1
@@ -5191,7 +5191,7 @@ dependencies = [
|
||||
|
||||
[[package]]
|
||||
name = "galaxy"
|
||||
version = "1.1.0"
|
||||
version = "1.2.1"
|
||||
dependencies = [
|
||||
"addr",
|
||||
"aho-corasick",
|
||||
|
||||
@@ -37,9 +37,9 @@ Environment variables:
|
||||
- `find . -name "*.wgsl" -exec wgslfmt --check {} +` - Check WGSL shader formatting
|
||||
|
||||
### Bedrock Diagnostics
|
||||
- Bedrock request or stream failures automatically write `Error_<timestamp>.txt` to the repository root.
|
||||
- The error snapshot file includes the serialized Bedrock context window, tool definitions, protobuf request debug payload, captured Bedrock diagnostic lines, and log tails.
|
||||
- Set `GALAXY_BEDROCK_DIAGNOSTICS=1` to additionally write per-event Bedrock diagnostic logs to `bedrock-diagnostics.log` in the active Warp log directory.
|
||||
- Set `GALAXY_BEDROCK_DIAGNOSTICS=1` to enable Bedrock diagnostic output, including:
|
||||
- `Error_<timestamp>.txt` snapshot files written to the repository root on request/stream failures (includes the serialized Bedrock context window, tool definitions, protobuf request debug payload, captured Bedrock diagnostic lines, and log tails)
|
||||
- Per-event Bedrock diagnostic logs written to `bedrock-diagnostics.log` in the active Warp log directory
|
||||
|
||||
### Platform Setup
|
||||
- `./script/bootstrap` - Platform-specific setup (calls platform-specific bootstrap scripts)
|
||||
|
||||
+1
-1
@@ -5,7 +5,7 @@ description = "Galaxy - AI-powered terminal"
|
||||
edition = "2021"
|
||||
autobins = false
|
||||
name = "galaxy"
|
||||
version = "1.2.0"
|
||||
version = "1.2.1"
|
||||
publish.workspace = true
|
||||
license.workspace = true
|
||||
|
||||
|
||||
@@ -211,6 +211,9 @@ impl BedrockDiagnosticLogger {
|
||||
}
|
||||
|
||||
pub fn dump_error_snapshot(&self, error: &str, debug_error: &str) -> Option<PathBuf> {
|
||||
if !is_enabled() {
|
||||
return None;
|
||||
}
|
||||
let conversation_id = self
|
||||
.conversation_id
|
||||
.lock()
|
||||
|
||||
Reference in New Issue
Block a user