Clarify agent setup summary

This commit is contained in:
Luca Martial 2026-05-11 22:05:59 -07:00
parent be90ad27f4
commit 626284c78d
2 changed files with 19 additions and 11 deletions

View file

@ -239,11 +239,11 @@ describe('setup agents', () => {
const output = io.stdout();
expect(output).toContain('Agent integration complete');
expect(output).toContain('Claude Code');
expect(output).toContain('+ Skill installed');
expect(output).toContain('+ Skill installed — teaches your agent which KTX commands to run');
expect(output).toContain('.claude/skills/ktx/SKILL.md');
expect(output).toContain('+ Rule installed');
expect(output).toContain('+ Rule installed — tells your agent when to use KTX');
expect(output).toContain('.claude/rules/ktx.md');
expect(output).toContain('+ MCP config added');
expect(output).toContain('+ MCP config added — lets your agent talk to KTX over MCP');
expect(output).toContain('.mcp.json');
});
@ -258,9 +258,9 @@ describe('setup agents', () => {
);
expect(summary).toContain('Cursor');
expect(summary).toContain('+ Rule installed');
expect(summary).toContain('+ Rule installed — tells your agent when to use KTX');
expect(summary).toContain('.cursor/rules/ktx.mdc');
expect(summary).toContain('+ MCP config added');
expect(summary).toContain('+ MCP config added — lets your agent talk to KTX over MCP');
expect(summary).toContain('.cursor/mcp.json');
expect(summary).not.toContain(tempDir);
});
@ -280,9 +280,9 @@ describe('setup agents', () => {
);
expect(summary).toContain('Claude Code');
expect(summary).toContain('+ Skill installed');
expect(summary).toContain('+ Rule installed');
expect(summary).toContain('+ Skill installed — teaches your agent which KTX commands to run');
expect(summary).toContain('+ Rule installed — tells your agent when to use KTX');
expect(summary).toContain('Codex');
expect(summary).toContain('+ MCP config added');
expect(summary).toContain('+ MCP config added — lets your agent talk to KTX over MCP');
});
});

View file

@ -348,6 +348,12 @@ export function formatInstallSummary(
idx += planned.length;
}
const fileHints: Record<string, string> = {
skill: 'teaches your agent which KTX commands to run',
rule: 'tells your agent when to use KTX',
mcp: 'lets your agent talk to KTX over MCP',
};
const lines: string[] = [];
for (const install of installs) {
const targetEntries = entriesByTarget.get(install.target) ?? [];
@ -356,11 +362,13 @@ export function formatInstallSummary(
const displayPath =
install.scope === 'global' ? entry.path : relative(projectDir, entry.path);
if (entry.kind === 'file') {
const label = entry.role === 'rule' ? 'Rule installed' : fileEntryLabels[install.target];
lines.push(` + ${label}`);
const isRule = entry.role === 'rule' || fileEntryLabels[install.target] === 'Rule installed';
const label = isRule ? 'Rule installed' : fileEntryLabels[install.target];
const hint = fileHints[isRule ? 'rule' : (entry.role ?? 'skill')] ?? '';
lines.push(` + ${label}${hint}`);
lines.push(` ${displayPath}`);
} else {
lines.push(` + MCP config added`);
lines.push(` + MCP config added${fileHints.mcp}`);
lines.push(` ${displayPath}`);
}
}