-
Notifications
You must be signed in to change notification settings - Fork 403
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
feat: Added support for Anthropic Claude 3 messages API #2278
Conversation
1048d44
to
1e788d3
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2278 +/- ##
==========================================
- Coverage 96.32% 96.27% -0.06%
==========================================
Files 284 284
Lines 45178 45230 +52
==========================================
+ Hits 43518 43543 +25
- Misses 1660 1687 +27
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
result = this.#body?.messages?.reduce((acc, curr) => { | ||
acc += curr?.content ?? '' | ||
return acc | ||
}, '') |
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.
We may need to come back and adjust this. It's not clear to me right now how we should handle multiple messages being sent to the LLM.
This PR resolves #2243.