@didim365/agent-cli 0.1.1 → 0.1.3

package/dist/src/config/settings.test.js

@@ -1,1822 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-/// <reference types="vitest/globals" />
-// Mock 'os' first.
-import * as osActual from 'node:os'; // Import for type info for the mock factory
-vi.mock('os', async (importOriginal) => {
-    const actualOs = await importOriginal();
-    return {
-        ...actualOs,
-        homedir: vi.fn(() => '/mock/home/user'),
-        platform: vi.fn(() => 'linux'),
-    };
-});
-// Mock './settings.js' to ensure it uses the mocked 'os.homedir()' for its internal constants.
-vi.mock('./settings.js', async (importActual) => {
-    const originalModule = await importActual();
-    return {
-        __esModule: true, // Ensure correct module shape
-        ...originalModule, // Re-export all original members
-        // We are relying on originalModule's USER_SETTINGS_PATH being constructed with mocked os.homedir()
-    };
-});
-// Mock trustedFolders
-vi.mock('./trustedFolders.js', () => ({
-    isWorkspaceTrusted: vi
-        .fn()
-        .mockReturnValue({ isTrusted: true, source: 'file' }),
-}));
-vi.mock('./settingsSchema.js', async (importOriginal) => {
-    const actual = await importOriginal();
-    return {
-        ...actual,
-        getSettingsSchema: vi.fn(actual.getSettingsSchema),
-    };
-});
-// NOW import everything else, including the (now effectively re-exported) settings.js
-import * as path from 'node:path'; // Restored for MOCK_WORKSPACE_SETTINGS_PATH
-import { describe, it, expect, vi, beforeEach, afterEach, } from 'vitest';
-import * as fs from 'node:fs'; // fs will be mocked separately
-import stripJsonComments from 'strip-json-comments'; // Will be mocked separately
-import { isWorkspaceTrusted } from './trustedFolders.js';
-// These imports will get the versions from the vi.mock('./settings.js', ...) factory.
-import { loadSettings, USER_SETTINGS_PATH, // This IS the mocked path.
-getSystemSettingsPath, getSystemDefaultsPath, saveSettings, getDefaultsFromSchema, loadEnvironment, migrateDeprecatedSettings, SettingScope, LoadedSettings, } from './settings.js';
-import { FatalConfigError, GEMINI_DIR } from '@didim365/agent-cli-core';
-import { updateSettingsFilePreservingFormat } from '../utils/commentJson.js';
-import { getSettingsSchema, MergeStrategy, } from './settingsSchema.js';
-const MOCK_WORKSPACE_DIR = '/mock/workspace';
-// Use the (mocked) GEMINI_DIR for consistency
-const MOCK_WORKSPACE_SETTINGS_PATH = path.join(MOCK_WORKSPACE_DIR, GEMINI_DIR, 'settings.json');
-vi.mock('fs', async (importOriginal) => {
-    // Get all the functions from the real 'fs' module
-    const actualFs = await importOriginal();
-    return {
-        ...actualFs, // Keep all the real functions
-        // Now, just override the ones we need for the test
-        existsSync: vi.fn(),
-        readFileSync: vi.fn(),
-        writeFileSync: vi.fn(),
-        mkdirSync: vi.fn(),
-        realpathSync: (p) => p,
-    };
-});
-vi.mock('./extension.js');
-const mockCoreEvents = vi.hoisted(() => ({
-    emitFeedback: vi.fn(),
-    emitSettingsChanged: vi.fn(),
-}));
-vi.mock('@didim365/agent-cli-core', async (importOriginal) => {
-    const actual = await importOriginal();
-    return {
-        ...actual,
-        coreEvents: mockCoreEvents,
-    };
-});
-vi.mock('../utils/commentJson.js', () => ({
-    updateSettingsFilePreservingFormat: vi.fn(),
-}));
-vi.mock('strip-json-comments', () => ({
-    default: vi.fn((content) => content),
-}));
-describe('Settings Loading and Merging', () => {
-    let mockFsExistsSync;
-    let mockStripJsonComments;
-    let mockFsMkdirSync;
-    beforeEach(() => {
-        vi.resetAllMocks();
-        mockFsExistsSync = vi.mocked(fs.existsSync);
-        mockFsMkdirSync = vi.mocked(fs.mkdirSync);
-        mockStripJsonComments = vi.mocked(stripJsonComments);
-        vi.mocked(osActual.homedir).mockReturnValue('/mock/home/user');
-        mockStripJsonComments.mockImplementation((jsonString) => jsonString);
-        mockFsExistsSync.mockReturnValue(false);
-        fs.readFileSync.mockReturnValue('{}'); // Return valid empty JSON
-        mockFsMkdirSync.mockImplementation(() => undefined);
-        vi.mocked(isWorkspaceTrusted).mockReturnValue({
-            isTrusted: true,
-            source: 'file',
-        });
-    });
-    afterEach(() => {
-        vi.restoreAllMocks();
-    });
-    describe('loadSettings', () => {
-        it.each([
-            {
-                scope: 'system',
-                path: getSystemSettingsPath(),
-                content: {
-                    ui: { theme: 'system-default' },
-                    tools: { sandbox: false },
-                },
-            },
-            {
-                scope: 'user',
-                path: USER_SETTINGS_PATH,
-                content: {
-                    ui: { theme: 'dark' },
-                    context: { fileName: 'USER_CONTEXT.md' },
-                },
-            },
-            {
-                scope: 'workspace',
-                path: MOCK_WORKSPACE_SETTINGS_PATH,
-                content: {
-                    tools: { sandbox: true },
-                    context: { fileName: 'WORKSPACE_CONTEXT.md' },
-                },
-            },
-        ])('should load $scope settings if only $scope file exists', ({ scope, path, content }) => {
-            mockFsExistsSync.mockImplementation((p) => p === path);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === path)
-                    return JSON.stringify(content);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(fs.readFileSync).toHaveBeenCalledWith(path, 'utf-8');
-            expect(settings[scope].settings).toEqual(content);
-            expect(settings.merged).toMatchObject(content);
-        });
-        it('should merge system, user and workspace settings, with system taking precedence over workspace, and workspace over user', () => {
-            mockFsExistsSync.mockImplementation((p) => p === getSystemSettingsPath() ||
-                p === USER_SETTINGS_PATH ||
-                p === MOCK_WORKSPACE_SETTINGS_PATH);
-            const systemSettingsContent = {
-                ui: {
-                    theme: 'system-theme',
-                },
-                tools: {
-                    sandbox: false,
-                },
-                mcp: {
-                    allowed: ['server1', 'server2'],
-                },
-                telemetry: { enabled: false },
-            };
-            const userSettingsContent = {
-                ui: {
-                    theme: 'dark',
-                },
-                tools: {
-                    sandbox: true,
-                },
-                context: {
-                    fileName: 'USER_CONTEXT.md',
-                },
-            };
-            const workspaceSettingsContent = {
-                tools: {
-                    sandbox: false,
-                    core: ['tool1'],
-                },
-                context: {
-                    fileName: 'WORKSPACE_CONTEXT.md',
-                },
-                mcp: {
-                    allowed: ['server1', 'server2', 'server3'],
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.system.settings).toEqual(systemSettingsContent);
-            expect(settings.user.settings).toEqual(userSettingsContent);
-            expect(settings.workspace.settings).toEqual(workspaceSettingsContent);
-            expect(settings.merged).toMatchObject({
-                ui: {
-                    theme: 'system-theme',
-                },
-                tools: {
-                    sandbox: false,
-                    core: ['tool1'],
-                },
-                telemetry: { enabled: false },
-                context: {
-                    fileName: 'WORKSPACE_CONTEXT.md',
-                },
-                mcp: {
-                    allowed: ['server1', 'server2'],
-                },
-            });
-        });
-        it('should merge all settings files with the correct precedence', () => {
-            // Mock schema to test defaults application
-            const mockSchema = {
-                ui: { type: 'object', default: {}, properties: {} },
-                tools: { type: 'object', default: {}, properties: {} },
-                context: {
-                    type: 'object',
-                    default: {},
-                    properties: {
-                        discoveryMaxDirs: { type: 'number', default: 200 },
-                        includeDirectories: {
-                            type: 'array',
-                            default: [],
-                            mergeStrategy: MergeStrategy.CONCAT,
-                        },
-                    },
-                },
-                mcpServers: { type: 'object', default: {} },
-            };
-            getSettingsSchema.mockReturnValue(mockSchema);
-            mockFsExistsSync.mockReturnValue(true);
-            const systemDefaultsContent = {
-                ui: {
-                    theme: 'default-theme',
-                },
-                tools: {
-                    sandbox: true,
-                },
-                telemetry: true,
-                context: {
-                    includeDirectories: ['/system/defaults/dir'],
-                },
-            };
-            const userSettingsContent = {
-                ui: {
-                    theme: 'user-theme',
-                },
-                context: {
-                    fileName: 'USER_CONTEXT.md',
-                    includeDirectories: ['/user/dir1', '/user/dir2'],
-                },
-            };
-            const workspaceSettingsContent = {
-                tools: {
-                    sandbox: false,
-                },
-                context: {
-                    fileName: 'WORKSPACE_CONTEXT.md',
-                    includeDirectories: ['/workspace/dir'],
-                },
-            };
-            const systemSettingsContent = {
-                ui: {
-                    theme: 'system-theme',
-                },
-                telemetry: false,
-                context: {
-                    includeDirectories: ['/system/dir'],
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemDefaultsPath())
-                    return JSON.stringify(systemDefaultsContent);
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.systemDefaults.settings).toEqual(systemDefaultsContent);
-            expect(settings.system.settings).toEqual(systemSettingsContent);
-            expect(settings.user.settings).toEqual(userSettingsContent);
-            expect(settings.workspace.settings).toEqual(workspaceSettingsContent);
-            expect(settings.merged).toEqual({
-                context: {
-                    discoveryMaxDirs: 200,
-                    includeDirectories: [
-                        '/system/defaults/dir',
-                        '/user/dir1',
-                        '/user/dir2',
-                        '/workspace/dir',
-                        '/system/dir',
-                    ],
-                    fileName: 'WORKSPACE_CONTEXT.md',
-                },
-                mcpServers: {},
-                ui: { theme: 'system-theme' },
-                tools: { sandbox: false },
-                telemetry: false,
-            });
-        });
-        it('should use folderTrust from workspace settings when trusted', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = {
-                security: {
-                    folderTrust: {
-                        enabled: true,
-                    },
-                },
-            };
-            const workspaceSettingsContent = {
-                security: {
-                    folderTrust: {
-                        enabled: false, // This should be used
-                    },
-                },
-            };
-            const systemSettingsContent = {
-            // No folderTrust here
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.security?.folderTrust?.enabled).toBe(false); // Workspace setting should be used
-        });
-        it('should use system folderTrust over user setting', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = {
-                security: {
-                    folderTrust: {
-                        enabled: false,
-                    },
-                },
-            };
-            const workspaceSettingsContent = {
-                security: {
-                    folderTrust: {
-                        enabled: true, // This should be ignored
-                    },
-                },
-            };
-            const systemSettingsContent = {
-                security: {
-                    folderTrust: {
-                        enabled: true,
-                    },
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.security?.folderTrust?.enabled).toBe(true); // System setting should be used
-        });
-        it('should not allow user or workspace to override system disableYoloMode', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = {
-                security: {
-                    disableYoloMode: false,
-                },
-            };
-            const workspaceSettingsContent = {
-                security: {
-                    disableYoloMode: false, // This should be ignored
-                },
-            };
-            const systemSettingsContent = {
-                security: {
-                    disableYoloMode: true,
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.security?.disableYoloMode).toBe(true); // System setting should be used
-        });
-        it.each([
-            {
-                description: 'contextFileName in user settings',
-                path: USER_SETTINGS_PATH,
-                content: { context: { fileName: 'CUSTOM.md' } },
-                expected: { key: 'context.fileName', value: 'CUSTOM.md' },
-            },
-            {
-                description: 'contextFileName in workspace settings',
-                path: MOCK_WORKSPACE_SETTINGS_PATH,
-                content: { context: { fileName: 'PROJECT_SPECIFIC.md' } },
-                expected: { key: 'context.fileName', value: 'PROJECT_SPECIFIC.md' },
-            },
-            {
-                description: 'excludedProjectEnvVars in user settings',
-                path: USER_SETTINGS_PATH,
-                content: {
-                    advanced: { excludedEnvVars: ['DEBUG', 'NODE_ENV', 'CUSTOM_VAR'] },
-                },
-                expected: {
-                    key: 'advanced.excludedEnvVars',
-                    value: ['DEBUG', 'DEBUG_MODE', 'NODE_ENV', 'CUSTOM_VAR'],
-                },
-            },
-            {
-                description: 'excludedProjectEnvVars in workspace settings',
-                path: MOCK_WORKSPACE_SETTINGS_PATH,
-                content: {
-                    advanced: { excludedEnvVars: ['WORKSPACE_DEBUG', 'WORKSPACE_VAR'] },
-                },
-                expected: {
-                    key: 'advanced.excludedEnvVars',
-                    value: ['DEBUG', 'DEBUG_MODE', 'WORKSPACE_DEBUG', 'WORKSPACE_VAR'],
-                },
-            },
-        ])('should handle $description correctly', ({ path, content, expected }) => {
-            mockFsExistsSync.mockImplementation((p) => p === path);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === path)
-                    return JSON.stringify(content);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            const keys = expected.key.split('.');
-            let result = settings.merged;
-            for (const key of keys) {
-                result = result[key];
-            }
-            expect(result).toEqual(expected.value);
-        });
-        it('should merge excludedProjectEnvVars with workspace taking precedence over user', () => {
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH || p === MOCK_WORKSPACE_SETTINGS_PATH);
-            const userSettingsContent = {
-                general: {},
-                advanced: { excludedEnvVars: ['DEBUG', 'NODE_ENV', 'USER_VAR'] },
-            };
-            const workspaceSettingsContent = {
-                general: {},
-                advanced: { excludedEnvVars: ['WORKSPACE_DEBUG', 'WORKSPACE_VAR'] },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings.advanced?.excludedEnvVars).toEqual([
-                'DEBUG',
-                'NODE_ENV',
-                'USER_VAR',
-            ]);
-            expect(settings.workspace.settings.advanced?.excludedEnvVars).toEqual([
-                'WORKSPACE_DEBUG',
-                'WORKSPACE_VAR',
-            ]);
-            expect(settings.merged.advanced?.excludedEnvVars).toEqual([
-                'DEBUG',
-                'DEBUG_MODE',
-                'NODE_ENV',
-                'USER_VAR',
-                'WORKSPACE_DEBUG',
-                'WORKSPACE_VAR',
-            ]);
-        });
-        it('should default contextFileName to undefined if not in any settings file', () => {
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH || p === MOCK_WORKSPACE_SETTINGS_PATH);
-            const userSettingsContent = { ui: { theme: 'dark' } };
-            const workspaceSettingsContent = { tools: { sandbox: true } };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.context?.fileName).toBeUndefined();
-        });
-        it.each([
-            {
-                scope: 'user',
-                path: USER_SETTINGS_PATH,
-                content: { telemetry: { enabled: true } },
-                expected: true,
-            },
-            {
-                scope: 'workspace',
-                path: MOCK_WORKSPACE_SETTINGS_PATH,
-                content: { telemetry: { enabled: false } },
-                expected: false,
-            },
-        ])('should load telemetry setting from $scope settings', ({ path, content, expected }) => {
-            mockFsExistsSync.mockImplementation((p) => p === path);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === path)
-                    return JSON.stringify(content);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.telemetry?.enabled).toBe(expected);
-        });
-        it('should prioritize workspace telemetry setting over user setting', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = { telemetry: { enabled: true } };
-            const workspaceSettingsContent = { telemetry: { enabled: false } };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.telemetry?.enabled).toBe(false);
-        });
-        it('should have telemetry as undefined if not in any settings file', () => {
-            mockFsExistsSync.mockReturnValue(false); // No settings files exist
-            fs.readFileSync.mockReturnValue('{}');
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.telemetry).toBeUndefined();
-            expect(settings.merged.ui).toBeDefined();
-            expect(settings.merged.mcpServers).toEqual({});
-        });
-        it('should merge MCP servers correctly, with workspace taking precedence', () => {
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH || p === MOCK_WORKSPACE_SETTINGS_PATH);
-            const userSettingsContent = {
-                mcpServers: {
-                    'user-server': {
-                        command: 'user-command',
-                        args: ['--user-arg'],
-                        description: 'User MCP server',
-                    },
-                    'shared-server': {
-                        command: 'user-shared-command',
-                        description: 'User shared server config',
-                    },
-                },
-            };
-            const workspaceSettingsContent = {
-                mcpServers: {
-                    'workspace-server': {
-                        command: 'workspace-command',
-                        args: ['--workspace-arg'],
-                        description: 'Workspace MCP server',
-                    },
-                    'shared-server': {
-                        command: 'workspace-shared-command',
-                        description: 'Workspace shared server config',
-                    },
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings).toEqual(userSettingsContent);
-            expect(settings.workspace.settings).toEqual(workspaceSettingsContent);
-            expect(settings.merged.mcpServers).toEqual({
-                'user-server': {
-                    command: 'user-command',
-                    args: ['--user-arg'],
-                    description: 'User MCP server',
-                },
-                'workspace-server': {
-                    command: 'workspace-command',
-                    args: ['--workspace-arg'],
-                    description: 'Workspace MCP server',
-                },
-                'shared-server': {
-                    command: 'workspace-shared-command',
-                    description: 'Workspace shared server config',
-                },
-            });
-        });
-        it.each([
-            {
-                scope: 'user',
-                path: USER_SETTINGS_PATH,
-                content: {
-                    mcpServers: {
-                        'user-only-server': {
-                            command: 'user-only-command',
-                            description: 'User only server',
-                        },
-                    },
-                },
-                expected: {
-                    'user-only-server': {
-                        command: 'user-only-command',
-                        description: 'User only server',
-                    },
-                },
-            },
-            {
-                scope: 'workspace',
-                path: MOCK_WORKSPACE_SETTINGS_PATH,
-                content: {
-                    mcpServers: {
-                        'workspace-only-server': {
-                            command: 'workspace-only-command',
-                            description: 'Workspace only server',
-                        },
-                    },
-                },
-                expected: {
-                    'workspace-only-server': {
-                        command: 'workspace-only-command',
-                        description: 'Workspace only server',
-                    },
-                },
-            },
-        ])('should handle MCP servers when only in $scope settings', ({ path, content, expected }) => {
-            mockFsExistsSync.mockImplementation((p) => p === path);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === path)
-                    return JSON.stringify(content);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.mcpServers).toEqual(expected);
-        });
-        it('should have mcpServers as undefined if not in any settings file', () => {
-            mockFsExistsSync.mockReturnValue(false); // No settings files exist
-            fs.readFileSync.mockReturnValue('{}');
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.mcpServers).toEqual({});
-        });
-        it('should merge MCP servers from system, user, and workspace with system taking precedence', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const systemSettingsContent = {
-                mcpServers: {
-                    'shared-server': {
-                        command: 'system-command',
-                        args: ['--system-arg'],
-                    },
-                    'system-only-server': {
-                        command: 'system-only-command',
-                    },
-                },
-            };
-            const userSettingsContent = {
-                mcpServers: {
-                    'user-server': {
-                        command: 'user-command',
-                    },
-                    'shared-server': {
-                        command: 'user-command',
-                        description: 'from user',
-                    },
-                },
-            };
-            const workspaceSettingsContent = {
-                mcpServers: {
-                    'workspace-server': {
-                        command: 'workspace-command',
-                    },
-                    'shared-server': {
-                        command: 'workspace-command',
-                        args: ['--workspace-arg'],
-                    },
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.mcpServers).toEqual({
-                'user-server': {
-                    command: 'user-command',
-                },
-                'workspace-server': {
-                    command: 'workspace-command',
-                },
-                'system-only-server': {
-                    command: 'system-only-command',
-                },
-                'shared-server': {
-                    command: 'system-command',
-                    args: ['--system-arg'],
-                },
-            });
-        });
-        it('should merge mcp allowed/excluded lists with system taking precedence over workspace', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const systemSettingsContent = {
-                mcp: {
-                    allowed: ['system-allowed'],
-                },
-            };
-            const userSettingsContent = {
-                mcp: {
-                    allowed: ['user-allowed'],
-                    excluded: ['user-excluded'],
-                },
-            };
-            const workspaceSettingsContent = {
-                mcp: {
-                    allowed: ['workspace-allowed'],
-                    excluded: ['workspace-excluded'],
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.mcp).toEqual({
-                allowed: ['system-allowed'],
-                excluded: ['workspace-excluded'],
-            });
-        });
-        describe('compressionThreshold settings', () => {
-            it.each([
-                {
-                    description: 'should be taken from user settings if only present there',
-                    userContent: { model: { compressionThreshold: 0.5 } },
-                    workspaceContent: {},
-                    expected: 0.5,
-                },
-                {
-                    description: 'should be taken from workspace settings if only present there',
-                    userContent: {},
-                    workspaceContent: { model: { compressionThreshold: 0.8 } },
-                    expected: 0.8,
-                },
-                {
-                    description: 'should prioritize workspace settings over user settings',
-                    userContent: { model: { compressionThreshold: 0.5 } },
-                    workspaceContent: { model: { compressionThreshold: 0.8 } },
-                    expected: 0.8,
-                },
-                {
-                    description: 'should be default if not in any settings file',
-                    userContent: {},
-                    workspaceContent: {},
-                    expected: 0.5,
-                },
-            ])('$description', ({ userContent, workspaceContent, expected }) => {
-                mockFsExistsSync.mockReturnValue(true);
-                fs.readFileSync.mockImplementation((p) => {
-                    if (p === USER_SETTINGS_PATH)
-                        return JSON.stringify(userContent);
-                    if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                        return JSON.stringify(workspaceContent);
-                    return '{}';
-                });
-                const settings = loadSettings(MOCK_WORKSPACE_DIR);
-                expect(settings.merged.model?.compressionThreshold).toEqual(expected);
-            });
-        });
-        it('should use user compressionThreshold if workspace does not define it', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = {
-                general: {},
-                model: { compressionThreshold: 0.5 },
-            };
-            const workspaceSettingsContent = {
-                general: {},
-                model: {},
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.model?.compressionThreshold).toEqual(0.5);
-        });
-        it('should merge includeDirectories from all scopes', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const systemSettingsContent = {
-                context: { includeDirectories: ['/system/dir'] },
-            };
-            const systemDefaultsContent = {
-                context: { includeDirectories: ['/system/defaults/dir'] },
-            };
-            const userSettingsContent = {
-                context: { includeDirectories: ['/user/dir1', '/user/dir2'] },
-            };
-            const workspaceSettingsContent = {
-                context: { includeDirectories: ['/workspace/dir'] },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath())
-                    return JSON.stringify(systemSettingsContent);
-                if (p === getSystemDefaultsPath())
-                    return JSON.stringify(systemDefaultsContent);
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.context?.includeDirectories).toEqual([
-                '/system/defaults/dir',
-                '/user/dir1',
-                '/user/dir2',
-                '/workspace/dir',
-                '/system/dir',
-            ]);
-        });
-        it('should handle JSON parsing errors gracefully', () => {
-            mockFsExistsSync.mockReturnValue(true); // Both files "exist"
-            const invalidJsonContent = 'invalid json';
-            const userReadError = new SyntaxError("Expected ',' or '}' after property value in JSON at position 10");
-            const workspaceReadError = new SyntaxError('Unexpected token i in JSON at position 0');
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH) {
-                    // Simulate JSON.parse throwing for user settings
-                    vi.spyOn(JSON, 'parse').mockImplementationOnce(() => {
-                        throw userReadError;
-                    });
-                    return invalidJsonContent; // Content that would cause JSON.parse to throw
-                }
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH) {
-                    // Simulate JSON.parse throwing for workspace settings
-                    vi.spyOn(JSON, 'parse').mockImplementationOnce(() => {
-                        throw workspaceReadError;
-                    });
-                    return invalidJsonContent;
-                }
-                return '{}'; // Default for other reads
-            });
-            try {
-                loadSettings(MOCK_WORKSPACE_DIR);
-                throw new Error('loadSettings should have thrown a FatalConfigError');
-            }
-            catch (e) {
-                expect(e).toBeInstanceOf(FatalConfigError);
-                const error = e;
-                expect(error.message).toContain(`Error in ${USER_SETTINGS_PATH}: ${userReadError.message}`);
-                expect(error.message).toContain(`Error in ${MOCK_WORKSPACE_SETTINGS_PATH}: ${workspaceReadError.message}`);
-                expect(error.message).toContain('Please fix the configuration file(s) and try again.');
-            }
-            // Restore JSON.parse mock if it was spied on specifically for this test
-            vi.restoreAllMocks(); // Or more targeted restore if needed
-        });
-        it('should resolve environment variables in user settings', () => {
-            process.env['TEST_API_KEY'] = 'user_api_key_from_env';
-            const userSettingsContent = {
-                apiKey: '$TEST_API_KEY',
-                someUrl: 'https://test.com/${TEST_API_KEY}',
-            };
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings['apiKey']).toBe('user_api_key_from_env');
-            expect(settings.user.settings['someUrl']).toBe('https://test.com/user_api_key_from_env');
-            expect(settings.merged['apiKey']).toBe('user_api_key_from_env');
-            delete process.env['TEST_API_KEY'];
-        });
-        it('should resolve environment variables in workspace settings', () => {
-            process.env['WORKSPACE_ENDPOINT'] = 'workspace_endpoint_from_env';
-            const workspaceSettingsContent = {
-                endpoint: '${WORKSPACE_ENDPOINT}/api',
-                nested: { value: '$WORKSPACE_ENDPOINT' },
-            };
-            mockFsExistsSync.mockImplementation((p) => p === MOCK_WORKSPACE_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.workspace.settings['endpoint']).toBe('workspace_endpoint_from_env/api');
-            const nested = settings.workspace.settings['nested'];
-            expect(nested['value']).toBe('workspace_endpoint_from_env');
-            expect(settings.merged['endpoint']).toBe('workspace_endpoint_from_env/api');
-            delete process.env['WORKSPACE_ENDPOINT'];
-        });
-        it('should correctly resolve and merge env variables from different scopes', () => {
-            process.env['SYSTEM_VAR'] = 'system_value';
-            process.env['USER_VAR'] = 'user_value';
-            process.env['WORKSPACE_VAR'] = 'workspace_value';
-            process.env['SHARED_VAR'] = 'final_value';
-            const systemSettingsContent = {
-                configValue: '$SHARED_VAR',
-                systemOnly: '$SYSTEM_VAR',
-            };
-            const userSettingsContent = {
-                configValue: '$SHARED_VAR',
-                userOnly: '$USER_VAR',
-                ui: {
-                    theme: 'dark',
-                },
-            };
-            const workspaceSettingsContent = {
-                configValue: '$SHARED_VAR',
-                workspaceOnly: '$WORKSPACE_VAR',
-                ui: {
-                    theme: 'light',
-                },
-            };
-            mockFsExistsSync.mockReturnValue(true);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath()) {
-                    return JSON.stringify(systemSettingsContent);
-                }
-                if (p === USER_SETTINGS_PATH) {
-                    return JSON.stringify(userSettingsContent);
-                }
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH) {
-                    return JSON.stringify(workspaceSettingsContent);
-                }
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            // Check resolved values in individual scopes
-            expect(settings.system.settings['configValue']).toBe('final_value');
-            expect(settings.system.settings['systemOnly']).toBe('system_value');
-            expect(settings.user.settings['configValue']).toBe('final_value');
-            expect(settings.user.settings['userOnly']).toBe('user_value');
-            expect(settings.workspace.settings['configValue']).toBe('final_value');
-            expect(settings.workspace.settings['workspaceOnly']).toBe('workspace_value');
-            // Check merged values (system > workspace > user)
-            expect(settings.merged['configValue']).toBe('final_value');
-            expect(settings.merged['systemOnly']).toBe('system_value');
-            expect(settings.merged['userOnly']).toBe('user_value');
-            expect(settings.merged['workspaceOnly']).toBe('workspace_value');
-            expect(settings.merged.ui?.theme).toBe('light'); // workspace overrides user
-            delete process.env['SYSTEM_VAR'];
-            delete process.env['USER_VAR'];
-            delete process.env['WORKSPACE_VAR'];
-            delete process.env['SHARED_VAR'];
-        });
-        it('should correctly merge dnsResolutionOrder with workspace taking precedence', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = {
-                advanced: { dnsResolutionOrder: 'ipv4first' },
-            };
-            const workspaceSettingsContent = {
-                advanced: { dnsResolutionOrder: 'verbatim' },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.advanced?.dnsResolutionOrder).toBe('verbatim');
-        });
-        it('should use user dnsResolutionOrder if workspace is not defined', () => {
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            const userSettingsContent = {
-                advanced: { dnsResolutionOrder: 'verbatim' },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.advanced?.dnsResolutionOrder).toBe('verbatim');
-        });
-        it('should leave unresolved environment variables as is', () => {
-            const userSettingsContent = { apiKey: '$UNDEFINED_VAR' };
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings['apiKey']).toBe('$UNDEFINED_VAR');
-            expect(settings.merged['apiKey']).toBe('$UNDEFINED_VAR');
-        });
-        it('should resolve multiple environment variables in a single string', () => {
-            process.env['VAR_A'] = 'valueA';
-            process.env['VAR_B'] = 'valueB';
-            const userSettingsContent = {
-                path: '/path/$VAR_A/${VAR_B}/end',
-            };
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings['path']).toBe('/path/valueA/valueB/end');
-            delete process.env['VAR_A'];
-            delete process.env['VAR_B'];
-        });
-        it('should resolve environment variables in arrays', () => {
-            process.env['ITEM_1'] = 'item1_env';
-            process.env['ITEM_2'] = 'item2_env';
-            const userSettingsContent = {
-                list: ['$ITEM_1', '${ITEM_2}', 'literal'],
-            };
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings['list']).toEqual([
-                'item1_env',
-                'item2_env',
-                'literal',
-            ]);
-            delete process.env['ITEM_1'];
-            delete process.env['ITEM_2'];
-        });
-        it('should correctly pass through null, boolean, and number types, and handle undefined properties', () => {
-            process.env['MY_ENV_STRING'] = 'env_string_value';
-            process.env['MY_ENV_STRING_NESTED'] = 'env_string_nested_value';
-            const userSettingsContent = {
-                nullVal: null,
-                trueVal: true,
-                falseVal: false,
-                numberVal: 123.45,
-                stringVal: '$MY_ENV_STRING',
-                nestedObj: {
-                    nestedNull: null,
-                    nestedBool: true,
-                    nestedNum: 0,
-                    nestedString: 'literal',
-                    anotherEnv: '${MY_ENV_STRING_NESTED}',
-                },
-            };
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings['nullVal']).toBeNull();
-            expect(settings.user.settings['trueVal']).toBe(true);
-            expect(settings.user.settings['falseVal']).toBe(false);
-            expect(settings.user.settings['numberVal']).toBe(123.45);
-            expect(settings.user.settings['stringVal']).toBe('env_string_value');
-            expect(settings.user.settings['undefinedVal']).toBeUndefined();
-            const nestedObj = settings.user.settings['nestedObj'];
-            expect(nestedObj['nestedNull']).toBeNull();
-            expect(nestedObj['nestedBool']).toBe(true);
-            expect(nestedObj['nestedNum']).toBe(0);
-            expect(nestedObj['nestedString']).toBe('literal');
-            expect(nestedObj['anotherEnv']).toBe('env_string_nested_value');
-            delete process.env['MY_ENV_STRING'];
-            delete process.env['MY_ENV_STRING_NESTED'];
-        });
-        it('should resolve multiple concatenated environment variables in a single string value', () => {
-            process.env['TEST_HOST'] = 'myhost';
-            process.env['TEST_PORT'] = '9090';
-            const userSettingsContent = {
-                serverAddress: '${TEST_HOST}:${TEST_PORT}/api',
-            };
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings['serverAddress']).toBe('myhost:9090/api');
-            delete process.env['TEST_HOST'];
-            delete process.env['TEST_PORT'];
-        });
-        describe('when GEMINI_CLI_SYSTEM_SETTINGS_PATH is set', () => {
-            const MOCK_ENV_SYSTEM_SETTINGS_PATH = '/mock/env/system/settings.json';
-            beforeEach(() => {
-                process.env['GEMINI_CLI_SYSTEM_SETTINGS_PATH'] =
-                    MOCK_ENV_SYSTEM_SETTINGS_PATH;
-            });
-            afterEach(() => {
-                delete process.env['GEMINI_CLI_SYSTEM_SETTINGS_PATH'];
-            });
-            it('should load system settings from the path specified in the environment variable', () => {
-                mockFsExistsSync.mockImplementation((p) => p === MOCK_ENV_SYSTEM_SETTINGS_PATH);
-                const systemSettingsContent = {
-                    ui: { theme: 'env-var-theme' },
-                    tools: { sandbox: true },
-                };
-                fs.readFileSync.mockImplementation((p) => {
-                    if (p === MOCK_ENV_SYSTEM_SETTINGS_PATH)
-                        return JSON.stringify(systemSettingsContent);
-                    return '{}';
-                });
-                const settings = loadSettings(MOCK_WORKSPACE_DIR);
-                expect(fs.readFileSync).toHaveBeenCalledWith(MOCK_ENV_SYSTEM_SETTINGS_PATH, 'utf-8');
-                expect(settings.system.path).toBe(MOCK_ENV_SYSTEM_SETTINGS_PATH);
-                expect(settings.system.settings).toEqual(systemSettingsContent);
-                expect(settings.merged).toMatchObject({
-                    ...systemSettingsContent,
-                });
-            });
-        });
-    });
-    describe('excludedProjectEnvVars integration', () => {
-        const originalEnv = { ...process.env };
-        beforeEach(() => {
-            process.env = { ...originalEnv };
-        });
-        afterEach(() => {
-            process.env = originalEnv;
-        });
-        it('should exclude DEBUG and DEBUG_MODE from project .env files by default', () => {
-            // Create a workspace settings file with excludedProjectEnvVars
-            const workspaceSettingsContent = {
-                general: {},
-                advanced: { excludedEnvVars: ['DEBUG', 'DEBUG_MODE'] },
-            };
-            mockFsExistsSync.mockImplementation((p) => p === MOCK_WORKSPACE_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            // Mock findEnvFile to return a project .env file
-            const originalFindEnvFile = loadSettings.findEnvFile;
-            loadSettings.findEnvFile =
-                () => '/mock/project/.env';
-            // Mock fs.readFileSync for .env file content
-            const originalReadFileSync = fs.readFileSync;
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === '/mock/project/.env') {
-                    return 'DEBUG=true\nDEBUG_MODE=1\nGEMINI_API_KEY=test-key';
-                }
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH) {
-                    return JSON.stringify(workspaceSettingsContent);
-                }
-                return '{}';
-            });
-            try {
-                // This will call loadEnvironment internally with the merged settings
-                const settings = loadSettings(MOCK_WORKSPACE_DIR);
-                // Verify the settings were loaded correctly
-                expect(settings.merged.advanced?.excludedEnvVars).toEqual([
-                    'DEBUG',
-                    'DEBUG_MODE',
-                ]);
-                // Note: We can't directly test process.env changes here because the mocking
-                // prevents the actual file system operations, but we can verify the settings
-                // are correctly merged and passed to loadEnvironment
-            }
-            finally {
-                loadSettings.findEnvFile =
-                    originalFindEnvFile;
-                fs.readFileSync.mockImplementation(originalReadFileSync);
-            }
-        });
-        it('should respect custom excludedProjectEnvVars from user settings', () => {
-            const userSettingsContent = {
-                general: {},
-                advanced: { excludedEnvVars: ['NODE_ENV', 'DEBUG'] },
-            };
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings.advanced?.excludedEnvVars).toEqual([
-                'NODE_ENV',
-                'DEBUG',
-            ]);
-            expect(settings.merged.advanced?.excludedEnvVars).toEqual([
-                'DEBUG',
-                'DEBUG_MODE',
-                'NODE_ENV',
-            ]);
-        });
-        it('should merge excludedProjectEnvVars with workspace taking precedence', () => {
-            const userSettingsContent = {
-                general: {},
-                advanced: { excludedEnvVars: ['DEBUG', 'NODE_ENV', 'USER_VAR'] },
-            };
-            const workspaceSettingsContent = {
-                general: {},
-                advanced: { excludedEnvVars: ['WORKSPACE_DEBUG', 'WORKSPACE_VAR'] },
-            };
-            mockFsExistsSync.mockReturnValue(true);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.user.settings.advanced?.excludedEnvVars).toEqual([
-                'DEBUG',
-                'NODE_ENV',
-                'USER_VAR',
-            ]);
-            expect(settings.workspace.settings.advanced?.excludedEnvVars).toEqual([
-                'WORKSPACE_DEBUG',
-                'WORKSPACE_VAR',
-            ]);
-            expect(settings.merged.advanced?.excludedEnvVars).toEqual([
-                'DEBUG',
-                'DEBUG_MODE',
-                'NODE_ENV',
-                'USER_VAR',
-                'WORKSPACE_DEBUG',
-                'WORKSPACE_VAR',
-            ]);
-        });
-    });
-    describe('with workspace trust', () => {
-        it('should merge workspace settings when workspace is trusted', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = {
-                ui: { theme: 'dark' },
-                tools: { sandbox: false },
-            };
-            const workspaceSettingsContent = {
-                tools: { sandbox: true },
-                context: { fileName: 'WORKSPACE.md' },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.tools?.sandbox).toBe(true);
-            expect(settings.merged.context?.fileName).toBe('WORKSPACE.md');
-            expect(settings.merged.ui?.theme).toBe('dark');
-        });
-        it('should NOT merge workspace settings when workspace is not trusted', () => {
-            vi.mocked(isWorkspaceTrusted).mockReturnValue({
-                isTrusted: false,
-                source: 'file',
-            });
-            mockFsExistsSync.mockReturnValue(true);
-            const userSettingsContent = {
-                ui: { theme: 'dark' },
-                tools: { sandbox: false },
-                context: { fileName: 'USER.md' },
-            };
-            const workspaceSettingsContent = {
-                tools: { sandbox: true },
-                context: { fileName: 'WORKSPACE.md' },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            expect(settings.merged.tools?.sandbox).toBe(false); // User setting
-            expect(settings.merged.context?.fileName).toBe('USER.md'); // User setting
-            expect(settings.merged.ui?.theme).toBe('dark'); // User setting
-        });
-    });
-    describe('loadEnvironment', () => {
-        function setup({ isFolderTrustEnabled = true, isWorkspaceTrustedValue = true, }) {
-            delete process.env['TESTTEST']; // reset
-            const geminiEnvPath = path.resolve(path.join(GEMINI_DIR, '.env'));
-            vi.mocked(isWorkspaceTrusted).mockReturnValue({
-                isTrusted: isWorkspaceTrustedValue,
-                source: 'file',
-            });
-            mockFsExistsSync.mockImplementation((p) => [USER_SETTINGS_PATH, geminiEnvPath].includes(p.toString()));
-            const userSettingsContent = {
-                ui: {
-                    theme: 'dark',
-                },
-                security: {
-                    folderTrust: {
-                        enabled: isFolderTrustEnabled,
-                    },
-                },
-                context: {
-                    fileName: 'USER_CONTEXT.md',
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === geminiEnvPath)
-                    return 'TESTTEST=1234';
-                return '{}';
-            });
-        }
-        it('sets environment variables from .env files', () => {
-            setup({ isFolderTrustEnabled: false, isWorkspaceTrustedValue: true });
-            loadEnvironment(loadSettings(MOCK_WORKSPACE_DIR).merged);
-            expect(process.env['TESTTEST']).toEqual('1234');
-        });
-        it('does not load env files from untrusted spaces', () => {
-            setup({ isFolderTrustEnabled: true, isWorkspaceTrustedValue: false });
-            loadEnvironment(loadSettings(MOCK_WORKSPACE_DIR).merged);
-            expect(process.env['TESTTEST']).not.toEqual('1234');
-        });
-    });
-    describe('migrateDeprecatedSettings', () => {
-        let mockFsExistsSync;
-        let mockFsReadFileSync;
-        beforeEach(() => {
-            vi.resetAllMocks();
-            mockFsExistsSync = vi.mocked(fs.existsSync);
-            mockFsExistsSync.mockReturnValue(true);
-            mockFsReadFileSync = vi.mocked(fs.readFileSync);
-            mockFsReadFileSync.mockReturnValue('{}');
-            vi.mocked(isWorkspaceTrusted).mockReturnValue({
-                isTrusted: true,
-                source: undefined,
-            });
-        });
-        afterEach(() => {
-            vi.restoreAllMocks();
-        });
-        it('should not do anything if there are no deprecated settings', () => {
-            const userSettingsContent = {
-                extensions: {
-                    enabled: ['user-ext-1'],
-                },
-            };
-            const workspaceSettingsContent = {
-                someOtherSetting: 'value',
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                if (p === MOCK_WORKSPACE_SETTINGS_PATH)
-                    return JSON.stringify(workspaceSettingsContent);
-                return '{}';
-            });
-            const setValueSpy = vi.spyOn(LoadedSettings.prototype, 'setValue');
-            const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR);
-            setValueSpy.mockClear();
-            migrateDeprecatedSettings(loadedSettings, true);
-            expect(setValueSpy).not.toHaveBeenCalled();
-        });
-        it('should migrate general.disableAutoUpdate to general.enableAutoUpdate with inverted value', () => {
-            const userSettingsContent = {
-                general: {
-                    disableAutoUpdate: true,
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const setValueSpy = vi.spyOn(LoadedSettings.prototype, 'setValue');
-            const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR);
-            migrateDeprecatedSettings(loadedSettings, true);
-            // Should set new value to false (inverted from true)
-            expect(setValueSpy).toHaveBeenCalledWith(SettingScope.User, 'general', expect.objectContaining({ enableAutoUpdate: false }));
-        });
-        it('should migrate all 4 inverted boolean settings', () => {
-            const userSettingsContent = {
-                general: {
-                    disableAutoUpdate: false,
-                    disableUpdateNag: true,
-                },
-                context: {
-                    fileFiltering: {
-                        disableFuzzySearch: false,
-                    },
-                },
-                ui: {
-                    accessibility: {
-                        disableLoadingPhrases: true,
-                    },
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const setValueSpy = vi.spyOn(LoadedSettings.prototype, 'setValue');
-            const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR);
-            migrateDeprecatedSettings(loadedSettings, true);
-            // Check that general settings were migrated with inverted values
-            expect(setValueSpy).toHaveBeenCalledWith(SettingScope.User, 'general', expect.objectContaining({ enableAutoUpdate: true }));
-            expect(setValueSpy).toHaveBeenCalledWith(SettingScope.User, 'general', expect.objectContaining({ enableAutoUpdateNotification: false }));
-            // Check context.fileFiltering was migrated
-            expect(setValueSpy).toHaveBeenCalledWith(SettingScope.User, 'context', expect.objectContaining({
-                fileFiltering: expect.objectContaining({ enableFuzzySearch: true }),
-            }));
-            // Check ui.accessibility was migrated
-            expect(setValueSpy).toHaveBeenCalledWith(SettingScope.User, 'ui', expect.objectContaining({
-                accessibility: expect.objectContaining({
-                    enableLoadingPhrases: false,
-                }),
-            }));
-        });
-        it('should prioritize new settings over deprecated ones and respect removeDeprecated flag', () => {
-            const userSettingsContent = {
-                general: {
-                    disableAutoUpdate: true,
-                    enableAutoUpdate: true, // Trust this (true) over disableAutoUpdate (true -> false)
-                },
-                context: {
-                    fileFiltering: {
-                        disableFuzzySearch: false,
-                        enableFuzzySearch: false, // Trust this (false) over disableFuzzySearch (false -> true)
-                    },
-                },
-            };
-            const loadedSettings = new LoadedSettings({
-                path: getSystemSettingsPath(),
-                settings: {},
-                originalSettings: {},
-            }, {
-                path: getSystemDefaultsPath(),
-                settings: {},
-                originalSettings: {},
-            }, {
-                path: USER_SETTINGS_PATH,
-                settings: userSettingsContent,
-                originalSettings: userSettingsContent,
-            }, {
-                path: MOCK_WORKSPACE_SETTINGS_PATH,
-                settings: {},
-                originalSettings: {},
-            }, true);
-            const setValueSpy = vi.spyOn(loadedSettings, 'setValue');
-            // 1. removeDeprecated = false (default)
-            migrateDeprecatedSettings(loadedSettings);
-            // Should still have old settings
-            expect(loadedSettings.forScope(SettingScope.User).settings.general).toHaveProperty('disableAutoUpdate');
-            expect(loadedSettings.forScope(SettingScope.User).settings.context.fileFiltering).toHaveProperty('disableFuzzySearch');
-            // 2. removeDeprecated = true
-            migrateDeprecatedSettings(loadedSettings, true);
-            // Should remove disableAutoUpdate and trust enableAutoUpdate: true
-            expect(setValueSpy).toHaveBeenCalledWith(SettingScope.User, 'general', {
-                enableAutoUpdate: true,
-            });
-            // Should remove disableFuzzySearch and trust enableFuzzySearch: false
-            expect(setValueSpy).toHaveBeenCalledWith(SettingScope.User, 'context', {
-                fileFiltering: { enableFuzzySearch: false },
-            });
-        });
-        it('should trigger migration automatically during loadSettings', () => {
-            mockFsExistsSync.mockImplementation((p) => p === USER_SETTINGS_PATH);
-            const userSettingsContent = {
-                general: {
-                    disableAutoUpdate: true,
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            // Verify it was migrated in the merged settings
-            expect(settings.merged.general?.enableAutoUpdate).toBe(false);
-            // Verify it was saved back to disk (via setValue calling updateSettingsFilePreservingFormat)
-            expect(updateSettingsFilePreservingFormat).toHaveBeenCalledWith(USER_SETTINGS_PATH, expect.objectContaining({
-                general: expect.objectContaining({ enableAutoUpdate: false }),
-            }));
-        });
-        it('should migrate disableUpdateNag to enableAutoUpdateNotification in system and system defaults settings', () => {
-            const systemSettingsContent = {
-                general: {
-                    disableUpdateNag: true,
-                },
-            };
-            const systemDefaultsContent = {
-                general: {
-                    disableUpdateNag: false,
-                },
-            };
-            vi.mocked(fs.existsSync).mockReturnValue(true);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath()) {
-                    return JSON.stringify(systemSettingsContent);
-                }
-                if (p === getSystemDefaultsPath()) {
-                    return JSON.stringify(systemDefaultsContent);
-                }
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            // Verify system settings were migrated
-            expect(settings.system.settings.general).toHaveProperty('enableAutoUpdateNotification');
-            expect(settings.system.settings.general['enableAutoUpdateNotification']).toBe(false);
-            // Verify system defaults settings were migrated
-            expect(settings.systemDefaults.settings.general).toHaveProperty('enableAutoUpdateNotification');
-            expect(settings.systemDefaults.settings.general['enableAutoUpdateNotification']).toBe(true);
-            // Merged should also reflect it (system overrides defaults, but both are migrated)
-            expect(settings.merged.general?.enableAutoUpdateNotification).toBe(false);
-        });
-        it('should migrate experimental agent settings to agents overrides', () => {
-            const userSettingsContent = {
-                experimental: {
-                    codebaseInvestigatorSettings: {
-                        enabled: true,
-                        maxNumTurns: 15,
-                        maxTimeMinutes: 5,
-                        thinkingBudget: 16384,
-                        model: 'gemini-1.5-pro',
-                    },
-                    cliHelpAgentSettings: {
-                        enabled: false,
-                    },
-                },
-            };
-            vi.mocked(fs.existsSync).mockReturnValue(true);
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === USER_SETTINGS_PATH)
-                    return JSON.stringify(userSettingsContent);
-                return '{}';
-            });
-            const settings = loadSettings(MOCK_WORKSPACE_DIR);
-            // Verify migration to agents.overrides
-            expect(settings.user.settings.agents?.overrides).toMatchObject({
-                codebase_investigator: {
-                    enabled: true,
-                    runConfig: {
-                        maxTurns: 15,
-                        maxTimeMinutes: 5,
-                    },
-                    modelConfig: {
-                        model: 'gemini-1.5-pro',
-                        generateContentConfig: {
-                            thinkingConfig: {
-                                thinkingBudget: 16384,
-                            },
-                        },
-                    },
-                },
-                cli_help: {
-                    enabled: false,
-                },
-            });
-        });
-    });
-    describe('saveSettings', () => {
-        it('should save settings using updateSettingsFilePreservingFormat', () => {
-            const mockUpdateSettings = vi.mocked(updateSettingsFilePreservingFormat);
-            const settingsFile = {
-                path: '/mock/settings.json',
-                settings: { ui: { theme: 'dark' } },
-                originalSettings: { ui: { theme: 'dark' } },
-            };
-            saveSettings(settingsFile);
-            expect(mockUpdateSettings).toHaveBeenCalledWith('/mock/settings.json', {
-                ui: { theme: 'dark' },
-            });
-        });
-        it('should create directory if it does not exist', () => {
-            const mockFsExistsSync = vi.mocked(fs.existsSync);
-            const mockFsMkdirSync = vi.mocked(fs.mkdirSync);
-            mockFsExistsSync.mockReturnValue(false);
-            const settingsFile = {
-                path: '/mock/new/dir/settings.json',
-                settings: {},
-                originalSettings: {},
-            };
-            saveSettings(settingsFile);
-            expect(mockFsExistsSync).toHaveBeenCalledWith('/mock/new/dir');
-            expect(mockFsMkdirSync).toHaveBeenCalledWith('/mock/new/dir', {
-                recursive: true,
-            });
-        });
-        it('should emit error feedback if saving fails', () => {
-            const mockUpdateSettings = vi.mocked(updateSettingsFilePreservingFormat);
-            const error = new Error('Write failed');
-            mockUpdateSettings.mockImplementation(() => {
-                throw error;
-            });
-            const settingsFile = {
-                path: '/mock/settings.json',
-                settings: {},
-                originalSettings: {},
-            };
-            saveSettings(settingsFile);
-            expect(mockCoreEvents.emitFeedback).toHaveBeenCalledWith('error', 'There was an error saving your latest settings changes.', error);
-        });
-    });
-    describe('LoadedSettings and remote admin settings', () => {
-        it('should prioritize remote admin settings over file-based admin settings', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const systemSettingsContent = {
-                admin: {
-                    // These should be ignored
-                    secureModeEnabled: true,
-                    mcp: { enabled: false },
-                    extensions: { enabled: false },
-                },
-                // A non-admin setting to ensure it's still processed
-                ui: { theme: 'system-theme' },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath()) {
-                    return JSON.stringify(systemSettingsContent);
-                }
-                return '{}';
-            });
-            const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR);
-            // 1. Verify that on initial load, file-based admin settings are ignored
-            //    and schema defaults are used instead.
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false); // default: false
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true); // default: true
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true); // default: true
-            expect(loadedSettings.merged.ui?.theme).toBe('system-theme'); // non-admin setting should be loaded
-            // 2. Now, set remote admin settings.
-            loadedSettings.setRemoteAdminSettings({
-                secureModeEnabled: true,
-                mcpSetting: { mcpEnabled: false },
-                cliFeatureSetting: { extensionsSetting: { extensionsEnabled: false } },
-            });
-            // 3. Verify that remote admin settings take precedence.
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(true);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(false);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(false);
-            // non-admin setting should remain unchanged
-            expect(loadedSettings.merged.ui?.theme).toBe('system-theme');
-        });
-        it('should set remote admin settings and recompute merged settings', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const systemSettingsContent = {
-                admin: {
-                    secureModeEnabled: false,
-                    mcp: { enabled: false },
-                    extensions: { enabled: false },
-                },
-                ui: { theme: 'initial-theme' },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath()) {
-                    return JSON.stringify(systemSettingsContent);
-                }
-                return '{}';
-            });
-            const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR);
-            // Ensure initial state from defaults (as file-based admin settings are ignored)
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true);
-            expect(loadedSettings.merged.ui?.theme).toBe('initial-theme');
-            const newRemoteSettings = {
-                secureModeEnabled: true,
-                mcpSetting: { mcpEnabled: false },
-                cliFeatureSetting: { extensionsSetting: { extensionsEnabled: false } },
-            };
-            loadedSettings.setRemoteAdminSettings(newRemoteSettings);
-            // Verify that remote admin settings are applied
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(true);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(false);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(false);
-            // Non-admin settings should remain untouched
-            expect(loadedSettings.merged.ui?.theme).toBe('initial-theme');
-            // Verify that calling setRemoteAdminSettings with partial data overwrites previous remote settings
-            // and missing properties revert to schema defaults.
-            loadedSettings.setRemoteAdminSettings({ secureModeEnabled: false });
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true); // Reverts to default: true
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true); // Reverts to default: true
-        });
-        it('should correctly handle undefined remote admin settings', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const systemSettingsContent = {
-                ui: { theme: 'initial-theme' },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath()) {
-                    return JSON.stringify(systemSettingsContent);
-                }
-                return '{}';
-            });
-            const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR);
-            // Should have default admin settings
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true);
-            loadedSettings.setRemoteAdminSettings({}); // Set empty remote settings
-            // Admin settings should revert to defaults because there are no remote overrides
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true);
-        });
-        it('should correctly handle missing properties in remote admin settings', () => {
-            mockFsExistsSync.mockReturnValue(true);
-            const systemSettingsContent = {
-                admin: {
-                    secureModeEnabled: true,
-                },
-            };
-            fs.readFileSync.mockImplementation((p) => {
-                if (p === getSystemSettingsPath()) {
-                    return JSON.stringify(systemSettingsContent);
-                }
-                return '{}';
-            });
-            const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR);
-            // Ensure initial state from defaults (as file-based admin settings are ignored)
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true);
-            // Set remote settings with only secureModeEnabled
-            loadedSettings.setRemoteAdminSettings({
-                secureModeEnabled: true,
-            });
-            // Verify secureModeEnabled is updated, others remain defaults
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(true);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true);
-            // Set remote settings with only mcpSetting.mcpEnabled
-            loadedSettings.setRemoteAdminSettings({
-                mcpSetting: { mcpEnabled: false },
-            });
-            // Verify mcpEnabled is updated, others remain defaults (secureModeEnabled reverts to default:false)
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(false);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(true);
-            // Set remote settings with only cliFeatureSetting.extensionsSetting.extensionsEnabled
-            loadedSettings.setRemoteAdminSettings({
-                cliFeatureSetting: { extensionsSetting: { extensionsEnabled: false } },
-            });
-            // Verify extensionsEnabled is updated, others remain defaults
-            expect(loadedSettings.merged.admin?.secureModeEnabled).toBe(false);
-            expect(loadedSettings.merged.admin?.mcp?.enabled).toBe(true);
-            expect(loadedSettings.merged.admin?.extensions?.enabled).toBe(false);
-        });
-        it('should set skills based on advancedFeaturesEnabled', () => {
-            const loadedSettings = loadSettings();
-            loadedSettings.setRemoteAdminSettings({
-                cliFeatureSetting: {
-                    advancedFeaturesEnabled: true,
-                },
-            });
-            expect(loadedSettings.merged.admin.skills?.enabled).toBe(true);
-            loadedSettings.setRemoteAdminSettings({
-                cliFeatureSetting: {
-                    advancedFeaturesEnabled: false,
-                },
-            });
-            expect(loadedSettings.merged.admin.skills?.enabled).toBe(false);
-        });
-    });
-    describe('getDefaultsFromSchema', () => {
-        it('should extract defaults from a schema', () => {
-            const mockSchema = {
-                prop1: {
-                    type: 'string',
-                    default: 'default1',
-                    label: 'Prop 1',
-                    category: 'General',
-                    requiresRestart: false,
-                },
-                nested: {
-                    type: 'object',
-                    label: 'Nested',
-                    category: 'General',
-                    requiresRestart: false,
-                    default: {},
-                    properties: {
-                        prop2: {
-                            type: 'number',
-                            default: 42,
-                            label: 'Prop 2',
-                            category: 'General',
-                            requiresRestart: false,
-                        },
-                    },
-                },
-            };
-            const defaults = getDefaultsFromSchema(mockSchema);
-            expect(defaults).toEqual({
-                prop1: 'default1',
-                nested: {
-                    prop2: 42,
-                },
-            });
-        });
-    });
-});
-//# sourceMappingURL=settings.test.js.map