@lisa.ai/agent 2.2.0 → 2.2.2

package/dist/commands/coverage.js

@@ -44,6 +44,7 @@ const telemetry_service_1 = require("../services/telemetry.service");
 const discovery_service_1 = require("../services/discovery.service");
 const installer_service_1 = require("../services/installer.service");
 const generator_service_1 = require("../services/generator.service");
+const parser_1 = require("../utils/parser");
 async function coverageCommand(command, modelProvider, attempt = 1, maxRetries = 3, projectId = 'local', apiKey) {
     // Bug 8 fix: Always run scanRepository() so we have a framework fingerprint regardless of
     // whether the user passed --command explicitly. scanRepository() is cheap (reads package.json)
@@ -60,6 +61,17 @@ async function coverageCommand(command, modelProvider, attempt = 1, maxRetries =
         }
         if (mutableFingerprint.suggestedTestCommand) {
             command = mutableFingerprint.suggestedTestCommand;
+            // Ensure coverage reporting is enabled. Without --coverage the test runner never
+            // writes coverage-summary.json, so the agent stays permanently stuck in cold-start
+            // discovery mode no matter how many tests it generates.
+            // Karma instruments coverage via karma.conf.js — don't touch it.
+            const fw = mutableFingerprint.testingFramework;
+            if ((fw === 'jest' || fw === 'vitest') && !command.includes('--coverage')) {
+                command = command.includes('npm run')
+                    ? `${command} -- --coverage`
+                    : `${command} --coverage`;
+                console.log(`[Lisa.ai Coverage] Coverage flag appended: ${command}`);
+            }
             console.log(`[Lisa.ai Auto-Discovery] Bootstrapping execution with natively discovered command: ${command}`);
         }
         else {
@@ -120,14 +132,24 @@ async function coverageCommand(command, modelProvider, attempt = 1, maxRetries =
         console.log(`\n✅ [Lisa.ai Coverage] Tests passed successfully on attempt ${attempt}.`);
     }
     catch (error) {
-        console.log(`\n❌ [Lisa.ai Coverage] Tests failed. Delegating to Auto-Heal...`);
-        // If tests themselves fail to compile or run, we fallback exactly to auto-heal
-        await (0, heal_1.healCommand)(command, modelProvider, 1, null, maxRetries, projectId, undefined, apiKey);
-        // Once the auto-heal engine successfully returns, compilation errors are fixed!
-        // We gracefully restart the coverage engine to evaluate the newly working tests.
-        console.log(`\n🔄 [Lisa.ai Coverage] Auto-Heal successful. Restarting coverage analysis...`);
-        await coverageCommand(command, modelProvider, attempt + 1, maxRetries, projectId, apiKey);
-        return;
+        const errorLog = (error.stderr || '') + '\n' + (error.stdout || '') + '\n' + (error.message || '');
+        // Check whether the error output contains an actual broken source file.
+        // If it does → real compilation/runtime failure → delegate to healCommand.
+        // If it does NOT (e.g. Jest's "No tests found, exiting with code 1" on a cold-start
+        // project, or a jest config error with no file reference) → delegating to healCommand
+        // would crash with "Could not parse a valid failing file path". Instead, fall through
+        // to the cold-start discovery block below which will generate the first test files.
+        const hasHealableFile = (0, parser_1.extractFilePath)(errorLog, [], process.cwd()) !== null;
+        if (hasHealableFile) {
+            console.log(`\n❌ [Lisa.ai Coverage] Tests failed. Delegating to Auto-Heal...`);
+            await (0, heal_1.healCommand)(command, modelProvider, 1, null, maxRetries, projectId, undefined, apiKey);
+            console.log(`\n🔄 [Lisa.ai Coverage] Auto-Heal successful. Restarting coverage analysis...`);
+            await coverageCommand(command, modelProvider, attempt + 1, maxRetries, projectId, apiKey);
+            return;
+        }
+        // No file path found in the error — most likely cold-start (no test files yet).
+        // Fall through to the coverage evaluation block which will trigger cold-start discovery.
+        console.log(`\n[Lisa.ai Coverage] No failing spec file detected in error output. Tests may not exist yet — initiating Cold-Start Discovery...`);
     }
     // 2. Tests passed — evaluate coverage.
     // Bug 5 fix: removed dead `executionPassed` variable. It was set to `true` on line 42
@@ -184,15 +206,26 @@ async function coverageCommand(command, modelProvider, attempt = 1, maxRetries =
         }
         let testCode = '';
         const fileContent = fs.readFileSync(absoluteTarget, 'utf-8');
-        if (existingSpecContent) {
-            console.log(`[Lisa.ai Coverage] Existing test suite discovered for ${targetFilePath}. Requesting single-pass logic coverage append...`);
-            // Bug 8 fix: pass detectedFramework so the LLM generates correct syntax for the project's framework.
-            testCode = await (0, llm_service_1.updateTestForFile)(targetFilePath, fileContent, path.relative(process.cwd(), targetSpecPath), existingSpecContent, modelProvider, apiKey, detectedFramework);
+        // Bug 9 fix: wrap the LLM call in its own try-catch so that a context-overflow
+        // or garbage response for ONE file doesn't crash the whole coverage loop.
+        // On failure we write a minimal placeholder spec so:
+        //   (a) Jest doesn't error on a missing/corrupt file
+        //   (b) The coverage report records this file so it won't be re-attempted
+        //   (c) The loop continues straight to the next uncovered file.
+        try {
+            if (existingSpecContent) {
+                console.log(`[Lisa.ai Coverage] Existing test suite discovered for ${targetFilePath}. Requesting single-pass logic coverage append...`);
+                testCode = await (0, llm_service_1.updateTestForFile)(targetFilePath, fileContent, path.relative(process.cwd(), targetSpecPath), existingSpecContent, modelProvider, apiKey, detectedFramework);
+            }
+            else {
+                console.log(`[Lisa.ai Coverage] Requesting newly generated test suite for ${targetFilePath}...`);
+                testCode = await (0, llm_service_1.generateTestForFile)(targetFilePath, fileContent, modelProvider, apiKey, detectedFramework);
+            }
         }
-        else {
-            console.log(`[Lisa.ai Coverage] Requesting newly generated test suite for ${targetFilePath}...`);
-            // Bug 8 fix: pass detectedFramework so the LLM generates correct syntax for the project's framework.
-            testCode = await (0, llm_service_1.generateTestForFile)(targetFilePath, fileContent, modelProvider, apiKey, detectedFramework);
+        catch (llmError) {
+            console.warn(`[Lisa.ai Coverage] Skipping ${targetFilePath} — LLM call failed: ${llmError.message}`);
+            // Placeholder keeps Jest happy and marks the file as "attempted"
+            testCode = `// Lisa.ai: Auto-skipped — ${llmError.message}\ntest('placeholder', () => { expect(true).toBe(true); });\n`;
         }
         fs.writeFileSync(targetSpecPath, testCode, 'utf-8');
         console.log(`[Lisa.ai Coverage] Wrote Spec File to ${targetSpecPath}`);

package/dist/services/discovery.service.js

@@ -83,8 +83,19 @@ class AutoDiscoveryService {
                 result.suggestedTestCommand = 'npm run test';
             }
             else {
-                // They have the library but no script; we execute binary directly
-                result.suggestedTestCommand = result.testingFramework === 'karma' ? 'npx karma start' : `npx ${result.testingFramework}`;
+                // They have the library but no script; execute the binary directly.
+                // Include --coverage for jest/vitest so coverage-summary.json is produced.
+                // Without it the coverage command has no report to evaluate and gets permanently
+                // stuck in cold-start discovery mode.
+                if (result.testingFramework === 'karma') {
+                    result.suggestedTestCommand = 'npx karma start';
+                }
+                else if (result.testingFramework === 'jest') {
+                    result.suggestedTestCommand = 'npx jest --coverage';
+                }
+                else if (result.testingFramework === 'vitest') {
+                    result.suggestedTestCommand = 'npx vitest run --coverage';
+                }
             }
         }
         return result;
@@ -98,7 +109,33 @@ class AutoDiscoveryService {
             return untested;
         const files = fs.readdirSync(dir);
         const ignoreDirs = ['node_modules', 'dist', 'build', '.git', '.angular', 'coverage', 'public', 'assets'];
-        const ignoreFiles = ['main.ts', 'index.ts', 'app.config.ts', 'app.routes.ts', 'styles.css', 'styles.scss', 'tailwind.config.js', 'eslint.config.js'];
+        const ignoreFiles = [
+            // Angular / framework entry points
+            'main.ts', 'index.ts', 'app.config.ts', 'app.routes.ts',
+            'styles.css', 'styles.scss',
+            // Common config files (exact names)
+            'tailwind.config.js', 'tailwind.config.ts',
+            'eslint.config.js', 'eslint.config.ts',
+            'jest.config.js', 'jest.config.ts', 'jest.config.mjs',
+            'jest.setup.js', 'jest.setup.ts',
+            'webpack.config.js', 'webpack.config.ts',
+            'babel.config.js', 'babel.config.ts',
+            'rollup.config.js', 'rollup.config.ts',
+            'vite.config.js', 'vite.config.ts',
+            'vitest.config.js', 'vitest.config.ts',
+            'karma.conf.js', 'karma.config.js',
+            'prettier.config.js', 'prettier.config.ts',
+            'postcss.config.js', 'postcss.config.ts',
+            'gulpfile.js', 'Gulpfile.js',
+            // Dotfile configs often present in Node projects
+            '.eslintrc.js', '.eslintrc.ts', '.babelrc.js',
+        ];
+        // Pattern-based exclusions: dotfiles and *.config.* files not in the list above
+        const ignorePatterns = [
+            /^\./, // any dotfile (e.g. .eslintrc.js, .prettierrc.js)
+            /\.config\.(js|ts|mjs|cjs)$/, // foo.config.js / foo.config.ts
+            /\.conf\.(js|ts|mjs|cjs)$/, // foo.conf.js
+        ];
         for (const file of files) {
             const fullPath = path.join(dir, file);
             if (ignoreDirs.includes(file))
@@ -116,6 +153,8 @@ class AutoDiscoveryService {
             else if (file.match(/\.(ts|tsx|js|jsx|vue)$/) && !file.includes('.spec.') && !file.includes('.test.')) {
                 if (ignoreFiles.includes(file))
                     continue;
+                if (ignorePatterns.some(p => p.test(file)))
+                    continue;
                 const ext = path.extname(file);
                 const base = file.slice(0, -ext.length);
                 const possibleSpecs = [

package/dist/services/llm.service.js

@@ -44,6 +44,26 @@ const google_1 = require("@ai-sdk/google");
 const path = __importStar(require("path"));
 const dotenv = __importStar(require("dotenv"));
 dotenv.config({ path: path.resolve(__dirname, '../../.env'), quiet: true });
+// Context-window guard: large files cause the LLM to hit its input limit and return
+// its internal context-compression prompt instead of code, corrupting the spec file.
+// 15 000 chars ≈ ~4 000 tokens — safe for all supported models.
+const MAX_SOURCE_CHARS = 15000;
+function truncateSource(content, label) {
+    if (content.length <= MAX_SOURCE_CHARS)
+        return content;
+    console.warn(`[Lisa.ai LLM] ${label} is ${content.length} chars — truncating to ${MAX_SOURCE_CHARS} to stay within context window.`);
+    return content.slice(0, MAX_SOURCE_CHARS) + '\n\n// ... (truncated)';
+}
+// Sanity-check the LLM response before it gets written to disk.
+// If the response contains none of the standard test primitives it is almost certainly
+// a meta-prompt leak or hallucination — throw so the caller can write a placeholder
+// instead of permanently corrupting the spec file.
+function assertLooksLikeTestCode(text, filePath) {
+    const testKeywords = /\b(describe|it\(|test\(|expect\(|beforeEach|afterEach|suite|assert\.|cy\.)\b/;
+    if (!testKeywords.test(text)) {
+        throw new Error(`LLM returned a non-code response for ${filePath} (possible context overflow). Skipping to prevent file corruption.`);
+    }
+}
 function getProvider(provider, apiKey) {
     if (provider === 'claude') {
         const key = apiKey || process.env.ANTHROPIC_API_KEY;
@@ -125,7 +145,7 @@ async function generateTestForFile(sourceFilePath, sourceFileContent, modelProvi
         : "3. Include all necessary imports assuming a standard testing framework (Jest/Karma/Vitest) is available.\n";
     const prompt = "You are Lisa.ai, an autonomous CI/CD expert platform.\n" +
         "A source file lacks 100% test coverage. Your task is to generate a comprehensive testing suite covering all branches, lines, and functions.\n\n" +
-        "--- Target File Content (" + sourceFilePath + ") ---\n" + sourceFileContent + "\n\n" +
+        "--- Target File Content (" + sourceFilePath + ") ---\n" + truncateSource(sourceFileContent, sourceFilePath) + "\n\n" +
         "--- Constraints ---\n" +
         "1. Return the generated test code wrapped in a markdown code block (```typescript ... ```).\n" +
         "2. Do not include any explanation or intro text.\n" +
@@ -136,7 +156,9 @@ async function generateTestForFile(sourceFilePath, sourceFileContent, modelProvi
         prompt,
     });
     const match = text.match(/```(?:typescript|ts|javascript|js)?\n([\s\S]*?)```/);
-    return match ? match[1].trim() : text.trim();
+    const result = match ? match[1].trim() : text.trim();
+    assertLooksLikeTestCode(result, sourceFilePath);
+    return result;
 }
 async function updateTestForFile(sourceFilePath, sourceFileContent, testFilePath, existingTestContent, modelProvider, apiKey, framework) {
     console.log(`[Lisa.ai Coverage] Requesting test update from ${modelProvider} for ${sourceFilePath}...`);
@@ -148,7 +170,7 @@ async function updateTestForFile(sourceFilePath, sourceFileContent, testFilePath
         : "3. Append missing tests to the existing suite. Do not delete existing passing tests unless they are fundamentally broken.\n";
     const prompt = "You are Lisa.ai, an autonomous CI/CD expert platform.\n" +
         "A source file lacks 100% test coverage. You must update its existing test suite to achieve full coverage.\n\n" +
-        "--- Target File Content (" + sourceFilePath + ") ---\n" + sourceFileContent + "\n\n" +
+        "--- Target File Content (" + sourceFilePath + ") ---\n" + truncateSource(sourceFileContent, sourceFilePath) + "\n\n" +
         "--- Existing Test Suite (" + testFilePath + ") ---\n" + existingTestContent + "\n\n" +
         "--- Constraints ---\n" +
         "1. Return the updated complete test code wrapped in a markdown code block (```typescript ... ```).\n" +
@@ -160,7 +182,9 @@ async function updateTestForFile(sourceFilePath, sourceFileContent, testFilePath
         prompt,
     });
     const match = text.match(/```(?:typescript|ts|javascript|js)?\n([\s\S]*?)```/);
-    return match ? match[1].trim() : text.trim();
+    const result = match ? match[1].trim() : text.trim();
+    assertLooksLikeTestCode(result, sourceFilePath);
+    return result;
 }
 async function askLisaGeneral(promptStr, modelProvider, apiKey) {
     const model = getProvider(modelProvider, apiKey);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lisa.ai/agent",
-  "version": "2.2.0",
+  "version": "2.2.2",
   "description": "Lisa.ai Autonomous CI/CD Worker Agent",
   "main": "dist/index.js",
   "bin": {