CODEX mcp fix

Author: HumbleBee14

src/vs/workbench/contrib/roopik/electron-main/mcp/installer/mcpInstaller.ts

@@ -169,15 +169,17 @@ export class McpInstaller extends Disposable {
 		} else if (agent === 'codex') {
 			// Codex extension - check extension binary
 			const extPath = this._platformAdapter?.getExternalExtensionPath(CODEX_EXTENSION_ID);
-			installed = !!getCodexBinaryPath(extPath);
+			installed = !!getCodexBinaryPath(extPath) || installed;
 			registrationMethod = 'cli-command';
 		} else if (agent === 'codex-cli') {
 			// Codex CLI - check global command
 			installed = await isCommandAvailable('codex');
 			registrationMethod = 'cli-command';
 		}
 
-		if (installed) {
+		// For CLI-based agents, we can't reliably check registration from config files
+		// because they manage their own storage locations
+		if (installed && registrationMethod !== 'cli-command') {
 			const config = readJsonConfig<{ mcpServers?: Record<string, unknown> }>(configPath);
 			registered = isRoopikRegistered(config);
 		}

src/vs/workbench/contrib/roopik/electron-main/mcp/installer/mcpInstallerUtils.ts

@@ -158,12 +158,14 @@ export function getAgentConfigPath(agent: AiAgent): string {
 			return path.join(home, '.gemini', 'settings.json');
 
 		case 'codex':
-			// Codex extension uses CLI commands, no config file
-			return path.join(home, '.codex', 'mcp.json');
+			// Codex extension (uses binary from VS Code extension folder)
+			// Both extension and CLI share the same config: ~/.codex/config.toml
+			return path.join(home, '.codex', 'config.toml');
 
 		case 'codex-cli':
-			// Codex CLI uses CLI commands, no config file
-			return path.join(home, '.codex', 'mcp.json');
+			// Codex global CLI (uses `codex` command in PATH)
+			// Both extension and CLI share the same config: ~/.codex/config.toml
+			return path.join(home, '.codex', 'config.toml');
 
 		default:
 			throw new Error(`Unknown agent: ${agent}`);