kodevu 0.1.6 → 0.1.8

package/README.md

@@ -65,7 +65,7 @@ npx kodevu /path/to/your/repo --config ./config.current.json
 ## Config
 
 - `target`: required repository target; can be provided by config or as the CLI positional argument
-- `reviewer`: `codex` or `gemini`; default `codex`
+- `reviewer`: `codex`, `gemini`, or `auto`; default `auto`
 - `reviewPrompt`: saved into the report as review context
 - `outputDir`: report output directory; default `~/.kodevu`
 - `stateFilePath`: review state file path; default `~/.kodevu/state.json`
@@ -76,7 +76,7 @@ Internal defaults:
 
 - by default, review reports and state are stored under `~/.kodevu`; first run starts from the current latest change instead of replaying full history
 - if `./config.json` is absent, Kodevu still runs with built-in defaults as long as you pass a positional `target`
-- Kodevu invokes `git`, `svn`, and the configured reviewer CLI from `PATH`; debug logging is enabled only by passing `--debug` or `-d`
+- Kodevu invokes `git`, `svn`, and the configured reviewer CLI from `PATH`; when `reviewer` is `auto`, it randomly selects one from the installed reviewer CLIs it can find in `PATH`; debug logging is enabled only by passing `--debug` or `-d`
 
 ## Target Rules
 
@@ -88,10 +88,11 @@ Internal defaults:
 
 - `reviewer: "codex"` uses `codex exec` with the diff embedded in the prompt.
 - `reviewer: "gemini"` uses `gemini -p` in non-interactive mode.
+- `reviewer: "auto"` probes `codex` and `gemini` in `PATH`, then randomly chooses one of the available CLIs for this run.
 - Large diffs are truncated before being sent to the reviewer or written into the report once they exceed the configured line or character limits.
 - For Git targets and local SVN working copies, the reviewer command runs from the repository workspace so it can inspect related files beyond the diff when needed.
 - For remote SVN URLs without a local working copy, the review still relies on the diff and change metadata only.
-- SVN reports keep the `r123.md` naming style.
-- Git reports are written as `git-<short-commit-hash>.md`.
+- SVN reports are written as `<YYYYMMDD-HHmmss>-svn-r<revision>.md`.
+- Git reports are written as `<YYYYMMDD-HHmmss>-git-<short-commit-hash>.md`.
 - `~/.kodevu/state.json` stores per-project checkpoints keyed by repository identity; only the v2 multi-project structure is supported.
 - If the reviewer command exits non-zero or times out, the report is still written, but the state is not advanced so the change can be retried later.

package/config.example.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write Markdown review reports.",
   "bin": {
@@ -13,7 +13,6 @@
   ],
   "scripts": {
     "start": "node src/index.js",
-    "once": "node src/index.js",
     "check": "node --check src/index.js && node --check src/config.js && node --check src/review-runner.js && node --check src/svn-client.js && node --check src/git-client.js && node --check src/vcs-client.js && node --check src/shell.js"
   },
   "engines": {

package/src/config.js

@@ -3,11 +3,13 @@ import { constants as fsConstants } from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
+import { findCommandOnPath } from "./shell.js";
 
 const defaultStorageDir = path.join(os.homedir(), ".kodevu");
+const SUPPORTED_REVIEWERS = ["codex", "gemini"];
 
 const defaultConfig = {
-  reviewer: "codex",
+  reviewer: "auto",
   target: "",
   outputDir: defaultStorageDir,
   stateFilePath: path.join(defaultStorageDir, "state.json"),
@@ -37,6 +39,27 @@ function resolveConfigPath(baseDir, value) {
   return path.isAbsolute(value) ? value : path.resolve(baseDir, value);
 }
 
+async function resolveAutoReviewer(debug, loadedConfigPath) {
+  const availableReviewers = [];
+
+  for (const reviewerName of SUPPORTED_REVIEWERS) {
+    const commandPath = await findCommandOnPath(reviewerName, { debug });
+    if (commandPath) {
+      availableReviewers.push({ reviewerName, commandPath });
+    }
+  }
+
+  if (availableReviewers.length === 0) {
+    throw new Error(
+      `No reviewer CLI was found in PATH for "reviewer": "auto". Install one of: ${SUPPORTED_REVIEWERS.join(", ")}${
+        loadedConfigPath ? ` (${loadedConfigPath})` : ""
+      }`
+    );
+  }
+
+  return availableReviewers[Math.floor(Math.random() * availableReviewers.length)];
+}
+
 export function parseCliArgs(argv) {
   const args = {
     command: "run",
@@ -120,11 +143,18 @@ export async function loadConfig(configPath, cliArgs = {}) {
     throw new Error('Missing target. Pass `npx kodevu <repo-path>` or set "target" in config.json.');
   }
 
-  config.reviewer = String(config.reviewer || "codex").toLowerCase();
   config.debug = Boolean(cliArgs.debug);
-
-  if (!["codex", "gemini"].includes(config.reviewer)) {
-    throw new Error(`"reviewer" must be one of "codex" or "gemini"${loadedConfigPath ? ` in ${loadedConfigPath}` : ""}`);
+  config.reviewer = String(config.reviewer || "auto").toLowerCase();
+
+  if (config.reviewer === "auto") {
+    const selectedReviewer = await resolveAutoReviewer(config.debug, loadedConfigPath);
+    config.reviewer = selectedReviewer.reviewerName;
+    config.reviewerCommandPath = selectedReviewer.commandPath;
+    config.reviewerWasAutoSelected = true;
+  } else if (!SUPPORTED_REVIEWERS.includes(config.reviewer)) {
+    throw new Error(
+      `"reviewer" must be one of "codex", "gemini", or "auto"${loadedConfigPath ? ` in ${loadedConfigPath}` : ""}`
+    );
   }
 
   config.configPath = loadedConfigPath;
@@ -162,7 +192,7 @@ Options:
   --help, -h     Show help
 
 Config highlights:
-  reviewer       codex | gemini
+  reviewer       codex | gemini | auto
   target         Repository target path (Git) or SVN working copy / URL; CLI positional target overrides config
 `);
 }

package/src/index.js

@@ -31,11 +31,19 @@ if (cliArgs.command === "init") {
 
 const config = await loadConfig(cliArgs.configPath, cliArgs);
 
+if (config.reviewerWasAutoSelected) {
+  console.log(
+    `Reviewer "auto" selected ${config.reviewer}${config.reviewerCommandPath ? ` (${config.reviewerCommandPath})` : ""}.`
+  );
+}
+
 if (config.debug) {
   console.error(
     `[debug] Loaded config: ${JSON.stringify({
       configPath: config.configPath,
       reviewer: config.reviewer,
+      reviewerCommandPath: config.reviewerCommandPath,
+      reviewerWasAutoSelected: config.reviewerWasAutoSelected,
       target: config.target,
       outputDir: config.outputDir,
       debug: config.debug

package/src/shell.js

@@ -103,3 +103,23 @@ export async function runCommand(command, args = [], options = {}) {
     }
   });
 }
+
+export async function findCommandOnPath(command, options = {}) {
+  const locator = process.platform === "win32" ? "where" : "which";
+  const result = await runCommand(locator, [command], {
+    allowFailure: true,
+    trim: true,
+    debug: options.debug
+  });
+
+  if (result.code !== 0 || result.timedOut || !result.stdout) {
+    return null;
+  }
+
+  return (
+    result.stdout
+      .split(/\r?\n/)
+      .map((item) => item.trim())
+      .find(Boolean) || null
+  );
+}

package/src/vcs-client.js

@@ -25,8 +25,17 @@ function createSvnBackend() {
       return `r${revision}`;
     },
     getReportFileName(revision) {
-      return `r${revision}.md`;
+      const now = new Date();
+      const datePrefix = now.getFullYear() +
+        String(now.getMonth() + 1).padStart(2, '0') +
+        String(now.getDate()).padStart(2, '0') +
+        '-' +
+        String(now.getHours()).padStart(2, '0') +
+        String(now.getMinutes()).padStart(2, '0') +
+        String(now.getSeconds()).padStart(2, '0');
+      return `${datePrefix}-svn-r${revision}.md`;
     },
+
     async getTargetInfo(config) {
       return await svnClient.getTargetInfo(config);
     },
@@ -86,8 +95,17 @@ function createGitBackend() {
       return commitHash.slice(0, 12);
     },
     getReportFileName(commitHash) {
-      return `git-${commitHash.slice(0, 12)}.md`;
+      const now = new Date();
+      const datePrefix = now.getFullYear() +
+        String(now.getMonth() + 1).padStart(2, '0') +
+        String(now.getDate()).padStart(2, '0') +
+        '-' +
+        String(now.getHours()).padStart(2, '0') +
+        String(now.getMinutes()).padStart(2, '0') +
+        String(now.getSeconds()).padStart(2, '0');
+      return `${datePrefix}-git-${commitHash.slice(0, 12)}.md`;
     },
+
     async getTargetInfo(config) {
       return await gitClient.getTargetInfo(config);
     },