Split pattern examples into PATTERNS.md; trim description - #191
Open
dsc-pliancy wants to merge 1 commit into
Open
Split pattern examples into PATTERNS.md; trim description#191dsc-pliancy wants to merge 1 commit into
dsc-pliancy wants to merge 1 commit into
Conversation
- SKILL.md keeps watch-words and problem statements for all 33 patterns, with a pointer to the matching example, instead of full before/after blocks inline. Drops from 412 to 247 lines; no pattern content, numbering, or behavior changes. - New PATTERNS.md holds the full before/after examples, keeping SKILL.md lean so a normal invocation doesn't load all 33 of them. - Frontmatter description trimmed: drops the Wikipedia-attribution line (already in Reference) and the itemized 10-pattern list, without broadening automatic invocation. - scripts/validate-package.py now also checks PATTERNS.md contains examples 1-33 in order. - Bumped metadata.version, plugin.json version, and README version history to 2.10.0 together per the maintenance contract. - Added .gitignore for .DS_Store and other local cruft. Note: blader#190 (open, unmerged, same base commit 523374d) also claims 2.10.0 for an unrelated change. Happy to rebase and renumber if that one lands first.
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.
What
SKILL.mdkeeps watch-words and problem statements for all 33 patterns, with a pointer to the matching example, instead of full before/after blocks inline. Drops from 412 to 247 lines; no pattern content, numbering, or behavior changes.PATTERNS.mdholds the full before/after examples, keepingSKILL.mdlean so a normal invocation doesn't load all 33 of them.descriptiontrimmed: drops the Wikipedia-attribution line (already in Reference) and the itemized 10-pattern list, without broadening automatic invocation.Why
SKILL.mdis model-invoked, so its full contents load into context on every relevant turn. The 33 patterns' before/after examples made up the bulk of that (roughly 300 of 412 lines) but are only needed for calibration, not detection, the condensed watch-words and problem statement are enough to spot a pattern; the example is useful on demand, not every time. Splitting them out cuts what loads by default without touching what the skill actually does.The description also carried body content that didn't earn its context cost: an itemized sample of 10 of the 33 patterns, and a Wikipedia-attribution line already stated in Reference. Trimmed to state the skill and its trigger once. This doesn't broaden automatic invocation scope, it should fire on the same text it already fires on today.
Maintenance contract
Per
AGENTS.md: bumpedmetadata.versioninSKILL.md,.claude-plugin/plugin.json, and theREADME.mdversion history together to 2.10.0; addedPATTERNS.mdto AGENTS.md's Key Files and to the patterns-sync clause; extendedscripts/validate-package.pyto also checkPATTERNS.mdcontains examples 1-33 in order.python3 scripts/validate-package.pypasses. Also added a.gitignorefor.DS_Storeand other local editor/OS cruft, which wasn't previously ignored.No numbered patterns were added, removed, or renumbered (still 33).
Version note: #190 is open against this same base commit and also claims 2.10.0 for an unrelated change. Happy to rebase and renumber if #190 lands first.