refactor: consolidate caveman-compress into skills/, drop renamed mirror#360
Open
JuliusBrussee wants to merge 1 commit into
Open
refactor: consolidate caveman-compress into skills/, drop renamed mirror#360JuliusBrussee wants to merge 1 commit into
JuliusBrussee wants to merge 1 commit into
Conversation
…eliminate compress mirror - Move caveman-compress/ source (SKILL.md, scripts/, README.md, SECURITY.md) to skills/caveman-compress/. - Delete skills/compress/ — the CI-generated rename mirror that caused dual-source confusion. - Move plugins/caveman/skills/compress/ to plugins/caveman/skills/caveman-compress/. Plugin keeps the consolidated name; CI no longer rewrites the frontmatter. - Replace the two sed-heavy CI sync steps with one verbatim cp -r from source to plugin. - Update verify_repo.py, test_compress_safety.py, test_validate_inline.py, GEMINI.md, AGENTS.md, CONTRIBUTING.md, README.md, and the workflow paths to reference the new location. - Use Path.resolve().parents[N] for the benchmark.py repo-root walk now that the directory depth changed. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
caveman-compress/source intoskills/caveman-compress/so the compress sub-skill lives where every other skill does.skills/compress/rename mirror that caused dual-source confusion.plugins/caveman/skills/compress/→plugins/caveman/skills/caveman-compress/so the plugin advertises the consolidated name.cp -rstep now that source and plugin share the samename: caveman-compressfrontmatter.What changed
git mv caveman-compress/* skills/caveman-compress/(history preserved onSKILL.md, scripts, README, SECURITY).Processstep references its ownscripts/dir instead ofcaveman-compress/scripts/. Thecd caveman-compressinvocation is gone..github/workflows/sync-skill.ymltriggers onskills/caveman-compress/...and copies the source dir into the plugin verbatim. No moresedrename or path rewrite.scripts/benchmark.pyupdated to usePath.resolve().parents[N]for the new directory depths (skills tree: 3, plugin tree: 5).tests/verify_repo.py,tests/test_compress_safety.py,tests/test_validate_inline.pyimport fromskills/caveman-compress/scripts/.GEMINI.md,AGENTS.md,CONTRIBUTING.md, and the README link toskills/caveman-compress/(README change is one line —caveman-compress READMEandSECURITY.mdlinks).Test plan
python3 -m unittest test_compress_safety test_validate_inline(15/15 pass)python3 tests/verify_repo.py— all checks pass (frontmatter, manifests, hooks, compress fixtures, compress CLI)python3 -c "import yaml; yaml.safe_load(open('.github/workflows/sync-skill.yml'))"— workflow parsesgrep -rn 'caveman-compress\b\|skills/compress'— no dangling path refs (remaining hits are intentional: brand mentions, slash-command name, English usage of "caveman-compressed", and CLAUDE.md which Unit 8 owns)🤖 Generated with Claude Code