Repository navigation
Record live Runs and add machine-wide status - #777
Merged
Merged
Conversation
This was referenced Oct 18, 2026
Merged
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
Adds host, issue/title, Harness/Model/Effort, Session log, stage history and stage age. Documents the version 1 JSON API for agents and the Console. Record write failures warn once without changing the Run's outcome. Broader Command kinds and history remain in #711's later Tickets.
Closes #768
Evidence
cargo test --test statusrejectedstatusas an Issue URL. Exact Spec reproductions failed on incorrect retained branch facts and colliding record filenames.After: All nine status tests, all nine Continuation tests, and all eleven Command-log tests pass. All three branch reproductions and the exact collision test pass; their tests are retained.
cargo fmt --check;cargo clippy --all-targets -- -D warnings;cargo nextest run --test-threads 8 --no-fail-fast --status-level fail --final-status-level fail— 1,881 passed, 2 skipped. The fullcargo testrun also passed; Nextest validates the final committed tree including the last record-isolation fix.proc_pidinfostart timestamps and zombie detection; macOS execution remains for CI.Test seams: the
thirdshiftbinary with the existing fakes and a waiting Harness; the existing recorder effects adapter for write failures and reused process ids.Review coverage: Standards and Spec each read all 18 changed files, including supporting sources. No generated-output exclusions. No changed file was left unread. The Standards stage-type finding and both Spec findings (branch retention and record ownership) are addressed.
Unaddressed findings
Merge Danger
Door: two-way
Additive Command records and a read-only CLI; existing logs remain usable after rollback.
Blast Radius: observability
Run lifecycle recording and machine-wide status. Recording failures leave delivery outcomes unchanged.
Built with codex · gpt-6.1-sol · high