Problem
Raised by Copilot on the v2.10.0 milestone merge (#2637). docs/environment-variables.md says it lists every environment variable that changes how the Inspector behaves, but mcpdo, which ships for the first time in 2.10.0, is missing from it:
- Three variables mcpdo reads are documented only in
specification/v2_cli_v2.md (and skills/mcpdo/SKILL.md):
MCP_INSPECTOR_DAEMON_DIR
MCP_INSPECTOR_DAEMON_TOKEN (both set by eval "$(mcpdo private)")
MCP_ALLOW_DEFAULT_CONNECTION
- The shared variables mcpdo also reads (
MCP_STORAGE_DIR, MCP_CATALOG_PATH, MCP_CLIENT_CONFIG_PATH, MCP_OAUTH_CALLBACK_URL, the OAuth state and secret-store variables reached through core/) do not list mcpdo in their Read by column, and the column legend names only web, CLI, TUI and launcher.
Expected
The page names mcpdo in its legend and in each Read-by cell it applies to, and documents the three daemon variables with their defaults and effects, taken from the source (clients/mcpdo/src/daemon/paths.ts, daemon/auth.ts, connection/dispatch.ts).
Priority
Medium (rubric total 7): Severity 2 (doc gap, workaround is reading the spec), Urgency 4 (ships with the first mcpdo release), +1 milestoned.
Problem
Raised by Copilot on the v2.10.0 milestone merge (#2637).
docs/environment-variables.mdsays it lists every environment variable that changes how the Inspector behaves, but mcpdo, which ships for the first time in 2.10.0, is missing from it:specification/v2_cli_v2.md(andskills/mcpdo/SKILL.md):MCP_INSPECTOR_DAEMON_DIRMCP_INSPECTOR_DAEMON_TOKEN(both set byeval "$(mcpdo private)")MCP_ALLOW_DEFAULT_CONNECTIONMCP_STORAGE_DIR,MCP_CATALOG_PATH,MCP_CLIENT_CONFIG_PATH,MCP_OAUTH_CALLBACK_URL, the OAuth state and secret-store variables reached throughcore/) do not list mcpdo in their Read by column, and the column legend names only web, CLI, TUI and launcher.Expected
The page names mcpdo in its legend and in each Read-by cell it applies to, and documents the three daemon variables with their defaults and effects, taken from the source (
clients/mcpdo/src/daemon/paths.ts,daemon/auth.ts,connection/dispatch.ts).Priority
Medium (rubric total 7): Severity 2 (doc gap, workaround is reading the spec), Urgency 4 (ships with the first mcpdo release), +1 milestoned.