Skip to content

fix(claude): remove --dangerously-skip-permissions flag#60

Open
rbpata wants to merge 1 commit into
ryanstephen:mainfrom
rbpata:fix/remove-dangerously-skip-permissions
Open

fix(claude): remove --dangerously-skip-permissions flag#60
rbpata wants to merge 1 commit into
ryanstephen:mainfrom
rbpata:fix/remove-dangerously-skip-permissions

Conversation

@rbpata

@rbpata rbpata commented May 27, 2026

Copy link
Copy Markdown

Summary

  • Removes the --dangerously-skip-permissions flag from the Claude CLI invocation in ClaudeSession.swift
  • Claude Code now runs with its standard permission system, prompting the user before executing file writes, shell commands, or other potentially destructive actions
  • No other behaviour is changed — streaming JSON I/O, session lifecycle, and history handling are all unaffected

Motivation

The flag was silently bypassing all of Claude's built-in safety guardrails. A user interacting via the lil agents chat UI had no indication that Claude could write files or run shell commands without any confirmation. Restoring the default permission flow gives users visibility and control over what the agent does on their machine.

Test plan

  • Build and run the app in Xcode
  • Open the Claude chat popover and send a prompt that triggers a tool use (e.g. "create a file called test.txt")
  • Confirm that Claude now shows a permission prompt / pauses for confirmation instead of executing silently
  • Confirm normal conversation (no tool use) still works as expected

🤖 Generated with Claude Code

Claude Code was launched with --dangerously-skip-permissions, which
silently bypasses all permission confirmations for file writes, shell
commands, and other potentially destructive operations. This removes
the flag so users are prompted before Claude executes actions, giving
them the standard safety guardrails.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant