Skip to content

feat(skills): support global skill removal - #142

Merged
codeaholicguy merged 1 commit into
mainfrom
feature-global-skill-remove
Aug 6, 2026
Merged

feat(skills): support global skill removal#142
codeaholicguy merged 1 commit into
mainfrom
feature-global-skill-remove

Conversation

@codeaholicguy

@codeaholicguy codeaholicguy commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • add ai-devkit skill remove SKILL_NAME --global
  • add --env ENVIRONMENT... filtering and reject it without --global
  • preserve project-local removal behavior when --global is absent
  • guard removal to direct children of configured home-relative global skill roots
  • preserve registry cache and safely handle symlinks, missing entries, duplicate paths, and partial failures
  • update CLI help and skills documentation

Validation

  • focused command/manager tests: 74 passed
  • full CLI package tests: 909 passed
  • repository commit hooks: all 6 lint and all 6 test targets passed
  • workspace build/type declarations: all 6 build targets passed
  • post-rebase CLI build: passed
  • isolated temporary-home built CLI test: symlink and copy removed; symlink target and registry cache preserved; missing skill succeeds; invalid --env usage fails
  • strict TDD reverse-fix proof: selected-global-environment test failed with dispatch disabled and passed after restoration

Risks

  • --global without --env intentionally targets every environment with a configured global skill root.
  • partial filesystem failures can leave some selected environments removed and others intact; the command continues and exits nonzero with a summary.

@codeaholicguy
codeaholicguy force-pushed the feature-global-skill-remove branch from 32b0707 to d39800e Compare August 6, 2026 08:59
@codeaholicguy
codeaholicguy merged commit f231c06 into main Aug 6, 2026
7 checks passed
@codeaholicguy
codeaholicguy deleted the feature-global-skill-remove branch August 6, 2026 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant