Skip to content

refactor(auth): simplify MCP consent screen - #463

Draft
irvinebroque wants to merge 4 commits into
mainfrom
codex/remove-redundant-oauth-approval
Draft

irvinebroque wants to merge 4 commits into
mainfrom
codex/remove-redundant-oauth-approval

Conversation

@irvinebroque

@irvinebroque irvinebroque commented Aug 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • retain the per-client MCP consent interstitial required before forwarding to Cloudflare OAuth
  • reduce the visible page to the requesting client name, full registered redirect URI, complete third-party API scope set, and explicit Cancel/Continue controls
  • continue requesting every configured Cloudflare scope so the upstream Cloudflare consent page can handle optional permission selection
  • preserve signed remembered-client consent, CSRF validation, restrictive CSP/clickjacking protection, and OAuth state/session binding
  • return OAuth access_denied to the validated client redirect when the user cancels

Verification

  • pnpm --filter @repo/mcp-common test (157 tests)
  • pnpm --filter @repo/mcp-common check:types
  • pnpm --filter @repo/mcp-common check:lint
  • targeted Prettier check for changed files
  • git diff --check

Notes

  • repository-wide pnpm check:format remains blocked by the pre-existing untracked .opencode/package.json; the changed files pass Prettier.

@irvinebroque

Copy link
Copy Markdown
Collaborator Author

WIP — think you are right that spec requires some kind of interstitial

@irvinebroque irvinebroque changed the title refactor(auth): remove redundant approval page refactor(auth): simplify MCP consent screen Aug 22, 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