@nucleoicons/skills 0.1.3 → 0.1.4

package/dist/api.d.ts

@@ -32,6 +32,10 @@ export interface SkillsReleaseData {
     files: SkillsReleaseFile[];
     message?: string;
 }
+export interface DownloadProgress {
+    transferred: number;
+    total: number | null;
+}
 /**
  * POST /skills/login
  * Body: { email, password }
@@ -43,5 +47,10 @@ export declare function skillsLogin(email: string, password: string): Promise<Sk
  * Response: { data: { version, families, files, downloadReady } }
  */
 export declare function fetchSkillsRelease(token: string, target: PackageTarget): Promise<SkillsReleaseData>;
-/** Download a remote file (signed R2 URL) to disk. */
-export declare function downloadUrlToFile(url: string, destPath: string): Promise<void>;
+declare function formatBytes(bytes: number): string;
+/** Download a remote file (signed R2 URL) to disk, with optional progress. */
+export declare function downloadUrlToFile(url: string, destPath: string, onProgress?: (progress: DownloadProgress) => void): Promise<{
+    bytes: number;
+    total: number | null;
+}>;
+export { formatBytes };

package/dist/api.js

@@ -1,6 +1,7 @@
 import { createWriteStream } from "node:fs";
 import { mkdir } from "node:fs/promises";
 import { dirname } from "node:path";
+import { Transform } from "node:stream";
 import { pipeline } from "node:stream/promises";
 import { Readable } from "node:stream";
 import { getApiBaseUrl } from "./config.js";
@@ -84,18 +85,58 @@ export async function fetchSkillsRelease(token, target) {
     }
     return json.data;
 }
-/** Download a remote file (signed R2 URL) to disk. */
-export async function downloadUrlToFile(url, destPath) {
-    const res = await fetch(url);
+function formatBytes(bytes) {
+    if (bytes < 1024) {
+        return `${bytes} B`;
+    }
+    if (bytes < 1024 * 1024) {
+        return `${(bytes / 1024).toFixed(1)} KB`;
+    }
+    return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
+}
+/** Download a remote file (signed R2 URL) to disk, with optional progress. */
+export async function downloadUrlToFile(url, destPath, onProgress) {
+    let res;
+    try {
+        res = await fetch(url);
+    }
+    catch (error) {
+        const detail = error instanceof Error ? error.message : String(error);
+        throw new ApiError(`Download failed (network): ${detail}`);
+    }
     if (!res.ok) {
-        throw new ApiError(`Download failed (${res.status}) for ${url}`, res.status);
+        throw new ApiError(`Download failed (${res.status}). The link may have expired — run setup again.`, res.status);
     }
     if (!res.body) {
         throw new ApiError("Download failed: empty response body.");
     }
+    const totalHeader = res.headers.get("content-length");
+    const total = totalHeader ? Number(totalHeader) : null;
+    let transferred = 0;
+    let lastLogged = 0;
     await mkdir(dirname(destPath), { recursive: true });
     const fileStream = createWriteStream(destPath);
-    // Node fetch body → Web ReadableStream; convert for pipeline
-    const nodeReadable = Readable.fromWeb(res.body);
-    await pipeline(nodeReadable, fileStream);
+    const progressTap = new Transform({
+        transform(chunk, _encoding, callback) {
+            transferred += chunk.length;
+            if (onProgress) {
+                onProgress({ transferred, total: Number.isFinite(total) ? total : null });
+            }
+            // Throttle console-friendly updates to ~every 2MB
+            if (transferred - lastLogged >= 2 * 1024 * 1024 || transferred === total) {
+                lastLogged = transferred;
+            }
+            callback(null, chunk);
+        },
+    });
+    try {
+        const nodeReadable = Readable.fromWeb(res.body);
+        await pipeline(nodeReadable, progressTap, fileStream);
+    }
+    catch (error) {
+        const detail = error instanceof Error ? error.message : String(error);
+        throw new ApiError(`Download interrupted after ${formatBytes(transferred)}${total ? ` of ${formatBytes(total)}` : ""}: ${detail}`);
+    }
+    return { bytes: transferred, total: Number.isFinite(total) ? total : null };
 }
+export { formatBytes };

package/dist/index.js

@@ -23,11 +23,32 @@ function parseTools(value) {
         .map((t) => t.trim().toLowerCase())
         .filter(Boolean);
 }
+let verbose = false;
+function printError(error) {
+    const message = error instanceof Error ? error.message : String(error);
+    console.error(`\nError: ${message}`);
+    if (verbose && error instanceof Error && error.stack) {
+        console.error(error.stack);
+    }
+    console.error("");
+}
+process.on("unhandledRejection", (reason) => {
+    printError(reason);
+    process.exit(1);
+});
+process.on("uncaughtException", (error) => {
+    printError(error);
+    process.exit(1);
+});
 const program = new Command();
 program
     .name("nucleo-skills")
     .description("Install and update Nucleo Pro MCP + icon skills")
-    .version(packageVersion());
+    .version(packageVersion())
+    .option("-v, --verbose", "Show full error details")
+    .hook("preAction", (thisCommand) => {
+    verbose = Boolean(thisCommand.opts().verbose);
+});
 program
     .command("setup")
     .description("Sign in, download your licensed package, configure your editor")
@@ -52,7 +73,7 @@ program
         });
     }
     catch (error) {
-        console.error(`\nError: ${error instanceof Error ? error.message : String(error)}\n`);
+        printError(error);
         process.exitCode = 1;
     }
 });
@@ -73,7 +94,7 @@ program
         });
     }
     catch (error) {
-        console.error(`\nError: ${error instanceof Error ? error.message : String(error)}\n`);
+        printError(error);
         process.exitCode = 1;
     }
 });
@@ -85,7 +106,7 @@ program
         await runStatus();
     }
     catch (error) {
-        console.error(`\nError: ${error instanceof Error ? error.message : String(error)}\n`);
+        printError(error);
         process.exitCode = 1;
     }
 });
@@ -97,7 +118,7 @@ program
         await runLogout();
     }
     catch (error) {
-        console.error(`\nError: ${error instanceof Error ? error.message : String(error)}\n`);
+        printError(error);
         process.exitCode = 1;
     }
 });

package/dist/install.d.ts

@@ -4,6 +4,8 @@ export declare function readInstallManifest(root?: string): Promise<InstallManif
 export declare function cleanInstallRoot(root?: string): Promise<void>;
 /**
  * Download all release files (signed URLs) and merge into the install root.
+ * Downloads complete first, then the install folder is replaced — so a failed
+ * download does not wipe a previous working install.
  */
 export declare function installFromRelease(release: SkillsReleaseData, target: PackageTarget, root?: string): Promise<void>;
 /**

package/dist/install.js

@@ -1,10 +1,10 @@
 import { existsSync } from "node:fs";
-import { cp, mkdir, readFile, rm, writeFile } from "node:fs/promises";
+import { cp, mkdir, readFile, rm, stat, writeFile } from "node:fs/promises";
 import { join } from "node:path";
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 import extract from "extract-zip";
-import { downloadUrlToFile } from "./api.js";
+import { downloadUrlToFile, formatBytes } from "./api.js";
 import { installRoot, nucleoHomeDir } from "./config.js";
 const execFileAsync = promisify(execFile);
 export async function readInstallManifest(root = installRoot()) {
@@ -39,26 +39,65 @@ async function extractZipMerge(zipPath, dest) {
 }
 /**
  * Download all release files (signed URLs) and merge into the install root.
+ * Downloads complete first, then the install folder is replaced — so a failed
+ * download does not wipe a previous working install.
  */
 export async function installFromRelease(release, target, root = installRoot()) {
     if (!release.downloadReady) {
         throw new Error(release.message ||
-            "Downloads are not ready on the server yet (R2 not configured or urls missing).");
+            "Downloads are not ready on the server yet. Please try again later.");
     }
     const missingUrl = release.files.filter((f) => !f.url);
     if (missingUrl.length > 0) {
         throw new Error(`Release files missing download urls: ${missingUrl.map((f) => f.id).join(", ")}`);
     }
-    await cleanInstallRoot(root);
     const downloadsDir = join(nucleoHomeDir(), "downloads");
     await mkdir(downloadsDir, { recursive: true });
+    const downloaded = [];
     try {
+        // 1) Download everything first (keeps existing install intact on failure)
         for (const file of release.files) {
             const zipPath = join(downloadsDir, `${file.id}-${release.version}.zip`);
-            console.log(`  ↓ ${file.id}`);
-            await downloadUrlToFile(file.url, zipPath);
-            await extractZipMerge(zipPath, root);
-            await rm(zipPath, { force: true }).catch(() => undefined);
+            process.stdout.write(`  ↓ ${file.id}`);
+            let lastLine = "";
+            try {
+                const result = await downloadUrlToFile(file.url, zipPath, ({ transferred, total }) => {
+                    const label = total
+                        ? `  ↓ ${file.id}  ${formatBytes(transferred)} / ${formatBytes(total)}`
+                        : `  ↓ ${file.id}  ${formatBytes(transferred)}`;
+                    // overwrite current line
+                    process.stdout.write(`\r${label}${" ".repeat(Math.max(0, lastLine.length - label.length))}`);
+                    lastLine = label;
+                });
+                const sizeLabel = result.total
+                    ? `${formatBytes(result.bytes)} / ${formatBytes(result.total)}`
+                    : formatBytes(result.bytes);
+                process.stdout.write(`\r  ↓ ${file.id}  ${sizeLabel}  ✓\n`);
+            }
+            catch (error) {
+                process.stdout.write("\n");
+                throw error;
+            }
+            const info = await stat(zipPath);
+            if (info.size <= 0) {
+                throw new Error(`Downloaded file is empty: ${file.id}`);
+            }
+            downloaded.push({ id: file.id, zipPath });
+        }
+        // 2) Only now replace the install root
+        console.log("  Extracting…");
+        await cleanInstallRoot(root);
+        for (const file of downloaded) {
+            console.log(`  ▸ ${file.id}`);
+            try {
+                await extractZipMerge(file.zipPath, root);
+            }
+            catch (error) {
+                const detail = error instanceof Error ? error.message : String(error);
+                throw new Error(`Failed extracting ${file.id} (${formatBytes((await stat(file.zipPath)).size)}). ${detail}\n` +
+                    `If this keeps happening, free disk space or try again with:\n` +
+                    `  NODE_OPTIONS=--max-old-space-size=8192 npx @nucleoicons/skills setup …`);
+            }
         }
     }
     finally {
@@ -116,11 +155,19 @@ export async function buildMcp(root = installRoot()) {
     }
     const distEntry = join(mcpDir, "dist", "index.js");
     const hasDist = existsSync(distEntry);
-    await execFileAsync("npm", ["install", "--omit=dev"], {
-        cwd: mcpDir,
-        env: process.env,
-    });
+    console.log("  npm install (mcp)…");
+    try {
+        await execFileAsync("npm", ["install", "--omit=dev"], {
+            cwd: mcpDir,
+            env: process.env,
+        });
+    }
+    catch (error) {
+        const detail = error instanceof Error ? error.message : String(error);
+        throw new Error(`MCP dependency install failed: ${detail}`);
+    }
     if (!hasDist || existsSync(join(mcpDir, "src"))) {
+        console.log("  npm run build (mcp)…");
         try {
             await execFileAsync("npm", ["run", "build"], {
                 cwd: mcpDir,
@@ -129,7 +176,8 @@ export async function buildMcp(root = installRoot()) {
         }
         catch (error) {
             if (!existsSync(distEntry)) {
-                throw error;
+                const detail = error instanceof Error ? error.message : String(error);
+                throw new Error(`MCP build failed: ${detail}`);
             }
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nucleoicons/skills",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "CLI to install and update Nucleo Pro MCP + icon skills",
   "type": "module",
   "bin": {