kodevu 0.1.57 → 0.1.59

package/README.md

@@ -6,11 +6,11 @@ A Node.js tool that fetches Git commits or SVN revisions, sends the diff to a su
 
 ## Pure & Zero Config
 
-Kodevu is designed to be stateless and requires no configuration files. It relies entirely on command-line arguments and environment variables.
+Kodevu is designed to be stateless and requires no mandatory configuration. All settings work out-of-the-box via command-line arguments and environment variables, with an optional persistent config file for convenience.
 
 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.
+3. **Flexible**: Every setting can be set via config file, ENV var, or CLI flag, with CLI taking highest priority.
 
 ## Quick Start
 
@@ -69,6 +69,28 @@ You can set these in your shell to change default behavior without typing flags
 - `KODEVU_OPENAI_ORG`: Optional organization ID for `openai`.
 - `KODEVU_OPENAI_PROJECT`: Optional project ID for `openai`.
 
+### Configuration File
+
+For persistent settings that survive across shells and AI tools, create `~/.kodevu/config.json`:
+
+```json
+{
+  "reviewer": "openai",
+  "openaiApiKey": "sk-...",
+  "openaiBaseUrl": "https://your-gateway.example.com/v1",
+  "openaiModel": "gpt-4o",
+  "lang": "zh"
+}
+```
+
+The file is optional and silently ignored if absent. **Priority order** (highest wins):
+
+```
+CLI flags  >  Environment variables  >  Config file  >  Built-in defaults
+```
+
+Supported keys: `reviewer`, `lang`, `outputDir`, `prompt`, `commandTimeoutMs`, `outputFormats`, `openaiApiKey`, `openaiBaseUrl`, `openaiModel`, `openaiOrganization`, `openaiProject`.
+
 ## Examples
 
 ### Selecting Revisions

package/SKILL.md

@@ -5,7 +5,7 @@ description: A tool to fetch Git/SVN diffs, send them to an AI reviewer, and gen
 
 # Kodevu Skill
 
-Kodevu is 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. It is designed to be **stateless** and requires **no configuration files**.
+Kodevu is 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. It supports an optional persistent config file at `~/.kodevu/config.json` for settings that should survive across sessions.
 
 ## Usage
 
@@ -71,6 +71,22 @@ All options can also be set via environment variables to avoid repetitive flags:
 - `KODEVU_OPENAI_BASE_URL` – Base URL for `openai`.
 - `KODEVU_OPENAI_MODEL` – Model for `openai`.
 
+### Configuration File
+
+For persistent settings that survive across shells and AI tool invocations, create `~/.kodevu/config.json`:
+
+```json
+{
+  "reviewer": "openai",
+  "openaiApiKey": "sk-...",
+  "openaiBaseUrl": "https://your-gateway.example.com/v1",
+  "openaiModel": "gpt-4o",
+  "lang": "zh"
+}
+```
+
+The file is optional and silently ignored if absent. Priority: **CLI flags > ENV vars > config file > defaults**.
+
 ## Working with Target Repositories
 
 - **Git**: `target` must be a local repository or subdirectory.

package/package.json

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

@@ -17,7 +17,7 @@ const defaultConfig = {
   lang: "auto",
   outputDir: defaultStorageDir,
   logsDir: path.join(defaultStorageDir, "logs"),
-  commandTimeoutMs: 600000,
+  commandTimeoutMs: 120000,
   prompt: "",
   maxRevisionsPerRun: 5,
   outputFormats: ["markdown"],
@@ -46,6 +46,9 @@ const ENV_MAP = {
   KODEVU_OPENAI_PROJECT: "openaiProject"
 };
 
+const CONFIG_FILE_KEYS = new Set(Object.values(ENV_MAP));
+const defaultConfigFilePath = path.join(defaultStorageDir, "config.json");
+
 function resolvePath(value) {
   if (!value) return value;
   if (value === "~") return os.homedir();
@@ -55,6 +58,33 @@ function resolvePath(value) {
   return path.isAbsolute(value) ? value : path.resolve(process.cwd(), value);
 }
 
+async function loadConfigFile(configPath = defaultConfigFilePath) {
+  const resolvedPath = resolvePath(configPath);
+  let content;
+  try {
+    content = await fs.readFile(resolvedPath, "utf8");
+  } catch (err) {
+    if (err.code === "ENOENT") return {};
+    throw new Error(`Failed to read config file ${resolvedPath}: ${err.message}`);
+  }
+  let parsed;
+  try {
+    parsed = JSON.parse(content);
+  } catch (err) {
+    throw new Error(`Invalid JSON in config file ${resolvedPath}: ${err.message}`);
+  }
+  if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed)) {
+    throw new Error(`Config file ${resolvedPath} must contain a JSON object`);
+  }
+  const result = {};
+  for (const [key, value] of Object.entries(parsed)) {
+    if (CONFIG_FILE_KEYS.has(key)) {
+      result[key] = value;
+    }
+  }
+  return result;
+}
+
 function normalizeOutputFormats(outputFormats) {
   const source = outputFormats == null ? ["markdown"] : outputFormats;
   const values = Array.isArray(source) ? source : String(source).split(",");
@@ -260,6 +290,14 @@ export function parseCliArgs(argv) {
 export async function resolveConfig(cliArgs = {}) {
   const config = { ...defaultConfig };
 
+  // 0. Merge Config File (lowest priority: overridden by env vars and CLI args)
+  const fileConfig = await loadConfigFile();
+  for (const key of CONFIG_FILE_KEYS) {
+    if (fileConfig[key] !== undefined && fileConfig[key] !== "") {
+      config[key] = fileConfig[key];
+    }
+  }
+
   // 1. Merge Environment Variables
   for (const [envVar, configKey] of Object.entries(ENV_MAP)) {
     if (process.env[envVar] !== undefined) {
@@ -388,6 +426,12 @@ Environment Variables:
   KODEVU_OPENAI_MODEL     Model for reviewer=openai
   KODEVU_OPENAI_ORG       Organization ID for reviewer=openai
   KODEVU_OPENAI_PROJECT   Project ID for reviewer=openai
+
+Config File:
+  ~/.kodevu/config.json   Optional persistent settings (overridden by env vars and CLI flags)
+  Supported keys: reviewer, lang, outputDir, prompt, commandTimeoutMs, outputFormats,
+                  openaiApiKey, openaiBaseUrl, openaiModel, openaiOrganization, openaiProject
+  Example: { "reviewer": "openai", "openaiApiKey": "sk-...", "openaiModel": "gpt-4o" }
 `);
 }
 

package/src/review-runner.js

@@ -105,7 +105,8 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
         reviewer: reviewerName,
         console: false
       });
-      // If it's the last one, it will throw below or break loop anyway
+      // Store the result so the final throw can include stderr/exit code details
+      reviewerResult = err?.result ?? null;
     }
 
     if (reviewerName !== reviewersToTry[reviewersToTry.length - 1]) {
@@ -120,9 +121,23 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
   }
 
   if (!reviewerResult || reviewerResult.code !== 0 || reviewerResult.timedOut) {
-    throw new Error(
-      `${reviewer?.displayName || config.reviewer} failed for ${details.displayId}`
-    );
+    const displayName = reviewer?.displayName || config.reviewer;
+    const reasonParts = [`${displayName} failed for ${details.displayId}`];
+
+    if (reviewerResult?.timedOut) {
+      reasonParts.push("(timed out)");
+    } else if (reviewerResult?.code != null && reviewerResult.code !== 0) {
+      reasonParts.push(`(exit code: ${reviewerResult.code})`);
+    } else if (!reviewerResult) {
+      reasonParts.push("(reviewer produced no result)");
+    }
+
+    const detail = (reviewerResult?.stderr || reviewerResult?.stdout || "").trim();
+    if (detail) {
+      reasonParts.push(`\n${detail}`);
+    }
+
+    throw new Error(reasonParts.join(" "));
   }
 
   progress?.update(0.82, "writing report");