Repository navigation
feat(cli): show per-step execution timelines - #157
Draft
Ehsan-hash wants to merge 1 commit into
Draft
Ehsan-hash wants to merge 1 commit into
Ehsan-hash wants to merge 1 commit into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This branch has not been deployed
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.
Summary
jamjet runcurrently ends its followed text output with the execution status. It now also shows recorded step results and durations, making slow or failed steps visible without opening the event JSON.Related
Closes #8.
Changes
The timeline displays completed, failed, cancelled and skipped node events in recorded order, including repeated node IDs. Known durations receive proportional bars. Missing timings remain unknown, and the total is labelled recorded step time because parallel steps need not equal elapsed time. JSON, streaming and
--no-followretain their existing behavior. Unavailable event history does not hide the final execution status.Testing
mypy jamjet/cloudpassed.python-sdk-v0.12.0runtime completed three isolated condition nodes and displayed their persisted durations, including zero, with the correct total. The original CLI omitted the timeline. This checks released-runtime compatibility; the current Rust source was not built locally.The two existing CLI-file mypy errors also reproduce on the baseline. Rust tooling and
justare unavailable locally, so their full recipes are not claimed. This PR stays draft until CI passes, following the contribution guide's review gate.Checklist
just test; Python SDK checks above passed)just lint; Python SDK checks above passed)CHANGELOG.mdupdated (CONTRIBUTING assigns this to the release PR)Disclosure: This contribution was developed with AI assistance (OpenAI Codex), with automated tests, an isolated runtime check and independent code review.