-
-
Notifications
You must be signed in to change notification settings - Fork 60
Input output schema #37
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
Conversation
|
View your CI Pipeline Execution ↗ for commit 5f5c59c
☁️ Nx Cloud last updated this comment at |
@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/react-ai-devtools
@tanstack/solid-ai-devtools
commit: |
AlemTuzlak
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.
I really like this PR and it makes tools SOOO MUCHHH nicer but I think inputSchema should be optional, just to satisfy the requirement of passing it in we had to add zod to all the adapters which should not be the case
* add improvements to input/output schema * remove old files * ci: apply automated fixes * remove old files * remove old files * fix types for anthropic and gemini * ci: apply automated fixes * lock fix --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
|
Awesome work Jack, love it! |
Moving from parameters (old skool) to inputSchema/outputSchema (new hotness).