Skip to content

Fix Codex MCP setup command#1031

Closed
Zahidul Islam (zahidoverflow) wants to merge 1 commit into
browseros-ai:mainfrom
zahidoverflow:fix-codex-mcp-command
Closed

Fix Codex MCP setup command#1031
Zahidul Islam (zahidoverflow) wants to merge 1 commit into
browseros-ai:mainfrom
zahidoverflow:fix-codex-mcp-command

Conversation

@zahidoverflow

Copy link
Copy Markdown

Summary

  • update the in-app MCP Quick Setup Codex command to use codex mcp add browseros --url <url>
  • update the matching Codex CLI docs example to use --url instead of the stale transport syntax

Verification

  • git diff --check
  • Could not run bunx biome check because bunx is not installed in this environment

Copilot AI review requested due to automatic review settings May 22, 2026 17:35
@greptile-apps

greptile-apps Bot commented May 22, 2026

Copy link
Copy Markdown
Contributor

PR author is not in the allowed authors list.

@github-actions

github-actions Bot commented May 22, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA. Thank you!
Posted by the CLA Assistant Lite bot.

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the OpenAI Codex CLI setup instructions/snippets to use the --url flag when adding the BrowserOS MCP server.

Changes:

  • Updated the in-app “Quick Setup” Codex snippet to use --url.
  • Updated the documentation Codex tab to use --url and refreshed the example.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/browseros-agent/apps/agent/entrypoints/app/mcp-settings/QuickSetupSection.tsx Adjusts the generated Codex CLI snippet to use the --url flag.
docs/features/use-with-claude-code.mdx Aligns Codex CLI docs with the new --url-based invocation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

name: 'Codex',
type: 'command',
getSnippet: (url) => `codex mcp add browseros ${url}`,
getSnippet: (url) => `codex mcp add browseros --url ${url}`,
# Example: codex mcp add browseros http://127.0.0.1:9239/mcp --transport http
codex mcp add browseros --url <mcp_url>
# Example: codex mcp add browseros --url http://127.0.0.1:9239/mcp
```
@zahidoverflow

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@zahidoverflow

Copy link
Copy Markdown
Author

recheck

Copilot AI 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.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

name: 'Codex',
type: 'command',
getSnippet: (url) => `codex mcp add browseros ${url}`,
getSnippet: (url) => `codex mcp add browseros --url ${url}`,
Comment on lines +88 to +89
codex mcp add browseros --url <mcp_url>
# Example: codex mcp add browseros --url http://127.0.0.1:9239/mcp
@shadowfax92

Copy link
Copy Markdown
Contributor

Thanks so much for contributing to BrowserOS! 🙏

We're doing some housekeeping on our PR queue and closing out PRs that have been open for more than 7 days. This isn't a reflection on your work — we really appreciate the time and effort you put in.

If this is still relevant, please feel free to reopen it or open a fresh PR and we'll take another look. Thanks again for being part of the project! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants