@shakecodeslikecray/whiterose 1.0.2 → 1.0.3

package/dist/cli/index.js

@@ -4329,7 +4329,18 @@ function loadAccumulatedBugs(cwd) {
     }
     stored.bugs = stored.bugs.map((b) => ({ ...b, kind: b.kind || "bug" }));
     return stored;
-  } catch {
+  } catch (error) {
+    const backupPath = `${bugsPath}.corrupted.${Date.now()}`;
+    try {
+      const corruptedContent = readFileSync(bugsPath, "utf-8");
+      writeFileSync(backupPath, corruptedContent);
+      console.warn(`Warning: ${BUGS_FILENAME} is corrupted and could not be parsed.`);
+      console.warn(`Corrupted file backed up to: ${backupPath}`);
+      console.warn("Bug history has been reset. Previous bugs will appear as new.");
+    } catch {
+      console.warn(`Warning: ${BUGS_FILENAME} is corrupted and could not be parsed.`);
+      console.warn("Bug history has been reset. Previous bugs will appear as new.");
+    }
     return {
       version: STORAGE_VERSION,
       lastUpdated: (/* @__PURE__ */ new Date()).toISOString(),
@@ -6323,73 +6334,111 @@ In: ${bug.file}:${bug.line}`
 async function runAgenticFix(bug, config, projectDir) {
   const providerCommand = getProviderCommand(config.provider);
   const prompt = buildAgenticFixPrompt(bug);
-  const args = [];
-  if (config.provider === "claude-code") {
-    args.push("-p", prompt, "--dangerously-skip-permissions");
-  } else if (config.provider === "gemini") {
-    args.push("-p", prompt);
-  } else if (config.provider === "aider") {
-    args.push("--message", prompt, bug.file);
-  } else if (config.provider === "codex") ; else {
-    args.push("-p", prompt);
-  }
+  const controller = new AbortController();
+  const timeoutId = setTimeout(() => controller.abort(), 3e5);
   let stdout = "";
   let stderr = "";
-  if (config.provider === "codex") {
-    const tempDir = mkdtempSync(join(tmpdir(), "whiterose-fix-"));
-    const outputFile = join(tempDir, "output.txt");
-    try {
+  try {
+    if (config.provider === "codex") {
+      const tempDir = mkdtempSync(join(tmpdir(), "whiterose-fix-"));
+      const outputFile = join(tempDir, "output.txt");
+      try {
+        const result = await execa(
+          providerCommand,
+          [
+            "exec",
+            "--full-auto",
+            // Allow workspace writes without approval prompts
+            "--skip-git-repo-check",
+            "-C",
+            projectDir,
+            // Set working directory for codex
+            "-o",
+            outputFile,
+            "-"
+            // Read prompt from stdin
+          ],
+          {
+            cwd: projectDir,
+            input: prompt,
+            // Pass prompt via stdin
+            timeout: 3e5,
+            env: { ...process.env, NO_COLOR: "1" },
+            reject: false,
+            cancelSignal: controller.signal
+          }
+        );
+        stderr = result.stderr || "";
+        if (existsSync(outputFile)) {
+          try {
+            stdout = readFileSync(outputFile, "utf-8");
+          } catch {
+            stdout = result.stdout || "";
+          }
+        } else {
+          stdout = result.stdout || "";
+        }
+      } finally {
+        try {
+          rmSync(tempDir, { recursive: true, force: true });
+        } catch {
+        }
+      }
+    } else if (config.provider === "claude-code") {
       const result = await execa(
         providerCommand,
-        [
-          "exec",
-          "--full-auto",
-          // Allow workspace writes without approval prompts
-          "--skip-git-repo-check",
-          "-C",
-          projectDir,
-          // Set working directory for codex
-          "-o",
-          outputFile,
-          "-"
-          // Read prompt from stdin
-        ],
+        ["--dangerously-skip-permissions", "-p"],
         {
           cwd: projectDir,
           input: prompt,
-          // Pass prompt via stdin
+          // Pass prompt via stdin (Claude reads from stdin when no prompt arg provided)
           timeout: 3e5,
           env: { ...process.env, NO_COLOR: "1" },
-          reject: false
+          reject: false,
+          cancelSignal: controller.signal
         }
       );
+      stdout = result.stdout || "";
+      stderr = result.stderr || "";
+    } else if (config.provider === "gemini") {
+      const result = await execa(providerCommand, ["-p", prompt], {
+        cwd: projectDir,
+        timeout: 3e5,
+        env: { ...process.env, NO_COLOR: "1" },
+        reject: false,
+        stdin: "ignore",
+        // Prevent stdin hangs
+        cancelSignal: controller.signal
+      });
+      stdout = result.stdout || "";
+      stderr = result.stderr || "";
+    } else if (config.provider === "aider") {
+      const result = await execa(providerCommand, ["--message", prompt, bug.file], {
+        cwd: projectDir,
+        timeout: 3e5,
+        env: { ...process.env, NO_COLOR: "1" },
+        reject: false,
+        stdin: "ignore",
+        // Prevent stdin hangs
+        cancelSignal: controller.signal
+      });
+      stdout = result.stdout || "";
+      stderr = result.stderr || "";
+    } else {
+      const result = await execa(providerCommand, ["-p", prompt], {
+        cwd: projectDir,
+        timeout: 3e5,
+        env: { ...process.env, NO_COLOR: "1" },
+        reject: false,
+        stdin: "ignore",
+        // Prevent stdin hangs
+        cancelSignal: controller.signal
+      });
+      stdout = result.stdout || "";
       stderr = result.stderr || "";
-      if (existsSync(outputFile)) {
-        try {
-          stdout = readFileSync(outputFile, "utf-8");
-        } catch {
-          stdout = result.stdout || "";
-        }
-      } else {
-        stdout = result.stdout || "";
-      }
-    } finally {
-      try {
-        rmSync(tempDir, { recursive: true, force: true });
-      } catch {
-      }
     }
-  } else {
-    const result = await execa(providerCommand, args, {
-      cwd: projectDir,
-      timeout: 3e5,
-      // 5 minute timeout for agentic operations
-      env: { ...process.env, NO_COLOR: "1" },
-      reject: false
-      // Don't throw on non-zero exit
-    });
-    stdout = result.stdout || "";
-    stderr = result.stderr || "";
+  } finally {
+    clearTimeout(timeoutId);
   }
   if (stderr) {
     const lowerStderr = stderr.toLowerCase();