Repository navigation
Disable orchestration context inheritance - #260
Merged
Merged
Conversation
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
Removes orchestration context inheritance from the active MCP behavior because the cloned/resumed transcript path is too unstable for child-agent work.
Children now start from clean provider conversations. The parent agent must include required context directly in the prompt. The old
inheritParentContextschema field is still tolerated for stale MCP tool caches, but the implementation forces it off.Refs #259 for bringing context inheritance back with a more stable design.
Changes
inheritParentContext: falsethrough the MCP/main/renderer creation path.Verification
npm run test:orchestration-mcp -- --runInBandnpm run build:appNote:
npx tsc --noEmit -p tsconfig.web.jsonis not usable directly in this checkout because the repo expects referenced.tsc-out/nodedeclaration outputs to exist first; it fails with TS6305 before checking this change.