kodevu 0.1.19 → 0.1.21

package/README.md

@@ -32,7 +32,7 @@ If you want a config file, run `npx kodevu init` to create `./config.json` in th
 npx kodevu init
 ```
 
-This creates `config.json` in the current directory from the packaged `config.example.json`.
+This creates `config.json` in the current directory from built-in defaults.
 You only need this when you want to override defaults such as `reviewer` or output paths.
 
 If you want a different path:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.19",
+  "version": "0.1.21",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",
   "bin": {
@@ -8,8 +8,7 @@
   },
   "files": [
     "src",
-    "README.md",
-    "config.example.json"
+    "README.md"
   ],
   "scripts": {
     "start": "node src/index.js",

package/src/config.js

@@ -1,8 +1,6 @@
 import fs from "node:fs/promises";
-import { constants as fsConstants } from "node:fs";
 import os from "node:os";
 import path from "node:path";
-import { fileURLToPath } from "node:url";
 import { findCommandOnPath } from "./shell.js";
 
 const defaultStorageDir = path.join(os.homedir(), ".kodevu");
@@ -16,10 +14,21 @@ const defaultConfig = {
   commandTimeoutMs: 600000,
   prompt:
     "请严格审查当前变更，优先指出 bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失测试。请使用简体中文输出 Markdown；如果没有明确缺陷，请写“未发现明确缺陷”，并补充剩余风险。",
-  maxRevisionsPerRun: 20,
+  maxRevisionsPerRun: 5,
   outputFormats: ["markdown"]
 };
 
+const configTemplate = {
+  target: "C:/path/to/your/repository-or-subdirectory",
+  reviewer: defaultConfig.reviewer,
+  prompt: defaultConfig.prompt,
+  outputDir: "~/.kodevu",
+  stateFilePath: "~/.kodevu/state.json",
+  commandTimeoutMs: defaultConfig.commandTimeoutMs,
+  maxRevisionsPerRun: defaultConfig.maxRevisionsPerRun,
+  outputFormats: defaultConfig.outputFormats
+};
+
 function resolveConfigPath(baseDir, value) {
   if (!value) {
     return value;
@@ -264,12 +273,12 @@ Config highlights:
 
 export async function initConfig(targetPath = "config.json") {
   const absoluteTargetPath = path.resolve(targetPath);
-  const packageRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "..");
-  const templatePath = path.join(packageRoot, "config.example.json");
 
   await fs.mkdir(path.dirname(absoluteTargetPath), { recursive: true });
+
+  const content = JSON.stringify(configTemplate, null, 2) + "\n";
   try {
-    await fs.copyFile(templatePath, absoluteTargetPath, fsConstants.COPYFILE_EXCL);
+    await fs.writeFile(absoluteTargetPath, content, { flag: "wx" });
   } catch (error) {
     if (error?.code === "EEXIST") {
       throw new Error(`Config file already exists: ${absoluteTargetPath}`);

package/src/progress-ui.js

@@ -1,9 +1,3 @@
-import readline from "node:readline";
-
-const SPINNER_FRAMES = ["|", "/", "-", "\\"];
-const ELLIPSIS = "...";
-const MIN_DYNAMIC_WIDTH = 60;
-
 function clampProgress(value) {
   if (!Number.isFinite(value)) {
     return 0;
@@ -12,78 +6,6 @@ function clampProgress(value) {
   return Math.max(0, Math.min(1, value));
 }
 
-function getCharacterWidth(character) {
-  const codePoint = character.codePointAt(0);
-
-  if (!codePoint || codePoint <= 0x1f || (codePoint >= 0x7f && codePoint <= 0x9f)) {
-    return 0;
-  }
-
-  if (
-    codePoint >= 0x1100 &&
-    (
-      codePoint <= 0x115f ||
-      codePoint === 0x2329 ||
-      codePoint === 0x232a ||
-      (codePoint >= 0x2e80 && codePoint <= 0xa4cf && codePoint !== 0x303f) ||
-      (codePoint >= 0xac00 && codePoint <= 0xd7a3) ||
-      (codePoint >= 0xf900 && codePoint <= 0xfaff) ||
-      (codePoint >= 0xfe10 && codePoint <= 0xfe19) ||
-      (codePoint >= 0xfe30 && codePoint <= 0xfe6f) ||
-      (codePoint >= 0xff00 && codePoint <= 0xff60) ||
-      (codePoint >= 0xffe0 && codePoint <= 0xffe6) ||
-      (codePoint >= 0x1f300 && codePoint <= 0x1f64f) ||
-      (codePoint >= 0x1f900 && codePoint <= 0x1f9ff) ||
-      (codePoint >= 0x20000 && codePoint <= 0x3fffd)
-    )
-  ) {
-    return 2;
-  }
-
-  return 1;
-}
-
-function getDisplayWidth(text) {
-  let width = 0;
-
-  for (const character of text) {
-    width += getCharacterWidth(character);
-  }
-
-  return width;
-}
-
-function truncateLine(line, maxWidth) {
-  if (!Number.isFinite(maxWidth) || maxWidth <= 0) {
-    return "";
-  }
-
-  if (getDisplayWidth(line) <= maxWidth) {
-    return line;
-  }
-
-  if (maxWidth <= 3) {
-    return ".".repeat(maxWidth);
-  }
-
-  const targetWidth = maxWidth - getDisplayWidth(ELLIPSIS);
-  let result = "";
-  let width = 0;
-
-  for (const character of line) {
-    const nextWidth = width + getCharacterWidth(character);
-
-    if (nextWidth > targetWidth) {
-      break;
-    }
-
-    result += character;
-    width = nextWidth;
-  }
-
-  return `${result}${ELLIPSIS}`;
-}
-
 class ProgressItem {
   constructor(display, label) {
     this.display = display;
@@ -91,20 +13,13 @@ class ProgressItem {
     this.progress = 0;
     this.stage = "";
     this.active = false;
-    this.lastFallbackLine = "";
+    this.lastStatusLine = "";
   }
 
   start(stage = "starting") {
     this.active = true;
     this.stage = stage;
-
-    if (!this.display.canRenderDynamically()) {
-      this.writeFallback(`... ${this.label}: ${stage}`);
-      return;
-    }
-
-    this.display.start();
-    this.display.activate(this);
+    this.writeStatus();
   }
 
   update(progress, stage) {
@@ -114,17 +29,11 @@ class ProgressItem {
       this.stage = stage;
     }
 
-    if (!this.display.canRenderDynamically()) {
-      this.writeFallback(`... ${this.label}: ${this.stage}`);
-      return;
-    }
-
-    this.display.start();
-    this.display.activate(this);
+    this.writeStatus();
   }
 
   log(message) {
-    this.display.log(message);
+    this.display.writeLine(message);
   }
 
   succeed(message) {
@@ -138,189 +47,36 @@ class ProgressItem {
   finish(prefix, progress, message) {
     this.progress = clampProgress(progress);
     this.active = false;
-    this.display.deactivate(this);
-    this.display.writeStaticLine(this.buildFinalLine(prefix, message));
+    this.display.writeLine(`${prefix} ${message}`);
   }
 
-  renderLine(frameIndex) {
-    return this.buildStatusLine(SPINNER_FRAMES[frameIndex], this.stage);
-  }
+  writeStatus() {
+    const line = this.buildStatusLine();
 
-  buildFinalLine(prefix, message) {
-    return this.buildStatusLine(prefix, message);
-  }
-
-  buildStatusLine(prefix, suffix) {
-    const availableWidth = this.display.getAvailableWidth();
-    const pct = `${Math.round(this.progress * 100)}`.padStart(3, " ");
-    const suffixText = suffix ? ` | ${suffix}` : "";
-    return truncateLine(`${prefix} ${pct}% ${this.label}${suffixText}`, availableWidth);
-  }
-
-  writeFallback(line) {
-    if (line === this.lastFallbackLine) {
+    if (line === this.lastStatusLine) {
       return;
     }
 
-    this.lastFallbackLine = line;
-    this.display.stream.write(`${line}\n`);
+    this.lastStatusLine = line;
+    this.display.writeLine(line);
+  }
+
+  buildStatusLine() {
+    const pct = `${Math.round(this.progress * 100)}`.padStart(3, " ");
+    return `[progress] ${pct}% ${this.label}${this.stage ? ` | ${this.stage}` : ""}`;
   }
 }
 
 export class ProgressDisplay {
   constructor(options = {}) {
     this.stream = options.stream || process.stdout;
-    this.enabled = Boolean(this.stream.isTTY);
-    this.frameIndex = 0;
-    this.timer = null;
-    this.items = [];
-    this.currentItem = null;
-    this.statusVisible = false;
-    this.lastStatusWidth = 0;
-    this.resizeAttached = false;
-    this.handleResize = this.handleResize.bind(this);
   }
 
   createItem(label) {
-    const item = new ProgressItem(this, label);
-    this.items.push(item);
-    return item;
-  }
-
-  start() {
-    if (!this.enabled || this.timer) {
-      return;
-    }
-
-    this.attachResizeHandler();
-    this.timer = setInterval(() => {
-      this.frameIndex = (this.frameIndex + 1) % SPINNER_FRAMES.length;
-      this.render();
-    }, 120);
-  }
-
-  activate(item) {
-    this.currentItem = item;
-    this.render();
-  }
-
-  deactivate(item) {
-    if (this.currentItem === item) {
-      this.currentItem = this.items.findLast((candidate) => candidate.active) || null;
-    }
-
-    this.stopIfIdle();
-    this.clearStatusLine();
-
-    if (this.currentItem) {
-      this.render();
-    }
-  }
-
-  log(message) {
-    if (!this.canRenderDynamically()) {
-      this.stream.write(`${message}\n`);
-      return;
-    }
-
-    this.clearStatusLine();
-    this.stream.write(`${truncateLine(message, this.getAvailableWidth())}\n`);
-    this.render();
-  }
-
-  writeStaticLine(message) {
-    if (!this.canRenderDynamically()) {
-      this.stream.write(`${message}\n`);
-      return;
-    }
-
-    this.clearStatusLine();
-    this.stream.write(`${truncateLine(message, this.getAvailableWidth())}\n`);
-    this.render();
-  }
-
-  render() {
-    if (!this.canRenderDynamically() || !this.currentItem?.active) {
-      this.clearStatusLine();
-      return;
-    }
-
-    this.clearStatusLine();
-    const line = this.currentItem.renderLine(this.frameIndex);
-    this.stream.write(line);
-    this.statusVisible = true;
-    this.lastStatusWidth = getDisplayWidth(line);
-  }
-
-  clearStatusLine() {
-    if (!this.enabled || !this.statusVisible) {
-      return;
-    }
-
-    const rows = Math.max(1, Math.ceil(this.lastStatusWidth / Math.max(this.stream.columns || 1, 1)));
-
-    readline.moveCursor(this.stream, 0, -Math.max(rows - 1, 0));
-
-    for (let index = 0; index < rows; index += 1) {
-      readline.clearLine(this.stream, 0);
-      readline.cursorTo(this.stream, 0);
-
-      if (index < rows - 1) {
-        readline.moveCursor(this.stream, 0, 1);
-      }
-    }
-
-    readline.moveCursor(this.stream, 0, -Math.max(rows - 1, 0));
-    readline.cursorTo(this.stream, 0);
-    this.statusVisible = false;
-    this.lastStatusWidth = 0;
-  }
-
-  stopIfIdle() {
-    if (this.items.some((item) => item.active)) {
-      return;
-    }
-
-    if (this.timer) {
-      clearInterval(this.timer);
-      this.timer = null;
-    }
-
-    this.detachResizeHandler();
-  }
-
-  getAvailableWidth() {
-    const columns = this.stream.columns || 80;
-    return Math.max(columns - 1, 1);
-  }
-
-  canRenderDynamically() {
-    return this.enabled && this.getAvailableWidth() >= MIN_DYNAMIC_WIDTH;
+    return new ProgressItem(this, label);
   }
 
-  handleResize() {
-    if (!this.enabled) {
-      return;
-    }
-
-    this.render();
-  }
-
-  attachResizeHandler() {
-    if (typeof this.stream.on !== "function" || this.resizeAttached) {
-      return;
-    }
-
-    this.stream.on("resize", this.handleResize);
-    this.resizeAttached = true;
-  }
-
-  detachResizeHandler() {
-    if (typeof this.stream.off !== "function" || !this.resizeAttached) {
-      return;
-    }
-
-    this.stream.off("resize", this.handleResize);
-    this.resizeAttached = false;
+  writeLine(message) {
+    this.stream.write(`${message}\n`);
   }
 }

package/config.example.json

@@ -1,12 +0,0 @@
-{
-  "target": "C:/path/to/your/repository-or-subdirectory",
-  "reviewer": "auto",
-  "prompt": "请严格审查当前变更，优先指出 bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失测试。请使用简体中文输出 Markdown；如果没有明确缺陷，请写“未发现明确缺陷”，并补充剩余风险。",
-  "outputDir": "~/.kodevu",
-  "stateFilePath": "~/.kodevu/state.json",
-  "commandTimeoutMs": 600000,
-  "maxRevisionsPerRun": 5,
-  "outputFormats": [
-    "markdown"
-  ]
-}