kodevu 0.1.31 → 0.1.33

package/README.md

@@ -2,98 +2,88 @@
 
 A Node.js tool that fetches Git commits or SVN revisions, sends the diff to a supported AI reviewer CLI, and writes review results to report files.
 
-## Workflow
+## Pure & Zero Config
 
-1. Detect the repository type automatically (Git or SVN).
-2. Fetch the specified revision(s) as requested by the user:
-   - A single specific revision/commit via `--rev`.
-   - The latest $N$ revisions/commits via `--last` (default: 1).
-3. For each change:
-   - Load metadata and changed paths from SVN or Git.
-   - Generate a unified diff for that single revision or commit.
-   - Send the diff and change metadata to the configured reviewer CLI.
-   - Allow the reviewer to inspect related local repository files in read-only mode when a local workspace is available.
-   - Write the result to `~/.kodevu/` (Markdown by default; optional JSON via config).
+Kodevu is designed to be stateless and requires no configuration files. It relies entirely on command-line arguments and environment variables.
 
-**Note**: Kodevu is stateless. It does not track which changes have been reviewed previously.
+1. **Automatic Detection**: Detects repository type (Git/SVN), language, and available reviewers.
+2. **Stateless**: Does not track history; reviews exactly what you ask for.
+3. **Flexible**: Every setting can be overridden via CLI flags or ENV vars.
 
-## Quick start
+## Quick Start
 
 Review the latest commit in your repository:
 
 ```bash
-npx kodevu /path/to/your/repo
+npx kodevu .
 ```
 
 Review the latest 3 commits:
 
 ```bash
-npx kodevu /path/to/your/repo --last 3
+npx kodevu . --last 3
 ```
 
 Review a specific commit:
 
 ```bash
-npx kodevu /path/to/your/repo --rev abc1234
+npx kodevu . --rev abc1234
 ```
 
-Review reports are written to `~/.kodevu/` as Markdown (`.md`) by default.
+Reports are written to `~/.kodevu/` by default.
 
-## Setup
-
-If you want to customize settings beyond the defaults:
+## Usage
 
 ```bash
-npx kodevu init
+npx kodevu [target] [options]
 ```
 
-This creates `config.json` in the current directory. You only need this when you want to override defaults such as `reviewer` or output paths.
+### Options
 
-## Run
+- `target`: Repository path (Git) or SVN URL/Working copy (default: `.`).
+- `--reviewer, -r`: `codex`, `gemini`, `copilot`, or `auto` (default: `auto`).
+- `--rev, -v`: A specific revision or commit hash to review.
+- `--last, -n`: Number of latest revisions to review (default: 1).
+- `--lang, -l`: Output language (e.g., `zh`, `en`, `auto`).
+- `--prompt, -p`: Additional instructions for the reviewer. Use `@file.txt` to read from a file.
+- `--output, -o`: Report output directory (default: `~/.kodevu`).
+- `--format, -f`: Output formats (e.g., `markdown`, `json`, or `markdown,json`).
+- `--debug, -d`: Print debug information.
 
-Specify the output language (e.g., Chinese):
+### Environment Variables
 
-```bash
-npx kodevu /path/to/your/repo --lang zh
-```
+You can set these in your shell to change default behavior without typing flags every time:
 
-Run with debug logs:
+- `KODEVU_REVIEWER`: Default reviewer.
+- `KODEVU_LANG`: Default language.
+- `KODEVU_OUTPUT_DIR`: Default output directory.
+- `KODEVU_PROMPT`: Default prompt instructions.
+- `KODEVU_TIMEOUT`: Reviewer execution timeout in milliseconds.
 
+## Examples
+
+**Review with a custom prompt from a file:**
 ```bash
-npx kodevu /path/to/your/repo --debug
+npx kodevu . --prompt @my-rules.txt
 ```
 
-Use a custom config path:
-
+**Generate JSON reports in a local folder:**
 ```bash
-npx kodevu --config ./my-config.json
+npx kodevu . --format json --output ./review-reports
 ```
 
-## Config / CLI Options
-
-- `target`: Repository target path or SVN URL. (CLI positional argument overrides config)
-- `reviewer`: `codex`, `gemini`, `copilot`, or `auto` (default: `auto`).
-- `rev`: A specific revision or commit hash to review.
-- `last`: Number of latest revisions to review (default: 1 if `rev` is not set).
-- `lang`: Output language for the review (e.g., `zh`, `en`, `auto`).
-- `prompt`: Additional instructions for the reviewer.
-- `outputDir`: Report output directory (default: `~/.kodevu`).
-- `outputFormats`: Report formats to generate (supports `markdown` and `json`; default: `["markdown"]`).
-- `commandTimeoutMs`: Timeout for a single review command execution in milliseconds.
-- `maxRevisionsPerRun`: Cap the number of revisions handled in one run.
-
-## Target Rules
-
-- For SVN, `target` can be a working copy path or repository URL.
-- For Git, `target` must be a local repository path or a subdirectory inside a local repository.
-- The tool tries Git first, then falls back to SVN.
+**Set a persistent reviewer via ENV:**
+```bash
+export KODEVU_REVIEWER=gemini
+npx kodevu .
+```
 
-## Notes
+## How it Works
 
-- `reviewer: "auto"` probes `codex`, `gemini`, and `copilot` in `PATH`, then selects an available one.
-- Large diffs are truncated based on internal limits to fit AI context windows.
-- 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.
-- If the reviewer command exits non-zero or times out, the report is still written containing the error details.
+- **Git Targets**: `target` must be a local repository or subdirectory.
+- **SVN Targets**: `target` can be a working copy path or repository URL.
+- **Reviewer "auto"**: Probes `codex`, `gemini`, and `copilot` in your `PATH` and selects one.
+- **Contextual Review**: For local repositories, the reviewer can inspect related files beyond the diff to provide deeper insights.
 
 ## License
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.31",
+  "version": "0.1.33",
   "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,7 +6,6 @@ import { findCommandOnPath } from "./shell.js";
 const defaultStorageDir = path.join(os.homedir(), ".kodevu");
 const SUPPORTED_REVIEWERS = ["codex", "gemini", "copilot"];
 
-
 const defaultConfig = {
   reviewer: "auto",
   target: "",
@@ -21,61 +20,36 @@ const defaultConfig = {
   last: 0
 };
 
-const configTemplate = {
-  target: "C:/path/to/your/repository-or-subdirectory",
-  reviewer: defaultConfig.reviewer,
-  lang: defaultConfig.lang,
-  prompt: defaultConfig.prompt,
-  outputDir: "~/.kodevu",
-  logsDir: "~/.kodevu/logs",
-  commandTimeoutMs: defaultConfig.commandTimeoutMs,
-  maxRevisionsPerRun: defaultConfig.maxRevisionsPerRun,
-  outputFormats: defaultConfig.outputFormats,
-  rev: "",
-  last: 1
+const ENV_MAP = {
+  KODEVU_REVIEWER: "reviewer",
+  KODEVU_LANG: "lang",
+  KODEVU_OUTPUT_DIR: "outputDir",
+  KODEVU_PROMPT: "prompt",
+  KODEVU_TIMEOUT: "commandTimeoutMs",
+  KODEVU_MAX_REVISIONS: "maxRevisionsPerRun",
+  KODEVU_FORMATS: "outputFormats"
 };
 
-function resolveConfigPath(baseDir, value) {
-  if (!value) {
-    return value;
-  }
-
-  if (typeof value !== "string") {
-    return path.resolve(baseDir, String(value));
-  }
-
-  if (value === "~") {
-    return os.homedir();
-  }
-
+function resolvePath(value) {
+  if (!value) return value;
+  if (value === "~") return os.homedir();
   if (value.startsWith("~/") || value.startsWith("~\\")) {
     return path.join(os.homedir(), value.slice(2));
   }
-
-  return path.isAbsolute(value) ? value : path.resolve(baseDir, value);
+  return path.isAbsolute(value) ? value : path.resolve(process.cwd(), value);
 }
 
-
-function normalizeOutputFormats(outputFormats, loadedConfigPath) {
+function normalizeOutputFormats(outputFormats) {
   const source = outputFormats == null ? ["markdown"] : outputFormats;
-  const values = Array.isArray(source) ? source : [source];
+  const values = Array.isArray(source) ? source : String(source).split(",");
   const normalized = [...new Set(values.map((item) => String(item || "").trim().toLowerCase()).filter(Boolean))];
   const supported = ["markdown", "json"];
   const invalid = normalized.filter((item) => !supported.includes(item));
 
   if (invalid.length > 0) {
-    throw new Error(
-      `"outputFormats" contains unsupported value(s): ${invalid.join(", ")}. Use any of: ${supported.join(", ")}${
-        loadedConfigPath ? ` in ${loadedConfigPath}` : ""
-      }`
-    );
-  }
-
-  if (normalized.length === 0) {
-    throw new Error(`"outputFormats" must include at least one format${loadedConfigPath ? ` in ${loadedConfigPath}` : ""}`);
+    throw new Error(`Unsupported output format(s): ${invalid.join(", ")}. Use: ${supported.join(", ")}`);
   }
-
-  return normalized;
+  return normalized.length === 0 ? ["markdown"] : normalized;
 }
 
 function detectLanguage() {
@@ -88,46 +62,28 @@ function detectLanguage() {
     }
   })();
 
-  // 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];
-  }
-
+  if (os.platform() === "win32" && intlLocale.startsWith("zh")) return "zh";
+  if (envLang.startsWith("zh")) return "zh";
+  if (envLang.startsWith("en")) return "en";
+  if (envLang) return envLang.split(/[._-]/)[0];
+  if (intlLocale.startsWith("zh")) return "zh";
+  if (intlLocale.startsWith("en")) return "en";
+  if (intlLocale) return intlLocale.split("-")[0];
   return "en";
 }
 
-async function resolveAutoReviewers(debug, loadedConfigPath) {
+async function resolveAutoReviewers(debug) {
   const availableReviewers = [];
-
   for (const reviewerName of SUPPORTED_REVIEWERS) {
     const commandPath = await findCommandOnPath(reviewerName, { debug });
-    if (commandPath) {
-      availableReviewers.push({ reviewerName, commandPath });
-    }
+    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})` : ""
-      }`
-    );
+    throw new Error(`No reviewer CLI found in PATH. Install one of: ${SUPPORTED_REVIEWERS.join(", ")}`);
   }
 
+  // Shuffle for variety
   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]];
@@ -138,9 +94,6 @@ async function resolveAutoReviewers(debug, loadedConfigPath) {
 
 export function parseCliArgs(argv) {
   const args = {
-    command: "run",
-    configPath: "config.json",
-    configExplicitlySet: false,
     target: "",
     debug: false,
     help: false,
@@ -149,18 +102,13 @@ export function parseCliArgs(argv) {
     prompt: "",
     rev: "",
     last: "",
-    commandExplicitlySet: false
+    outputDir: "",
+    outputFormats: ""
   };
 
   for (let index = 0; index < argv.length; index += 1) {
     const value = argv[index];
 
-    if (value === "init" && !args.commandExplicitlySet && index === 0) {
-      args.command = "init";
-      args.commandExplicitlySet = true;
-      continue;
-    }
-
     if (value === "--help" || value === "-h") {
       args.help = true;
       continue;
@@ -171,68 +119,59 @@ export function parseCliArgs(argv) {
       continue;
     }
 
-    if (value === "--config" || value === "-c") {
-      const configPath = argv[index + 1];
-      if (!configPath || configPath.startsWith("-")) {
-        throw new Error(`Missing value for ${value}`);
-      }
-      args.configPath = configPath;
-      args.configExplicitlySet = true;
-      index += 1;
-      continue;
-    }
+    const nextValue = argv[index + 1];
+    const hasNextValue = nextValue && !nextValue.startsWith("-");
 
     if (value === "--reviewer" || value === "-r") {
-      const reviewer = argv[index + 1];
-      if (!reviewer || reviewer.startsWith("-")) {
-        throw new Error(`Missing value for ${value}`);
-      }
-      args.reviewer = reviewer;
+      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      args.reviewer = nextValue;
       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;
+      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      args.prompt = nextValue;
       index += 1;
       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;
+      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      args.lang = nextValue;
       index += 1;
       continue;
     }
 
     if (value === "--rev" || value === "-v") {
-      const rev = argv[index + 1];
-      if (!rev || rev.startsWith("-")) {
-        throw new Error(`Missing value for ${value}`);
-      }
-      args.rev = rev;
+      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      args.rev = nextValue;
       index += 1;
       continue;
     }
 
     if (value === "--last" || value === "-n") {
-      const last = argv[index + 1];
-      if (!last || last.startsWith("-")) {
-        throw new Error(`Missing value for ${value}`);
-      }
-      args.last = last;
+      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      args.last = nextValue;
+      index += 1;
+      continue;
+    }
+
+    if (value === "--output" || value === "-o") {
+      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      args.outputDir = nextValue;
       index += 1;
       continue;
     }
 
-    if (!value.startsWith("-") && args.command === "run" && !args.target) {
+    if (value === "--format" || value === "-f") {
+      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      args.outputFormats = nextValue;
+      index += 1;
+      continue;
+    }
+
+    if (!value.startsWith("-") && !args.target) {
       args.target = value;
       continue;
     }
@@ -240,100 +179,78 @@ export function parseCliArgs(argv) {
     throw new Error(`Unexpected argument: ${value}`);
   }
 
-  delete args.commandExplicitlySet;
   return args;
 }
 
-export async function loadConfig(configPath, cliArgs = {}) {
-  const absoluteConfigPath = path.resolve(configPath);
-  let loadedConfig = {};
-  let loadedConfigPath = null;
-  let baseDir = process.cwd();
-
-  try {
-    const raw = await fs.readFile(absoluteConfigPath, "utf8");
-    loadedConfig = JSON.parse(raw);
-    loadedConfigPath = absoluteConfigPath;
-    baseDir = path.dirname(absoluteConfigPath);
-  } catch (error) {
-    if (!(error?.code === "ENOENT" && !cliArgs.configExplicitlySet)) {
-      throw error;
+export async function resolveConfig(cliArgs = {}) {
+  const config = { ...defaultConfig };
+
+  // 1. Merge Environment Variables
+  for (const [envVar, configKey] of Object.entries(ENV_MAP)) {
+    if (process.env[envVar] !== undefined) {
+      config[configKey] = process.env[envVar];
     }
   }
 
-  const config = {
-    ...defaultConfig,
-    ...loadedConfig
+  // 2. Merge CLI Arguments
+  const cliMapping = {
+    target: "target",
+    reviewer: "reviewer",
+    prompt: "prompt",
+    lang: "lang",
+    rev: "rev",
+    last: "last",
+    outputDir: "outputDir",
+    outputFormats: "outputFormats"
   };
 
-  if (cliArgs.target) {
-    config.target = cliArgs.target;
-  }
-
-  if (cliArgs.reviewer) {
-    config.reviewer = cliArgs.reviewer;
-  }
-
-  if (cliArgs.prompt) {
-    config.prompt = cliArgs.prompt;
-  }
-
-  if (cliArgs.lang) {
-    config.lang = cliArgs.lang;
-  }
-
-  if (cliArgs.rev) {
-    config.rev = cliArgs.rev;
-  }
-
-  if (cliArgs.last) {
-    config.last = cliArgs.last;
+  for (const [cliKey, configKey] of Object.entries(cliMapping)) {
+    if (cliArgs[cliKey]) {
+      config[configKey] = cliArgs[cliKey];
+    }
   }
 
   if (!config.target) {
-    throw new Error('Missing target. Pass `npx kodevu <repo-path>` or set "target" in config.json.');
+    config.target = process.cwd();
   }
 
   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;
 
+  // Handle @file prompt
+  if (config.prompt.startsWith("@")) {
+    const promptPath = resolvePath(config.prompt.slice(1));
+    try {
+      config.prompt = await fs.readFile(promptPath, "utf8");
+    } catch (err) {
+      throw new Error(`Failed to read prompt file: ${promptPath} (${err.message})`);
+    }
+  }
+
   if (config.reviewer === "auto") {
-    const availableReviewers = await resolveAutoReviewers(config.debug, loadedConfigPath);
+    const availableReviewers = await resolveAutoReviewers(config.debug);
     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(
-      `"reviewer" must be one of "codex", "gemini", "copilot", or "auto"${loadedConfigPath ? ` in ${loadedConfigPath}` : ""}`
-    );
+    throw new Error(`"reviewer" must be one of: ${SUPPORTED_REVIEWERS.join(", ")}, or "auto"`);
   }
 
-  config.configPath = loadedConfigPath;
-  config.baseDir = baseDir;
-  config.outputDir = resolveConfigPath(config.baseDir, config.outputDir);
-  config.logsDir = resolveConfigPath(config.baseDir, config.logsDir);
+  config.outputDir = resolvePath(config.outputDir);
+  config.logsDir = path.join(config.outputDir, "logs");
   config.maxRevisionsPerRun = Number(config.maxRevisionsPerRun);
   config.commandTimeoutMs = Number(config.commandTimeoutMs);
   config.last = Number(config.last);
-  config.outputFormats = normalizeOutputFormats(config.outputFormats, loadedConfigPath);
+  config.outputFormats = normalizeOutputFormats(config.outputFormats);
 
   if (!config.rev && (isNaN(config.last) || config.last <= 0)) {
     config.last = 1;
   }
 
-  if (!Number.isInteger(config.maxRevisionsPerRun) || config.maxRevisionsPerRun <= 0) {
-    throw new Error(`"maxRevisionsPerRun" must be a positive integer${loadedConfigPath ? ` in ${loadedConfigPath}` : ""}`);
-  }
-
-  if (!Number.isInteger(config.commandTimeoutMs) || config.commandTimeoutMs <= 0) {
-    throw new Error(`"commandTimeoutMs" must be a positive integer${loadedConfigPath ? ` in ${loadedConfigPath}` : ""}`);
-  }
-
   return config;
 }
 
@@ -341,44 +258,25 @@ export function printHelp() {
   console.log(`Kodevu
 
 Usage:
-  kodevu init
-  npx kodevu init
-  kodevu <target> [--debug]
-  npx kodevu <target> [--debug]
-  kodevu [--config config.json]
-  npx kodevu [--config config.json]
+  npx kodevu [target] [options]
 
 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)
-  --rev, -v      Review a specific revision or commit hash
-  --last, -n     Review the latest N revisions (default: 1)
-  --debug, -d    Print extra debug information to the console
-  --help, -h     Show help
-
-Config highlights:
-  reviewer       codex | gemini | copilot | auto
-  target         Repository target path (Git) or SVN working copy / URL; CLI positional target overrides config
-  outputFormats  ["markdown"] by default; set to include "json" when needed
+  --target, <path>  Target repository path (default: current directory)
+  --reviewer, -r    Reviewer (codex | gemini | copilot | auto, default: auto)
+  --prompt, -p      Additional instructions or @file.txt to read from file
+  --lang, -l        Output language (e.g. zh, en, auto)
+  --rev, -v         Review a specific revision or commit hash
+  --last, -n        Review the latest N revisions (default: 1)
+  --output, -o      Output directory (default: ~/.kodevu)
+  --format, -f      Output formats (markdown, json, comma-separated)
+  --debug, -d       Print extra debug information
+  --help, -h        Show help
+
+Environment Variables:
+  KODEVU_REVIEWER   Default reviewer
+  KODEVU_LANG       Default language
+  KODEVU_OUTPUT_DIR Default output directory
+  KODEVU_PROMPT     Default prompt text
+  KODEVU_TIMEOUT    Reviewer timeout in ms
 `);
 }
-
-export async function initConfig(targetPath = "config.json") {
-  const absoluteTargetPath = path.resolve(targetPath);
-
-  await fs.mkdir(path.dirname(absoluteTargetPath), { recursive: true });
-
-  const content = JSON.stringify(configTemplate, null, 2) + "\n";
-  try {
-    await fs.writeFile(absoluteTargetPath, content, { flag: "wx" });
-  } catch (error) {
-    if (error?.code === "EEXIST") {
-      throw new Error(`Config file already exists: ${absoluteTargetPath}`);
-    }
-    throw error;
-  }
-
-  return absoluteTargetPath;
-}

package/src/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-import { initConfig, loadConfig, parseCliArgs, printHelp } from "./config.js";
+import { resolveConfig, parseCliArgs, printHelp } from "./config.js";
 import { runReviewCycle } from "./review-runner.js";
 import { logger } from "./logger.js";
 
@@ -19,19 +19,8 @@ if (cliArgs.help) {
   process.exit(0);
 }
 
-if (cliArgs.command === "init") {
-  try {
-    const createdPath = await initConfig(cliArgs.configPath);
-    console.log(`Created config: ${createdPath}`);
-    process.exit(0);
-  } catch (error) {
-    console.error(error?.stack || String(error));
-    process.exit(1);
-  }
-}
-
 try {
-  const config = await loadConfig(cliArgs.configPath, cliArgs);
+  const config = await resolveConfig(cliArgs);
   logger.init(config);
 
   if (config.reviewerWasAutoSelected) {
@@ -42,8 +31,7 @@ try {
 
   if (config.debug) {
     logger.debug(
-      `Loaded config: ${JSON.stringify({
-        configPath: config.configPath,
+      `Resolved config: ${JSON.stringify({
         reviewer: config.reviewer,
         reviewerCommandPath: config.reviewerCommandPath,
         reviewerWasAutoSelected: config.reviewerWasAutoSelected,
@@ -59,7 +47,6 @@ try {
   await runReviewCycle(config);
   logger.info("Session completed successfully.");
 } catch (error) {
-  // If config was loaded, logger might be initialized, otherwise it will fall back to stderr
   logger.error("Session failed with error", error);
   process.exitCode = 1;
 }

package/src/report-generator.js

@@ -1,5 +1,13 @@
-export 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.";
+export function getCoreReviewInstruction(lang) {
+  const lowLang = (lang || "").toLowerCase();
+  if (lowLang.startsWith("zh")) {
+    if (lowLang === "zh-tw" || lowLang === "zh-hk") {
+      return "請嚴格審查目前的更改，優先考慮錯誤、回歸風險、相容性問題、安全問題、邊界條件缺陷和缺失的測試。請使用 Markdown 進行回覆。如果未發現明顯缺陷，請寫「未發現明顯缺陷」並補充風險。";
+    }
+    return "请严格审查当前的更改，优先处理 Bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失的测试。请使用 Markdown 进行回复。如果未发现明显缺陷，请写“未发现明显缺陷”并补充残留风险。";
+  }
+  return "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.";
+}
 
 export function getReviewWorkspaceRoot(config, backend, targetInfo) {
   if (backend.kind === "git" && targetInfo.repoRootPath) {
@@ -30,13 +38,77 @@ function getLanguageDisplayName(lang) {
   return lang;
 }
 
+const LOCALIZED_PHRASES = {
+  en: {
+    workspaceRoot: "You are running inside a read-only workspace rooted at:",
+    noWorkspace: "No local repository workspace is available for this review run.",
+    besidesDiff: "Besides the diff below, you may read other related files in the workspace when needed to understand call sites, shared utilities, configuration, tests, or data flow. Do not modify files or rely on shell commands.",
+    reviewFromDiff: "Review primarily from the diff below. Do not assume access to other local files, shell commands, or repository history.",
+    fileRefs: "Use plain text file references like path/to/file.js:123. Do not emit clickable workspace links.",
+    repoType: "Repository Type",
+    changeId: "Change ID",
+    author: "Author",
+    date: "Date",
+    changedFiles: "Changed files",
+    commitMessage: "Commit message",
+    diffNoteTruncated: "Diff delivery note: the diff was truncated before being sent to the reviewer to stay within configured size limits. Original diff size was {originalLineCount} lines / {originalCharCount} chars, and the included excerpt is {outputLineCount} lines / {outputCharCount} chars. Use the changed file list and inspect related workspace files when needed.",
+    diffNoteFull: "Diff delivery note: the full diff is included. Size is {originalLineCount} lines / {originalCharCount} chars.",
+    langRule: "--- IMPORTANT LANGUAGE RULE ---\nYou MUST respond strictly in {langName}. No other language should be used for the explanation and comments."
+  },
+  zh: {
+    workspaceRoot: "你正运行在一个只读工作区内，根目录为：",
+    noWorkspace: "此审查运行没有可用的本地仓库工作区。",
+    besidesDiff: "除了下面的 Diff，你可以在需要时阅读工作区中的其他相关文件，以了解调用点、共享工具、配置、测试或数据流。请勿修改文件或依赖 Shell 命令。",
+    reviewFromDiff: "主要根据下面的 Diff 进行审查。不要假设可以访问其他本地文件、Shell 命令或仓库历史。",
+    fileRefs: "使用纯文本文件引用，如 path/to/file.js:123。不要生成可点击的工作区链接。",
+    repoType: "仓库类型",
+    changeId: "变更 ID",
+    author: "作者",
+    date: "日期",
+    changedFiles: "已变更文件",
+    commitMessage: "提交信息",
+    diffNoteTruncated: "Diff 交付说明：Diff 在发送给审查者之前已被截斷，以保持在配置的大小限制内。原始 Diff 大小为 {originalLineCount} 行 / {originalCharCount} 个字符，包含的摘录为 {outputLineCount} 行 / {outputCharCount} 个字符。在需要时使用已更正文件列表并检查相关工作区文件。",
+    diffNoteFull: "Diff 交付说明：包含完整的 Diff。大小为 {originalLineCount} 行 / {originalCharCount} 个字符。",
+    langRule: "--- 重要语言规则 ---\n你必须严格使用 {langName} 进行回复。解释、评论和总结均不得使用其他语言。"
+  },
+  "zh-tw": {
+    workspaceRoot: "你正運行在一個唯讀工作區內，根目錄為：",
+    noWorkspace: "此審查運行沒有可用的本地倉庫工作區。",
+    besidesDiff: "除了下面的 Diff，你可以在需要時閱讀工作區中的其他相關文件，以了解調用點、共享工具、配置、測試或資料流。請勿修改文件或依賴 Shell 命令。",
+    reviewFromDiff: "主要根據下面的 Diff 進行審查。不要假設可以訪問其他本地文件、Shell 命令或倉庫歷史。",
+    fileRefs: "使用純文本文件引用，如 path/to/file.js:123。不要生成可點擊的工作區連結。",
+    repoType: "倉庫類型",
+    changeId: "變更 ID",
+    author: "作者",
+    date: "日期",
+    changedFiles: "已變更文件",
+    commitMessage: "提交信息",
+    diffNoteTruncated: "Diff 交付說明：Diff 在傳送給審查者之前已被截斷，以保持在配置的大小限制內。原始 Diff 大小為 {originalLineCount} 行 / {originalCharCount} 個字符，包含的摘錄為 {outputLineCount} 行 / {outputCharCount} 個字符。在需要時使用已更正文件列表並檢查相關工作區文件。",
+    diffNoteFull: "Diff 交付說明：包含完整的 Diff。大小為 {originalLineCount} 行 / {originalCharCount} 個字符。",
+    langRule: "--- 重要語言規則 ---\n你必須嚴格使用 {langName} 進行回覆。解釋、評論和總結均不得使用其他語言。"
+  }
+};
+
+function getPhrase(key, lang, placeholders = {}) {
+  const lowLang = (lang || "en").toLowerCase();
+  const langKey = lowLang.startsWith("zh") ? (lowLang === "zh-tw" || lowLang === "zh-hk" ? "zh-tw" : "zh") : "en";
+  let phrase = LOCALIZED_PHRASES[langKey][key] || LOCALIZED_PHRASES.en[key];
+
+  for (const [k, v] of Object.entries(placeholders)) {
+    phrase = phrase.replace(`{${k}}`, v);
+  }
+  return phrase;
+}
+
 export function buildPrompt(config, backend, targetInfo, details, reviewDiffPayload) {
   const fileList = details.changedPaths.map((item) => `${item.action} ${item.relativePath}`).join("\n");
   const workspaceRoot = getReviewWorkspaceRoot(config, backend, targetInfo);
   const canReadRelatedFiles = backend.kind === "git" || Boolean(targetInfo.workingCopyPath);
 
-  const langName = getLanguageDisplayName(config.resolvedLang);
-  const lowLang = (config.resolvedLang || "").toLowerCase();
+  const lang = config.resolvedLang || "en";
+  const langName = getLanguageDisplayName(lang);
+  const lowLang = lang.toLowerCase();
+
   let langInstruction = `IMPORTANT: Your entire response MUST be in ${langName}. All explanations, comments, and structure should strictly follow the ${langName} language rules.`;
 
   if (lowLang.startsWith("zh")) {
@@ -47,39 +119,41 @@ export function buildPrompt(config, backend, targetInfo, details, reviewDiffPayl
     }
   }
 
-
-
   return [
-    CORE_REVIEW_INSTRUCTION,
     langInstruction,
+    getCoreReviewInstruction(lang),
     config.prompt,
     canReadRelatedFiles
-      ? `You are running inside a read-only workspace rooted at: ${workspaceRoot}`
-      : "No local repository workspace is available for this review run.",
+      ? `${getPhrase("workspaceRoot", lang)} ${workspaceRoot}`
+      : getPhrase("noWorkspace", lang),
     canReadRelatedFiles
-      ? "Besides the diff below, you may read other related files in the workspace when needed to understand call sites, shared utilities, configuration, tests, or data flow. Do not modify files or rely on shell commands."
-      : "Review primarily from the diff below. Do not assume access to other local files, shell commands, or repository history.",
-    "Use plain text file references like path/to/file.js:123. Do not emit clickable workspace links.",
-    `Repository Type: ${backend.displayName}`,
-    `Change ID: ${details.displayId}`,
-    `Author: ${details.author}`,
-    `Date: ${details.date || "unknown"}`,
-    `Changed files:\n${fileList || "(none)"}`,
-    `Commit message:\n${details.message || "(empty)"}`,
+      ? getPhrase("besidesDiff", lang)
+      : getPhrase("reviewFromDiff", lang),
+    getPhrase("fileRefs", lang),
+    `${getPhrase("repoType", lang)}: ${backend.displayName}`,
+    `${getPhrase("changeId", lang)}: ${details.displayId}`,
+    `${getPhrase("author", lang)}: ${details.author}`,
+    `${getPhrase("date", lang)}: ${details.date || "unknown"}`,
+    `${getPhrase("changedFiles", lang)}:\n${fileList || "(none)"}`,
+    `${getPhrase("commitMessage", lang)}:\n${details.message || "(empty)"}`,
     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.`,
-    `--- IMPORTANT LANGUAGE RULE ---\nYou MUST respond strictly in ${langName}. No other language should be used for the explanation and comments.${
-      lowLang.startsWith("zh")
+      ? getPhrase("diffNoteTruncated", lang, {
+          originalLineCount: reviewDiffPayload.originalLineCount,
+          originalCharCount: reviewDiffPayload.originalCharCount,
+          outputLineCount: reviewDiffPayload.outputLineCount,
+          outputCharCount: reviewDiffPayload.outputCharCount
+        })
+      : getPhrase("diffNoteFull", lang, {
+          originalLineCount: reviewDiffPayload.originalLineCount,
+          originalCharCount: reviewDiffPayload.originalCharCount
+        }),
+    getPhrase("langRule", lang, { langName }) +
+      (lowLang.startsWith("zh")
         ? lowLang === "zh-tw" || lowLang === "zh-hk"
           ? "\n請務必完全使用繁體中文進行回覆，所有的審查分析、注釋和總結都必須使用繁體中文。"
           : "\n请务必完全使用简体中文进行回复，所有的审查分析、注释和总结都必须使用简体中文。"
-        : ""
-    }`
-
+        : "")
   ].filter(Boolean).join("\n\n");
-
-
 }
 
 export function formatTokenUsage(tokenUsage) {