Skip to content

Align MCP port configuration to BrowserOS env-driven model (without changing default 3100)#19

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/update-browseros-port-configuration
Draft

Align MCP port configuration to BrowserOS env-driven model (without changing default 3100)#19
Copilot wants to merge 2 commits intomainfrom
copilot/update-browseros-port-configuration

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 30, 2026

BrowserOS does not hardcode an MCP port and relies on environment configuration; this repo still had several fixed 3100 references that could drift from runtime settings. This PR aligns configuration surfaces and docs to the same env-driven port model while retaining 3100 as the intentional fallback/default.

  • Runtime port model clarity

    • Clarified server/mcp-server.js behavior as environment-first port resolution with final fallback.
    • Kept existing precedence and default intact (MCP_SERVER_PORTMCP_PORTBROWSEROS_GUIDES_PORT3100).
  • Container/config alignment

    • Updated docker-compose.yml to use BROWSEROS_GUIDES_PORT interpolation for:
      • MCP server published port mapping
      • inter-service MCP URL
      • MCP server environment assignment
    • This removes hardcoded compose wiring and keeps compose/runtime behavior consistent.
  • Configuration discoverability

    • Added MCP host/port entries to .env.template so env-based port control is explicit and documented.
  • Documentation synchronization

    • Updated README.md and QUICKSTART_MCP.md with explicit port override examples.
    • Added explicit port precedence note in Quickstart.
    • Added targeted comments in config.yml MCP sections indicating env-driven runtime resolution.
// server/mcp-server.js
const PORT =
  process.env.MCP_SERVER_PORT ||
  process.env.MCP_PORT ||
  process.env.BROWSEROS_GUIDES_PORT ||
  3100;

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/search/code
    • Triggering command: /usr/bin/python python scripts/check_browseros_ports.py (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Update BrowserOS port configuration settings Align MCP port configuration to BrowserOS env-driven model (without changing default 3100) Mar 30, 2026
Copilot AI requested a review from AccidentalJedi March 30, 2026 14:48
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.

⚠️ BrowserOS Port Configuration Mismatch Detected

2 participants