@yasserkhanorg/impact-gate 2.1.3 → 2.1.5

package/dist/esm/qa-agent/phase25/fix_loop.js

@@ -94,6 +94,7 @@ export async function runFixLoop(config, findings, browser, projectRoot) {
         screenshotDir,
         screenshotCounter: 100, // Start at 100 to avoid collisions with Phase 2 screenshots
         qaCommitHashes: new Set(),
+        pendingWrittenFiles: new Set(),
     };
     for (const finding of fixable) {
         if (wtf.shouldStop()) {

package/dist/esm/qa-agent/phase25/fix_tools.js

@@ -159,8 +159,21 @@ export function executeFixTool(ctx, name, input) {
             if (!isPathSafe(ctx.projectRoot, filePath)) {
                 return { output: `Blocked: "${filePath}" is outside the project or a restricted path.` };
             }
+            // Refuse to overwrite files that already have uncommitted user changes
+            if (!ctx.pendingWrittenFiles.has(filePath)) {
+                try {
+                    const status = execFileSync('git', ['status', '--porcelain', '--', filePath], { cwd: ctx.projectRoot, encoding: 'utf-8' }).trim();
+                    if (status.length > 0) {
+                        return { output: `Blocked: "${filePath}" has uncommitted user changes. Choose a different file or skip this fix.` };
+                    }
+                }
+                catch {
+                    // git status failed — allow the write (file may be new/untracked)
+                }
+            }
             const fullPath = resolve(ctx.projectRoot, filePath);
             writeFileSync(fullPath, String(input.content), 'utf-8');
+            ctx.pendingWrittenFiles.add(filePath);
             return { output: `Written: ${filePath}`, filesChanged: [filePath] };
         }
         case 'search_code': {
@@ -227,6 +240,7 @@ export function executeFixTool(ctx, name, input) {
                 execFileSync('git', ['commit', '-m', message], { cwd: ctx.projectRoot, encoding: 'utf-8' });
                 const hash = execFileSync('git', ['rev-parse', '--short', 'HEAD'], { cwd: ctx.projectRoot, encoding: 'utf-8' }).trim();
                 ctx.qaCommitHashes.add(hash);
+                ctx.pendingWrittenFiles.clear();
                 return { output: `Committed: ${hash} — ${message}`, commitHash: hash, filesChanged: files };
             }
             catch (err) {
@@ -252,9 +266,14 @@ export function executeFixTool(ctx, name, input) {
             }
         }
         case 'git_restore': {
+            const filesToRestore = [...ctx.pendingWrittenFiles];
+            if (filesToRestore.length === 0) {
+                return { output: 'No pending edits to restore.' };
+            }
             try {
-                execFileSync('git', ['checkout', '--', '.'], { cwd: ctx.projectRoot, encoding: 'utf-8' });
-                return { output: 'Restored working tree to last commit state. All uncommitted edits discarded.' };
+                execFileSync('git', ['checkout', '--', ...filesToRestore], { cwd: ctx.projectRoot, encoding: 'utf-8' });
+                ctx.pendingWrittenFiles.clear();
+                return { output: `Restored ${filesToRestore.length} file(s): ${filesToRestore.join(', ')}` };
             }
             catch (err) {
                 const error = err;

package/dist/qa-agent/phase25/fix_loop.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fix_loop.d.ts","sourceRoot":"","sources":["../../../src/qa-agent/phase25/fix_loop.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAC,OAAO,EAAqC,aAAa,EAAE,QAAQ,EAAC,MAAM,aAAa,CAAC;AACrG,OAAO,KAAK,EAAC,YAAY,EAAC,MAAM,4BAA4B,CAAC;AAyD7D,wBAAsB,UAAU,CAC5B,MAAM,EAAE,QAAQ,EAChB,QAAQ,EAAE,OAAO,EAAE,EACnB,OAAO,EAAE,YAAY,EACrB,WAAW,EAAE,MAAM,GACpB,OAAO,CAAC,aAAa,CAAC,CA4FxB"}
+{"version":3,"file":"fix_loop.d.ts","sourceRoot":"","sources":["../../../src/qa-agent/phase25/fix_loop.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAC,OAAO,EAAqC,aAAa,EAAE,QAAQ,EAAC,MAAM,aAAa,CAAC;AACrG,OAAO,KAAK,EAAC,YAAY,EAAC,MAAM,4BAA4B,CAAC;AAyD7D,wBAAsB,UAAU,CAC5B,MAAM,EAAE,QAAQ,EAChB,QAAQ,EAAE,OAAO,EAAE,EACnB,OAAO,EAAE,YAAY,EACrB,WAAW,EAAE,MAAM,GACpB,OAAO,CAAC,aAAa,CAAC,CA6FxB"}

package/dist/qa-agent/phase25/fix_loop.js

@@ -100,6 +100,7 @@ async function runFixLoop(config, findings, browser, projectRoot) {
         screenshotDir,
         screenshotCounter: 100, // Start at 100 to avoid collisions with Phase 2 screenshots
         qaCommitHashes: new Set(),
+        pendingWrittenFiles: new Set(),
     };
     for (const finding of fixable) {
         if (wtf.shouldStop()) {

package/dist/qa-agent/phase25/fix_tools.d.ts

@@ -9,6 +9,8 @@ export interface FixToolContext {
     screenshotCounter: number;
     /** Commit hashes created by the fix loop. Only these can be reverted. */
     qaCommitHashes: Set<string>;
+    /** Files written by the current fix attempt. Only these are restored on failure. */
+    pendingWrittenFiles: Set<string>;
 }
 export interface FixToolResult {
     output: string;

package/dist/qa-agent/phase25/fix_tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fix_tools.d.ts","sourceRoot":"","sources":["../../../src/qa-agent/phase25/fix_tools.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,SAAS,MAAM,mBAAmB,CAAC;AAE/C,OAAO,KAAK,EAAC,YAAY,EAAC,MAAM,4BAA4B,CAAC;AAM7D,eAAO,MAAM,oBAAoB,EAAE,SAAS,CAAC,IAAI,EAgGhD,CAAC;AAMF,MAAM,WAAW,cAAc;IAC3B,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,YAAY,CAAC;IACtB,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,MAAM,CAAC;IACtB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,yEAAyE;IACzE,cAAc,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;CAC/B;AAED,MAAM,WAAW,aAAa;IAC1B,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,0DAA0D;IAC1D,aAAa,CAAC,EAAE,OAAO,CAAC;CAC3B;AA2CD,wBAAgB,cAAc,CAC1B,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC/B,aAAa,CA8Jf"}
+{"version":3,"file":"fix_tools.d.ts","sourceRoot":"","sources":["../../../src/qa-agent/phase25/fix_tools.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,SAAS,MAAM,mBAAmB,CAAC;AAE/C,OAAO,KAAK,EAAC,YAAY,EAAC,MAAM,4BAA4B,CAAC;AAM7D,eAAO,MAAM,oBAAoB,EAAE,SAAS,CAAC,IAAI,EAgGhD,CAAC;AAMF,MAAM,WAAW,cAAc;IAC3B,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,YAAY,CAAC;IACtB,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,MAAM,CAAC;IACtB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,yEAAyE;IACzE,cAAc,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAC5B,oFAAoF;IACpF,mBAAmB,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;CACpC;AAED,MAAM,WAAW,aAAa;IAC1B,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,0DAA0D;IAC1D,aAAa,CAAC,EAAE,OAAO,CAAC;CAC3B;AA2CD,wBAAgB,cAAc,CAC1B,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC/B,aAAa,CAmLf"}

package/dist/qa-agent/phase25/fix_tools.js

@@ -163,8 +163,21 @@ function executeFixTool(ctx, name, input) {
             if (!isPathSafe(ctx.projectRoot, filePath)) {
                 return { output: `Blocked: "${filePath}" is outside the project or a restricted path.` };
             }
+            // Refuse to overwrite files that already have uncommitted user changes
+            if (!ctx.pendingWrittenFiles.has(filePath)) {
+                try {
+                    const status = (0, child_process_1.execFileSync)('git', ['status', '--porcelain', '--', filePath], { cwd: ctx.projectRoot, encoding: 'utf-8' }).trim();
+                    if (status.length > 0) {
+                        return { output: `Blocked: "${filePath}" has uncommitted user changes. Choose a different file or skip this fix.` };
+                    }
+                }
+                catch {
+                    // git status failed — allow the write (file may be new/untracked)
+                }
+            }
             const fullPath = (0, path_1.resolve)(ctx.projectRoot, filePath);
             (0, fs_1.writeFileSync)(fullPath, String(input.content), 'utf-8');
+            ctx.pendingWrittenFiles.add(filePath);
             return { output: `Written: ${filePath}`, filesChanged: [filePath] };
         }
         case 'search_code': {
@@ -231,6 +244,7 @@ function executeFixTool(ctx, name, input) {
                 (0, child_process_1.execFileSync)('git', ['commit', '-m', message], { cwd: ctx.projectRoot, encoding: 'utf-8' });
                 const hash = (0, child_process_1.execFileSync)('git', ['rev-parse', '--short', 'HEAD'], { cwd: ctx.projectRoot, encoding: 'utf-8' }).trim();
                 ctx.qaCommitHashes.add(hash);
+                ctx.pendingWrittenFiles.clear();
                 return { output: `Committed: ${hash} — ${message}`, commitHash: hash, filesChanged: files };
             }
             catch (err) {
@@ -256,9 +270,14 @@ function executeFixTool(ctx, name, input) {
             }
         }
         case 'git_restore': {
+            const filesToRestore = [...ctx.pendingWrittenFiles];
+            if (filesToRestore.length === 0) {
+                return { output: 'No pending edits to restore.' };
+            }
             try {
-                (0, child_process_1.execFileSync)('git', ['checkout', '--', '.'], { cwd: ctx.projectRoot, encoding: 'utf-8' });
-                return { output: 'Restored working tree to last commit state. All uncommitted edits discarded.' };
+                (0, child_process_1.execFileSync)('git', ['checkout', '--', ...filesToRestore], { cwd: ctx.projectRoot, encoding: 'utf-8' });
+                ctx.pendingWrittenFiles.clear();
+                return { output: `Restored ${filesToRestore.length} file(s): ${filesToRestore.join(', ')}` };
             }
             catch (err) {
                 const error = err;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yasserkhanorg/impact-gate",
-  "version": "2.1.3",
+  "version": "2.1.5",
   "description": "Diff-aware E2E impact analysis and coverage gating for Playwright/Cypress teams. Optional AI features can suggest, generate, and heal tests once your project has a route-families.json manifest.",
   "main": "dist/index.js",
   "module": "dist/esm/index.js",