faf-mcp 2.3.0 → 3.0.0

package/dist/src/handlers/engine-adapter.js

@@ -34,86 +34,43 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.FafEngineAdapter = void 0;
-const child_process_1 = require("child_process");
-const util_1 = require("util");
+/**
+ * FafEngineAdapter — routes the interop tools to their bundled, in-process
+ * commands.
+ *
+ * Every faf-mcp tool runs in-process: scoring, init, auto, trust, status and
+ * the resources go through src/utils/faf-cli-bridge.ts (the faf-cli this
+ * package bundles); the seven interop commands below run the vendored
+ * faf-core modules. Nothing shells out.
+ *
+ * History (3.0 P2): this file used to carry 22 command branches, a
+ * `which faf` detector that ran at every server start, and an exec fallback
+ * that spawned whatever binary the detector found — on machines with an
+ * unrelated `faf` on PATH that was a different program entirely. Only seven
+ * branches were ever reached by a registered tool; the rest, the detector
+ * and the fallback are gone (archive tag: archive/pre-p2-dead-code).
+ */
 const path = __importStar(require("path"));
 const fs = __importStar(require("fs"));
-const type_guards_js_1 = require("../utils/type-guards.js"); // ✅ FIXED: Removed unused isDefined
-const cli_detector_js_1 = require("../utils/cli-detector.js");
-const score_js_1 = require("../faf-core/commands/score.js");
-const init_js_1 = require("../faf-core/commands/init.js");
-const auto_js_1 = require("../faf-core/commands/auto.js");
+const type_guards_js_1 = require("../utils/type-guards.js");
 const sync_js_1 = require("../faf-core/commands/sync.js");
 const bi_sync_js_1 = require("../faf-core/commands/bi-sync.js");
-const formats_js_1 = require("../faf-core/commands/formats.js");
-const doctor_js_1 = require("../faf-core/commands/doctor.js");
-const validate_js_1 = require("../faf-core/commands/validate.js");
-const audit_js_1 = require("../faf-core/commands/audit.js");
-const update_js_1 = require("../faf-core/commands/update.js");
-const migrate_js_1 = require("../faf-core/commands/migrate.js");
-const innit_js_1 = require("../faf-core/commands/innit.js");
-const quick_js_1 = require("../faf-core/commands/quick.js");
-const human_js_1 = require("../faf-core/commands/human.js");
-const readme_js_1 = require("../faf-core/commands/readme.js");
-// v4.5.0 Interop commands
 const agents_js_1 = require("../faf-core/commands/agents.js");
 const cursor_js_1 = require("../faf-core/commands/cursor.js");
 const gemini_js_1 = require("../faf-core/commands/gemini.js");
 const conductor_js_1 = require("../faf-core/commands/conductor.js");
 const git_context_js_1 = require("../faf-core/commands/git-context.js");
-const execAsync = (0, util_1.promisify)(child_process_1.exec);
-// Enhanced PATH for FAF CLI discovery
-const getEnhancedEnv = () => ({
-    ...process.env,
-    PATH: [
-        '/usr/local/bin',
-        '/opt/homebrew/bin',
-        process.env.HOME ? `${process.env.HOME}/.npm/bin` : undefined,
-        process.env.HOME ? `${process.env.HOME}/.npm-global/bin` : undefined,
-        '/usr/bin',
-        '/bin',
-        process.env.PATH
-    ].filter(Boolean).join(':')
-});
 class FafEngineAdapter {
     enginePath;
-    detectedCliPath = null;
-    cliVersion;
-    timeout;
     workingDirectory;
-    constructor(enginePath = 'faf', timeout = 30000) {
+    /**
+     * `enginePath` is kept for the config surface (server.ts passes it through;
+     * faf_debug reports it). `_timeout` was the exec fallback's — retained in
+     * the signature so callers do not break, unused now that nothing spawns.
+     */
+    constructor(enginePath = 'native', _timeout = 30000) {
         this.enginePath = enginePath;
-        this.timeout = timeout;
         this.workingDirectory = this.findBestWorkingDirectory();
-        // Auto-detect CLI on initialization
-        this.detectCli();
-    }
-    detectCli() {
-        const detection = (0, cli_detector_js_1.detectFafCli)();
-        if (detection.found && detection.path) {
-            this.detectedCliPath = detection.path;
-            this.cliVersion = detection.version;
-            // Validate version meets minimum requirement
-            if (this.cliVersion && !(0, cli_detector_js_1.validateCliVersion)(this.cliVersion, '3.1.1')) {
-                console.warn(`FAF CLI version ${this.cliVersion} is below minimum required version 3.1.1. Some features may not work correctly.`);
-            }
-            console.error(`✅ FAF CLI detected: ${this.detectedCliPath} (v${this.cliVersion || 'unknown'}) via ${detection.method}`);
-        }
-        else {
-            console.error('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-            console.error('⚠️  FAF CLI NOT DETECTED');
-            console.error('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-            console.error('');
-            console.error('INSTALLATION ORDER REQUIRED:');
-            console.error('  1️⃣  npm install -g faf-cli        (REQUIRED FIRST)');
-            console.error('  2️⃣  npm install -g faf-mcp  (THEN THIS)');
-            console.error('');
-            console.error('Most MCP tools require faf-cli to be installed.');
-            console.error('Only basic file operations will work without it.');
-            console.error('');
-            console.error('After installing faf-cli, restart your MCP host.');
-            console.error('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-        }
     }
     findBestWorkingDirectory() {
         // Priority 1: Environment variable for explicit control
@@ -126,7 +83,27 @@ class FafEngineAdapter {
         if (mcpWorkingDir && fs.existsSync(mcpWorkingDir)) {
             return mcpWorkingDir;
         }
-        // Priority 3: FORCE ~/Projects as universal default
+        // Priority 3 (FIX 2026-06-30): the caller's ACTUAL working directory — the
+        // workspace an IDE / MCP host (Cursor, VS Code, Claude Desktop) launches the
+        // server in. This MUST win over the ~/Projects convention below. Previously
+        // ~/Projects was FORCED here, so every no-path tool call (faf_score, …)
+        // operated on ~/Projects instead of the project the user actually had open —
+        // the editor surface's whole value, silently broken (scored ~/Projects's
+        // stale .faf for everyone). Prefer a real FAF project (cwd contains
+        // project.faf — the path-check), else the cwd itself when it's a usable,
+        // non-root directory (so faf_init/score act on the open workspace even
+        // before a .faf exists).
+        const currentDir = process.cwd();
+        const usableCwd = currentDir !== '/' && currentDir !== '/root' && fs.existsSync(currentDir);
+        if (usableCwd && fs.existsSync(path.join(currentDir, 'project.faf'))) {
+            return currentDir;
+        }
+        if (usableCwd) {
+            return currentDir;
+        }
+        // Priority 4: ~/Projects convention — a soft landing ONLY when the host gave
+        // us no usable workspace (e.g. cwd is the filesystem root). Never overrides a
+        // real cwd above.
         const homeDir = process.env.HOME ?? process.env.USERPROFILE;
         if (homeDir) {
             // Try capitalized Projects first (macOS/Windows convention)
@@ -149,632 +126,99 @@ class FafEngineAdapter {
                 return homeDir;
             }
         }
-        // Priority 4: Current directory if not root
-        const currentDir = process.cwd();
-        if (currentDir !== '/' && currentDir !== '/root' && fs.existsSync(currentDir)) {
-            return currentDir;
-        }
         // Last resort: /tmp (should rarely happen)
         return '/tmp';
     }
+    /**
+     * Wrap a bundled command's outcome. A failed command's reason travels in
+     * `error` so handlers can print it — previously the bundled branches set
+     * only `success` and users saw "❌ undefined".
+     */
+    outcome(result, fallback, startTime) {
+        return {
+            success: result.success,
+            data: result,
+            error: result.success ? undefined : (result.message || result.error || fallback),
+            duration: Date.now() - startTime,
+        };
+    }
+    failure(error, fallback, startTime) {
+        return { success: false, error: (0, type_guards_js_1.isError)(error) ? error.message : fallback, duration: Date.now() - startTime };
+    }
     async callEngine(command, args = []) {
         const startTime = Date.now();
-        // Input validation
         if (!command || typeof command !== 'string') {
-            return {
-                success: false,
-                error: 'Command must be a non-empty string',
-                duration: 0
-            };
-        }
-        // ============================================================================
-        // MK3 BUNDLED ENGINE - Direct function calls (no CLI dependency!)
-        // ============================================================================
-        // SCORE command - use bundled FafCompiler
-        if (command === 'score') {
-            try {
-                const filePath = args[0] || this.workingDirectory;
-                const result = await (0, score_js_1.scoreFafFile)(filePath, { json: true });
-                const duration = Date.now() - startTime;
-                return {
-                    success: true,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                const errorMessage = (0, type_guards_js_1.isError)(error) ? error.message : 'Score command failed';
-                return {
-                    success: false,
-                    error: errorMessage,
-                    duration
-                };
-            }
-        }
-        // INIT command - use bundled init
-        if (command === 'init') {
-            try {
-                // Filter out flags to get the actual path argument
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const force = args.includes('--force');
-                const result = await (0, init_js_1.initFafFile)(projectPath, { force });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                const errorMessage = (0, type_guards_js_1.isError)(error) ? error.message : 'Init command failed';
-                return {
-                    success: false,
-                    error: errorMessage,
-                    duration
-                };
-            }
-        }
-        // AUTO command - use bundled auto (init + score)
-        if (command === 'auto') {
-            try {
-                // Filter out flags to get the actual path argument
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const directory = pathArgs[0] || this.workingDirectory;
-                const force = args.includes('--force');
-                const result = await (0, auto_js_1.autoCommand)(directory, { force });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                const errorMessage = (0, type_guards_js_1.isError)(error) ? error.message : 'Auto command failed';
-                return {
-                    success: false,
-                    error: errorMessage,
-                    duration
-                };
-            }
-        }
-        // SYNC command - use bundled sync
-        if (command === 'sync') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const auto = args.includes('--auto');
-                const result = await (0, sync_js_1.syncFafFile)(projectPath, { auto, json: true });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Sync command failed',
-                    duration
-                };
-            }
-        }
-        // BI-SYNC command - use bundled bi-sync (v4.5.0: supports agents/cursor/gemini/all flags)
-        if (command === 'bi-sync' || command === 'bisync') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const result = await (0, bi_sync_js_1.syncBiDirectional)(projectPath, {
-                    json: true,
-                    agents: args.includes('--agents'),
-                    cursor: args.includes('--cursor'),
-                    gemini: args.includes('--gemini'),
-                    all: args.includes('--all'),
-                });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Bi-sync command failed',
-                    duration
-                };
-            }
-        }
-        // FORMATS command - use bundled formats
-        if (command === 'formats') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const result = await (0, formats_js_1.formatsCommand)(projectPath, { json: true });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Formats command failed',
-                    duration
-                };
-            }
-        }
-        // DOCTOR command - use bundled doctor
-        if (command === 'doctor') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const result = await (0, doctor_js_1.doctorCommand)(projectPath);
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Doctor command failed',
-                    duration
-                };
-            }
-        }
-        // VALIDATE command - use bundled validate
-        if (command === 'validate') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const result = await (0, validate_js_1.validateFafFile)(projectPath, { json: true });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Validate command failed',
-                    duration
-                };
-            }
-        }
-        // AUDIT command - use bundled audit
-        if (command === 'audit') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const result = await (0, audit_js_1.auditFafFile)(projectPath, { json: true });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Audit command failed',
-                    duration
-                };
-            }
-        }
-        // UPDATE command - use bundled update
-        if (command === 'update') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const result = await (0, update_js_1.updateFafFile)(projectPath, { json: true });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Update command failed',
-                    duration
-                };
-            }
-        }
-        // MIGRATE command - use bundled migrate
-        if (command === 'migrate') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const force = args.includes('--force');
-                const result = await (0, migrate_js_1.migrateFafFile)(projectPath, { force, json: true });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Migrate command failed',
-                    duration
-                };
-            }
-        }
-        // INNIT command - use bundled innit (British init)
-        if (command === 'innit') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const force = args.includes('--force');
-                const result = await (0, innit_js_1.innitFafFile)(projectPath, { force });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Innit command failed',
-                    duration
-                };
-            }
-        }
-        // QUICK command - use bundled quick
-        if (command === 'quick') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const input = pathArgs[1]; // optional input string
-                const force = args.includes('--force');
-                const json = args.includes('--json');
-                const result = await (0, quick_js_1.quickCommand)(projectPath, input, { force, json });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Quick command failed',
-                    duration
-                };
-            }
-        }
-        // HUMAN command - use bundled human (add/set human_context)
-        if (command === 'human' || command === 'human-add') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                // Extract YAML from args if provided
-                const yamlArg = args.find(arg => arg.startsWith('--yaml='));
-                const yaml = yamlArg ? yamlArg.substring(7) : undefined;
-                // Extract field/value for single field mode
-                const fieldArg = args.find(arg => arg.startsWith('--field='));
-                const valueArg = args.find(arg => arg.startsWith('--value='));
-                const field = fieldArg ? fieldArg.substring(8) : undefined;
-                const value = valueArg ? valueArg.substring(8) : undefined;
-                const result = await (0, human_js_1.humanAddCommand)(projectPath, { yaml, field, value });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Human command failed',
-                    duration
-                };
-            }
+            return { success: false, error: 'Command must be a non-empty string', duration: 0 };
         }
-        // HUMAN-SET command - set single field
-        if (command === 'human-set') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const field = pathArgs[1];
-                const value = pathArgs[2];
-                if (!field || !value) {
+        const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
+        const projectPath = pathArgs[0] || this.workingDirectory;
+        const actionArg = args.find(arg => arg.startsWith('--action='));
+        const force = args.includes('--force');
+        const merge = args.includes('--merge');
+        try {
+            switch (command) {
+                case 'sync': {
+                    const result = await (0, sync_js_1.syncFafFile)(projectPath, { auto: args.includes('--auto'), json: true });
+                    return this.outcome(result, 'Sync command failed', startTime);
+                }
+                case 'bi-sync':
+                case 'bisync': {
+                    const result = await (0, bi_sync_js_1.syncBiDirectional)(projectPath, {
+                        json: true,
+                        agents: args.includes('--agents'),
+                        cursor: args.includes('--cursor'),
+                        gemini: args.includes('--gemini'),
+                        all: args.includes('--all'),
+                    });
+                    return this.outcome(result, 'Bi-sync command failed', startTime);
+                }
+                case 'agents': {
+                    const action = actionArg ? actionArg.substring(9) : 'sync';
+                    const result = action === 'import' ? await (0, agents_js_1.agentsImportCommand)(projectPath, { merge })
+                        : action === 'export' ? await (0, agents_js_1.agentsExportCommand)(projectPath, { force })
+                            : await (0, agents_js_1.agentsSyncCommand)(projectPath);
+                    return this.outcome(result, 'Agents command failed', startTime);
+                }
+                case 'cursor': {
+                    const action = actionArg ? actionArg.substring(9) : 'sync';
+                    const result = action === 'import' ? await (0, cursor_js_1.cursorImportCommand)(projectPath, { merge })
+                        : action === 'export' ? await (0, cursor_js_1.cursorExportCommand)(projectPath, { force })
+                            : await (0, cursor_js_1.cursorSyncCommand)(projectPath);
+                    return this.outcome(result, 'Cursor command failed', startTime);
+                }
+                case 'gemini': {
+                    const action = actionArg ? actionArg.substring(9) : 'sync';
+                    const result = action === 'import' ? await (0, gemini_js_1.geminiImportCommand)(projectPath, { merge })
+                        : action === 'export' ? await (0, gemini_js_1.geminiExportCommand)(projectPath, { force })
+                            : await (0, gemini_js_1.geminiSyncCommand)(projectPath);
+                    return this.outcome(result, 'Gemini command failed', startTime);
+                }
+                case 'conductor': {
+                    const action = actionArg ? actionArg.substring(9) : 'import';
+                    const result = action === 'export'
+                        ? await (0, conductor_js_1.conductorExportCommand)(projectPath, { force })
+                        : await (0, conductor_js_1.conductorImportCommand)(projectPath, { merge });
+                    return this.outcome(result, 'Conductor command failed', startTime);
+                }
+                case 'git': {
+                    const url = args[0];
+                    const outputPath = args[1]; // optional
+                    if (!url) {
+                        return { success: false, error: 'URL is required', duration: Date.now() - startTime };
+                    }
+                    const result = await (0, git_context_js_1.gitContextCommand)(url, outputPath);
+                    return this.outcome(result, 'Git command failed', startTime);
+                }
+                default:
                     return {
                         success: false,
-                        error: 'Usage: human-set <field> <value>',
-                        duration: Date.now() - startTime
+                        error: `Unknown engine command '${command}'. Every faf-mcp tool runs in-process; there is no CLI to fall back to.`,
+                        duration: Date.now() - startTime,
                     };
-                }
-                const result = await (0, human_js_1.humanSetCommand)(projectPath, field, value);
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'Human-set command failed',
-                    duration
-                };
-            }
-        }
-        // README command - extract context from README.md
-        if (command === 'readme' || command === 'readme-extract') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const result = await (0, readme_js_1.readmeExtractCommand)(projectPath);
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'README extract command failed',
-                    duration
-                };
-            }
-        }
-        // README-MERGE command - extract and merge into .faf
-        if (command === 'readme-merge') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                // Check for overwrite flag
-                const overwrite = args.includes('--overwrite');
-                const result = await (0, readme_js_1.readmeMergeCommand)(projectPath, { overwrite });
-                const duration = Date.now() - startTime;
-                return {
-                    success: result.success,
-                    data: result,
-                    duration
-                };
-            }
-            catch (error) {
-                const duration = Date.now() - startTime;
-                return {
-                    success: false,
-                    error: (0, type_guards_js_1.isError)(error) ? error.message : 'README merge command failed',
-                    duration
-                };
-            }
-        }
-        // ============================================================================
-        // v4.5.0 INTEROP COMMANDS
-        // ============================================================================
-        // AGENTS command - import/export/sync AGENTS.md
-        if (command === 'agents') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const actionArg = args.find(arg => arg.startsWith('--action='));
-                const action = actionArg ? actionArg.substring(9) : 'sync';
-                const force = args.includes('--force');
-                const merge = args.includes('--merge');
-                let result;
-                if (action === 'import') {
-                    result = await (0, agents_js_1.agentsImportCommand)(projectPath, { merge });
-                }
-                else if (action === 'export') {
-                    result = await (0, agents_js_1.agentsExportCommand)(projectPath, { force });
-                }
-                else {
-                    result = await (0, agents_js_1.agentsSyncCommand)(projectPath);
-                }
-                return { success: result.success, data: result, duration: Date.now() - startTime };
-            }
-            catch (error) {
-                return { success: false, error: (0, type_guards_js_1.isError)(error) ? error.message : 'Agents command failed', duration: Date.now() - startTime };
-            }
-        }
-        // CURSOR command - import/export/sync .cursorrules
-        if (command === 'cursor') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const actionArg = args.find(arg => arg.startsWith('--action='));
-                const action = actionArg ? actionArg.substring(9) : 'sync';
-                const force = args.includes('--force');
-                const merge = args.includes('--merge');
-                let result;
-                if (action === 'import') {
-                    result = await (0, cursor_js_1.cursorImportCommand)(projectPath, { merge });
-                }
-                else if (action === 'export') {
-                    result = await (0, cursor_js_1.cursorExportCommand)(projectPath, { force });
-                }
-                else {
-                    result = await (0, cursor_js_1.cursorSyncCommand)(projectPath);
-                }
-                return { success: result.success, data: result, duration: Date.now() - startTime };
-            }
-            catch (error) {
-                return { success: false, error: (0, type_guards_js_1.isError)(error) ? error.message : 'Cursor command failed', duration: Date.now() - startTime };
-            }
-        }
-        // GEMINI command - import/export/sync GEMINI.md
-        if (command === 'gemini') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const actionArg = args.find(arg => arg.startsWith('--action='));
-                const action = actionArg ? actionArg.substring(9) : 'sync';
-                const force = args.includes('--force');
-                const merge = args.includes('--merge');
-                let result;
-                if (action === 'import') {
-                    result = await (0, gemini_js_1.geminiImportCommand)(projectPath, { merge });
-                }
-                else if (action === 'export') {
-                    result = await (0, gemini_js_1.geminiExportCommand)(projectPath, { force });
-                }
-                else {
-                    result = await (0, gemini_js_1.geminiSyncCommand)(projectPath);
-                }
-                return { success: result.success, data: result, duration: Date.now() - startTime };
-            }
-            catch (error) {
-                return { success: false, error: (0, type_guards_js_1.isError)(error) ? error.message : 'Gemini command failed', duration: Date.now() - startTime };
-            }
-        }
-        // CONDUCTOR command - import/export conductor/
-        if (command === 'conductor') {
-            try {
-                const pathArgs = args.filter(arg => !arg.startsWith('--') && !arg.startsWith('-'));
-                const projectPath = pathArgs[0] || this.workingDirectory;
-                const actionArg = args.find(arg => arg.startsWith('--action='));
-                const action = actionArg ? actionArg.substring(9) : 'import';
-                const force = args.includes('--force');
-                const merge = args.includes('--merge');
-                let result;
-                if (action === 'export') {
-                    result = await (0, conductor_js_1.conductorExportCommand)(projectPath, { force });
-                }
-                else {
-                    result = await (0, conductor_js_1.conductorImportCommand)(projectPath, { merge });
-                }
-                return { success: result.success, data: result, duration: Date.now() - startTime };
-            }
-            catch (error) {
-                return { success: false, error: (0, type_guards_js_1.isError)(error) ? error.message : 'Conductor command failed', duration: Date.now() - startTime };
-            }
-        }
-        // GIT command - generate .faf from GitHub repo
-        if (command === 'git') {
-            try {
-                const url = args[0];
-                const outputPath = args[1]; // optional
-                if (!url) {
-                    return { success: false, error: 'URL is required', duration: Date.now() - startTime };
-                }
-                const result = await (0, git_context_js_1.gitContextCommand)(url, outputPath);
-                return { success: result.success, data: result, duration: Date.now() - startTime };
-            }
-            catch (error) {
-                return { success: false, error: (0, type_guards_js_1.isError)(error) ? error.message : 'Git command failed', duration: Date.now() - startTime };
-            }
-        }
-        // ============================================================================
-        // FALLBACK: Shell out to CLI for commands not yet bundled
-        // ============================================================================
-        // Check if CLI is available
-        if (!this.detectedCliPath) {
-            const duration = Date.now() - startTime;
-            return {
-                success: false,
-                error: `FAF CLI not detected. Command '${command}' requires faf-cli.\n\nINSTALLATION ORDER:\n  1️⃣  npm install -g faf-cli\n  2️⃣  npm install -g faf-mcp\n\nAfter installing faf-cli, restart your MCP host.`,
-                duration
-            };
-        }
-        // Sanitize arguments to prevent injection
-        const sanitizedArgs = args.map(arg => typeof arg === 'string' ? arg.replace(/[;&|`$(){}[\]]/g, '') : '');
-        try {
-            // Use detected absolute path to CLI
-            const fullCommand = `"${this.detectedCliPath}" ${command} ${sanitizedArgs.join(' ')}`;
-            const { stdout, stderr } = await execAsync(fullCommand, {
-                env: getEnhancedEnv(),
-                timeout: this.timeout,
-                maxBuffer: 1024 * 1024, // 1MB buffer limit
-                cwd: this.workingDirectory
-            });
-            const duration = Date.now() - startTime;
-            if (stderr?.trim()) {
-                console.warn(`FAF CLI warning: ${stderr.trim()}`);
             }
-            return {
-                success: true,
-                data: this.parseOutput(stdout),
-                duration
-            };
         }
         catch (error) {
-            const duration = Date.now() - startTime;
-            let errorMessage = 'Unknown error occurred';
-            if ((0, type_guards_js_1.isError)(error)) {
-                errorMessage = error.message;
-                // Handle specific error codes
-                if ('code' in error) {
-                    if (error.code === 'ETIMEDOUT') {
-                        errorMessage = `Command timed out after ${this.timeout}ms`;
-                    }
-                    else if (error.code === 'ENOENT') {
-                        errorMessage = `FAF CLI not found at ${this.detectedCliPath}. Please reinstall: npm install -g faf-cli`;
-                    }
-                }
-                // Handle signal termination
-                if ('signal' in error && error.signal === 'SIGTERM') {
-                    errorMessage = 'Command was terminated';
-                }
-            }
-            console.error(`FAF CLI error: ${errorMessage}`);
-            return {
-                success: false,
-                error: errorMessage,
-                duration
-            };
-        }
-    }
-    parseOutput(output) {
-        if (!output || output.trim() === '') {
-            return { output: '' };
-        }
-        try {
-            return JSON.parse(output);
-        }
-        catch {
-            return { output: output.trim() };
-        }
-    }
-    // Method to check if FAF CLI is available
-    async checkHealth() {
-        try {
-            const result = await this.callEngine('--version');
-            return result.success;
-        }
-        catch {
-            return false;
+            return this.failure(error, `${command} command failed`, startTime);
         }
     }
     // Get the current working directory used by the adapter
@@ -800,14 +244,6 @@ class FafEngineAdapter {
     getEnginePath() {
         return this.enginePath;
     }
-    // Get CLI detection info for debugging
-    getCliInfo() {
-        return {
-            detected: this.detectedCliPath !== null,
-            path: this.detectedCliPath || undefined,
-            version: this.cliVersion
-        };
-    }
 }
 exports.FafEngineAdapter = FafEngineAdapter;
 //# sourceMappingURL=engine-adapter.js.map