Skip to content

fix(basic-host): use tool display-name precedence in headers - #778

Open
Michael-WhiteCapData wants to merge 1 commit into
modelcontextprotocol:mainfrom
Michael-WhiteCapData:fix/tool-display-name-precedence
Open

Michael-WhiteCapData wants to merge 1 commit into
modelcontextprotocol:mainfrom
Michael-WhiteCapData:fix/tool-display-name-precedence

Conversation

@Michael-WhiteCapData

Copy link
Copy Markdown

Summary

  • render the reference host tool-call header with title, then annotations.title, then name
  • keep the precedence in a small testable helper
  • add regression coverage for all three precedence levels

Closes #767.

Verification

  • npm test — 441 passed, 2 platform-specific skipped
  • npm run --workspace examples/basic-host build
  • prettier --check on changed files
  • git diff --check

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.

Reference host renders tool.name in the app header; base spec display precedence is title → annotations.title → name

1 participant