docs(readme): list every skill; fix the breakdown; sweep stale references - #17
Merged
Conversation
…nces The README's skill tables listed 27 of 29 skills. The two missing were capturing-learnings and -- more awkwardly -- data-science-project, the front door, the skill this whole rework was about. It was named twice in prose and linked from no table, so a reader browsing the catalogue could not find it. The prose breakdown also summed to 28: "15 pipeline + 11 domain + 1 methodology + 1 entry" against 29 on disk. Domain is 12. Two tests now guard both independently, because they drift independently: one asserts every skill directory appears in a table and no table row points at a skill that is gone, the other parses the breakdown sentence and checks it sums to the real count. Swept the rest of the repo for stale references. Fixed: eval 6's gate_under_test still named validation-strategy, a skill merged into ds-validate. Removed an empty references/ directory left from an earlier phase. Deliberately left alone: the validation-strategy mentions in benchmarks/house-prices/ are the narrative of a completed run, and rewriting a finished run's record to match a later refactor would falsify it. The .gitignore entries for venv/ and .last-ds-mile/*.csv match nothing today because they are prospective -- they cover files a user's project will produce. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
StamKavid
added a commit
that referenced
this pull request
Aug 8, 2026
docs(readme): list every skill; fix the breakdown; sweep stale references
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.
The README skill tables listed 27 of 29 skills. The two missing were
capturing-learningsand — more awkwardly —data-science-project, the front door, the skill this entire rework was about. Named twice in prose, linked from no table.The prose breakdown also summed to 28: "15 pipeline + 11 domain + 1 methodology + 1 entry" against 29 on disk. Domain is 12.
Two tests now guard these independently, because they drift independently:
Staleness sweep
Fixed: eval 6's
gate_under_teststill namedvalidation-strategy, merged intods-validateearlier. Removed an emptyreferences/directory left from an earlier phase.Left alone, deliberately: the
validation-strategymentions underbenchmarks/house-prices/are the narrative of a completed run — rewriting a finished run's record to match a later refactor would falsify it. The.gitignoreentries forvenv/and.last-ds-mile/*.csvmatch nothing today because they are prospective: they cover files a user's project will produce.Verified clean
plugin.json0.9.0 ·package.json0.9.0 · marketplace refv0.9.0🤖 Generated with Claude Code