kodevu 0.1.62 → 0.1.64

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

package/package.json

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

@@ -139,7 +139,7 @@ async function resolveAutoReviewers(debug) {
   }
 
   if (availableReviewers.length === 0) {
-    throw new Error(`No reviewer CLI found in PATH. Install one of: ${AUTO_SUPPORTED_REVIEWERS.join(", ")}`);
+    throw mkCliError(`No reviewer CLI found in PATH. Install one of: ${AUTO_SUPPORTED_REVIEWERS.join(", ")}`);
   }
 
   // Shuffle for variety
@@ -335,11 +335,11 @@ export async function resolveConfig(cliArgs = {}) {
   }
 
   if (cliArgs.rev && cliArgs.last) {
-    throw new Error("Parameters --rev and --last are mutually exclusive. Please specify only one.");
+    throw mkCliError("Parameters --rev and --last are mutually exclusive. Please specify only one.");
   }
 
   if (cliArgs.uncommitted && (cliArgs.rev || cliArgs.last)) {
-    throw new Error("Parameter --uncommitted is mutually exclusive with --rev and --last.");
+    throw mkCliError("Parameter --uncommitted is mutually exclusive with --rev and --last.");
   }
 
   if (!config.target) {
@@ -358,7 +358,7 @@ export async function resolveConfig(cliArgs = {}) {
     try {
       config.prompt = await fs.readFile(promptPath, "utf8");
     } catch (err) {
-      throw new Error(`Failed to read prompt file: ${promptPath} (${err.message})`);
+      throw mkCliError(`Failed to read prompt file: ${promptPath} (${err.message})`);
     }
   }
 
@@ -370,7 +370,7 @@ export async function resolveConfig(cliArgs = {}) {
     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: ${SUPPORTED_REVIEWERS.join(", ")}, or "auto"`);
+    throw mkCliError(`"reviewer" must be one of: ${SUPPORTED_REVIEWERS.join(", ")}, or "auto"`);
   }
 
   config.outputDir = resolvePath(config.outputDir);
@@ -391,7 +391,7 @@ export async function resolveConfig(cliArgs = {}) {
   }
 
   if (config.reviewer === "openai" && !config.openaiApiKey) {
-    throw new Error('Reviewer "openai" requires an API key. Set KODEVU_OPENAI_API_KEY or pass --openai-api-key.');
+    throw mkCliError('Reviewer "openai" requires an API key. Set KODEVU_OPENAI_API_KEY or pass --openai-api-key.');
   }
 
   return config;

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/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");