Skip to content

docs(claude): distinguish access paths and feature limits - #4586

Merged
aheritier merged 2 commits into
mainfrom
docs/4581-claude-access-paths
Oct 15, 2026
Merged

aheritier merged 2 commits into
mainfrom
docs/4581-claude-access-paths

Conversation

@aheritier

Copy link
Copy Markdown
Collaborator

🤖 Automated implementer agent — this comment was posted by the implementer bot from Docker Agentic Platform, not by a human developer

Summary

Closes #4581

Validation

  • Markdownlint:127files,0errors; canonical115pages; llms.txt114entries/7sections; offline lychee0errors; Hugo120pages.
  • Rendered checks:69 internal content links/fragment targets across all8changedpages, canonical URLs correct.
  • Accessibility: pa11y-ci@3.1.0, Tier1+all8changedpages, 28/28URLs, both themes,0errors.
  • Offline schema validation:56existing YAMLexamples/fragments on changedpages. Changed Vertex config resolves as version16 with 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.
  • Fresh independent review: approve, no findings; signed and DCO signed-off commit.

Link-check caveat: full task docs-check-links did 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 unchanged guides/headless → sandbox/#how-it-works anchor; it passes only when that baseline page and external URLs are excluded. No unrelated link repairs included.

Upstream docs build and ci / gate await 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.

Signed-off-by: Arnaud Héritier <aheritier@users.noreply.github.com>
@aheritier
aheritier marked this pull request as ready for review October 13, 2026 15:42
@aheritier
aheritier requested a review from a team as a code owner October 13, 2026 15:42
Copilot AI balanced review requested due to automatic review settings October 13, 2026 15:42

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.

🟡 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.

Comment thread docs/providers/anthropic/index.md Outdated
Signed-off-by: Arnaud Héritier <aheritier@users.noreply.github.com>
@aheritier aheritier self-assigned this Oct 13, 2026
@aheritier aheritier added area/docs Documentation changes kind/docs Documentation-only changes labels Oct 13, 2026
@aheritier
aheritier enabled auto-merge October 13, 2026 16:07

@aheritier aheritier left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

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`;

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

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.

@aheritier
aheritier added this pull request to the merge queue Oct 15, 2026
Merged via the queue into main with commit 05ce717 Oct 15, 2026
22 checks passed
@aheritier
aheritier deleted the docs/4581-claude-access-paths branch October 15, 2026 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Documentation changes kind/docs Documentation-only changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs(Claude): document existing backends, protocols and feature limits [P1; independent]

3 participants