From 90809c47ab62c17ffe133be0c6cb8afd3e60e4b8 Mon Sep 17 00:00:00 2001
From: chenbaowang <49091147+Rsweater@users.noreply.github.com>
Date: Fri, 17 Apr 2026 18:49:23 +0800
Subject: [PATCH] fix(cli): stop spinner before version selection to prevent UI
 conflict

The ora spinner was still running during p.select() interactive prompt,
causing the spinner to continuously overwrite the terminal and block
the version selection UI. The user would see "Fetching..." stuck until
pressing Enter.

Stop spinner before p.select() and restart after selection completes.
---
 skillhub-cli/src/commands/install.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/skillhub-cli/src/commands/install.ts b/skillhub-cli/src/commands/install.ts
index 1217308e..c8e27706 100644
--- a/skillhub-cli/src/commands/install.ts
+++ b/skillhub-cli/src/commands/install.ts
@@ -260,6 +260,8 @@ async function installFromRegistry(slug: string, opts: Record<string, string | s
     }
   } else {
     // Interactive: show version selection
+    spinner.stop();
+
     const picked = await p.select({
       message: "Select version",
       options: versions.map((v) => ({
@@ -275,6 +277,7 @@ async function installFromRegistry(slug: string, opts: Record<string, string | s
     }
 
     selectedVersion = picked as string;
+    spinner.start("Downloading");
   }
 
   const baseUrl = config.registry.replace(/\/$/, "");