Merge pull request #491 from Mng-dev-ai/fix/default-permission-defaults

Set default permission modes per agent

Author: Mng-dev-ai

backend/app/api/endpoints/chat.py

@@ -110,7 +110,7 @@ async def send_message(
     prompt: str = Form(...),
     chat_id: str = Form(...),
     model_id: str = Form(...),
-    permission_mode: PermissionMode = Form("acceptEdits"),
+    permission_mode: PermissionMode = Form("bypassPermissions"),
     thinking_mode: str | None = Form(None),
     worktree: bool = Form(False),
     plan_mode: bool = Form(False),
@@ -430,7 +430,7 @@ async def queue_message(
     chat_id: UUID,
     content: str = Form(...),
     model_id: str = Form(...),
-    permission_mode: PermissionMode = Form("acceptEdits"),
+    permission_mode: PermissionMode = Form("bypassPermissions"),
     thinking_mode: str | None = Form(None),
     worktree: bool = Form(False),
     plan_mode: bool = Form(False),

backend/app/models/schemas/chat.py

@@ -31,7 +31,7 @@ class ChatRequest(BaseModel):
     chat_id: UUID
     model_id: str = Field(..., min_length=1, max_length=255)
     attached_files: list[UploadFile] | None = None
-    permission_mode: PermissionMode = "acceptEdits"
+    permission_mode: PermissionMode = "bypassPermissions"
     thinking_mode: str | None = Field(None, max_length=50)
     worktree: bool = False
     plan_mode: bool = False

backend/app/models/schemas/queue.py

@@ -11,7 +11,7 @@
 class QueuedMessageBase(BaseModel):
     content: str = Field(..., min_length=1, max_length=100000)
     model_id: str = Field(..., min_length=1, max_length=255)
-    permission_mode: PermissionMode = "acceptEdits"
+    permission_mode: PermissionMode = "bypassPermissions"
     thinking_mode: str | None = None
     worktree: bool = False
     plan_mode: bool = False

backend/app/services/queue.py

@@ -38,7 +38,7 @@ async def add_message(
         chat_id: str,
         content: str,
         model_id: str,
-        permission_mode: str = "acceptEdits",
+        permission_mode: str = "bypassPermissions",
         thinking_mode: str | None = None,
         worktree: bool = False,
         plan_mode: bool = False,

frontend/src/components/chat/permission-mode-selector/PermissionModeSelector.tsx

@@ -49,8 +49,8 @@ export const MODES_BY_AGENT: Record<AgentKind, PermissionModeOption[]> = {
 };
 
 const DEFAULT_BY_AGENT: Record<AgentKind, PermissionMode> = {
-  claude: 'acceptEdits',
-  codex: 'auto',
+  claude: 'bypassPermissions',
+  codex: 'full-access',
 };
 
 export function coercePermissionModeForAgent(

frontend/src/components/chat/sub-threads/CreateSubThreadDialog.tsx

@@ -26,7 +26,11 @@ import { useCreateSubThreadMutation } from '@/hooks/queries/useChatQueries';
 import { useSlashCommandSuggestions } from '@/hooks/useSlashCommandSuggestions';
 import { useChatContext } from '@/hooks/useChatContext';
 import { useModelStore } from '@/store/modelStore';
-import { useChatSettingsStore, DEFAULT_PERSONA } from '@/store/chatSettingsStore';
+import {
+  useChatSettingsStore,
+  DEFAULT_PERSONA,
+  DEFAULT_PERMISSION_MODE,
+} from '@/store/chatSettingsStore';
 import { useAuthStore } from '@/store/authStore';
 import type { Chat } from '@/types/chat.types';
 import type { SlashCommand } from '@/types/ui.types';
@@ -47,7 +51,7 @@ export function CreateSubThreadDialog({ parentChat, onClose }: CreateSubThreadDi
   const [personaName, setPersonaName] = useState(DEFAULT_PERSONA);
   const [message, setMessage] = useState('');
   const [thinkingMode, setThinkingMode] = useState<ThinkingModeOption>(CLAUDE_THINKING_MODES[1]);
-  const [permissionMode, setPermissionMode] = useState<PermissionMode>('acceptEdits');
+  const [permissionMode, setPermissionMode] = useState<PermissionMode>(DEFAULT_PERMISSION_MODE);
 
   const { customSkills, builtinSlashCommands } = useChatContext();
 

frontend/src/services/queueService.ts

@@ -1,6 +1,6 @@
 import { apiClient } from '@/lib/api';
 import { ensureResponse, serviceCall } from '@/services/base/BaseService';
-import { DEFAULT_PERSONA } from '@/store/chatSettingsStore';
+import { DEFAULT_PERSONA, DEFAULT_PERMISSION_MODE } from '@/store/chatSettingsStore';
 import type { PermissionMode } from '@/store/chatSettingsStore';
 import { validateId, validateRequired } from '@/utils/validation';
 import type { QueuedMessage, QueueAddResponse } from '@/types/queue.types';
@@ -9,7 +9,7 @@ async function queueMessage(
   chatId: string,
   content: string,
   modelId: string,
-  permissionMode: PermissionMode = 'acceptEdits',
+  permissionMode: PermissionMode = DEFAULT_PERMISSION_MODE,
   thinkingMode: string | null = null,
   worktree: boolean = false,
   planMode: boolean = false,

frontend/src/store/chatSettingsStore.ts

@@ -11,7 +11,7 @@ type PermissionMode =
   | 'full-access';
 
 const DEFAULT_KEY = '__default__';
-export const DEFAULT_PERMISSION_MODE: PermissionMode = 'acceptEdits';
+export const DEFAULT_PERMISSION_MODE: PermissionMode = 'bypassPermissions';
 export const DEFAULT_THINKING_MODE: string = 'medium';
 export const DEFAULT_WORKTREE = false;
 export const DEFAULT_PLAN_MODE = false;

frontend/src/store/messageQueueStore.ts

@@ -1,7 +1,7 @@
 import { create } from 'zustand';
 import type { LocalQueuedMessage } from '@/types/queue.types';
 import { queueService } from '@/services/queueService';
-import { DEFAULT_PERSONA } from '@/store/chatSettingsStore';
+import { DEFAULT_PERSONA, DEFAULT_PERMISSION_MODE } from '@/store/chatSettingsStore';
 import type { PermissionMode } from '@/store/chatSettingsStore';
 
 export const EMPTY_QUEUE: LocalQueuedMessage[] = [];
@@ -40,7 +40,7 @@ export const useMessageQueueStore = create<MessageQueueState>((set, get) => ({
     chatId: string,
     content: string,
     modelId: string,
-    permissionMode: PermissionMode = 'acceptEdits',
+    permissionMode: PermissionMode = DEFAULT_PERMISSION_MODE,
     thinkingMode: string | null = null,
     worktree: boolean = false,
     planMode: boolean = false,