github/ui-ux-pro-max-skill
1
0
Fork 0
mirror of https://github.com/nextlevelbuilder/ui-ux-pro-max-skill.git synced 2026-04-25 11:18:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(cli): check dir existence before reporting removal in uninstall

Browse Source 
Use stat() to verify directory exists before rm + push to removed[].
Prevents misleading "Skill files removed!" when dir never existed.
This commit is contained in:
Goon 2026-03-24 17:49:02 +07:00
parent 035df47a56
commit 0d58617a42
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
cli/src/commands/uninstall.ts
View File

@ -1,4 +1,4 @@
import { rm } from 'node:fs/promises';
import { rm, stat } from 'node:fs/promises';
import { join } from 'node:path';
import { homedir } from 'node:os';
import chalk from 'chalk';
@ -24,10 +24,11 @@ async function removeSkillDir(baseDir: string, aiType: Exclude<AIType, 'all'>):
for (const folder of folders) {
const skillDir = join(baseDir, folder, 'skills', 'ui-ux-pro-max');
try {
await stat(skillDir);
await rm(skillDir, { recursive: true, force: true });
removed.push(`${folder}/skills/ui-ux-pro-max`);
} catch (err: unknown) {
// Re-throw permission errors; force:true already handles ENOENT
// Skip non-existent dirs; re-throw permission or other errors
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') throw err;
}
}