@magicpixelart/cli 0.3.0 → 0.4.0

package/dist/commands/sync.js

@@ -1,16 +1,19 @@
 import kleur from 'kleur';
 import ora from 'ora';
 import { createHash } from 'node:crypto';
-import { mkdir, writeFile, unlink, readdir, rm, rename } from 'node:fs/promises';
+import { mkdir, writeFile, unlink, rename } from 'node:fs/promises';
 import { existsSync } from 'node:fs';
 import { dirname, relative, resolve } from 'node:path';
 import { loadConfig, loadState, saveState } from '../config.js';
 import { fetchAllManifest, fetchAssetBytes, ApiError } from '../api.js';
 import { fileSha256 } from '../util/hash.js';
-import { assetDiskPath, assetDiskPathFromKey } from '../util/paths.js';
+import { assetDiskPath, assetDiskPathFromKey, pruneEmptyDirs, walkOutDirPngs } from '../util/paths.js';
 import { createLimit } from '../util/limit.js';
 import { emitTypedIndex, ensureAgentsDoc, scanDiskAssets } from '../util/emitIndex.js';
 import { assertPathInsideRoot, tmpPathFor } from '../util/security.js';
+import { friendlyFsError } from '../util/errors.js';
+import { maxIsoTimestamp } from '../util/iso.js';
+import { formatBytes } from '../util/format.js';
 export async function syncCommand(opts) {
     if (opts.watch) {
         await watchLoop(opts);
@@ -19,7 +22,13 @@ export async function syncCommand(opts) {
     await runOnce(opts);
 }
 async function watchLoop(opts) {
-    const intervalSec = typeof opts.watch === 'string' ? Math.max(2, parseInt(opts.watch, 10) || 10) : 10;
+    // Default 2s — matches the perceived-instant UX users expect when editing
+    // sprites in MagicPixel and watching them refresh in their game's dev
+    // server. Incremental polls send `?since=<lastSync>` so an empty manifest
+    // round-trip is cheap (a single HTTP HEAD-sized response with `count: 0`).
+    // Commander already validated the numeric value (2–3600); the `?? 2`
+    // covers the bare boolean `-w` form.
+    const intervalSec = typeof opts.watch === 'string' ? parseInt(opts.watch, 10) : 2;
     // Header — print once on start. Project/asset count is best-effort; if the
     // first manifest fetch fails we still want the loop to come up and retry.
     let headerCount = null;
@@ -34,38 +43,97 @@ async function watchLoop(opts) {
     console.log(`   Edit at:  ${kleur.cyan('https://magicpixel.art')}`);
     if (headerCount !== null)
         console.log(kleur.dim(`   Sprites:  ${headerCount}`));
-    console.log(kleur.dim(`   Polling:  every ${intervalSec}s   ·   Stop: Ctrl+C`));
+    console.log(kleur.dim(`   Polling:  every ${intervalSec}s (slows when idle)   ·   Stop: Ctrl+C`));
     console.log();
     let stopping = false;
     let inFlight = false;
-    let resolveIdle = null;
     let backoffSec = intervalSec;
     let pausedForAuth = false;
-    const onSigint = () => {
+    // Mirrors pausedForAuth for the network-offline path: once we've told the
+    // user "MagicPixel is offline", the next successful tick prints a single
+    // "back online" recovery line. Without this flag a user who walked away
+    // during an outage has no signal that things are healthy again.
+    let pausedForNetwork = false;
+    // After this many consecutive 401/403s we give up and exit non-zero so a
+    // parent process (Vite plugin, systemd, pm2) can tell the watcher is
+    // genuinely broken (revoked key) rather than transiently blipped.
+    const MAX_AUTH_FAILURES = 5;
+    let consecutiveAuthFailures = 0;
+    // Adaptive idle backoff: after a few minutes of nothing-to-do we slow the
+    // poll from intervalSec → 5s → 10s so a dev who walked away isn't hammering
+    // the manifest endpoint. ANY change OR error resets this back to intervalSec,
+    // so the "edit → see it" promise stays intact the moment the user comes
+    // back. Error backoff (2→60s) is separate and continues to win.
+    let consecutiveIdleTicks = 0;
+    // Thresholds in seconds (NOT ticks) so a `--watch 10` user gets the same
+    // ~3 min / ~15 min UX as a default `--watch 2` user. Previously these were
+    // tick counts (90 / 300) which made the slowdown timing scale with intervalSec.
+    const IDLE_SOFT_BACKOFF_SECONDS = 180;
+    const IDLE_HARD_BACKOFF_SECONDS = 900;
+    // Cancellable idle sleep — `onStopSignal` calls `wakeStop()` so the next
+    // tick exits immediately instead of waiting out the full backoff (which can
+    // be 60s at the error ceiling). Initialised to a no-op so the first
+    // pre-loop `await tick()` is safe even before the first sleep.
+    let wakeStop = () => { };
+    // Handle both SIGINT (Ctrl+C) and SIGTERM (`kill`, `docker stop`, systemd,
+    // pm2). Without the SIGTERM listener a supervisor-managed watcher would die
+    // mid-sync without draining in-flight work or preserving the exit code.
+    const onStopSignal = (signal) => {
         if (stopping)
             return;
         stopping = true;
+        wakeStop();
         process.stdout.write('\x1b[2K\r');
         if (inFlight) {
-            console.log(kleur.dim('[watch] finishing current sync… (Ctrl+C again to force quit)'));
-            process.once('SIGINT', () => process.exit(130));
+            console.log(kleur.dim(`[watch] finishing current sync… (${signal} again to force quit)`));
+            process.once(signal, () => process.exit(signal === 'SIGINT' ? 130 : 143));
         }
         else {
-            console.log(kleur.dim('[watch] stopped.'));
-            process.exit(0);
+            if (!opts.quiet)
+                console.log(kleur.dim('[watch] stopped.'));
+            // Preserve any exit code already set by a prior failed tick.
+            process.exit(process.exitCode ?? 0);
         }
     };
-    process.on('SIGINT', onSigint);
+    process.on('SIGINT', onStopSignal);
+    process.on('SIGTERM', onStopSignal);
     const tick = async () => {
         if (inFlight || stopping)
             return;
         inFlight = true;
         try {
             const r = await runOnce({ ...opts, watch: false }, { watchMode: true });
-            // Reset backoff + auth-pause flags on any successful tick.
+            // Reset backoff on any successful tick. Resume message fires once when
+            // we recover from an auth pause — `getApiKey()` re-reads
+            // .magicpixel/credentials on every call, so a `magicpixel login` in
+            // another terminal is picked up automatically by the next tick.
+            const wasPausedForAuth = pausedForAuth;
+            const wasPausedForNetwork = pausedForNetwork;
             backoffSec = intervalSec;
             pausedForAuth = false;
+            pausedForNetwork = false;
+            consecutiveAuthFailures = 0;
+            if (wasPausedForAuth && !opts.quiet) {
+                process.stdout.write('\x1b[2K\r');
+                console.log(`${kleur.dim(timestamp())} ${kleur.green('✓')} Key accepted again — resuming.`);
+            }
+            if (wasPausedForNetwork && !wasPausedForAuth && !opts.quiet) {
+                process.stdout.write('\x1b[2K\r');
+                console.log(`${kleur.dim(timestamp())} ${kleur.green('✓')} Back online — resuming.`);
+            }
             const changedCount = r.added.length + r.modified.length + r.removed.length + r.renamed.length;
+            if (changedCount > 0) {
+                // Snap back to the fast interval the moment anything changes.
+                consecutiveIdleTicks = 0;
+                backoffSec = intervalSec;
+            }
+            else {
+                consecutiveIdleTicks++;
+                backoffSec = nextBackoffForIdle(consecutiveIdleTicks, intervalSec, {
+                    softSec: IDLE_SOFT_BACKOFF_SECONDS,
+                    hardSec: IDLE_HARD_BACKOFF_SECONDS,
+                });
+            }
             if (opts.quiet)
                 return;
             if (changedCount > 0) {
@@ -84,54 +152,141 @@ async function watchLoop(opts) {
             const apiErr = err instanceof ApiError ? err : null;
             const firstLine = err.message.split('\n')[0];
             process.stdout.write('\x1b[2K\r');
-            if (apiErr && (apiErr.status === 401 || apiErr.status === 403)) {
+            // Any error breaks the idle streak so we come back fast once it clears.
+            consecutiveIdleTicks = 0;
+            const decision = classifyTickError(err, {
+                backoffSec,
+                pausedForAuth,
+                pausedForNetwork,
+                consecutiveAuthFailures,
+                maxAuthFailures: MAX_AUTH_FAILURES,
+            });
+            if (decision.kind === 'auth') {
                 if (!pausedForAuth) {
-                    console.log(`${kleur.dim(timestamp())} ${kleur.red('✗')} Your key looks invalid or rotated.`);
+                    // Surface the request id (from B1) so the user can paste it into a
+                    // support thread and we can correlate against edge function logs.
+                    const idSuffix = apiErr?.requestId ? kleur.dim(` (request id: ${apiErr.requestId})`) : '';
+                    console.log(`${kleur.dim(timestamp())} ${kleur.red('✗')} Your key looks invalid or rotated.${idSuffix}`);
                     console.log(kleur.dim('   Fix: run `magicpixel login` (this watcher will keep retrying every 30s).'));
                 }
                 pausedForAuth = true;
-                backoffSec = 30;
+                consecutiveAuthFailures = decision.consecutiveAuthFailures;
+                backoffSec = decision.nextBackoffSec;
+                if (decision.giveUp) {
+                    console.log(`${kleur.dim(timestamp())} ${kleur.red('✗')} Giving up after ${MAX_AUTH_FAILURES} consecutive auth failures.`);
+                    console.log(kleur.dim('   Fix: run `magicpixel login` with a fresh key, then restart the watcher.'));
+                    // Surface this to /admin/errors — persistent watcher auth failure
+                    // means a key is mass-rejected (revoked, project deleted, edge
+                    // misconfig) and we want visibility without waiting for a support
+                    // ping. Awaited so the report flushes before exit.
+                    const { reportAndExit } = await import('../util/telemetry.js');
+                    await reportAndExit(err, 'sync (watch)', 2);
+                }
             }
-            else if (isNetworkError(err)) {
-                if (backoffSec < 30) {
+            else if (decision.kind === 'network') {
+                consecutiveAuthFailures = 0;
+                if (decision.printMessage) {
+                    const idSuffix = apiErr?.requestId ? kleur.dim(` (request id: ${apiErr.requestId})`) : '';
                     console.log(`${kleur.dim(timestamp())} ${kleur.yellow('!')} MagicPixel is offline or your internet is. ` +
-                        `Sprites you already have still work. Retrying in ${Math.min(backoffSec * 2, 60)}s.`);
+                        `Sprites you already have still work. Retrying in ${decision.nextBackoffSec}s.${idSuffix}`);
                 }
-                backoffSec = Math.min(backoffSec * 2, 60);
+                pausedForNetwork = true;
+                backoffSec = decision.nextBackoffSec;
             }
             else {
-                console.log(`${kleur.dim(timestamp())} ${kleur.red('!')} ${firstLine}`);
-                backoffSec = Math.min(backoffSec * 2, 60);
+                consecutiveAuthFailures = 0;
+                const idSuffix = apiErr?.requestId ? kleur.dim(` (request id: ${apiErr.requestId})`) : '';
+                console.log(`${kleur.dim(timestamp())} ${kleur.red('!')} ${firstLine}${idSuffix}`);
+                backoffSec = decision.nextBackoffSec;
             }
         }
         finally {
             inFlight = false;
-            resolveIdle?.();
-            resolveIdle = null;
         }
     };
     await tick();
     while (!stopping) {
-        await new Promise((r) => setTimeout(r, backoffSec * 1000));
+        // Race the sleep against a stop signal so Ctrl+C during a long error-
+        // backoff (up to 60s) exits within a tick instead of after the full wait.
+        await new Promise((resolve) => {
+            const timer = setTimeout(() => {
+                wakeStop = () => { };
+                resolve();
+            }, backoffSec * 1000);
+            wakeStop = () => {
+                clearTimeout(timer);
+                wakeStop = () => { };
+                resolve();
+            };
+        });
         if (stopping)
             break;
+        // `await tick()` only returns once its `finally` runs (clearing
+        // `inFlight`), so the loop naturally drains the in-flight sync on Ctrl+C
+        // before we exit — no separate idle-promise dance required.
         await tick();
     }
-    if (inFlight) {
-        await new Promise((r) => {
-            resolveIdle = r;
-        });
-    }
-    console.log(kleur.dim('[watch] stopped.'));
-    process.exit(0);
+    if (!opts.quiet)
+        console.log(kleur.dim('[watch] stopped.'));
+    // Preserve any non-zero exit code set by a failed tick — don't mask a
+    // download failure with a clean exit just because the user pressed Ctrl+C.
+    process.exit(process.exitCode ?? 0);
 }
 function isNetworkError(err) {
     // The api layer wraps fetch failures as `manifest: network error (...)`;
     // bare ENOTFOUND/ETIMEDOUT/etc. also surface here from `fetchAssetBytes`.
+    // Include 502/504 strings — corporate proxies often surface upstream gateway
+    // failures as terse text rather than as ApiError (e.g. 502 from a TLS
+    // terminator before our edge function ever sees the request).
     const msg = err.message;
     return (/network error/i.test(msg) ||
-        /ENOTFOUND|ETIMEDOUT|ECONNRESET|ECONNREFUSED|EAI_AGAIN/i.test(msg) ||
-        /fetch failed/i.test(msg));
+        /ENOTFOUND|ETIMEDOUT|ECONNRESET|ECONNREFUSED|EAI_AGAIN|ENETUNREACH|EHOSTUNREACH/i.test(msg) ||
+        /fetch failed/i.test(msg) ||
+        /\b(502|504)\b.*\b(bad gateway|gateway timeout)\b/i.test(msg));
+}
+/**
+ * Pure helper: given how many consecutive idle ticks we've seen and the
+ * configured poll interval, return the next backoff in seconds. Exported so
+ * the watch-mode regression test can guard the "ticks vs seconds" bug fixed
+ * in 0.4.0 without exercising the full watch loop.
+ */
+export function nextBackoffForIdle(consecutiveIdleTicks, intervalSec, thresholds = { softSec: 180, hardSec: 900 }) {
+    const idleSeconds = consecutiveIdleTicks * intervalSec;
+    if (idleSeconds >= thresholds.hardSec)
+        return Math.max(intervalSec, 10);
+    if (idleSeconds >= thresholds.softSec)
+        return Math.max(intervalSec, 5);
+    return intervalSec;
+}
+/**
+ * Pure helper: classify a per-tick error and compute the next backoff +
+ * counter updates. Extracted so the auth-failure watchdog and the
+ * network-recovery message can be unit-tested without spinning up signals or
+ * mocking timers.
+ *
+ * Caller is responsible for performing side effects (logging, `process.exit`,
+ * assigning the returned values back onto loop state).
+ */
+export function classifyTickError(err, state) {
+    const apiErr = err instanceof ApiError ? err : null;
+    if (apiErr && (apiErr.status === 401 || apiErr.status === 403)) {
+        const next = state.consecutiveAuthFailures + 1;
+        return {
+            kind: 'auth',
+            nextBackoffSec: 30,
+            consecutiveAuthFailures: next,
+            giveUp: next >= state.maxAuthFailures,
+        };
+    }
+    if (isNetworkError(err)) {
+        const nextBackoffSec = Math.min(state.backoffSec * 2, 60);
+        // Print on the first offline tick, then again whenever backoff changes —
+        // capped at 60s so users who walked away still see periodic confirmation
+        // the watcher is alive and retrying.
+        const printMessage = !state.pausedForNetwork || nextBackoffSec !== state.backoffSec;
+        return { kind: 'network', nextBackoffSec, printMessage };
+    }
+    return { kind: 'other', nextBackoffSec: Math.min(state.backoffSec * 2, 60) };
 }
 async function runOnce(opts, runOpts = {}) {
     const config = await loadConfig();
@@ -165,13 +320,28 @@ async function runOnce(opts, runOpts = {}) {
             renamed.push({ id: entry.id, oldKey: prevKey, newKey: entry.key });
         }
     }
-    // Diff against disk
+    // Compute the disk path once per entry and reuse it across the diff /
+    // orphan / download loops — saves three resolves + two security asserts
+    // per asset on large projects. Every entry in `manifest` is pre-seeded, so
+    // the lookup never misses; non-null assertion is safe.
+    const diskPathById = new Map();
+    for (const entry of manifest) {
+        diskPathById.set(entry.id, assetDiskPath(config.outDir, entry));
+    }
+    const pathFor = (entry) => diskPathById.get(entry.id);
+    // Diff against disk. SHA pre-check runs through the same concurrency pool
+    // used for downloads — on a 1k-asset project that's all-unchanged this is
+    // the dominant wall-clock cost of an incremental sync. We cache each result
+    // so the download loop can reuse it for `If-None-Match` ETags instead of
+    // re-hashing the same file moments later.
+    const diffLimit = createLimit(concurrency);
+    const shaByEntryId = new Map();
     const toDownload = [];
     let bytesSaved = 0;
     let unchanged = 0;
-    for (const entry of manifest) {
-        const diskPath = assetDiskPath(config.outDir, entry);
-        const localSha = await fileSha256(diskPath);
+    await Promise.all(manifest.map((entry) => diffLimit(async () => {
+        const localSha = await fileSha256(pathFor(entry));
+        shaByEntryId.set(entry.id, localSha);
         if (entry.sha256 && localSha && entry.sha256 === localSha) {
             unchanged++;
             if (entry.size_bytes)
@@ -180,14 +350,14 @@ async function runOnce(opts, runOpts = {}) {
         else {
             toDownload.push(entry);
         }
-    }
+    })));
     // Orphan detection only when we have the full picture.
     // (Renames also produce a stale path on disk — collected separately below.)
     let orphans = [];
     if (!since) {
-        const remoteDiskPaths = new Set(manifest.map((e) => assetDiskPath(config.outDir, e)));
-        orphans = (await findLocalPngs(resolve(process.cwd(), config.outDir)))
-            .filter((p) => !remoteDiskPaths.has(p));
+        const remoteDiskPaths = new Set(manifest.map((e) => pathFor(e)));
+        const localPngs = await walkOutDirPngs(config.outDir);
+        orphans = localPngs.map((a) => a.abs).filter((p) => !remoteDiskPaths.has(p));
     }
     // Stale paths from detected renames (always pruned, even in incremental mode —
     // otherwise the old PNG silently lingers next to the renamed copy).
@@ -251,13 +421,25 @@ async function runOnce(opts, runOpts = {}) {
     const run = createLimit(concurrency);
     let done = 0;
     await Promise.all(toDownload.map((entry) => run(async () => {
-        const diskPath = assetDiskPath(config.outDir, entry);
+        const diskPath = pathFor(entry);
         const existedBefore = existsSync(diskPath);
         try {
-            const localSha = await fileSha256(diskPath);
+            // Reuse the SHA computed during the diff pre-check — re-hashing the
+            // same file moments later was a measurable cost on large projects.
+            const localSha = shaByEntryId.get(entry.id) ?? null;
             const bytes = await fetchAssetBytes(config, entry.key, localSha);
             if (bytes === null) {
+                // Server returned 304 (ETag matched). Credit the asset's manifest
+                // size to bytesSaved so the end-of-sync summary reflects the
+                // bandwidth the conditional GET avoided.
+                //
+                // Invariant: we only reach here for entries that made it into
+                // `toDownload`, i.e. the disk-SHA pre-check above did NOT credit
+                // them (typically because `entry.sha256` was null in the manifest
+                // but the ETag still matched server-side). So no double-counting.
                 result.unchanged++;
+                if (entry.size_bytes)
+                    result.bytesSaved += entry.size_bytes;
             }
             else {
                 if (entry.sha256) {
@@ -266,11 +448,22 @@ async function runOnce(opts, runOpts = {}) {
                         throw new Error(`sha256 mismatch (expected ${entry.sha256.slice(0, 12)}…, got ${actual.slice(0, 12)}…)`);
                     }
                 }
-                await mkdir(dirname(diskPath), { recursive: true });
-                const tmp = tmpPathFor(diskPath);
-                assertPathInsideRoot(tmp, resolve(process.cwd(), config.outDir), 'outDir');
-                await writeFile(tmp, bytes);
-                await rename(tmp, diskPath);
+                // Wrap any of mkdir/writeFile/rename so the user sees a friendly
+                // multi-line diagnostic instead of a raw `EACCES: permission denied`.
+                try {
+                    await mkdir(dirname(diskPath), { recursive: true });
+                    const tmp = tmpPathFor(diskPath);
+                    assertPathInsideRoot(tmp, resolve(process.cwd(), config.outDir), 'outDir');
+                    await writeFile(tmp, bytes);
+                    await rename(tmp, diskPath);
+                }
+                catch (fsErr) {
+                    throw friendlyFsError(fsErr, {
+                        operation: `Writing asset`,
+                        path: diskPath,
+                        hint: `Sync can't continue until outDir (${config.outDir}) is writable.`,
+                    });
+                }
                 if (existedBefore)
                     result.modified.push(entry.key);
                 else
@@ -281,7 +474,17 @@ async function runOnce(opts, runOpts = {}) {
         catch (e) {
             result.failed++;
             progress?.stop();
-            console.log(`  ${kleur.red('!')} ${entry.key}: ${e.message.split('\n')[0]}`);
+            // Multi-line messages come from friendlyFsError — print all lines
+            // so the user sees the fix hint. Single-line errors stay terse.
+            const msg = e.message ?? String(e);
+            if (msg.includes('\n')) {
+                console.log(`  ${kleur.red('!')} ${entry.key}:`);
+                for (const line of msg.split('\n'))
+                    console.log(`     ${line}`);
+            }
+            else {
+                console.log(`  ${kleur.red('!')} ${entry.key}: ${msg}`);
+            }
             progress?.start();
         }
         finally {
@@ -339,7 +542,27 @@ async function runOnce(opts, runOpts = {}) {
     }
     const nextState = { ...state, assets: nextAssets };
     if (result.failed === 0) {
-        nextState.lastSync = startedAt;
+        // Advance the cursor to the newest row we actually observed, NOT the
+        // wall-clock time the sync started. Using startedAt silently skipped
+        // rows whose `updated_at` lived in the gap between the manifest snapshot
+        // and the next poll (notably when only metadata changed — e.g. artboard
+        // renames — and the row updated between fetch start and save).
+        const maxUpdatedAt = maxIsoTimestamp(manifest.map((e) => e.updated_at));
+        if (maxUpdatedAt) {
+            // Take the later of: newest row we saw, or the prior cursor. Never
+            // rewind — an incremental sync that returned 0 rows must keep the
+            // existing lastSync (otherwise we'd re-download history next tick).
+            nextState.lastSync =
+                state.lastSync && state.lastSync > maxUpdatedAt ? state.lastSync : maxUpdatedAt;
+        }
+        else if (state.lastSync) {
+            nextState.lastSync = state.lastSync;
+        }
+        else {
+            // No prior cursor and an empty manifest (fresh project): fall back to
+            // startedAt so future incremental polls have a baseline.
+            nextState.lastSync = startedAt;
+        }
         delete nextState.lastError;
     }
     else {
@@ -361,18 +584,20 @@ async function runOnce(opts, runOpts = {}) {
         const indexPath = await emitTypedIndex(config.outDir, diskEntries);
         if (verbose)
             console.log(kleur.dim(`  index → ${relative(process.cwd(), indexPath)}`));
-        // Nudge AI tools toward the typed index. Idempotent — `ensureAgentsDoc`
-        // no-ops once our marker section is present.
-        try {
-            const agentsResult = await ensureAgentsDoc(config.outDir);
-            if (verbose && agentsResult !== 'unchanged') {
-                console.log(kleur.dim(`  AGENTS.md ${agentsResult}`));
-            }
-        }
-        catch {
-            // Never let an AGENTS.md write failure break sync.
+    }
+    // AGENTS.md hint is always written — `public/` and `static/` users
+    // (emitIndex: false) need the absolute-URL snippet just as much as
+    // bundler-importable outDir users need the ES-import one.
+    // `ensureAgentsDoc` no-ops once our marker section is present.
+    try {
+        const agentsResult = await ensureAgentsDoc(config.outDir);
+        if (verbose && agentsResult !== 'unchanged') {
+            console.log(kleur.dim(`  AGENTS.md ${agentsResult}`));
         }
     }
+    catch {
+        // Never let an AGENTS.md write failure break sync.
+    }
     await saveState(nextState);
     if (result.failed > 0 && verbose) {
         console.log(kleur.yellow(`\n${result.failed} download${result.failed === 1 ? '' : 's'} failed — lastSync not advanced. Re-run to retry.`));
@@ -386,7 +611,10 @@ async function runOnce(opts, runOpts = {}) {
             (renamed.length ? `, renamed ${renamed.length}` : '') +
             (result.failed ? `, failed ${result.failed}` : '');
         console.log(result.failed ? kleur.yellow(`done with errors. ${summary}`) : kleur.green(`✓ done. ${summary}`));
-        printChanges(result, '  ');
+        // Per-file change list. Suppress renames here — printRenames below owns
+        // the rename block (it adds the import-update hints). Letting both fire
+        // would print each rename twice.
+        printChanges(result, '  ', { includeRenames: renamed.length === 0 });
         if (renamed.length > 0)
             printRenames(renamed, { withHints: true });
     }
@@ -395,14 +623,16 @@ async function runOnce(opts, runOpts = {}) {
     return result;
 }
 const CHANGE_PRINT_CAP = 50;
-function printChanges(r, indent) {
+function printChanges(r, indent, opts = { includeRenames: true }) {
     const lines = [];
     for (const k of r.added)
         lines.push(`${indent}${kleur.green('+')} ${k}`);
     for (const k of r.modified)
         lines.push(`${indent}${kleur.cyan('~')} ${k}`);
-    for (const r2 of r.renamed)
-        lines.push(`${indent}${kleur.cyan('↪')} ${r2.oldKey} → ${r2.newKey}`);
+    if (opts.includeRenames) {
+        for (const r2 of r.renamed)
+            lines.push(`${indent}${kleur.cyan('↪')} ${r2.oldKey} → ${r2.newKey}`);
+    }
     for (const k of r.removed)
         lines.push(`${indent}${kleur.red('-')} ${k}`);
     if (lines.length === 0)
@@ -437,13 +667,6 @@ function progressText(done, total, bytes) {
     const bar = '█'.repeat(filled) + '░'.repeat(barWidth - filled);
     return `${bar}  ${done}/${total}  ${kleur.dim(formatBytes(bytes))}`;
 }
-function formatBytes(n) {
-    if (n < 1024)
-        return `${n} B`;
-    if (n < 1024 * 1024)
-        return `${(n / 1024).toFixed(1)} KB`;
-    return `${(n / (1024 * 1024)).toFixed(1)} MB`;
-}
 function timestamp() {
     const d = new Date();
     return `[${String(d.getHours()).padStart(2, '0')}:${String(d.getMinutes()).padStart(2, '0')}:${String(d.getSeconds()).padStart(2, '0')}]`;
@@ -462,68 +685,7 @@ function printOrphans(orphans) {
     for (const p of orphans) {
         console.log(`  ${kleur.dim('?')} ${relative(process.cwd(), p)}`);
     }
-    console.log(kleur.dim('  Pruning is on by default. Re-run without --no-prune to delete them.'));
-}
-async function findLocalPngs(dir) {
-    if (!existsSync(dir))
-        return [];
-    const root = resolve(dir);
-    const out = [];
-    async function walk(d) {
-        const entries = await readdir(d, { withFileTypes: true });
-        for (const ent of entries) {
-            if (ent.isSymbolicLink())
-                continue;
-            const full = resolve(d, ent.name);
-            try {
-                assertPathInsideRoot(full, root, 'outDir');
-            }
-            catch {
-                continue;
-            }
-            if (ent.isDirectory()) {
-                if (ent.name.startsWith('.'))
-                    continue;
-                await walk(full);
-            }
-            else if (ent.isFile() && ent.name.toLowerCase().endsWith('.png')) {
-                out.push(full);
-            }
-        }
-    }
-    await walk(root);
-    return out;
-}
-async function pruneEmptyDirs(root) {
-    if (!existsSync(root))
-        return;
-    async function walk(d) {
-        const entries = await readdir(d, { withFileTypes: true });
-        let isEmpty = true;
-        for (const ent of entries) {
-            const full = resolve(d, ent.name);
-            if (ent.isDirectory()) {
-                const childEmpty = await walk(full);
-                if (childEmpty) {
-                    await rm(full, { recursive: true, force: true });
-                }
-                else {
-                    isEmpty = false;
-                }
-            }
-            else {
-                isEmpty = false;
-            }
-        }
-        return isEmpty;
-    }
-    const entries = await readdir(root, { withFileTypes: true });
-    for (const ent of entries) {
-        if (ent.isDirectory()) {
-            const full = resolve(root, ent.name);
-            const empty = await walk(full);
-            if (empty)
-                await rm(full, { recursive: true, force: true });
-        }
-    }
+    console.log(kleur.dim('  You passed --no-prune, so these were kept. Remove the flag to delete them.'));
 }
+// Note: orphan scan delegates to `walkOutDirPngs` in util/paths.ts (shared
+// with the typed-index emitter). `maxIsoTimestamp` lives in util/iso.ts.