kodevu 0.1.8 → 0.1.10

package/README.md

@@ -66,7 +66,7 @@ npx kodevu /path/to/your/repo --config ./config.current.json
 
 - `target`: required repository target; can be provided by config or as the CLI positional argument
 - `reviewer`: `codex`, `gemini`, or `auto`; default `auto`
-- `reviewPrompt`: saved into the report as review context
+- `prompt`: saved into the report as review context
 - `outputDir`: report output directory; default `~/.kodevu`
 - `stateFilePath`: review state file path; default `~/.kodevu/state.json`
 - `commandTimeoutMs`: timeout for a single review command execution in milliseconds

package/config.example.json

@@ -1,7 +1,7 @@
 {
   "target": "C:/path/to/your/repository-or-subdirectory",
   "reviewer": "auto",
-  "reviewPrompt": "请严格审查当前变更，优先指出 bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失测试。请使用简体中文输出 Markdown；如果没有明确缺陷，请写“未发现明确缺陷”，并补充剩余风险。",
+  "prompt": "请严格审查当前变更，优先指出 bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失测试。请使用简体中文输出 Markdown；如果没有明确缺陷，请写“未发现明确缺陷”，并补充剩余风险。",
   "outputDir": "~/.kodevu",
   "stateFilePath": "~/.kodevu/state.json",
   "commandTimeoutMs": 600000,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.8",
+  "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

@@ -14,7 +14,7 @@ const defaultConfig = {
   outputDir: defaultStorageDir,
   stateFilePath: path.join(defaultStorageDir, "state.json"),
   commandTimeoutMs: 600000,
-  reviewPrompt:
+  prompt:
     "请严格审查当前变更，优先指出 bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失测试。请使用简体中文输出 Markdown；如果没有明确缺陷，请写“未发现明确缺陷”，并补充剩余风险。",
   maxRevisionsPerRun: 20
 };
@@ -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) {
@@ -68,6 +73,8 @@ export function parseCliArgs(argv) {
     target: "",
     debug: false,
     help: false,
+    reviewer: "",
+    prompt: "",
     commandExplicitlySet: false
   };
 
@@ -101,6 +108,26 @@ export function parseCliArgs(argv) {
       continue;
     }
 
+    if (value === "--reviewer" || value === "-r") {
+      const reviewer = argv[index + 1];
+      if (!reviewer || reviewer.startsWith("-")) {
+        throw new Error(`Missing value for ${value}`);
+      }
+      args.reviewer = reviewer;
+      index += 1;
+      continue;
+    }
+
+    if (value === "--prompt" || value === "-p") {
+      const prompt = argv[index + 1];
+      if (!prompt || prompt.startsWith("-")) {
+        throw new Error(`Missing value for ${value}`);
+      }
+      args.prompt = prompt;
+      index += 1;
+      continue;
+    }
+
     if (!value.startsWith("-") && args.command === "run" && !args.target) {
       args.target = value;
       continue;
@@ -139,6 +166,14 @@ export async function loadConfig(configPath, cliArgs = {}) {
     config.target = cliArgs.target;
   }
 
+  if (cliArgs.reviewer) {
+    config.reviewer = cliArgs.reviewer;
+  }
+
+  if (cliArgs.prompt) {
+    config.prompt = cliArgs.prompt;
+  }
+
   if (!config.target) {
     throw new Error('Missing target. Pass `npx kodevu <repo-path>` or set "target" in config.json.');
   }
@@ -147,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(
@@ -188,6 +225,8 @@ Usage:
 
 Options:
   --config, -c   Optional config json path. If omitted, ./config.json is loaded only when present
+  --reviewer, -r Override reviewer (codex | gemini | auto)
+  --prompt, -p   Override prompt
   --debug, -d    Print extra debug information to the console
   --help, -h     Show help
 

package/src/review-runner.js

@@ -284,7 +284,7 @@ function buildPrompt(config, backend, targetInfo, details, reviewDiffPayload) {
   const canReadRelatedFiles = backend.kind === "git" || Boolean(targetInfo.workingCopyPath);
 
   return [
-    config.reviewPrompt,
+    config.prompt,
     canReadRelatedFiles
       ? `You are running inside a read-only workspace rooted at: ${workspaceRoot}`
       : "No local repository workspace is available for this review run.",
@@ -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);