Mount skill search and load tools on every worker - #982
Merged
TheGreatAxios merged 7 commits intoSep 13, 2026
Merged
Conversation
Collaborator
Author
|
Audit: Approve. Problem/solution fit: baked SKILL.md dumps per spawn removed in favor of scoped skill_search/use_skill mount with mount-time deny for grok/kimi. File-for-later (do not implement here): (1) optional-compliance drift, (2) unset-allowlist widening log. |
Baked skill bodies cost roughly 38KB per worker prompt; workers now search and load only the skills the brief names.
Search is the noisy surface on small models; use_skill stays available so brief-named skills still load.
TheGreatAxios
force-pushed
the
cl-7668-mount-skill_search-and-use_skill-on-every-worker-stop-baking
branch
from
September 13, 2026 19:32
94cb600 to
55a893b
Compare
Warden's test pinned the pre-skill review surface; skill tools now ride REVIEW_TOOLS via READ_TOOLS, scoped at mount to optionalSkills. Update the expectation and the prompt line to match critic.
TheGreatAxios
added a commit
that referenced
this pull request
Sep 14, 2026
* Mount skill_search and use_skill on every worker tool surface * Mount scoped skill_search and use_skill on every worker * Spread skill tools across build, docs, review, and orchestrator surfaces * Load worker skills on demand instead of baking bodies Baked skill bodies cost roughly 38KB per worker prompt; workers now search and load only the skills the brief names. * Deny skill_search on grok and kimi leaf workers Search is the noisy surface on small models; use_skill stays available so brief-named skills still load. * Enforce the skill search deny at the worker mount * fix: scope warden to the on-demand skill surface Warden's test pinned the pre-skill review surface; skill tools now ride REVIEW_TOOLS via READ_TOOLS, scoped at mount to optionalSkills. Update the expectation and the prompt line to match critic.
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.
Summary
Workers mount skill_search and use_skill scoped to their optionalSkills and load skill bodies on demand instead of baked SKILL.md dumps. Grok and Kimi leaf workers deny skill_search through the family policy while use_skill stays available; brief-named skills load straight through use_skill.
Verification
bun run check passes (7132 tests, 0 fail). Prompt sizes fall 64-96 percent per worker. Live tier-med eval cells blocked on missing model credentials in this environment.
Fixes CL-7668