ht-skills 0.2.8 → 0.2.9

package/lib/cli.js

@@ -1538,6 +1538,10 @@ async function cmdPublish(flags, deps = {}) {
       width: Math.max(44, Math.min(outputWidth - 4, 72)),
     }));
   };
+  const setFlowStatus = (stepId, status) => {
+    flowState[stepId] = status;
+    renderPublishFlow();
+  };
 
   if (ui) {
     // eslint-disable-next-line no-console
@@ -1557,48 +1561,70 @@ async function cmdPublish(flags, deps = {}) {
     printFallbackPublishIntro({ registry, skillDir, archiveName, visibility }, log);
   }
 
-  flowState.auth = "active";
+  setFlowStatus("auth", "active");
   if (!ui) {
     log(`Checking login for ${registry}...`);
   }
   const { token } = await ensureValidAuthToken(registry, flags, deps);
-  flowState.auth = "done";
+  setFlowStatus("auth", "done");
 
   const packSpinner = ui ? ui.spinner() : null;
-  flowState.pack = "active";
+  setFlowStatus("pack", "active");
   if (packSpinner) {
     packSpinner.start(`Packing ${path.basename(skillDir) || skillDir}`);
   } else {
     log(`Packing skill directory: ${skillDir}`);
   }
-  const archiveBuffer = await createZipFromDirectory(skillDir);
-  flowState.pack = "done";
-  if (packSpinner) {
-    packSpinner.stop(`Created ${archiveName} (${formatBytes(archiveBuffer.length)})`);
-  } else {
-    log(`Created archive ${archiveName} (${formatBytes(archiveBuffer.length)})`);
+  let archiveBuffer;
+  try {
+    archiveBuffer = await createZipFromDirectory(skillDir);
+    setFlowStatus("pack", "done");
+    if (packSpinner) {
+      packSpinner.stop(`Created ${archiveName} (${formatBytes(archiveBuffer.length)})`);
+    } else {
+      log(`Created archive ${archiveName} (${formatBytes(archiveBuffer.length)})`);
+    }
+  } catch (error) {
+    setFlowStatus("pack", "error");
+    if (packSpinner) {
+      packSpinner.stop("Packing failed");
+    }
+    throw error;
   }
 
   const uploadSpinner = ui ? ui.spinner() : null;
-  flowState.inspect = "active";
+  setFlowStatus("inspect", "active");
   if (uploadSpinner) {
     uploadSpinner.start("Uploading archive for package inspection");
   } else {
     log("Uploading archive for package inspection...");
   }
-  const job = await requestJsonImpl(
-    `${registry}/api/skills/inspect-package-jobs/upload?archive_name=${encodeURIComponent(archiveName)}`,
-    {
-      method: "POST",
-      headers: withBearerToken({
-        "content-type": "application/zip",
-      }, token),
-      body: archiveBuffer,
-    },
-  );
+  let job;
+  try {
+    job = await requestJsonImpl(
+      `${registry}/api/skills/inspect-package-jobs/upload?archive_name=${encodeURIComponent(archiveName)}`,
+      {
+        method: "POST",
+        headers: withBearerToken({
+          "content-type": "application/zip",
+        }, token),
+        body: archiveBuffer,
+      },
+    );
+  } catch (error) {
+    setFlowStatus("inspect", "error");
+    if (uploadSpinner) {
+      uploadSpinner.stop("Inspection upload failed");
+    }
+    throw error;
+  }
 
   const jobId = String(job.job_id || "").trim();
   if (!jobId) {
+    setFlowStatus("inspect", "error");
+    if (uploadSpinner) {
+      uploadSpinner.stop("Inspection upload failed");
+    }
     throw new Error("registry did not return an inspection job id");
   }
   if (uploadSpinner) {
@@ -1617,7 +1643,7 @@ async function cmdPublish(flags, deps = {}) {
   }
   while (inspection.status !== "succeeded" && inspection.status !== "failed") {
     if (Date.now() > publishDeadline) {
-      flowState.inspect = "error";
+      setFlowStatus("inspect", "error");
       if (inspectSpinner) {
         inspectSpinner.stop("Inspection timed out");
       }
@@ -1636,7 +1662,7 @@ async function cmdPublish(flags, deps = {}) {
     } catch (error) {
       consecutivePollErrors += 1;
       if (consecutivePollErrors >= pollErrorLimit) {
-        flowState.inspect = "error";
+        setFlowStatus("inspect", "error");
         if (inspectSpinner) {
           inspectSpinner.stop("Inspection polling failed");
         }
@@ -1650,7 +1676,7 @@ async function cmdPublish(flags, deps = {}) {
 
     const runtimeError = String(inspection.error?.message || inspection.error || "").trim();
     if (runtimeError && inspection.status !== "succeeded") {
-      flowState.inspect = "error";
+      setFlowStatus("inspect", "error");
       if (inspectSpinner) {
         inspectSpinner.stop("Inspection failed");
       }
@@ -1674,13 +1700,13 @@ async function cmdPublish(flags, deps = {}) {
   }
 
   if (inspection.status !== "succeeded") {
-    flowState.inspect = "error";
+    setFlowStatus("inspect", "error");
     if (inspectSpinner) {
       inspectSpinner.stop("Inspection failed");
     }
     throw new Error(inspection.error || "skill archive inspection failed");
   }
-  flowState.inspect = "done";
+  setFlowStatus("inspect", "done");
   if (inspectSpinner) {
     inspectSpinner.stop("Inspection passed");
   } else {
@@ -1689,6 +1715,7 @@ async function cmdPublish(flags, deps = {}) {
 
   const preview = inspection.result || {};
   if (!preview.valid || !preview.preview_token) {
+    setFlowStatus("inspect", "error");
     throw new Error(summarizePreviewErrors(preview));
   }
   if (!ui) {
@@ -1710,24 +1737,33 @@ async function cmdPublish(flags, deps = {}) {
   }
 
   const submitSpinner = ui ? ui.spinner() : null;
-  flowState.submit = "active";
+  setFlowStatus("submit", "active");
   if (submitSpinner) {
     submitSpinner.start(`Submitting review request (${visibility})`);
   } else {
     log(`Submitting review request with access=${visibility}...`);
   }
-  const result = await requestJsonImpl(`${registry}/api/skills/submit`, {
-    method: "POST",
-    headers: withBearerToken({
-      "content-type": "application/json",
-    }, token),
-    body: JSON.stringify(body),
-  });
-  flowState.submit = "done";
-  if (submitSpinner) {
-    submitSpinner.stop(`Review submission created (${result.submission_id || "pending"})`);
-  } else {
-    log(`Review submission created: ${result.submission_id || "(no submission id returned)"}`);
+  let result;
+  try {
+    result = await requestJsonImpl(`${registry}/api/skills/submit`, {
+      method: "POST",
+      headers: withBearerToken({
+        "content-type": "application/json",
+      }, token),
+      body: JSON.stringify(body),
+    });
+    setFlowStatus("submit", "done");
+    if (submitSpinner) {
+      submitSpinner.stop(`Review submission created (${result.submission_id || "pending"})`);
+    } else {
+      log(`Review submission created: ${result.submission_id || "(no submission id returned)"}`);
+    }
+  } catch (error) {
+    setFlowStatus("submit", "error");
+    if (submitSpinner) {
+      submitSpinner.stop("Submit failed");
+    }
+    throw error;
   }
 
   const summaryPayload = {
@@ -1750,7 +1786,6 @@ async function cmdPublish(flags, deps = {}) {
       ].join("\n"),
       "Published",
     );
-    renderPublishFlow();
     ui.outro(`Submitted ${ui.pc.cyan(archiveName)} for review.`);
   } else {
     log(JSON.stringify(summaryPayload, null, 2));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ht-skills",
-  "version": "0.2.8",
+  "version": "0.2.9",
   "description": "CLI for installing and submitting skills from HT Skills Marketplace.",
   "type": "commonjs",
   "bin": {