@magicpixelart/cli 0.5.11 → 0.5.12

package/CHANGELOG.md

@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.5.12] — 2026-08-26
+
+### Fixed
+
+- **Watch counting no longer looks hung.** Sprite counting used to stay on
+  `Counting sprites in your game…` with no numbers until the first PNG, which
+  on a large Unity tree can take a long time. The spinner now updates as
+  folders are visited (`0 sprites · 128 folders`) and walks `Assets/` before
+  sibling project folders so the sprite count appears sooner.
+
 ## [0.5.11] — 2026-08-26
 
 ### Changed

package/dist/commands/connect.js

@@ -29,7 +29,7 @@ export async function connectCommand(glob) {
     }
     const spinner = ora({ text: countingSpritesText(0), spinner: 'dots' }).start();
     const index = await indexGamePngs(kind, process.cwd(), config.outDir, {
-        onProgress: (n) => { spinner.text = countingSpritesText(n); },
+        onProgress: (p) => { spinner.text = countingSpritesText(p); },
     });
     spinner.stop();
     if (index.capped) {

package/dist/commands/push.js

@@ -47,7 +47,7 @@ export async function runPushWith(config, state, opts = {}) {
     const index = opts.gameIndex
         ?? (isEngineKind(kind)
             ? await indexGamePngs(kind, process.cwd(), config.outDir, {
-                onProgress: spinner ? (n) => { spinner.text = countingSpritesText(n); } : undefined,
+                onProgress: spinner ? (p) => { spinner.text = countingSpritesText(p); } : undefined,
             })
             : { files: [], capped: false });
     const matched = matchConnectGlobs(index, config.connect ?? []);

package/dist/commands/search.js

@@ -28,7 +28,7 @@ export async function searchCommand(query) {
     }
     const spinner = ora({ text: countingSpritesText(0), spinner: 'dots' }).start();
     const index = await indexGamePngs(kind, process.cwd(), outDir, {
-        onProgress: (n) => { spinner.text = countingSpritesText(n); },
+        onProgress: (p) => { spinner.text = countingSpritesText(p); },
     });
     spinner.stop();
     if (index.capped) {

package/dist/commands/sync.js

@@ -68,7 +68,7 @@ async function watchLoop(opts) {
             /* header is cosmetic */
         }
     }
-    const header = await loadWatchHeader(countSpinner ? (n) => { countSpinner.text = countingSpritesText(n); } : undefined);
+    const header = await loadWatchHeader(countSpinner ? (p) => { countSpinner.text = countingSpritesText(p); } : undefined);
     if (countSpinner) {
         if (header.line)
             countSpinner.succeed(header.line.trim());
@@ -410,7 +410,13 @@ async function runOnce(opts, runOpts = {}) {
     const gameIndex = runOpts.gameIndex
         ?? (isEngineKind(projectKind)
             ? await indexGamePngs(projectKind, process.cwd(), config.outDir, {
-                onProgress: (n) => onStatus?.(`Looking through your game sprites…  ${n.toLocaleString('en-US')}`),
+                onProgress: (p) => {
+                    const n = p.pngs.toLocaleString('en-US');
+                    const f = p.folders.toLocaleString('en-US');
+                    onStatus?.(p.folders > 0
+                        ? `Looking through your game sprites…  ${n} sprites · ${f} folders`
+                        : `Looking through your game sprites…  ${n}`);
+                },
             })
             : { files: [], capped: false });
     const connected = matchConnectGlobs(gameIndex, config.connect ?? []);

package/dist/util/gameScan.js

@@ -23,6 +23,10 @@ export const GAME_SCAN_SKIP_DIRS = new Set([
     'streamingassets',
     'node_modules',
     'magicpixel',
+    'build',
+    'builds',
+    'recordings',
+    'memorycaptures',
 ]);
 /**
  * Dot-directories we never walk. Other hidden folders (e.g. `.SpineRaw_*`)
@@ -94,28 +98,41 @@ function toEntry(abs, cwdAbs, assetRoot) {
         return null;
     return { abs, sourceRel, adoptRel, key: adopted.path.join('/') };
 }
+function fmtCount(n, noun) {
+    return `${n.toLocaleString('en-US')} ${noun}${n === 1 ? '' : 's'}`;
+}
 /** Spinner / status copy while walking the game tree. */
-export function countingSpritesText(found) {
-    return found > 0
-        ? `Counting sprites in your game…  ${found.toLocaleString('en-US')}`
-        : 'Counting sprites in your game…';
+export function countingSpritesText(progress = 0) {
+    const pngs = typeof progress === 'number' ? progress : progress.pngs;
+    const folders = typeof progress === 'number' ? 0 : progress.folders;
+    if (pngs === 0 && folders === 0)
+        return 'Counting sprites in your game…';
+    const spriteBit = fmtCount(pngs, 'sprite');
+    if (folders > 0)
+        return `Counting sprites in your game…  ${spriteBit} · ${fmtCount(folders, 'folder')}`;
+    return `Counting sprites in your game…  ${spriteBit}`;
 }
 function bindScanProgress(onProgress) {
     if (!onProgress)
         return () => { };
     let lastAt = 0;
-    let lastN = 0;
-    return (n) => {
-        if (n === lastN)
+    let lastPngs = -1;
+    let lastFolders = -1;
+    return (pngs, folders) => {
+        if (pngs === lastPngs && folders === lastFolders)
             return;
         const now = Date.now();
-        if (lastN === 0 || now - lastAt >= 80) {
+        if (lastPngs < 0 || now - lastAt >= 80) {
             lastAt = now;
-            lastN = n;
-            onProgress(n);
+            lastPngs = pngs;
+            lastFolders = folders;
+            onProgress({ pngs, folders });
         }
     };
 }
+function samePath(a, b) {
+    return a.replace(/\\/g, '/').toLowerCase() === b.replace(/\\/g, '/').toLowerCase();
+}
 /** Walk the engine tree and return index entries. Does not copy files. */
 export async function indexGamePngs(kind, cwd = process.cwd(), outDir = '', opts = {}) {
     const empty = { files: [], capped: false };
@@ -128,11 +145,11 @@ export async function indexGamePngs(kind, cwd = process.cwd(), outDir = '', opts
     const namedRoot = gameScanRoot(kind);
     const assetRoot = namedRoot ? resolveChildDir(cwd, namedRoot) : null;
     const found = [];
+    const stats = { folders: 0 };
     const io = createLimit(SCAN_WALK_CONCURRENCY);
     const report = bindScanProgress(opts.onProgress);
-    await walkPngs(scanRoot, scanRoot, destNorm, found, MAX_GAME_INDEX + 1, io, report);
-    if (found.length > 0)
-        opts.onProgress?.(found.length);
+    await walkPngs(scanRoot, scanRoot, destNorm, found, MAX_GAME_INDEX + 1, io, report, stats, assetRoot);
+    opts.onProgress?.({ pngs: found.length, folders: stats.folders });
     found.sort((a, b) => a.localeCompare(b));
     const capped = found.length > MAX_GAME_INDEX;
     if (capped)
@@ -161,9 +178,12 @@ export function searchGameIndex(index, query) {
         return [];
     return index.files.filter((e) => e.sourceRel.toLowerCase().includes(q) || e.key.includes(q) || e.adoptRel.toLowerCase().includes(q));
 }
-async function walkPngs(dir, root, destNorm, out, limit, io, onFound) {
+async function walkPngs(dir, root, destNorm, out, limit, io, onVisit, stats, preferFirst) {
     if (out.length >= limit)
         return;
+    // Count the folder *before* readdir so a huge listing still shows movement.
+    stats.folders++;
+    onVisit(out.length, stats.folders);
     let entries;
     try {
         entries = await readdir(dir, { withFileTypes: true });
@@ -172,7 +192,8 @@ async function walkPngs(dir, root, destNorm, out, limit, io, onFound) {
         return;
     }
     entries.sort((a, b) => a.name.localeCompare(b.name));
-    const subdirs = [];
+    const preferred = [];
+    const other = [];
     for (const ent of entries) {
         if (out.length >= limit)
             return;
@@ -190,14 +211,25 @@ async function walkPngs(dir, root, destNorm, out, limit, io, onFound) {
                 continue;
             if (destNorm && full.replace(/\\/g, '/').toLowerCase() === destNorm)
                 continue;
-            subdirs.push(full);
+            if (preferFirst && samePath(full, preferFirst))
+                preferred.push(full);
+            else
+                other.push(full);
         }
         else if (ent.isFile() && ent.name.toLowerCase().endsWith('.png')) {
             out.push(full);
-            onFound?.(out.length);
+            onVisit(out.length, stats.folders);
         }
     }
-    if (subdirs.length === 0 || out.length >= limit)
+    if (out.length >= limit)
+        return;
+    // Engine asset tree first (Unity `Assets/`) so sprite counts appear before
+    // sibling folders like Builds/Editor. Nested walks keep preferFirst so a
+    // misplaced `Assets` dir is still prioritized if we ever see it.
+    for (const d of preferred) {
+        await walkPngs(d, root, destNorm, out, limit, io, onVisit, stats, null);
+    }
+    if (other.length === 0 || out.length >= limit)
         return;
-    await Promise.all(subdirs.map((d) => io(() => walkPngs(d, root, destNorm, out, limit, io, onFound))));
+    await Promise.all(other.map((d) => io(() => walkPngs(d, root, destNorm, out, limit, io, onVisit, stats, null))));
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@magicpixelart/cli",
-  "version": "0.5.11",
+  "version": "0.5.12",
   "description": "Sync MagicPixel pixel-art assets to your local project.",
   "type": "module",
   "bin": {