osagent 0.1.40 → 0.1.42

package/dist/src/utils/settingsUtils.test.js

@@ -1,808 +0,0 @@
-/**
- * @license
- * Copyright 2025 OSAgent OC
- * SPDX-License-Identifier: Apache-2.0
- */
-import { describe, it, expect } from 'vitest';
-import { 
-// Schema utilities
-getSettingsByCategory, getSettingDefinition, requiresRestart, getDefaultValue, getRestartRequiredSettings, getEffectiveValue, getAllSettingKeys, getSettingsByType, getSettingsRequiringRestart, isValidSettingKey, getSettingCategory, shouldShowInDialog, getDialogSettingsByCategory, getDialogSettingsByType, getDialogSettingKeys, 
-// Business logic utilities
-getSettingValue, isSettingModified, TEST_ONLY, settingExistsInScope, setPendingSettingValue, hasRestartRequiredSettings, getRestartRequiredFromModified, getDisplayValue, isDefaultValue, isValueInherited, getEffectiveDisplayValue, } from './settingsUtils.js';
-import { getSettingsSchema, } from '../config/settingsSchema.js';
-vi.mock('../config/settingsSchema.js', async (importOriginal) => {
-    const original = await importOriginal();
-    return {
-        ...original,
-        getSettingsSchema: vi.fn(),
-    };
-});
-function makeMockSettings(settings) {
-    return settings;
-}
-describe('SettingsUtils', () => {
-    beforeEach(() => {
-        const SETTINGS_SCHEMA = {
-            mcpServers: {
-                type: 'object',
-                label: 'MCP Servers',
-                category: 'Advanced',
-                requiresRestart: true,
-                default: {},
-                description: 'Configuration for MCP servers.',
-                showInDialog: false,
-            },
-            test: {
-                type: 'string',
-                label: 'Test',
-                category: 'Basic',
-                requiresRestart: false,
-                default: 'hello',
-                description: 'A test field',
-                showInDialog: true,
-            },
-            advanced: {
-                type: 'object',
-                label: 'Advanced',
-                category: 'Advanced',
-                requiresRestart: true,
-                default: {},
-                description: 'Advanced settings for power users.',
-                showInDialog: false,
-            },
-            ui: {
-                type: 'object',
-                label: 'UI',
-                category: 'UI',
-                requiresRestart: false,
-                default: {},
-                description: 'User interface settings.',
-                showInDialog: false,
-                properties: {
-                    theme: {
-                        type: 'string',
-                        label: 'Theme',
-                        category: 'UI',
-                        requiresRestart: false,
-                        default: undefined,
-                        description: 'The color theme for the UI.',
-                        showInDialog: false,
-                    },
-                    requiresRestart: {
-                        type: 'boolean',
-                        label: 'Requires Restart',
-                        category: 'UI',
-                        default: false,
-                        requiresRestart: true,
-                    },
-                    accessibility: {
-                        type: 'object',
-                        label: 'Accessibility',
-                        category: 'UI',
-                        requiresRestart: true,
-                        default: {},
-                        description: 'Accessibility settings.',
-                        showInDialog: false,
-                        properties: {
-                            disableLoadingPhrases: {
-                                type: 'boolean',
-                                label: 'Disable Loading Phrases',
-                                category: 'UI',
-                                requiresRestart: true,
-                                default: false,
-                                description: 'Disable loading phrases for accessibility',
-                                showInDialog: true,
-                            },
-                        },
-                    },
-                },
-            },
-            tools: {
-                type: 'object',
-                label: 'Tools',
-                category: 'Tools',
-                requiresRestart: false,
-                default: {},
-                description: 'Tool settings.',
-                showInDialog: false,
-                properties: {
-                    shell: {
-                        type: 'object',
-                        label: 'Shell',
-                        category: 'Tools',
-                        requiresRestart: false,
-                        default: {},
-                        description: 'Shell tool settings.',
-                        showInDialog: false,
-                        properties: {
-                            pager: {
-                                type: 'string',
-                                label: 'Pager',
-                                category: 'Tools',
-                                requiresRestart: false,
-                                default: 'less',
-                                description: 'The pager to use for long output.',
-                                showInDialog: true,
-                            },
-                        },
-                    },
-                },
-            },
-        };
-        vi.mocked(getSettingsSchema).mockReturnValue(SETTINGS_SCHEMA);
-    });
-    afterEach(() => {
-        TEST_ONLY.clearFlattenedSchema();
-        vi.clearAllMocks();
-        vi.resetAllMocks();
-    });
-    describe('Schema Utilities', () => {
-        describe('getSettingsByCategory', () => {
-            it('should group settings by category', () => {
-                const categories = getSettingsByCategory();
-                expect(categories).toHaveProperty('Advanced');
-                expect(categories).toHaveProperty('Basic');
-            });
-            it('should include key property in grouped settings', () => {
-                const categories = getSettingsByCategory();
-                Object.entries(categories).forEach(([_category, settings]) => {
-                    settings.forEach((setting) => {
-                        expect(setting.key).toBeDefined();
-                    });
-                });
-            });
-        });
-        describe('getSettingDefinition', () => {
-            it('should return definition for valid setting', () => {
-                const definition = getSettingDefinition('ui.theme');
-                expect(definition).toBeDefined();
-                expect(definition?.label).toBe('Theme');
-            });
-            it('should return undefined for invalid setting', () => {
-                const definition = getSettingDefinition('invalidSetting');
-                expect(definition).toBeUndefined();
-            });
-        });
-        describe('requiresRestart', () => {
-            it('should return true for settings that require restart', () => {
-                expect(requiresRestart('ui.requiresRestart')).toBe(true);
-            });
-            it('should return false for settings that do not require restart', () => {
-                expect(requiresRestart('ui.theme')).toBe(false);
-            });
-            it('should return false for invalid settings', () => {
-                expect(requiresRestart('invalidSetting')).toBe(false);
-            });
-        });
-        describe('getDefaultValue', () => {
-            it('should return correct default values', () => {
-                expect(getDefaultValue('test')).toBe('hello');
-                expect(getDefaultValue('ui.requiresRestart')).toBe(false);
-            });
-            it('should return undefined for invalid settings', () => {
-                expect(getDefaultValue('invalidSetting')).toBeUndefined();
-            });
-        });
-        describe('getRestartRequiredSettings', () => {
-            it('should return all settings that require restart', () => {
-                const restartSettings = getRestartRequiredSettings();
-                expect(restartSettings).toContain('mcpServers');
-                expect(restartSettings).toContain('ui.requiresRestart');
-            });
-        });
-        describe('getEffectiveValue', () => {
-            it('should return value from settings when set', () => {
-                const settings = makeMockSettings({ ui: { requiresRestart: true } });
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                });
-                const value = getEffectiveValue('ui.requiresRestart', settings, mergedSettings);
-                expect(value).toBe(true);
-            });
-            it('should return value from merged settings when not set in current scope', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: true },
-                });
-                const value = getEffectiveValue('ui.requiresRestart', settings, mergedSettings);
-                expect(value).toBe(true);
-            });
-            it('should return default value when not set anywhere', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({});
-                const value = getEffectiveValue('ui.requiresRestart', settings, mergedSettings);
-                expect(value).toBe(false); // default value
-            });
-            it('should handle nested settings correctly', () => {
-                const settings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: true } },
-                });
-                const mergedSettings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: false } },
-                });
-                const value = getEffectiveValue('ui.accessibility.disableLoadingPhrases', settings, mergedSettings);
-                expect(value).toBe(true);
-            });
-            it('should return undefined for invalid settings', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({});
-                const value = getEffectiveValue('invalidSetting', settings, mergedSettings);
-                expect(value).toBeUndefined();
-            });
-        });
-        describe('getAllSettingKeys', () => {
-            it('should return all setting keys', () => {
-                const keys = getAllSettingKeys();
-                expect(keys).toContain('test');
-                expect(keys).toContain('ui.accessibility.disableLoadingPhrases');
-            });
-        });
-        describe('getSettingsByType', () => {
-            it('should return only boolean settings', () => {
-                const booleanSettings = getSettingsByType('boolean');
-                expect(booleanSettings.length).toBeGreaterThan(0);
-                booleanSettings.forEach((setting) => {
-                    expect(setting.type).toBe('boolean');
-                });
-            });
-        });
-        describe('getSettingsRequiringRestart', () => {
-            it('should return only settings that require restart', () => {
-                const restartSettings = getSettingsRequiringRestart();
-                expect(restartSettings.length).toBeGreaterThan(0);
-                restartSettings.forEach((setting) => {
-                    expect(setting.requiresRestart).toBe(true);
-                });
-            });
-        });
-        describe('isValidSettingKey', () => {
-            it('should return true for valid setting keys', () => {
-                expect(isValidSettingKey('ui.requiresRestart')).toBe(true);
-                expect(isValidSettingKey('ui.accessibility.disableLoadingPhrases')).toBe(true);
-            });
-            it('should return false for invalid setting keys', () => {
-                expect(isValidSettingKey('invalidSetting')).toBe(false);
-                expect(isValidSettingKey('')).toBe(false);
-            });
-        });
-        describe('getSettingCategory', () => {
-            it('should return correct category for valid settings', () => {
-                expect(getSettingCategory('ui.requiresRestart')).toBe('UI');
-                expect(getSettingCategory('ui.accessibility.disableLoadingPhrases')).toBe('UI');
-            });
-            it('should return undefined for invalid settings', () => {
-                expect(getSettingCategory('invalidSetting')).toBeUndefined();
-            });
-        });
-        describe('shouldShowInDialog', () => {
-            it('should return true for settings marked to show in dialog', () => {
-                expect(shouldShowInDialog('ui.requiresRestart')).toBe(true);
-                expect(shouldShowInDialog('general.vimMode')).toBe(true);
-                expect(shouldShowInDialog('ui.hideWindowTitle')).toBe(true);
-            });
-            it('should return false for settings marked to hide from dialog', () => {
-                expect(shouldShowInDialog('ui.theme')).toBe(false);
-            });
-            it('should return true for invalid settings (default behavior)', () => {
-                expect(shouldShowInDialog('invalidSetting')).toBe(true);
-            });
-        });
-        describe('getDialogSettingsByCategory', () => {
-            it('should only return settings marked for dialog display', async () => {
-                const categories = getDialogSettingsByCategory();
-                // Should include UI settings that are marked for dialog
-                expect(categories['UI']).toBeDefined();
-                const uiSettings = categories['UI'];
-                const uiKeys = uiSettings.map((s) => s.key);
-                expect(uiKeys).toContain('ui.requiresRestart');
-                expect(uiKeys).toContain('ui.accessibility.disableLoadingPhrases');
-                expect(uiKeys).not.toContain('ui.theme'); // This is now marked false
-            });
-            it('should not include Advanced category settings', () => {
-                const categories = getDialogSettingsByCategory();
-                // Advanced settings should be filtered out
-                expect(categories['Advanced']).toBeUndefined();
-            });
-            it('should include settings with showInDialog=true', () => {
-                const categories = getDialogSettingsByCategory();
-                const allSettings = Object.values(categories).flat();
-                const allKeys = allSettings.map((s) => s.key);
-                expect(allKeys).toContain('test');
-                expect(allKeys).toContain('ui.requiresRestart');
-                expect(allKeys).not.toContain('ui.theme'); // Now hidden
-                expect(allKeys).not.toContain('general.preferredEditor'); // Now hidden
-            });
-        });
-        describe('getDialogSettingsByType', () => {
-            it('should return only boolean dialog settings', () => {
-                const booleanSettings = getDialogSettingsByType('boolean');
-                const keys = booleanSettings.map((s) => s.key);
-                expect(keys).toContain('ui.requiresRestart');
-                expect(keys).toContain('ui.accessibility.disableLoadingPhrases');
-                expect(keys).not.toContain('privacy.usageStatisticsEnabled');
-                expect(keys).not.toContain('security.auth.selectedType'); // Advanced setting
-                expect(keys).not.toContain('security.auth.useExternal'); // Advanced setting
-            });
-            it('should return only string dialog settings', () => {
-                const stringSettings = getDialogSettingsByType('string');
-                const keys = stringSettings.map((s) => s.key);
-                // Note: theme and preferredEditor are now hidden from dialog
-                expect(keys).not.toContain('ui.theme'); // Now marked false
-                expect(keys).not.toContain('general.preferredEditor'); // Now marked false
-                expect(keys).not.toContain('security.auth.selectedType'); // Advanced setting
-                // Check that user-facing tool settings are included
-                expect(keys).toContain('tools.shell.pager');
-                // Check that advanced/hidden tool settings are excluded
-                expect(keys).not.toContain('tools.discoveryCommand');
-                expect(keys).not.toContain('tools.callCommand');
-                expect(keys.every((key) => !key.startsWith('advanced.'))).toBe(true);
-            });
-        });
-        describe('getDialogSettingKeys', () => {
-            it('should return only settings marked for dialog display', () => {
-                const dialogKeys = getDialogSettingKeys();
-                // Should include settings marked for dialog
-                expect(dialogKeys).toContain('ui.requiresRestart');
-                // Should include nested settings marked for dialog
-                expect(dialogKeys).toContain('ui.accessibility.disableLoadingPhrases');
-                // Should NOT include settings marked as hidden
-                expect(dialogKeys).not.toContain('ui.theme'); // Hidden
-            });
-            it('should return fewer keys than getAllSettingKeys', () => {
-                const allKeys = getAllSettingKeys();
-                const dialogKeys = getDialogSettingKeys();
-                expect(dialogKeys.length).toBeLessThan(allKeys.length);
-                expect(dialogKeys.length).toBeGreaterThan(0);
-            });
-            it('should handle nested settings display correctly', () => {
-                vi.mocked(getSettingsSchema).mockReturnValue({
-                    context: {
-                        type: 'object',
-                        label: 'Context',
-                        category: 'Context',
-                        requiresRestart: false,
-                        default: {},
-                        description: 'Settings for managing context provided to the model.',
-                        showInDialog: false,
-                        properties: {
-                            fileFiltering: {
-                                type: 'object',
-                                label: 'File Filtering',
-                                category: 'Context',
-                                requiresRestart: true,
-                                default: {},
-                                description: 'Settings for git-aware file filtering.',
-                                showInDialog: false,
-                                properties: {
-                                    respectGitIgnore: {
-                                        type: 'boolean',
-                                        label: 'Respect .gitignore',
-                                        category: 'Context',
-                                        requiresRestart: true,
-                                        default: true,
-                                        description: 'Respect .gitignore files when searching',
-                                        showInDialog: true,
-                                    },
-                                },
-                            },
-                        },
-                    },
-                });
-                // Test the specific issue with fileFiltering.respectGitIgnore
-                const key = 'context.fileFiltering.respectGitIgnore';
-                const initialSettings = makeMockSettings({});
-                const pendingSettings = makeMockSettings({});
-                // Set the nested setting to true
-                const updatedPendingSettings = setPendingSettingValue(key, true, pendingSettings);
-                // Check if the setting exists in pending settings
-                const existsInPending = settingExistsInScope(key, updatedPendingSettings);
-                expect(existsInPending).toBe(true);
-                // Get the value from pending settings
-                const valueFromPending = getSettingValue(key, updatedPendingSettings, {});
-                expect(valueFromPending).toBe(true);
-                // Test getDisplayValue should show the pending change
-                const displayValue = getDisplayValue(key, initialSettings, {}, new Set(), updatedPendingSettings);
-                expect(displayValue).toBe('true'); // Should show true (no * since value matches default)
-                // Test that modified settings also show the * indicator
-                const modifiedSettings = new Set([key]);
-                const displayValueWithModified = getDisplayValue(key, initialSettings, {}, modifiedSettings, {});
-                expect(displayValueWithModified).toBe('true*'); // Should show true* because it's in modified settings and default is true
-            });
-        });
-    });
-    describe('Business Logic Utilities', () => {
-        describe('getSettingValue', () => {
-            it('should return value from settings when set', () => {
-                const settings = makeMockSettings({ ui: { requiresRestart: true } });
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                });
-                const value = getSettingValue('ui.requiresRestart', settings, mergedSettings);
-                expect(value).toBe(true);
-            });
-            it('should return value from merged settings when not set in current scope', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: true },
-                });
-                const value = getSettingValue('ui.requiresRestart', settings, mergedSettings);
-                expect(value).toBe(true);
-            });
-            it('should return default value for invalid setting', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({});
-                const value = getSettingValue('invalidSetting', settings, mergedSettings);
-                expect(value).toBe(false); // Default fallback
-            });
-        });
-        describe('isSettingModified', () => {
-            it('should return true when value differs from default', () => {
-                expect(isSettingModified('ui.requiresRestart', true)).toBe(true);
-                expect(isSettingModified('ui.accessibility.disableLoadingPhrases', true)).toBe(true);
-            });
-            it('should return false when value matches default', () => {
-                expect(isSettingModified('ui.requiresRestart', false)).toBe(false);
-                expect(isSettingModified('ui.accessibility.disableLoadingPhrases', false)).toBe(false);
-            });
-        });
-        describe('settingExistsInScope', () => {
-            it('should return true for top-level settings that exist', () => {
-                const settings = makeMockSettings({ ui: { requiresRestart: true } });
-                expect(settingExistsInScope('ui.requiresRestart', settings)).toBe(true);
-            });
-            it('should return false for top-level settings that do not exist', () => {
-                const settings = makeMockSettings({});
-                expect(settingExistsInScope('ui.requiresRestart', settings)).toBe(false);
-            });
-            it('should return true for nested settings that exist', () => {
-                const settings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: true } },
-                });
-                expect(settingExistsInScope('ui.accessibility.disableLoadingPhrases', settings)).toBe(true);
-            });
-            it('should return false for nested settings that do not exist', () => {
-                const settings = makeMockSettings({});
-                expect(settingExistsInScope('ui.accessibility.disableLoadingPhrases', settings)).toBe(false);
-            });
-            it('should return false when parent exists but child does not', () => {
-                const settings = makeMockSettings({ ui: { accessibility: {} } });
-                expect(settingExistsInScope('ui.accessibility.disableLoadingPhrases', settings)).toBe(false);
-            });
-        });
-        describe('setPendingSettingValue', () => {
-            it('should set top-level setting value', () => {
-                const pendingSettings = makeMockSettings({});
-                const result = setPendingSettingValue('ui.hideWindowTitle', true, pendingSettings);
-                expect(result.ui?.hideWindowTitle).toBe(true);
-            });
-            it('should set nested setting value', () => {
-                const pendingSettings = makeMockSettings({});
-                const result = setPendingSettingValue('ui.accessibility.disableLoadingPhrases', true, pendingSettings);
-                expect(result.ui?.accessibility?.disableLoadingPhrases).toBe(true);
-            });
-            it('should preserve existing nested settings', () => {
-                const pendingSettings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: false } },
-                });
-                const result = setPendingSettingValue('ui.accessibility.disableLoadingPhrases', true, pendingSettings);
-                expect(result.ui?.accessibility?.disableLoadingPhrases).toBe(true);
-            });
-            it('should not mutate original settings', () => {
-                const pendingSettings = makeMockSettings({});
-                setPendingSettingValue('ui.requiresRestart', true, pendingSettings);
-                expect(pendingSettings).toEqual({});
-            });
-        });
-        describe('hasRestartRequiredSettings', () => {
-            it('should return true when modified settings require restart', () => {
-                const modifiedSettings = new Set([
-                    'advanced.autoConfigureMemory',
-                    'ui.requiresRestart',
-                ]);
-                expect(hasRestartRequiredSettings(modifiedSettings)).toBe(true);
-            });
-            it('should return false when no modified settings require restart', () => {
-                const modifiedSettings = new Set(['test']);
-                expect(hasRestartRequiredSettings(modifiedSettings)).toBe(false);
-            });
-            it('should return false for empty set', () => {
-                const modifiedSettings = new Set();
-                expect(hasRestartRequiredSettings(modifiedSettings)).toBe(false);
-            });
-        });
-        describe('getRestartRequiredFromModified', () => {
-            it('should return only settings that require restart', () => {
-                const modifiedSettings = new Set([
-                    'ui.requiresRestart',
-                    'test',
-                ]);
-                const result = getRestartRequiredFromModified(modifiedSettings);
-                expect(result).toContain('ui.requiresRestart');
-                expect(result).not.toContain('test');
-            });
-            it('should return empty array when no settings require restart', () => {
-                const modifiedSettings = new Set([
-                    'requiresRestart',
-                    'hideTips',
-                ]);
-                const result = getRestartRequiredFromModified(modifiedSettings);
-                expect(result).toEqual([]);
-            });
-        });
-        describe('getDisplayValue', () => {
-            describe('enum behavior', () => {
-                let StringEnum;
-                (function (StringEnum) {
-                    StringEnum["FOO"] = "foo";
-                    StringEnum["BAR"] = "bar";
-                    StringEnum["BAZ"] = "baz";
-                })(StringEnum || (StringEnum = {}));
-                let NumberEnum;
-                (function (NumberEnum) {
-                    NumberEnum[NumberEnum["ONE"] = 1] = "ONE";
-                    NumberEnum[NumberEnum["TWO"] = 2] = "TWO";
-                    NumberEnum[NumberEnum["THREE"] = 3] = "THREE";
-                })(NumberEnum || (NumberEnum = {}));
-                const SETTING = {
-                    type: 'enum',
-                    label: 'Theme',
-                    options: [
-                        {
-                            value: StringEnum.FOO,
-                            label: 'Foo',
-                        },
-                        {
-                            value: StringEnum.BAR,
-                            label: 'Bar',
-                        },
-                        {
-                            value: StringEnum.BAZ,
-                            label: 'Baz',
-                        },
-                    ],
-                    category: 'UI',
-                    requiresRestart: false,
-                    default: StringEnum.BAR,
-                    description: 'The color theme for the UI.',
-                    showInDialog: false,
-                };
-                it('handles display of number-based enums', () => {
-                    vi.mocked(getSettingsSchema).mockReturnValue({
-                        ui: {
-                            properties: {
-                                theme: {
-                                    ...SETTING,
-                                    options: [
-                                        {
-                                            value: NumberEnum.ONE,
-                                            label: 'One',
-                                        },
-                                        {
-                                            value: NumberEnum.TWO,
-                                            label: 'Two',
-                                        },
-                                        {
-                                            value: NumberEnum.THREE,
-                                            label: 'Three',
-                                        },
-                                    ],
-                                },
-                            },
-                        },
-                    });
-                    const settings = makeMockSettings({
-                        ui: { theme: NumberEnum.THREE },
-                    });
-                    const mergedSettings = makeMockSettings({
-                        ui: { theme: NumberEnum.THREE },
-                    });
-                    const modifiedSettings = new Set();
-                    const result = getDisplayValue('ui.theme', settings, mergedSettings, modifiedSettings);
-                    expect(result).toBe('Three*');
-                });
-                it('handles default values for number-based enums', () => {
-                    vi.mocked(getSettingsSchema).mockReturnValue({
-                        ui: {
-                            properties: {
-                                theme: {
-                                    ...SETTING,
-                                    default: NumberEnum.THREE,
-                                    options: [
-                                        {
-                                            value: NumberEnum.ONE,
-                                            label: 'One',
-                                        },
-                                        {
-                                            value: NumberEnum.TWO,
-                                            label: 'Two',
-                                        },
-                                        {
-                                            value: NumberEnum.THREE,
-                                            label: 'Three',
-                                        },
-                                    ],
-                                },
-                            },
-                        },
-                    });
-                    const modifiedSettings = new Set();
-                    const result = getDisplayValue('ui.theme', makeMockSettings({}), makeMockSettings({}), modifiedSettings);
-                    expect(result).toBe('Three');
-                });
-                it('shows the enum display value', () => {
-                    vi.mocked(getSettingsSchema).mockReturnValue({
-                        ui: { properties: { theme: { ...SETTING } } },
-                    });
-                    const settings = makeMockSettings({ ui: { theme: StringEnum.BAR } });
-                    const mergedSettings = makeMockSettings({
-                        ui: { theme: StringEnum.BAR },
-                    });
-                    const modifiedSettings = new Set();
-                    const result = getDisplayValue('ui.theme', settings, mergedSettings, modifiedSettings);
-                    expect(result).toBe('Bar*');
-                });
-                it('passes through unknown values verbatim', () => {
-                    vi.mocked(getSettingsSchema).mockReturnValue({
-                        ui: {
-                            properties: {
-                                theme: { ...SETTING },
-                            },
-                        },
-                    });
-                    const settings = makeMockSettings({ ui: { theme: 'xyz' } });
-                    const mergedSettings = makeMockSettings({ ui: { theme: 'xyz' } });
-                    const modifiedSettings = new Set();
-                    const result = getDisplayValue('ui.theme', settings, mergedSettings, modifiedSettings);
-                    expect(result).toBe('xyz*');
-                });
-                it('shows the default value for string enums', () => {
-                    vi.mocked(getSettingsSchema).mockReturnValue({
-                        ui: {
-                            properties: {
-                                theme: { ...SETTING, default: StringEnum.BAR },
-                            },
-                        },
-                    });
-                    const modifiedSettings = new Set();
-                    const result = getDisplayValue('ui.theme', makeMockSettings({}), makeMockSettings({}), modifiedSettings);
-                    expect(result).toBe('Bar');
-                });
-            });
-            it('should show value without * when setting matches default', () => {
-                const settings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                }); // false matches default, so no *
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                });
-                const modifiedSettings = new Set();
-                const result = getDisplayValue('ui.requiresRestart', settings, mergedSettings, modifiedSettings);
-                expect(result).toBe('false*');
-            });
-            it('should show default value when setting is not in scope', () => {
-                const settings = makeMockSettings({}); // no setting in scope
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                });
-                const modifiedSettings = new Set();
-                const result = getDisplayValue('ui.requiresRestart', settings, mergedSettings, modifiedSettings);
-                expect(result).toBe('false'); // shows default value
-            });
-            it('should show value with * when changed from default', () => {
-                const settings = makeMockSettings({ ui: { requiresRestart: true } }); // true is different from default (false)
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: true },
-                });
-                const modifiedSettings = new Set();
-                const result = getDisplayValue('ui.requiresRestart', settings, mergedSettings, modifiedSettings);
-                expect(result).toBe('true*');
-            });
-            it('should show default value without * when setting does not exist in scope', () => {
-                const settings = makeMockSettings({}); // setting doesn't exist in scope, show default
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                });
-                const modifiedSettings = new Set();
-                const result = getDisplayValue('ui.requiresRestart', settings, mergedSettings, modifiedSettings);
-                expect(result).toBe('false'); // default value (false) without *
-            });
-            it('should show value with * when user changes from default', () => {
-                const settings = makeMockSettings({}); // setting doesn't exist in scope originally
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                });
-                const modifiedSettings = new Set(['ui.requiresRestart']);
-                const pendingSettings = makeMockSettings({
-                    ui: { requiresRestart: true },
-                }); // user changed to true
-                const result = getDisplayValue('ui.requiresRestart', settings, mergedSettings, modifiedSettings, pendingSettings);
-                expect(result).toBe('true*'); // changed from default (false) to true
-            });
-        });
-        describe('isDefaultValue', () => {
-            it('should return true when setting does not exist in scope', () => {
-                const settings = makeMockSettings({}); // setting doesn't exist
-                const result = isDefaultValue('ui.requiresRestart', settings);
-                expect(result).toBe(true);
-            });
-            it('should return false when setting exists in scope', () => {
-                const settings = makeMockSettings({ ui: { requiresRestart: true } }); // setting exists
-                const result = isDefaultValue('ui.requiresRestart', settings);
-                expect(result).toBe(false);
-            });
-            it('should return true when nested setting does not exist in scope', () => {
-                const settings = makeMockSettings({}); // nested setting doesn't exist
-                const result = isDefaultValue('ui.accessibility.disableLoadingPhrases', settings);
-                expect(result).toBe(true);
-            });
-            it('should return false when nested setting exists in scope', () => {
-                const settings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: true } },
-                }); // nested setting exists
-                const result = isDefaultValue('ui.accessibility.disableLoadingPhrases', settings);
-                expect(result).toBe(false);
-            });
-        });
-        describe('isValueInherited', () => {
-            it('should return false for top-level settings that exist in scope', () => {
-                const settings = makeMockSettings({ ui: { requiresRestart: true } });
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: true },
-                });
-                const result = isValueInherited('ui.requiresRestart', settings, mergedSettings);
-                expect(result).toBe(false);
-            });
-            it('should return true for top-level settings that do not exist in scope', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: true },
-                });
-                const result = isValueInherited('ui.requiresRestart', settings, mergedSettings);
-                expect(result).toBe(true);
-            });
-            it('should return false for nested settings that exist in scope', () => {
-                const settings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: true } },
-                });
-                const mergedSettings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: true } },
-                });
-                const result = isValueInherited('ui.accessibility.disableLoadingPhrases', settings, mergedSettings);
-                expect(result).toBe(false);
-            });
-            it('should return true for nested settings that do not exist in scope', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({
-                    ui: { accessibility: { disableLoadingPhrases: true } },
-                });
-                const result = isValueInherited('ui.accessibility.disableLoadingPhrases', settings, mergedSettings);
-                expect(result).toBe(true);
-            });
-        });
-        describe('getEffectiveDisplayValue', () => {
-            it('should return value from settings when available', () => {
-                const settings = makeMockSettings({ ui: { requiresRestart: true } });
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: false },
-                });
-                const result = getEffectiveDisplayValue('ui.requiresRestart', settings, mergedSettings);
-                expect(result).toBe(true);
-            });
-            it('should return value from merged settings when not in scope', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({
-                    ui: { requiresRestart: true },
-                });
-                const result = getEffectiveDisplayValue('ui.requiresRestart', settings, mergedSettings);
-                expect(result).toBe(true);
-            });
-            it('should return default value for undefined values', () => {
-                const settings = makeMockSettings({});
-                const mergedSettings = makeMockSettings({});
-                const result = getEffectiveDisplayValue('ui.requiresRestart', settings, mergedSettings);
-                expect(result).toBe(false); // Default value
-            });
-        });
-    });
-});
-//# sourceMappingURL=settingsUtils.test.js.map