moonming commented on PR #13048: URL: https://github.com/apache/apisix/pull/13048#issuecomment-4064896848
**Architectural Note (self-review)**: This PR (#13048) and #13050 (anthropic-native driver by @iakuf) address the same need — Anthropic/Claude support — but with different approaches: - **This PR**: Protocol translation (Claude ↔ OpenAI format conversion) — more general, single code path - **#13050**: Native Anthropic driver — more efficient, preserves Anthropic-specific features **My current thinking**: Protocol translation as the default approach, with native drivers as optional optimizations for providers that have significantly different capabilities (like Anthropic's tool use format). Need to finalize this direction before either PR can move forward. Will update both PRs once the decision is made. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
