Relocate SKILL.md rationale into docs/RATIONALE.md#13
Merged
Conversation
Move three rationale-only passages out of the SKILL.md body into a new docs/RATIONALE.md, leaving every operational rule in place, to reduce the per-session read cost (SKILL.md loads on every session in Claude Code and Codex). Relocated (verbatim, with one-line pointers left at each excision): - two-channel v1-regression history (Sonnet 4.6 benchmark story) — the load-bearing rule "Do not narrate the contract in prose" stays in SKILL.md - helper "universal floor" design philosophy - the "What this skill is not" positioning section SKILL.md: 295 -> 289 lines, 23642 -> 22441 chars (-5.1%, ~-300 est tokens). KEEP-region diff is empty; all moved text present verbatim in RATIONALE.md; every pointer anchor resolves. Tests: 35 pytest + 8 shell assertions pass. Co-Authored-By: Claude Opus 4.8 (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
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
Relocates rationale-only prose out of the SKILL.md body into a new
docs/RATIONALE.md, leaving every operational rule in place. SKILL.md loads on every session (Claude Code + Codex), so the win is a leaner per-session read. This is a discretionary optimization, not a bloat fix — at 295 lines the body was already under the ~500-line progressive-disclosure ideal.Passages moved (verbatim →
docs/RATIONALE.md)#two-channel-history#universal-floor#what-this-skill-is-notThe load-bearing rule "Do not narrate the contract in prose" stays in SKILL.md — only the story that justified it moved. Each excision left exactly one pointer line.
Size delta (SKILL.md)
Pre/post figures saved under
benchmarks/.Invariants satisfied
cfab952present verbatim — KEEP-region diff is empty.docs/RATIONALE.md.SKILL.md(modified),docs/RATIONALE.md, and the twobenchmarks/files changed.Verification
python3 -m pytest tests/→ 35 passedsh tests/test_contract_session_init.sh→ 8/8 oklimnblock, declares lists beforeadd✓ (minor prose narration leak observed; rule itself unchanged — reviewed and accepted at the manual gate)cite, disclosed unverified in prose, cited only the true figure (58) ✓🤖 Generated with Claude Code