refactor(@angular/cli): rewrite modernize MCP tool's description and schemas for clarity

The `modernize` MCP tool's description and schemas are rewritten to clarify its primary function: providing instructions and commands for an AI model to execute.

The new description includes detailed operational notes on how to use the tool's output, especially for the multi-step 'standalone' migration. The input and output schema descriptions are also enhanced to provide better context.

Author: clydin

packages/angular/cli/src/commands/mcp/tools/modernize.ts

@@ -70,7 +70,11 @@ const modernizeInputSchema = z.object({
   // Casting to [string, ...string[]] since the enum definition requires a nonempty array.
   transformations: z
     .array(z.enum(TRANSFORMATIONS.map((t) => t.name) as [string, ...string[]]))
-    .optional(),
+    .optional()
+    .describe(
+      'A list of specific transformations to get instructions for. ' +
+        'If omitted, general guidance is provided.',
+    ),
 });
 
 export type ModernizeInput = z.infer<typeof modernizeInputSchema>;
@@ -93,10 +97,14 @@ function generateInstructions(transformationNames: string[]): string[] {
     } else {
       // If no instructions are included, default to running a cli schematic with the transformation name.
       const command = `ng generate @angular/core:${transformation.name}`;
-      transformationInstructions = `To run the ${transformation.name} migration, execute the following command: \`${command}\`.`;
+      transformationInstructions = `To run the ${transformation.name} migration, execute the following command: 
+
+${command}
+.`;
     }
     if (transformation.documentationUrl) {
-      transformationInstructions += `\nFor more information, see ${transformation.documentationUrl}.`;
+      transformationInstructions += `
+For more information, see ${transformation.documentationUrl}.`;
     }
     instructions.push(transformationInstructions);
   }
@@ -116,28 +124,38 @@ export async function runModernization(input: ModernizeInput) {
 export const MODERNIZE_TOOL = declareTool({
   name: 'modernize',
   title: 'Modernize Angular Code',
-  description:
-    '<Purpose>\n' +
-    'This tool modernizes Angular code by applying the latest best practices and syntax improvements, ' +
-    'ensuring it is idiomatic, readable, and maintainable.\n\n' +
-    '</Purpose>\n' +
-    '<Use Cases>\n' +
-    '* After generating new code: Run this tool immediately after creating new Angular components, directives, ' +
-    'or services to ensure they adhere to modern standards.\n' +
-    '* On existing code: Apply to existing TypeScript files (.ts) and Angular templates (.html) to update ' +
-    'them with the latest features, such as the new built-in control flow syntax.\n\n' +
-    '* When the user asks for a specific transformation: When the transformation list is populated, ' +
-    'these specific ones will be ran on the inputs.\n' +
-    '</Use Cases>\n' +
-    '<Transformations>\n' +
-    TRANSFORMATIONS.map((t) => `* ${t.name}: ${t.description}`).join('\n') +
-    '\n</Transformations>\n',
+  description: `
+<Purpose>
+Provides instructions and commands for modernizing Angular code to align with the latest best
+practices and syntax. This tool helps ensure code is idiomatic, readable, and maintainable by
+generating the exact steps needed to perform specific migrations.
+</Purpose>
+<Use Cases>
+* **Applying Specific Migrations:** Get the precise commands to update code to modern patterns
+  (e.g., selecting 'control-flow-migration' to replace *ngIf with @if).
+* **Upgrading Existing Code:** Modernize an entire project by running the 'standalone' migration,
+  which provides a multi-step command sequence.
+* **Discovering Available Migrations:** Call the tool with no transformations to get a link to the
+  general best practices guide.
+</Use Cases>
+<Operational Notes>
+* **Execution:** This tool **provides instructions**, which you **MUST** then execute as shell commands.
+  It does not modify code directly.
+* **Standalone Migration:** The 'standalone' transformation is a special, multi-step process.
+  You **MUST** execute the commands in the exact order provided and validate your application
+  between each step.
+* **Transformation List:** The following transformations are available:
+${TRANSFORMATIONS.map((t) => `  * ${t.name}: ${t.description}`).join('\n')}
+</Operational Notes>`,
   inputSchema: modernizeInputSchema.shape,
   outputSchema: {
     instructions: z
       .array(z.string())
       .optional()
-      .describe('A list of instructions on how to run the migrations.'),
+      .describe(
+        'A list of instructions and shell commands to run the requested modernizations. ' +
+          'Each string in the array is a separate step or command.',
+      ),
   },
   isLocalOnly: true,
   isReadOnly: true,