[Crane: crane-migration-python-to-go-full-apm-cli-rewrite] Iteration 148: fix mcp install probe parity (revert iter148/8f799e3b name-collection regression)

Changes:
  Python Click ignore_unknown_options=True sends --X to ctx.args, not NAME.
  Remove the dead startsWith(name, "-") block added by 8f799e3 which emitted
  a wrong stdout [!] line and wrong stderr install-context error.
  Now 'apm mcp install --definitely-not-an-apm-option' -> name=="" ->
  4-line missing-arg error (Usage/Try/blank/Error) matching Python rc=2.

Run: https://github.com/githubnext/apm/actions/runs/28279263309

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

Author: github-actions[bot]

cmd/apm/cmd_mcp.go

@@ -94,9 +94,9 @@ func runMCPInstall(args []string) int {
 		case "--limit":
 			i++ // skip value
 		default:
-			// Python Click ignore_unknown_options=True accepts any first arg as NAME,
-			// even if it starts with '-'.
-			if !startsWith(a, "--limit=") && name == "" {
+			// Python Click ignore_unknown_options=True puts --X args into ctx.args,
+			// not into the NAME positional. Only non-dash args are positional.
+			if !startsWith(a, "--limit=") && !startsWith(a, "-") && name == "" {
 				name = a
 			}
 		}
@@ -108,14 +108,6 @@ func runMCPInstall(args []string) int {
 		fmt.Fprintln(os.Stderr, "Error: Missing argument 'NAME'.")
 		return 2
 	}
-	if startsWith(name, "-") {
-		fmt.Print("[!] Install interrupted after 0.0s.\n")
-		fmt.Fprintln(os.Stderr, "Usage: apm install [OPTIONS] [PACKAGES]...")
-		fmt.Fprintln(os.Stderr, "Try 'apm install --help' for help.")
-		fmt.Fprintln(os.Stderr, "")
-		fmt.Fprintln(os.Stderr, "Error: MCP name cannot start with '-'; did you forget a value for --mcp?")
-		return 2
-	}
 
 	cwd, _ := os.Getwd()
 	ymlPath, err := findApmYML(cwd)