-
-
Notifications
You must be signed in to change notification settings - Fork 26
feat: context aware tools #93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
this was my initial thought, feel free to suggest enhancement or if there is a better way |
jherr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small API change, otherwise I love it!
|
hey @jherr , thanks for this suggestion ! yes it's much better to make this prop scalable.. |
|
View your CI Pipeline Execution ↗ for commit c818617
☁️ Nx Cloud last updated this comment at |
|
I have 2 questions on my mind here:
|
@tanstack/ai
@tanstack/ai-anthropic
@tanstack/ai-client
@tanstack/ai-devtools-core
@tanstack/ai-gemini
@tanstack/ai-ollama
@tanstack/ai-openai
@tanstack/ai-react
@tanstack/ai-react-ui
@tanstack/ai-solid
@tanstack/ai-solid-ui
@tanstack/react-ai-devtools
@tanstack/solid-ai-devtools
commit: |
hey @AlemTuzlak .. .server() will have server context isomorphic context :'D |
|
@AlemTuzlak , do you think we better also pass current serialisable_only within the request ? |
I think that was bad initial idea, or maybe not and we can make use of it but only serialisable will be sent |
🎯 Changes
Supporting context in tool execution..
to make tools behaviour context aware, the only way is to reconstruct them everytime you want to invoke a new trigger..
this PR suggests supporting context.
✅ Checklist
pnpm run test:pr.pnpm run test:lib.🚀 Release Impact