Skip to content

docs(skills): document API7 Gateway Bedrock support - #74

Open
kayx23 wants to merge 3 commits into
masterfrom
agent/restore-ai-proxy-bedrock
Open

docs(skills): document API7 Gateway Bedrock support#74
kayx23 wants to merge 3 commits into
masterfrom
agent/restore-ai-proxy-bedrock

Conversation

@kayx23

@kayx23 kayx23 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update the a7-plugin-ai-proxy guidance for Amazon Bedrock and the current protocol surface.
  • Document the required auth.aws credentials, provider_conf.region, model configuration, URI-based protocol detection, and Converse streaming behavior.
  • Clarify Anthropic authentication and add the missing Gemini and custom OpenAI-compatible authentication examples.

Why

API7 Gateway 3.10.4 includes the Bedrock provider and protocol implementation, but the A7 skill still describes the earlier nine-provider surface. The generated AI Agent Skills catalog would otherwise publish stale API7 Gateway guidance.

Verified against the released API7 Gateway 3.10.4 ai-providers, ai-proxy, Anthropic Messages, and Bedrock Converse implementation.

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

Validation

  • make check
  • git diff --check

Required by api7/docs#2067.

Summary by CodeRabbit

  • Documentation
    • Expanded AI proxy setup guidance for Anthropic, Gemini, and Amazon Bedrock.
    • Added details on protocol detection, provider endpoints, authentication, streaming, and configuration.
    • Included authentication and custom OpenAI-compatible API examples.
    • Documented Bedrock credentials, model configuration, and the Vertex AI service account requirement.
    • Updated supported-provider counts and endpoint descriptions.

@kayx23
kayx23 requested a balanced review from Copilot August 11, 2026 09:37
@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Changes

AI Proxy Provider Documentation

Layer / File(s) Summary
Provider protocols and gateway behavior
skills/a7-plugin-ai-proxy/SKILL.md
The documentation now covers protocol detection, supported request types, provider endpoints, Amazon Bedrock streaming, and expanded provider support.
Provider authentication and configuration
skills/a7-plugin-ai-proxy/SKILL.md
The documentation adds Anthropic and Gemini authentication examples, Vertex AI service-account configuration, Amazon Bedrock credentials, and custom OpenAI-compatible API settings.

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, with no runtime or API flow; E2E criteria are not applicable, and static checks show valid JSON, 10 providers, and no whitespace errors.
Security Check ✅ Passed Only SKILL.md changed; credential examples are placeholders, no production secret patterns exist, and no logging, database, endpoint authorization, ownership, TLS, resource-sharing, or resolver cod...
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: documenting Amazon Bedrock support in the API7 Gateway skill.
✨ 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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates AI proxy skill guidance to cover Amazon Bedrock and clarify provider-specific behavior.

Changes:

  • Documents Bedrock configuration, authentication, protocol detection, and streaming.
  • Clarifies Anthropic, Gemini, Vertex AI, and OpenAI-compatible authentication.
  • Updates supported protocols, providers, and endpoint behavior.

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

@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: 1

🤖 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/a7-plugin-ai-proxy/SKILL.md`:
- Around line 43-48: Update the Protocol Detection section in SKILL.md to
document that requests containing input without messages must use a URI ending
in /v1/responses to be detected as OpenAI Responses, and that custom routes
should use /v1/embeddings as the fallback when applicable.
🪄 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: ce90575b-42ca-4258-bb7c-fd09b2900b93

📥 Commits

Reviewing files that changed from the base of the PR and between 43c11b6 and 816799f.

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

Comment thread skills/a7-plugin-ai-proxy/SKILL.md

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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
Contributor

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 API7 Gateway Bedrock guidance docs(skills): document API7 Gateway Bedrock support Aug 11, 2026
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