chore(skills): trim sdlc/update SKILL.md below 5K-token threshold (v1.48.0) - #273
Merged
Conversation
….48.0) PR #272's audit-session-load.sh flagged 2 of 4 SKILL.md files: - skills/sdlc/SKILL.md: 12,427 → 4,995 tokens (-60%) - skills/update/SKILL.md: 8,555 → 4,044 tokens (-53%) Acted on the tool's own findings — closes the Prove-It loop. Aggressive prose compression with no operational content lost: every TodoWrite item, every hard rule (NEVER AUTO-MERGE, ALL TESTS MUST PASS, NEVER MODIFY CLAUDE.md), every Codex command, every test anchor preserved. Added test_wizard_own_skills_below_threshold to test-audit-session-load.sh — RED on old files, GREEN on trimmed. 0 trim candidates from the audit now. 45 unit suites + 4 e2e quick-tests green. Codex round 1 CERTIFIED 10/10 (no findings; verified RED/GREEN proof, full CONTRIBUTING.md test suite, end-to-end semantic comparison vs git show HEAD).
BaseInfinity
added a commit
that referenced
this pull request
May 5, 2026
….48.0) (#273) PR #272's audit-session-load.sh flagged 2 of 4 SKILL.md files: - skills/sdlc/SKILL.md: 12,427 → 4,995 tokens (-60%) - skills/update/SKILL.md: 8,555 → 4,044 tokens (-53%) Acted on the tool's own findings — closes the Prove-It loop. Aggressive prose compression with no operational content lost: every TodoWrite item, every hard rule (NEVER AUTO-MERGE, ALL TESTS MUST PASS, NEVER MODIFY CLAUDE.md), every Codex command, every test anchor preserved. Added test_wizard_own_skills_below_threshold to test-audit-session-load.sh — RED on old files, GREEN on trimmed. 0 trim candidates from the audit now. 45 unit suites + 4 e2e quick-tests green. Codex round 1 CERTIFIED 10/10 (no findings; verified RED/GREEN proof, full CONTRIBUTING.md test suite, end-to-end semantic comparison vs git show HEAD).
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
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
PR #272's
scripts/audit-session-load.shflagged 2 of 4 SKILL.md files as TRIM candidates the same day we shipped the tool:skills/sdlc/SKILL.md: 12,427 tokensskills/update/SKILL.md: 8,555 tokensActed on the tool's findings — closes the Prove-It loop. A tool that surfaces real issues whose owner ignores them is just a louder lint warning.
Result:
skills/sdlc/SKILL.md: 12,427 → 4,995 tokens (-60%, 49,709 → 19,983 chars)skills/update/SKILL.md: 8,555 → 4,044 tokens (-53%, 34,220 → 16,179 chars)What was trimmed
No operational content lost. Aggressive prose compression — removed ASCII-art decoration boxes (kept the bold sentences they contained), tightened cross-model review section while preserving every Codex command/sandbox note/dialogue-loop template/convergence rule. TodoWrite checklist intact (all 30 items, with
activeFormremoved since the spinner falls back tosubject). Step 1.5's 30-line Nodecmp()helper replaced with a precise prose description of the algorithm. Step 3's frozen 20-line changelog example shortened to a placeholder pointing at the real fetched CHANGELOG.Every test anchor traced and preserved: all grep'd phrases across
tests/test-{doc-consistency,self-update,update-skill-step-7-7,update-skill-cli-version,memory-audit-protocol,docs-usability,cli,prove-it,hooks}.shstill match.New quality test
test_wizard_own_skills_below_thresholdintests/test-audit-session-load.sh— runs the audit on the wizard repo itself and fails if any SKILL.md flags TRIM. RED before this PR (both files flagged), GREEN after. Mutation-verifiable: bumping either file ~200 tokens flips the test red.Codex review
Round 1 CERTIFIED 10/10. No findings. Codex did its own RED/GREEN proof (stashed only the trimmed skill files to keep the new test active), verified every checklist item with shell evidence, ran the full CONTRIBUTING.md test suite, and read both files end-to-end against
git show HEAD:...for semantic completeness. Full review at.reviews/latest-review.md.Test plan
tests/test-audit-session-load.sh— 9/9 (was 8/8)tests/test-doc-consistency.sh— 30/30tests/test-docs-usability.sh— 29/29tests/test-self-update.sh— 153/153tests/test-cli.sh— 78/78tests/test-hooks.sh— 154/154tests/test-update-skill-step-7-7.sh— 8/8tests/test-update-skill-cli-version.sh— 8/8tests/test-memory-audit-protocol.sh— 12/12