@grafema/core 0.1.1-alpha → 0.2.1-beta

package/dist/plugins/analysis/ExpressResponseAnalyzer.js

@@ -0,0 +1,495 @@
+/**
+ * ExpressResponseAnalyzer - detects Express response patterns
+ *
+ * For each http:route node:
+ * 1. Follow HANDLED_BY edge to get handler function
+ * 2. Traverse handler AST for res.json(...), res.send(...) patterns
+ * 3. Create RESPONDS_WITH edge from http:route to response argument node
+ *
+ * Patterns:
+ * - res.json({ data })
+ * - res.send(variable)
+ * - res.status(200).json(data)
+ */
+import { readFileSync } from 'fs';
+import { parse } from '@babel/parser';
+import traverseModule from '@babel/traverse';
+import { Plugin, createSuccessResult, createErrorResult } from '../Plugin.js';
+import { getLine } from './ast/utils/location.js';
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+const traverse = traverseModule.default || traverseModule;
+const RESPONSE_METHODS = ['json', 'send'];
+export class ExpressResponseAnalyzer extends Plugin {
+    responseNodeCounter = 0;
+    get metadata() {
+        return {
+            name: 'ExpressResponseAnalyzer',
+            phase: 'ANALYSIS',
+            priority: 74, // After ExpressRouteAnalyzer (75)
+            creates: {
+                nodes: [],
+                edges: ['RESPONDS_WITH']
+            },
+            dependencies: ['ExpressRouteAnalyzer', 'JSASTAnalyzer']
+        };
+    }
+    async execute(context) {
+        const logger = this.log(context);
+        try {
+            const { graph } = context;
+            // Get all http:route nodes
+            const routes = [];
+            for await (const node of graph.queryNodes({ type: 'http:route' })) {
+                routes.push(node);
+            }
+            logger.info('Processing routes', { count: routes.length });
+            let edgesCreated = 0;
+            for (const route of routes) {
+                const result = await this.analyzeRouteResponses(route, graph);
+                edgesCreated += result;
+            }
+            logger.info('Analysis complete', { edgesCreated });
+            return createSuccessResult({ nodes: 0, edges: edgesCreated }, { routesAnalyzed: routes.length });
+        }
+        catch (error) {
+            logger.error('Analysis failed', { error });
+            return createErrorResult(error);
+        }
+    }
+    /**
+     * Analyze a single http:route for response patterns
+     */
+    async analyzeRouteResponses(route, graph) {
+        let edgesCreated = 0;
+        try {
+            // Get HANDLED_BY edges to find handler function
+            const handledByEdges = await graph.getOutgoingEdges(route.id, ['HANDLED_BY']);
+            if (handledByEdges.length === 0) {
+                return 0;
+            }
+            // Get handler function node
+            const handlerEdge = handledByEdges[0];
+            const handlerNode = await graph.getNode(handlerEdge.dst);
+            if (!handlerNode || !handlerNode.file) {
+                return 0;
+            }
+            // Parse the file and find response calls in handler
+            const code = readFileSync(handlerNode.file, 'utf-8');
+            const ast = parse(code, {
+                sourceType: 'module',
+                plugins: ['jsx', 'typescript']
+            });
+            // Find response calls within the handler's line range
+            const responseCalls = this.findResponseCalls(ast, handlerNode.file, handlerNode.line);
+            // Create RESPONDS_WITH edges
+            for (const call of responseCalls) {
+                // Try to resolve to existing variable, or create a unique response node
+                const dstNodeId = await this.resolveOrCreateResponseNode(graph, handlerNode.file, call, route.id, handlerNode.id // Handler's semantic ID for scope resolution
+                );
+                await graph.addEdge({
+                    type: 'RESPONDS_WITH',
+                    src: route.id,
+                    dst: dstNodeId,
+                    metadata: {
+                        responseMethod: call.method
+                    }
+                });
+                edgesCreated++;
+            }
+        }
+        catch {
+            // Silent - per-route errors shouldn't spam logs
+        }
+        return edgesCreated;
+    }
+    /**
+     * Find res.json/res.send calls within a function at given line
+     */
+    findResponseCalls(ast, file, handlerLine) {
+        const calls = [];
+        // Using object wrapper to satisfy TypeScript's control flow analysis
+        const found = { path: null };
+        // First pass: find the handler function at the specified line
+        traverse(ast, {
+            'ArrowFunctionExpression|FunctionExpression|FunctionDeclaration': (path) => {
+                const node = path.node;
+                const line = getLine(node);
+                if (line === handlerLine) {
+                    found.path = path;
+                    path.stop();
+                }
+            }
+        });
+        if (!found.path) {
+            return calls;
+        }
+        const handlerPath = found.path;
+        // Second pass: traverse only within the handler to find response calls
+        const handlerNode = handlerPath.node;
+        // Get parameter names to identify 'res'
+        const resParamName = this.getResponseParamName(handlerNode);
+        if (!resParamName) {
+            return calls;
+        }
+        // Traverse the handler body for res.json/res.send calls
+        handlerPath.traverse({
+            CallExpression: (callPath) => {
+                const callNode = callPath.node;
+                const callee = callNode.callee;
+                // Check for res.json() or res.send() or res.status().json() patterns
+                const responseInfo = this.extractResponseInfo(callee, resParamName);
+                if (!responseInfo) {
+                    return;
+                }
+                // Get the argument being sent
+                if (callNode.arguments.length === 0) {
+                    return;
+                }
+                const arg = callNode.arguments[0];
+                const argLine = getLine(arg);
+                const argColumn = arg.loc?.start.column ?? 0;
+                calls.push({
+                    method: responseInfo.method,
+                    argLine,
+                    argColumn,
+                    argType: arg.type,
+                    line: getLine(callNode),
+                    identifierName: arg.type === 'Identifier' ? arg.name : undefined
+                });
+            }
+        });
+        return calls;
+    }
+    /**
+     * Get response parameter name from function params (typically 'res')
+     */
+    getResponseParamName(func) {
+        const params = func.params;
+        // Express handlers: (req, res) or (req, res, next)
+        if (params.length >= 2) {
+            const resParam = params[1];
+            if (resParam.type === 'Identifier') {
+                return resParam.name;
+            }
+        }
+        return null;
+    }
+    /**
+     * Extract response method info from callee
+     * Handles: res.json(), res.send(), res.status(200).json()
+     */
+    extractResponseInfo(callee, resParamName) {
+        // Direct call: res.json() or res.send()
+        if (callee.type === 'MemberExpression') {
+            const memberExpr = callee;
+            const property = memberExpr.property;
+            if (property.type !== 'Identifier') {
+                return null;
+            }
+            const methodName = property.name;
+            // Check for res.json() or res.send()
+            if (memberExpr.object.type === 'Identifier' &&
+                memberExpr.object.name === resParamName &&
+                RESPONSE_METHODS.includes(methodName)) {
+                return { method: methodName };
+            }
+            // Check for res.status(200).json() chain
+            if (memberExpr.object.type === 'CallExpression' &&
+                RESPONSE_METHODS.includes(methodName)) {
+                const chainedCall = memberExpr.object;
+                if (this.isResMethodCall(chainedCall.callee, resParamName, 'status')) {
+                    return { method: methodName };
+                }
+            }
+        }
+        return null;
+    }
+    /**
+     * Check if callee is res.methodName()
+     */
+    isResMethodCall(callee, resParamName, methodName) {
+        if (callee.type !== 'MemberExpression') {
+            return false;
+        }
+        const memberExpr = callee;
+        return (memberExpr.object.type === 'Identifier' &&
+            memberExpr.object.name === resParamName &&
+            memberExpr.property.type === 'Identifier' &&
+            memberExpr.property.name === methodName);
+    }
+    /**
+     * Resolve response node: find existing variable or create stub.
+     *
+     * For Identifier arguments (e.g., res.json(statusData)):
+     * 1. Try to find existing VARIABLE/PARAMETER/CONSTANT with same name in handler scope
+     * 2. If found, return existing node ID (no stub needed)
+     * 3. If not found, fall back to creating stub (external/global variables)
+     *
+     * For non-Identifier arguments (ObjectExpression, CallExpression, etc.):
+     * - Always create stub node (existing behavior)
+     *
+     * @param graph - Graph backend
+     * @param file - Handler file path
+     * @param call - Response call info (includes identifierName)
+     * @param routeId - Route ID (for metadata)
+     * @param handlerSemanticId - Handler function's semantic ID (for scope matching)
+     * @returns Node ID (existing or newly created)
+     */
+    async resolveOrCreateResponseNode(graph, file, call, routeId, handlerSemanticId) {
+        const { argLine, argColumn, argType, identifierName } = call;
+        // For Identifier arguments, try to find existing variable/parameter
+        if (argType === 'Identifier' && identifierName) {
+            const existingNodeId = await this.findIdentifierInScope(graph, file, identifierName, handlerSemanticId, argLine);
+            if (existingNodeId) {
+                return existingNodeId; // Use existing node, no stub needed
+            }
+            // Fall through to create stub if not found (external/global variables)
+        }
+        // For non-Identifier or not-found, create stub node (existing logic)
+        return this.createResponseArgumentNode(graph, file, argLine, argColumn, argType, routeId);
+    }
+    /**
+     * Find existing VARIABLE/CONSTANT/PARAMETER node in handler scope.
+     *
+     * Strategy:
+     * 1. Parse handler semantic ID to extract scope prefix
+     * 2. Query VARIABLE/CONSTANT nodes: match by name, file, scope prefix, and line <= useLine
+     * 3. Query PARAMETER nodes: match by name, file, parentFunctionId === handlerSemanticId
+     *
+     * Scope matching:
+     * - Handler ID: "routes.js->anonymous[1]->FUNCTION->anonymous[1]"
+     * - Scope prefix: "routes.js->anonymous[1]->"
+     * - Variable ID: "routes.js->anonymous[1]->VARIABLE->statusData" (matches prefix)
+     * - External ID: "utils.js->VARIABLE->config" (different file)
+     *
+     * @param graph - Graph backend
+     * @param file - File path
+     * @param name - Variable name to find
+     * @param handlerSemanticId - Handler function's semantic ID
+     * @param useLine - Line where identifier is used (variable must be declared before this)
+     * @returns Node ID if found, null otherwise
+     */
+    async findIdentifierInScope(graph, file, name, handlerSemanticId, useLine) {
+        // Extract scope prefix from handler semantic ID
+        const handlerScopePrefix = this.extractScopePrefix(handlerSemanticId);
+        // Query VARIABLE nodes
+        for await (const node of graph.queryNodes({ type: 'VARIABLE' })) {
+            if (node.name === name && node.file === file) {
+                // Check if in handler scope and declared before usage
+                if (node.id.startsWith(handlerScopePrefix) && node.line <= useLine) {
+                    return node.id;
+                }
+            }
+        }
+        // Query CONSTANT nodes
+        for await (const node of graph.queryNodes({ type: 'CONSTANT' })) {
+            if (node.name === name && node.file === file) {
+                if (node.id.startsWith(handlerScopePrefix) && node.line <= useLine) {
+                    return node.id;
+                }
+            }
+        }
+        // Query PARAMETER nodes
+        for await (const node of graph.queryNodes({ type: 'PARAMETER' })) {
+            if (node.name === name && node.file === file) {
+                // Parameters belong to the function directly
+                const parentFunctionId = node.parentFunctionId;
+                if (parentFunctionId === handlerSemanticId) {
+                    return node.id;
+                }
+            }
+        }
+        // Also check module-level variables (scope prefix would be just "file.js->")
+        // For module-level constants, they should be accessible from any function in the file
+        const modulePrefix = this.extractModulePrefix(handlerSemanticId);
+        if (modulePrefix) {
+            // Check module-level VARIABLE
+            for await (const node of graph.queryNodes({ type: 'VARIABLE' })) {
+                if (node.name === name && node.file === file) {
+                    // Module-level variables have IDs like "file.js->VARIABLE->name" (3 parts)
+                    // Function-local variables have IDs like "file.js->funcName->VARIABLE->name" (4+ parts)
+                    // Only match true module-level variables by checking structure
+                    if (this.isModuleLevelId(node.id, modulePrefix) && node.line <= useLine) {
+                        return node.id;
+                    }
+                }
+            }
+            // Check module-level CONSTANT
+            for await (const node of graph.queryNodes({ type: 'CONSTANT' })) {
+                if (node.name === name && node.file === file) {
+                    if (this.isModuleLevelId(node.id, modulePrefix) && node.line <= useLine) {
+                        return node.id;
+                    }
+                }
+            }
+        }
+        return null; // Not found - will create stub
+    }
+    /**
+     * Extract scope prefix from handler function's semantic ID.
+     *
+     * Handler function semantic IDs follow the pattern:
+     *   {file}->{scope_path}->{type}->{name}
+     *
+     * Variables declared INSIDE the handler have IDs where the handler's NAME
+     * becomes part of THEIR scope path:
+     *   {file}->{handler_name}->{type}->{var_name}
+     *
+     * Examples:
+     * - Handler: "index.js->global->FUNCTION->anonymous[0]"
+     *   -> Variables inside: "index.js->anonymous[0]->CONSTANT->statusData"
+     *   -> Scope prefix: "index.js->anonymous[0]->"
+     *
+     * - Handler: "routes.js->anonymous[1]->FUNCTION->anonymous[1]"
+     *   -> Variables inside: "routes.js->anonymous[1]->VARIABLE->data"
+     *   -> Scope prefix: "routes.js->anonymous[1]->"
+     *
+     * - Handler: "app.js->global->FUNCTION->handleRequest"
+     *   -> Variables inside: "app.js->handleRequest->VARIABLE->result"
+     *   -> Scope prefix: "app.js->handleRequest->"
+     *
+     * Algorithm:
+     * 1. Split by "->"
+     * 2. Take file (first part) and handler name (last part)
+     * 3. Join with "->" and add trailing "->"
+     *
+     * @param semanticId - Handler function's semantic ID
+     * @returns Scope prefix for matching variables declared inside the handler
+     */
+    extractScopePrefix(semanticId) {
+        const parts = semanticId.split('->');
+        // Semantic ID format: file->scope->TYPE->name
+        // We need file + function name (last part) to match variables inside the function
+        if (parts.length >= 4) {
+            const file = parts[0];
+            const functionName = parts[parts.length - 1]; // Function name is the last part
+            return `${file}->${functionName}->`;
+        }
+        // Fallback: use first two parts (shouldn't happen for well-formed IDs)
+        if (parts.length >= 2) {
+            return `${parts[0]}->${parts[1]}->`;
+        }
+        return semanticId;
+    }
+    /**
+     * Extract module prefix from semantic ID (for module-level variable access).
+     *
+     * Examples:
+     * - "routes.js->anonymous[1]->FUNCTION->anonymous[1]" -> "routes.js->"
+     * - "app.js->startServer->FUNCTION->startServer" -> "app.js->"
+     *
+     * @param semanticId - Handler function's semantic ID
+     * @returns Module prefix for matching module-level variables
+     */
+    extractModulePrefix(semanticId) {
+        const parts = semanticId.split('->');
+        if (parts.length >= 1 && parts[0]) {
+            return `${parts[0]}->`;
+        }
+        return null;
+    }
+    /**
+     * Check if a semantic ID represents a true module-level variable.
+     *
+     * Semantic IDs have format: file->scope->TYPE->name
+     * - Module-level variables have "global" as the scope: "file.js->global->TYPE->name"
+     * - Function-local variables have function name as scope: "file.js->funcName->TYPE->name"
+     *
+     * Examples:
+     * - "index.js->global->CONSTANT->CONFIG" -> true (module-level)
+     * - "index.js->global->VARIABLE->counter" -> true (module-level)
+     * - "index.js->anonymous[0]->CONSTANT->data" -> false (function-local)
+     * - "routes.js->handler->VARIABLE->result" -> false (function-local)
+     *
+     * @param nodeId - The node's semantic ID
+     * @param modulePrefix - The module prefix (e.g., "index.js->")
+     * @returns true if this is a module-level variable
+     */
+    isModuleLevelId(nodeId, modulePrefix) {
+        if (!nodeId.startsWith(modulePrefix)) {
+            return false;
+        }
+        // Check if the scope part (second component) is "global"
+        const parts = nodeId.split('->');
+        // Expected format: ["file.js", "global", "TYPE", "name"]
+        // Check that second part is "global" (module scope)
+        return parts.length >= 4 && parts[1] === 'global';
+    }
+    /**
+     * Create a node for the response argument
+     */
+    async createResponseArgumentNode(graph, file, line, column, astType, routeId) {
+        // Map AST type to node type and create appropriate node
+        switch (astType) {
+            case 'ObjectExpression': {
+                // Include counter to make the node unique even for same location
+                const counter = this.responseNodeCounter++;
+                const id = `OBJECT_LITERAL#response:${counter}#${file}#${line}:${column}`;
+                await graph.addNode({
+                    id,
+                    type: 'OBJECT_LITERAL',
+                    name: '<response>',
+                    file,
+                    line,
+                    column,
+                    parentRouteId: routeId
+                });
+                return id;
+            }
+            case 'Identifier': {
+                // For identifiers, we link to the variable that's being returned
+                const counter = this.responseNodeCounter++;
+                const id = `VARIABLE#response:${counter}#${file}#${line}:${column}`;
+                await graph.addNode({
+                    id,
+                    type: 'VARIABLE',
+                    name: '<response>',
+                    file,
+                    line,
+                    column
+                });
+                return id;
+            }
+            case 'CallExpression': {
+                const counter = this.responseNodeCounter++;
+                const id = `CALL#response:${counter}#${file}#${line}:${column}`;
+                await graph.addNode({
+                    id,
+                    type: 'CALL',
+                    name: '<response>',
+                    file,
+                    line,
+                    column
+                });
+                return id;
+            }
+            case 'ArrayExpression': {
+                const counter = this.responseNodeCounter++;
+                const id = `ARRAY_LITERAL#response:${counter}#${file}#${line}:${column}`;
+                await graph.addNode({
+                    id,
+                    type: 'ARRAY_LITERAL',
+                    name: '<response>',
+                    file,
+                    line,
+                    column
+                });
+                return id;
+            }
+            default: {
+                // Generic expression node
+                const counter = this.responseNodeCounter++;
+                const id = `EXPRESSION#response:${counter}#${file}#${line}:${column}`;
+                await graph.addNode({
+                    id,
+                    type: 'EXPRESSION',
+                    name: '<response>',
+                    file,
+                    line,
+                    column
+                });
+                return id;
+            }
+        }
+    }
+}

package/dist/plugins/analysis/ExpressRouteAnalyzer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ExpressRouteAnalyzer.d.ts","sourceRoot":"","sources":["../../../src/plugins/analysis/ExpressRouteAnalyzer.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAOH,OAAO,EAAE,MAAM,EAA0C,MAAM,cAAc,CAAC;AAC9E,OAAO,KAAK,EAAE,aAAa,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AA+ChF,qBAAa,oBAAqB,SAAQ,MAAM;IAC9C,IAAI,QAAQ,IAAI,cAAc,CAW7B;IAEK,OAAO,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,YAAY,CAAC;YAmD9C,aAAa;CA8R5B"}
+{"version":3,"file":"ExpressRouteAnalyzer.d.ts","sourceRoot":"","sources":["../../../src/plugins/analysis/ExpressRouteAnalyzer.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAOH,OAAO,EAAE,MAAM,EAA0C,MAAM,cAAc,CAAC;AAC9E,OAAO,KAAK,EAAE,aAAa,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAkDhF,qBAAa,oBAAqB,SAAQ,MAAM;IAC9C,IAAI,QAAQ,IAAI,cAAc,CAW7B;IAEK,OAAO,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,YAAY,CAAC;YAmD9C,aAAa;CAkU5B"}

package/dist/plugins/analysis/ExpressRouteAnalyzer.js

@@ -11,7 +11,7 @@ import { readFileSync } from 'fs';
 import { parse } from '@babel/parser';
 import traverseModule from '@babel/traverse';
 import { Plugin, createSuccessResult, createErrorResult } from '../Plugin.js';
-import { getLine } from './ast/utils/location.js';
+import { getLine, getColumn } from './ast/utils/location.js';
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const traverse = traverseModule.default || traverseModule;
 const HTTP_METHODS = ['get', 'post', 'put', 'patch', 'delete', 'options', 'head'];
@@ -140,6 +140,32 @@ export class ExpressRouteAnalyzer extends Plugin {
                                     const handlers = args.slice(1);
                                     // Последний handler - это route handler
                                     const mainHandler = handlers[handlers.length - 1];
+                                    // Unwrap wrapper functions (asyncHandler, catchAsync, etc.)
+                                    // Pattern: wrapper(async (req, res) => {...}) -> extract inner function
+                                    // Also handles nested wrappers: outer(inner(handler))
+                                    let actualHandler = mainHandler;
+                                    while (actualHandler.type === 'CallExpression') {
+                                        const callExpr = actualHandler;
+                                        const firstArg = callExpr.arguments[0];
+                                        if (!firstArg) {
+                                            // No arguments - not a wrapper pattern
+                                            break;
+                                        }
+                                        if (firstArg.type === 'ArrowFunctionExpression' ||
+                                            firstArg.type === 'FunctionExpression') {
+                                            // Found the actual handler function
+                                            actualHandler = firstArg;
+                                            break;
+                                        }
+                                        else if (firstArg.type === 'CallExpression') {
+                                            // Nested wrapper: outer(inner(...)) - continue unwrapping
+                                            actualHandler = firstArg;
+                                        }
+                                        else {
+                                            // First arg is not a function or CallExpression - not a wrapper pattern
+                                            break;
+                                        }
+                                    }
                                     // Все предыдущие - middleware
                                     const middlewareHandlers = handlers.slice(0, -1);
                                     // Создаём http:route
@@ -151,10 +177,14 @@ export class ExpressRouteAnalyzer extends Plugin {
                                         path: routePath,
                                         file: module.file,
                                         line: getLine(node),
+                                        column: getColumn(node),
                                         routerName: objectName,
-                                        handlerLine: mainHandler.loc
-                                            ? getLine(mainHandler)
-                                            : getLine(node)
+                                        handlerLine: actualHandler.loc
+                                            ? getLine(actualHandler)
+                                            : getLine(node),
+                                        handlerColumn: actualHandler.loc
+                                            ? getColumn(actualHandler)
+                                            : getColumn(node)
                                     });
                                     // Обрабатываем middleware
                                     middlewareHandlers.forEach((mw, index) => {
@@ -181,6 +211,7 @@ export class ExpressRouteAnalyzer extends Plugin {
                                                 name: middlewareName,
                                                 file: module.file,
                                                 line: mwNode.loc ? getLine(mwNode) : getLine(node),
+                                                column: mwNode.loc ? getColumn(mwNode) : getColumn(node),
                                                 endpointId: endpointId,
                                                 order: index // Порядок в цепочке
                                             });
@@ -218,6 +249,7 @@ export class ExpressRouteAnalyzer extends Plugin {
                                             name: middlewareName,
                                             file: module.file,
                                             line: getLine(node),
+                                            column: getColumn(node),
                                             mountPath: mountPath,
                                             isGlobal: mountPath === '/' // Global middleware если нет path
                                         });
@@ -230,10 +262,11 @@ export class ExpressRouteAnalyzer extends Plugin {
             });
             // Создаём ENDPOINT ноды
             for (const endpoint of endpoints) {
-                // Сохраняем handlerLine ПЕРЕД destructuring
+                // Сохраняем handler location ПЕРЕД destructuring
                 const handlerLine = endpoint.handlerLine;
+                const handlerColumn = endpoint.handlerColumn;
                 // eslint-disable-next-line @typescript-eslint/no-unused-vars
-                const { handlerLine: _, routerName, ...endpointData } = endpoint;
+                const { handlerLine: _hl, handlerColumn: _hc, routerName, ...endpointData } = endpoint;
                 await graph.addNode(endpointData);
                 endpointsCreated++;
                 // MODULE -> CONTAINS -> ENDPOINT
@@ -243,22 +276,24 @@ export class ExpressRouteAnalyzer extends Plugin {
                     dst: endpoint.id
                 });
                 edgesCreated++;
-                // Ищем FUNCTION ноду для handler (arrow function на той же строке)
+                // Ищем FUNCTION ноду для handler по line+column
+                // NOTE: queryNodes не поддерживает line/column фильтрацию, поэтому фильтруем вручную
                 if (handlerLine) {
-                    // Используем queryNodes вместо прямого доступа к graph.nodes
                     for await (const fn of graph.queryNodes({
                         type: 'FUNCTION',
-                        file: module.file,
-                        line: handlerLine
+                        file: module.file
                     })) {
-                        // ENDPOINT -> HANDLED_BY -> FUNCTION
-                        await graph.addEdge({
-                            type: 'HANDLED_BY',
-                            src: endpoint.id,
-                            dst: fn.id
-                        });
-                        edgesCreated++;
-                        break; // Берём только первую найденную функцию
+                        // Проверяем точное совпадение line и column
+                        if (fn.line === handlerLine && fn.column === handlerColumn) {
+                            // ENDPOINT -> HANDLED_BY -> FUNCTION
+                            await graph.addEdge({
+                                type: 'HANDLED_BY',
+                                src: endpoint.id,
+                                dst: fn.id
+                            });
+                            edgesCreated++;
+                            break;
+                        }
                     }
                 }
             }