Fix ~2% skill usage: selective install (--only/--exclude) + skill-budget warning (v3.7.0) - #2
Draft
GopalGB wants to merge 1 commit into
Draft
Fix ~2% skill usage: selective install (--only/--exclude) + skill-budget warning (v3.7.0)#2GopalGB wants to merge 1 commit into
GopalGB wants to merge 1 commit into
Conversation
…kill-usage) (v3.7.0) Codex shows its skill list within only ~2% of the context window (~8000 chars). With all 383 skills installed it shortens descriptions and OMITS most from the auto-trigger list — the "only ~2% of skills get used" symptom. (Ref: OpenAI Codex Skills docs.) - install.sh: --only=<list> (allowlist) and --exclude=<list> (denylist), taking comma-separated pack prefixes (e.g. xls) or exact skill names (e.g. production-audit), so you install a focused subset that fits the budget. - Post-install warning when >25 skills are installed, with the exact --only/ --exclude/prune commands. Default (install all) unchanged. - README "Skill budget" callout + OFFICE-SETUP office-subset guidance. Verified: --only/--exclude filter exactly (prefix + exact-name match, no over-match); --help clean; smoke 19/19 green. Follow-up to merged PR #1. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QTbNNSTfHHsQXrR1FgyK6t
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Problem — "only ~2% of skills get used"
Per OpenAI's Codex Skills docs, Codex loads the initial skill list within only ~2% of the context window (~8,000 characters). With all 383 skills installed (descriptions total ~100 KB / ~25k tokens — ~12× over budget), Codex shortens descriptions and omits most skills from the auto-trigger list. They still run when invoked by path, but won't fire on their own — the "2% skill usage" symptom. Shipping 383 skills fundamentally exceeds this cap; the fix is to install a focused subset.
Fix
install.shselective install (non-breaking — default still installs all):--only=<list>— install ONLY the listed packs/skills--exclude=<list>— install everything EXCEPT the listed packs/skillsxls) or exact skill names (e.g.production-audit)--only/--exclude/prune commandsDocs: README gains a "⚠ Skill budget" callout;
docs/OFFICE-SETUP.mdgains office-subset guidance. Both point to/prompts:absorbfor per-project scoping into./.agents/skills/.Verification
--only=ponytail,production-audit→ installs exactly those 2 (real install to temp dir confirmed).--only=xls,ponytail→ 27;--exclude=ml,ai,sde,agentic,craft→ 270 (verified: no prefix over-match;mlintentionally also matches the coreml-engineer).bash install.sh --helpclean;bash -n install.shOK; smoke 19/19 green.Notes
main); this is a separate, new change.🤖 Generated with Claude Code
https://claude.ai/code/session_01QTbNNSTfHHsQXrR1FgyK6t
Generated by Claude Code