deflake 1.2.31 → 1.2.33

package/cli.js

@@ -116,10 +116,8 @@ async function analyzeAndFix(artifacts, client, argv, capturedOutput = '') {
         return 0;
     }
     
-    // Debug: verify captured output content
-    const hasErrorContext = capturedOutput.includes('Error Context:');
-    const errorContextCount = (capturedOutput.match(/Error Context:/g) || []).length;
-    console.log(`${C.GRAY}📊 Captured output: ${capturedOutput.length} chars, Error Context lines: ${errorContextCount}, has 'at ': ${capturedOutput.includes(' at ')}${C.RESET}`);
+    // Strip ANSI escape codes from captured output — they break regex matching
+    const cleanOutput = capturedOutput.replace(/\x1b\[[0-9;]*[a-zA-Z]/g, '');
     
     console.log(`${C.BRIGHT}🔍 Analyzing ${artifacts.length} failure(s)...${C.RESET}\n`);
     let count = 0;
@@ -133,10 +131,11 @@ async function analyzeAndFix(artifacts, client, argv, capturedOutput = '') {
             let loc = extractLoc(content);
             let locSource = 'error-context.md';
             
-            if (!loc && capturedOutput) {
+            if (!loc && cleanOutput) {
                 // Extract the relevant section from captured output for this artifact
-                const relevantOutput = extractRelevantOutput(capturedOutput, art.name);
-                loc = extractLoc(relevantOutput || capturedOutput);
+                const relevantOutput = extractRelevantOutput(cleanOutput, art.name);
+                console.log(`   ${C.GRAY}🔬 Relevant output: ${relevantOutput.length} chars for artifact${C.RESET}`);
+                loc = extractLoc(relevantOutput || cleanOutput);
                 locSource = 'console output';
             }
             
@@ -262,13 +261,9 @@ async function applyFix(res, loc) {
 }
 
 function extractRelevantOutput(fullOutput, artifactName) {
-    // SMART: Use Playwright's "Error Context:" line to find the EXACT error block for this artifact.
-    // Each Playwright failure ends with: "Error Context: test-results/<artifact-name>/error-context.md"
-    // We find that line, then walk backwards to capture the full error + stack trace.
-    
+    // Find the Error Context line for this artifact, then grab 30 lines before it
     const lines = fullOutput.split('\n');
     
-    // Find the Error Context line that matches this artifact
     let anchorIdx = -1;
     for (let i = 0; i < lines.length; i++) {
         if (lines[i].includes('Error Context:') && lines[i].includes(artifactName)) {
@@ -278,7 +273,7 @@ function extractRelevantOutput(fullOutput, artifactName) {
     }
     
     if (anchorIdx === -1) {
-        // Fallback: try partial match on artifact name parts
+        // Fallback: partial match on artifact name parts
         const nameParts = artifactName.split('-').filter(p => p.length > 3);
         for (let i = 0; i < lines.length; i++) {
             if (lines[i].includes('Error Context:') && nameParts.some(part => lines[i].includes(part))) {
@@ -290,16 +285,8 @@ function extractRelevantOutput(fullOutput, artifactName) {
     
     if (anchorIdx === -1) return '';
     
-    // Walk backwards from anchor to find the start of this error block
-    // Error blocks in Playwright start with "N) [chromium] › tests/..."
-    let startIdx = anchorIdx;
-    for (let i = anchorIdx; i >= 0; i--) {
-        if (lines[i].match(/^\s*\d+\)\s+\[/)) {
-            startIdx = i;
-            break;
-        }
-    }
-    
+    // Grab 30 lines before the anchor — guaranteed to include the full stack trace
+    const startIdx = Math.max(0, anchorIdx - 30);
     return lines.slice(startIdx, anchorIdx + 1).join('\n');
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "deflake",
-  "version": "1.2.31",
+  "version": "1.2.33",
   "description": "AI-powered self-healing tool for Playwright, Cypress, and WebdriverIO tests.",
   "main": "client.js",
   "bin": {