refactor(mcp): drop explanatory comments added in the uninstall picker

Author: rafa-thayto

packages/cli-core/src/commands/mcp/uninstall.test.ts

@@ -17,8 +17,6 @@ mock.module("../../mode.ts", () => ({
 let mockHome = realOs.tmpdir();
 mock.module("node:os", () => ({ ...realOs, homedir: () => mockHome }));
 
-// The human picker resolves the Clack multiselect lazily from lib/prompts.ts;
-// stub it so tests drive which clients get selected without a real prompt.
 const mockMultiselect = mock();
 mock.module("../../lib/prompts.ts", () => ({
   multiselect: (...args: unknown[]) => mockMultiselect(...args),

packages/cli-core/src/commands/mcp/uninstall.ts

@@ -23,17 +23,13 @@ function printResult(client: McpClient, result: RemoveResult): void {
   log.info(`${label}: ${result.removed ? green("removed") : yellow("not present")}`);
 }
 
-/** Supported clients that currently have the `name` entry registered. */
 async function installedClients(name: string, cwd: string): Promise<McpClient[]> {
   const present = await Promise.all(
     CLIENTS.map(async (client) => (await client.list(cwd)).some((entry) => entry.name === name)),
   );
   return CLIENTS.filter((_, i) => present[i]);
 }
 
-// `--client` targets exactly those; `--all` (and agent mode, which can't
-// prompt) targets every client; otherwise the human picks which of the clients
-// that actually have the entry to remove it from.
 async function chooseClients(options: McpOptions, name: string, cwd: string): Promise<McpClient[]> {
   if (options.client && options.client.length > 0) return resolveClients(options.client);
   if (options.all || isAgent()) return Array.from(CLIENTS);