kodevu 0.1.63 → 0.1.65

package/README.md

@@ -23,6 +23,16 @@ npx kodevu .
 Review reports are saved to `~/.kodevu/` by default.
 Console output is intentionally concise by default; detailed execution logs are written to `~/.kodevu/logs/`.
 
+## Install as an AI Agent Skill
+
+Kodevu includes a natively supported `SKILL.md` file, which allows it to be installed as a specialized skill in AI agent coding assistants.
+
+To install:
+
+```bash
+npx skills add gyteng/kodevu
+```
+
 ## Usage
 
 ```bash
@@ -32,7 +42,7 @@ npx kodevu [target] [options]
 ### Options
 
 - `target`: Repository path (Git) or SVN URL/Working copy (default: `.`).
-- `--reviewer, -r`: `codex`, `gemini`, `copilot`, `openai`, or `auto` (default: `auto`).
+- `--reviewer, -r`: `codex`, `gemini`, `copilot`, `openai`, `opencode` or `auto` (default: `auto`).
 - `--rev, -v`: A specific revision or commit hash to review.
 - `--last, -n`: Number of latest revisions to review (default: 1). Use negative values (e.g., `-3`) to review only the 3rd commit from the top.
 - `--uncommitted, -u`: Review current uncommitted changes in the target working tree.

package/package.json

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

@@ -8,8 +8,8 @@ const require = createRequire(import.meta.url);
 const { version: packageVersion } = require("../package.json");
 
 const defaultStorageDir = path.join(os.homedir(), ".kodevu");
-const SUPPORTED_REVIEWERS = ["codex", "gemini", "copilot", "openai"];
-const AUTO_SUPPORTED_REVIEWERS = ["codex", "gemini", "copilot"];
+const SUPPORTED_REVIEWERS = ["codex", "gemini", "copilot", "openai", "opencode"];
+const AUTO_SUPPORTED_REVIEWERS = ["codex", "gemini", "copilot", "opencode"];
 
 const defaultConfig = {
   reviewer: "auto",
@@ -405,7 +405,7 @@ Usage:
 
 Options:
   --target, <path>  Target repository path (default: current directory)
-  --reviewer, -r    Reviewer (codex | gemini | copilot | openai | auto, default: auto)
+  --reviewer, -r    Reviewer (codex | gemini | copilot | openai | opencode | 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 specific revision(s), hashes, branches or ranges (comma-separated)

package/src/logger.js

@@ -2,87 +2,73 @@ import fs from "node:fs";
 import path from "node:path";
 import { formatDate } from "./utils.js";
 
-function formatValue(value) {
-  if (value == null) {
-    return "";
-  }
+// Top-level helpers moved into Logger as private methods (#name).
 
-  if (typeof value === "string") {
-    return value.trim();
+class Logger {
+  constructor () {
+    this.config = null;
+    this.logFile = null;
+    this.sessionId = null;
+    this.initialized = false;
   }
 
-  if (typeof value === "number" || typeof value === "boolean") {
-    return String(value);
-  }
+  #formatValue(value) {
+    if (value == null) return "";
 
-  try {
-    return JSON.stringify(value);
-  } catch {
-    return String(value);
-  }
-}
+    if (typeof value === "string") return value.trim();
 
-function formatMeta(meta = {}) {
-  const parts = [];
+    if (typeof value === "number" || typeof value === "boolean") return String(value);
 
-  for (const [key, rawValue] of Object.entries(meta)) {
-    if (rawValue == null || rawValue === "") {
-      continue;
+    try {
+      return JSON.stringify(value);
+    } catch {
+      return String(value);
     }
+  }
 
-    const value = formatValue(rawValue);
-    if (!value) {
-      continue;
-    }
+  #formatMeta(meta = {}) {
+    const parts = [];
 
-    if (typeof rawValue === "string") {
-      parts.push(`${key}=${JSON.stringify(value)}`);
-      continue;
-    }
+    for (const [key, rawValue] of Object.entries(meta || {})) {
+      if (rawValue == null || rawValue === "") continue;
 
-    if (typeof rawValue === "number" || typeof rawValue === "boolean") {
-      parts.push(`${key}=${String(rawValue)}`);
-      continue;
-    }
+      const value = this.#formatValue(rawValue);
+      if (!value) continue;
 
-    parts.push(`${key}=${value}`);
-  }
+      if (typeof rawValue === "string") {
+        parts.push(`${key}=${JSON.stringify(value)}`);
+        continue;
+      }
 
-  return parts.join(" ");
-}
+      if (typeof rawValue === "number" || typeof rawValue === "boolean") {
+        parts.push(`${key}=${rawValue}`);
+        continue;
+      }
 
-function formatErrorDetails(error) {
-  if (!error) {
-    return "";
-  }
+      parts.push(`${key}=${value}`);
+    }
 
-  if (error instanceof Error) {
-    return error.stack || error.message || String(error);
+    return parts.join(" ");
   }
 
-  if (typeof error === "string") {
-    return error;
-  }
+  #formatErrorDetails(error) {
+    if (!error) return "";
 
-  try {
-    return JSON.stringify(error, null, 2);
-  } catch {
-    return String(error);
-  }
-}
+    if (error instanceof Error) return error.stack ?? error.message ?? String(error);
 
-class Logger {
-  constructor() {
-    this.config = null;
-    this.logFile = null;
-    this.sessionId = null;
-    this.initialized = false;
+    if (typeof error === "string") return error;
+
+    try {
+      return JSON.stringify(error, null, 2);
+    } catch {
+      return String(error);
+    }
   }
 
   init(config) {
     if (this.initialized) return;
     this.config = config;
-    this.sessionId = this._createSessionId();
+    this.sessionId = this.#createSessionId();
     
     if (config.logsDir) {
       try {
@@ -93,7 +79,7 @@ class Logger {
         this.logFile = path.join(config.logsDir, `run-${date}.log`);
         
         // Simple rotation: Clean up logs older than 7 days
-        this._cleanupOldLogs(config.logsDir);
+        this.#cleanupOldLogs(config.logsDir);
         this.initialized = true;
       } catch (err) {
         console.error(`[logger] Failed to initialize log file: ${err.message}`);
@@ -102,33 +88,33 @@ class Logger {
   }
 
   info(message, meta) {
-    this._log("INFO", message, meta);
+    this.#log("INFO", message, meta);
   }
 
   warn(message, meta) {
-    this._log("WARN", message, { ...meta, console: meta?.console ?? true });
+    this.#log("WARN", message, { ...meta, console: meta?.console ?? true });
   }
 
   error(message, error, meta) {
-    this._log("ERROR", message, {
+    this.#log("ERROR", message, {
       ...meta,
       console: meta?.console ?? true,
-      error: formatErrorDetails(error)
+      error: this.#formatErrorDetails(error)
     });
   }
 
   debug(message, meta) {
-    this._log("DEBUG", message, meta);
+    this.#log("DEBUG", message, meta);
   }
 
-  _log(level, message, meta = {}) {
+  #log(level, message, meta = {}) {
     const timestamp = formatDate(new Date());
     const { console: consoleMode, ...details } = meta;
     const fields = {
       session: this.sessionId || "uninitialized",
       ...details
     };
-    const metaSuffix = formatMeta(fields);
+    const metaSuffix = this.#formatMeta(fields);
     const logLine = `[${timestamp}] [${level}] ${message}${metaSuffix ? ` | ${metaSuffix}` : ""}`;
 
     if (this.logFile) {
@@ -139,7 +125,7 @@ class Logger {
       }
     }
 
-    if (!this._shouldWriteToConsole(level, consoleMode)) {
+    if (!this.#shouldWriteToConsole(level, consoleMode)) {
       return;
     }
 
@@ -150,34 +136,24 @@ class Logger {
     }
   }
 
-  _shouldWriteToConsole(level, consoleMode) {
-    if (consoleMode === false) {
-      return false;
-    }
+  #shouldWriteToConsole(level, consoleMode) {
+    if (consoleMode === false) return false;
 
-    if (level === "ERROR" || level === "WARN") {
-      return true;
-    }
+    if (level === "ERROR" || level === "WARN") return true;
 
     if (level === "DEBUG") {
-      if (consoleMode === true) {
-        return Boolean(this.config?.debug);
-      }
+      if (consoleMode === true) return Boolean(this.config?.debug);
       return false;
     }
 
-    if (consoleMode === true) {
-      return true;
-    }
+    if (consoleMode === true) return true;
 
-    if (consoleMode === "debug") {
-      return Boolean(this.config?.debug);
-    }
+    if (consoleMode === "debug") return Boolean(this.config?.debug);
 
     return false;
   }
 
-  _createSessionId() {
+  #createSessionId() {
     return [
       Date.now().toString(36),
       process.pid.toString(36),
@@ -185,7 +161,9 @@ class Logger {
     ].join("-");
   }
 
-  _cleanupOldLogs(logsDir) {
+
+
+  #cleanupOldLogs(logsDir) {
     try {
       const files = fs.readdirSync(logsDir);
       const now = Date.now();

package/src/reviewers.js

@@ -165,6 +165,45 @@ export const REVIEWERS = {
       }
     }
   },
+  opencode: {
+    displayName: "OpenCode",
+    responseSectionTitle: "OpenCode Response",
+    emptyResponseText: "_No final response returned from opencode._",
+    async run(config, workingDir, promptText, diffText) {
+      const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), "kodevu-opencode-"));
+      const reviewInputFile = path.join(tempDir, "review-input.md");
+
+      try {
+        await fs.writeFile(
+          reviewInputFile,
+          [promptText, "### Unified Diff", "```diff", diffText, "```"].join("\n\n"),
+          "utf8"
+        );
+
+        const args = [
+          "run",
+          "Please perform the code review strictly following the instructions and unified diff provided in the attached file.",
+          "-f",
+          reviewInputFile,
+          "--pure"
+        ];
+
+        const execResult = await runCommand("opencode", args, {
+          cwd: workingDir,
+          allowFailure: true,
+          timeoutMs: config.commandTimeoutMs,
+          debug: config.debug
+        });
+
+        return {
+          ...execResult,
+          message: execResult.stdout
+        };
+      } finally {
+        await fs.rm(tempDir, { recursive: true, force: true });
+      }
+    }
+  },
   openai: {
     displayName: "OpenAI API",
     responseSectionTitle: "OpenAI Response",

package/src/utils.js

@@ -55,6 +55,7 @@ export function formatDate(dateInput) {
 
   return `${year}-${month}-${day} ${hours}:${minutes}:${seconds}.${milliseconds} ${offset}`;
 }
+
 export function getTimestampPrefix() {
   const now = new Date();
   const pad = (n) => String(n).padStart(2, "0");