From 844be657c88b59efcef2f38f230ff2a5f720f17d Mon Sep 17 00:00:00 2001 From: webdevcody Date: Thu, 1 Jan 2026 17:08:15 -0500 Subject: [PATCH] feat: add skipSandboxWarning to settings and sync function - Introduced skipSandboxWarning property in GlobalSettings interface to manage user preference for sandbox risk warnings. - Updated syncSettingsToServer function to include skipSandboxWarning in the settings synchronization process. - Set default value for skipSandboxWarning to false in DEFAULT_GLOBAL_SETTINGS. --- apps/ui/src/hooks/use-settings-migration.ts | 1 + libs/types/src/settings.ts | 3 +++ 2 files changed, 4 insertions(+) diff --git a/apps/ui/src/hooks/use-settings-migration.ts b/apps/ui/src/hooks/use-settings-migration.ts index 54b24b24..7abc86c2 100644 --- a/apps/ui/src/hooks/use-settings-migration.ts +++ b/apps/ui/src/hooks/use-settings-migration.ts @@ -226,6 +226,7 @@ export async function syncSettingsToServer(): Promise { validationModel: state.validationModel, autoLoadClaudeMd: state.autoLoadClaudeMd, enableSandboxMode: state.enableSandboxMode, + skipSandboxWarning: state.skipSandboxWarning, keyboardShortcuts: state.keyboardShortcuts, aiProfiles: state.aiProfiles, mcpServers: state.mcpServers, diff --git a/libs/types/src/settings.ts b/libs/types/src/settings.ts index 990c2ff6..309703ce 100644 --- a/libs/types/src/settings.ts +++ b/libs/types/src/settings.ts @@ -353,6 +353,8 @@ export interface GlobalSettings { autoLoadClaudeMd?: boolean; /** Enable sandbox mode for bash commands (default: false, enable for additional security) */ enableSandboxMode?: boolean; + /** Skip showing the sandbox risk warning dialog */ + skipSandboxWarning?: boolean; // MCP Server Configuration /** List of configured MCP servers for agent use */ @@ -531,6 +533,7 @@ export const DEFAULT_GLOBAL_SETTINGS: GlobalSettings = { lastSelectedSessionByProject: {}, autoLoadClaudeMd: false, enableSandboxMode: false, + skipSandboxWarning: false, mcpServers: [], // Default to true for autonomous workflow. Security is enforced when adding servers // via the security warning dialog that explains the risks.