Refactor config handling and update documentation

- Move runtime settings to VS Code config (kfc.* namespace)
- Simplify project config to only handle paths in kfc-settings.json
- Update README to clarify configuration structure
- Remove unused config interfaces and methods from config-manager
- Add note about fixed settings file location

This change separates project-specific path configuration from runtime settings, making the extension more maintainable and user-configurable through VS Code settings UI.

Author: atman-33

.codex/settings/kfc-settings.json

@@ -0,0 +1,8 @@
+{
+  "paths": {
+    "specs": ".codex/specs",
+    "steering": ".codex/steering",
+    "settings": ".codex/settings",
+    "prompts": ".codex/prompts"
+  }
+}

README.md

@@ -146,7 +146,9 @@ Core commands registered by the extension:
 
 ## Configuration
 
-Settings are stored in `.codex/settings/kfc-settings.json`:
+Project-local settings are stored in `.codex/settings/kfc-settings.json` and only contain paths. UI visibility and Codex runtime options live in VS Code settings under the `kfc.*` namespace.
+
+Minimal settings file:
 
 ```json
 {
@@ -155,26 +157,15 @@ Settings are stored in `.codex/settings/kfc-settings.json`:
     "steering": ".codex/steering",
     "settings": ".codex/settings",
     "prompts": ".codex/prompts"
-  },
-  "views": {
-    "specs": { "visible": true },
-    "steering": { "visible": true },
-    "prompts": { "visible": true },
-    "mcp": { "visible": false },
-    "hooks": { "visible": false },
-    "agents": { "visible": false },
-    "settings": { "visible": false }
-  },
-  "codex": {
-    "path": "codex",
-    "defaultApprovalMode": "interactive",
-    "defaultModel": "gpt-5",
-    "timeout": 30000,
-    "terminalDelay": 1000
   }
 }
 ```
 
+Notes:
+- Only the `paths.*` values are honored by the extension at runtime.
+- Changing `paths.*` may require a window reload to take effect.
+- The location of `kfc-settings.json` itself is fixed to `.codex/settings/kfc-settings.json` (editing `paths.settings` does not relocate the file).
+
 ## Workspace Structure
 
 The extension creates the following structure in your workspace:

src/utils/config-manager.ts

@@ -1,43 +1,16 @@
 import * as path from 'path';
 import * as vscode from 'vscode';
-import { CONFIG_FILE_NAME, DEFAULT_PATHS, DEFAULT_VIEW_VISIBILITY } from '../constants';
-
-export enum ApprovalMode {
-    Interactive = 'interactive',
-    AutoEdit = 'auto-edit',
-    FullAuto = 'full-auto'
-}
-
-export interface CodexConfig {
-    path: string;
-    defaultApprovalMode: ApprovalMode;
-    defaultModel?: string;
-    timeout: number;
-    terminalDelay: number;
-}
-
-export interface MigrationConfig {
-    backupOriginalFiles: boolean;
-    migrationCompleted: boolean;
-}
+import { CONFIG_FILE_NAME, DEFAULT_PATHS } from '../constants';
 
+// Minimal project-local settings persisted under .codex/settings/kfc-settings.json
+// Only "paths" are honored by the extension. Other runtime configs live in VS Code settings (kfc.*).
 export interface KfcSettings {
     paths: {
         specs: string;
         steering: string;
         settings: string;
         prompts: string;
     };
-    views: {
-        specs: { visible: boolean; };
-        steering: { visible: boolean; };
-        mcp: { visible: boolean; };
-        hooks: { visible: boolean; };
-        prompts: { visible: boolean; };
-        settings: { visible: boolean; };
-    };
-    codex?: CodexConfig;
-    migration?: MigrationConfig;
 }
 
 export class ConfigManager {
@@ -108,34 +81,7 @@ export class ConfigManager {
 
     private getDefaultSettings(): KfcSettings {
         return {
-            paths: { ...DEFAULT_PATHS },
-            views: {
-                specs: { visible: DEFAULT_VIEW_VISIBILITY.specs },
-                steering: { visible: DEFAULT_VIEW_VISIBILITY.steering },
-                mcp: { visible: DEFAULT_VIEW_VISIBILITY.mcp },
-                hooks: { visible: DEFAULT_VIEW_VISIBILITY.hooks },
-                prompts: { visible: DEFAULT_VIEW_VISIBILITY.prompts },
-                settings: { visible: DEFAULT_VIEW_VISIBILITY.settings }
-            },
-            codex: this.getDefaultCodexConfig(),
-            migration: this.getDefaultMigrationConfig()
-        };
-    }
-
-    private getDefaultCodexConfig(): CodexConfig {
-        return {
-            path: 'codex',
-            defaultApprovalMode: ApprovalMode.Interactive,
-            defaultModel: 'gpt-5',
-            timeout: 30000,
-            terminalDelay: 1000
-        };
-    }
-
-    private getDefaultMigrationConfig(): MigrationConfig {
-        return {
-            backupOriginalFiles: true,
-            migrationCompleted: false
+            paths: { ...DEFAULT_PATHS }
         };
     }
 
@@ -162,135 +108,5 @@ export class ConfigManager {
         this.settings = settings;
     }
 
-    // Codex Configuration Methods
-    getCodexConfig(): CodexConfig {
-        const settings = this.getSettings();
-        return settings.codex || this.getDefaultCodexConfig();
-    }
-
-    async updateCodexConfig(config: Partial<CodexConfig>): Promise<void> {
-        const settings = this.getSettings();
-        settings.codex = { ...this.getCodexConfig(), ...config };
-        await this.saveSettings(settings);
-    }
-
-    async validateCodexPath(codexPath?: string): Promise<{ isValid: boolean; error?: string; }> {
-        const pathToCheck = codexPath || this.getCodexConfig().path;
-
-        try {
-            // Check if codex command is available
-            const { spawn } = require('child_process');
-
-            return new Promise((resolve) => {
-                const process = spawn(pathToCheck, ['--version'], {
-                    stdio: 'pipe',
-                    timeout: 5000
-                });
-
-                let output = '';
-                process.stdout?.on('data', (data: Buffer) => {
-                    output += data.toString();
-                });
-
-                process.on('close', (code: number) => {
-                    if (code === 0) {
-                        resolve({ isValid: true });
-                    } else {
-                        resolve({
-                            isValid: false,
-                            error: `Codex CLI returned exit code ${code}`
-                        });
-                    }
-                });
-
-                process.on('error', (error: Error) => {
-                    resolve({
-                        isValid: false,
-                        error: `Failed to execute Codex CLI: ${error.message}`
-                    });
-                });
-            });
-        } catch (error) {
-            return {
-                isValid: false,
-                error: `Error validating Codex path: ${error instanceof Error ? error.message : 'Unknown error'}`
-            };
-        }
-    }
-
-    // Migration Methods
-    getMigrationConfig(): MigrationConfig {
-        const settings = this.getSettings();
-        return settings.migration || this.getDefaultMigrationConfig();
-    }
-
-    async updateMigrationConfig(config: Partial<MigrationConfig>): Promise<void> {
-        const settings = this.getSettings();
-        settings.migration = { ...this.getMigrationConfig(), ...config };
-        await this.saveSettings(settings);
-    }
-
-    // Approval Mode Management
-    async setApprovalMode(mode: ApprovalMode): Promise<void> {
-        await this.updateCodexConfig({ defaultApprovalMode: mode });
-    }
-
-    getApprovalMode(): ApprovalMode {
-        return this.getCodexConfig().defaultApprovalMode;
-    }
-
-    // Codex CLI Availability Check
-    async checkCodexAvailability(): Promise<{ available: boolean; version?: string; error?: string; }> {
-        const validation = await this.validateCodexPath();
-
-        if (!validation.isValid) {
-            return {
-                available: false,
-                error: validation.error
-            };
-        }
-
-        try {
-            const codexPath = this.getCodexConfig().path;
-            const { spawn } = require('child_process');
-
-            return new Promise((resolve) => {
-                const process = spawn(codexPath, ['--version'], {
-                    stdio: 'pipe',
-                    timeout: 5000
-                });
-
-                let output = '';
-                process.stdout?.on('data', (data: Buffer) => {
-                    output += data.toString();
-                });
-
-                process.on('close', (code: number) => {
-                    if (code === 0) {
-                        // Extract version from output
-                        const versionMatch = output.match(/(\d+\.\d+\.\d+)/);
-                        const version = versionMatch ? versionMatch[1] : 'unknown';
-                        resolve({ available: true, version });
-                    } else {
-                        resolve({
-                            available: false,
-                            error: `Codex CLI returned exit code ${code}`
-                        });
-                    }
-                });
-
-                process.on('error', (error: Error) => {
-                    resolve({
-                        available: false,
-                        error: `Failed to check Codex version: ${error.message}`
-                    });
-                });
-            });
-        } catch (error) {
-            return {
-                available: false,
-                error: `Error checking Codex availability: ${error instanceof Error ? error.message : 'Unknown error'}`
-            };
-        }
-    }
+    // (Intentionally minimal) — legacy config sections (views/codex/migration) have been removed.
 }