@grafema/core 0.1.0-alpha.5 → 0.1.1-alpha

package/src/plugins/analysis/ReactAnalyzer.ts

@@ -18,6 +18,7 @@ import type { Node, CallExpression, JSXElement, JSXAttribute, VariableDeclarator
 import { Plugin, createSuccessResult, createErrorResult } from '../Plugin.js';
 import type { PluginContext, PluginResult, PluginMetadata } from '../Plugin.js';
 import type { NodeRecord } from '@grafema/types';
+import { getLine, getColumn } from './ast/utils/location.js';
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const traverse = (traverseModule as any).default || traverseModule;
@@ -224,6 +225,8 @@ export class ReactAnalyzer extends Plugin {
   }
 
   async execute(context: PluginContext): Promise<PluginResult> {
+    const logger = this.log(context);
+
     try {
       const { graph } = context;
       const modules = await this.getModules(graph);
@@ -252,11 +255,16 @@ export class ReactAnalyzer extends Plugin {
           stats.issues += result.issues;
           stats.edges += result.edges;
         } catch (err) {
-          console.error(`[ReactAnalyzer] Error analyzing ${module.file}:`, (err as Error).message);
+          // Silent - per-module errors shouldn't spam logs
         }
       }
 
-      console.log(`[ReactAnalyzer] Found ${stats.components} components, ${stats.hooks} hooks, ${stats.events} events, ${stats.issues} issues`);
+      logger.info('Analysis complete', {
+        components: stats.components,
+        hooks: stats.hooks,
+        events: stats.events,
+        issues: stats.issues
+      });
 
       return createSuccessResult(
         {
@@ -266,7 +274,7 @@ export class ReactAnalyzer extends Plugin {
         stats
       );
     } catch (error) {
-      console.error(`[ReactAnalyzer] Error:`, error);
+      logger.error('Analysis failed', { error });
       return createErrorResult(error as Error);
     }
   }
@@ -328,14 +336,13 @@ export class ReactAnalyzer extends Plugin {
         if (this.isReactComponent(path)) {
           const node = path.node;
           const name = (node.id as { name: string }).name;
-          const loc = node.loc!;
           const component: ComponentNode = {
-            id: `react:component#${name}#${filePath}:${loc.start.line}`,
+            id: `react:component#${name}#${filePath}:${getLine(node)}`,
             type: 'react:component',
             name,
             file: filePath,
-            line: loc.start.line,
-            column: loc.start.column,
+            line: getLine(node),
+            column: getColumn(node),
             kind: 'arrow'
           };
           analysis.components.push(component);
@@ -348,14 +355,13 @@ export class ReactAnalyzer extends Plugin {
           const name = path.node.id?.name;
           if (!name) return;
 
-          const loc = path.node.loc!;
           const component: ComponentNode = {
-            id: `react:component#${name}#${filePath}:${loc.start.line}`,
+            id: `react:component#${name}#${filePath}:${getLine(path.node)}`,
             type: 'react:component',
             name,
             file: filePath,
-            line: loc.start.line,
-            column: loc.start.column,
+            line: getLine(path.node),
+            column: getColumn(path.node),
             kind: 'function'
           };
           analysis.components.push(component);
@@ -459,13 +465,12 @@ export class ReactAnalyzer extends Plugin {
   private analyzeHook(path: NodePath<CallExpression>, filePath: string): HookNode | null {
     const callee = path.node.callee as { name: string };
     const hookName = callee.name;
-    const loc = path.node.loc!;
     const args = path.node.arguments;
 
     const hookBase = {
       file: filePath,
-      line: loc.start.line,
-      column: loc.start.column,
+      line: getLine(path.node),
+      column: getColumn(path.node),
       hookName
     };
 
@@ -481,7 +486,7 @@ export class ReactAnalyzer extends Plugin {
           const initialValue = args[0];
 
           return {
-            id: `react:state#${stateName}#${filePath}:${loc.start.line}`,
+            id: `react:state#${stateName}#${filePath}:${hookBase.line}`,
             type: 'react:state',
             ...hookBase,
             stateName,
@@ -506,7 +511,7 @@ export class ReactAnalyzer extends Plugin {
                           'react:insertion-effect';
 
         return {
-          id: `${effectType}#${filePath}:${loc.start.line}`,
+          id: `${effectType}#${filePath}:${hookBase.line}`,
           type: effectType,
           ...hookBase,
           deps,
@@ -523,7 +528,7 @@ export class ReactAnalyzer extends Plugin {
         const deps = this.extractDeps(depsArg as Node);
 
         return {
-          id: `react:callback#${callbackName || 'anonymous'}#${filePath}:${loc.start.line}`,
+          id: `react:callback#${callbackName || 'anonymous'}#${filePath}:${hookBase.line}`,
           type: 'react:callback',
           ...hookBase,
           callbackName,
@@ -539,7 +544,7 @@ export class ReactAnalyzer extends Plugin {
         const deps = this.extractDeps(depsArg as Node);
 
         return {
-          id: `react:memo#${memoName || 'anonymous'}#${filePath}:${loc.start.line}`,
+          id: `react:memo#${memoName || 'anonymous'}#${filePath}:${hookBase.line}`,
           type: 'react:memo',
           ...hookBase,
           memoName,
@@ -554,7 +559,7 @@ export class ReactAnalyzer extends Plugin {
         const initialValue = args[0];
 
         return {
-          id: `react:ref#${refName || 'anonymous'}#${filePath}:${loc.start.line}`,
+          id: `react:ref#${refName || 'anonymous'}#${filePath}:${hookBase.line}`,
           type: 'react:ref',
           ...hookBase,
           refName,
@@ -574,7 +579,7 @@ export class ReactAnalyzer extends Plugin {
           const reducerName = reducerArg?.type === 'Identifier' ? (reducerArg as { name: string }).name : null;
 
           return {
-            id: `react:reducer#${stateName}#${filePath}:${loc.start.line}`,
+            id: `react:reducer#${stateName}#${filePath}:${hookBase.line}`,
             type: 'react:reducer',
             ...hookBase,
             stateName,
@@ -593,7 +598,7 @@ export class ReactAnalyzer extends Plugin {
         const contextName = contextArg?.type === 'Identifier' ? (contextArg as { name: string }).name : null;
 
         return {
-          id: `react:context-use#${contextName || 'unknown'}#${filePath}:${loc.start.line}`,
+          id: `react:context-use#${contextName || 'unknown'}#${filePath}:${hookBase.line}`,
           type: 'react:context-use',
           ...hookBase,
           valueName,
@@ -623,7 +628,7 @@ export class ReactAnalyzer extends Plugin {
         }
 
         return {
-          id: `react:imperative-handle#${filePath}:${loc.start.line}`,
+          id: `react:imperative-handle#${filePath}:${hookBase.line}`,
           type: 'react:imperative-handle',
           ...hookBase,
           refName,
@@ -963,7 +968,6 @@ export class ReactAnalyzer extends Plugin {
     }
 
     // This is a React component being rendered
-    const loc = openingElement.loc!;
 
     // Find parent component
     let parentComponent: string | null = null;
@@ -988,7 +992,7 @@ export class ReactAnalyzer extends Plugin {
         src: `react:component#${parentComponent}`,
         dst: `react:component#${elementName}`,
         file: filePath,
-        line: loc.start.line
+        line: getLine(openingElement)
       });
     }
   }
@@ -1026,7 +1030,6 @@ export class ReactAnalyzer extends Plugin {
     if (!attr.name || attr.name.type !== 'JSXIdentifier') return;
 
     const attrName = attr.name.name;
-    const loc = attr.loc!;
 
     // Get parent JSX element info first
     const jsxOpeningElement = path.parent as { type: string; name?: Node };
@@ -1054,13 +1057,13 @@ export class ReactAnalyzer extends Plugin {
       }
 
       const event: EventNode = {
-        id: `dom:event#${eventType}#${filePath}:${loc.start.line}`,
+        id: `dom:event#${eventType}#${filePath}:${getLine(attr)}`,
         type: 'dom:event',
         eventType,
         reactProp: attrName,
         handler: handlerName,
         file: filePath,
-        line: loc.start.line
+        line: getLine(attr)
       };
       analysis.events.push(event);
     }
@@ -1105,7 +1108,7 @@ export class ReactAnalyzer extends Plugin {
           propName: attrName,
           propValue,
           file: filePath,
-          line: loc.start.line
+          line: getLine(attr)
         });
       }
     }
@@ -1115,18 +1118,17 @@ export class ReactAnalyzer extends Plugin {
    * Analyze forwardRef usage
    */
   private analyzeForwardRef(path: NodePath<CallExpression>, filePath: string, analysis: AnalysisResult): void {
-    const loc = path.node.loc!;
     const parent = path.parent as { type: string; id?: { name: string } };
     const componentName = parent.type === 'VariableDeclarator' ? parent.id?.name : null;
 
     if (componentName) {
       analysis.components.push({
-        id: `react:component#${componentName}#${filePath}:${loc.start.line}`,
+        id: `react:component#${componentName}#${filePath}:${getLine(path.node)}`,
         type: 'react:component',
         name: componentName,
         file: filePath,
-        line: loc.start.line,
-        column: loc.start.column,
+        line: getLine(path.node),
+        column: getColumn(path.node),
         kind: 'forwardRef'
       });
     }
@@ -1136,19 +1138,18 @@ export class ReactAnalyzer extends Plugin {
    * Analyze createContext usage
    */
   private analyzeCreateContext(path: NodePath<CallExpression>, filePath: string, analysis: AnalysisResult): void {
-    const loc = path.node.loc!;
     const parent = path.parent as { type: string; id?: { name: string } };
     const contextName = parent.type === 'VariableDeclarator' ? parent.id?.name : null;
 
     if (contextName) {
       const defaultValue = path.node.arguments[0];
       analysis.hooks.push({
-        id: `react:context#${contextName}#${filePath}:${loc.start.line}`,
+        id: `react:context#${contextName}#${filePath}:${getLine(path.node)}`,
         type: 'react:context',
         contextName,
         file: filePath,
-        line: loc.start.line,
-        column: loc.start.column,
+        line: getLine(path.node),
+        column: getColumn(path.node),
         hookName: 'createContext',
         defaultValue: this.getExpressionValue(defaultValue as Node)
       });
@@ -1160,7 +1161,6 @@ export class ReactAnalyzer extends Plugin {
    */
   private analyzeBrowserAPI(path: NodePath<CallExpression>, filePath: string, analysis: AnalysisResult): void {
     const callee = path.node.callee;
-    const loc = path.node.loc!;
 
     // Direct function call: setTimeout, fetch, alert
     if (callee.type === 'Identifier') {
@@ -1169,11 +1169,11 @@ export class ReactAnalyzer extends Plugin {
       // Timers
       if (BROWSER_APIS.timers.includes(name)) {
         analysis.browserAPIs.push({
-          id: `browser:timer#${name}#${filePath}:${loc.start.line}`,
+          id: `browser:timer#${name}#${filePath}:${getLine(path.node)}`,
           type: 'browser:timer',
           api: name,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1181,11 +1181,11 @@ export class ReactAnalyzer extends Plugin {
       // Blocking APIs
       if (BROWSER_APIS.blocking.includes(name)) {
         analysis.browserAPIs.push({
-          id: `browser:blocking#${name}#${filePath}:${loc.start.line}`,
+          id: `browser:blocking#${name}#${filePath}:${getLine(path.node)}`,
           type: 'browser:blocking',
           api: name,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1193,11 +1193,11 @@ export class ReactAnalyzer extends Plugin {
       // Fetch
       if (name === 'fetch') {
         analysis.browserAPIs.push({
-          id: `browser:async#fetch#${filePath}:${loc.start.line}`,
+          id: `browser:async#fetch#${filePath}:${getLine(path.node)}`,
           type: 'browser:async',
           api: 'fetch',
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1215,12 +1215,12 @@ export class ReactAnalyzer extends Plugin {
                          method === 'removeItem' ? 'delete' : method;
 
         analysis.browserAPIs.push({
-          id: `browser:storage#${storage}:${operation}#${filePath}:${loc.start.line}`,
+          id: `browser:storage#${storage}:${operation}#${filePath}:${getLine(path.node)}`,
           type: 'browser:storage',
           storage,
           operation,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1229,12 +1229,12 @@ export class ReactAnalyzer extends Plugin {
       if (fullName.startsWith('document.') &&
           (fullName.includes('querySelector') || fullName.includes('getElementById'))) {
         analysis.browserAPIs.push({
-          id: `browser:dom#query#${filePath}:${loc.start.line}`,
+          id: `browser:dom#query#${filePath}:${getLine(path.node)}`,
           type: 'browser:dom',
           operation: 'query',
           api: fullName,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1242,11 +1242,11 @@ export class ReactAnalyzer extends Plugin {
       // History API
       if (fullName.startsWith('history.') || fullName.startsWith('window.history.')) {
         analysis.browserAPIs.push({
-          id: `browser:history#${filePath}:${loc.start.line}`,
+          id: `browser:history#${filePath}:${getLine(path.node)}`,
           type: 'browser:history',
           api: fullName,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1254,11 +1254,11 @@ export class ReactAnalyzer extends Plugin {
       // Clipboard API
       if (fullName.includes('clipboard')) {
         analysis.browserAPIs.push({
-          id: `browser:clipboard#${filePath}:${loc.start.line}`,
+          id: `browser:clipboard#${filePath}:${getLine(path.node)}`,
           type: 'browser:clipboard',
           api: fullName,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1266,11 +1266,11 @@ export class ReactAnalyzer extends Plugin {
       // Geolocation
       if (fullName.includes('geolocation')) {
         analysis.browserAPIs.push({
-          id: `browser:geolocation#${filePath}:${loc.start.line}`,
+          id: `browser:geolocation#${filePath}:${getLine(path.node)}`,
           type: 'browser:geolocation',
           api: fullName,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1279,11 +1279,11 @@ export class ReactAnalyzer extends Plugin {
       if (fullName.match(/\.(fillRect|strokeRect|fillText|strokeText|beginPath|closePath|moveTo|lineTo|arc|fill|stroke|clearRect|drawImage|save|restore|translate|rotate|scale)$/)) {
         const method = fullName.split('.').pop();
         analysis.browserAPIs.push({
-          id: `canvas:draw#${method}#${filePath}:${loc.start.line}`,
+          id: `canvas:draw#${method}#${filePath}:${getLine(path.node)}`,
           type: 'canvas:draw',
           method,
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }
@@ -1291,11 +1291,11 @@ export class ReactAnalyzer extends Plugin {
       // matchMedia
       if (fullName === 'window.matchMedia' || fullName === 'matchMedia') {
         analysis.browserAPIs.push({
-          id: `browser:media-query#${filePath}:${loc.start.line}`,
+          id: `browser:media-query#${filePath}:${getLine(path.node)}`,
           type: 'browser:media-query',
           api: 'matchMedia',
           file: filePath,
-          line: loc.start.line
+          line: getLine(path.node)
         });
         return;
       }

package/src/plugins/analysis/RustAnalyzer.ts

@@ -140,19 +140,20 @@ interface EdgeToAdd {
 let parseRustFile: ((code: string) => RustParseResult) | undefined;
 
 // Try to load the native binding
+// Path: from dist/plugins/analysis/ go up 5 levels to reach project root, then rust-engine/
 try {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  const nativeBinding = await import('../../../../rust-engine/grafema-graph-engine.node' as any);
+  const nativeBinding = await import('../../../../../rust-engine/grafema-graph-engine.node' as any);
   parseRustFile = nativeBinding.parseRustFile;
 } catch {
   // Fallback: try require
   try {
     const { createRequire } = await import('module');
     const require = createRequire(import.meta.url);
-    const nativeBinding = require('../../../../rust-engine/grafema-graph-engine.node');
+    const nativeBinding = require('../../../../../rust-engine/grafema-graph-engine.node');
     parseRustFile = nativeBinding.parseRustFile;
-  } catch (e2) {
-    console.warn('[RustAnalyzer] Native binding not available:', (e2 as Error).message);
+  } catch {
+    // Silent - will be reported during execute if needed
   }
 }
 
@@ -179,9 +180,10 @@ export class RustAnalyzer extends Plugin {
 
   async execute(context: PluginContext): Promise<PluginResult> {
     const { graph, onProgress } = context;
+    const logger = this.log(context);
 
     if (!parseRustFile) {
-      console.log('[RustAnalyzer] Skipping - native binding not available');
+      logger.info('Skipping - native binding not available');
       return createSuccessResult(
         { nodes: 0, edges: 0 },
         { skipped: true, reason: 'Native binding not available' }
@@ -195,11 +197,11 @@ export class RustAnalyzer extends Plugin {
     }
 
     if (modules.length === 0) {
-      console.log('[RustAnalyzer] No RUST_MODULE nodes found, skipping');
+      logger.info('No RUST_MODULE nodes found, skipping');
       return createSuccessResult({ nodes: 0, edges: 0 }, { skipped: true, reason: 'No modules' });
     }
 
-    console.log(`[RustAnalyzer] Analyzing ${modules.length} Rust modules...`);
+    logger.info('Analyzing Rust modules', { count: modules.length });
 
     const stats: AnalysisStats = {
       functions: 0,
@@ -238,15 +240,18 @@ export class RustAnalyzer extends Plugin {
         }
       } catch (err) {
         errors.push({ file: module.file!, error: (err as Error).message });
-        console.error(`[RustAnalyzer] Error parsing ${module.file}:`, (err as Error).message);
+        logger.warn('Error parsing module', {
+          file: module.file,
+          error: (err as Error).message
+        });
       }
     }
 
     if (errors.length > 0) {
-      console.warn(`[RustAnalyzer] ${errors.length} errors during analysis`);
+      logger.warn('Analysis completed with errors', { errorCount: errors.length });
     }
 
-    console.log(`[RustAnalyzer] Created: ${JSON.stringify(stats)}`);
+    logger.info('Analysis complete', { ...stats });
     return createSuccessResult(
       { nodes: stats.functions + stats.structs + stats.impls + stats.methods + stats.traits + stats.calls, edges: stats.edges },
       { ...stats, errors: errors.length }

package/src/plugins/analysis/SQLiteAnalyzer.ts

@@ -16,6 +16,7 @@ import type { NodePath } from '@babel/traverse';
 import { Plugin, createSuccessResult, createErrorResult } from '../Plugin.js';
 import type { PluginContext, PluginResult, PluginMetadata } from '../Plugin.js';
 import type { NodeRecord } from '@grafema/types';
+import { getLine } from './ast/utils/location.js';
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const traverse = (traverseModule as any).default || traverseModule;
@@ -62,6 +63,8 @@ export class SQLiteAnalyzer extends Plugin {
   }
 
   async execute(context: PluginContext): Promise<PluginResult> {
+    const logger = this.log(context);
+
     try {
       const { graph } = context;
 
@@ -80,14 +83,14 @@ export class SQLiteAnalyzer extends Plugin {
         edgesCreated += result.edges;
       }
 
-      console.log(`[SQLiteAnalyzer] Found ${queriesCreated} queries, ${operationsCreated} operations`);
+      logger.info('Analysis complete', { queriesCreated, operationsCreated });
 
       return createSuccessResult(
         { nodes: queriesCreated + operationsCreated, edges: edgesCreated },
         { modulesAnalyzed: modules.length, queries: queriesCreated, operations: operationsCreated }
       );
     } catch (error) {
-      console.error(`[SQLiteAnalyzer] Error:`, error);
+      logger.error('Analysis failed', { error });
       return createErrorResult(error as Error);
     }
   }
@@ -167,7 +170,7 @@ export class SQLiteAnalyzer extends Plugin {
                 const operationType = this.detectOperationType(query);
                 const tableName = this.extractTableName(query, operationType);
 
-                const queryId = `${module.file}:DATABASE_QUERY:${method}:${node.loc!.start.line}`;
+                const queryId = `${module.file}:DATABASE_QUERY:${method}:${getLine(node)}`;
 
                 queries.push({
                   id: queryId,
@@ -178,7 +181,7 @@ export class SQLiteAnalyzer extends Plugin {
                   operationType: operationType,
                   tableName: tableName,
                   file: module.file!,
-                  line: node.loc!.start.line
+                  line: getLine(node)
                 });
               }
             }
@@ -239,7 +242,7 @@ export class SQLiteAnalyzer extends Plugin {
                         const operationType = this.detectOperationType(query);
                         const tableName = this.extractTableName(query, operationType);
 
-                        const queryId = `${module.file}:DATABASE_QUERY:${method}:${innerNode.loc!.start.line}`;
+                        const queryId = `${module.file}:DATABASE_QUERY:${method}:${getLine(innerNode)}`;
 
                         queries.push({
                           id: queryId,
@@ -250,7 +253,7 @@ export class SQLiteAnalyzer extends Plugin {
                           operationType: operationType,
                           tableName: tableName,
                           file: module.file!,
-                          line: innerNode.loc!.start.line,
+                          line: getLine(innerNode),
                           promiseWrapped: true
                         });
                       }
@@ -307,7 +310,7 @@ export class SQLiteAnalyzer extends Plugin {
         }
       }
     } catch (error) {
-      console.error(`[SQLiteAnalyzer] Error analyzing ${module.file}:`, (error as Error).message);
+      // Silent - per-module errors shouldn't spam logs
     }
 
     return {

package/src/plugins/analysis/ServiceLayerAnalyzer.ts

@@ -24,6 +24,7 @@ import type { NodePath } from '@babel/traverse';
 import { Plugin, createSuccessResult, createErrorResult } from '../Plugin.js';
 import type { PluginContext, PluginResult, PluginMetadata } from '../Plugin.js';
 import type { NodeRecord } from '@grafema/types';
+import { getLine } from './ast/utils/location.js';
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const traverse = (traverseModule as any).default || traverseModule;
@@ -99,12 +100,14 @@ export class ServiceLayerAnalyzer extends Plugin {
   }
 
   async execute(context: PluginContext): Promise<PluginResult> {
+    const logger = this.log(context);
+
     try {
       const { graph } = context;
 
       // Получаем все модули
       const modules = await this.getModules(graph);
-      console.log(`[ServiceLayerAnalyzer] Processing ${modules.length} modules...`);
+      logger.info('Processing modules', { count: modules.length });
 
       let classesCount = 0;
       let instancesCount = 0;
@@ -124,15 +127,21 @@ export class ServiceLayerAnalyzer extends Plugin {
         if ((i + 1) % 20 === 0 || i === modules.length - 1) {
           const elapsed = ((Date.now() - startTime) / 1000).toFixed(1);
           const avgTime = ((Date.now() - startTime) / (i + 1)).toFixed(0);
-          console.log(
-            `[ServiceLayerAnalyzer] Progress: ${i + 1}/${modules.length} (${elapsed}s, avg ${avgTime}ms/module)`
-          );
+          logger.debug('Progress', {
+            current: i + 1,
+            total: modules.length,
+            elapsed: `${elapsed}s`,
+            avgTime: `${avgTime}ms/module`
+          });
         }
       }
 
-      console.log(
-        `[ServiceLayerAnalyzer] Found ${classesCount} service classes, ${instancesCount} instances, ${registrationsCount} registrations, ${usagesCount} usages`
-      );
+      logger.info('Analysis complete', {
+        classesCount,
+        instancesCount,
+        registrationsCount,
+        usagesCount
+      });
 
       return createSuccessResult(
         {
@@ -147,7 +156,7 @@ export class ServiceLayerAnalyzer extends Plugin {
         }
       );
     } catch (error) {
-      console.error('[ServiceLayerAnalyzer] Error:', error);
+      logger.error('Analysis failed', { error });
       return createErrorResult(error as Error);
     }
   }
@@ -186,7 +195,7 @@ export class ServiceLayerAnalyzer extends Plugin {
           const className = node.id?.name;
 
           if (className && this.isServiceClass(className)) {
-            const line = node.loc!.start.line;
+            const line = getLine(node);
 
             // Извлекаем методы сервиса
             const methods = node.body.body
@@ -213,7 +222,7 @@ export class ServiceLayerAnalyzer extends Plugin {
             const className = (callee as Identifier).name;
 
             if (this.isServiceClass(className)) {
-              const line = node.loc!.start.line;
+              const line = getLine(node);
 
               serviceInstances.push({
                 id: `${module.file}:SERVICE_INSTANCE:${className}:${line}`,
@@ -240,7 +249,7 @@ export class ServiceLayerAnalyzer extends Plugin {
             const objectName = this.getObjectName(callee.object);
             if (objectName === 'app' && node.arguments.length >= 2) {
               const serviceName = this.extractStringArg(node.arguments[0]);
-              const line = node.loc!.start.line;
+              const line = getLine(node);
 
               // Проверяем что это похоже на service (имя содержит 'service' или '*Service')
               if (
@@ -273,7 +282,7 @@ export class ServiceLayerAnalyzer extends Plugin {
             ) {
               if (node.arguments.length >= 1) {
                 const serviceName = this.extractStringArg(node.arguments[0]);
-                const line = node.loc!.start.line;
+                const line = getLine(node);
 
                 // Проверяем что это похоже на service
                 if (
@@ -358,10 +367,7 @@ export class ServiceLayerAnalyzer extends Plugin {
         usages: serviceUsages.length
       };
     } catch (error) {
-      console.error(
-        `[ServiceLayerAnalyzer] Error analyzing ${module.file}:`,
-        (error as Error).message
-      );
+      // Silent - per-module errors shouldn't spam logs
       return { classes: 0, instances: 0, registrations: 0, usages: 0 };
     }
   }