[GH-ISSUE #2159] [Bug] rea uninstall leaves empty reverse-engineer-anything skill directories behind #709

New issue

Closed

opened 2026-10-15 18:27:36 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-15 18:27:36 +00:00

Owner

Originally created by @kaoru0822-kitauji on GitHub (Oct 12, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/2159

Observed behavior

rea uninstall reports each managed skill as removed, but leaves the emptied skill directory trees behind in every skill destination:

$ rea setup --client claude_code --client hermes … --yes && rea uninstall
skill,removed,"…/.agents/skills/reverse-engineer-anything: removed (REA-managed skill files were removed; other files were preserved.) …"

$ find ~/.agents ~/.claude/skills ~/.gemini/config/skills ~/.hermes/skills
~/.agents/skills/reverse-engineer-anything
~/.agents/skills/reverse-engineer-anything/references
~/.claude/skills/reverse-engineer-anything
~/.claude/skills/reverse-engineer-anything/references
~/.gemini/config/skills/reverse-engineer-anything
~/.gemini/config/skills/reverse-engineer-anything/references
~/.hermes/skills/reverse-engineer-anything
~/.hermes/skills/reverse-engineer-anything/references

All of these directories are empty. Setup created them and they hold no user files.

Reproduction

Isolated HOME on current main 5190843f8: run rea setup --yes with every supported client, then rea uninstall, then list the skill destinations.

Expected behavior

When no user files remain, uninstall also removes the empty references/ and reverse-engineer-anything/ directories that setup created. Directories that still contain user files are preserved, as now.

Cause

a01148553 changed skill removal from a recursive rm of the skill directory to per-file removal of MANAGED_SKILL_FILES, so that user files are preserved. Nothing removes the managed directories once they are empty. Before that change (as in #661), uninstall removed ~/.agents/skills/reverse-engineer-anything itself.

Originally created by @kaoru0822-kitauji on GitHub (Oct 12, 2026). Original GitHub issue: https://github.com/morluto/rea/issues/2159 ### Observed behavior `rea uninstall` reports each managed skill as `removed`, but leaves the emptied skill directory trees behind in every skill destination: ```text $ rea setup --client claude_code --client hermes … --yes && rea uninstall skill,removed,"…/.agents/skills/reverse-engineer-anything: removed (REA-managed skill files were removed; other files were preserved.) …" $ find ~/.agents ~/.claude/skills ~/.gemini/config/skills ~/.hermes/skills ~/.agents/skills/reverse-engineer-anything ~/.agents/skills/reverse-engineer-anything/references ~/.claude/skills/reverse-engineer-anything ~/.claude/skills/reverse-engineer-anything/references ~/.gemini/config/skills/reverse-engineer-anything ~/.gemini/config/skills/reverse-engineer-anything/references ~/.hermes/skills/reverse-engineer-anything ~/.hermes/skills/reverse-engineer-anything/references ``` All of these directories are empty. Setup created them and they hold no user files. ### Reproduction Isolated HOME on current main 5190843f8: run `rea setup --yes` with every supported client, then `rea uninstall`, then list the skill destinations. ### Expected behavior When no user files remain, uninstall also removes the empty `references/` and `reverse-engineer-anything/` directories that setup created. Directories that still contain user files are preserved, as now. ### Cause a01148553 changed skill removal from a recursive `rm` of the skill directory to per-file removal of `MANAGED_SKILL_FILES`, so that user files are preserved. Nothing removes the managed directories once they are empty. Before that change (as in #661), uninstall removed `~/.agents/skills/reverse-engineer-anything` itself.

bsakaguchi

2026-10-15 18:27:36 +00:00