fix(cli): resolve settings test failures by updating mock and using truly invalid JSON

packages/cli/src/config/settings_repro.test.ts

@@ -80,9 +80,14 @@ vi.mock('@google/gemini-cli-core', async (importOriginal) => {
   };
 });
 
-vi.mock('../utils/commentJson.js', () => ({
-  updateSettingsFilePreservingFormat: vi.fn(),
-}));
+vi.mock('../utils/commentJson.js', async (importOriginal) => {
+  const actual =
+    await importOriginal<typeof import('../utils/commentJson.js')>();
+  return {
+    ...actual,
+    updateSettingsFilePreservingFormat: vi.fn(),
+  };
+});
 
 vi.mock('strip-json-comments', () => ({
   default: vi.fn((content) => content),

packages/cli/src/config/settings_validation_warning.test.ts

@@ -152,7 +152,7 @@ describe('Settings Validation Warning', () => {
     );
 
     (fs.readFileSync as Mock).mockImplementation((p: string) => {
-      if (p === USER_SETTINGS_PATH) return '{ "invalid": "json", }'; // Trailing comma is invalid in standard JSON
+      if (p === USER_SETTINGS_PATH) return '{ "invalid": "json"'; // Unclosed brace is invalid
       return '{}';
     });