api-tests-coverage 1.0.20 → 1.0.22

package/dist/src/inference/businessRuleInference.js

@@ -138,104 +138,12 @@ const INFERENCE_PATTERNS = [
         behaviorTemplate: (m) => `${m[1]} must not exceed ${m[2]}`,
         conditionTemplate: (m) => `${m[1]} > ${m[2]}`,
     },
-    // ── Flask / Python-specific patterns ──────────────────────────────────────
-    // Broader Python raise: any raise X() or raise X.y() not already ending in Exception/Error
-    // (Ordered after the existing raise patterns so deduplification by source location handles overlap)
-    {
-        type: 'business_logic',
-        pattern: /\braise\s+([\w]+(?:\.[\w]+)*)\s*(?:\(([^)]{0,60})\))?/,
-        nameTemplate: (m) => { var _a; return toSnakeCase((_a = m[1].split('.').pop()) !== null && _a !== void 0 ? _a : m[1]); },
-        behaviorTemplate: (m) => `Operation rejected: ${m[1]}`,
-        conditionTemplate: (m) => { var _a; return `raise ${m[1]}(${((_a = m[2]) !== null && _a !== void 0 ? _a : '').trim()})`; },
-    },
-    // Python `if not X:` null guard — idiomatic Python nil check
-    {
-        type: 'validation',
-        pattern: /if\s+not\s+([\w.]+)\s*(?::|and|or)/,
-        nameTemplate: (m) => `require_${toSnakeCase(m[1].replace(/\./g, '_'))}`,
-        behaviorTemplate: (m) => `${m[1]} must exist`,
-        conditionTemplate: (m) => `if not ${m[1]}`,
-    },
-    // Flask/Django auth decorators: @jwt_required, @login_required, etc.
-    {
-        type: 'authorization',
-        pattern: /@(jwt_required|login_required|require_auth|requires_auth|permission_required|auth_required|authenticated_user|require_permissions?)\b/i,
-        nameTemplate: (m) => `require_${toSnakeCase(m[1])}`,
-        behaviorTemplate: (m) => `Request requires authentication (${m[1]})`,
-        conditionTemplate: (m) => `@${m[1]}`,
-    },
-    // Flask `@use_kwargs` / `@validate_arguments` input validation
-    {
-        type: 'validation',
-        pattern: /@(use_kwargs|validate_arguments?|expects_json|validate_body)\s*\(\s*([\w]+)/i,
-        nameTemplate: (m) => { var _a; return `validate_input_${toSnakeCase((_a = m[2]) !== null && _a !== void 0 ? _a : 'schema')}`; },
-        behaviorTemplate: (m) => { var _a; return `Input validated against ${(_a = m[2]) !== null && _a !== void 0 ? _a : 'schema'}`; },
-        conditionTemplate: (m) => { var _a; return `@${m[1]}(${(_a = m[2]) !== null && _a !== void 0 ? _a : ''})`; },
-    },
-    // Python ownership / attribute comparison guard
-    {
-        type: 'authorization',
-        pattern: /if\s+[\w.]+\s*!=\s*[\w.]+\.(?:id|user_id|author_id|owner_id|profile\.id)/,
-        nameTemplate: () => 'ownership_check',
-        behaviorTemplate: () => 'Resource must belong to the requesting user',
-        conditionTemplate: (m) => m[0].trim(),
-    },
-    // ── Generic JS/TS/frontend patterns ───────────────────────────────────────
-    // Generic JS/TS `if (!expression)` null/falsy guard
-    // Matches meaningful presence checks like !JWT.get(), !User.current, !token
-    // Skips pure normalisation calls like .trim(), .length, .toLowerCase(), etc.
-    // via the skipIfGroup1Matches post-match filter.
-    {
-        type: 'validation',
-        pattern: /if\s*\(\s*!\s*([\w][\w.[\]]*(?:\.\w+\(\s*\))?)\s*\)/,
-        skipIfGroup1Matches: /\.(trim|length|split|join|toLowerCase|toUpperCase|toString|valueOf|slice|substr|substring|replace|indexOf|includes|startsWith|endsWith)\(\s*\)$/,
-        nameTemplate: (m) => `require_${toSnakeCase(m[1].replace(/[^a-zA-Z0-9]/g, '_').replace(/_+/g, '_').replace(/^_|_$/g, ''))}`,
-        behaviorTemplate: (m) => `${m[1]} must be present/truthy`,
-        conditionTemplate: (m) => `!${m[1]}`,
-    },
-    // HTTP status comparison guard (frontend interceptors): rejection.status === 401
-    {
-        type: 'validation',
-        pattern: /\.status\s*={1,3}\s*(4\d\d|5\d\d)/,
-        nameTemplate: (m) => `http_status_${m[1]}_check`,
-        behaviorTemplate: (m) => `Handle HTTP ${m[1]} error response`,
-        conditionTemplate: (m) => `status == ${m[1]}`,
-    },
-    // Ownership / identity comparison guard (username === author.username)
-    {
-        type: 'authorization',
-        pattern: /if\s*\(?\s*[\w.]+\.(?:username|userId|user_id|email|id)\s*!==?\s*[\w.]+\.(?:username|userId|user_id|email|id|author\.username)/,
-        nameTemplate: () => 'ownership_identity_check',
-        behaviorTemplate: () => 'Resource belongs to a specific user identity',
-        conditionTemplate: (m) => m[0].trim().replace(/^if\s*\(?/, '').replace(/\)?\s*$/, ''),
-    },
-    // Generic early-return guard: if (cond) return/throw on same or adjacent line
-    {
-        type: 'business_logic',
-        pattern: /if\s*\([^)]{3,80}\)\s*(?:\{[^}]{0,40}\})?\s*(?:return|throw|raise)\b/,
-        nameTemplate: () => 'guard_condition',
-        behaviorTemplate: () => 'Early return or error on condition',
-        conditionTemplate: (m) => {
-            const cond = m[0].match(/if\s*\(([^)]{3,80})\)/);
-            return cond ? cond[1].trim() : m[0].trim();
-        },
-    },
 ];
 // ─── Endpoint heuristics ──────────────────────────────────────────────────────
 /** Attempt to associate a rule with the nearest HTTP route/endpoint annotation. */
 function guessEndpoint(lines, ruleLineIdx) {
     // Search up to 40 lines above for common routing patterns
     const lookupLines = lines.slice(Math.max(0, ruleLineIdx - 40), ruleLineIdx);
-    // Flask: @blueprint.route('/path', methods=...) / @app.route('/path', ...)
-    for (let i = lookupLines.length - 1; i >= 0; i--) {
-        const fm = lookupLines[i].match(/@\w+\.route\s*\(\s*['"]([^'"]+)['"]/);
-        if (fm) {
-            // Try to find method from same line or near
-            const methodsMatch = lookupLines[i].match(/methods\s*=\s*[\[(]['"]?([\w]+)['"]?/);
-            const method = methodsMatch ? methodsMatch[1].toUpperCase() : 'GET';
-            return `${method} ${fm[1]}`;
-        }
-    }
     // Spring: @GetMapping("/path") / @PostMapping / @RequestMapping
     for (let i = lookupLines.length - 1; i >= 0; i--) {
         const m = lookupLines[i].match(/@(Get|Post|Put|Patch|Delete|Request)Mapping\s*\(\s*["']([^"']+)["']/i);
@@ -283,9 +191,6 @@ function inferRulesFromFile(filePath) {
             const match = line.match(ip.pattern);
             if (!match)
                 continue;
-            // Post-match filter: skip false positives based on first captured group
-            if (ip.skipIfGroup1Matches && match[1] && ip.skipIfGroup1Matches.test(match[1]))
-                continue;
             const condition = ip.conditionTemplate(match);
             const sourceLocation = `${filePath}:${lineIdx + 1}`;
             const dedupKey = `${filePath}:${lineIdx}:${condition}`;

package/dist/src/inference/integrationFlowInference.d.ts

@@ -44,20 +44,10 @@ export interface IntegrationFlowInferenceResult {
  * Infer integration flows from a single test file.
  */
 export declare function inferFlowsFromFile(filePath: string): InferredIntegrationFlow[];
-/**
- * Infer integration flows from service source files (not test files).
- *
- * Used as a fallback when no test files are present (e.g. frontend-only projects).
- * Only produces flows for source files that contain 2+ HTTP calls in the same function.
- */
-export declare function inferFlowsFromSourceFiles(serviceFiles: string[]): InferredIntegrationFlow[];
 /**
  * Run flow inference across all provided test files.
- *
- * When `testFiles` is empty and `serviceFiles` is provided, flows are inferred
- * from the service source code instead and tagged accordingly.
  */
-export declare function inferIntegrationFlows(testFiles: string[], warnings?: string[], serviceFiles?: string[]): IntegrationFlowInferenceResult;
+export declare function inferIntegrationFlows(testFiles: string[], warnings?: string[]): IntegrationFlowInferenceResult;
 /**
  * Write inferred integration flows to the reports directory.
  * Returns the path of the written file.

package/dist/src/inference/integrationFlowInference.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"integrationFlowInference.d.ts","sourceRoot":"","sources":["../../../src/inference/integrationFlowInference.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAOH,MAAM,MAAM,UAAU,GAAG,UAAU,GAAG,UAAU,CAAC;AAEjD,MAAM,WAAW,QAAQ;IACvB,kBAAkB;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,yCAAyC;IACzC,IAAI,EAAE,MAAM,CAAC;IACb,uBAAuB;IACvB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,MAAM,WAAW,uBAAuB;IACtC,wBAAwB;IACxB,EAAE,EAAE,MAAM,CAAC;IACX,+BAA+B;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,UAAU,CAAC;IACxB,+BAA+B;IAC/B,KAAK,EAAE,QAAQ,EAAE,CAAC;IAClB,iDAAiD;IACjD,eAAe,EAAE,MAAM,CAAC;IACxB,oDAAoD;IACpD,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,8BAA8B;IAC7C,KAAK,EAAE,uBAAuB,EAAE,CAAC;IACjC,aAAa,EAAE,MAAM,CAAC;IACtB,QAAQ,EAAE,OAAO,CAAC;IAClB,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAqLD;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,GAAG,uBAAuB,EAAE,CAW9E;AAED;;;;;GAKG;AACH,wBAAgB,yBAAyB,CAAC,YAAY,EAAE,MAAM,EAAE,GAAG,uBAAuB,EAAE,CAgB3F;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CACnC,SAAS,EAAE,MAAM,EAAE,EACnB,QAAQ,GAAE,MAAM,EAAO,EACvB,YAAY,CAAC,EAAE,MAAM,EAAE,GACtB,8BAA8B,CAoChC;AAED;;;GAGG;AACH,wBAAgB,6BAA6B,CAC3C,MAAM,EAAE,8BAA8B,EACtC,UAAU,EAAE,MAAM,GACjB,MAAM,CAYR"}
+{"version":3,"file":"integrationFlowInference.d.ts","sourceRoot":"","sources":["../../../src/inference/integrationFlowInference.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAOH,MAAM,MAAM,UAAU,GAAG,UAAU,GAAG,UAAU,CAAC;AAEjD,MAAM,WAAW,QAAQ;IACvB,kBAAkB;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,yCAAyC;IACzC,IAAI,EAAE,MAAM,CAAC;IACb,uBAAuB;IACvB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,MAAM,WAAW,uBAAuB;IACtC,wBAAwB;IACxB,EAAE,EAAE,MAAM,CAAC;IACX,+BAA+B;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,UAAU,CAAC;IACxB,+BAA+B;IAC/B,KAAK,EAAE,QAAQ,EAAE,CAAC;IAClB,iDAAiD;IACjD,eAAe,EAAE,MAAM,CAAC;IACxB,oDAAoD;IACpD,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,8BAA8B;IAC7C,KAAK,EAAE,uBAAuB,EAAE,CAAC;IACjC,aAAa,EAAE,MAAM,CAAC;IACtB,QAAQ,EAAE,OAAO,CAAC;IAClB,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AA4KD;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,GAAG,uBAAuB,EAAE,CAW9E;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CACnC,SAAS,EAAE,MAAM,EAAE,EACnB,QAAQ,GAAE,MAAM,EAAO,GACtB,8BAA8B,CA2BhC;AAED;;;GAGG;AACH,wBAAgB,6BAA6B,CAC3C,MAAM,EAAE,8BAA8B,EACtC,UAAU,EAAE,MAAM,GACjB,MAAM,CAYR"}

package/dist/src/inference/integrationFlowInference.js

@@ -48,7 +48,6 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.inferFlowsFromFile = inferFlowsFromFile;
-exports.inferFlowsFromSourceFiles = inferFlowsFromSourceFiles;
 exports.inferIntegrationFlows = inferIntegrationFlows;
 exports.writeInferredIntegrationFlows = writeInferredIntegrationFlows;
 const fs = __importStar(require("fs"));
@@ -64,11 +63,6 @@ const HTTP_CALL_PATTERNS = [
     { pattern: /\b(get|post|put|patch|delete)\s+['"`]([^'"`\s]+)['"`]/i, methodGroup: 1, pathGroup: 2 },
     // fetch('/path', { method: 'POST' })
     { pattern: /fetch\s*\(\s*['"`]([^'"`]+)['"`]\s*,\s*\{[^}]*method\s*:\s*['"`](GET|POST|PUT|PATCH|DELETE|HEAD)['"`]/i, methodGroup: 2, pathGroup: 1 },
-    // WebTest TestApp: testapp.post_json(url_for('endpoint'), data)
-    // Also handles: testapp.get(url_for('endpoint')), testapp.delete_json(...)
-    { pattern: /(?:testapp|self\.testapp)\.(get_json|post_json|put_json|patch_json|delete_json|get|post|put|patch|delete)\s*\(\s*url_for\s*\(\s*['"]([^'"]+)['"]/i, methodGroup: 1, pathGroup: 2 },
-    // WebTest without url_for: testapp.get('/path')
-    { pattern: /(?:testapp|self\.testapp)\.(get_json|post_json|put_json|patch_json|delete_json|get|post|put|patch|delete)\s*\(\s*['"`]([^'"`]+)['"`]/i, methodGroup: 1, pathGroup: 2 },
 ];
 /** Patterns for test function / scenario boundaries */
 const TEST_FUNCTION_PATTERNS = [
@@ -91,8 +85,6 @@ function extractHttpCallsFromLines(lines) {
             const m = line.match(p.pattern);
             if (m) {
                 const method = m[p.methodGroup].toUpperCase();
-                // Normalize WebTest _json suffix: POST_JSON → POST, GET_JSON → GET
-                const normalizedMethod = method.replace(/_JSON$/, '');
                 const rawPath = m[p.pathGroup];
                 // Skip unlikely paths (full URLs with domain, non-path values)
                 if (rawPath.startsWith('http') && !rawPath.includes('/api'))
@@ -100,7 +92,7 @@ function extractHttpCallsFromLines(lines) {
                 const cleanPath = extractPathFromUrl(rawPath);
                 if (!cleanPath)
                     continue;
-                calls.push({ method: normalizedMethod, path: cleanPath, lineIdx: i });
+                calls.push({ method, path: cleanPath, lineIdx: i });
                 break; // only first pattern match per line
             }
         }
@@ -110,9 +102,6 @@ function extractHttpCallsFromLines(lines) {
 function extractPathFromUrl(raw) {
     if (raw.startsWith('/'))
         return raw;
-    // url_for endpoint name: 'blueprint.function' — treat as pseudo-path
-    if (/^\w+\.\w+$/.test(raw))
-        return `/${raw.replace('.', '/')}`;
     try {
         const u = new URL(raw);
         return u.pathname || undefined;
@@ -214,46 +203,10 @@ function inferFlowsFromFile(filePath) {
     const calls = extractHttpCallsFromLines(lines);
     return groupCallsIntoFlows(calls, filePath, lines);
 }
-/**
- * Infer integration flows from service source files (not test files).
- *
- * Used as a fallback when no test files are present (e.g. frontend-only projects).
- * Only produces flows for source files that contain 2+ HTTP calls in the same function.
- */
-function inferFlowsFromSourceFiles(serviceFiles) {
-    const allFlows = [];
-    for (const fp of serviceFiles) {
-        let content;
-        try {
-            content = fs.readFileSync(fp, 'utf-8');
-        }
-        catch {
-            continue;
-        }
-        const lines = content.split('\n');
-        const calls = extractHttpCallsFromLines(lines);
-        if (calls.length < 2)
-            continue;
-        const flows = groupCallsIntoFlows(calls, fp, lines);
-        allFlows.push(...flows);
-    }
-    return allFlows;
-}
 /**
  * Run flow inference across all provided test files.
- *
- * When `testFiles` is empty and `serviceFiles` is provided, flows are inferred
- * from the service source code instead and tagged accordingly.
  */
-function inferIntegrationFlows(testFiles, warnings = [], serviceFiles) {
-    if (testFiles.length === 0 && serviceFiles && serviceFiles.length > 0) {
-        warnings.push('No test files found; integration flows inferred from service source code.');
-        const flows = inferFlowsFromSourceFiles(serviceFiles);
-        if (flows.length === 0) {
-            warnings.push('No multi-step HTTP call sequences detected in service source files; integration flow inference produced no results.');
-        }
-        return { flows, filesAnalyzed: serviceFiles.length, inferred: true, warnings };
-    }
+function inferIntegrationFlows(testFiles, warnings = []) {
     if (testFiles.length === 0) {
         warnings.push('No test files provided; integration flow inference skipped.');
         return { flows: [], filesAnalyzed: 0, inferred: true, warnings };

package/dist/src/inference/routeInference.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"routeInference.d.ts","sourceRoot":"","sources":["../../../src/inference/routeInference.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAOH,eAAO,MAAM,YAAY,uEAAwE,CAAC;AAClG,MAAM,MAAM,UAAU,GAAG,OAAO,YAAY,CAAC,MAAM,CAAC,CAAC;AAErD,MAAM,WAAW,aAAa;IAC5B,MAAM,EAAE,UAAU,CAAC;IACnB,IAAI,EAAE,MAAM,CAAC;IACb,0EAA0E;IAC1E,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,oCAAoC;IACpC,aAAa,EAAE,MAAM,GAAG,OAAO,CAAC;CACjC;AAED,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,aAAa,EAAE,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAmGD;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,MAAM,GAAG,aAAa,EAAE,CAoJrE;AAkXD;;GAEG;AACH,wBAAgB,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,GAAG,oBAAoB,CAgBxE;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,oBAAoB,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAY5F"}
+{"version":3,"file":"routeInference.d.ts","sourceRoot":"","sources":["../../../src/inference/routeInference.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAOH,eAAO,MAAM,YAAY,uEAAwE,CAAC;AAClG,MAAM,MAAM,UAAU,GAAG,OAAO,YAAY,CAAC,MAAM,CAAC,CAAC;AAErD,MAAM,WAAW,aAAa;IAC5B,MAAM,EAAE,UAAU,CAAC;IACnB,IAAI,EAAE,MAAM,CAAC;IACb,0EAA0E;IAC1E,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,oCAAoC;IACpC,aAAa,EAAE,MAAM,GAAG,OAAO,CAAC;CACjC;AAED,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,aAAa,EAAE,CAAC;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAqFD;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,MAAM,GAAG,aAAa,EAAE,CAkIrE;AAiWD;;GAEG;AACH,wBAAgB,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,GAAG,oBAAoB,CAgBxE;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,oBAAoB,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAY5F"}

package/dist/src/inference/routeInference.js

@@ -80,18 +80,7 @@ const JSDOC_ROUTE_PATTERN = /@route\s+\{(GET|POST|PUT|PATCH|DELETE|HEAD|OPTIONS)
  * Used to gate the URL+method object-form scan so we only activate it in
  * files that actually make HTTP calls (avoids scanning every JS file).
  */
-const HTTP_CLIENT_SIGNAL = /\$http|\bfetch\s*\(|\baxios\b|\bsuperagent\b/;
-/**
- * Matches client-side HTTP method calls: requests.get('/path'), api.post('/path'), etc.
- * Group 1 = HTTP method, Group 2 = path literal starting with /
- * Also handles: superagent.get(`${base}/path`) → captures the static suffix
- */
-const HTTP_CLIENT_METHOD_CALL = /\b\w+\s*\.\s*(get|post|put|patch|delete|del|head|options)\s*\(\s*(?:[^'"`\n]*?\+\s*)?['"`](\/[^'"`?\n]+)/i;
-/**
- * Signals that this JS/TS file is a client-side HTTP service layer.
- * Expands on HTTP_CLIENT_SIGNAL to include superagent and common wrapper patterns.
- */
-const HTTP_SERVICE_SIGNAL = /\bsuperagent\b|\brequests\s*\.\s*(?:get|post|put|delete|del)\b|\bapi\s*\.\s*(?:get|post|put|delete)\b/;
+const HTTP_CLIENT_SIGNAL = /\$http|\bfetch\s*\(|\baxios\b/;
 /**
  * Matches a `url:` property that ends with a literal path segment starting
  * with `/`.  Handles both bare and concatenated forms:
@@ -173,8 +162,6 @@ function inferRoutesFromFile(filePath) {
     // We only enable the object-form method+url scanner for such files to
     // avoid false positives in ordinary JS/TS source.
     const usesHttpClient = HTTP_CLIENT_SIGNAL.test(content);
-    // Pre-check: does this file use a client-side HTTP service wrapper (requests.*, api.*, etc.)?
-    const usesHttpServiceWrapper = HTTP_SERVICE_SIGNAL.test(content);
     for (let lineIdx = 0; lineIdx < lines.length; lineIdx++) {
         const line = lines[lineIdx];
         // 1a. router.method('/path', ...) — all on one line
@@ -252,21 +239,6 @@ function inferRoutesFromFile(filePath) {
                 }
             }
         }
-        // 6. Client-side HTTP method calls: requests.get('/path'), api.post('/path')
-        //    Only activate for files that contain HTTP service signals to avoid false positives
-        if (usesHttpServiceWrapper) {
-            const clientCallMatch = line.match(HTTP_CLIENT_METHOD_CALL);
-            if (clientCallMatch) {
-                let httpMethod = clientCallMatch[1].toLowerCase();
-                // Normalize 'del' → 'delete'
-                if (httpMethod === 'del')
-                    httpMethod = 'delete';
-                if (exports.HTTP_METHODS.includes(httpMethod)) {
-                    const routePath = clientCallMatch[2].split('?')[0]; // strip query string
-                    addRoute(httpMethod, routePath, lineIdx + 1, 'code');
-                }
-            }
-        }
     }
     return [...byKey.values()];
 }
@@ -428,7 +400,7 @@ function findNextMethodNameInJava(javaLines, fromLine) {
  *   @blueprint.route('/articles/<slug>', methods=['PUT'])
  *   @app.route('/tags')   ← defaults to GET
  */
-const FLASK_ROUTE = /@(\w+)\.route\s*\(\s*['"]([^'"]+)['"](?:[^\n]*?\bmethods\s*=\s*[\[(]([^\])\n]+)[\])])?/;
+const FLASK_ROUTE = /@(\w+)\.route\s*\(\s*['"]([^'"]+)['"](?:\s*,\s*methods\s*=\s*\[([^\]]+)\])?\s*\)/;
 /**
  * Matches FastAPI decorators.
  *   @app.get('/articles')
@@ -447,22 +419,6 @@ const FLASK_BLUEPRINT_CTOR = /(\w+)\s*=\s*Blueprint\s*\(\s*['"][^'"]+['"](?:\s*,
  *   router = APIRouter(prefix="/articles")
  */
 const FASTAPI_APIROUTER = /(\w+)\s*=\s*APIRouter\s*\([^)]*?prefix\s*=\s*['"]([^'"]+)['"]/;
-/**
- * Scan ahead for a Python `def function_name(` line following a decorator.
- * Skips additional decorator lines and stops at the first non-decorator,
- * non-blank, non-comment line that is not a def.
- */
-function findPythonHandlerFunction(lines, fromLine) {
-    for (let j = fromLine + 1; j < Math.min(fromLine + 6, lines.length); j++) {
-        const m = lines[j].match(/^def\s+(\w+)\s*\(/);
-        if (m)
-            return m[1];
-        // Skip decorator lines
-        if (!lines[j].trim().startsWith('@') && lines[j].trim().length > 0 && !lines[j].trim().startsWith('#'))
-            break;
-    }
-    return undefined;
-}
 /**
  * Infer routes from a Python (Flask/FastAPI) source file.
  */
@@ -476,10 +432,10 @@ function inferRoutesFromPythonFile(filePath) {
     }
     const lines = content.split('\n');
     const byKey = new Map();
-    const addRoute = (method, routePath, lineNumber, handlerFunction) => {
+    const addRoute = (method, routePath, lineNumber) => {
         const key = `${method}:${routePath}`;
         if (!byKey.has(key)) {
-            byKey.set(key, { method, path: routePath, handlerFunction, sourceFile: filePath, lineNumber, discoveredVia: 'code' });
+            byKey.set(key, { method, path: routePath, sourceFile: filePath, lineNumber, discoveredVia: 'code' });
         }
     };
     // Pass 1: Detect blueprint/router prefix for the file
@@ -507,19 +463,18 @@ function inferRoutesFromPythonFile(filePath) {
             const fullPath = localPrefix + (routePath.startsWith('/') ? routePath : '/' + routePath);
             // Normalize Flask <param> to {param}
             const normalizedPath = fullPath.replace(/<(?:\w+:)?(\w+)>/g, '{$1}');
-            const handlerFn = findPythonHandlerFunction(lines, i);
             if (methodsList) {
-                // Parse methods=['GET', 'POST'] or methods=('GET', 'POST') → individual routes
+                // Parse methods=['GET', 'POST'] → individual routes
                 const methods = methodsList.replace(/['"]/g, '').split(',').map((m) => m.trim().toLowerCase());
                 for (const m of methods) {
                     if (exports.HTTP_METHODS.includes(m)) {
-                        addRoute(m, normalizedPath, i + 1, handlerFn);
+                        addRoute(m, normalizedPath, i + 1);
                     }
                 }
             }
             else {
                 // Default to GET
-                addRoute('get', normalizedPath, i + 1, handlerFn);
+                addRoute('get', normalizedPath, i + 1);
             }
             continue;
         }
@@ -529,9 +484,8 @@ function inferRoutesFromPythonFile(filePath) {
             const httpMethod = fastapiMatch[2].toLowerCase();
             const routePath = fastapiMatch[3];
             const fullPath = localPrefix + (routePath.startsWith('/') ? routePath : '/' + routePath);
-            const handlerFn = findPythonHandlerFunction(lines, i);
             // FastAPI uses {param} natively
-            addRoute(httpMethod, fullPath, i + 1, handlerFn);
+            addRoute(httpMethod, fullPath, i + 1);
             continue;
         }
     }

package/dist/src/languages/java/semanticBuilder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"semanticBuilder.d.ts","sourceRoot":"","sources":["../../../../src/languages/java/semanticBuilder.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAEV,cAAc,EACd,gBAAgB,EAChB,gBAAgB,EAChB,iBAAiB,EACjB,eAAe,EACf,OAAO,EACR,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAKL,KAAK,MAAM,EACZ,MAAM,2BAA2B,CAAC;AAOnC,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,MAAM,GACX;IAAE,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC;IAAC,KAAK,EAAE,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAA;CAAE,CAmErF;AAID,wBAAgB,oBAAoB,CAClC,IAAI,EAAE,MAAM,EACZ,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,GACrC,GAAG,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAwC/B;AAID,wBAAgB,oBAAoB,CAClC,KAAK,EAAE,MAAM,EACb,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,EACtC,GAAG,EAAE,gBAAgB,EAAE,GACtB,IAAI,CA+EN;AAID,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,MAAM,GAAG,iBAAiB,EAAE,CAyBvE;AAID,wBAAgB,2BAA2B,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,EAAE,CAa3E;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,EAAE,CAa3D"}
+{"version":3,"file":"semanticBuilder.d.ts","sourceRoot":"","sources":["../../../../src/languages/java/semanticBuilder.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAEV,cAAc,EACd,gBAAgB,EAChB,gBAAgB,EAChB,iBAAiB,EACjB,eAAe,EACf,OAAO,EACR,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAKL,KAAK,MAAM,EACZ,MAAM,2BAA2B,CAAC;AAOnC,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,MAAM,GACX;IAAE,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,CAAC;IAAC,KAAK,EAAE,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAA;CAAE,CAmErF;AAID,wBAAgB,oBAAoB,CAClC,IAAI,EAAE,MAAM,EACZ,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,GACrC,GAAG,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAqC/B;AAID,wBAAgB,oBAAoB,CAClC,KAAK,EAAE,MAAM,EACb,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,EACtC,GAAG,EAAE,gBAAgB,EAAE,GACtB,IAAI,CA6DN;AAID,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,MAAM,GAAG,iBAAiB,EAAE,CAyBvE;AAID,wBAAgB,2BAA2B,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,EAAE,CAa3E;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,EAAE,CAa3D"}

package/dist/src/languages/java/semanticBuilder.js

@@ -97,8 +97,6 @@ function extractJavaFunctions(root, constants) {
             continue;
         const annotations = extractAnnotationNames(method);
         const block = (_e = (_d = method.childForFieldName) === null || _d === void 0 ? void 0 : _d.call(method, 'body')) !== null && _e !== void 0 ? _e : (0, treeSitterUtils_1.firstChildOfType)(method, 'block');
-        // Extract method parameters
-        const parameters = extractMethodParameters(method);
         const bodyHttpCalls = [];
         const calledFunctions = [];
         let returnValue;
@@ -111,7 +109,7 @@ function extractJavaFunctions(root, constants) {
         const cucumberPattern = extractCucumberPattern(annotations, method);
         graph.set(name, {
             name,
-            parameters,
+            parameters: [],
             bodyHttpCalls,
             calledFunctions,
             returnValue,
@@ -123,44 +121,14 @@ function extractJavaFunctions(root, constants) {
 }
 // ─── HTTP call extraction ─────────────────────────────────────────────────────
 function extractJavaHttpCalls(block, constants, out) {
-    var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s;
+    var _a, _b, _c, _d, _e, _f, _g;
     const methodInvocations = (0, treeSitterUtils_1.findNodes)(block, ['method_invocation']);
     for (const invoc of methodInvocations) {
         const methodName = (_e = (_c = (_b = (_a = invoc.childForFieldName) === null || _a === void 0 ? void 0 : _a.call(invoc, 'name')) === null || _b === void 0 ? void 0 : _b.text) !== null && _c !== void 0 ? _c : (_d = firstChildNamed(invoc, 'identifier')) === null || _d === void 0 ? void 0 : _d.text) !== null && _e !== void 0 ? _e : '';
         const lowerMethod = methodName.toLowerCase();
-        // MockMvc: perform(get("/path")) — extract the inner HTTP method call
-        if (lowerMethod === 'perform') {
-            const argList = (_g = (_f = invoc.childForFieldName) === null || _f === void 0 ? void 0 : _f.call(invoc, 'arguments')) !== null && _g !== void 0 ? _g : (0, treeSitterUtils_1.firstChildOfType)(invoc, 'argument_list');
-            if (!argList)
-                continue;
-            // The argument to perform() is a method invocation like get("/path") or post("/path")
-            const innerInvocations = (0, treeSitterUtils_1.findNodes)(argList, ['method_invocation']);
-            for (const innerInvoc of innerInvocations) {
-                const innerMethodName = (_m = (_k = (_j = (_h = innerInvoc.childForFieldName) === null || _h === void 0 ? void 0 : _h.call(innerInvoc, 'name')) === null || _j === void 0 ? void 0 : _j.text) !== null && _k !== void 0 ? _k : (_l = firstChildNamed(innerInvoc, 'identifier')) === null || _l === void 0 ? void 0 : _l.text) !== null && _m !== void 0 ? _m : '';
-                const innerLower = innerMethodName.toLowerCase();
-                if (!HTTP_METHODS.has(innerLower))
-                    continue;
-                const innerArgList = (_p = (_o = innerInvoc.childForFieldName) === null || _o === void 0 ? void 0 : _o.call(innerInvoc, 'arguments')) !== null && _p !== void 0 ? _p : (0, treeSitterUtils_1.firstChildOfType)(innerInvoc, 'argument_list');
-                if (!innerArgList)
-                    continue;
-                const innerStringArg = findFirstStringInArgList(innerArgList, constants);
-                if (!innerStringArg)
-                    continue;
-                const normalizedPath = innerStringArg.value.startsWith('/') ? (0, resolvePaths_1.normalizePathToTemplate)(innerStringArg.value) : undefined;
-                out.push({
-                    method: innerLower.toUpperCase(),
-                    rawPathArg: (_q = innerStringArg.varName) !== null && _q !== void 0 ? _q : innerStringArg.value,
-                    resolvedPath: innerStringArg.value,
-                    normalizedPath,
-                    resolutionType: innerStringArg.isDirect ? 'direct' : 'constant',
-                    confidence: innerStringArg.isDirect ? 'high' : 'medium',
-                });
-            }
-            continue;
-        }
         if (!HTTP_METHODS.has(lowerMethod))
             continue;
-        const argList = (_s = (_r = invoc.childForFieldName) === null || _r === void 0 ? void 0 : _r.call(invoc, 'arguments')) !== null && _s !== void 0 ? _s : (0, treeSitterUtils_1.firstChildOfType)(invoc, 'argument_list');
+        const argList = (_g = (_f = invoc.childForFieldName) === null || _f === void 0 ? void 0 : _f.call(invoc, 'arguments')) !== null && _g !== void 0 ? _g : (0, treeSitterUtils_1.firstChildOfType)(invoc, 'argument_list');
         if (!argList)
             continue;
         // Extract the first string argument (path)
@@ -170,6 +138,15 @@ function extractJavaHttpCalls(block, constants, out) {
         const { value: path, isDirect, varName } = firstStringArg;
         // Try to detect the HTTP method from context
         let httpMethod = lowerMethod.toUpperCase();
+        // RestAssured: .when().get("/path") or .given().get("/path")
+        // The method name IS the HTTP method in RestAssured style
+        if (!HTTP_METHODS.has(lowerMethod))
+            continue;
+        // MockMvc: perform(get("/path")) — perform wraps a method call
+        if (lowerMethod === 'perform') {
+            // The arg is another method invocation
+            continue; // Handled when we process the inner get/post
+        }
         if (lowerMethod === 'request') {
             // generic request(method, path) — try to get HTTP method from first arg
             const stringArgs = findAllStringsInArgList(argList, constants);
@@ -247,40 +224,6 @@ function extractJavaFlowRefs(root) {
     return refs;
 }
 // ─── Helpers ──────────────────────────────────────────────────────────────────
-/**
- * Extract method parameters with their annotations (e.g. @RequestParam, @PathVariable, @RequestBody).
- */
-function extractMethodParameters(method) {
-    var _a;
-    const params = [];
-    const formalParams = (0, treeSitterUtils_1.firstChildOfType)(method, 'formal_parameters');
-    if (!formalParams)
-        return params;
-    const paramNodes = (0, treeSitterUtils_1.findNodes)(formalParams, ['formal_parameter', 'spread_parameter']);
-    for (const paramNode of paramNodes) {
-        const paramAnnotations = [];
-        (0, treeSitterUtils_1.walkTree)(paramNode, (n) => {
-            var _a;
-            if (n.type === 'annotation' || n.type === 'marker_annotation') {
-                const nameNode = (0, treeSitterUtils_1.firstChildOfType)(n, 'identifier');
-                if (nameNode)
-                    paramAnnotations.push('@' + ((_a = nameNode.text) !== null && _a !== void 0 ? _a : ''));
-            }
-        });
-        const nameNode = firstChildNamed(paramNode, 'identifier');
-        const name = (_a = nameNode === null || nameNode === void 0 ? void 0 : nameNode.text) !== null && _a !== void 0 ? _a : '';
-        if (!name)
-            continue;
-        // Include annotation prefix for annotated params (Spring @RequestParam, @PathVariable, etc.)
-        if (paramAnnotations.length > 0) {
-            params.push(`${paramAnnotations.join(' ')} ${name}`);
-        }
-        else {
-            params.push(name);
-        }
-    }
-    return params;
-}
 function extractModifierTexts(node) {
     const modifiers = [];
     (0, treeSitterUtils_1.walkTree)(node, (n) => {

package/dist/src/languages/javascript/angularDetector.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"angularDetector.d.ts","sourceRoot":"","sources":["../../../../src/languages/javascript/angularDetector.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,kBAAkB,EAAE,OAAO,CAAC;IAC5B,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,gBAAgB;IAC/B,wCAAwC;IACxC,aAAa,EAAE,MAAM,CAAC;IACtB,mCAAmC;IACnC,YAAY,EAAE,MAAM,CAAC;IACrB,+DAA+D;IAC/D,KAAK,EAAE,aAAa,GAAG,WAAW,CAAC;IACnC,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,YAAY,GAAG,aAAa,CAAC;IACnC,SAAS,EAAE,aAAa,GAAG,kBAAkB,GAAG,eAAe,GAAG,SAAS,GAAG,UAAU,CAAC;IACzF,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,kBAAkB;IACjC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,aAAa,GAAG,YAAY,CAAC;IACnC,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,gBAAgB;IAC/B,2BAA2B,EAAE,OAAO,CAAC;IACrC,oBAAoB,EAAE,MAAM,EAAE,CAAC;IAC/B,UAAU,EAAE,MAAM,CAAC;CACpB;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,eAAe,EAAE,CAyC9F;AAED;;GAEG;AACH,wBAAgB,uBAAuB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,gBAAgB,EAAE,CA2EhG;AAED;;GAEG;AACH,wBAAgB,mBAAmB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,YAAY,EAAE,CAyCxF;AAED;;GAEG;AACH,wBAAgB,yBAAyB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,kBAAkB,EAAE,CAgCpG;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,gBAAgB,CAY7F;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAEhE"}
+{"version":3,"file":"angularDetector.d.ts","sourceRoot":"","sources":["../../../../src/languages/javascript/angularDetector.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,kBAAkB,EAAE,OAAO,CAAC;IAC5B,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,gBAAgB;IAC/B,wCAAwC;IACxC,aAAa,EAAE,MAAM,CAAC;IACtB,mCAAmC;IACnC,YAAY,EAAE,MAAM,CAAC;IACrB,+DAA+D;IAC/D,KAAK,EAAE,aAAa,GAAG,WAAW,CAAC;IACnC,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,YAAY,GAAG,aAAa,CAAC;IACnC,SAAS,EAAE,aAAa,GAAG,kBAAkB,GAAG,eAAe,GAAG,SAAS,GAAG,UAAU,CAAC;IACzF,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,kBAAkB;IACjC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,aAAa,GAAG,YAAY,CAAC;IACnC,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,gBAAgB;IAC/B,2BAA2B,EAAE,OAAO,CAAC;IACrC,oBAAoB,EAAE,MAAM,EAAE,CAAC;IAC/B,UAAU,EAAE,MAAM,CAAC;CACpB;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,eAAe,EAAE,CAyC9F;AAED;;GAEG;AACH,wBAAgB,uBAAuB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,gBAAgB,EAAE,CAuChG;AAED;;GAEG;AACH,wBAAgB,mBAAmB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,YAAY,EAAE,CAyCxF;AAED;;GAEG;AACH,wBAAgB,yBAAyB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,kBAAkB,EAAE,CAgCpG;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,gBAAgB,CAY7F;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAEhE"}

package/dist/src/languages/javascript/angularDetector.js

@@ -68,66 +68,33 @@ function detectAngularInjections(sourceText, filePath) {
     const injections = [];
     const lines = sourceText.split('\n');
     let currentClass = '';
-    let currentConstHost = '';
-    let inConstructorParams = false;
     for (let i = 0; i < lines.length; i++) {
         const line = lines[i];
         // Track current class
         const classMatch = line.match(/class\s+(\w+)/);
         if (classMatch)
             currentClass = classMatch[1];
-        // Track current const/export const assignment as potential host for inject() outside class.
-        // Only update when the line is NOT itself an inject() call (e.g. skip `const x = inject(Y)`).
-        if (!currentClass) {
-            const constHostMatch = line.match(/(?:export\s+)?(?:const|let)\s+(\w+)\s*(?::\s*\S+)?\s*=/);
-            if (constHostMatch && !line.match(/=\s*inject\s*\(/)) {
-                currentConstHost = constHostMatch[1];
-            }
-        }
-        // Track constructor parameter block boundaries
-        if (/\bconstructor\s*\(/.test(line)) {
-            inConstructorParams = true;
-        }
-        if (inConstructorParams && line.includes(')')) {
-            // Process this line (it's still inside the constructor params), then close
-            // We'll close after checking for injections below
-        }
         // Constructor injection: constructor(private http: HttpClient)
-        // Only match visibility-modified parameters when inside a constructor(...) block
-        if (inConstructorParams) {
-            const ctorParamPattern = /(?:private|protected|public|readonly)\s+(\w+)\s*:\s*(\w+)/g;
-            let ctorMatch;
-            while ((ctorMatch = ctorParamPattern.exec(line)) !== null) {
-                if (currentClass) {
-                    injections.push({
-                        consumerClass: currentClass,
-                        serviceClass: ctorMatch[2],
-                        style: 'constructor',
-                        sourceFile: filePath,
-                        line: i + 1,
-                    });
-                }
-            }
-        }
-        // Close the constructor param block after processing (handles closing paren on same line)
-        if (inConstructorParams && line.includes(')')) {
-            inConstructorParams = false;
+        const ctorMatch = line.match(/(?:private|protected|public|readonly)\s+(\w+)\s*:\s*(\w+)/);
+        if (ctorMatch && currentClass) {
+            injections.push({
+                consumerClass: currentClass,
+                serviceClass: ctorMatch[2],
+                style: 'constructor',
+                sourceFile: filePath,
+                line: i + 1,
+            });
         }
         // Functional inject: inject(HttpClient)
         const injectMatch = line.match(/(\w+)\s*=\s*inject\s*\(\s*(\w+)\s*\)/);
-        if (injectMatch) {
-            // Use the enclosing class, or fall back to the enclosing const/export const host
-            // (e.g. functional guards/interceptors: `export const authGuard: CanActivateFn = ...`)
-            const hostName = currentClass || currentConstHost;
-            if (hostName) {
-                injections.push({
-                    consumerClass: hostName,
-                    serviceClass: injectMatch[2],
-                    style: 'inject-fn',
-                    sourceFile: filePath,
-                    line: i + 1,
-                });
-            }
+        if (injectMatch && currentClass) {
+            injections.push({
+                consumerClass: currentClass,
+                serviceClass: injectMatch[2],
+                style: 'inject-fn',
+                sourceFile: filePath,
+                line: i + 1,
+            });
         }
     }
     return injections;

package/dist/src/languages/javascript/assertionResolver.js

@@ -66,8 +66,8 @@ function tryExtractAssertion(callNode) {
             if (!subject)
                 return null;
             const methodName = (_j = (_h = (_g = callee.property) === null || _g === void 0 ? void 0 : _g.name) === null || _h === void 0 ? void 0 : _h.toLowerCase()) !== null && _j !== void 0 ? _j : '';
-            const { variable, assertionType: subjectType } = classifyExpectSubject(subject);
-            const assertionType = classifyAssertionMethod(methodName, subjectType);
+            const assertionType = classifyAssertionMethod(methodName);
+            const { variable } = classifyExpectSubject(subject);
             return { assertionType, subjectVariable: variable, line: (_l = (_k = callNode.loc) === null || _k === void 0 ? void 0 : _k.start) === null || _l === void 0 ? void 0 : _l.line };
         }
     }
@@ -132,15 +132,13 @@ function classifyExpectSubject(subject) {
     }
     return { variable: undefined, assertionType: 'body-field' };
 }
-function classifyAssertionMethod(methodName, subjectType) {
+function classifyAssertionMethod(methodName) {
     if (methodName === 'tobe' ||
         methodName === 'toequal' ||
         methodName === 'tostrictequal' ||
         methodName === 'tobetruthy' ||
         methodName === 'tobefalsy') {
-        // These methods are ambiguous — use subject context to decide.
-        // Only classify as status-code when the subject involves status.
-        return subjectType === 'status-code' ? 'status-code' : 'body-field';
+        return 'status-code'; // Could be either, default to status-code
     }
     if (methodName === 'tohaveproperty' || methodName === 'tocontain' || methodName === 'tomatch') {
         return 'body-field';

package/dist/src/languages/javascript/hapiDetector.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"hapiDetector.d.ts","sourceRoot":"","sources":["../../../../src/languages/javascript/hapiDetector.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,QAAQ,CAAC;IAChB,UAAU,CAAC,EAAE,cAAc,CAAC;IAC5B,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,QAAQ;IACvB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,EAAE,UAAU,GAAG,UAAU,GAAG,KAAK,CAAC;CACvC;AAED,MAAM,WAAW,cAAc;IAC7B,WAAW,EAAE,MAAM,EAAE,CAAC;IACtB,aAAa,EAAE,MAAM,EAAE,CAAC;CACzB;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAkBD;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,SAAS,EAAE,CA6HlF;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,aAAa,EAAE,CAmBtF"}
+{"version":3,"file":"hapiDetector.d.ts","sourceRoot":"","sources":["../../../../src/languages/javascript/hapiDetector.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,QAAQ,CAAC;IAChB,UAAU,CAAC,EAAE,cAAc,CAAC;IAC5B,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,QAAQ;IACvB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,EAAE,UAAU,GAAG,UAAU,GAAG,KAAK,CAAC;CACvC;AAED,MAAM,WAAW,cAAc;IAC7B,WAAW,EAAE,MAAM,EAAE,CAAC;IACtB,aAAa,EAAE,MAAM,EAAE,CAAC;CACzB;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAkBD;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,SAAS,EAAE,CAuFlF;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,aAAa,EAAE,CAmBtF"}