ht-skills 0.2.6 → 0.2.7

package/lib/cli.js

@@ -819,6 +819,64 @@ function formatSearchCard(item, { index = 0, colorize = (text) => text, width =
   return [title, blank, firstLine, ...detailRendered, blank, bottom].join("\n");
 }
 
+const PUBLISH_FLOW_STEPS = [
+  { id: "auth", label: "Login" },
+  { id: "pack", label: "Pack" },
+  { id: "inspect", label: "Inspect" },
+  { id: "submit", label: "Submit" },
+];
+
+function getPublishStepStatusLabel(status) {
+  switch (status) {
+    case "done":
+      return "Done";
+    case "active":
+      return "Active";
+    case "error":
+      return "Error";
+    default:
+      return "Pending";
+  }
+}
+
+function renderPublishFlowCard(flowState, { colorize = (text) => text, width = 80 } = {}) {
+  const maxInnerWidth = Math.max(28, width - 6);
+  const lines = PUBLISH_FLOW_STEPS.map((step, index) => {
+    const status = flowState?.[step.id] || "pending";
+    const symbol = status === "done"
+      ? colorize("●", "success")
+      : status === "active"
+        ? colorize("◉", "accent")
+        : status === "error"
+          ? colorize("▲", "warn")
+          : colorize("○", "muted");
+    const label = status === "done"
+      ? colorize(step.label, "success")
+      : status === "active"
+        ? colorize(step.label, "accent")
+        : status === "error"
+          ? colorize(step.label, "warn")
+          : step.label;
+    const statusLabel = colorize(getPublishStepStatusLabel(status), status === "pending" ? "muted" : status === "error" ? "warn" : status === "active" ? "accent" : "success");
+    const plainLine = `${index + 1}. ${step.label}  ${getPublishStepStatusLabel(status)}`;
+    const paddedWidth = Math.max(0, maxInnerWidth - getDisplayWidth(plainLine));
+    return `│ ${symbol} ${label}${" ".repeat(paddedWidth)} ${statusLabel} │`;
+  });
+
+  const contentWidth = Math.max(
+    28,
+    ...PUBLISH_FLOW_STEPS.map((step, index) => getDisplayWidth(`${index + 1}. ${step.label}  Pending`)),
+  );
+  const title = `◇  ${colorize("Publish Flow", "accent")} ${colorize("─".repeat(Math.max(0, contentWidth - getDisplayWidth("Publish Flow") - 1)), "muted")}╮`;
+  const normalizedLines = lines.map((line) => {
+    const plainWidth = getDisplayWidth(line);
+    const targetWidth = contentWidth + 4;
+    return plainWidth < targetWidth ? `${line.slice(0, -2)}${" ".repeat(targetWidth - plainWidth)} │` : line;
+  });
+  const bottom = `╰${"─".repeat(contentWidth + 2)}╯`;
+  return [title, ...normalizedLines, bottom].join("\n");
+}
+
 function printFallbackIntro({ registry, slug, version, skillName, skillDescription, installTargets }, log = console.log) {
   log("");
   log("ht-skills");
@@ -848,6 +906,19 @@ function printFallbackSearchIntro({ registry, query, limit }, log = console.log)
   log("");
 }
 
+function printFallbackPublishIntro({ registry, skillDir, archiveName, visibility }, log = console.log) {
+  log("");
+  log(renderGradientBanner());
+  log("");
+  log("Publish");
+  log("");
+  log(`Source: ${registry}`);
+  log(`Directory: ${skillDir}`);
+  log(`Archive: ${archiveName}`);
+  log(`Access: ${visibility}`);
+  log("");
+}
+
 async function loadTerminalUi() {
   const [{ intro, outro, note, multiselect, select, spinner, isCancel, cancel }, pcModule] = await Promise.all([
     import("@clack/prompts"),
@@ -1440,17 +1511,88 @@ async function cmdPublish(flags, deps = {}) {
   const log = deps.log || ((message) => console.log(message));
   const registry = getRegistryUrl(flags);
   const skillDir = path.resolve(flags._[0] || ".");
-  log(`Checking login for ${registry}...`);
+  const visibility = String(flags.access || flags.visibility || "public").trim().toLowerCase() || "public";
+  const archiveName = `${path.basename(skillDir) || "skill"}.zip`;
+  const isInteractive = isInteractiveSession(deps);
+  const canUseFancyUi = Boolean(isInteractive && !deps.disableUi);
+  const ui = deps.ui || (canUseFancyUi ? await loadTerminalUi().catch(() => null) : null);
+  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 flowState = {
+    auth: "pending",
+    pack: "pending",
+    inspect: "pending",
+    submit: "pending",
+  };
+
+  const renderPublishFlow = () => {
+    if (!ui) return;
+    // eslint-disable-next-line no-console
+    console.log(renderPublishFlowCard(flowState, {
+      colorize,
+      width: Math.max(44, Math.min(outputWidth - 4, 72)),
+    }));
+  };
+
+  if (ui) {
+    // eslint-disable-next-line no-console
+    console.log(`\n${renderGradientBanner()}\n`);
+    ui.intro(ui.pc.bgCyan(ui.pc.black(" ht-skills ")));
+    ui.note(
+      [
+        `${colorize("Source", "muted")}: ${registry}`,
+        `${colorize("Directory", "muted")}: ${skillDir}`,
+        `${colorize("Archive", "muted")}: ${archiveName}`,
+        `${colorize("Access", "muted")}: ${visibility}`,
+      ].join("\n"),
+      "Publish",
+    );
+    renderPublishFlow();
+  } else {
+    printFallbackPublishIntro({ registry, skillDir, archiveName, visibility }, log);
+  }
+
+  const authSpinner = ui ? ui.spinner() : null;
+  flowState.auth = "active";
+  renderPublishFlow();
+  if (authSpinner) {
+    authSpinner.start("Checking login");
+  } else {
+    log(`Checking login for ${registry}...`);
+  }
   const { token } = await ensureValidAuthToken(registry, flags, deps);
+  flowState.auth = "done";
+  renderPublishFlow();
+  if (authSpinner) {
+    authSpinner.stop("Login confirmed");
+  }
 
-  const archiveName = `${path.basename(skillDir) || "skill"}.zip`;
-  log(`Packing skill directory: ${skillDir}`);
+  const packSpinner = ui ? ui.spinner() : null;
+  flowState.pack = "active";
+  renderPublishFlow();
+  if (packSpinner) {
+    packSpinner.start(`Packing ${path.basename(skillDir) || skillDir}`);
+  } else {
+    log(`Packing skill directory: ${skillDir}`);
+  }
   const archiveBuffer = await createZipFromDirectory(skillDir);
-  log(`Created archive ${archiveName} (${formatBytes(archiveBuffer.length)})`);
-
-  const pollIntervalMs = Math.max(500, Number(flags["poll-interval"] || deps.pollIntervalMs || DEFAULT_PUBLISH_POLL_MS));
+  flowState.pack = "done";
+  renderPublishFlow();
+  if (packSpinner) {
+    packSpinner.stop(`Created ${archiveName} (${formatBytes(archiveBuffer.length)})`);
+  } else {
+    log(`Created archive ${archiveName} (${formatBytes(archiveBuffer.length)})`);
+  }
 
-  log("Uploading archive for package inspection...");
+  const uploadSpinner = ui ? ui.spinner() : null;
+  flowState.inspect = "active";
+  renderPublishFlow();
+  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)}`,
     {
@@ -1466,10 +1608,18 @@ async function cmdPublish(flags, deps = {}) {
   if (!jobId) {
     throw new Error("registry did not return an inspection job id");
   }
-  log(`Inspection job created: ${jobId}`);
+  if (uploadSpinner) {
+    uploadSpinner.stop(`Inspection job created (${jobId})`);
+  } else {
+    log(`Inspection job created: ${jobId}`);
+  }
 
   let inspection = job;
   let lastProgressKey = "";
+  const inspectSpinner = ui ? ui.spinner() : null;
+  if (inspectSpinner) {
+    inspectSpinner.start("Inspecting archive");
+  }
   while (inspection.status !== "succeeded" && inspection.status !== "failed") {
     await sleep(pollIntervalMs);
     inspection = await requestJsonImpl(
@@ -1487,22 +1637,38 @@ async function cmdPublish(flags, deps = {}) {
       const progressLabel = step
         ? `${step}${typeof percent === "number" ? ` ${percent}%` : ""}`
         : inspection.status;
-      log(`Inspection in progress: ${progressLabel}`);
+      if (inspectSpinner) {
+        inspectSpinner.message(`Inspecting archive (${progressLabel})`);
+      } else {
+        log(`Inspection in progress: ${progressLabel}`);
+      }
     }
   }
 
   if (inspection.status !== "succeeded") {
+    flowState.inspect = "error";
+    renderPublishFlow();
+    if (inspectSpinner) {
+      inspectSpinner.stop("Inspection failed");
+    }
     throw new Error(inspection.error || "skill archive inspection failed");
   }
-  log("Inspection passed.");
+  flowState.inspect = "done";
+  renderPublishFlow();
+  if (inspectSpinner) {
+    inspectSpinner.stop("Inspection passed");
+  } else {
+    log("Inspection passed.");
+  }
 
   const preview = inspection.result || {};
   if (!preview.valid || !preview.preview_token) {
     throw new Error(summarizePreviewErrors(preview));
   }
-  log(`Preview token created: ${preview.preview_token}`);
+  if (!ui) {
+    log(`Preview token created: ${preview.preview_token}`);
+  }
 
-  const visibility = String(flags.access || flags.visibility || "public").trim().toLowerCase() || "public";
   const body = {
     preview_token: preview.preview_token,
     visibility,
@@ -1517,7 +1683,14 @@ async function cmdPublish(flags, deps = {}) {
     body.publish_now = true;
   }
 
-  log(`Submitting review request with access=${visibility}...`);
+  const submitSpinner = ui ? ui.spinner() : null;
+  flowState.submit = "active";
+  renderPublishFlow();
+  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({
@@ -1525,9 +1698,15 @@ async function cmdPublish(flags, deps = {}) {
     }, token),
     body: JSON.stringify(body),
   });
-  log(`Review submission created: ${result.submission_id || "(no submission id returned)"}`);
+  flowState.submit = "done";
+  renderPublishFlow();
+  if (submitSpinner) {
+    submitSpinner.stop(`Review submission created (${result.submission_id || "pending"})`);
+  } else {
+    log(`Review submission created: ${result.submission_id || "(no submission id returned)"}`);
+  }
 
-  log(JSON.stringify({
+  const summaryPayload = {
     status: result.status,
     submission_id: result.submission_id || null,
     created_at: result.created_at || null,
@@ -1535,7 +1714,22 @@ async function cmdPublish(flags, deps = {}) {
     publication: result.publication || null,
     preview_token: preview.preview_token,
     archive_name: archiveName,
-  }, null, 2));
+  };
+
+  if (ui) {
+    ui.note(
+      [
+        `${colorize("Status", "muted")}: ${result.status || "pending"}`,
+        `${colorize("Submission", "muted")}: ${result.submission_id || "-"}`,
+        `${colorize("Preview", "muted")}: ${preview.preview_token}`,
+        `${colorize("Access", "muted")}: ${summaryPayload.visibility || visibility}`,
+      ].join("\n"),
+      "Published",
+    );
+    ui.outro(`Submitted ${ui.pc.cyan(archiveName)} for review.`);
+  } else {
+    log(JSON.stringify(summaryPayload, null, 2));
+  }
 
   return result;
 }

package/package.json

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