feat(angular-mcp-server): separate reporter for single violation

Author: Karnaukhov-kh

packages/angular-mcp-server/src/lib/tools/ds/report-violations/index.ts

@@ -4,6 +4,8 @@ export { reportAllViolationsTools } from './report-all-violations.tool.js';
 export type {
   ViolationEntry,
   ComponentViolationReport,
+  AllViolationsEntry,
+  AllViolationsComponentReport,
   AllViolationsReport,
   ComponentViolationInFile,
   FileViolationReport,

packages/angular-mcp-server/src/lib/tools/ds/report-violations/models/types.ts

@@ -1,21 +1,33 @@
-// JSON output types
+// Types for report-violations (single component, no replacement field needed)
 export interface ViolationEntry {
   file: string;
   lines: number[];
   violation: string;
-  replacement: string;
 }
 
 export interface ComponentViolationReport {
   component: string;
   violations: ViolationEntry[];
 }
 
+// Types for report-all-violations (multiple components, replacement field needed)
+export interface AllViolationsEntry {
+  file: string;
+  lines: number[];
+  violation: string;
+  replacement: string;
+}
+
+export interface AllViolationsComponentReport {
+  component: string;
+  violations: AllViolationsEntry[];
+}
+
 export interface AllViolationsReport {
-  components: ComponentViolationReport[];
+  components: AllViolationsComponentReport[];
 }
 
-// File-grouped output types
+// File-grouped output types for report-all-violations
 export interface ComponentViolationInFile {
   component: string;
   lines: number[];

packages/angular-mcp-server/src/lib/tools/ds/report-violations/report-all-violations.tool.ts

@@ -17,9 +17,9 @@ import {
 import type { BaseViolationAudit } from '../shared/violation-analysis/types.js';
 import { loadAndValidateDsComponentsFile } from '../../../validation/ds-components-file-loader.validation.js';
 import { 
-  AllViolationsReport, 
-  ComponentViolationReport, 
-  ViolationEntry,
+  AllViolationsReport,
+  AllViolationsComponentReport,
+  AllViolationsEntry,
   AllViolationsReportByFile,
   FileViolationReport,
   ComponentViolationInFile
@@ -161,7 +161,7 @@ export const reportAllViolationsHandler = createHandler<
 
     // Group by component (default behavior)
     if (params.groupBy !== 'file') {
-      const componentMap = new Map<string, ViolationEntry[]>();
+      const componentMap = new Map<string, AllViolationsEntry[]>();
 
       for (const item of processed) {
         if (!componentMap.has(item.component)) {
@@ -179,7 +179,7 @@ export const reportAllViolationsHandler = createHandler<
         }
       }
 
-      const components: ComponentViolationReport[] = Array.from(
+      const components: AllViolationsComponentReport[] = Array.from(
         componentMap.entries(),
         ([component, violations]) => ({ component, violations }),
       );

packages/angular-mcp-server/src/lib/tools/ds/report-violations/report-violations.tool.ts

@@ -1,7 +1,6 @@
 import {
   createHandler,
   BaseHandlerOptions,
-  RESULT_FORMATTERS,
 } from '../shared/utils/handler-helpers.js';
 import {
   createViolationReportingSchema,
@@ -29,21 +28,15 @@ export const reportViolationsSchema = {
 };
 
 /**
- * Extracts deprecated class and replacement from violation message
+ * Extracts deprecated class from violation message
  */
-function parseViolationMessage(message: string): { violation: string; replacement: string } {
+function parseViolationMessage(message: string): string {
   // Clean up HTML tags
   const cleanMessage = message.replace(/<code>/g, '`').replace(/<\/code>/g, '`');
 
   // Extract deprecated class - look for patterns like "class `offer-badge`" or "class `btn, btn-primary`"
   const classMatch = cleanMessage.match(/class `([^`]+)`/);
-  const violation = classMatch ? classMatch[1] : 'unknown';
-  
-  // Extract replacement component - look for "Use `ComponentName`"
-  const replacementMatch = cleanMessage.match(/Use `([^`]+)`/);
-  const replacement = replacementMatch ? replacementMatch[1] : 'unknown';
-  
-  return { violation, replacement };
+  return classMatch ? classMatch[1] : 'unknown';
 }
 
 export const reportViolationsHandler = createHandler<
@@ -72,13 +65,12 @@ export const reportViolationsHandler = createHandler<
       );
 
       for (const [fileName, { lines, message }] of Object.entries(fileGroups)) {
-        const { violation, replacement } = parseViolationMessage(message);
+        const violation = parseViolationMessage(message);
 
         violations.push({
           file: fileName,
           lines: lines.sort((a, b) => a - b),
           violation,
-          replacement,
         });
       }
     }
@@ -88,7 +80,25 @@ export const reportViolationsHandler = createHandler<
       violations,
     };
   },
-  (result) => RESULT_FORMATTERS.json(result),
+  (result) => {
+    if (result.violations.length === 0) {
+      return [`No violations found for component: ${result.component}`];
+    }
+
+    const message = [
+      `Found violations for component: ${result.component}`,
+      'Use this output to identify:',
+      '  - Which files contain violations',
+      '  - The specific line numbers where violations occur',
+      '  - What is being used that violates the rules (violation field)',
+      '',
+      'Violation Report:',
+      '',
+      JSON.stringify(result, null, 2),
+    ];
+
+    return [message.join('\n')];
+  },
 );
 
 export const reportViolationsTools = [