Skip to content
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

svelte use assistant hook #1559

Closed
wants to merge 0 commits into from
Closed

svelte use assistant hook #1559

wants to merge 0 commits into from

Conversation

jaycoolslm
Copy link
Contributor

Implementation of useAssistant for Svelte based on the

https://github.com/vercel/ai/blob/main/packages/core/react/use-assistant.ts directory

and following the same coding standards as found in the other svelte files

@lgrammel
Copy link
Collaborator

The lockfile changes are unnecessary. Please revert that part.

@lgrammel
Copy link
Collaborator

Can you add an example under examples/sveltekit-openai similar to examples/next-openai/app/assistant and examples/next-openai/app/api/assistant so I can try it out?

@yaqubhall
Copy link

Yep, will try get that over tonight

@jaycoolslm
Copy link
Contributor Author

Have added assistant page and api endpoint - its more similar to the current the svelte implementations and not as thorough as the nextjs. Low in time atm lol (aren't we all)... I ran pnpm i again and it did change the lock file, but not to the same extent as previous time so hopefully it's okay. Looks like it just bumped a few packages etc, pushing as a separate commit anyway so easy to revert

@lgrammel
Copy link
Collaborator

@jaycoolslm can you see if you can resolve the conflicts? Usually a merge with the latest main + pnpm i fixes the lockfile conflicts

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.

None yet

3 participants