fix: preserve existing Codex settings during setup

docs/client-setup.md

@@ -117,7 +117,7 @@ Claude Code writes `.mcp.json`; Codex writes `.codex/config.toml`. Both configs
 
 `--yes` writes the config without prompts. Run `init` without flags to choose a client interactively and optionally add project instructions. Cursor and OpenCode still use HTTP and need a separately running server.
 
-Existing settings are kept. If a different Codebase Context entry is already present, setup leaves the file unchanged and reports the conflict. Restart the client after setup and approve the server if asked. The first use builds the code index. Rerun setup if the Codebase Context installation moves, because the config uses its installed location.
+Existing settings are kept. If a different Codebase Context entry is already present, setup leaves the file unchanged and reports the conflict. If the Codebase Context installation moves, remove only its existing entry from the client config. Keep the other settings, then run setup again. Restart the client after setup and approve the server if asked. The first use builds the code index.
 
 ## Optional fixed-folder setup and CLI preparation
 
@@ -129,7 +129,7 @@ To prepare the index separately, run from the project folder:
 npx -y codebase-context@2.4.0 reindex
 ```
 
-The interactive wizard limitations described above are still present in the 2.4.0 source. A successful wizard exit alone does not establish working client registration.
+A successful setup run writes the config but does not confirm that the client connected.
 
 ## Gemini CLI
 

src/cli-init.ts

@@ -339,7 +339,7 @@ export async function _buildMergedCodexContent(
   }
 
   const dottedEntry = new RegExp(
-    `^\\s*${mcpServersKey}\\s*\\.\\s*${codebaseContextKey}\\s*\\.`,
+    `^\\s*${mcpServersKey}\\s*\\.\\s*${codebaseContextKey}\\s*(?:\\.|=)`,
     'm'
   );
   if (dottedEntry.test(normalizedExisting)) {

tests/cli-init-saved-file.test.ts

@@ -0,0 +1,59 @@
+import { mkdtemp, mkdir, readFile, rm, writeFile } from 'node:fs/promises';
+import os from 'node:os';
+import path from 'node:path';
+import { describe, expect, it, vi } from 'vitest';
+import { handleInitCli } from '../src/cli-init.js';
+
+describe('Codex init with an existing saved config', () => {
+  it.each([
+    {
+      name: 'bare dotted keys',
+      toml: 'mcp_servers.codebase-context = { command = "old", args = [] }\n'
+    },
+    {
+      name: 'a double-quoted root key',
+      toml: '"mcp_servers".codebase-context = { command = "old", args = [] }\n'
+    },
+    {
+      name: 'a double-quoted server key',
+      toml: 'mcp_servers."codebase-context" = { command = "old", args = [] }\n'
+    },
+    {
+      name: 'double-quoted root and server keys',
+      toml: '"mcp_servers"."codebase-context" = { command = "old", args = [] }\n'
+    },
+    {
+      name: 'single-quoted root and server keys',
+      toml: "'mcp_servers'.'codebase-context' = { command = \"old\", args = [] }\n"
+    }
+  ])('refuses $name without changing the saved file', async ({ toml }) => {
+    const projectRoot = await mkdtemp(path.join(os.tmpdir(), 'cbc-codex-init-'));
+    const configPath = path.join(projectRoot, '.codex', 'config.toml');
+    const initialBytes = Buffer.from(toml, 'utf8');
+    const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+    const previousExitCode = process.exitCode;
+
+    try {
+      await mkdir(path.dirname(configPath), { recursive: true });
+      await writeFile(configPath, initialBytes);
+      process.exitCode = undefined;
+
+      await handleInitCli(['--client', 'codex', '--yes', '--root', projectRoot]);
+
+      const persistedBytes = await readFile(configPath);
+      expect({ exitCode: process.exitCode, persistedBytes }).toEqual({
+        exitCode: 1,
+        persistedBytes: initialBytes
+      });
+      expect(errorSpy).toHaveBeenCalledWith(
+        expect.stringContaining('already contains a different codebase-context configuration')
+      );
+    } finally {
+      process.exitCode = previousExitCode;
+      errorSpy.mockRestore();
+      logSpy.mockRestore();
+      await rm(projectRoot, { recursive: true, force: true });
+    }
+  });
+});