ht-skills 0.2.7 → 0.2.8

package/lib/cli.js

@@ -36,6 +36,8 @@ const CONFIG_FILE_NAME = "config.json";
 const DEFAULT_LOGIN_TIMEOUT_MS = 5 * 60 * 1000;
 const DEFAULT_LOGIN_POLL_MS = 1500;
 const DEFAULT_PUBLISH_POLL_MS = 1500;
+const DEFAULT_PUBLISH_TIMEOUT_MS = 5 * 60 * 1000;
+const DEFAULT_PUBLISH_POLL_ERROR_LIMIT = 3;
 
 const BANNER_TEXT = String.raw`  _  _ _____   ___ _   _ _ _      __  __          _       _        _         
  | || |_   _| / __| |_(_) | |___ |  \/  |__ _ _ _| |_____| |_ _ __| |__ _ __ 
@@ -1519,6 +1521,8 @@ async function cmdPublish(flags, deps = {}) {
   const colorize = ui ? createUiColorizer(ui.pc) : (text) => String(text);
   const outputWidth = getOutputWidth(deps.outputWidth);
   const pollIntervalMs = Math.max(500, Number(flags["poll-interval"] || deps.pollIntervalMs || DEFAULT_PUBLISH_POLL_MS));
+  const publishTimeoutMs = Math.max(10_000, Number(flags.timeout || deps.timeoutMs || DEFAULT_PUBLISH_TIMEOUT_MS));
+  const pollErrorLimit = Math.max(1, Number(flags["poll-error-limit"] || deps.pollErrorLimit || DEFAULT_PUBLISH_POLL_ERROR_LIMIT));
   const flowState = {
     auth: "pending",
     pack: "pending",
@@ -1553,24 +1557,15 @@ async function cmdPublish(flags, deps = {}) {
     printFallbackPublishIntro({ registry, skillDir, archiveName, visibility }, log);
   }
 
-  const authSpinner = ui ? ui.spinner() : null;
   flowState.auth = "active";
-  renderPublishFlow();
-  if (authSpinner) {
-    authSpinner.start("Checking login");
-  } else {
+  if (!ui) {
     log(`Checking login for ${registry}...`);
   }
   const { token } = await ensureValidAuthToken(registry, flags, deps);
   flowState.auth = "done";
-  renderPublishFlow();
-  if (authSpinner) {
-    authSpinner.stop("Login confirmed");
-  }
 
   const packSpinner = ui ? ui.spinner() : null;
   flowState.pack = "active";
-  renderPublishFlow();
   if (packSpinner) {
     packSpinner.start(`Packing ${path.basename(skillDir) || skillDir}`);
   } else {
@@ -1578,7 +1573,6 @@ async function cmdPublish(flags, deps = {}) {
   }
   const archiveBuffer = await createZipFromDirectory(skillDir);
   flowState.pack = "done";
-  renderPublishFlow();
   if (packSpinner) {
     packSpinner.stop(`Created ${archiveName} (${formatBytes(archiveBuffer.length)})`);
   } else {
@@ -1587,7 +1581,6 @@ async function cmdPublish(flags, deps = {}) {
 
   const uploadSpinner = ui ? ui.spinner() : null;
   flowState.inspect = "active";
-  renderPublishFlow();
   if (uploadSpinner) {
     uploadSpinner.start("Uploading archive for package inspection");
   } else {
@@ -1616,18 +1609,53 @@ async function cmdPublish(flags, deps = {}) {
 
   let inspection = job;
   let lastProgressKey = "";
+  let consecutivePollErrors = 0;
+  const publishDeadline = Date.now() + publishTimeoutMs;
   const inspectSpinner = ui ? ui.spinner() : null;
   if (inspectSpinner) {
     inspectSpinner.start("Inspecting archive");
   }
   while (inspection.status !== "succeeded" && inspection.status !== "failed") {
+    if (Date.now() > publishDeadline) {
+      flowState.inspect = "error";
+      if (inspectSpinner) {
+        inspectSpinner.stop("Inspection timed out");
+      }
+      throw new Error(`Inspection timed out after ${Math.round(publishTimeoutMs / 1000)}s`);
+    }
+
     await sleep(pollIntervalMs);
-    inspection = await requestJsonImpl(
-      `${registry}/api/skills/inspect-package-jobs/${encodeURIComponent(jobId)}`,
-      {
-        headers: withBearerToken({}, token),
-      },
-    );
+    try {
+      inspection = await requestJsonImpl(
+        `${registry}/api/skills/inspect-package-jobs/${encodeURIComponent(jobId)}`,
+        {
+          headers: withBearerToken({}, token),
+        },
+      );
+      consecutivePollErrors = 0;
+    } catch (error) {
+      consecutivePollErrors += 1;
+      if (consecutivePollErrors >= pollErrorLimit) {
+        flowState.inspect = "error";
+        if (inspectSpinner) {
+          inspectSpinner.stop("Inspection polling failed");
+        }
+        throw new Error(`Inspection polling failed ${consecutivePollErrors} times: ${error.message}`);
+      }
+      if (!inspectSpinner) {
+        log(`Inspection polling retry ${consecutivePollErrors}/${pollErrorLimit}: ${error.message}`);
+      }
+      continue;
+    }
+
+    const runtimeError = String(inspection.error?.message || inspection.error || "").trim();
+    if (runtimeError && inspection.status !== "succeeded") {
+      flowState.inspect = "error";
+      if (inspectSpinner) {
+        inspectSpinner.stop("Inspection failed");
+      }
+      throw new Error(runtimeError);
+    }
 
     const step = String(inspection.progress?.step || inspection.result?.step || "").trim();
     const percent = inspection.progress?.percent;
@@ -1647,14 +1675,12 @@ async function cmdPublish(flags, deps = {}) {
 
   if (inspection.status !== "succeeded") {
     flowState.inspect = "error";
-    renderPublishFlow();
     if (inspectSpinner) {
       inspectSpinner.stop("Inspection failed");
     }
     throw new Error(inspection.error || "skill archive inspection failed");
   }
   flowState.inspect = "done";
-  renderPublishFlow();
   if (inspectSpinner) {
     inspectSpinner.stop("Inspection passed");
   } else {
@@ -1685,7 +1711,6 @@ async function cmdPublish(flags, deps = {}) {
 
   const submitSpinner = ui ? ui.spinner() : null;
   flowState.submit = "active";
-  renderPublishFlow();
   if (submitSpinner) {
     submitSpinner.start(`Submitting review request (${visibility})`);
   } else {
@@ -1699,7 +1724,6 @@ async function cmdPublish(flags, deps = {}) {
     body: JSON.stringify(body),
   });
   flowState.submit = "done";
-  renderPublishFlow();
   if (submitSpinner) {
     submitSpinner.stop(`Review submission created (${result.submission_id || "pending"})`);
   } else {
@@ -1726,6 +1750,7 @@ 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.7",
+  "version": "0.2.8",
   "description": "CLI for installing and submitting skills from HT Skills Marketplace.",
   "type": "commonjs",
   "bin": {