Implementing more Anthropic messages API schema #1837
Closed
changminbark wants to merge 11 commits intoenvoyproxy:mainfrom
Closed
Implementing more Anthropic messages API schema #1837changminbark wants to merge 11 commits intoenvoyproxy:mainfrom
changminbark wants to merge 11 commits intoenvoyproxy:mainfrom
Conversation
dosubot
bot
added
the
size:XL
Contributor
Author
|
Hello @nacx, this is the PR that introduces the rest of the Anthropic messages API schema. Btw, is there any way for me to change the PR title? It is not passing the PR style check for titles. |
Signed-off-by: Chang Min <changminbark@gmail.com>
Signed-off-by: Chang Min <changminbark@gmail.com>
Signed-off-by: Chang Min <changminbark@gmail.com>
Signed-off-by: Chang Min <changminbark@gmail.com>
Signed-off-by: Chang Min <changminbark@gmail.com>
…1842) **Description** Claude Opus 4.6 has new value `adaptive` for thinking [1] type 1. https://platform.claude.com/docs/en/about-claude/models/whats-new-claude-4-6#adaptive-thinking-mode Signed-off-by: Xiaolin Lin <xlin158@bloomberg.net>
…nvoyproxy#1648) **Description** Add InvokeModel API support for claude models in aws bedrock. The motivation is to provide consistent services cross providers, envoyproxy#1644 for more details about the motivation. Other Changes: I put common codes related to anthropic into `anthropic_helper.go`, so that both aws and gcp can share these codes. --------- Signed-off-by: yxia216 <yxia216@bloomberg.net> Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com> Signed-off-by: Aaron Choo <achoo30@bloomberg.net> Signed-off-by: Dan Sun <dsun20@bloomberg.net> Co-authored-by: Takeshi Yoneda <t.y.mathetake@gmail.com> Co-authored-by: Aaron Choo <achoo30@bloomberg.net> Co-authored-by: Dan Sun <dsun20@bloomberg.net>
**Description** This adds support for API Key upstream auth as a HTTP query parameter for MCP servers. There are several public MCP server that employ this pattern, thus this unlocks several patterns. To adds support for that, this remove the use of CredentialsInjector for inserting API Keys as it doesn't support the query parameter as a location. **Related Issues/PRs (if applicable)** Closes envoyproxy#1731 --------- Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com> Co-authored-by: Ignasi Barrera <ignasi@tetrate.io>
Signed-off-by: Chang Min <changminbark@gmail.com>
changminbark
force-pushed
the
anthropic-messages-schema-tracing
branch
from
f3ea05e to
a669424
Compare
dosubot
bot
added
size:XXL
Contributor
Author
|
@nutanix-Hrushikesh @nacx I created a new PR because of the git mess I created (while trying to signoff on previous commits) and not being able to change the PR title. The new PR is here: #1843 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds more struct definitions and their corresponding JSONMarshal and JSONUnmarshal methods according to the Anthropic Messages API.
It does not introduce any tracing as that should be opened in a new issue.
Related Issues/PRs (if applicable)
Fixes #1799
Special notes for reviewers (if applicable)
Claude code was used to write most of the code here, but I manually checked all of code that it generated. One thing that I noticed is that some of the previous API schema for Anthropic implemented in this repository were moved to beta status by Anthropic. This is something that the maintainers should discuss.