scriptoriumadmin/GitNexus
1
0
Fork 0
mirror of https://github.com/abhigyanpatwari/GitNexus.git synced 2026-08-28 05:25:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

fix(setup): group skill-rename leftover notices into one line per rename

Browse source 
A multi-tool setup printed the "skill X was renamed to Y" notice once per
agent target (4x for Claude Code/Cursor/OpenCode/Codex). Leftover legacy
dirs are now collected during install and flushed as a single grouped
notice per rename - all target paths on one line - just before the
summary block.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
abhigyantrumio 2026-07-26 06:30:43 +05:30
parent 7126e67d1a
commit 62219e8b53
1 changed files with 26 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

31
gitnexus/src/cli/setup.ts
View file

@ -1031,6 +1031,23 @@ export const RENAMED_SKILL_DIRS: Readonly<Record<string, readonly string[]>> = {
*/
export const LEGACY_SKILL_DIR_NAMES: readonly string[] = Object.values(RENAMED_SKILL_DIRS).flat();
/** Legacy skill dirs found during this run, keyed `oldName|newName` — flushed
* as one grouped notice per rename by {@link flushSkillRenameNotices}. */
const pendingSkillRenameNotices = new Map<string, string[]>();
/** Print the collected rename leftovers (one line per rename, all target
* paths grouped) and reset the collector. */
export function flushSkillRenameNotices(): void {
for (const [key, paths] of pendingSkillRenameNotices) {
const [oldName, skillName] = key.split('|');
console.log(
` Note: skill "${oldName}" was renamed to "${skillName}". Left in place ` +
`(delete manually if you have not customized them): ${paths.join(', ')}`,
);
}
pendingSkillRenameNotices.clear();
}
/**
* Install GitNexus skills to a target directory.
* Each skill is installed as {targetDir}/gitnexus-{skillName}/SKILL.md
@ -1102,14 +1119,16 @@ async function installSkillsTo(targetDir: string): Promise<string[]> {
// A directory superseded by a shipped rename is warned about, never
// deleted: the installer cannot prove it owns the contents (users
// customize installed skills or hand-write their own under these
// names), so an upgrade must not destroy data.
// names), so an upgrade must not destroy data. Collected instead of
// printed here so a multi-tool setup emits one grouped notice per
// rename, not one line per target directory.
for (const oldName of RENAMED_SKILL_DIRS[skillName] ?? []) {
const legacyDir = path.join(targetDir, oldName);
if (await dirExists(legacyDir)) {
console.log(
`[gitnexus] skill "${oldName}" was renamed to "${skillName}"; ` +
`left ${legacyDir} in place — delete it manually if you have not customized it.`,
);
const key = `${oldName}|${skillName}`;
const paths = pendingSkillRenameNotices.get(key) ?? [];
paths.push(legacyDir);
pendingSkillRenameNotices.set(key, paths);
}
}
installed.push(skillName);
@ -1267,6 +1286,8 @@ export const setupCommand = async (options?: { codingAgent?: string[] | string }
}
}
flushSkillRenameNotices();
console.log('');
console.log(' Summary:');
console.log(