Skip to content

fix: keep caller recovery on the existing rebind surface - #294

Closed
James3014 wants to merge 1 commit into
mainfrom
fix/issue240-rebind-owner-recovery
Closed

James3014 wants to merge 1 commit into
mainfrom
fix/issue240-rebind-owner-recovery

Conversation

@James3014

Copy link
Copy Markdown
Owner

Issue #240 exposed a remaining host-coupling bug: the live DevSpace catalog can add the owner orphan-PROCESS recovery tool while an already-open ChatGPT conversation keeps its older callable projection. That made a critical continuation path depend on the host hot-adding a new top-level tool.

This keeps the existing core_mutation_session_rebind schema stable and lets that already-projected surface consume a strict owner-recovery evidence envelope only when the bound session is PROCESS / OUTCOME_UNKNOWN. The exact configured OAuth owner is still required, physical snapshot/scope/deletion evidence is verified by the existing Core recovery path, and only after the PROCESS pin is cleared does the normal atomic caller handoff run. Ordinary rebind behavior is unchanged and foreign owner clients remain fail-closed.

Verification on the exact patch: new positive/hostile recovery cases 3/3 PASS; caller-rebind suite 8/8 PASS; typecheck PASS; production build PASS; git diff --check PASS; full server.test.ts 87/88 with the sole failure the pre-existing Issue #194 G2 ECONNRESET transport test. Core Candidate a363e2970320625daf481ad5670f4a255ee6b9bb, tree 2bc511a88ff9dbb47bda58df65dd0597f062fea4, diff hash sha256:15feaf08d18493d3ada5dd476e4639765d2bbff65546a594c2608527eb1dfc63.

Copy link
Copy Markdown
Owner Author

Superseded by #296 after main advanced via #295 / #242. The same bounded #240 repair was rebound and reverified on exact latest main c03753296ef2127457e9a83274c028b77915e384; do not merge this older-base PR.

@James3014 James3014 closed this Sep 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant