Skip to content

docs: clarify ChatGPT Codex requirement#25649

Open
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:codex/fix-issue-25648-chatgpt-prerequisite
Open

docs: clarify ChatGPT Codex requirement#25649
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:codex/fix-issue-25648-chatgpt-prerequisite

Conversation

@dvdksn

@dvdksn dvdksn commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Clarify that ChatGPT's remote server requires the codex command in the sandbox and that the Codex sandbox template used by the Connect steps includes it. This preserves the app-specific requirement without implying that readers must create the sandbox before following the connection instructions.

@netlify /ai/sandboxes/integrations/chatgpt/

Preview: https://deploy-preview-25649--docsdocker.netlify.app/ai/sandboxes/integrations/chatgpt/

Closes #25648

Generated by Codex

The ChatGPT integration listed an existing Codex sandbox as a prerequisite even though the Connect section creates it. Remove the contradictory prerequisite so readers begin with the documented sandbox creation step.
@netlify

netlify Bot commented Jul 26, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c6044a6
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a65a357935a640009f844b2
😎 Deploy Preview https://deploy-preview-25649--docsdocker.netlify.app/ai/sandboxes/integrations/chatgpt/
📱 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 Jul 26, 2026
@dvdksn
dvdksn requested a review from a team July 26, 2026 05:57

@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 change correctly removes the circular prerequisite. The Connect section already creates the Codex sandbox as its first step, so requiring an existing sandbox before reaching that step was redundant and confusing. The removal is minimal, accurate, and well-scoped — no style issues, no markdown problems, no missing redirects, and all front matter fields remain intact.

@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 removes a circular prerequisite from the ChatGPT integration guide. The removed lines required users to have an existing Codex-template sandbox before starting — but the Connect section's very first step already creates that sandbox (with sbx create --name demo codex . and the note "if you don't already have one"). The removal is clean: no AI-isms, no broken markdown, no vendored-file edits, no missing redirects, no front-matter issues, and the resulting document guides users correctly.

ChatGPT requires the Codex CLI on the remote host, which made removing the prerequisite context incomplete. Explain that the Codex sandbox template used by the connection steps includes the required command.
@dvdksn dvdksn changed the title docs: remove circular ChatGPT prerequisite docs: clarify ChatGPT Codex requirement Jul 26, 2026
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.

[docs-scanner] Confusing prerequisite in ChatGPT sandbox integration

2 participants