@google/gemini-cli 0.3.0-nightly.20250822.15c62bad → 0.3.0-preview.1

package/dist/src/config/settings.d.ts

@@ -3,18 +3,20 @@
  * Copyright 2025 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-import { Settings, MemoryImportFormat } from './settingsSchema.js';
+import type { Settings, MemoryImportFormat } from './settingsSchema.js';
 export type { Settings, MemoryImportFormat };
 export declare const SETTINGS_DIRECTORY_NAME = ".gemini";
 export declare const USER_SETTINGS_PATH: string;
 export declare const USER_SETTINGS_DIR: string;
 export declare const DEFAULT_EXCLUDED_ENV_VARS: string[];
 export declare function getSystemSettingsPath(): string;
+export declare function getSystemDefaultsPath(): string;
 export type { DnsResolutionOrder } from './settingsSchema.js';
 export declare enum SettingScope {
     User = "User",
     Workspace = "Workspace",
-    System = "System"
+    System = "System",
+    SystemDefaults = "SystemDefaults"
 }
 export interface CheckpointingSettings {
     enabled?: boolean;
@@ -34,18 +36,21 @@ export interface SettingsFile {
     settings: Settings;
     path: string;
 }
+export declare function migrateSettingsToV1(v2Settings: Record<string, unknown>): Record<string, unknown>;
 export declare class LoadedSettings {
-    constructor(system: SettingsFile, user: SettingsFile, workspace: SettingsFile, errors: SettingsError[], isTrusted: boolean);
+    constructor(system: SettingsFile, systemDefaults: SettingsFile, user: SettingsFile, workspace: SettingsFile, errors: SettingsError[], isTrusted: boolean, migratedInMemorScopes: Set<SettingScope>);
     readonly system: SettingsFile;
+    readonly systemDefaults: SettingsFile;
     readonly user: SettingsFile;
     readonly workspace: SettingsFile;
     readonly errors: SettingsError[];
     readonly isTrusted: boolean;
+    readonly migratedInMemorScopes: Set<SettingScope>;
     private _merged;
     get merged(): Settings;
     private computeMergedSettings;
     forScope(scope: SettingScope): SettingsFile;
-    setValue<K extends keyof Settings>(scope: SettingScope, key: K, value: Settings[K]): void;
+    setValue(scope: SettingScope, key: string, value: unknown): void;
 }
 export declare function setUpCloudShellEnvironment(envFilePath: string | null): void;
 export declare function loadEnvironment(settings?: Settings): void;

package/dist/src/config/settings.js

@@ -3,19 +3,70 @@
  * Copyright 2025 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-import * as fs from 'fs';
-import * as path from 'path';
-import { homedir, platform } from 'os';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import { homedir, platform } from 'node:os';
 import * as dotenv from 'dotenv';
 import { GEMINI_CONFIG_DIR as GEMINI_DIR, getErrorMessage, Storage, } from '@google/gemini-cli-core';
 import stripJsonComments from 'strip-json-comments';
 import { DefaultLight } from '../ui/themes/default-light.js';
 import { DefaultDark } from '../ui/themes/default.js';
 import { isWorkspaceTrusted } from './trustedFolders.js';
+import { mergeWith } from 'lodash-es';
 export const SETTINGS_DIRECTORY_NAME = '.gemini';
 export const USER_SETTINGS_PATH = Storage.getGlobalSettingsPath();
 export const USER_SETTINGS_DIR = path.dirname(USER_SETTINGS_PATH);
 export const DEFAULT_EXCLUDED_ENV_VARS = ['DEBUG', 'DEBUG_MODE'];
+const MIGRATE_V2_OVERWRITE = false;
+// As defined in spec.md
+const MIGRATION_MAP = {
+    preferredEditor: 'general.preferredEditor',
+    vimMode: 'general.vimMode',
+    disableAutoUpdate: 'general.disableAutoUpdate',
+    disableUpdateNag: 'general.disableUpdateNag',
+    checkpointing: 'general.checkpointing',
+    theme: 'ui.theme',
+    customThemes: 'ui.customThemes',
+    hideWindowTitle: 'ui.hideWindowTitle',
+    hideTips: 'ui.hideTips',
+    hideBanner: 'ui.hideBanner',
+    hideFooter: 'ui.hideFooter',
+    showMemoryUsage: 'ui.showMemoryUsage',
+    showLineNumbers: 'ui.showLineNumbers',
+    accessibility: 'ui.accessibility',
+    ideMode: 'ide.enabled',
+    hasSeenIdeIntegrationNudge: 'ide.hasSeenNudge',
+    usageStatisticsEnabled: 'privacy.usageStatisticsEnabled',
+    telemetry: 'telemetry',
+    model: 'model.name',
+    maxSessionTurns: 'model.maxSessionTurns',
+    summarizeToolOutput: 'model.summarizeToolOutput',
+    chatCompression: 'model.chatCompression',
+    skipNextSpeakerCheck: 'model.skipNextSpeakerCheck',
+    contextFileName: 'context.fileName',
+    memoryImportFormat: 'context.importFormat',
+    memoryDiscoveryMaxDirs: 'context.discoveryMaxDirs',
+    includeDirectories: 'context.includeDirectories',
+    loadMemoryFromIncludeDirectories: 'context.loadFromIncludeDirectories',
+    fileFiltering: 'context.fileFiltering',
+    sandbox: 'tools.sandbox',
+    shouldUseNodePtyShell: 'tools.usePty',
+    coreTools: 'tools.core',
+    excludeTools: 'tools.exclude',
+    toolDiscoveryCommand: 'tools.discoveryCommand',
+    toolCallCommand: 'tools.callCommand',
+    mcpServerCommand: 'mcp.serverCommand',
+    allowMCPServers: 'mcp.allowed',
+    excludeMCPServers: 'mcp.excluded',
+    folderTrustFeature: 'security.folderTrust.featureEnabled',
+    folderTrust: 'security.folderTrust.enabled',
+    selectedAuthType: 'security.auth.selectedType',
+    useExternalAuth: 'security.auth.useExternal',
+    autoConfigureMaxOldSpaceSize: 'advanced.autoConfigureMemory',
+    dnsResolutionOrder: 'advanced.dnsResolutionOrder',
+    excludedProjectEnvVars: 'advanced.excludedEnvVars',
+    bugCommand: 'advanced.bugCommand',
+};
 export function getSystemSettingsPath() {
     if (process.env['GEMINI_CLI_SYSTEM_SETTINGS_PATH']) {
         return process.env['GEMINI_CLI_SYSTEM_SETTINGS_PATH'];
@@ -30,63 +81,258 @@ export function getSystemSettingsPath() {
         return '/etc/gemini-cli/settings.json';
     }
 }
+export function getSystemDefaultsPath() {
+    if (process.env['GEMINI_CLI_SYSTEM_DEFAULTS_PATH']) {
+        return process.env['GEMINI_CLI_SYSTEM_DEFAULTS_PATH'];
+    }
+    return path.join(path.dirname(getSystemSettingsPath()), 'system-defaults.json');
+}
 export var SettingScope;
 (function (SettingScope) {
     SettingScope["User"] = "User";
     SettingScope["Workspace"] = "Workspace";
     SettingScope["System"] = "System";
+    SettingScope["SystemDefaults"] = "SystemDefaults";
 })(SettingScope || (SettingScope = {}));
-function mergeSettings(system, user, workspace, isTrusted) {
+function setNestedProperty(obj, path, value) {
+    const keys = path.split('.');
+    const lastKey = keys.pop();
+    if (!lastKey)
+        return;
+    let current = obj;
+    for (const key of keys) {
+        if (current[key] === undefined) {
+            current[key] = {};
+        }
+        const next = current[key];
+        if (typeof next === 'object' && next !== null) {
+            current = next;
+        }
+        else {
+            // This path is invalid, so we stop.
+            return;
+        }
+    }
+    current[lastKey] = value;
+}
+function needsMigration(settings) {
+    return !('general' in settings);
+}
+function migrateSettingsToV2(flatSettings) {
+    if (!needsMigration(flatSettings)) {
+        return null;
+    }
+    const v2Settings = {};
+    const flatKeys = new Set(Object.keys(flatSettings));
+    for (const [oldKey, newPath] of Object.entries(MIGRATION_MAP)) {
+        if (flatKeys.has(oldKey)) {
+            setNestedProperty(v2Settings, newPath, flatSettings[oldKey]);
+            flatKeys.delete(oldKey);
+        }
+    }
+    // Preserve mcpServers at the top level
+    if (flatSettings['mcpServers']) {
+        v2Settings['mcpServers'] = flatSettings['mcpServers'];
+        flatKeys.delete('mcpServers');
+    }
+    // Carry over any unrecognized keys
+    for (const remainingKey of flatKeys) {
+        v2Settings[remainingKey] = flatSettings[remainingKey];
+    }
+    return v2Settings;
+}
+function getNestedProperty(obj, path) {
+    const keys = path.split('.');
+    let current = obj;
+    for (const key of keys) {
+        if (typeof current !== 'object' || current === null || !(key in current)) {
+            return undefined;
+        }
+        current = current[key];
+    }
+    return current;
+}
+const REVERSE_MIGRATION_MAP = Object.fromEntries(Object.entries(MIGRATION_MAP).map(([key, value]) => [value, key]));
+// Dynamically determine the top-level keys from the V2 settings structure.
+const KNOWN_V2_CONTAINERS = new Set(Object.values(MIGRATION_MAP).map((path) => path.split('.')[0]));
+export function migrateSettingsToV1(v2Settings) {
+    const v1Settings = {};
+    const v2Keys = new Set(Object.keys(v2Settings));
+    for (const [newPath, oldKey] of Object.entries(REVERSE_MIGRATION_MAP)) {
+        const value = getNestedProperty(v2Settings, newPath);
+        if (value !== undefined) {
+            v1Settings[oldKey] = value;
+            v2Keys.delete(newPath.split('.')[0]);
+        }
+    }
+    // Preserve mcpServers at the top level
+    if (v2Settings['mcpServers']) {
+        v1Settings['mcpServers'] = v2Settings['mcpServers'];
+        v2Keys.delete('mcpServers');
+    }
+    // Carry over any unrecognized keys
+    for (const remainingKey of v2Keys) {
+        const value = v2Settings[remainingKey];
+        if (value === undefined) {
+            continue;
+        }
+        // Don't carry over empty objects that were just containers for migrated settings.
+        if (KNOWN_V2_CONTAINERS.has(remainingKey) &&
+            typeof value === 'object' &&
+            value !== null &&
+            !Array.isArray(value) &&
+            Object.keys(value).length === 0) {
+            continue;
+        }
+        v1Settings[remainingKey] = value;
+    }
+    return v1Settings;
+}
+function mergeSettings(system, systemDefaults, user, workspace, isTrusted) {
     const safeWorkspace = isTrusted ? workspace : {};
     // folderTrust is not supported at workspace level.
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    const { folderTrust, ...safeWorkspaceWithoutFolderTrust } = safeWorkspace;
+    const { security, ...restOfWorkspace } = safeWorkspace;
+    const safeWorkspaceWithoutFolderTrust = security
+        ? {
+            ...restOfWorkspace,
+            // eslint-disable-next-line @typescript-eslint/no-unused-vars
+            security: (({ folderTrust, ...rest }) => rest)(security),
+        }
+        : {
+            ...restOfWorkspace,
+            security: {},
+        };
+    // Settings are merged with the following precedence (last one wins for
+    // single values):
+    // 1. System Defaults
+    // 2. User Settings
+    // 3. Workspace Settings
+    // 4. System Settings (as overrides)
+    //
+    // For properties that are arrays (e.g., includeDirectories), the arrays
+    // are concatenated. For objects (e.g., customThemes), they are merged.
     return {
+        ...systemDefaults,
         ...user,
         ...safeWorkspaceWithoutFolderTrust,
         ...system,
-        customThemes: {
-            ...(user.customThemes || {}),
-            ...(safeWorkspace.customThemes || {}),
-            ...(system.customThemes || {}),
+        ui: {
+            ...(systemDefaults.ui || {}),
+            ...(user.ui || {}),
+            ...(safeWorkspaceWithoutFolderTrust.ui || {}),
+            ...(system.ui || {}),
+            customThemes: {
+                ...(systemDefaults.ui?.customThemes || {}),
+                ...(user.ui?.customThemes || {}),
+                ...(safeWorkspaceWithoutFolderTrust.ui?.customThemes || {}),
+                ...(system.ui?.customThemes || {}),
+            },
+        },
+        security: {
+            ...(systemDefaults.security || {}),
+            ...(user.security || {}),
+            ...(safeWorkspaceWithoutFolderTrust.security || {}),
+            ...(system.security || {}),
+        },
+        mcp: {
+            ...(systemDefaults.mcp || {}),
+            ...(user.mcp || {}),
+            ...(safeWorkspaceWithoutFolderTrust.mcp || {}),
+            ...(system.mcp || {}),
         },
         mcpServers: {
+            ...(systemDefaults.mcpServers || {}),
             ...(user.mcpServers || {}),
-            ...(safeWorkspace.mcpServers || {}),
+            ...(safeWorkspaceWithoutFolderTrust.mcpServers || {}),
             ...(system.mcpServers || {}),
         },
-        includeDirectories: [
-            ...(system.includeDirectories || []),
-            ...(user.includeDirectories || []),
-            ...(safeWorkspace.includeDirectories || []),
-        ],
-        chatCompression: {
-            ...(system.chatCompression || {}),
-            ...(user.chatCompression || {}),
-            ...(safeWorkspace.chatCompression || {}),
+        context: {
+            ...(systemDefaults.context || {}),
+            ...(user.context || {}),
+            ...(safeWorkspaceWithoutFolderTrust.context || {}),
+            ...(system.context || {}),
+            includeDirectories: [
+                ...(systemDefaults.context?.includeDirectories || []),
+                ...(user.context?.includeDirectories || []),
+                ...(safeWorkspaceWithoutFolderTrust.context?.includeDirectories || []),
+                ...(system.context?.includeDirectories || []),
+            ],
+        },
+        model: {
+            ...(systemDefaults.model || {}),
+            ...(user.model || {}),
+            ...(safeWorkspaceWithoutFolderTrust.model || {}),
+            ...(system.model || {}),
+            chatCompression: {
+                ...(systemDefaults.model?.chatCompression || {}),
+                ...(user.model?.chatCompression || {}),
+                ...(safeWorkspaceWithoutFolderTrust.model?.chatCompression || {}),
+                ...(system.model?.chatCompression || {}),
+            },
+        },
+        advanced: {
+            ...(systemDefaults.advanced || {}),
+            ...(user.advanced || {}),
+            ...(safeWorkspaceWithoutFolderTrust.advanced || {}),
+            ...(system.advanced || {}),
+            excludedEnvVars: [
+                ...new Set([
+                    ...(systemDefaults.advanced?.excludedEnvVars || []),
+                    ...(user.advanced?.excludedEnvVars || []),
+                    ...(safeWorkspaceWithoutFolderTrust.advanced?.excludedEnvVars || []),
+                    ...(system.advanced?.excludedEnvVars || []),
+                ]),
+            ],
+        },
+        extensions: {
+            ...(systemDefaults.extensions || {}),
+            ...(user.extensions || {}),
+            ...(safeWorkspaceWithoutFolderTrust.extensions || {}),
+            ...(system.extensions || {}),
+            disabled: [
+                ...new Set([
+                    ...(systemDefaults.extensions?.disabled || []),
+                    ...(user.extensions?.disabled || []),
+                    ...(safeWorkspaceWithoutFolderTrust.extensions?.disabled || []),
+                    ...(system.extensions?.disabled || []),
+                ]),
+            ],
+            workspacesWithMigrationNudge: [
+                ...new Set([
+                    ...(systemDefaults.extensions?.workspacesWithMigrationNudge || []),
+                    ...(user.extensions?.workspacesWithMigrationNudge || []),
+                    ...(safeWorkspaceWithoutFolderTrust.extensions
+                        ?.workspacesWithMigrationNudge || []),
+                    ...(system.extensions?.workspacesWithMigrationNudge || []),
+                ]),
+            ],
         },
     };
 }
 export class LoadedSettings {
-    constructor(system, user, workspace, errors, isTrusted) {
+    constructor(system, systemDefaults, user, workspace, errors, isTrusted, migratedInMemorScopes) {
         this.system = system;
+        this.systemDefaults = systemDefaults;
         this.user = user;
         this.workspace = workspace;
         this.errors = errors;
         this.isTrusted = isTrusted;
+        this.migratedInMemorScopes = migratedInMemorScopes;
         this._merged = this.computeMergedSettings();
     }
     system;
+    systemDefaults;
     user;
     workspace;
     errors;
     isTrusted;
+    migratedInMemorScopes;
     _merged;
     get merged() {
         return this._merged;
     }
     computeMergedSettings() {
-        return mergeSettings(this.system.settings, this.user.settings, this.workspace.settings, this.isTrusted);
+        return mergeSettings(this.system.settings, this.systemDefaults.settings, this.user.settings, this.workspace.settings, this.isTrusted);
     }
     forScope(scope) {
         switch (scope) {
@@ -96,13 +342,15 @@ export class LoadedSettings {
                 return this.workspace;
             case SettingScope.System:
                 return this.system;
+            case SettingScope.SystemDefaults:
+                return this.systemDefaults;
             default:
                 throw new Error(`Invalid scope: ${scope}`);
         }
     }
     setValue(scope, key, value) {
         const settingsFile = this.forScope(scope);
-        settingsFile.settings[key] = value;
+        setNestedProperty(settingsFile.settings, key, value);
         this._merged = this.computeMergedSettings();
         saveSettings(settingsFile);
     }
@@ -219,7 +467,8 @@ export function loadEnvironment(settings) {
         try {
             const envFileContent = fs.readFileSync(envFilePath, 'utf-8');
             const parsedEnv = dotenv.parse(envFileContent);
-            const excludedVars = resolvedSettings?.excludedProjectEnvVars || DEFAULT_EXCLUDED_ENV_VARS;
+            const excludedVars = resolvedSettings?.advanced?.excludedEnvVars ||
+                DEFAULT_EXCLUDED_ENV_VARS;
             const isProjectEnvFile = !envFilePath.includes(GEMINI_DIR);
             for (const key in parsedEnv) {
                 if (Object.hasOwn(parsedEnv, key)) {
@@ -245,10 +494,13 @@ export function loadEnvironment(settings) {
  */
 export function loadSettings(workspaceDir) {
     let systemSettings = {};
+    let systemDefaultSettings = {};
     let userSettings = {};
     let workspaceSettings = {};
     const settingsErrors = [];
     const systemSettingsPath = getSystemSettingsPath();
+    const systemDefaultsPath = getSystemDefaultsPath();
+    const migratedInMemorScopes = new Set();
     // Resolve paths to their canonical representation to handle symlinks
     const resolvedWorkspaceDir = path.resolve(workspaceDir);
     const resolvedHomeDir = path.resolve(homedir());
@@ -263,66 +515,77 @@ export function loadSettings(workspaceDir) {
     // We expect homedir to always exist and be resolvable.
     const realHomeDir = fs.realpathSync(resolvedHomeDir);
     const workspaceSettingsPath = new Storage(workspaceDir).getWorkspaceSettingsPath();
-    // Load system settings
-    try {
-        if (fs.existsSync(systemSettingsPath)) {
-            const systemContent = fs.readFileSync(systemSettingsPath, 'utf-8');
-            systemSettings = JSON.parse(stripJsonComments(systemContent));
-        }
-    }
-    catch (error) {
-        settingsErrors.push({
-            message: getErrorMessage(error),
-            path: systemSettingsPath,
-        });
-    }
-    // Load user settings
-    try {
-        if (fs.existsSync(USER_SETTINGS_PATH)) {
-            const userContent = fs.readFileSync(USER_SETTINGS_PATH, 'utf-8');
-            userSettings = JSON.parse(stripJsonComments(userContent));
-            // Support legacy theme names
-            if (userSettings.theme && userSettings.theme === 'VS') {
-                userSettings.theme = DefaultLight.name;
-            }
-            else if (userSettings.theme && userSettings.theme === 'VS2015') {
-                userSettings.theme = DefaultDark.name;
-            }
-        }
-    }
-    catch (error) {
-        settingsErrors.push({
-            message: getErrorMessage(error),
-            path: USER_SETTINGS_PATH,
-        });
-    }
-    if (realWorkspaceDir !== realHomeDir) {
-        // Load workspace settings
+    const loadAndMigrate = (filePath, scope) => {
         try {
-            if (fs.existsSync(workspaceSettingsPath)) {
-                const projectContent = fs.readFileSync(workspaceSettingsPath, 'utf-8');
-                workspaceSettings = JSON.parse(stripJsonComments(projectContent));
-                if (workspaceSettings.theme && workspaceSettings.theme === 'VS') {
-                    workspaceSettings.theme = DefaultLight.name;
+            if (fs.existsSync(filePath)) {
+                const content = fs.readFileSync(filePath, 'utf-8');
+                const rawSettings = JSON.parse(stripJsonComments(content));
+                if (typeof rawSettings !== 'object' ||
+                    rawSettings === null ||
+                    Array.isArray(rawSettings)) {
+                    settingsErrors.push({
+                        message: 'Settings file is not a valid JSON object.',
+                        path: filePath,
+                    });
+                    return {};
                 }
-                else if (workspaceSettings.theme &&
-                    workspaceSettings.theme === 'VS2015') {
-                    workspaceSettings.theme = DefaultDark.name;
+                let settingsObject = rawSettings;
+                if (needsMigration(settingsObject)) {
+                    console.error(`Legacy settings file detected at: ${filePath}`);
+                    const migratedSettings = migrateSettingsToV2(settingsObject);
+                    if (migratedSettings) {
+                        if (MIGRATE_V2_OVERWRITE) {
+                            try {
+                                fs.renameSync(filePath, `${filePath}.orig`);
+                                fs.writeFileSync(filePath, JSON.stringify(migratedSettings, null, 2), 'utf-8');
+                                console.log(`Successfully migrated and saved settings file: ${filePath}`);
+                            }
+                            catch (e) {
+                                console.error(`Error migrating settings file on disk: ${getErrorMessage(e)}`);
+                            }
+                        }
+                        else {
+                            console.log(`Successfully migrated settings for ${filePath} in-memory for the current session.`);
+                            migratedInMemorScopes.add(scope);
+                        }
+                        settingsObject = migratedSettings;
+                    }
                 }
+                return settingsObject;
             }
         }
         catch (error) {
             settingsErrors.push({
                 message: getErrorMessage(error),
-                path: workspaceSettingsPath,
+                path: filePath,
             });
         }
+        return {};
+    };
+    systemSettings = loadAndMigrate(systemSettingsPath, SettingScope.System);
+    systemDefaultSettings = loadAndMigrate(systemDefaultsPath, SettingScope.SystemDefaults);
+    userSettings = loadAndMigrate(USER_SETTINGS_PATH, SettingScope.User);
+    if (realWorkspaceDir !== realHomeDir) {
+        workspaceSettings = loadAndMigrate(workspaceSettingsPath, SettingScope.Workspace);
+    }
+    // Support legacy theme names
+    if (userSettings.ui?.theme === 'VS') {
+        userSettings.ui.theme = DefaultLight.name;
+    }
+    else if (userSettings.ui?.theme === 'VS2015') {
+        userSettings.ui.theme = DefaultDark.name;
+    }
+    if (workspaceSettings.ui?.theme === 'VS') {
+        workspaceSettings.ui.theme = DefaultLight.name;
+    }
+    else if (workspaceSettings.ui?.theme === 'VS2015') {
+        workspaceSettings.ui.theme = DefaultDark.name;
     }
     // For the initial trust check, we can only use user and system settings.
-    const initialTrustCheckSettings = { ...systemSettings, ...userSettings };
+    const initialTrustCheckSettings = mergeWith({}, systemSettings, userSettings);
     const isTrusted = isWorkspaceTrusted(initialTrustCheckSettings) ?? true;
     // Create a temporary merged settings object to pass to loadEnvironment.
-    const tempMergedSettings = mergeSettings(systemSettings, userSettings, workspaceSettings, isTrusted);
+    const tempMergedSettings = mergeSettings(systemSettings, systemDefaultSettings, userSettings, workspaceSettings, isTrusted);
     // loadEnviroment depends on settings so we have to create a temp version of
     // the settings to avoid a cycle
     loadEnvironment(tempMergedSettings);
@@ -334,21 +597,16 @@ export function loadSettings(workspaceDir) {
     const loadedSettings = new LoadedSettings({
         path: systemSettingsPath,
         settings: systemSettings,
+    }, {
+        path: systemDefaultsPath,
+        settings: systemDefaultSettings,
     }, {
         path: USER_SETTINGS_PATH,
         settings: userSettings,
     }, {
         path: workspaceSettingsPath,
         settings: workspaceSettings,
-    }, settingsErrors, isTrusted);
-    // Validate chatCompression settings
-    const chatCompression = loadedSettings.merged.chatCompression;
-    const threshold = chatCompression?.contextPercentageThreshold;
-    if (threshold != null &&
-        (typeof threshold !== 'number' || threshold < 0 || threshold > 1)) {
-        console.warn(`Invalid value for chatCompression.contextPercentageThreshold: "${threshold}". Please use a value between 0 and 1. Using default compression settings.`);
-        delete loadedSettings.merged.chatCompression;
-    }
+    }, settingsErrors, isTrusted, migratedInMemorScopes);
     return loadedSettings;
 }
 export function saveSettings(settingsFile) {
@@ -358,7 +616,11 @@ export function saveSettings(settingsFile) {
         if (!fs.existsSync(dirPath)) {
             fs.mkdirSync(dirPath, { recursive: true });
         }
-        fs.writeFileSync(settingsFile.path, JSON.stringify(settingsFile.settings, null, 2), 'utf-8');
+        let settingsToSave = settingsFile.settings;
+        if (!MIGRATE_V2_OVERWRITE) {
+            settingsToSave = migrateSettingsToV1(settingsToSave);
+        }
+        fs.writeFileSync(settingsFile.path, JSON.stringify(settingsToSave, null, 2), 'utf-8');
     }
     catch (error) {
         console.error('Error saving user settings file:', error);