Remove caveman skill from style harness#2
Merged
Conversation
Agents kept loading a persona-style response skill even after the harness moved to concise normal prose, and existing installs could keep stale managed copies after reinstall. Remove the skill, catalog route, and routing fixture; document the decision in ADR-0013; and delete obsolete managed per-file outputs during normal install so old native skill files do not linger. AI assistance used; reviewed, tested, and verified with temp-root install proof.
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.
Why
Agents could still discover or keep loading a persona-style
cavemanskill even after the harness moved to concise normal prose, and existing installs could retain stale managed native skill files after reinstall.What
cavemansource skill, catalog row, and routing fixture reference..ai-ackfor the AI assistance policy.How
Normal install now computes the managed files it still writes and deletes any other managed files under per-file output directories. Merged AGENTS-style outputs keep their existing replace/prune behavior.
Test plan
.venv/bin/python -m scripts.lint_source.venv/bin/python -m scripts.build --dry-run.venv/bin/python -m pytest tests/ -q.venv/bin/python -m ruff check scripts testsgit diff --checkcavemanfiles deleted stale managed files for Claude, Codex, Cursor, OpenCode, and Vibe.cavemanskill/catalog route matches.Risk + rollback
Risk is limited to managed per-file install cleanup. User-authored files without the harness marker are ignored. Rollback is restoring the removed skill and ADR status, or reverting this PR.
AI Assistance