Conversation
Co-Authored-By: mecatl <noreply@stacklok.com>
Co-Authored-By: mecatl <noreply@stacklok.com>
Co-Authored-By: mecatl <noreply@stacklok.com>
Co-Authored-By: mecatl <noreply@stacklok.com>
Co-Authored-By: OpenAI Codex <noreply@openai.com>
Co-Authored-By: mecatl <noreply@stacklok.com>
Co-Authored-By: mecatl <noreply@stacklok.com>
Co-authored-by: mecatl <noreply@mecatl.dev>
Co-Authored-By: mecatl <noreply@stacklok.com>
Co-Authored-By: OpenAI Codex <codex@openai.com>
Preserve incoming client-MCP authority hardening alongside immutable runtime publication, renumber the proposed reconciliation ADR and named proofs to 0346, and repair the landed SDK acceptance proof to match main's authorized callback behavior. Co-Authored-By: OpenAI Codex <codex@openai.com>
Co-Authored-By: OpenAI Codex <codex@openai.com>
Co-Authored-By: Mecatl <mecatl@users.noreply.github.com>
Co-Authored-By: Mecatl <mecatl@users.noreply.github.com>
Co-Authored-By: Mecatl <mecatl@users.noreply.github.com>
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.
Stage
Implementation — stacked on #1527
Contract and baseline
316f99d100298f7da57966de25c322baac0a86616c75a42d9cad502685a42f5418f2bc99b9ae6376without rewriting integrated history.plan/mcp-source-reconciliation; humans merge the plan first, then retarget/review this PR for main.Summary
Session.GrantToolAuthority, direct refresh gRPC/HTTP controls, cached revision/status, SDK support, and unified mecatui/mcp-refreshrouting./tools-connectas a broker-only compatibility alias.Acceptance coverage
All 12 amended ACs are implemented and traced:
The plan's
landedstatus in this diff is a proposed transition, effective only when the implementation merges.Verification
Passed on final candidate:
task generatetask linttask test(race and module/standalone gates)task api:checktask sdk:api:check,task sdk:test(276 tests),task sdk:buildtask docs,task site:buildtask ac-trace-strictgo run ./cmd/mecademoOptional Bun-only e2e was not rerun locally because Bun is unavailable; required SDK gates pass.
Final panel:
PANEL: ship_blockers=0 important=1 advisory=2 reviewer_failures=0Non-blocking suggestions retained as optional follow-ups: consolidate candidate pagination if the abstraction proves worthwhile; use typed sorting and non-cryptographic polling jitter. No technical contract deviations.
Two pre-existing strict-trace references were corrected to the actual current provider/SDK proof names without changing tested behavior. Generated protobuf/API output accounts for a substantial portion of the diff. The discarded broader implementation is preserved separately and is not part of this stack.
Tracking: #1511