Skip to content

fix: Rename bypass permissions to auto-accept permissions#1469

Merged
charlesvien merged 3 commits intomainfrom
04-02-permissions_ui_nits
Apr 3, 2026
Merged

fix: Rename bypass permissions to auto-accept permissions#1469
charlesvien merged 3 commits intomainfrom
04-02-permissions_ui_nits

Conversation

@charlesvien
Copy link
Copy Markdown
Member

@charlesvien charlesvien commented Apr 2, 2026

Problem

"Bypass Permissions" is confusing and sounds like a security exploit rather than a deliberate user choice.

Changes

  1. Rename "Bypass Permissions" to "Auto-accept permissions" across settings UI and agent
  2. Update callout text to mention shift+tab shortcut for toggling the mode
  3. Reword confirmation dialog to use "auto-accept" language

How did you test this?

Manually

@charlesvien charlesvien changed the title Rename bypass permissions to auto-accept permissions fix: Rename bypass permissions to auto-accept permissions Apr 2, 2026
@charlesvien charlesvien marked this pull request as ready for review April 2, 2026 22:21
@charlesvien charlesvien force-pushed the 04-02-permissions_ui_nits branch 2 times, most recently from c9f077b to fdc1564 Compare April 2, 2026 22:55
@charlesvien charlesvien force-pushed the 04-02-permissions_ui_nits branch 2 times, most recently from e002336 to 2a03b44 Compare April 2, 2026 22:57
Copy link
Copy Markdown
Contributor

@adboio adboio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i like this better, but as a user i am still confused on what exactly it does

would be nice to have some quick reference here, particularly if the user has never enabled this before

i avoid it every time bc i have no clue what i'm allowing

@charlesvien
Copy link
Copy Markdown
Member Author

i like this better, but as a user i am still confused on what exactly it does

would be nice to have some quick reference here, particularly if the user has never enabled this before

i avoid it every time bc i have no clue what i'm allowing

I just made some clarifying changes to the wording in 9301abf

We do heavily warn them incl. a popup when they try to enable it, but I think this might help clarify what exactly they are enabling.

@charlesvien charlesvien force-pushed the 04-02-permissions_ui_nits branch from 9301abf to 5cf1b30 Compare April 3, 2026 00:56
Copy link
Copy Markdown
Member Author

charlesvien commented Apr 3, 2026

Merge activity

  • Apr 3, 1:05 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 3, 1:07 AM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 3, 1:11 AM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien changed the base branch from 04-02-ui_nits to graphite-base/1469 April 3, 2026 01:05
@charlesvien charlesvien changed the base branch from graphite-base/1469 to main April 3, 2026 01:05
@charlesvien charlesvien force-pushed the 04-02-permissions_ui_nits branch from 5cf1b30 to 935058b Compare April 3, 2026 01:06
@charlesvien charlesvien merged commit 613748b into main Apr 3, 2026
16 checks passed
@charlesvien charlesvien deleted the 04-02-permissions_ui_nits branch April 3, 2026 01:11
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