ht-skills 0.2.2 → 0.2.4

package/lib/cli.js

@@ -242,7 +242,7 @@ function summarizePreviewErrors(preview) {
 async function promptToOpenBrowser(url, deps = {}) {
   const ask = deps.ask || null;
   if (ask) {
-    await ask(`Press Enter to open the browser for:\n${url}\n`);
+    await ask("Press ENTER to open in the browser...\n");
     return;
   }
 
@@ -259,12 +259,38 @@ async function promptToOpenBrowser(url, deps = {}) {
     output: process.stdout,
   });
   try {
-    await rl.question(`Press Enter to open the browser for:\n${url}\n`);
+    await rl.question("Press ENTER to open in the browser...\n");
   } finally {
     rl.close();
   }
 }
 
+function startLoginWaitingIndicator(deps = {}) {
+  const isInteractive = typeof deps.isInteractive === "boolean"
+    ? deps.isInteractive
+    : Boolean(process.stdin.isTTY && process.stdout.isTTY);
+
+  if (!isInteractive || !process.stdout.isTTY) {
+    return () => {};
+  }
+
+  const frames = ["|", "/", "-", "\\"];
+  let frameIndex = 0;
+  process.stdout.write("Waiting for browser sign-in... ");
+  const timer = setInterval(() => {
+    const frame = frames[frameIndex % frames.length];
+    frameIndex += 1;
+    process.stdout.write(`\rWaiting for browser sign-in... ${frame}`);
+  }, 120);
+
+  return () => {
+    clearInterval(timer);
+    process.stdout.write("\r");
+    process.stdout.clearLine(0);
+    process.stdout.cursorTo(0);
+  };
+}
+
 function formatInstallError(error, { slug, version = null, registry, stage = "resolve" }) {
   const rawMessage = String(error?.message || "Install failed").trim();
   const status = Number(error?.status || 0);
@@ -1279,33 +1305,38 @@ async function cmdLogin(flags, deps = {}) {
     throw new Error("registry did not return a usable CLI login flow");
   }
 
-  log(`Complete CLI sign-in here:\n${browserUrl}`);
+  log(`Authenticate your account at:\n${browserUrl}`);
   await promptToOpenBrowser(browserUrl, deps);
 
   try {
     await Promise.resolve(openBrowser(browserUrl));
-    log(`Opened browser for ${registry}. Complete sign-in to continue...`);
   } catch (error) {
     log(`Open this URL in your browser and sign in:\n${browserUrl}`);
   }
 
   const expiresAtMs = Date.parse(started.expires_at || "") || (Date.now() + timeoutMs);
   const deadline = Math.min(Date.now() + timeoutMs, expiresAtMs);
+  const stopIndicator = startLoginWaitingIndicator(deps);
 
-  while (Date.now() <= deadline) {
-    await sleep(pollIntervalMs);
-    const status = await requestJsonImpl(pollUrl);
-    if (status.status === "approved") {
-      await setStoredRegistryAuth(registry, {
-        token: status.token,
-        user: status.user || null,
-        tokenExpiresAt: status.token_expires_at || null,
-        savedAt: new Date().toISOString(),
-      }, { homeDir });
-      const userLabel = status.user?.name || status.user?.email || status.user?.id || "user";
-      log(`Signed in to ${registry} as ${userLabel}.`);
-      return status;
+  try {
+    while (Date.now() <= deadline) {
+      await sleep(pollIntervalMs);
+      const status = await requestJsonImpl(pollUrl);
+      if (status.status === "approved") {
+        await setStoredRegistryAuth(registry, {
+          token: status.token,
+          user: status.user || null,
+          tokenExpiresAt: status.token_expires_at || null,
+          savedAt: new Date().toISOString(),
+        }, { homeDir });
+        const userLabel = status.user?.name || status.user?.email || status.user?.id || "user";
+        stopIndicator();
+        log(`Signed in to ${registry} as ${userLabel}.`);
+        return status;
+      }
     }
+  } finally {
+    stopIndicator();
   }
 
   throw new Error(`Timed out waiting for browser sign-in at ${registry}`);

package/package.json

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