Repository navigation
Split large skill script tables out of INDEX.md into a lazy INDEX-scripts.md - #1955
Merged
Merged
Conversation
…ipts.md agentic-delivery/INDEX.md is what a run reads when SKILL.md routing has no match; 78 script rows were 12.6 KB of its 16,962 bytes. skill_index.py now moves the script table of a skill with more than 12 scripts into INDEX-scripts.md and leaves the references table plus a pointer in INDEX.md (4,533 bytes). deep-code-review (24 scripts) splits the same way; smaller skills are unchanged. The saving applies to reference lookups only. A script lookup reads INDEX.md plus INDEX-scripts.md, 17,367 bytes against the old 16,962 (+405, 2.4%). The pointer tells the reader to open INDEX-scripts.md when a script is needed; the entry index keeps only one-line purpose and args. --check now flags an orphan INDEX-scripts.md. land-release.sh, test-conflict-free-prs.sh and the CI comment treat INDEX-scripts.md as a derived file, and --regen keeps size rows for both split skills' index files. The entry index text is trimmed to stay under its frozen budget (12,352 to 12,344). Tests: 190/190 Co-Authored-By: Claude Sonnet 5.5 <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
A run reads
agentic-delivery/INDEX.mdwhen SKILL.md routing has no match. 78 script rows took up 12.6 KB of the file's 16,962 bytes.skill_index.pynow handles skills with more than 12 scripts differently. It moves their script table into a lazyINDEX-scripts.mdand leaves the references table plus a pointer inINDEX.md.agentic-delivery/INDEX.mdshrinks to 4,533 bytes, a 73% cut for reference lookups.deep-code-review, which has 24 scripts, splits the same way: itsINDEX.mdgoes from 22,660 to 18,650 bytes on current main, with 4,415 bytes in the lazy scripts file.Measurement (offline byte counts)
INDEX.md+INDEX-scripts.md)The pointer says to open
INDEX-scripts.mdonly when a script is needed.Safety
--checknow flags an orphanINDEX-scripts.md.land-release.sh,test-conflict-free-prs.shand the CI comment treatINDEX-scripts.mdas a derived file.--regenkeeps size rows for both split skills.Review
INDEX-scripts.md;d33a09377efe5b111ded98ffde5f502ff3abfaf4.This branch is that commit cherry-picked onto main
061e61883aff2be6411223ff95a197a191a8cce5. The only conflict was insize-budgets.tsv. I kept main's new rows, took the trimmed.claude/skills/INDEX.mdbudget (12,344), and set thedeep-code-review/INDEX.mdrow to 18,650 to match the regenerated file, which now includes the method-split references from #1954.Tests
🤖 Generated with Claude Code