kodevu 0.1.9 → 0.1.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.9",
+  "version": "0.1.10",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write Markdown review reports.",
   "bin": {

package/src/config.js

@@ -39,7 +39,7 @@ function resolveConfigPath(baseDir, value) {
   return path.isAbsolute(value) ? value : path.resolve(baseDir, value);
 }
 
-async function resolveAutoReviewer(debug, loadedConfigPath) {
+async function resolveAutoReviewers(debug, loadedConfigPath) {
   const availableReviewers = [];
 
   for (const reviewerName of SUPPORTED_REVIEWERS) {
@@ -57,7 +57,12 @@ async function resolveAutoReviewer(debug, loadedConfigPath) {
     );
   }
 
-  return availableReviewers[Math.floor(Math.random() * availableReviewers.length)];
+  for (let i = availableReviewers.length - 1; i > 0; i--) {
+    const j = Math.floor(Math.random() * (i + 1));
+    [availableReviewers[i], availableReviewers[j]] = [availableReviewers[j], availableReviewers[i]];
+  }
+
+  return availableReviewers;
 }
 
 export function parseCliArgs(argv) {
@@ -177,9 +182,11 @@ export async function loadConfig(configPath, cliArgs = {}) {
   config.reviewer = String(config.reviewer || "auto").toLowerCase();
 
   if (config.reviewer === "auto") {
-    const selectedReviewer = await resolveAutoReviewer(config.debug, loadedConfigPath);
+    const availableReviewers = await resolveAutoReviewers(config.debug, loadedConfigPath);
+    const selectedReviewer = availableReviewers[0];
     config.reviewer = selectedReviewer.reviewerName;
     config.reviewerCommandPath = selectedReviewer.commandPath;
+    config.fallbackReviewers = availableReviewers.map(r => r.reviewerName).slice(1);
     config.reviewerWasAutoSelected = true;
   } else if (!SUPPORTED_REVIEWERS.includes(config.reviewer)) {
     throw new Error(

package/src/review-runner.js

@@ -402,14 +402,38 @@ async function reviewChange(config, backend, targetInfo, changeId) {
   }
 
   const diffText = await backend.getChangeDiff(config, targetInfo, changeId);
-  const { reviewer, diffPayloads, result: reviewerResult } = await runReviewerPrompt(
-    config,
-    backend,
-    targetInfo,
-    details,
-    diffText
-  );
-  const report = buildReport(config, backend, targetInfo, details, diffPayloads, reviewer, reviewerResult);
+  const reviewersToTry = [config.reviewer, ...(config.fallbackReviewers || [])];
+
+  let reviewer;
+  let diffPayloads;
+  let reviewerResult;
+  let currentReviewerConfig;
+
+  for (const reviewerName of reviewersToTry) {
+    currentReviewerConfig = { ...config, reviewer: reviewerName };
+    debugLog(config, `Trying reviewer: ${reviewerName}`);
+
+    const res = await runReviewerPrompt(
+      currentReviewerConfig,
+      backend,
+      targetInfo,
+      details,
+      diffText
+    );
+    reviewer = res.reviewer;
+    diffPayloads = res.diffPayloads;
+    reviewerResult = res.result;
+
+    if (reviewerResult.code === 0 && !reviewerResult.timedOut) {
+      break;
+    }
+
+    if (reviewerName !== reviewersToTry[reviewersToTry.length - 1]) {
+      console.log(`${reviewer.displayName} failed for ${details.displayId}; trying next reviewer...`);
+    }
+  }
+
+  const report = buildReport(currentReviewerConfig, backend, targetInfo, details, diffPayloads, reviewer, reviewerResult);
   const outputFile = path.join(config.outputDir, backend.getReportFileName(changeId));
   await writeTextFile(outputFile, report);