Skip to content

Fix chat-sample tool calling for Claude models#1289

Open
fhwvtqdc2q-svg wants to merge 1 commit into
microsoft:mainfrom
fhwvtqdc2q-svg:fix/chat-sample-claude-tools
Open

Fix chat-sample tool calling for Claude models#1289
fhwvtqdc2q-svg wants to merge 1 commit into
microsoft:mainfrom
fhwvtqdc2q-svg:fix/chat-sample-claude-tools

Conversation

@fhwvtqdc2q-svg

Copy link
Copy Markdown
Contributor

Summary

  • Upgrade @vscode/chat-extension-utils to alpha.5 for improved tool-calling support across language models.
  • Add lmTools.ts to map registered tools to LanguageModelChatTool with a valid inputSchema (required by Claude).
  • Update @catTools and @tools participants to use normalized tools and resolve #findFiles references correctly.
  • Add extension tests, a npm run verify script, and vscode-test-cli configuration.

Fixes #1282

Test plan

  • npm run compile
  • npm run lint
  • npm run verify (7/7 checks)
  • Manual: F5 -> Run Extension -> Copilot Chat with Claude -> @catTools search the package.json file

Upgrade chat-extension-utils and normalize tool schemas so @cattools and @tools work with Claude, not only GPT.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

chat-sample does not work with Claude model

1 participant