Repository navigation
Conversation
Signed-off-by: Arnaud Héritier <aheritier@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
The custom Messages authentication table omits credential fallback behavior that can expose provider credentials to the configured host.
1 open finding
What changed in this PR
Documents Claude access paths, model availability, authentication, and provider-specific limitations.
Changes:
- Adds a source-dated Claude transport and model-support matrix.
- Clarifies Vertex, Bedrock, custom-host, and OpenCode behavior.
- Cross-links structured-output and provider guidance.
| File | Description |
|---|---|
docs/providers/overview/index.md |
Links to Claude access-path guidance. |
docs/providers/anthropic/index.md |
Adds the primary access-path and compatibility guide. |
docs/providers/google/index.md |
Clarifies native Vertex behavior and prerequisites. |
docs/providers/bedrock/index.md |
Documents ConverseStream and feature limitations. |
docs/providers/custom/index.md |
Distinguishes custom Messages and Azure paths. |
docs/providers/opencode-zen/index.md |
Separates OpenAI-compatible and Messages paths. |
docs/providers/opencode-go/index.md |
Clarifies alias protocol and feature guarantees. |
docs/configuration/structured-output/index.md |
Refines provider-specific structured-output support. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Signed-off-by: Arnaud Héritier <aheritier@users.noreply.github.com>
aheritier
left a comment
There was a problem hiding this comment.
Reviewed the eight-file documentation change for quality, performance, security, and relevance against #4581 and the implementation at this head. No blocking defects found. The transport/authentication distinctions and credential-disclosure warning align with the implementation; the evidence matrix distinguishes corroborated availability from limited-access announcements rather than claiming live-tested support. Future implementation work remains explicitly out of scope.
One optional readability improvement is noted inline. Relative links/heading targets and YAML examples were checked successfully. CI reports SUCCESS; documentation checks passed, while docs-only Go test/race/build jobs were skipped. Rendering/accessibility checks were verified through CI, not rerun locally; no live inference or entitlement tests were performed. This COMMENT is not an approval.
| mean this client's complete 5.x compatibility work has shipped. | ||
|
|
||
| [AWS compaction](https://docs.aws.amazon.com/bedrock/latest/userguide/claude-messages-compaction.html) | ||
| requires InvokeModel for both `compact-2026-01-12` and `compact-2026-09-04`; |
There was a problem hiding this comment.
Optional: Could we name the two mechanisms beside their beta IDs? AWS distinguishes threshold-triggered compaction from explicit on-demand compaction, but this paragraph identifies them only by dates. For example: ‘requires InvokeModel for both threshold-triggered compaction (compact-2026-01-12) and on-demand compaction (compact-2026-09-04); neither works through Converse.’ This makes the limitation understandable without opening the vendor reference and keeps the existing meaning.

Summary
ConverseStream, and OpenAI-compatible aggregators. Distinguish OpenCode aliases from custom Messages and Azure aliases from native Foundry.Closes #4581
Validation
pa11y-ci@3.1.0, Tier1+all8changedpages, 28/28URLs, both themes,0errors.debug config; no inference.task build,CGO_ENABLED=1 task test,CGO_ENABLED=1 task lint:passed (golangci-lint2.14.0 zeroissues). Compiler/linter/browser libraries provisioned after initial missing-tool/CGO setup failures.Link-check caveat: full
task docs-check-linksdid not pass: external AWS403s and pre-existing stable-docs404s/other forbidden endpoints. A newly detected tool-mode anchor was fixed; changed-page rendered links pass. Internal-only muffet also found an unchangedguides/headless→sandbox/#how-it-worksanchor; it passes only when that baseline page and external URLs are excluded. No unrelated link repairs included.Upstream docs build and
ci / gateawait the PR workflow. Docs-only Go CI jobs are expected skipped, not passed; local Go checks above are separate. No vendor inference, entitlement test or cloud/consent-setting change.