docs(bedrock): clarify Mantle model-family API roots - #885
Conversation
1566b0e to
4b7a6c0
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1566b0e22b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Addressed the P2 in @codex review |
|
Note You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
@codex review Please review the current head |
|
Note You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
fscfede-beep
left a comment
There was a problem hiding this comment.
RUMBO audit note: current PR head 41ff61a1f4ec9eeb1c67fb87e1dc14af85d4593a is still open but currently reports mergeable=false. I am not pushing or requesting another automated review in this pass because the previous current-head review attempts were blocked by Codex review limits, and repeated pings would add noise without new evidence. Next useful contributor action is to reconcile the branch against current upstream main and revalidate the documentation-only diff before any further review request.
Summary
Clarifies the route selection described in #812 without changing the existing Mantle default.
AWS currently documents different
bedrock-mantleAPI roots for different model families:/openai/v1./v1.This documents that
EndpointMantleintentionally preserves/openai/v1for backwards compatibility, does not infer the route from the model name, and shows the existingConfig.BaseURLescape hatch for deployments whose AWS model card requires/v1.No SDK behavior, authentication, signing, generated code, or exported API is changed.
Validation
git diff --checkLocal Go tests were not run because Go is not installed on this host; this is a documentation-only change and normal repository CI remains authoritative.
Closes #812.