Skip to content

feat(mcp): Phase 9 — optional overlay auth preflight - #377

Merged
erishforG merged 1 commit into
release/1.0from
feat/auto-mcp-phase9-validation
Jul 6, 2026
Merged

feat(mcp): Phase 9 — optional overlay auth preflight#377
erishforG merged 1 commit into
release/1.0from
feat/auto-mcp-phase9-validation

Conversation

@erishforG

Copy link
Copy Markdown
Owner

무엇

@erishforG MCP tools/call preflight now rejects explicitly requested GitHub-backed optional overlays when no delegated token is present.

Refs #294. docs/mcp/spec.md defines optional overlays such as PR/CI status as GitHub-backed behavior that should require delegated auth when requested. This keeps local-only calls working while making explicit overlay requests fail predictably.

변경

  • Pass tools/call arguments into MCP preflight.
  • Detect explicit include_pr / include_ci requests and return the existing AUTH_REQUIRED MCP error envelope without dispatching handlers.
  • Add unit coverage and a stdio smoke fixture for the optional overlay auth contract.

다음 Phase 힌트

  • Track delegated token scopes separately from token presence so INSUFFICIENT_SCOPE can be returned when a token lacks pull_request:read or checks:read.

리스크

Low. Change is isolated to src/mcp/ dispatcher preflight plus MCP fixtures; no CLI surface or worktree core logic changes.

롤백

Revert this commit to restore previous behavior where optional overlay arguments were not checked before handler dispatch.

Test plan

  • cargo build --quiet
  • cargo fmt --check
  • cargo clippy --all-targets -- -D warnings
  • cargo test --quiet

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@erishforG erishforG added the auto-improvement Daily auto-generated improvement PR/issue label Jul 4, 2026
@erishforG
erishforG merged commit 73ddea8 into release/1.0 Jul 6, 2026
12 checks passed
@erishforG
erishforG deleted the feat/auto-mcp-phase9-validation branch July 6, 2026 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-improvement Daily auto-generated improvement PR/issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant