Skip to content

fix(mcp): consume auth prompt signal - #3059

Open
tandede wants to merge 1 commit into
upstash:masterfrom
tandede:fix/mcp-consume-auth-prompt
Open

fix(mcp): consume auth prompt signal#3059
tandede wants to merge 1 commit into
upstash:masterfrom
tandede:fix/mcp-consume-auth-prompt

Conversation

@tandede

@tandede tandede commented Aug 22, 2026

Copy link
Copy Markdown

Summary

  • consume the backend sign-in prompt signal once elicitation starts
  • prevent later stdio tool calls from reopening the same dialog
  • add regression coverage and a patch changeset

Why

The backend sends X-Context7-Auth-Prompt: 1 at most once per MCP session. The server captured that signal in ctx.shouldPrompt but never cleared it, so clients advertising elicitation could reopen the same sign-in dialog after every subsequent tool call.

Test plan

  • pnpm lint:check
  • pnpm format:check
  • pnpm build
  • pnpm typecheck
  • MCP test suite — 80 passed

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