kodevu 0.1.15 → 0.1.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.15",
+  "version": "0.1.17",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",
   "bin": {

package/src/progress-ui.js

@@ -1,8 +1,6 @@
 import readline from "node:readline";
 
 const SPINNER_FRAMES = ["|", "/", "-", "\\"];
-const DEFAULT_BAR_WIDTH = 24;
-const MIN_BAR_WIDTH = 8;
 
 function clampProgress(value) {
   if (!Number.isFinite(value)) {
@@ -12,21 +10,6 @@ function clampProgress(value) {
   return Math.max(0, Math.min(1, value));
 }
 
-function buildBar(progress, width) {
-  const safeProgress = clampProgress(progress);
-  const filled = Math.max(Math.round(safeProgress * width), safeProgress > 0 ? 1 : 0);
-
-  if (filled >= width) {
-    return `[${"=".repeat(width)}]`;
-  }
-
-  if (filled === 0) {
-    return `[${" ".repeat(width)}]`;
-  }
-
-  return `[${"=".repeat(Math.max(filled - 1, 0))}>${" ".repeat(width - filled)}]`;
-}
-
 function truncateLine(line, maxWidth) {
   if (!Number.isFinite(maxWidth) || maxWidth <= 0) {
     return "";
@@ -44,10 +27,9 @@ function truncateLine(line, maxWidth) {
 }
 
 class ProgressItem {
-  constructor(display, label, options = {}) {
+  constructor(display, label) {
     this.display = display;
     this.label = label;
-    this.barWidth = options.barWidth || DEFAULT_BAR_WIDTH;
     this.progress = 0;
     this.stage = "";
     this.active = false;
@@ -55,8 +37,8 @@ class ProgressItem {
   }
 
   start(stage = "starting") {
-    this.stage = stage;
     this.active = true;
+    this.stage = stage;
 
     if (!this.display.enabled) {
       this.writeFallback(`... ${this.label}: ${stage}`);
@@ -64,7 +46,7 @@ class ProgressItem {
     }
 
     this.display.start();
-    this.display.render();
+    this.display.activate(this);
   }
 
   update(progress, stage) {
@@ -79,7 +61,7 @@ class ProgressItem {
       return;
     }
 
-    this.display.render();
+    this.display.activate(this);
   }
 
   log(message) {
@@ -97,28 +79,23 @@ class ProgressItem {
   finish(prefix, progress, message) {
     this.progress = clampProgress(progress);
     this.active = false;
-    this.display.stopIfIdle();
-    this.display.writeStaticLine(this.buildLine(prefix, message));
+    this.display.deactivate(this);
+    this.display.writeStaticLine(this.buildFinalLine(prefix, message));
   }
 
   renderLine(frameIndex) {
-    const spinner = SPINNER_FRAMES[frameIndex];
-    return this.buildLine(spinner, this.stage);
+    return this.buildStatusLine(SPINNER_FRAMES[frameIndex], this.stage);
   }
 
-  buildLine(prefix, suffix) {
+  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 reservedWidth = prefix.length + this.label.length + pct.length + 6;
-    const dynamicBarWidth = Math.min(
-      this.barWidth,
-      Math.max(MIN_BAR_WIDTH, availableWidth - reservedWidth - (suffix ? suffix.length + 1 : 0))
-    );
-    const bar = buildBar(this.progress, dynamicBarWidth);
-    return truncateLine(
-      `${prefix} ${this.label} ${bar} ${pct}%${suffix ? ` ${suffix}` : ""}`,
-      availableWidth
-    );
+    const suffixText = suffix ? ` | ${suffix}` : "";
+    return truncateLine(`${prefix} ${pct}% ${this.label}${suffixText}`, availableWidth);
   }
 
   writeFallback(line) {
@@ -138,12 +115,14 @@ export class ProgressDisplay {
     this.frameIndex = 0;
     this.timer = null;
     this.items = [];
-    this.renderedLineCount = 0;
+    this.currentItem = null;
+    this.statusVisible = false;
+    this.resizeAttached = false;
     this.handleResize = this.handleResize.bind(this);
   }
 
-  createItem(label, options = {}) {
-    const item = new ProgressItem(this, label, options);
+  createItem(label) {
+    const item = new ProgressItem(this, label);
     this.items.push(item);
     return item;
   }
@@ -154,21 +133,38 @@ export class ProgressDisplay {
     }
 
     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.enabled) {
       this.stream.write(`${message}\n`);
       return;
     }
 
-    this.clearRender();
-    this.stream.write(`${message}\n`);
+    this.clearStatusLine();
+    this.stream.write(`${truncateLine(message, this.getAvailableWidth())}\n`);
     this.render();
   }
 
@@ -178,48 +174,29 @@ export class ProgressDisplay {
       return;
     }
 
-    this.clearRender();
+    this.clearStatusLine();
     this.stream.write(`${truncateLine(message, this.getAvailableWidth())}\n`);
     this.render();
   }
 
   render() {
-    if (!this.enabled) {
+    if (!this.enabled || !this.currentItem?.active) {
       return;
     }
 
-    const activeItems = this.items.filter((item) => item.active);
-    this.clearRender();
-
-    if (activeItems.length === 0) {
-      this.renderedLineCount = 0;
-      return;
-    }
-
-    const lines = activeItems.map((item) => item.renderLine(this.frameIndex));
-    this.stream.write(lines.join("\n"));
-    this.renderedLineCount = lines.length;
+    this.clearStatusLine();
+    this.stream.write(this.currentItem.renderLine(this.frameIndex));
+    this.statusVisible = true;
   }
 
-  clearRender() {
-    if (!this.enabled || this.renderedLineCount === 0) {
+  clearStatusLine() {
+    if (!this.enabled || !this.statusVisible) {
       return;
     }
 
-    readline.moveCursor(this.stream, 0, -Math.max(this.renderedLineCount - 1, 0));
-
-    for (let index = 0; index < this.renderedLineCount; index += 1) {
-      readline.clearLine(this.stream, 0);
-      readline.cursorTo(this.stream, 0);
-
-      if (index < this.renderedLineCount - 1) {
-        readline.moveCursor(this.stream, 0, 1);
-      }
-    }
-
-    readline.moveCursor(this.stream, 0, -Math.max(this.renderedLineCount - 1, 0));
+    readline.clearLine(this.stream, 0);
     readline.cursorTo(this.stream, 0);
-    this.renderedLineCount = 0;
+    this.statusVisible = false;
   }
 
   stopIfIdle() {
@@ -237,7 +214,7 @@ export class ProgressDisplay {
 
   getAvailableWidth() {
     const columns = this.stream.columns || 80;
-    return Math.max(columns - 1, 20);
+    return Math.max(columns - 1, 1);
   }
 
   handleResize() {

package/src/review-runner.js

@@ -592,24 +592,16 @@ export async function runReviewCycle(config) {
 
   console.log(`Reviewing ${backend.displayName} ${backend.changeName}s ${formatChangeList(backend, changeIdsToReview)}`);
   const progressDisplay = new ProgressDisplay();
-  const overallProgress = progressDisplay.createItem(
-    `${backend.displayName} ${backend.changeName} batch`,
-    { barWidth: 30 }
-  );
-  overallProgress.start("0/" + changeIdsToReview.length + " completed");
+  const progress = progressDisplay.createItem(`${backend.displayName} ${backend.changeName} batch`);
+  progress.start("0/" + changeIdsToReview.length + " completed");
 
   for (const [index, changeId] of changeIdsToReview.entries()) {
     debugLog(config, `Starting review for ${backend.formatChangeId(changeId)}.`);
     const displayId = backend.formatChangeId(changeId);
-    const progress = progressDisplay.createItem(
-      `${backend.displayName} ${backend.changeName} ${displayId} (${index + 1}/${changeIdsToReview.length})`
-    );
-    progress.start("queued");
-    updateOverallProgress(overallProgress, index, changeIdsToReview.length, 0, `starting ${displayId}`);
+    updateOverallProgress(progress, index, changeIdsToReview.length, 0, `starting ${displayId}`);
 
     const syncOverallProgress = (fraction, stage) => {
-      progress.update(fraction, stage);
-      updateOverallProgress(overallProgress, index, changeIdsToReview.length, fraction, `${displayId} | ${stage}`);
+      updateOverallProgress(progress, index, changeIdsToReview.length, fraction, `${displayId} | ${stage}`);
     };
 
     let result;
@@ -618,8 +610,7 @@ export async function runReviewCycle(config) {
       result = await reviewChange(config, backend, targetInfo, changeId, { update: syncOverallProgress, log: (message) => progress.log(message) });
       syncOverallProgress(0.94, "saving checkpoint");
     } catch (error) {
-      progress.fail(`failed at ${displayId}`);
-      overallProgress.fail(`failed at ${displayId} (${index}/${changeIdsToReview.length} completed)`);
+      progress.fail(`failed at ${displayId} (${index}/${changeIdsToReview.length} completed)`);
       throw error;
     }
 
@@ -631,11 +622,11 @@ export async function runReviewCycle(config) {
     await saveState(config.stateFilePath, updateProjectState(stateFile, targetInfo, nextProjectState));
     stateFile.projects[targetInfo.stateKey] = nextProjectState;
     debugLog(config, `Saved checkpoint for ${backend.formatChangeId(changeId)} to ${config.stateFilePath}.`);
-    progress.succeed(`reviewed ${displayId}: ${outputLabels.join(" | ") || "(no report file generated)"}`);
-    updateOverallProgress(overallProgress, index + 1, changeIdsToReview.length, 0, `finished ${displayId}`);
+    progress.log(`[done] reviewed ${displayId}: ${outputLabels.join(" | ") || "(no report file generated)"}`);
+    updateOverallProgress(progress, index + 1, changeIdsToReview.length, 0, `finished ${displayId}`);
   }
 
-  overallProgress.succeed(`completed ${changeIdsToReview.length}/${changeIdsToReview.length}`);
+  progress.succeed(`completed ${changeIdsToReview.length}/${changeIdsToReview.length}`);
 
   const remainingChanges = await backend.getPendingChangeIds(
     config,