Skip to content

feat(skills): add model-invocable SkillTool#28

Open
yqmai wants to merge 2 commits into
e10nMa2k:mainfrom
yqmai:feat-skillTool
Open

feat(skills): add model-invocable SkillTool#28
yqmai wants to merge 2 commits into
e10nMa2k:mainfrom
yqmai:feat-skillTool

Conversation

@yqmai

@yqmai yqmai commented Jun 15, 2026

Copy link
Copy Markdown

Summary

Adds model-invocable skills through a new SkillTool execution path.

  • Adds explicit skill metadata for model invocation eligibility
  • Runs model-invocable skills in isolated child Engines through SkillRunner
  • Adds SkillTool with frozen authorization schema and structured results
  • Preserves manual slash skill behavior
  • Adds worker skill authorization for Coordinator workers
  • Keeps Coordinator, Explore, and Plan Mode boundaries enforced

Safety / Permissions

  • Skills are not model-invocable by default
  • A skill must explicitly set model-invocable: true and declare allowed-tools
  • SkillTool does not bypass existing tool permissions or sandbox rules
  • /commit and /simplify remain manual-only
  • Plan Mode and Coordinator main agents do not expose SkillTool
  • Worker skill grants are fixed at worker creation

Tests

  • pytest tests -q
  • Diff whitespace check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant