diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-27 13:58:35 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-27 13:58:35 -0500 |
| commit | 931f364864441676eb250cecd074c4876012a1cd (patch) | |
| tree | 3531a0fc7f8e8d8af1bcfab4083952e0876eb35b /frontend-design/SKILL.md | |
| parent | d74d98d03e7321b6c69dae9ef427da66b0f6fbc6 (diff) | |
| download | rulesets-931f364864441676eb250cecd074c4876012a1cd.tar.gz rulesets-931f364864441676eb250cecd074c4876012a1cd.zip | |
refactor(rules): split commits.md into invariants plus a publish skill
commits.md was 12,800 tokens, larger than the next four rules combined, and it loaded in every session in every project whether or not anything was being published. Most of it is procedure that only applies when a commit or PR is in play.
I split it by blast radius rather than by size. What stays always-loaded is what must hold whether or not I'm publishing, and where a violation is permanent and reaches other people: author identity, the no-AI-attribution ban, the generated-document byline rule, and the public-artifact content-scope rules. That core is 1,027 words, about 2,342 tokens.
What moved into the publish skill is everything about how a message gets written, reviewed, approved, and published. The pre-flight reconcile, the code-review gate, the draft and voice and approval gate, conventional-commit format, Voice and Focus, PR description structure, the three review shapes, merge strategy, and the pre-commit checklist.
The failure modes are why the line falls there. If the skill doesn't load I don't know the publish flow and have to be told, which is visible and recoverable. I don't silently commit with AI attribution, because that guard never moved. Only the recoverable half is on the skill-triggering bet.
I verified the split by using it. The skill registered mid-session and I invoked it to publish this commit, and it loaded with the flow intact.
Content is conserved: 5,561 words in, 5,898 across both files, the difference being the frontmatter and the pointer I added to the core. I repointed five cross-references in voice, review-code, inbox.org, and no-approvals.org that named sections which moved.
One thing I left alone. The skill is a single 4,871-word file, and both posts argue a long skill should split internally rather than sit as one blob. It loads on demand now, which is the win worth taking today. Splitting it further is its own change.
Diffstat (limited to 'frontend-design/SKILL.md')
0 files changed, 0 insertions, 0 deletions
