Skip to content

fix: ServerHandler defaults answer prompts/list and completion/complete when neither capability is declared - #1446

Open
xixi-box wants to merge 1 commit into
modelcontextprotocol:mainfrom
xixi-box:fix/issue-1445-f852ec0-6401d18d
Open

xixi-box wants to merge 1 commit into
modelcontextprotocol:mainfrom
xixi-box:fix/issue-1445-f852ec0-6401d18d

Conversation

@xixi-box

Copy link
Copy Markdown
Contributor

Fixes #1445

Verified by running the failing test at f852ec0e9281 (fails) and again with this patch applied (passes).

AI assistance: OpenCode generated the fix and tests. oss-scout verified the regression and opened this PR automatically.

Signed-off-by: wangshun <96341168+xixi-box@users.noreply.github.com>
@xixi-box
xixi-box requested a review from a team as a code owner November 11, 2026 22:05
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-11T22:08:01.900245Z e28c827 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions github-actions Bot added T-core Core library changes T-handler Handler implementation changes labels Nov 11, 2026
@github-actions
github-actions Bot requested a review from jokemanfire November 11, 2026 22:05
@github-actions

Copy link
Copy Markdown
Contributor

@jokemanfire is on triage rotation this week and is the reviewer for this one.

Labels come from the PR labeler. Please review the change.

Automated triage. Labels are a first pass and the assigned maintainer has the final say.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-core Core library changes T-handler Handler implementation changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ServerHandler defaults answer prompts/list and completion/complete when neither capability is declared

1 participant