kodevu 0.1.26 → 0.1.28

package/README.md

@@ -59,6 +59,12 @@ Run with debug logs:
 npx kodevu /path/to/your/repo --debug
 ```
 
+Specify the output language (e.g., Chinese):
+
+```bash
+npx kodevu /path/to/your/repo --lang zh
+```
+
 Use a custom config path only when needed:
 
 ```bash
@@ -77,7 +83,8 @@ 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`, `copilot`, or `auto`; default `auto`
-- `prompt`: saved into the report as review context
+- `prompt`: additional instructions for the reviewer; usually empty by default as the core instructions are built-in
+- `lang`: output language for the review (e.g., `zh`, `en`, `auto`); default `auto`
 - `outputDir`: report output directory; default `~/.kodevu`
 - `outputFormats`: report formats to generate; supports `markdown` and `json`; default `["markdown"]`
 - `stateFilePath`: review state file path; default `~/.kodevu/state.json`
@@ -110,6 +117,7 @@ Internal defaults:
 - If `outputFormats` includes `json`, matching `.json` files are generated alongside Markdown reports.
 - `~/.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.
+- Review instructions are built into the tool in English to ensure consistent logic across reviewers. The `lang` setting (CLI `--lang` or config `lang`) determines the language AI uses for the resulting review. When set to `auto`, Kodevu detects the language from the system environment (`LANG`, `LC_ALL`, or system locale).
 - Each report includes a `Token Usage` section recording token consumption for the review task. When the reviewer CLI outputs token statistics (via stderr), those are used directly (`source: "reviewer"`). Otherwise tokens are estimated at ~4 characters per token (`source: "estimate"`). The JSON report contains a `tokenUsage` object with `inputTokens`, `outputTokens`, `totalTokens`, and `source`.
 
 ## License

package/package.json

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

@@ -6,15 +6,16 @@ import { findCommandOnPath } from "./shell.js";
 const defaultStorageDir = path.join(os.homedir(), ".kodevu");
 const SUPPORTED_REVIEWERS = ["codex", "gemini", "copilot"];
 
+
 const defaultConfig = {
   reviewer: "auto",
   target: "",
+  lang: "auto",
   outputDir: defaultStorageDir,
   stateFilePath: path.join(defaultStorageDir, "state.json"),
   logsDir: path.join(defaultStorageDir, "logs"),
   commandTimeoutMs: 600000,
-  prompt:
-    "请严格审查当前变更，优先指出 bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失测试。请使用简体中文输出 Markdown；如果没有明确缺陷，请写“未发现明确缺陷”，并补充剩余风险。",
+  prompt: "",
   maxRevisionsPerRun: 5,
   outputFormats: ["markdown"]
 };
@@ -22,6 +23,7 @@ const defaultConfig = {
 const configTemplate = {
   target: "C:/path/to/your/repository-or-subdirectory",
   reviewer: defaultConfig.reviewer,
+  lang: defaultConfig.lang,
   prompt: defaultConfig.prompt,
   outputDir: "~/.kodevu",
   stateFilePath: "~/.kodevu/state.json",
@@ -74,6 +76,38 @@ function normalizeOutputFormats(outputFormats, loadedConfigPath) {
   return normalized;
 }
 
+function detectLanguage() {
+  const envLang = (process.env.LANG || process.env.LC_ALL || process.env.LC_MESSAGES || "").toLowerCase();
+  const intlLocale = (() => {
+    try {
+      return Intl.DateTimeFormat().resolvedOptions().locale.toLowerCase();
+    } catch {
+      return "";
+    }
+  })();
+
+  // On Windows, the LANG environment variable is often set to 'en_US.UTF-8' by default
+  // in many shells (like Git Bash/MSYS2), regardless of the actual OS display language.
+  // We prefer the system locale (Intl) on Windows if it's Chinese.
+  if (os.platform() === "win32" && intlLocale.startsWith("zh")) {
+    return "zh";
+  }
+
+  if (envLang) {
+    if (envLang.startsWith("zh")) return "zh";
+    if (envLang.startsWith("en")) return "en";
+    return envLang.split(/[._-]/)[0];
+  }
+
+  if (intlLocale) {
+    if (intlLocale.startsWith("zh")) return "zh";
+    if (intlLocale.startsWith("en")) return "en";
+    return intlLocale.split("-")[0];
+  }
+
+  return "en";
+}
+
 async function resolveAutoReviewers(debug, loadedConfigPath) {
   const availableReviewers = [];
 
@@ -109,6 +143,7 @@ export function parseCliArgs(argv) {
     debug: false,
     help: false,
     reviewer: "",
+    lang: "",
     prompt: "",
     commandExplicitlySet: false
   };
@@ -163,6 +198,16 @@ export function parseCliArgs(argv) {
       continue;
     }
 
+    if (value === "--lang" || value === "-l") {
+      const lang = argv[index + 1];
+      if (!lang || lang.startsWith("-")) {
+        throw new Error(`Missing value for ${value}`);
+      }
+      args.lang = lang;
+      index += 1;
+      continue;
+    }
+
     if (!value.startsWith("-") && args.command === "run" && !args.target) {
       args.target = value;
       continue;
@@ -209,12 +254,19 @@ export async function loadConfig(configPath, cliArgs = {}) {
     config.prompt = cliArgs.prompt;
   }
 
+  if (cliArgs.lang) {
+    config.lang = cliArgs.lang;
+  }
+
   if (!config.target) {
     throw new Error('Missing target. Pass `npx kodevu <repo-path>` or set "target" in config.json.');
   }
 
   config.debug = Boolean(cliArgs.debug);
   config.reviewer = String(config.reviewer || "auto").toLowerCase();
+  config.lang = String(config.lang || "auto").toLowerCase();
+
+  config.resolvedLang = config.lang === "auto" ? detectLanguage() : config.lang;
 
   if (config.reviewer === "auto") {
     const availableReviewers = await resolveAutoReviewers(config.debug, loadedConfigPath);
@@ -264,6 +316,7 @@ Options:
   --config, -c   Optional config json path. If omitted, ./config.json is loaded only when present
   --reviewer, -r Override reviewer (codex | gemini | copilot | auto)
   --prompt, -p   Override prompt
+  --lang, -l     Override output language (e.g. zh, en, auto)
   --debug, -d    Print extra debug information to the console
   --help, -h     Show help
 

package/src/index.js

@@ -49,6 +49,7 @@ try {
         reviewerWasAutoSelected: config.reviewerWasAutoSelected,
         target: config.target,
         outputDir: config.outputDir,
+        lang: config.lang,
         debug: config.debug
       })}`
     );

package/src/logger.js

@@ -38,6 +38,10 @@ class Logger {
     this._log("INFO", message);
   }
 
+  warn(message) {
+    this._log("WARN", message);
+  }
+
   error(message, error) {
     let msg = message;
     if (error) {
@@ -68,6 +72,7 @@ class Logger {
     // Console output
     const isDebug = level === "DEBUG";
     const isError = level === "ERROR";
+    const isWarn = level === "WARN";
 
     // If it's debug and debug mode is off, skip console
     if (isDebug && !this.config?.debug) return;
@@ -75,10 +80,10 @@ class Logger {
     if (this.progressDisplay) {
       this.progressDisplay.log(logLine);
     } else {
-      if (isError) {
+      if (isError || isWarn) {
         console.error(logLine);
       } else {
-        console.error(logLine);
+        console.log(logLine);
       }
     }
   }

package/src/progress-ui.js

@@ -1,3 +1,5 @@
+import { logger } from "./logger.js";
+
 function clampProgress(value) {
   if (!Number.isFinite(value)) {
     return 0;
@@ -19,6 +21,7 @@ class ProgressItem {
   start(stage = "starting") {
     this.active = true;
     this.stage = stage;
+    logger.debug(`${this.label} batch start: ${stage}`);
     this.writeStatus();
   }
 
@@ -27,21 +30,28 @@ class ProgressItem {
 
     if (stage) {
       this.stage = stage;
+      logger.debug(`${this.label} stage: ${stage} (${Math.round(this.progress * 100)}%)`);
     }
 
     this.writeStatus();
   }
 
   log(message) {
+    // We don't log to file here because usually progress.log() is called alongside logger.info()
+    // or we want the caller to decide whether it goes to the log file.
     this.display.writeLine(message);
   }
 
   succeed(message) {
-    this.finish("[done]", 1, message || `${this.label} complete`);
+    const finalMsg = message || `${this.label} complete`;
+    this.finish("[done]", 1, finalMsg);
+    logger.debug(`${this.label} batch succeed: ${finalMsg}`);
   }
 
   fail(message) {
-    this.finish("[fail]", this.progress, message || `${this.label} failed`);
+    const finalMsg = message || `${this.label} failed`;
+    this.finish("[fail]", this.progress, finalMsg);
+    logger.error(`${this.label} batch fail: ${finalMsg}`);
   }
 
   finish(prefix, progress, message) {

package/src/review-runner.js

@@ -18,6 +18,9 @@ const DIFF_LIMITS = {
   tailLines: 200
 };
 
+const CORE_REVIEW_INSTRUCTION =
+  "Please strictly review the current changes, prioritizing bugs, regression risks, compatibility issues, security concerns, boundary condition flaws, and missing tests. Please use Markdown for your response. If no clear flaws are found, write \"No clear flaws found\" and supplement with residual risks.";
+
 
 function estimateTokenCount(text) {
   if (!text) {
@@ -419,7 +422,13 @@ function buildPrompt(config, backend, targetInfo, details, reviewDiffPayload) {
   const workspaceRoot = getReviewWorkspaceRoot(config, backend, targetInfo);
   const canReadRelatedFiles = backend.kind === "git" || Boolean(targetInfo.workingCopyPath);
 
+  const langInstruction = config.resolvedLang === "zh"
+    ? "Please use Simplified Chinese for your response."
+    : `Please use ${config.resolvedLang || "English"} for your response.`;
+
   return [
+    CORE_REVIEW_INSTRUCTION,
+    langInstruction,
     config.prompt,
     canReadRelatedFiles
       ? `You are running inside a read-only workspace rooted at: ${workspaceRoot}`
@@ -437,7 +446,7 @@ function buildPrompt(config, backend, targetInfo, details, reviewDiffPayload) {
     reviewDiffPayload.wasTruncated
       ? `Diff delivery note: the diff was truncated before being sent to the reviewer to stay within configured size limits. Original diff size was ${reviewDiffPayload.originalLineCount} lines / ${reviewDiffPayload.originalCharCount} chars, and the included excerpt is ${reviewDiffPayload.outputLineCount} lines / ${reviewDiffPayload.outputCharCount} chars. Use the changed file list and inspect related workspace files when needed.`
       : `Diff delivery note: the full diff is included. Size is ${reviewDiffPayload.originalLineCount} lines / ${reviewDiffPayload.originalCharCount} chars.`
-  ].join("\n\n");
+  ].filter(Boolean).join("\n\n");
 }
 
 function formatTokenUsage(tokenUsage) {
@@ -582,6 +591,8 @@ function buildStateSnapshot(backend, targetInfo, changeId) {
 }
 
 async function reviewChange(config, backend, targetInfo, changeId, progress) {
+  const displayId = backend.formatChangeId(changeId);
+  logger.info(`Starting review for ${backend.changeName} ${displayId}`);
   progress?.update(0.05, "loading change details");
   const details = await backend.getChangeDetails(config, targetInfo, changeId);
 
@@ -650,7 +661,8 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
     }
 
     if (reviewerName !== reviewersToTry[reviewersToTry.length - 1]) {
-      progress?.log(`${reviewer.displayName} failed for ${details.displayId}; trying next reviewer...`);
+      const msg = `${reviewer.displayName} failed for ${details.displayId}; trying next reviewer...`;
+      logger.warn(msg);
     }
   }
 
@@ -679,6 +691,13 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
     );
   }
 
+  const outputLabels = [
+    shouldWriteFormat(config, "markdown") ? `md: ${outputFile}` : null,
+    shouldWriteFormat(config, "json") ? `json: ${jsonOutputFile}` : null
+  ].filter(Boolean);
+  
+  logger.info(`Completed review for ${displayId}: ${outputLabels.join(" | ") || "(no report file generated)"}`);
+
   return {
     success: true,
     outputFile: shouldWriteFormat(config, "markdown") ? outputFile : null,
@@ -772,15 +791,10 @@ export async function runReviewCycle(config) {
       throw error;
     }
 
-    const outputLabels = [
-      result.outputFile ? `md: ${result.outputFile}` : null,
-      result.jsonOutputFile ? `json: ${result.jsonOutputFile}` : null
-    ].filter(Boolean);
     const nextProjectState = buildStateSnapshot(backend, targetInfo, changeId);
     await saveState(config.stateFilePath, updateProjectState(stateFile, targetInfo, nextProjectState));
     stateFile.projects[targetInfo.stateKey] = nextProjectState;
     logger.debug(`Saved checkpoint for ${backend.formatChangeId(changeId)} to ${config.stateFilePath}.`);
-    progress.log(`[done] reviewed ${displayId}: ${outputLabels.join(" | ") || "(no report file generated)"}`);
     updateOverallProgress(progress, index + 1, changeIdsToReview.length, 0, `finished ${displayId}`);
   }
 

package/src/shell.js

@@ -25,7 +25,9 @@ export async function runCommand(command, args = [], options = {}) {
   } = options;
 
   logger.debug(
-    `run: ${command} ${args.join(" ")}${cwd ? ` | cwd=${cwd}` : ""}${timeoutMs > 0 ? ` | timeoutMs=${timeoutMs}` : ""}`
+    `run: ${command} ${args.join(" ")}${cwd ? ` | cwd=${cwd}` : ""}${timeoutMs > 0 ? ` | timeoutMs=${timeoutMs}` : ""}${
+      input ? ` | input=${summarizeOutput(input)}` : ""
+    }`
   );
 
   return await new Promise((resolve, reject) => {
@@ -51,7 +53,10 @@ export async function runCommand(command, args = [], options = {}) {
       stderrChunks.push(Buffer.from(chunk));
     });
 
-    child.on("error", reject);
+    child.on("error", (err) => {
+      logger.error(`spawn error: ${command}`, err);
+      reject(err);
+    });
 
     child.on("close", (code) => {
       if (timer) {
@@ -67,9 +72,14 @@ export async function runCommand(command, args = [], options = {}) {
         stderr: trim ? stderr.trim() : stderr
       };
 
-      logger.debug(
-        `exit: ${command} code=${result.code} timedOut=${result.timedOut} stdout=${summarizeOutput(result.stdout)} stderr=${summarizeOutput(result.stderr)}`
-      );
+      const level = (result.code !== 0 || result.timedOut) && !allowFailure ? "ERROR" : "DEBUG";
+      const exitMsg = `exit: ${command} code=${result.code} timedOut=${result.timedOut} stdout=${summarizeOutput(result.stdout)} stderr=${summarizeOutput(result.stderr)}`;
+      
+      if (level === "ERROR") {
+        logger.error(exitMsg);
+      } else {
+        logger.debug(exitMsg);
+      }
 
       if ((result.code !== 0 || result.timedOut) && !allowFailure) {
         const error = new Error(