Skip to content

Commit f3f6fb1

Browse files
committed
docs: update stale enableAllProjectMcpServers notes for new input
Addresses claude-review finding on PR #1250 — base-action/README.md:255 and docs/configuration.md:332 still said the setting is 'always set to true'. :house: Remote-Dev: homespace
1 parent c352b33 commit f3f6fb1

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

base-action/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ The settings file supports all Claude Code settings options including:
252252
- `includeCoAuthoredBy`: Include co-authored-by in git commits
253253
- And more...
254254

255-
**Note**: The `enableAllProjectMcpServers` setting is always set to `true` by this action to ensure MCP servers work correctly.
255+
**Note**: The `enableAllProjectMcpServers` setting is controlled by the `enable_all_project_mcp_servers` input (default `false`) and overrides any value provided via `settings`. Set the input to `true` only when the checkout's `.mcp.json` is trusted.
256256

257257
## Using MCP Config
258258

docs/configuration.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@ For a complete list of available settings and their descriptions, see the [Claud
329329

330330
**Notes**:
331331

332-
- The `enableAllProjectMcpServers` setting is always set to `true` by this action to ensure MCP servers work correctly.
332+
- The `enableAllProjectMcpServers` setting is controlled by the `enable_all_project_mcp_servers` input (default `true` for this action) and overrides any value provided via `settings`.
333333
- The `claude_args` input provides direct access to Claude Code CLI arguments and takes precedence over settings.
334334
- We recommend using `claude_args` for simple configurations and `settings` for complex configurations with hooks and environment variables.
335335

0 commit comments

Comments
 (0)