Add /yolo slash command to toggle skip-permissions live - #485
Merged
TheGreatAxios merged 2 commits intoAug 19, 2026
Merged
Conversation
TheGreatAxios
commented
Aug 19, 2026
TheGreatAxios
left a comment
Collaborator
Author
There was a problem hiding this comment.
Reviewed as reviewer of record against CL-6297.
Verdict: approve (GitHub will not let this identity formally approve its own PR).
/yolo is a live twin of the CLI skip-permissions flag: it flips setSkipPermissions on the existing gate and pre-gate sandboxes consult a getter, so outside-workspace access unlocks (or re-locks) without rebuilding plugins.
Checked:
- Command:
/yolo,/yolo on,/yolo off; bare invocation toggles. Missing context (exec) returns not available in this mode. - Enable message states the envelope: gate bypassed, secret-guard and authz remain.
- Envelope holds:
secretGuardPluginandauthzPluginsit beforepermissionPluginand do not read skip-permissions. Gateevaluate()early-returns the same way the CLI flag already does. - Live sandbox consult:
pathEscapePlugin,deleteFilePlugin,shellGuardPlugin/ persistent cwd, andlist_dirall takeallowOutside: () => permissionGate.getSkipPermissions(). - Tests cover command on/off/toggle/unknown/unwired, gate live toggle, and per-call allowOutside getters.
- ARCHITECTURE / IMPLEMENTATION / PRODUCT (secret guard, path sandbox, recovery) mention
/yolo.
Nits (not blocking):
- Ticket listed an explicit
toggleargument. Handler treats only empty args as toggle and rejectstogglewithUsage: /yolo [on|off]. Easy follow-up if you want the ticket text to be literal. - PRODUCT.md's built-in slash-command list still omits
/yoloeven though nearby paragraphs mention it.
Commit message is plain English and matches the change.
Mirror setAuto with setSkipPermissions, resolve allowOutside per call so sandboxes honor the toggle, and wire /yolo on|off|toggle in the TUI.
Bare /yolo already toggled; the ticket also named an explicit toggle argument. PRODUCT now includes /yolo in the slash list.
TheGreatAxios
force-pushed
the
cl-6297-add-yolo-slash-command-to-toggle-skip-permissions-mid
branch
from
August 19, 2026 07:22
394972a to
fc08b88
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a
/yoloslash command that toggles--dangerously-skip-permissionsfor the rest of the session, with a clear status flash and transcript notice.Closes CL-6297.
Changes
/yoloflips liveskipPermissionsvia the permission gateTest plan
/yoloenables skip-permissions; consequential tools do not ask/yoloagain disables; asks resume