@lisa.ai/agent 2.2.1 → 2.2.2

package/dist/commands/coverage.js

@@ -206,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

@@ -109,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))
@@ -127,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.1",
+  "version": "2.2.2",
   "description": "Lisa.ai Autonomous CI/CD Worker Agent",
   "main": "dist/index.js",
   "bin": {