kodevu 0.1.56 → 0.1.58

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.56",
+  "version": "0.1.58",
   "license": "MIT",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",

package/src/config.js

@@ -17,7 +17,7 @@ const defaultConfig = {
   lang: "auto",
   outputDir: defaultStorageDir,
   logsDir: path.join(defaultStorageDir, "logs"),
-  commandTimeoutMs: 600000,
+  commandTimeoutMs: 120000,
   prompt: "",
   maxRevisionsPerRun: 5,
   outputFormats: ["markdown"],

package/src/logger.js

@@ -90,7 +90,7 @@ class Logger {
           fs.mkdirSync(config.logsDir, { recursive: true });
         }
         const date = formatDate(new Date()).split(" ")[0];
-        this.logFile = path.join(config.logsDir, `run-${date}-${this.sessionId}.log`);
+        this.logFile = path.join(config.logsDir, `run-${date}.log`);
         
         // Simple rotation: Clean up logs older than 7 days
         this._cleanupOldLogs(config.logsDir);

package/src/review-runner.js

@@ -105,7 +105,8 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
         reviewer: reviewerName,
         console: false
       });
-      // If it's the last one, it will throw below or break loop anyway
+      // Store the result so the final throw can include stderr/exit code details
+      reviewerResult = err?.result ?? null;
     }
 
     if (reviewerName !== reviewersToTry[reviewersToTry.length - 1]) {
@@ -120,9 +121,23 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
   }
 
   if (!reviewerResult || reviewerResult.code !== 0 || reviewerResult.timedOut) {
-    throw new Error(
-      `${reviewer?.displayName || config.reviewer} failed for ${details.displayId}`
-    );
+    const displayName = reviewer?.displayName || config.reviewer;
+    const reasonParts = [`${displayName} failed for ${details.displayId}`];
+
+    if (reviewerResult?.timedOut) {
+      reasonParts.push("(timed out)");
+    } else if (reviewerResult?.code != null && reviewerResult.code !== 0) {
+      reasonParts.push(`(exit code: ${reviewerResult.code})`);
+    } else if (!reviewerResult) {
+      reasonParts.push("(reviewer produced no result)");
+    }
+
+    const detail = (reviewerResult?.stderr || reviewerResult?.stdout || "").trim();
+    if (detail) {
+      reasonParts.push(`\n${detail}`);
+    }
+
+    throw new Error(reasonParts.join(" "));
   }
 
   progress?.update(0.82, "writing report");