fix(mcp): clarify install help forwarded options (#2018)

* fix(mcp): clarify install help forwarded options

* docs(mcp): sync reference page label to 'Forwarded install options'

Align docs/reference/cli/mcp.md terminology with the CLI help text
changed in the parent commit. The CLI now says 'Forwarded install
options'; the reference page still said 'Common forwarded options'.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Ching Wei Kang <WilliamK112@users.noreply.github.com>
Co-authored-by: danielmeppiel <danielmeppiel@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: 3 people

docs/src/content/docs/reference/cli/mcp.md

@@ -89,7 +89,7 @@ apm mcp install fetch -- npx -y @modelcontextprotocol/server-fetch
 apm mcp install api --transport http --url https://example.com/mcp
 ```
 
-Common forwarded options (see `apm install --help` for the full
+Forwarded install options (see `apm install --help` for the full
 list):
 
 | Flag | Description |

src/apm_cli/commands/mcp.py

@@ -89,18 +89,16 @@ def mcp():
         "Add an MCP server to apm.yml. Alias for 'apm install --mcp'.\n\n"
         "Examples:\n\n"
         "  apm mcp install fetch -- npx -y @modelcontextprotocol/server-fetch\n\n"
-        "  apm mcp install api --transport http --url https://example.com/mcp"
-    ),
-    epilog=(
+        "  apm mcp install api --transport http --url https://example.com/mcp\n\n"
         "\b\n"
-        "Common options (see 'apm install --help' for full list):\n"
+        "Forwarded install options (see 'apm install --help' for the full list):\n"
         "  --transport [stdio|http|sse|streamable-http]\n"
         "  --url URL           Server URL for remote transports\n"
         "  --env KEY=VALUE     Environment variable (repeatable)\n"
         "  --header KEY=VALUE  HTTP header (repeatable)\n"
         "  --registry URL      Custom registry URL\n"
         "  --mcp-version VER   Pin registry entry to a specific version\n"
-        "  --dev / --dry-run / --force / --verbose / --no-policy\n"
+        "  --dev / --dry-run / --force / --verbose / --no-policy"
     ),
 )
 @click.argument("name", required=True)

tests/integration/test_commands_mcp_flow.py

@@ -121,6 +121,12 @@ def test_mcp_install_help(self) -> None:
         runner = CliRunner()
         result = runner.invoke(mcp, ["install", "--help"])
         assert result.exit_code == 0
+        description, options = result.output.split("Options:", 1)
+        assert "Forwarded install options" in description
+        assert "--transport [stdio|http|sse|streamable-http]" in description
+        assert "--mcp-version VER" in description
+        assert "Common options" not in options
+        assert "--transport [stdio|http|sse|streamable-http]" not in options
 
 
 class TestMcpSearch:

tests/integration/test_commands_mcp_phase3c.py

@@ -121,6 +121,12 @@ def test_mcp_install_help(self) -> None:
         runner = CliRunner()
         result = runner.invoke(mcp, ["install", "--help"])
         assert result.exit_code == 0
+        description, options = result.output.split("Options:", 1)
+        assert "Forwarded install options" in description
+        assert "--transport [stdio|http|sse|streamable-http]" in description
+        assert "--mcp-version VER" in description
+        assert "Common options" not in options
+        assert "--transport [stdio|http|sse|streamable-http]" not in options
 
 
 class TestMcpSearch: