Fix: check TRACE_TOKEN env var before keyring#58
Merged
Conversation
The examples README referenced non-existent commands (trace start, trace stop, trace list, trace show, etc.). Rewrite with the actual CLI surface: trace enable, trace status, trace session list/info/resume/export/replay, trace checkpoint list/rewind, trace fork, trace investigate.
The version check in PersistentPostRun was synchronous with a 2s timeout, adding up to 2 seconds of latency to every command invocation. Run it in a goroutine so it never blocks command completion.
Adds a --json flag to 'trace session list' that outputs the filtered session list as indented JSON instead of styled cards. Mirrors the existing --json support on 'trace session info'.
…p/ pkg
- Add --json flag to `trace recap` emitting MeRecapResponse as indented
JSON (encoder writes to cmd.OutOrStdout(), consistent with sessions list).
- Add --json flag to `trace plugin list` emitting []InstalledPlugin.
- Add --json flag to `trace agent list` emitting [{name, installed}] entries.
- Add JSON tags to InstalledPlugin (Name, Path, Symlink, LinkTarget).
- Delete the dead mcp/ package — a basic JSON-RPC MCP server that was
never imported or referenced anywhere in the codebase.
- Add tests: JSON output paths for recap (mock HTTP server + git repo),
plugin list, and agent list. Update existing agent list tests for the
new jsonOut parameter. Update flag-registration test to include --json.
All cli, cli/recap, and cli/api tests pass.
Co-Authored-By: Grok <noreply@xai.com>
- Add cli/graph_cmd.go with graph command - Add cli/graph_cmd_test.go with test coverage - Update cli/root.go, root_test.go, go.mod - Update README
- Add JournalGraph for tracking execution flow - Record transitions, durations, and status - Support JSON serialization and portable GraphSpec conversion
Patel230
force-pushed
the
fix/trace-token-env
branch
from
July 26, 2026 08:52
0512cf5 to
d11c1c9
Compare
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.
Check TRACE_TOKEN environment variable before falling back to keyring lookup, fixing CI test failures where no keyring is available