Repository navigation
Conversation
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
trissim
marked this pull request as draft
October 19, 2026 18:37
Integrates the source of pi-bridge-compact checkpoint d9e6dae without importing its malformed prose budget trailer. No published branch is rewritten. Registered input invokes frame behavior; ready reader variants own submission and acknowledgment. External stdout PIPE validation belongs to StreamingChildStdio, which declares that pipe.
Only native replayed user UUIDs acknowledge managed input. Other SDK output uses the existing OpenWire foreign case and remains visible. Watch states own enrollment and session hook decisions; startup waits on the socket watcher without sleeps. Managed input holds the existing reader lock, so WatchGuard no longer falsely reports a missing Monitor.
Compaction supplies its registry observer to the existing watch-state capability, preserving the stdlib-only prompt/tool hook path without function-local imports. The integrated SessionStart daemon, typed transport protocol, admission, owner lifetime and reader-state family add 1188 measured Core source lines relative to current main. Formatting and retained hook source are visible in the same PR; no baseline or checker is edited. Src-Budget: 1188
Drop formatter-only edits on three pre-existing absence sites. This does not introduce or redesign their existing semantics; the changed implementation remains typed and state-owned.
Contributor
Author
|
Closing stale draft: no commit since yesterday and nobody has mentioned it on either commander's bus since midnight. Branch kept; reopen with a current owner if the work is still wanted. |
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.
P205 checkpoint — not merge-ready
Current pushed head:
2e82c0959. Includes installed mainb793c7325by normal merge; published history was not rewritten.Working path
Runonly owns real Claude stdin/output.IncomingFrame.turn_startcapability selects the existing reader variant.InputReaderacknowledges only Claude's replayed UUID, not a pipe flush. Monitor retains its challenge and bounded window.OpenWire: unrelated SDK records remain visible but cannot acknowledge input. This fixed a realcommand_lifecyclefailure; that failed session and its uncertain input are preserved, never replayed.Actual Claude Code 2.1.303 / configured
claude-opus-5-5acceptance: fresh session9242e154-b222-456c-871e-dff3ec12ab25, no bootstrap prompt. Bus inputs3c31b1a33c89and37f94195537fproducedP205 answer: 999andP205 second answer: 51; both answers were read from Core's retained external history before shutdown. Reader lock observed held; frontend, Claude and daemon stopped. Raw evidence:/home/ts/.cache/agent-scratch/pi-toad-queue/p205-reader-family-3/. Later changes moved only compaction observer wiring, restored unchanged formatting and integrated main; this is not a fresh installed-main acceptance claim.Checks and remaining gap
d7dbb9555against main6a0f5e931: exit 0, no positive keys or introduced sites. Tools comparison retained at that head is also clean; the later restart interrupted its enclosing command, so no stronger exit receipt is claimed.pi-bridge-compactis replacing its baseline. No rewritten base has been taken, and this PR must not be merged as if that dependency were finished.Source ownership evidence
NRA parsed both source roots with no omissions before integration:
/home/ts/.cache/agent-scratch/pi-toad-queue/p205-repair-source-before.txt; complete earlier owner census:p205-source-owners-complete.txt. Payload identity belongs to the input frame; registration retains its capability; the reader state owns readiness and delivery; the session daemon alone owns endpoint/history lifetime. External Claude dispatch is dynamic and is established by the real run, not AST evidence.Assumed: managed input remains opt-in; native provider/auth/model configuration is retained; verification uses fresh private sessions with tools/MCP disabled, not original user sessions. No installation was changed. P233 retains shared-lifecycle ownership; I retain managed-input integration after that replacement is published.
Agent-Comms-Order: aa96f89f9e55