Merge pull request #22 from push-based/feat/extend-reporter

feat: extend reporter

Author: Karnaukhov-kh

packages/angular-mcp-server/src/lib/angular-mcp-server.ts

@@ -15,6 +15,7 @@ import { TOOLS } from './tools/tools.js';
 import { toolNotFound } from './tools/utils.js';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
+import { fileURLToPath } from 'node:url';
 import {
   AngularMcpServerOptionsSchema,
   AngularMcpServerOptions,
@@ -94,11 +95,11 @@ export class AngularMcpServerWrapper {
 
         // Try to read the llms.txt file from the package root (optional)
         try {
-          const filePath = path.resolve(__dirname, '../../llms.txt');
+          const currentDir = path.dirname(fileURLToPath(import.meta.url));
+          const filePath = path.resolve(currentDir, '../../llms.txt');
 
           // Only attempt to read if file exists
           if (fs.existsSync(filePath)) {
-            console.log('Reading llms.txt from:', filePath);
             const content = fs.readFileSync(filePath, 'utf-8');
             const lines = content.split('\n');
 
@@ -143,13 +144,9 @@ export class AngularMcpServerWrapper {
                 });
               }
             }
-          } else {
-            console.log('llms.txt not found at:', filePath, '(skipping)');
-          }
-        } catch (ctx: unknown) {
-          if (ctx instanceof Error) {
-            console.error('Error reading llms.txt (non-fatal):', ctx.message);
           }
+        } catch {
+          // Silently ignore errors reading llms.txt (non-fatal)
         }
 
         // Scan available design system components to add them as discoverable resources
@@ -183,13 +180,8 @@ export class AngularMcpServerWrapper {
               }
             }
           }
-        } catch (ctx: unknown) {
-          if (ctx instanceof Error) {
-            console.error(
-              'Error scanning DS components (non-fatal):',
-              ctx.message,
-            );
-          }
+        } catch {
+          // Silently ignore errors scanning DS components (non-fatal)
         }
 
         return {

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

@@ -2,13 +2,12 @@ export { reportViolationsTools } from './report-violations.tool.js';
 export { reportAllViolationsTools } from './report-all-violations.tool.js';
 
 export type {
-  ReportViolationsOptions,
-  ViolationResult,
-  ViolationIssue,
-  ViolationAudit,
-  FileViolation,
-  FileViolationGroup,
-  FileViolationGroups,
-  FolderViolationSummary,
-  FolderViolationGroups,
+  ViolationEntry,
+  ComponentViolationReport,
+  AllViolationsEntry,
+  AllViolationsComponentReport,
+  AllViolationsReport,
+  ComponentViolationInFile,
+  FileViolationReport,
+  AllViolationsReportByFile,
 } from './models/types.js';

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

@@ -1,40 +1,45 @@
-import {
-  BaseViolationOptions,
-  BaseViolationResult,
-  BaseViolationIssue,
-  BaseViolationAudit,
-} from '../../shared/violation-analysis/types.js';
-
-export interface ReportViolationsOptions extends BaseViolationOptions {
-  groupBy?: 'file' | 'folder';
+// Types for report-violations (single component, no replacement field needed)
+export interface ViolationEntry {
+  file: string;
+  lines: number[];
+  violation: string;
 }
 
-export type ViolationResult = BaseViolationResult;
-export type ViolationIssue = BaseViolationIssue;
-export type ViolationAudit = BaseViolationAudit;
+export interface ComponentViolationReport {
+  component: string;
+  violations: ViolationEntry[];
+}
 
-// File-specific types (when groupBy: 'file')
-export interface FileViolation {
-  fileName: string;
-  message: string;
+// Types for report-all-violations (multiple components, replacement field needed)
+export interface AllViolationsEntry {
+  file: string;
   lines: number[];
+  violation: string;
+  replacement: string;
 }
 
-export interface FileViolationGroup {
-  message: string;
-  lines: number[];
+export interface AllViolationsComponentReport {
+  component: string;
+  violations: AllViolationsEntry[];
 }
 
-export interface FileViolationGroups {
-  [fileName: string]: FileViolationGroup;
+export interface AllViolationsReport {
+  components: AllViolationsComponentReport[];
+}
+
+// File-grouped output types for report-all-violations
+export interface ComponentViolationInFile {
+  component: string;
+  lines: number[];
+  violation: string;
+  replacement: string;
 }
 
-// Folder-specific types (when groupBy: 'folder')
-export interface FolderViolationSummary {
-  violations: number;
-  files: string[];
+export interface FileViolationReport {
+  file: string;
+  components: ComponentViolationInFile[];
 }
 
-export interface FolderViolationGroups {
-  [folderPath: string]: FolderViolationSummary;
+export interface AllViolationsReportByFile {
+  files: FileViolationReport[];
 }