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

Ai assistclaude #656

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Ai assistclaude #656

wants to merge 2 commits into from

Conversation

CJary
Copy link
Collaborator

@CJary CJary commented Apr 12, 2024

Review the API request on claude3: https://docs.anthropic.com/claude/reference/complete_post
Adjusted the startAIAssist.ts with the new body request that claude3 wants. Still missing the API key for Claude so couldn't test it. But this is what I have so far.

Copy link
Contributor

@curran curran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent start! A few key changes to the overall approach are needed:

  • Based on the way VZCode is set up, the streaming AI response is added server-side, so the client side of this should ideally remain untouched
  • We want to use the streaming variant of the API, so we don't need to wait for the entire response
  • I think it should be possible to get an API key on a free Anthropic account, but I will investigate a little more and, if needed, I can provide an API key for you to use while testing.

Thank you so much for this work!

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