kodevu 0.1.66 → 0.1.68

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.66",
+  "version": "0.1.68",
   "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/logger.js

@@ -78,7 +78,7 @@ class Logger {
         const date = formatDate(new Date()).split(" ")[0];
         this.logFile = path.join(config.logsDir, `run-${date}.log`);
         
-        // Simple rotation: Clean up logs older than 7 days
+        // Simple rotation: Keep the most recent 7 log files
         this.#cleanupOldLogs(config.logsDir);
         this.initialized = true;
       } catch (err) {
@@ -161,21 +161,32 @@ class Logger {
     ].join("-");
   }
 
-
-
   #cleanupOldLogs(logsDir) {
     try {
       const files = fs.readdirSync(logsDir);
-      const now = Date.now();
-      const MAX_AGE_MS = 7 * 24 * 60 * 60 * 1000; // 7 days
 
-      for (const file of files) {
-        if (file.startsWith("run-") && file.endsWith(".log")) {
+      const logFiles = files
+        .filter((file) => file.startsWith("run-") && file.endsWith(".log"))
+        .map((file) => {
           const filePath = path.join(logsDir, file);
-          const stats = fs.statSync(filePath);
-          if (now - stats.mtimeMs > MAX_AGE_MS) {
-            fs.unlinkSync(filePath);
+          try {
+            const stats = fs.statSync(filePath);
+            return { file, path: filePath, mtime: stats.mtimeMs };
+          } catch {
+            return null;
           }
+        })
+        .filter(Boolean)
+        .sort((a, b) => b.mtime - a.mtime);
+
+      const KEEP_COUNT = 7;
+      const toRemove = logFiles.slice(KEEP_COUNT);
+
+      for (const entry of toRemove) {
+        try {
+          fs.unlinkSync(entry.path);
+        } catch {
+      // Ignore individual deletion errors
         }
       }
     } catch (err) {

package/src/reviewers.js

@@ -190,9 +190,21 @@ export const REVIEWERS = {
           debug: config.debug
         });
 
+        const message = execResult.stdout || execResult.stderr || "";
+
+        // Treat non-zero exit codes as failures and throw so callers can handle them.
+        if (typeof execResult.code === "number" && execResult.code !== 0) {
+          const err = new Error(
+            `OpenCode exited with code ${execResult.code}: ${execResult.stderr || execResult.stdout || ""}`
+          );
+          // attach execResult for callers that want more details
+          err.execResult = execResult;
+          throw err;
+        }
+
         return {
           ...execResult,
-          message: execResult.stdout || execResult.stderr || ""
+          message
         };
       } finally {
         await fs.rm(tempDir, { recursive: true, force: true });