Skip to content

docs(skills): document APISIX 3.17 Bedrock support - #59

Merged
kayx23 merged 3 commits into
mainfrom
agent/restore-ai-proxy-bedrock
Aug 12, 2026
Merged

docs(skills): document APISIX 3.17 Bedrock support#59
kayx23 merged 3 commits into
mainfrom
agent/restore-ai-proxy-bedrock

Conversation

@kayx23

@kayx23 kayx23 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update the a6-plugin-ai-proxy guidance for Amazon Bedrock and the current protocol surface.
  • Document the required auth.aws credentials, provider_conf.region, model configuration, and Converse streaming behavior.
  • Clarify the supported request protocols and Anthropic authentication.

Why

APISIX 3.17.0 includes the Bedrock provider, but the A6 skill still described only the earlier nine-provider surface. Regenerating the AI Agent Skills pages in api7/docs#2067 exposed that stale guidance.

Verified against the released Apache APISIX 3.17.0 ai-providers, ai-proxy, and Bedrock protocol implementation.

This changes skill guidance only. It does not change A6 or APISIX runtime behavior, the skill's version: "1.0.0" metadata, or repository releases.

Validation

  • make check
  • go test ./test/skills
  • git diff --check

Required by api7/docs#2067.

Summary by CodeRabbit

  • Documentation
    • Expanded AI provider documentation to include Anthropic, Gemini, Vertex AI, and Amazon Bedrock.
    • Added guidance for Responses API, Anthropic Messages, and Bedrock Converse request protocols.
    • Documented provider endpoints, configuration options, protocol detection, authentication examples, and Bedrock streaming requirements.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The AI proxy skill documentation now covers Amazon Bedrock, request protocols, endpoint behavior, streaming, and provider-specific authentication. It adds configuration details for Vertex AI and Amazon Bedrock.

Changes

AI proxy documentation

Layer / File(s) Summary
Provider and protocol coverage
skills/a6-plugin-ai-proxy/SKILL.md
The documentation adds Amazon Bedrock. It describes protocol detection, endpoint selection, forwarding or conversion, streaming, endpoint overrides, and provider configuration.
Provider authentication configuration
skills/a6-plugin-ai-proxy/SKILL.md
Anthropic uses x-api-key and requires anthropic-version for native Messages requests. Amazon Bedrock uses AWS credentials, an optional session token, an AWS region, and a model ID.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
E2e Test Quality Review ✅ Passed The PR changes only SKILL.md documentation; it adds no runtime flow or testable external-service behavior. E2E completeness, mocks, concurrency, and error handling are not applicable, while generic...
Security Check ✅ Passed Only skills/a6-plugin-ai-proxy/SKILL.md changed; credential examples use placeholders, and no logging, persistence, authorization, ownership, TLS, or secret-resolution code changed.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: documenting Amazon Bedrock support for APISIX 3.17.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/restore-ai-proxy-bedrock

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/a6-plugin-ai-proxy/SKILL.md`:
- Around line 35-36: Update the protocol support description in SKILL.md to
document that Anthropic Messages requests use a URI ending in /v1/messages and
Bedrock Converse requests use /converse. Clarify that Bedrock streaming still
uses /converse for detection, with stream: true selecting the outgoing
/converse-stream endpoint, and that omitting these suffixes may cause body-only
protocol detection such as OpenAI Chat.
- Around line 97-111: Add the required "anthropic-version": "2023-06-01" entry
to the Anthropic auth.header example, alongside x-api-key, so the documented
native Messages configuration includes both required headers.
- Line 68: Update the options.stream documentation in SKILL.md to explain that
Bedrock Converse with stream: true uses the /model/{modelId}/converse-stream
endpoint and returns unmodified AWS EventStream binary frames with Content-Type
application/vnd.amazon.eventstream rather than SSE; explicitly instruct clients
to parse EventStream responses.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f97133c2-b4f7-4718-9aca-3e87762b5afe

📥 Commits

Reviewing files that changed from the base of the PR and between 83da9bf and 11d4718.

📒 Files selected for processing (1)
  • skills/a6-plugin-ai-proxy/SKILL.md

Comment thread skills/a6-plugin-ai-proxy/SKILL.md
Comment thread skills/a6-plugin-ai-proxy/SKILL.md Outdated
Comment thread skills/a6-plugin-ai-proxy/SKILL.md

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the AI proxy skill to reflect APISIX 3.17 support for Amazon Bedrock and additional request protocols.

Changes:

  • Adds Bedrock provider, authentication, region, model, and streaming guidance.
  • Clarifies supported protocols, endpoint selection, and Anthropic authentication.
  • Expands provider configuration reference details.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@kayx23 kayx23 changed the title docs(skills): restore APISIX 3.17 Bedrock guidance docs(skills): document APISIX 3.17 Bedrock support Aug 11, 2026
@kayx23
kayx23 merged commit 3ce8877 into main Aug 12, 2026
7 checks passed
@kayx23
kayx23 deleted the agent/restore-ai-proxy-bedrock branch August 12, 2026 06:02
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