[codex] bind plugin routing to request identity#1055
Merged
TheGreenCedar merged 2 commits intoJul 13, 2026
Conversation
Owner
Author
|
The two independent P2 review findings are fixed at exact head What changed
Verification
The PR remains draft while hosted checks rerun against this exact head. |
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.
Outcome
CodeStory's plugin bootstrap now binds readiness to the repository named by the current request. Hook-written active-project files remain diagnostics only, filesystem aliases use native identity, and stale readiness from another project is rejected before packet/search surfaces are reported ready.
Closes #1035
Refs #1032
Refs #913
Refs #899
What changed
cache identitywith the lossless schema-3 project, workspace, and artifact-scope ids plus explicit legacy-alias disposition.How to review
plugins/codestory/scripts/codestory-mcp.cjs:resolveProjectRoot,sameFilesystemPath,probeProjectIdentity, andprojectIdentityMismatchdefine the routing boundary.crates/codestory-runtime/src/repository_identity.rsand the CLI renderer for the schema-3 exchange.Verification
node --check plugins/codestory/scripts/codestory-mcp.cjsnode --test plugins/codestory/tests/plugin-static.test.mjs— 72 passed, 1 Windows-only skipnode .github/scripts/check-doc-links.mjscargo fmt --all -- --checkcargo check --workspace --lockedcargo test --workspace --locked -qcargo clippy --workspace --locked --all-targets --all-features -- -D warningsThe branch was rebased onto
9c97808a; targeted identity tests and the workspace check were rerun after that rebase.Risk and follow-up
Bootstrap performs one additional read-only
cache identityprobe. It fails open to diagnostic MCP output if that exchange is invalid, and only ready broker evidence is required to carry a matching schema-3 identity. Legacy alias reuse remains unavailable unless provenance proves it safe.