refactor(@angular/cli): promote experimental MCP tools to stable

Promote `run_target` and `devserver` tools (which includes `devserver.start`, `devserver.stop`, and `devserver.wait_for_build`) from experimental to stable so they are registered and available by default on the MCP server.

Update the tool assembly and warning logic to allow enabling these now-stable tools via `-E` or `--experimental-tool` without generating "Unknown experimental tool" warnings, preserving backward compatibility. Update run-target e2e tests accordingly.

Author: clydin

packages/angular/cli/src/commands/mcp/mcp-server.ts

@@ -41,13 +41,15 @@ const STABLE_TOOLS = [
   DOC_SEARCH_TOOL,
   LIST_PROJECTS_TOOL,
   ZONELESS_MIGRATION_TOOL,
+  RUN_TARGET_TOOL,
+  ...DEVSERVER_TOOLS,
 ] as const;
 
 /**
  * The set of tools that are available but not enabled by default.
  * These tools are considered experimental and may have limitations.
  */
-export const EXPERIMENTAL_TOOLS = [RUN_TARGET_TOOL, ...DEVSERVER_TOOLS] as const;
+export const EXPERIMENTAL_TOOLS: readonly AnyMcpToolDeclaration[] = [] as const;
 
 /**
  * Experimental tools that are grouped together under a single name.
@@ -56,7 +58,6 @@ export const EXPERIMENTAL_TOOLS = [RUN_TARGET_TOOL, ...DEVSERVER_TOOLS] as const
  */
 export const EXPERIMENTAL_TOOL_GROUPS = {
   'all': EXPERIMENTAL_TOOLS,
-  'devserver': DEVSERVER_TOOLS,
 };
 
 export async function createMcpServer(
@@ -205,7 +206,7 @@ export function assembleToolDeclarations(
       const tool = experimentalToolsMap.get(toolName);
       if (tool) {
         toolDeclarations.push(tool);
-      } else {
+      } else if (!stableDeclarations.some((t) => t.name === toolName)) {
         options.logger.warn(`Unknown experimental tool: ${toolName}`);
       }
     }

tests/e2e/tests/mcp/run-target.ts

@@ -29,11 +29,11 @@ export default async function () {
   );
 
   try {
-    // 1. Ensure `run_target` is NOT registered by default (stable-only tools registered)
+    // 1. Ensure `run_target` is registered by default (now stable)
     const { stdout: stdoutDefault } = await runInspector('--method', 'tools/list');
-    assert.doesNotMatch(stdoutDefault, /"run_target"/);
+    assert.match(stdoutDefault, /"run_target"/);
 
-    // 2. Ensure `run_target` is registered when explicitly enabled via experimental-tool flag
+    // 2. Ensure `run_target` is also registered when using the deprecated experimental-tool flag
     const { stdout: stdoutEnabled } = await runInspector(
       '-E',
       'run_target',
@@ -44,8 +44,6 @@ export default async function () {
 
     // 3. Call run_target with build target
     const { stdout: stdoutCall } = await runInspector(
-      '-E',
-      'run_target',
       '--method',
       'tools/call',
       '--tool-name',
@@ -66,8 +64,6 @@ export default async function () {
     // 4. Call run_target with test target (only for esbuild/Vite test runner, as webpack-based Karma fails on this bazel CI headless runner)
     if (esbuild) {
       const { stdout: stdoutTestCall } = await runInspector(
-        '-E',
-        'run_target',
         '--method',
         'tools/call',
         '--tool-name',