Repository navigation
feat(identity): pin Claude sends to observed process generations - #286
Merged
Merged
Conversation
abruption
marked this pull request as ready for review
October 9, 2026 00:17
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. |
8 of 9 tasks
Open
3 tasks
9 tasks
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
Refs #180, which remains open: Codex and other native profiles do not provide a proven atomic generation precondition.
Add opt-in
list --with-target-generationandsend --target-generation. Claude tokens bind the home, native process-creation evidence and inbox endpoint. Before any message or Windows authentication bytes, check the connected socket/pipe server PID and original process evidence. Changed, missing or unavailable identity is refused; unsupported Codex, Antigravity and paired-device paths refuse before submission or wake. Unpinned output and Python Claude optional absence remain unchanged.Forward and check generation results over source-streamed SSH without retries. Missing or incorrect generation echoes retain returned native target, chars, dry-run and SSH metadata while reporting unknown/no automatic retry. Completed generation refusals remain refused. A
stale_targetrefusal now includes the closed, boundedlastSeenTarget: {agent: "claude", targetGeneration: ORIGINAL_TOKEN}: the original requested token, not successor metadata. Unavailable/unsupported/invalid/default results do not gain this field. No raw creation values, paths, user names or host names are added.The optional SSH refusal metadata must match an original single Claude
sendtarget and generation. Malformed optional metadata cannot establish a completed refusal. Complete positive results accepted by the existing ordinary parser retain their existing semantics with invalid optional metadata removed; this is not a new full native-context validator.A private POSIX effect/total monotonic deadline interface bounds socket connect/write/drain for handoff integration. Windows bounded pipe submission refuses before effect. Post-write failure stays unknown/no-retry rather than inventing
submitted:false. Filesystem/native identity inspection is not universally cancellable.Validation
At
df610896cc0c889c767f6096e6c685de4bccf97d:packaging==26.3: 623 tests passed, 29 skipped. No optional Relay/MCP packages were installed in this environment. An earlier system-interpreter run failed only because the release-artifact test lackedpackaging; the isolated rerun above supplies that existing test dependency without changing shared installations.6e2f9f8had a separate complete extras run of 805 tests (12 skipped); that historical run does not cover this new metadata implementation.7e8afb7, notdf61089. No current-head automatic-review approval is inferred; no mentions or manual review triggers were posted.Boundaries
No actual Claude/Codex/model message, physical remote-host SSH delivery, real Windows pipe experiment or new Linux live-platform experiment was performed locally. The owned Unix protocol inbox is not a native agent. Fake SSH proves source/argv/wire integration, not SSH authentication or fleet acceptance. It does not prove actual PID recycling, receiver deduplication or exactly-once delivery.
Native creation values have OS-specific granularity and are not globally unique or clone-proof authentication. Same-account adversarial endpoint manipulation is outside this precondition. The generation token is not a secret, permission, ACK, retry key or cryptographic SSH host-key continuity. Native Codex queue insertion cannot atomically constrain a later consuming writer; its unsupported boundary stays explicit and #180 remains open.
No merge, release, version bump, deployment, policy change or credentials/private session identifiers in the diff. Automatic Codex review and required current-head checks are inspected without mentions or manual review triggers.