Repository navigation
feat(handoff): add read-only status and bounded request watch - #329
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
CI follow-up: 9000ef6 bounds each sleep slice while retaining the same sampling interval, original total deadline, 100ms finish reserve, exit contract and no-submit boundary. A deterministic 10% sleep-slack model fails on the original code and passes after the change; it demonstrates a counterexample, not proof of the historical CI scheduler cause. Bounded output-phase/remaining-budget/helper-exit diagnostics were added to the two real-output assertions. e69e582 removes an expired relay room before asynchronous cancellation cleanup can yield, and rejects new arrivals against the original room deadline. Two deterministic regressions cover cleanup-yield and delayed timeout callback races; one was confirmed failing against the original code. The separate native relay-send failure did not reproduce in 30 fresh isolated executions; its assertion now reports only protocol diagnostics, without weakening success requirements or adding retries. Validation: Python 3.13 extras full suite 1,381 OK (19 skips); Python 3.9 watch/CLI 36 OK; relay suite 238 OK; generator/diff checks clean. Read-only subagent audit found no blockers. No claim that all historical CI causes are proven. New exact-head CI is required before merge. |
|
Exact-head CI verification: e69e582 now has 28/28 successful checks, including macOS Python 3.9/3.13, both macOS relay jobs and the release gate. The Python 3.13 log explicitly records both previously failing real-output tests and the new sleep-slack regression as OK. The separate native relay-send test also passed in the new run; its original failure reason remains unproven, so this is not a claim that every historical failure cause has been established. No CI protection bypass or test-success weakening was used. |
Summary
Adds a read-only local handoff status view and bounded sampled
handoff watchfor #296. No new receipt, native submission or observation capability is introduced.handoff watch --correlation-id UUID [--watch-timeout 1..60] [--interval-ms 1000..5000] [--json]observes one existing local POSIX intent. Defaults are 30 seconds/1,000 ms; only ASCII integer tokens are accepted. It emits the first valid snapshot if budget permits, then changed public metadata/history phase, and an end frame when output permits. This is polling, not a lossless event feed.Refs #296. Candidate functionality; no version bump or release is included.
Validation
claude-opus-5-5; Codex CLI 0.163.0 used its isolated default (actual model name was not exposed). Both had the same frozen sanitized input per round, successful completion and no tool/test/network authority. Input SHA-256:336ec24930a54ef662dbef901647a33d00806619e211a79ce45a1d1c80beefcbandd15033c2881081b5f84d2bc548f49b253a1cb3b5353b0b1c4f32d242c50cd900.Translation quality review (2026-10-23)
At the user's request, Antigravity CLI 1.4.5 explicitly selected available
gemini-3.8-flash-medium, medium effort, plan mode and sandbox with a five-minute limit. A task-local runner omitted permission-bypass flags. One fresh read-only review completed with exit 0 and structured output (160.625 seconds). It covered all 152 added/changed lines across eight English/Korean/Japanese/Simplified Chinese CLI reference/runtime guide files atb3c4929, plus heading/context, preserving code and links. Frozen evidence SHA-256:7c43a4a465784dacf087eed07afa208c8e557b34f6d14c23674536df5c7b7e0e; full prompt SHA-256:3bb94a8909bf7e01f507386c04492ac88443ea14c073dc663d895b2de04838d6. The model selector is recorded; independent backend identity is not exposed by the normalized result.Codex verified the suggestions against source and applied four narrow Korean wording changes: submission terminology, snapshot/no-lossless-stream separation, and two OS real-time-limit sentences. No runtime, numbers, flags, code blocks, links, English source or manifest change. Inherited consumed-intent wording and an unsupported inferred state distinction were not changed. Final documentation tests: 65 OK on both Python 3.9 and 3.13; diff check clean. Static translation QA only, not native/runtime/security validation; earlier CLI code-review evidence remains scoped to its recorded input, not a re-review of this documentation-only follow-up.
Remaining limits (unchanged)