scriptoriumadmin/skillhub
1
0
Fork 0
mirror of https://github.com/iflytek/skillhub.git synced 2026-09-15 23:31:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix(cli): sort agent lists alphabetically and fix spinner residue

Browse source 
- Sort universal and non-universal agent lists by name in selectAgentsInteractive
- Exclude agents with showInUniversalList===false from locked section
- Replace spinner.stop() with spinner.succeed() before interactive prompts
  to prevent 'Fetching' text from appearing as the step title
- Also fix spinner.stop -> spinner.succeed for 'Installation complete'
This commit is contained in:
chenbaowang 2026-04-20 16:36:09 +08:00
parent 3353953675
commit aaf80dbfd8
1 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
skillhub-cli/src/commands/install.ts
View file

@ -57,8 +57,13 @@ async function selectAgentsInteractive(isGlobal: boolean): Promise<string[] | nu
// Use scope-aware universal check: an agent is "universal" when its target
// install dir equals the canonical .agents/skills directory for the given scope.
const universalAgents = allAgents.filter((a) => isUniversalForScope(a, isGlobal));
const nonUniversalAgents = allAgents.filter((a) => !isUniversalForScope(a, isGlobal));
// Exclude agents with showInUniversalList === false from the locked section.
const universalAgents = allAgents
.filter((a) => isUniversalForScope(a, isGlobal) && a.showInUniversalList !== false)
.sort((a, b) => a.name.localeCompare(b.name));
const nonUniversalAgents = allAgents
.filter((a) => !isUniversalForScope(a, isGlobal))
.sort((a, b) => a.name.localeCompare(b.name));
const canonicalLabel = isGlobal ? "Universal (~/.agents/skills)" : "Universal (.agents/skills)";
const lockedSection = {
@ -214,7 +219,7 @@ async function installFromRegistry(slug: string, opts: Record<string, string | s
ns = uniqueResults[0].namespace;
actualSlug = uniqueResults[0].name;
} else {
spinner.stop();
spinner.succeed(`Found ${actualSlug}`);
const selected = await runInteractiveSearch(client, actualSlug);
if (!selected) {
console.log("Cancelled.");
@ -265,7 +270,7 @@ async function installFromRegistry(slug: string, opts: Record<string, string | s
}
} else {
// Interactive: show version selection
spinner.stop();
spinner.succeed(`Found ${ns}/${actualSlug}`);
const picked = await p.select({
message: "Select version",
@ -492,7 +497,7 @@ async function installFromRegistry(slug: string, opts: Record<string, string | s
}
}
spinner.stop("Installation complete");
spinner.succeed("Installation complete");
console.log("");
const successful = results.filter((r) => r.success);
@ -756,7 +761,7 @@ async function installFromGit(skillName: string, source: string, sourceType: Sou
}
}
spinner.stop("Installation complete");
spinner.succeed("Installation complete");
console.log("");
const successful = results.filter((r) => r.success);