chore(skills): reduce token cost of skill descriptions and bodies - #227
Merged
stijnvanhulle merged 1 commit intoSep 18, 2026
Merged
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
Contributor
⏭️ Comment Ratio: SkippedOnly 0 lines added, below the minimum of 50 for this check to apply.
Comment ratio 0% · Limit 15% 2 files analyzed
The ratio is the share of comment lines among all lines added. Lines are counted per changed file before and after the change; positive deltas are summed. Comparing |
Contributor
|
Size Change: 0 B Total Size: 273 B ℹ️ View Unchanged
|
commit: |
stijnvanhulle
force-pushed
the
chore/reduce-skill-token-cost
branch
from
September 18, 2026 09:59
9334201 to
b43e12f
Compare
stijnvanhulle
changed the base branch from
main
to
claude/eloquent-maxwell-1eg8x2
September 18, 2026 09:59
Split the jsdoc skill's example gallery into references/examples.md so the default load only carries the tag tables and guidelines, shortened its frontmatter description (always in context in every skill listing), and tightened repetitive prose in the pr skill's steps without dropping any instruction. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN
stijnvanhulle
force-pushed
the
chore/reduce-skill-token-cost
branch
from
September 18, 2026 10:05
b43e12f to
ca4a97b
Compare
stijnvanhulle
merged commit Sep 18, 2026
44eda88
into
claude/eloquent-maxwell-1eg8x2
10 checks passed
stijnvanhulle
added a commit
that referenced
this pull request
Sep 18, 2026
…eviewer (#226) * feat(claude,cursor): merge deslop and ponytail into one trim skill Runs the reuse-first ladder and the AI style-tell checklist in one pass, reports every finding, and applies only what the user confirms through the native picker (or a lettered list where none exists), matching how ponytail already asked before editing. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN * fix(claude,cursor): rename the merged slop/over-engineering skill to deslop Keep the existing deslop name instead of introducing trim, since it already covers the code-slop half of the merge and needs no new name to learn. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN * chore: ignore local agent worktree scratch checkouts .claude/worktrees/ holds a background agent's isolated git worktree, a local scratch checkout that never belongs in version control. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN * feat(claude,cursor): run humanizer as deslop's third pass over prose Fold prose review into deslop instead of pointing to humanizer as a separate step, so one /deslop run covers over-engineering, code style tells, and AI writing tells in any changed README, doc, or comment. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN * feat(claude,cursor): fold deslop, jsdoc, humanizer, and documentation into code-reviewer Have code-reviewer check for over-engineering, code and prose AI tells, and JSDoc conventions in one pass, ordered substance before style before prose, then ask which findings to hand off for fixing. It stays read-only: it reports by these skills' criteria rather than running their apply steps. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN * feat(claude,cursor): always reach for AskUserQuestion when info is needed Broaden the user-questions rule past fixed-choice forks: any time a task is blocked on information only the user can supply, use the native picker with best-guess options and Other, instead of a plain question. Secrets and credentials are the one exception and stay plain. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN * chore(skills): reduce token cost of skill descriptions and bodies (#227) Split the jsdoc skill's example gallery into references/examples.md so the default load only carries the tag tables and guidelines, shortened its frontmatter description (always in context in every skill listing), and tightened repetitive prose in the pr skill's steps without dropping any instruction. Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN Co-authored-by: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <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
2 participants
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.
🎯 Changes
Stacked on #226 — merge that one first, then this one shows only the diff below.
Cuts the token cost of the two heaviest skills without dropping any instruction.
jsdoc's@examplegallery and documentation patterns move to.agents/skills/jsdoc/references/examples.md,so the default load carries only the tag tables and guidelines (189 → 71 lines), and its
frontmatter description is shorter since it stays in every skill listing whether or not the
skill loads.
pr's later steps lose repeated phrasing. Bumps.codex-plugin/plugin.jsonandgemini-extension.json(on top of #226's bump) since a skill description changed, andregenerates
GEMINI.md.🧪 How to test
cat .agents/skills/jsdoc/SKILL.mdand.agents/skills/jsdoc/references/examples.md, confirm every original example and pattern is still present in one file or the other.node --experimental-strip-types scripts/agentFiles.tsfrom a clean checkout.agent files in sync (3 command sets)with no stale warning.✅ Checklist
pnpm format && pnpm lint:fix && pnpm typecheck && pnpm test.🚀 Release Impact
🤖 Generated with Claude Code
https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN