Skip to content

feat: sequential per-tool approval instead of batched all-or-nothing#937

Merged
njbrake merged 2 commits intomainfrom
feat/777-skip-only-plan-approval
Apr 13, 2026
Merged

feat: sequential per-tool approval instead of batched all-or-nothing#937
njbrake merged 2 commits intomainfrom
feat/777-skip-only-plan-approval

Conversation

@njbrake
Copy link
Copy Markdown
Collaborator

@njbrake njbrake commented Apr 13, 2026

Description

Replaces the batched approval prompt (approve all or deny all) with sequential per-tool prompts. Users can now approve or deny each tool independently when the agent requests multiple actions in a single round.

Before: "I need your approval for: Send text, Create calendar event, Upload file. Yes or no?" (all-or-nothing)

After: Three individual prompts, each with "I'd like to: X. Yes or no?" The user can approve sending the text, deny the calendar event, and approve the upload.

Key behaviors:

  • APPROVED/DENIED: applies to the current tool only, continues to next
  • ALWAYS/NEVER: persists permission for that specific tool, continues to next
  • INTERRUPTED (user changes subject): errors current + all remaining tools, stops asking

Closes #777

Type

  • Feature

Checklist

  • Tests pass (uv run pytest -v)
  • Lint passes (ruff check backend/ && ruff format --check backend/)
  • New tests added for new functionality (test_selective_approval)
  • Bug fixes include regression tests

AI Usage

  • AI-assisted (Claude Code planned and implemented the change)

njbrake and others added 2 commits April 13, 2026 15:17
…777)

Replace the batched approval prompt (approve all or deny all) with
sequential per-tool prompts so users can approve or deny each tool
independently. This gives granular control without needing skip/only
syntax or new data types.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Rename _format_approval_message to format_approval_message (public API)
- Update module docstring to reflect sequential approval behavior

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@njbrake njbrake merged commit b2878a8 into main Apr 13, 2026
8 checks passed
@njbrake njbrake deleted the feat/777-skip-only-plan-approval branch April 13, 2026 15:37
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.

skip N / only N response parsing for plan approval

1 participant