Add dangerous permission bypass flag - #268
Conversation
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
Included review availability: 3 reviews are currently available. Based on recent review activity, included reviews refill at 4 per hour. 📝 WalkthroughWalkthroughThe experimental ChangesCLI permission mode
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This localized change adds an explicit permission-bypass option to the experimental CLI while preserving the normal approval flow; no actionable merge-blocking risk remains after normal checks and review. Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Summary
--dangerously-skip-permissionsto the experimentaldiveCLIpermission.ModeBypassPermissionsmode when the flag is present while preserving the default approval flowWhy
Externally sandboxed or otherwise pre-authorized runs need an explicit way to execute tool calls without stopping for interactive approval prompts.
Validation
go test ./...cd experimental/cmd/dive && go test -race ./...cd experimental/cmd/dive && go run . --helpconfirms the public flaggit diff --checkSummary by CodeRabbit
New Features
diveCLI option to bypass tool approval prompts.Documentation