Skip to content

feat: Handle steer tool results with approval requests#17

Open
lan17 wants to merge 3 commits intomainfrom
lev/steer
Open

feat: Handle steer tool results with approval requests#17
lan17 wants to merge 3 commits intomainfrom
lev/steer

Conversation

@lan17
Copy link
Copy Markdown
Contributor

@lan17 lan17 commented Mar 30, 2026

Summary

  • add configurable steerBehavior support with requireApproval as the default
  • map tool-step steer matches to OpenClaw requireApproval responses while keeping deny as a hard block
  • document the new behavior and extend tests for steer approval, steer blocking, precedence, and guidance fallback

Verification

  • npm run lint
  • npm run typecheck
  • npm test
  • npm run coverage

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 30, 2026

Codecov Report

❌ Patch coverage is 91.66667% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/agent-control-plugin.ts 91.66% 1 Missing and 3 partials ⚠️

📢 Thoughts on this report? Let us know!

@lan17 lan17 changed the title Handle steer tool results with approval requests feat: Handle steer tool results with approval requests Mar 30, 2026
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.

2 participants