Skip to content

Add safe managed skill uninstall - #12

Merged
dd3ok merged 1 commit into
mainfrom
codex/add-safe-uninstall
Aug 28, 2026
Merged

Add safe managed skill uninstall#12
dd3ok merged 1 commit into
mainfrom
codex/add-safe-uninstall

Conversation

@dd3ok

@dd3ok dd3ok commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • add stash uninstall for verified inactive managed skills
  • use operation-owned tree and record tombstones with fail-closed crash recovery
  • document the contract and regenerate every vendor adapter

Safety

  • reject active deployments, links, non-directories, and hash drift without mutation
  • preserve external catalogs, source skills, host deployments, and host settings
  • support stale-record cleanup when the managed tree is already missing

Platform impact

  • use the existing Node.js rename and removal path on every supported platform
  • CI passed on Ubuntu (Node 20 and 24), macOS (Node 24), and Windows (Node 24)

Verification

  • npm run test:all (80 source tests passed, 1 existing Windows fixture skipped; 8 distribution tests passed)
  • canonical and installed Codex skill validators passed
  • npm run pack:check passed
  • git diff --check passed

@dd3ok
dd3ok merged commit ee083d8 into main Aug 28, 2026
4 checks passed
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