Skip to content

docs: warn about extended-agent OAuth - #25810

Open
dvdksn wants to merge 3 commits into
docker:mainfrom
dvdksn:fix/issue-434-credential-binding
Open

docs: warn about extended-agent OAuth#25810
dvdksn wants to merge 3 commits into
docker:mainfrom
dvdksn:fix/issue-434-credential-binding

Conversation

@dvdksn

@dvdksn dvdksn commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

Explain that proxy-managed OAuth does not work for third-party kits that extend built-in agents, and document the working host-stored API-key flow and generated credential-binding approval.

Closes docker/sbx-releases#434

@netlify /ai/sandboxes/customize/kit-examples/

Generated by Codex

The Claude extension example implied that proxy-managed OAuth works for a third-party child kit. Document the working API-key setup, explain that sbx generates the binding during first-run approval, and warn that the prompt's OAuth option does not activate interception.

Co-Authored-By: Claude <noreply@anthropic.com>
@netlify

netlify Bot commented Aug 12, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit e39fd2c
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a7c486189a6f900083c4af5
😎 Deploy Preview https://deploy-preview-25810--docsdocker.netlify.app/ai/sandboxes/customize/kit-examples/
📱 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.

@dvdksn dvdksn added the status/review Pull requests that are ready for review label Aug 12, 2026
@dvdksn
dvdksn requested a review from a team August 12, 2026 10:09

@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 correctly documents that proxy-managed OAuth is not supported for third-party kits that extend built-in agents, and guides users to the API-key setup flow as the working alternative. The added content in both kit-examples.md and credentials.md is accurate, well-scoped, and consistent with the existing document style. No style violations, broken links, or structural issues were found in the changed lines.

@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: 🟡 NEEDS ATTENTION

Two medium findings in the new content. No high-severity or style issues detected.

Comment thread content/manuals/ai/sandboxes/customize/kit-examples.md Outdated
Comment thread content/manuals/ai/sandboxes/security/credentials.md Outdated
dvdksn and others added 2 commits August 12, 2026 10:16
Tell users to review credential mechanisms individually and decline unsupported OAuth. Clarify the first-run behavior for third-party sandbox agents without removing OAuth binding coverage from the general documentation.

Co-Authored-By: Claude <noreply@anthropic.com>
Keep the credential approval flow unchanged and state only that OAuth is ineffective for the extended agent. Restore the general first-run explanation for both API-key and OAuth bindings.

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ai status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extending built-in claude kit as documented breaks credentials

2 participants