koguma 2.3.2 → 2.3.4

package/cli/log.ts

@@ -27,23 +27,30 @@ export const ANSI = {
 } as const;
 
 // ── Logging functions ──────────────────────────────────────────────
+// On TTY, prepend \r\x1b[K (carriage return + clear line) so any
+// in-place status line (spinner, reload counter) is cleared before
+// the permanent message, preventing interleaved output.
+
+const CR = process.stdout.isTTY ? '\r\x1b[K' : '';
 
 export function log(msg: string): void {
-  console.log(`  ${msg}`);
+  process.stdout.write(`${CR}  ${msg}\n`);
 }
 
 export function ok(msg: string): void {
-  console.log(`  ${ANSI.BRAND_TEAL}✓${ANSI.RESET} ${msg}`);
+  process.stdout.write(`${CR}  ${ANSI.BRAND_TEAL}✓${ANSI.RESET} ${msg}\n`);
 }
 
 export function warn(msg: string): void {
-  console.log(`  ${ANSI.YELLOW}⚠${ANSI.RESET} ${msg}`);
+  process.stdout.write(`${CR}  ${ANSI.YELLOW}⚠${ANSI.RESET} ${msg}\n`);
 }
 
 export function fail(msg: string): void {
-  console.error(`  ${ANSI.BRAND_RED}✗${ANSI.RESET} ${msg}`);
+  process.stderr.write(`${CR}  ${ANSI.BRAND_RED}✗${ANSI.RESET} ${msg}\n`);
 }
 
 export function header(msg: string): void {
-  console.log(`\n${ANSI.BOLD}${ANSI.BRAND_TEAL}🐻 ${msg}${ANSI.RESET}\n`);
+  process.stdout.write(
+    `\n${ANSI.BOLD}${ANSI.BRAND_TEAL}🐻 ${msg}${ANSI.RESET}\n\n`
+  );
 }

package/cli/wrangler.ts

@@ -337,97 +337,32 @@ export function wranglerDev(
       /^▲\s+/, // npm dep install warning lines
       /^>\s+/, // npm progress lines
       /^\s*$/, // blank lines
-      /Reloading local server/, // wrangler dev reload spam
-      /Local server updated/, // wrangler dev reload (handled as status line)
+      /Reloading local server/, // wrangler dev reload — suppress
+      /Local server updated/, // wrangler dev reload — suppress
       /Unable to find and open the program executable/ // benign diagnostic
     ];
 
     const shouldSuppress = (line: string): boolean =>
       suppressPatterns.some(p => p.test(line));
 
-    // ── Animated spinner for reload status ──
-    const SPINNER_FRAMES = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'];
-    let reloadCount = 0;
-    let hasStatusLine = false;
-    let spinnerFrame = 0;
-    let spinnerTimer: ReturnType<typeof setInterval> | null = null;
-    const isTTY = process.stdout.isTTY;
-
-    const drawStatus = (suffix = '') => {
-      if (!isTTY) return;
-      const frame = SPINNER_FRAMES[spinnerFrame % SPINNER_FRAMES.length]!;
-      process.stdout.write(`\r\x1b[K  ${frame} reloading${suffix}`);
-      hasStatusLine = true;
-    };
-
-    const startSpinner = () => {
-      if (!isTTY) return;
-      if (spinnerTimer) return; // already running
-      drawStatus();
-      spinnerTimer = setInterval(() => {
-        spinnerFrame++;
-        drawStatus();
-      }, 80);
-    };
-
-    const stopSpinner = (finalMsg?: string) => {
-      if (spinnerTimer) {
-        clearInterval(spinnerTimer);
-        spinnerTimer = null;
-      }
-      if (!isTTY) return;
-      if (finalMsg) {
-        process.stdout.write(`\r\x1b[K  ✓ ${finalMsg}\n`);
-        hasStatusLine = false;
-      }
-    };
-
-    /** Clear the status line before printing a permanent line */
-    const clearStatus = () => {
-      if (spinnerTimer) {
-        clearInterval(spinnerTimer);
-        spinnerTimer = null;
-      }
-      if (hasStatusLine && isTTY) {
-        process.stdout.write('\r\x1b[K');
-        hasStatusLine = false;
-      }
-    };
-
     const handleOutput = (data: Buffer, isErr: boolean) => {
       const text = data.toString();
       for (const line of text.split('\n')) {
         const trimmed = line.trim();
         if (!trimmed) continue;
 
-        // Rebrand the "Ready" message before suppress check
+        // Rebrand the "Ready" message
         if (trimmed.includes('Ready on http')) {
           const urlMatch = trimmed.match(/(https?:\/\/[^\s]+)/);
           const url = urlMatch?.[1] ?? 'http://localhost:8787';
-          stopSpinner();
-          clearStatus();
           ok(`Server ready → ${url}`);
           continue;
         }
 
-        // Reload events → animated spinner (overwrites in place)
-        if (/Reloading local server/.test(trimmed)) {
-          reloadCount++;
-          startSpinner();
-          continue;
-        }
-        if (/Local server updated/.test(trimmed)) {
-          stopSpinner(`reload #${reloadCount}`);
-          continue;
-        }
-
         if (shouldSuppress(line)) continue;
-
         if (trimmed.includes('Starting local server')) continue;
         if (trimmed.includes('Shutting down')) continue;
 
-        // Permanent line — clear status first
-        clearStatus();
         if (isErr) {
           warn(trimmed);
         } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "koguma",
-  "version": "2.3.2",
+  "version": "2.3.4",
   "description": "🐻 A little CMS with big heart — schema-driven, runs on Cloudflare's free tier",
   "type": "module",
   "license": "MIT",