Skip to content

docs: sync built-in credential services with sbx secret set#25473

Open
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:worktree-credentials-builtin-services
Open

docs: sync built-in credential services with sbx secret set#25473
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:worktree-credentials-builtin-services

Conversation

@dvdksn

@dvdksn dvdksn commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

What

Updates the built-in services table on the Credentials page so it matches what sbx secret set recognizes and the domains the proxy actually injects credentials for.

  • Add cursor, droid, and openrouter services
  • Fix the nebius API domain (api.studio.nebius.aiapi.studio.nebius.com)
  • Expand the API domains column to the full set each service authenticates for, instead of a single representative host (for example, anthropic also covers console.anthropic.com, claude.ai, and mcp-proxy.anthropic.com)
  • Remove the aws row — that service can't authenticate a Bedrock request and is being removed at the source

Update the built-in services table to match the services `sbx secret
set` recognizes and the domains the proxy injects for.

- Add cursor, droid, and openrouter services
- Fix nebius domain (api.studio.nebius.ai -> api.studio.nebius.com)
- List the full set of API domains each service authenticates for,
  taken from the agent network.serviceDomains blocks and the proxy
  service detector defaults in docker/sandboxes

bedrock is intentionally omitted; it is contributed only by the
flag-gated claude-bedrock agent and is not yet GA.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c9688da
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a44cb6b7ae561000834517e
😎 Deploy Preview https://deploy-preview-25473--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@docker-agent docker-agent 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.

Assessment: 🟢 APPROVE

The diff updates the built-in credential services table in the credentials page to add cursor, droid, and openrouter services, fixes the Nebius domain from .ai to .com, and expands the API domains column for anthropic, aws, github, google, and openai. The PR description states all values were verified against the docker/sandboxes source repo.

No style violations, markdown formatting issues, or content accuracy problems were found in the changed lines. The Nebius domain correction (api.studio.nebius.aiapi.studio.nebius.com) is explicitly noted in the PR description as an intentional fix verified against source.

@dvdksn dvdksn requested a review from a team July 1, 2026 07:50

@docker-agent docker-agent 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.

Assessment: 🟢 APPROVE

The PR updates the built-in credential services table with new services (cursor, droid, openrouter), expanded API domain lists, and a corrected Nebius domain. No style violations, markdown issues, vendored-file edits, or content accuracy problems were found. The changes are factual, concise, and match the existing document's tone and scope.

The opencode aws credential service can't authenticate a Bedrock
request (no SigV4 signing, no secret key) and is being removed at
the source. Drop the corresponding row so the table only lists
services that actually work.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dvdksn dvdksn force-pushed the worktree-credentials-builtin-services branch from 8199489 to c9688da Compare July 1, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants