-
Notifications
You must be signed in to change notification settings - Fork 51
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
anthropic provider #420
anthropic provider #420
Conversation
} | ||
|
||
// Make request | ||
let response = self.post(payload).await?; |
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.
similar to openai provider, can we check for context limit exceeded here? i think we'll want to raise a specific error for this which then gets handled by the agent / shown to the user.
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.
LGTM! Left a comment on checking for context length exceeded in the response
* v1.0: anthropic provider (#420)
* refs/heads/v1.0: Extract common renderings (#432) fix: theme default (#431) anthropic provider (#420) fix: lib - add libxcb1-dev (#430) feat: launch GUI from shell (#423) [feat]: Dark mode support (#345) Update README.md (#426) fix: slow GUI with stdio (#425) Fixed icon position (#422) fix: replace empty window (#417) Download script for goose cli pre-release (#413) # Conflicts: # ui/desktop/src/ChatWindow.tsx # ui/desktop/src/components/Splash.tsx # ui/desktop/src/main.ts # ui/desktop/src/preload.js
it works with the basics at least