Skip to content

docs(cli): document MCP load blocking setting#1328

Merged
factory-ain3sh merged 1 commit into
mainfrom
ainesh/cli-1450-document-block-mcp
Jul 24, 2026
Merged

docs(cli): document MCP load blocking setting#1328
factory-ain3sh merged 1 commit into
mainfrom
ainesh/cli-1450-document-block-mcp

Conversation

@factory-ain3sh

@factory-ain3sh factory-ain3sh commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

What

The public settings reference omits blockOnMcpLoad, so users cannot discover the option that controls whether agent turns wait for MCP readiness.

Why

The CLI now exposes this settings.json control, but the English and Japanese references were not updated with the feature.

How

This PR adds the boolean setting, its default (false), and its behavior to both localized settings tables.

Related Issue

Closes CLI-1450

Reviewer Guide

Diff shape: ~90% documentation tables, ~10% translation; no runtime code or generated files.
Review depth: Skim — verify the setting name, default, and behavior match the CLI contract in both locales.
Read order:

  1. docs/cli/configuration/settings.mdx — English setting reference.
  2. docs/jp/cli/configuration/settings.mdx — Japanese translation of the same entry.

Risk & Impact

Low risk. The change is documentation-only and does not alter CLI behavior, configuration parsing, or generated artifacts.

Verification

Behavior verified. Confirmed both public settings tables now list blockOnMcpLoad as a boolean defaulting to false, verified @ 4ed6b54.
Regression coverage. No runtime regression test applies because this PR changes documentation only; the two localized references are the owning documentation surfaces.
Not tested. Mintlify preview was not run because the worktree has no installable project manifest or dependency tree; git diff --check and worktree structural verification pass.
Standard validators. No formatter, linter, typecheck, dead-code, AI-slop, React, or test configuration is present in this docs-only repository; whitespace validation passed.

Add blockOnMcpLoad to the English and Japanese public settings references so users can discover the agent-loop readiness control introduced by the CLI.

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

factory-droid Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

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


Security Review

LGTM, documentation-only update adding the missing blockOnMcpLoad setting entry (default false) to both the English and Japanese settings references.

@mintlify

mintlify Bot commented Jul 24, 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 Jul 24, 2026, 2:01 AM

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

@factory-ain3sh factory-ain3sh self-assigned this Jul 24, 2026
@factory-ain3sh factory-ain3sh added the documentation Improvements or additions to documentation label Jul 24, 2026
@mintlify

mintlify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
factory 🟡 Building Jul 24, 2026, 2:00 AM

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

@factory-ain3sh
factory-ain3sh merged commit a80aa7c into main Jul 24, 2026
5 checks passed
@factory-ain3sh
factory-ain3sh deleted the ainesh/cli-1450-document-block-mcp branch July 24, 2026 21:10
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