seqcli trace command - #490
Merged
Merged
Conversation
Assisted-by: Claude:claude-fable-5
Assisted-by: Claude:claude-fable-5
Assisted-by: Claude:claude-fable-5
Assisted-by: Claude:claude-fable-5
nblumhardt-ro
commented
Aug 18, 2026
|
|
||
| if (_json) | ||
| { | ||
| // TODO - better to support JSON and set custom help text. |
nblumhardt-ro
commented
Aug 18, 2026
| these results. | ||
| Find traces by searching for their included spans. | ||
|
|
||
| To reconstruct a trace as a call tree, use the `seq_load_trace` MCP tool, or `seqcli trace -i <trace-id> --json`, |
Author
There was a problem hiding this comment.
Might be better not to show the full CLI command, here; callers should typically provide --columns, so either we include examples or force seqcli trace --help usage.
nblumhardt
marked this pull request as ready for review
August 19, 2026 07:38
KodrAus
approved these changes
Aug 20, 2026
KodrAus
left a comment
Member
There was a problem hiding this comment.
Some of the test cases here look a bit dubious; they're likely to break in difficult to fix ways if we adjust the output format, but we can reconsider them if/when that actually happens.
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
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.
Pretty-prints hierarchical traces:
A
--jsonversion, and MCP tool, serve automation use cases.I like the
--columnmechanism, here. We can't do precisely the same thing for searches, since they can't currently include arbitrary expressions to attach to results server-side, but local expression evaluation would actually get us pretty far 🤔