Skip to content

Define overlapping skill command policy handling#246

Merged
shiny-code-bot merged 5 commits into
mainfrom
feat-skill-policy-overlap
May 30, 2026
Merged

Define overlapping skill command policy handling#246
shiny-code-bot merged 5 commits into
mainfrom
feat-skill-policy-overlap

Conversation

@shiny-code-bot
Copy link
Copy Markdown
Collaborator

Summary

  • collect all matching skill command policies and choose a primary match by matcher specificity, matcher width, and stable load/declaration order
  • include lower-priority matching policies in guard guidance while de-duplicating repeated preferred actions
  • document command policy overlap behavior and canonical ownership guidance for skill authors

Validation

  • cargo test -p code-core skills::command_policy -- --nocapture
  • cargo test -p code-core --test skill_command_policy -- --nocapture
  • exec harness overlap smoke with the freshly built code binary, real repo-local overlapping skill policies, and a fake Responses endpoint: verified the raw command was blocked, overlap guidance was emitted, no exec_command_begin appeared, and the run became quiescent
  • ./build-fast.sh
  • after merging current origin/main, reran ./build-fast.sh

Closes #244

@shiny-code-bot shiny-code-bot merged commit d13cda8 into main May 30, 2026
1 check passed
@shiny-code-bot shiny-code-bot deleted the feat-skill-policy-overlap branch May 30, 2026 23:20
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.

Define overlap handling for skill command policies

2 participants