Skip to content

docs(cli): update /mcp list to show connection/auth status#1230

Merged
factory-ain3sh merged 1 commit into
mainfrom
ain3sh/cli-933-mcp-list-status
Jun 13, 2026
Merged

docs(cli): update /mcp list to show connection/auth status#1230
factory-ain3sh merged 1 commit into
mainfrom
ain3sh/cli-933-mcp-list-status

Conversation

@factory-ain3sh

Copy link
Copy Markdown
Contributor

Description

The CLI walkthrough docs described /mcp list as showing each MCP's "enabled/disabled status". That behavior changed in factory-mono#14725: droid mcp list now connects to each configured server and reports its live connection/authentication status (connected, connecting, needs authentication, failed) rather than a static enabled/disabled flag. This PR updates the EN and JP video-walkthrough docs to match, and ties the needs authentication state to the existing OAuth sign-in flow in the /mcp manager.

Related Issue

Closes CLI-933

Reviewer Guide

Read order: docs/cli/getting-started/video-walkthrough.mdx (English source of truth) > docs/jp/cli/getting-started/video-walkthrough.mdx (JP mirror).
Review depth: Skim — single-line copy change in each file.
Note: Status names are kept verbatim in the JP doc since they render in English in the CLI UI; terminology ("connection status" / "needs authentication") matches docs/cli/configuration/mcp.mdx.

Risk & Impact

Low risk — documentation-only copy change, no code or config touched.

Verification

Behavior verified. Confirmed the new wording matches the states surfaced by the source PR (connected, connecting, needs authentication, failed) and aligns with the OAuth flow already documented on the MCP configuration page.
Regression coverage. N/A — no executable behavior changed.
Not tested. No docs build run locally; this is a single-sentence MDX edit with balanced inline formatting, mirrored across both locales.

factory-mono#14725 changed `droid mcp list` to connect to each server and report live connection/authentication status (connected, connecting, needs authentication, failed) instead of a static enabled/disabled state. Update the EN and JP video walkthrough docs to match.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@mintlify

mintlify Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
factory 🟢 Ready View Preview Jun 13, 2026, 6:33 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@factory-droid

factory-droid Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Droid finished @factory-ain3sh's task —— View job


Security Review

LGTM. The docs now match the current /mcp list behavior (live connection and authentication status), and clarify that OAuth servers show needs authentication until you complete the sign-in flow in the interactive /mcp manager.

@factory-ain3sh factory-ain3sh merged commit f53eb02 into main Jun 13, 2026
5 checks passed
@factory-ain3sh factory-ain3sh deleted the ain3sh/cli-933-mcp-list-status branch June 13, 2026 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants