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

feat(cli): merge install results by path for cleaner output

Browse source 
Group agents that share the same install path into a single line.
When more than 5 agents share a path, show first 5 names + count.
Sort agent names alphabetically within each group.
Extract buildInstallResultLines utility function to DRY up
duplicate output logic in installFromRegistry and installFromGit.
This commit is contained in:
chenbaowang 2026-04-20 16:55:54 +08:00
parent aaf80dbfd8
commit 86b8de1fbc
1 changed files with 60 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -52,6 +52,64 @@ function getInstallSpinner(sourceType: SourceType, arg: string): string {
return `Resolving ${arg}`;
}
interface InstallResult {
skill: string;
agent: string;
success: boolean;
path: string;
error?: string;
}
/**
* Build grouped install result lines: agents sharing the same install path
* are merged into one line for cleaner output.
*
* Example:
* ✓ fork-workflow
* → Amp, Cline, Codex, Cursor, Deep Agents +7: .agents/skills/fork-workflow
* → Claude Code: .claude/skills/fork-workflow
*/
function buildInstallResultLines(
selectedSkills: { name: string }[],
results: InstallResult[],
): string[] {
const MAX_NAMES = 5;
const resultLines: string[] = [];
for (const skill of selectedSkills) {
const skillResults = results.filter((r) => r.skill === skill.name && r.success);
if (skillResults.length === 0) continue;
resultLines.push(`${pc.green("✓")} ${skill.name}`);
// Group agents by install path
const pathGroups = new Map<string, string[]>();
for (const r of skillResults) {
const agents = pathGroups.get(r.path) || [];
agents.push(r.agent);
pathGroups.set(r.path, agents);
}
// Sort groups: put the group with the most agents first
const sortedGroups = [...pathGroups.entries()].sort((a, b) => b[1].length - a[1].length);
for (const [path, agents] of sortedGroups) {
const sorted = agents.sort((a, b) => a.localeCompare(b));
let label: string;
if (sorted.length <= MAX_NAMES) {
label = sorted.join(", ");
} else {
const shown = sorted.slice(0, MAX_NAMES).join(", ");
const extra = sorted.length - MAX_NAMES;
label = `${shown} ${pc.dim(`+${extra}`)}`;
}
resultLines.push(` ${pc.dim("→")} ${label}: ${pc.dim(path)}`);
}
}
return resultLines;
}
async function selectAgentsInteractive(isGlobal: boolean): Promise<string[] | null> {
const allAgents = getAllAgents();
@ -503,16 +561,7 @@ async function installFromRegistry(slug: string, opts: Record<string, string | s
const successful = results.filter((r) => r.success);
if (successful.length > 0) {
const resultLines: string[] = [];
for (const skill of selectedSkills) {
const skillResults = results.filter((r) => r.skill === skill.name && r.success);
if (skillResults.length > 0) {
resultLines.push(`${pc.green("✓")} ${skill.name}`);
for (const r of skillResults) {
resultLines.push(` ${pc.dim("→")} ${r.agent}: ${r.path}`);
}
}
}
const resultLines = buildInstallResultLines(selectedSkills, results);
p.note(resultLines.join("\n"), `Installed ${successful.length} skill(s)`);
}
@ -767,16 +816,7 @@ async function installFromGit(skillName: string, source: string, sourceType: Sou
const successful = results.filter((r) => r.success);
if (successful.length > 0) {
const resultLines: string[] = [];
for (const skill of selectedSkills) {
const skillResults = results.filter((r) => r.skill === skill.name && r.success);
if (skillResults.length > 0) {
resultLines.push(`${pc.green("✓")} ${skill.name}`);
for (const r of skillResults) {
resultLines.push(` ${pc.dim("→")} ${r.agent}: ${r.path}`);
}
}
}
const resultLines = buildInstallResultLines(selectedSkills, results);
p.note(resultLines.join("\n"), `Installed ${successful.length} skill(s)`);
}