Skip to content

add Claude NativeTool support#397

Open
KristerV wants to merge 1 commit into
brainlid:mainfrom
KristerV:main
Open

add Claude NativeTool support#397
KristerV wants to merge 1 commit into
brainlid:mainfrom
KristerV:main

Conversation

@KristerV
Copy link
Copy Markdown
Contributor

i was trying to use the Claude web search tool, but NativeTool crashed on me. So I made a few changes.

This solution works, but the type/name situation is a tad confusing. Claude wants both keys to be present and one can be derived from the other. Docs here: https://anthropic.mintlify.app/en/docs/agents-and-tools/tool-use/web-search-tool

Anyway ready to be merged imo, but open to suggestions also. I used this in prod, but now am actually switching to Gemini, because it reads handwriting the best :D

@xxdavid
Copy link
Copy Markdown
Contributor

xxdavid commented Dec 5, 2025

Hi, thanks for the PR. Works great!

I think it would also be good to handle the web fetch results. Like this: xxdavid@814c493. What do you think?

@KristerV
Copy link
Copy Markdown
Contributor Author

personally not using Claude atm. but yeah, good idea.

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.

2 participants