trace: field-report findings 3-5 — command evidence, container warnings, width control - #26
Open
lroolle wants to merge 4 commits into
Open
trace: field-report findings 3-5 — command evidence, container warnings, width control#26lroolle wants to merge 4 commits into
lroolle wants to merge 4 commits into
Conversation
session_git_root_missing fired whenever the session's recorded cwd
didn't resolve on the current filesystem — even when the process-cwd
fallback then found the right repo. In containerized/remote setups the
recorded path is a host path, so every trace carried the warning
despite correct correlation.
A successful fallback now records git.resolved_from ("session_cwd" |
"process_cwd") on the correlation block; the warning fires only when
nothing resolves.
Closes #22
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ToolCallEvidence answered "did this call touch the workspace" (paths, mutation flags) while hiding how: for Bash only redirect targets survive path extraction, so auditing a Bash mutation meant opening the raw JSONL. An audit trace's whole job is "what did it touch". Evidence now carries a bounded one-line summary of the command for command-carrying tools (providers normalize their shell dialects onto the "command" input key). Paths are unchanged; non-command tools carry no summary. Closes #21 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
outlineHeadlineRunes was a constant: headlines cut mid-word at 160 runes and the only escape was --full, which dumps the entire bundle. The cap also ran before JSON marshaling, so script/skill consumers got the same truncation with no recourse. BuildOutline now takes the width; `ccx trace --width N` sets it, 0 disables truncation, default stays 160, and it applies to text and JSON outlines alike. Closes #23 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This was referenced Jul 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes out the 0.11.0 field report (findings 1-2 shipped in c9c9a7b for v0.12.0). One commit per finding:
session_git_root_missingfired on every containerized trace even when the process-cwd fallback found the right repo. A successful fallback now recordsgit.resolved_from("session_cwd"|"process_cwd") on the correlation block; the warning fires only when nothing resolves.ToolCallEvidencegains a bounded one-linesummaryfor command-carrying tools (providers already normalize their shell dialects onto thecommandinput key). Paths answered "did it touch the workspace" while hiding how; auditing a Bash mutation meant opening the raw JSONL.ccx trace --width Ncontrols outline headline truncation. Previously a 160-rune constant applied before JSON marshaling, so script/skill consumers were stuck too; the only escape was--full's entire bundle. 0 = untruncated, default stays 160, applies to text and JSON outlines alike.Closes #21, closes #22, closes #23.
Test plan
go test ./...green,go build ./...clean🤖 Generated with Claude Code