-
-
Notifications
You must be signed in to change notification settings - Fork 52
feat: Add Vue useChat() support
#84
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
| export function useChat<TTools extends ReadonlyArray<AnyClientTool> = any>( | ||
| options: UseChatOptions<TTools> = {} as UseChatOptions<TTools>, | ||
| ): UseChatReturn<TTools> { | ||
| const hookId = useId() // Available in Vue 3.5+ |
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.
If we want to support Vue <3.5, we can use a random ID generator instead
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 guess this should be fine unless someone explicitly asks. Upgrading 3.3 to 3.5 should not be a problem though?
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.
yep! no problems here
|
View your CI Pipeline Execution β for commit b6ce347
βοΈ 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/ai-solid-ui
@tanstack/ai-vue
@tanstack/react-ai-devtools
@tanstack/solid-ai-devtools
commit: |
|
@wobsoriano for the provenance issues I suggest copying the lock from main over to your branch and running pnpm install |
| export function useChat<TTools extends ReadonlyArray<AnyClientTool> = any>( | ||
| options: UseChatOptions<TTools> = {} as UseChatOptions<TTools>, | ||
| ): UseChatReturn<TTools> { | ||
| const hookId = useId() // Available in Vue 3.5+ |
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 guess this should be fine unless someone explicitly asks. Upgrading 3.3 to 3.5 should not be a problem though?
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.
update provinence, package version and the getter and we're good to go!! thank you
the provenance action is still failing after plenty of attempts checking out lockfile from main and reinstalling |
|
Maybe it indeed loses provencence when installing some of the deps from vue? @lachlancollins mind checking it out? |
Its due to: And it's transitive dep of |
π― Changes
This PR adds
useChat()for Vue! The UI components will come in a separate PR.β Checklist
pnpm run test:pr.π Release Impact