Skip to content

sbx: document codex auth prompt before sandbox launch#25315

Merged
dvdksn merged 1 commit into
docker:mainfrom
dvdksn:worktree-codex-auth-prompt-docs
Jun 9, 2026
Merged

sbx: document codex auth prompt before sandbox launch#25315
dvdksn merged 1 commit into
docker:mainfrom
dvdksn:worktree-codex-auth-prompt-docs

Conversation

@dvdksn

@dvdksn dvdksn commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

sbx now prompts you to authenticate to Codex on the host before launching the sandbox when no OpenAI credential is stored. Updated the Codex agent page and the credentials best practices to describe this flow, and reordered the Codex auth methods so OAuth (the host-side prompted flow) leads.

Generated by Claude Code

sbx now prompts you to authenticate to Codex on the host before launching
the sandbox when no OpenAI credential is stored. Update the Codex agent
page and the credentials best practices to describe this flow.

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

netlify Bot commented Jun 9, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 0271101
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a2807cb93eaa800071b62c5
😎 Deploy Preview https://deploy-preview-25315--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.

@dvdksn dvdksn added the status/review Pull requests that are ready for review label Jun 9, 2026
@dvdksn dvdksn added this to the sbx/v0.32.0 milestone Jun 9, 2026
@dvdksn dvdksn requested a review from a team June 9, 2026 12:32

@docker-agent docker-agent left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION


**OAuth**: If you prefer not to use an API key, start the OAuth flow on your
host with:
**OAuth**: Start the OAuth flow on your host with:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[MEDIUM] Bold lead-in **OAuth**: violates STYLE.md — bold only for UI elements

**OAuth**: uses bold as a section/method label, which STYLE.md explicitly disallows. Bold is reserved for UI elements (buttons, menus, field labels) only. The **Term**: description format is called out as "marketing-style list formatting" that should be avoided.

Consider restructuring without the bold lead-in, for example:

To authenticate using OAuth, run:

```console
$ sbx secret set -g openai --oauth

Or use a sub-heading if the section warrants one.

<!-- cagent-review -->

in your OS keychain. The OAuth flow runs on the host, not inside the sandbox,
so browser-based authentication works without any extra setup.

**API key**: Store your OpenAI API key using

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[MEDIUM] Bold lead-in **API key**: violates STYLE.md — bold only for UI elements

Same issue as the **OAuth**: lead-in above: **API key**: uses bold as a term label rather than for a UI element. Per STYLE.md, the **Term**: description pattern in lists is disallowed.

Consider plain prose, for example:

To store your OpenAI API key using [stored secrets](../security/credentials.md#stored-secrets), run:

@dvdksn dvdksn merged commit f530aa3 into docker:main Jun 9, 2026
19 checks passed
@dvdksn dvdksn deleted the worktree-codex-auth-prompt-docs branch June 9, 2026 13:58
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.

3 participants