@lisa.ai/agent 2.1.0 → 2.1.1

package/dist/services/llm.service.js

@@ -43,8 +43,6 @@ const anthropic_1 = require("@ai-sdk/anthropic");
 const google_1 = require("@ai-sdk/google");
 const path = __importStar(require("path"));
 const dotenv = __importStar(require("dotenv"));
-// Force dotenv to load the `.env` file from the agent's root directory
-// regardless of which folder the user is currently running the CLI inside.
 dotenv.config({ path: path.resolve(__dirname, '../../.env') });
 function getProvider(provider, apiKey) {
     if (provider === 'claude') {
@@ -67,35 +65,41 @@ function getProvider(provider, apiKey) {
 async function askLisaForFix(filePath, fileContent, errorLog, modelProvider, apiKey, previousFixAttempt, siblingContext) {
     console.log(`[Lisa.ai Auto-Heal] Requesting fix from ${modelProvider} for ${filePath}...`);
     const model = getProvider(modelProvider, apiKey);
-    let prompt = `You are Lisa.ai, an autonomous CI/CD expert platform.
-A build/compilation error occurred. Your task is to fix the provided file so that the error resolves.
-
---- Error Log ---
-${errorLog}
-
---- Target File Content (${filePath}) ---
-${fileContent}
-
---- Constraints ---
-1. Do not delete business logic unless absolutely necessary to resolve a missing dependency.
-2. Do not suppress TypeScript errors with @ts-ignore or any type assertions unless absolutely unavoidable.
-3. Fix the underlying type or logic issue.
-4. CRITICAL AGILE RULE: If the error is 'Cannot find module' or 'Property DOES NOT EXIST' and you do NOT know the correct import path or cannot resolve it from the context, you MUST silently comment out the failing import and completely comment out/stub the code that relies on it. Your ultimate goal is to make the file compile successfully so the Agent can proceed to the next file.
-5. FRAMEWORK TEST RULE: If fixing a testing file (\`.spec\` or \`.test\`) and an Angular error happens like 'NullInjectorError: No provider for X', you MUST add X to the 'providers' array in 'TestBed.configureTestingModule' or import the required testing module. If a React testing library error arises, fix the render context.
-6. Return the code wrapped in a markdown code block (\`\`\`typescript ... \`\`\`). Do not include any explanation or intro text.`;
+    let prompt = "You are Lisa.ai, an autonomous CI/CD expert platform.\n" +
+        "A build/compilation error occurred. Your task is to fix the provided file so that the error resolves.\n\n" +
+        "--- Error Log ---\n" + errorLog + "\n\n" +
+        "--- Target File Content (" + filePath + ") ---\n" + fileContent + "\n\n" +
+        "--- Constraints ---\n" +
+        "1. Do not delete business logic unless absolutely necessary to resolve a missing dependency.\n" +
+        "2. Do not suppress TypeScript errors with @ts-ignore or any type assertions unless absolutely unavoidable.\n" +
+        "3. Fix the underlying type or logic issue.\n" +
+        "4. CRITICAL AGILE RULE: If the error is 'Cannot find module' or 'Property DOES NOT EXIST' and you do NOT know the correct import path or cannot resolve it from the context, you MUST silently comment out the failing import and completely comment out/stub the code that relies on it. Your ultimate goal is to make the file compile successfully so the Agent can proceed to the next file.\n" +
+        "5. FRAMEWORK TEST RULE: If fixing a testing file ('.spec' or '.test') and an Angular error happens like 'NullInjectorError: No provider for X', you MUST add X to the 'providers' array in 'TestBed.configureTestingModule' or import the required testing module. If a React testing library error arises, fix the render context.\n" +
+        "6. CRITICAL ANGULAR RULE: If an Angular component imports 'RouterLink' or uses '[routerLink]' in its template, you MUST provide 'ActivatedRoute' in the providers array or import 'RouterTestingModule'. If it uses HTTP, import 'HttpClientTestingModule'.\n" +
+        "7. CRITICAL STANDALONE RULE: If a component has 'standalone: true' in its decorator (check the Sibling Context!), you MUST NOT add it to the 'declarations' array. Instead, you MUST add it to the 'imports' array within 'TestBed.configureTestingModule'.\n" +
+        "8. Return the code wrapped in a markdown code block (```typescript ... ```). Do not include any explanation or intro text.";
     if (siblingContext) {
-        prompt += `\n\n--- Sibling Component / Service Context ---
-You are fixing a \`.spec\` test file. Here is the actual implementation code for the component you are testing.
-Use this to identify EXACTLY which imports, Services, and Variables need to be mocked inside your \`TestBed\`.
-${siblingContext}`;
+        const isStandalone = /standalone\s*:\s*true/.test(siblingContext);
+        if (isStandalone) {
+            const classMatch = siblingContext.match(/export class (\w+)/);
+            const className = classMatch ? classMatch[1] : 'the component';
+            prompt += "\n\n--- CRITICAL ARCHITECTURAL REQUIREMENT ---\n" +
+                "The component " + className + " is marked as STANDALONE (standalone: true).\n" +
+                "You MUST add " + className + " to the 'imports' array within 'TestBed.configureTestingModule'.\n" +
+                "DO NOT add it to the 'declarations' array. If you omit " + className + " from 'imports', the test will fail.";
+        }
+        prompt += "\n\n--- Sibling Component / Service Context ---\n" +
+            "You are fixing a '.spec' test file. Here is the actual implementation code for the component you are testing.\n" +
+            "Use this to identify EXACTLY which imports, Services, and Variables need to be mocked inside your 'TestBed'.\n" +
+            siblingContext;
     }
     if (previousFixAttempt) {
         console.log(`[Lisa.ai Auto-Heal] Warning! Agent is looping on ${filePath}. Injecting previous failed context...`);
-        prompt += `\n\n--- CRITICAL WARNING ---\nYou previously attempted to fix this file but the compiler REJECTED your fix!
-Here is the previous analysis and failed fix you attempted:
-${previousFixAttempt}
-
-DO NOT repeat the identical code changes. Try a completely different programming approach, fix syntax typos, or check for missing imports.`;
+        prompt += "\n\n--- CRITICAL WARNING ---\n" +
+            "You previously attempted to fix this file but the compiler REJECTED your fix!\n" +
+            "Here is the previous analysis and failed fix you attempted:\n" +
+            previousFixAttempt + "\n\n" +
+            "DO NOT repeat the identical code changes. Try a completely different programming approach, fix syntax typos, or check for missing imports.";
     }
     const { text } = await (0, ai_1.generateText)({
         model,
@@ -107,17 +111,14 @@ DO NOT repeat the identical code changes. Try a completely different programming
 async function generateTestForFile(sourceFilePath, sourceFileContent, modelProvider, apiKey) {
     console.log(`[Lisa.ai Coverage] Requesting test generation from ${modelProvider} for ${sourceFilePath}...`);
     const model = getProvider(modelProvider, apiKey);
-    const prompt = `You are Lisa.ai, an autonomous CI/CD expert platform.
-A source file lacks 100% test coverage. Your task is to generate a comprehensive testing suite covering all branches, lines, and functions.
-
---- Target File Content (${sourceFilePath}) ---
-${sourceFileContent}
-
---- Constraints ---
-1. Return the generated test code wrapped in a markdown code block (\`\`\`typescript ... \`\`\`).
-2. Do not include any explanation or intro text.
-3. Include all necessary imports assuming a standard testing framework (Jest/Karma/Vitest) is available.
-4. Aim for 100% logic coverage.`;
+    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" +
+        "--- 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" +
+        "3. Include all necessary imports assuming a standard testing framework (Jest/Karma/Vitest) is available.\n" +
+        "4. Aim for 100% logic coverage.";
     const { text } = await (0, ai_1.generateText)({
         model,
         prompt,
@@ -128,20 +129,15 @@ ${sourceFileContent}
 async function updateTestForFile(sourceFilePath, sourceFileContent, testFilePath, existingTestContent, modelProvider, apiKey) {
     console.log(`[Lisa.ai Coverage] Requesting test update from ${modelProvider} for ${sourceFilePath}...`);
     const model = getProvider(modelProvider, apiKey);
-    const prompt = `You are Lisa.ai, an autonomous CI/CD expert platform.
-A source file lacks 100% test coverage. You must update its existing test suite to achieve full coverage.
-
---- Target File Content (${sourceFilePath}) ---
-${sourceFileContent}
-
---- Existing Test Suite (${testFilePath}) ---
-${existingTestContent}
-
---- Constraints ---
-1. Return the updated complete test code wrapped in a markdown code block (\`\`\`typescript ... \`\`\`).
-2. Do not include any explanation or intro text.
-3. Append missing tests to the existing suite. Do not delete existing passing tests unless they are fundamentally broken.
-4. Aim for 100% logic coverage across branches, lines, and functions.`;
+    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" +
+        "--- 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" +
+        "2. Do not include any explanation or intro text.\n" +
+        "3. Append missing tests to the existing suite. Do not delete existing passing tests unless they are fundamentally broken.\n" +
+        "4. Aim for 100% logic coverage across branches, lines, and functions.";
     const { text } = await (0, ai_1.generateText)({
         model,
         prompt,

package/dist/utils/parser.js

@@ -43,24 +43,25 @@ function extractFilePath(errorLog, skipFiles = [], searchDir = process.cwd()) {
     const normalizedSkips = skipFiles.map(p => path.resolve(p));
     // 1. First Pass: Try to match typical JS/TS/Vue error patterns with line numbers
     // e.g., "src/components/Button.tsx:12:3 - error"
-    const exactErrorRegex = /([a-zA-Z0-9_.\-\/\\]+\.(?:ts|tsx|js|jsx|vue))(?:\s*[:(])/g;
+    // Supporting Windows absolute paths (e.g. C:\...)
+    const exactErrorRegex = /([a-zA-Z]:[a-zA-Z0-9_.\-\/\\]+\.(?:ts|tsx|js|jsx|vue)|[a-zA-Z0-9_.\-\/\\]+\.(?:ts|tsx|js|jsx|vue))(?:\s*[:(])/g;
     let match;
     while ((match = exactErrorRegex.exec(cleanLog)) !== null) {
         const foundPath = match[1];
         if (foundPath) {
-            const absoluteFoundPath = path.resolve(searchDir, foundPath);
+            const absoluteFoundPath = path.isAbsolute(foundPath) ? foundPath : path.resolve(searchDir, foundPath);
             if (!normalizedSkips.includes(absoluteFoundPath) && fs.existsSync(absoluteFoundPath)) {
                 return foundPath;
             }
         }
     }
     // 2. Second Pass (Fallback): Find anything that looks like a source file
-    const fallbackRegex = /([a-zA-Z0-9_.\-\/\\]+\.(?:ts|tsx|js|jsx|vue))/g;
+    const fallbackRegex = /([a-zA-Z]:[a-zA-Z0-9_.\-\/\\]+\.(?:ts|tsx|js|jsx|vue)|[a-zA-Z0-9_.\-\/\\]+\.(?:ts|tsx|js|jsx|vue))/g;
     let fallbackMatch;
     while ((fallbackMatch = fallbackRegex.exec(cleanLog)) !== null) {
         const foundPath = fallbackMatch[1];
         if (foundPath) {
-            const absoluteFoundPath = path.resolve(searchDir, foundPath);
+            const absoluteFoundPath = path.isAbsolute(foundPath) ? foundPath : path.resolve(searchDir, foundPath);
             if (!normalizedSkips.includes(absoluteFoundPath) && fs.existsSync(absoluteFoundPath)) {
                 return foundPath;
             }
@@ -72,7 +73,7 @@ function extractFilePath(errorLog, skipFiles = [], searchDir = process.cwd()) {
     let symbolMatch;
     const searchedSymbols = new Set();
     // Filter out common JS/Browser built-ins and framework generics that are not user files
-    const ignoreList = ['Error', 'TypeError', 'SyntaxError', 'NullInjectorError', 'Object', 'Boolean', 'String', 'Number', 'Array', 'Chrome', 'Windows', 'Linux', 'Macintosh', 'UserContext', 'TestBed', 'Module'];
+    const ignoreList = ['Error', 'TypeError', 'SyntaxError', 'NullInjectorError', 'Object', 'Boolean', 'String', 'Number', 'Array', 'Chrome', 'Windows', 'Linux', 'Macintosh', 'UserContext', 'TestBed', 'Module', 'Unexpected', 'Expected', 'ChromeHeadless', 'Users', 'AppData', 'Local', 'Temp', 'Process', 'Component'];
     while ((symbolMatch = symbolRegex.exec(cleanLog)) !== null) {
         const symbolName = symbolMatch[1];
         if (symbolName && !ignoreList.includes(symbolName) && !searchedSymbols.has(symbolName)) {

package/package.json

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