Repository navigation
feat(cli): explain delivery evidence across supported routes - #325
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. |
|
The unchanged Control dependency-audit blocker is now tracked separately in #326 and fixed by dependency-only PR #327. That PR updates compatible Vite/Vitest patches, preserves the existing audit gates, and has successful local production/full audits. After normal approval and merge of #327, refresh or rerun this PR against the updated main; this comment does not claim current #325 CI is green. |
|
The owner-approved additional #323/#324 implementations are pushed to the existing PRs:
Independent Claude/Codex code-review findings and verified Gemini translation suggestions were addressed before the review handoff. Precise model/input/version limits are recorded in the PR bodies; this does not imply a new cloud review of an identical final tree. A separate local combined tree includes all six ordinary v1.3 feature PRs (#325/#328/#329/#330/#331/#332), detached head No PR/main merge, version bump, tag, release, publication, installation or operations change was performed. The integration commits are local only; normal sequential merges will still need current-head protections and checks. Research #245/#247 and Draft #246/#248 are assigned to Long-term. |
Summary
Completes the approved v1.3 explanation scope for #323, expanding the original local/SSH text dry-run slice:
send --dry-run --explaincovers local, SSH and paired receivers;doctor --explainexplains the existing diagnostic snapshot;handoff status --explainexplains an existing read-only intent snapshot.delivery-explanation-v1; an old receiver gets the unchanged resolve body and preserves known native facts with unknown context. No extra probe, connection, native submission, collector or fallback is introduced.Refs #323. No merge, runtime version bump, tag, release, package publication or operational change.
Validation
Tests use isolated offline fixtures/mocked native operations and loopback components; no real agent/model/SSH-account/hosted deployment acceptance is claimed.
Independent code review
Before push,
independent-cli-reviewran Claude Code 2.1.308 (requested and observedclaude-opus-5-5) and Codex CLI 0.163.0 (isolated default; actual model not exposed) in parallel. Both received identical frozen self-contained source/test excerpts, completed with exit 0, and left input SHA-256a6e7c8b3f931c111104ff26ac08361b34c391663904ecbb5a9a4393f580c35b1unchanged. Claude reported no execution error; Codex completed its turn without command-execution events. Reviewers had no file-edit, test, network or external-session authority.Confirmed findings were corrected: Claude context prefix, paired real-CLI validation-stage attribution, actual-route display, unsolicited context rejection, failure evidence labels, stored receipt time wording and bidi sanitization. Root additionally guarded the optional-context byte-cap boundary. Regressions exercise actual generated code and real CLI-to-renderer paths rather than mocking the formatter. A separate read-only subagent inspected the corrected integration and found no blocker. These corrections are not represented as a second exact-head CLI review or live security/transport acceptance.
Translation quality review
Antigravity CLI 1.4.5 explicitly selected available
gemini-3.8-flash-medium, medium effort, plan mode and sandbox, five-minute limit. One fresh structured review completed with exit 0 in 147.064 seconds; no permission bypass, repository exposure, tools, network or edits were authorized. Input covered the frozen changed explanation sections in English/Korean/Japanese/Simplified Chinese. Prompt SHA-256:a6e19a9283239a750f16029835e2c0242ac4f553ee1108901bd4f78effed3caf; evidence SHA-256:fc8d9da6c7e1e2f723da1a519b9d14f36ee96cc126ecb5b7cba823949def4588.Codex checked proposed edits against the English contract and corrected Japanese context wording and Chinese no-new-schema wording. Overstated model claims about doctor wording were not adopted. Code, options, links, numbers and supported-scope limits are preserved. The selected model is recorded; independent backend identity is not exposed. This is linguistic/semantic QA, not runtime/native validation.
Integration gates