@patchen0518/agentbrew 1.2.5 → 1.3.0

package/dist/sync.js

@@ -36,8 +36,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.INSTRUCTIONS_FILE = exports.MARKER_END = exports.MARKER_START = void 0;
+exports.unsyncInstructions = exports.syncInstructions = exports.removeFromFile = exports.injectIntoFile = exports.buildInjectedSection = exports.getInstructionsPath = exports.getDefaultTargets = exports.INSTRUCTIONS_FILE = exports.MARKER_END = exports.MARKER_START = exports.unsyncMcpServerFromKiro = exports.syncMcpServerToKiro = exports.unsyncMcpServerFromCodex = exports.syncMcpServerToCodex = exports.unsyncMcpServerFromCursor = exports.syncMcpServerToCursor = void 0;
 exports.extractSkillEntries = extractSkillEntries;
+exports.syncSkillsToAgent = syncSkillsToAgent;
+exports.unsyncSkillsFromAgent = unsyncSkillsFromAgent;
 exports.syncSkillsToClaudeCode = syncSkillsToClaudeCode;
 exports.unsyncSkillsFromClaudeCode = unsyncSkillsFromClaudeCode;
 exports.syncSkillsToGeminiCLI = syncSkillsToGeminiCLI;
@@ -46,30 +48,18 @@ exports.syncSkillsToWindsurf = syncSkillsToWindsurf;
 exports.unsyncSkillsFromWindsurf = unsyncSkillsFromWindsurf;
 exports.syncSkillsToAntigravityCLI = syncSkillsToAntigravityCLI;
 exports.unsyncSkillsFromAntigravityCLI = unsyncSkillsFromAntigravityCLI;
+exports.syncSkillsToKiro = syncSkillsToKiro;
+exports.unsyncSkillsFromKiro = unsyncSkillsFromKiro;
 exports.cleanOrphanSkills = cleanOrphanSkills;
-exports.syncMcpServerToCursor = syncMcpServerToCursor;
-exports.unsyncMcpServerFromCursor = unsyncMcpServerFromCursor;
-exports.syncMcpServerToCodex = syncMcpServerToCodex;
-exports.unsyncMcpServerFromCodex = unsyncMcpServerFromCodex;
 exports.syncSkillsToCursor = syncSkillsToCursor;
 exports.unsyncSkillsFromCursor = unsyncSkillsFromCursor;
-exports.syncSkillsToKiro = syncSkillsToKiro;
-exports.unsyncSkillsFromKiro = unsyncSkillsFromKiro;
-exports.syncMcpServerToKiro = syncMcpServerToKiro;
-exports.unsyncMcpServerFromKiro = unsyncMcpServerFromKiro;
-exports.getDefaultTargets = getDefaultTargets;
-exports.getInstructionsPath = getInstructionsPath;
-exports.buildInjectedSection = buildInjectedSection;
-exports.injectIntoFile = injectIntoFile;
-exports.removeFromFile = removeFromFile;
-exports.syncInstructions = syncInstructions;
-exports.unsyncInstructions = unsyncInstructions;
 const fs_1 = __importDefault(require("fs"));
 const os_1 = __importDefault(require("os"));
 const path_1 = __importDefault(require("path"));
 const toml = __importStar(require("smol-toml"));
-const config_1 = require("./config");
-const package_json_1 = __importDefault(require("../package.json"));
+const sync_state_1 = require("./sync-state");
+const agent_registry_1 = require("./agent-registry");
+const logger_1 = require("./logger");
 /**
  * Extracts SkillEntry objects from discovered packages by scanning for SKILL.md prompts.
  */
@@ -91,40 +81,7 @@ function extractSkillEntries(packages) {
     }
     return skills;
 }
-const SYNCED_SKILLS_FILE = 'synced-skills.json';
-const AGENTBREW_EXTENSION_NAME = 'agentbrew';
 const CURSOR_SKILLS_INDEX_FILE = 'agentbrew-skills-index.md';
-function getSyncedSkillsPath(brewRoot) {
-    return path_1.default.join(brewRoot ?? (0, config_1.getBrewRoot)(), SYNCED_SKILLS_FILE);
-}
-function loadSyncedState(brewRoot) {
-    try {
-        const raw = JSON.parse(fs_1.default.readFileSync(getSyncedSkillsPath(brewRoot), 'utf-8'));
-        // Migrate old flat format: { skills: [...] } → { claude: [...] }
-        if (raw.skills && !raw.claude) {
-            return { claude: raw.skills, gemini: [], windsurf: [], cursor: false, cursorMcp: false, antigravity: [], codexMcp: false, kiro: [], kiroMcp: false };
-        }
-        return {
-            claude: raw.claude ?? [],
-            gemini: raw.gemini ?? [],
-            windsurf: raw.windsurf ?? [],
-            cursor: raw.cursor ?? false,
-            cursorMcp: raw.cursorMcp ?? false,
-            antigravity: raw.antigravity ?? [],
-            codexMcp: raw.codexMcp ?? false,
-            kiro: raw.kiro ?? [],
-            kiroMcp: raw.kiroMcp ?? false,
-        };
-    }
-    catch {
-        return { claude: [], gemini: [], windsurf: [], cursor: false, cursorMcp: false, antigravity: [], codexMcp: false, kiro: [], kiroMcp: false };
-    }
-}
-function saveSyncedState(state, brewRoot) {
-    const p = getSyncedSkillsPath(brewRoot);
-    fs_1.default.mkdirSync(path_1.default.dirname(p), { recursive: true });
-    fs_1.default.writeFileSync(p, JSON.stringify(state, null, 2), 'utf-8');
-}
 function symlinkSkills(skills, targetDir, state, agentKey, brewRoot) {
     const results = [];
     const newEntries = [];
@@ -140,13 +97,19 @@ function symlinkSkills(skills, targetDir, state, agentKey, brewRoot) {
             fs_1.default.lstatSync(entryPath);
             exists = true;
         }
-        catch { }
+        catch (e) {
+            if (e.code !== 'ENOENT')
+                logger_1.Logger.warn(`Unexpected error checking ${entryName}: ${e.message}`);
+        }
         if (exists) {
             let currentTarget = null;
             try {
                 currentTarget = fs_1.default.readlinkSync(entryPath);
             }
-            catch { }
+            catch (e) {
+                if (e.code !== 'ENOENT')
+                    logger_1.Logger.warn(`Unexpected error checking ${entryName}: ${e.message}`);
+            }
             if (currentTarget === null) {
                 // Not a symlink — not created by AgentBrew, leave it alone
                 results.push({ entryName, status: 'skipped', note: 'Path exists and is not a symlink' });
@@ -161,7 +124,9 @@ function symlinkSkills(skills, targetDir, state, agentKey, brewRoot) {
             try {
                 fs_1.default.rmSync(entryPath, { force: true });
             }
-            catch { }
+            catch (e) {
+                logger_1.Logger.warn(`Could not remove ${entryPath}: ${e.message}`);
+            }
         }
         try {
             fs_1.default.symlinkSync(skill.skillDir, entryPath);
@@ -173,7 +138,7 @@ function symlinkSkills(skills, targetDir, state, agentKey, brewRoot) {
         }
     }
     state[agentKey] = [...new Set([...state[agentKey], ...newEntries])];
-    saveSyncedState(state, brewRoot);
+    (0, sync_state_1.saveSyncedState)(state, brewRoot);
     return results;
 }
 function removeTrackedSymlinks(tracked, skillsDir) {
@@ -185,7 +150,10 @@ function removeTrackedSymlinks(tracked, skillsDir) {
             fs_1.default.lstatSync(entryPath);
             exists = true;
         }
-        catch { }
+        catch (e) {
+            if (e.code !== 'ENOENT')
+                logger_1.Logger.warn(`Unexpected error checking ${entryName}: ${e.message}`);
+        }
         if (!exists) {
             results.push({ entryName, status: 'skipped', note: 'Not found' });
             continue;
@@ -200,157 +168,68 @@ function removeTrackedSymlinks(tracked, skillsDir) {
     }
     return results;
 }
-// ─── Claude Code ────────────────────────────────────────────────────────────
 /**
- * Symlinks each skill directory into ~/.claude/skills/<pkgName>-<skillName>
- * so Claude Code can discover them as invocable skills.
+ * Generic skill sync for any agent in AGENT_SKILL_REGISTRY.
  */
-function syncSkillsToClaudeCode(skills, brewRoot) {
-    const claudeDir = path_1.default.join(os_1.default.homedir(), '.claude');
-    if (!fs_1.default.existsSync(claudeDir))
+function syncSkillsToAgent(agent, skills, brewRoot) {
+    const rootDir = agent.agentRootDir?.();
+    if (rootDir && !fs_1.default.existsSync(rootDir))
         return [];
-    const skillsDir = path_1.default.join(claudeDir, 'skills');
+    const skillsDir = agent.skillsDir();
     fs_1.default.mkdirSync(skillsDir, { recursive: true });
-    const state = loadSyncedState(brewRoot);
-    return symlinkSkills(skills, skillsDir, state, 'claude', brewRoot);
+    const state = (0, sync_state_1.loadSyncedState)(brewRoot);
+    const results = symlinkSkills(skills, skillsDir, state, agent.key, brewRoot);
+    if (results.some(r => r.status === 'linked' || r.status === 'already_exists')) {
+        agent.onAfterSync?.();
+    }
+    return results;
 }
 /**
- * Removes all skill symlinks previously created by syncSkillsToClaudeCode.
+ * Generic skill unsync for any agent in AGENT_SKILL_REGISTRY.
  */
-function unsyncSkillsFromClaudeCode(brewRoot) {
-    const skillsDir = path_1.default.join(os_1.default.homedir(), '.claude', 'skills');
-    const state = loadSyncedState(brewRoot);
-    const results = removeTrackedSymlinks(state.claude, skillsDir);
-    state.claude = [];
-    saveSyncedState(state, brewRoot);
+function unsyncSkillsFromAgent(agent, brewRoot) {
+    const skillsDir = agent.skillsDir();
+    const state = (0, sync_state_1.loadSyncedState)(brewRoot);
+    const results = removeTrackedSymlinks(state[agent.key], skillsDir);
+    state[agent.key] = [];
+    (0, sync_state_1.saveSyncedState)(state, brewRoot);
+    agent.onAfterUnsync?.();
     return results;
 }
-// ─── Gemini CLI ──────────────────────────────────────────────────────────────
-/**
- * Registers skills with Gemini CLI by creating an agentbrew extension at
- * ~/.gemini/extensions/agentbrew/ and symlinking each skill's directory into
- * ~/.gemini/extensions/agentbrew/skills/<pkgName>-<skillName>.
- */
-function syncSkillsToGeminiCLI(skills, brewRoot) {
-    const geminiDir = path_1.default.join(os_1.default.homedir(), '.gemini');
-    if (!fs_1.default.existsSync(geminiDir))
-        return [];
-    const extensionDir = path_1.default.join(geminiDir, 'extensions', AGENTBREW_EXTENSION_NAME);
-    const skillsDir = path_1.default.join(extensionDir, 'skills');
-    fs_1.default.mkdirSync(skillsDir, { recursive: true });
-    // Write extension manifest (we own this file)
-    const manifestPath = path_1.default.join(extensionDir, 'gemini-extension.json');
-    fs_1.default.writeFileSync(manifestPath, JSON.stringify({ name: AGENTBREW_EXTENSION_NAME, version: package_json_1.default.version }, null, 2), 'utf-8');
-    // Enable extension in extension-enablement.json
-    _enableGeminiExtension(geminiDir);
-    const state = loadSyncedState(brewRoot);
-    return symlinkSkills(skills, skillsDir, state, 'gemini', brewRoot);
+// ─── Claude Code ────────────────────────────────────────────────────────────
+function syncSkillsToClaudeCode(skills, brewRoot) {
+    return syncSkillsToAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'claude'), skills, brewRoot);
 }
-/**
- * Removes Gemini CLI skill symlinks, the extension manifest, and the
- * agentbrew extension entry from extension-enablement.json.
- */
-function unsyncSkillsFromGeminiCLI(brewRoot) {
-    const geminiDir = path_1.default.join(os_1.default.homedir(), '.gemini');
-    const extensionDir = path_1.default.join(geminiDir, 'extensions', AGENTBREW_EXTENSION_NAME);
-    const skillsDir = path_1.default.join(extensionDir, 'skills');
-    const state = loadSyncedState(brewRoot);
-    const results = removeTrackedSymlinks(state.gemini, skillsDir);
-    // Clean up extension dir (best-effort; ignores non-empty)
-    try {
-        fs_1.default.rmSync(path_1.default.join(extensionDir, 'gemini-extension.json'), { force: true });
-    }
-    catch { }
-    try {
-        fs_1.default.rmdirSync(skillsDir);
-    }
-    catch { }
-    try {
-        fs_1.default.rmdirSync(extensionDir);
-    }
-    catch { }
-    _disableGeminiExtension(geminiDir);
-    state.gemini = [];
-    saveSyncedState(state, brewRoot);
-    return results;
+function unsyncSkillsFromClaudeCode(brewRoot) {
+    return unsyncSkillsFromAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'claude'), brewRoot);
 }
-function _enableGeminiExtension(geminiDir) {
-    const enablementPath = path_1.default.join(geminiDir, 'extensions', 'extension-enablement.json');
-    let data = {};
-    try {
-        data = JSON.parse(fs_1.default.readFileSync(enablementPath, 'utf-8'));
-    }
-    catch { }
-    if (!data[AGENTBREW_EXTENSION_NAME]) {
-        data[AGENTBREW_EXTENSION_NAME] = { overrides: [`${os_1.default.homedir()}/*`] };
-        fs_1.default.writeFileSync(enablementPath, JSON.stringify(data, null, 2), 'utf-8');
-    }
+// ─── Gemini CLI ───────────────────────────────────────────────────────────────
+function syncSkillsToGeminiCLI(skills, brewRoot) {
+    return syncSkillsToAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'gemini'), skills, brewRoot);
 }
-function _disableGeminiExtension(geminiDir) {
-    const enablementPath = path_1.default.join(geminiDir, 'extensions', 'extension-enablement.json');
-    try {
-        const data = JSON.parse(fs_1.default.readFileSync(enablementPath, 'utf-8'));
-        if (AGENTBREW_EXTENSION_NAME in data) {
-            delete data[AGENTBREW_EXTENSION_NAME];
-            if (Object.keys(data).length === 0) {
-                fs_1.default.rmSync(enablementPath, { force: true });
-            }
-            else {
-                fs_1.default.writeFileSync(enablementPath, JSON.stringify(data, null, 2), 'utf-8');
-            }
-        }
-    }
-    catch { }
+function unsyncSkillsFromGeminiCLI(brewRoot) {
+    return unsyncSkillsFromAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'gemini'), brewRoot);
 }
-// ─── Windsurf ────────────────────────────────────────────────────────────────
-/**
- * Symlinks each skill directory into ~/.codeium/windsurf/skills/<pkgName>-<skillName>
- * so Windsurf can discover them.
- */
+// ─── Windsurf ─────────────────────────────────────────────────────────────────
 function syncSkillsToWindsurf(skills, brewRoot) {
-    const windsurfDir = path_1.default.join(os_1.default.homedir(), '.codeium', 'windsurf');
-    if (!fs_1.default.existsSync(windsurfDir))
-        return [];
-    const skillsDir = path_1.default.join(windsurfDir, 'skills');
-    fs_1.default.mkdirSync(skillsDir, { recursive: true });
-    const state = loadSyncedState(brewRoot);
-    return symlinkSkills(skills, skillsDir, state, 'windsurf', brewRoot);
+    return syncSkillsToAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'windsurf'), skills, brewRoot);
 }
-/**
- * Removes all Windsurf skill symlinks previously created by syncSkillsToWindsurf.
- */
 function unsyncSkillsFromWindsurf(brewRoot) {
-    const skillsDir = path_1.default.join(os_1.default.homedir(), '.codeium', 'windsurf', 'skills');
-    const state = loadSyncedState(brewRoot);
-    const results = removeTrackedSymlinks(state.windsurf, skillsDir);
-    state.windsurf = [];
-    saveSyncedState(state, brewRoot);
-    return results;
+    return unsyncSkillsFromAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'windsurf'), brewRoot);
 }
-// ─── Antigravity CLI ─────────────────────────────────────────────────────────
-/**
- * Symlinks each skill directory into ~/.gemini/antigravity-cli/skills/<pkgName>-<skillName>
- * so Antigravity CLI can auto-discover them.
- */
+// ─── Antigravity CLI ──────────────────────────────────────────────────────────
 function syncSkillsToAntigravityCLI(skills, brewRoot) {
-    const antigravityDir = path_1.default.join(os_1.default.homedir(), '.gemini', 'antigravity-cli');
-    if (!fs_1.default.existsSync(antigravityDir))
-        return [];
-    const skillsDir = path_1.default.join(antigravityDir, 'skills');
-    fs_1.default.mkdirSync(skillsDir, { recursive: true });
-    const state = loadSyncedState(brewRoot);
-    return symlinkSkills(skills, skillsDir, state, 'antigravity', brewRoot);
+    return syncSkillsToAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'antigravity'), skills, brewRoot);
 }
-/**
- * Removes all Antigravity CLI skill symlinks previously created by syncSkillsToAntigravityCLI.
- */
 function unsyncSkillsFromAntigravityCLI(brewRoot) {
-    const skillsDir = path_1.default.join(os_1.default.homedir(), '.gemini', 'antigravity-cli', 'skills');
-    const state = loadSyncedState(brewRoot);
-    const results = removeTrackedSymlinks(state.antigravity, skillsDir);
-    state.antigravity = [];
-    saveSyncedState(state, brewRoot);
-    return results;
+    return unsyncSkillsFromAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'antigravity'), brewRoot);
+}
+// ─── Kiro ─────────────────────────────────────────────────────────────────────
+function syncSkillsToKiro(skills, brewRoot) {
+    return syncSkillsToAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'kiro'), skills, brewRoot);
+}
+function unsyncSkillsFromKiro(brewRoot) {
+    return unsyncSkillsFromAgent(agent_registry_1.AGENT_SKILL_REGISTRY.find(a => a.key === 'kiro'), brewRoot);
 }
 // ─── Orphan cleanup ──────────────────────────────────────────────────────────
 /**
@@ -358,24 +237,21 @@ function unsyncSkillsFromAntigravityCLI(brewRoot) {
  * Call after `agentbrew uninstall` to prevent stale entries.
  */
 function cleanOrphanSkills(brewRoot) {
-    const state = loadSyncedState(brewRoot);
+    const state = (0, sync_state_1.loadSyncedState)(brewRoot);
     const results = [];
-    const agentDirs = [
-        { key: 'claude', dir: path_1.default.join(os_1.default.homedir(), '.claude', 'skills') },
-        { key: 'gemini', dir: path_1.default.join(os_1.default.homedir(), '.gemini', 'extensions', AGENTBREW_EXTENSION_NAME, 'skills') },
-        { key: 'windsurf', dir: path_1.default.join(os_1.default.homedir(), '.codeium', 'windsurf', 'skills') },
-        { key: 'antigravity', dir: path_1.default.join(os_1.default.homedir(), '.gemini', 'antigravity-cli', 'skills') },
-        { key: 'kiro', dir: path_1.default.join(os_1.default.homedir(), '.kiro', 'skills') },
-    ];
-    for (const { key, dir } of agentDirs) {
+    for (const agent of agent_registry_1.AGENT_SKILL_REGISTRY) {
+        const dir = agent.skillsDir();
         const remaining = [];
-        for (const entryName of state[key]) {
+        for (const entryName of state[agent.key]) {
             const entryPath = path_1.default.join(dir, entryName);
             let symlinkTarget = null;
             try {
                 symlinkTarget = fs_1.default.readlinkSync(entryPath);
             }
-            catch { }
+            catch (e) {
+                if (e.code !== 'ENOENT')
+                    logger_1.Logger.warn(`Unexpected error checking ${entryName}: ${e.message}`);
+            }
             if (symlinkTarget !== null && !fs_1.default.existsSync(symlinkTarget)) {
                 try {
                     fs_1.default.rmSync(entryPath, { force: true });
@@ -390,7 +266,7 @@ function cleanOrphanSkills(brewRoot) {
                 remaining.push(entryName);
             }
         }
-        state[key] = remaining;
+        state[agent.key] = remaining;
     }
     // Handle Cursor index file: parse referenced SKILL.md paths and remove the file if any are stale
     if (state.cursor) {
@@ -418,190 +294,29 @@ function cleanOrphanSkills(brewRoot) {
             }
         }
     }
-    if (state.codexMcp && !fs_1.default.existsSync(path_1.default.join(os_1.default.homedir(), '.codex'))) {
-        state.codexMcp = false;
-    }
-    if (state.kiroMcp && !fs_1.default.existsSync(path_1.default.join(os_1.default.homedir(), '.kiro'))) {
-        state.kiroMcp = false;
-    }
-    saveSyncedState(state, brewRoot);
-    return results;
-}
-// ─── Cursor MCP server registration ─────────────────────────────────────────
-const CURSOR_MCP_ENTRY = 'agentbrew';
-/**
- * Adds agentbrew to ~/.cursor/mcp.json so Cursor can discover MCP tools directly.
- * Merges into any existing config without disturbing other servers.
- */
-function syncMcpServerToCursor(brewRoot) {
-    const cursorDir = path_1.default.join(os_1.default.homedir(), '.cursor');
-    if (!fs_1.default.existsSync(cursorDir))
-        return [];
-    const mcpJsonPath = path_1.default.join(cursorDir, 'mcp.json');
-    const entryName = 'agentbrew (Cursor MCP)';
-    let config = {};
-    try {
-        config = JSON.parse(fs_1.default.readFileSync(mcpJsonPath, 'utf-8'));
-    }
-    catch { }
-    const mcpServers = config.mcpServers ?? {};
-    const existing = mcpServers[CURSOR_MCP_ENTRY];
-    if (existing?.command === 'agentbrew') {
-        const state = loadSyncedState(brewRoot);
-        state.cursorMcp = true;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'already_exists', path: mcpJsonPath }];
-    }
-    config.mcpServers = { ...mcpServers, [CURSOR_MCP_ENTRY]: { command: 'agentbrew' } };
-    try {
-        fs_1.default.writeFileSync(mcpJsonPath, JSON.stringify(config, null, 2), 'utf-8');
-        const state = loadSyncedState(brewRoot);
-        state.cursorMcp = true;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'linked', path: mcpJsonPath }];
-    }
-    catch (e) {
-        return [{ entryName, status: 'error', note: e.message }];
-    }
-}
-/**
- * Removes the agentbrew entry from ~/.cursor/mcp.json.
- * Leaves other servers intact; removes the file only if it becomes empty.
- */
-function unsyncMcpServerFromCursor(brewRoot) {
-    const state = loadSyncedState(brewRoot);
-    if (!state.cursorMcp)
-        return [];
-    const mcpJsonPath = path_1.default.join(os_1.default.homedir(), '.cursor', 'mcp.json');
-    const entryName = 'agentbrew (Cursor MCP)';
-    let config = {};
-    try {
-        config = JSON.parse(fs_1.default.readFileSync(mcpJsonPath, 'utf-8'));
-    }
-    catch {
-        state.cursorMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'skipped', note: 'Not found' }];
-    }
-    if (!config.mcpServers?.[CURSOR_MCP_ENTRY]) {
-        state.cursorMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'skipped', note: 'Not found' }];
-    }
-    delete config.mcpServers[CURSOR_MCP_ENTRY];
-    if (Object.keys(config.mcpServers).length === 0)
-        delete config.mcpServers;
-    try {
-        if (Object.keys(config).length === 0) {
-            fs_1.default.rmSync(mcpJsonPath, { force: true });
-        }
-        else {
-            fs_1.default.writeFileSync(mcpJsonPath, JSON.stringify(config, null, 2), 'utf-8');
-        }
-        state.cursorMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'removed', path: mcpJsonPath }];
-    }
-    catch (e) {
-        return [{ entryName, status: 'error', note: e.message }];
-    }
-}
-// ─── Codex MCP server registration ──────────────────────────────────────────
-function _removeTomlSection(content, sectionHeader) {
-    const lines = content.split('\n');
-    const headerLine = `[${sectionHeader}]`;
-    const startIdx = lines.findIndex(l => l.trim() === headerLine);
-    if (startIdx === -1)
-        return content;
-    let endIdx = lines.length;
-    for (let i = startIdx + 1; i < lines.length; i++) {
-        if (lines[i].trim().startsWith('[')) {
-            endIdx = i;
-            break;
+    if (state.codexMcp) {
+        const codexConfig = path_1.default.join(os_1.default.homedir(), '.codex', 'config.toml');
+        let codexEntryPresent = false;
+        try {
+            const raw = fs_1.default.readFileSync(codexConfig, 'utf-8');
+            codexEntryPresent = toml.parse(raw)?.mcp_servers?.agentbrew?.command === 'agentbrew';
         }
+        catch { }
+        if (!codexEntryPresent)
+            state.codexMcp = false;
     }
-    // Also absorb a preceding blank line
-    let removeFrom = startIdx;
-    if (removeFrom > 0 && lines[removeFrom - 1].trim() === '')
-        removeFrom--;
-    return [...lines.slice(0, removeFrom), ...lines.slice(endIdx)].join('\n');
-}
-/**
- * Adds agentbrew to ~/.codex/config.toml so Codex CLI can discover MCP tools directly.
- * Appends the [mcp_servers.agentbrew] section without disturbing existing content.
- */
-function syncMcpServerToCodex(brewRoot) {
-    const codexDir = path_1.default.join(os_1.default.homedir(), '.codex');
-    if (!fs_1.default.existsSync(codexDir))
-        return [];
-    const configPath = path_1.default.join(codexDir, 'config.toml');
-    const entryName = 'agentbrew (Codex MCP)';
-    let raw = '';
-    try {
-        raw = fs_1.default.readFileSync(configPath, 'utf-8');
-    }
-    catch { }
-    // Check if already registered with the right command
-    try {
-        const parsed = toml.parse(raw);
-        if (parsed?.mcp_servers?.agentbrew?.command === 'agentbrew') {
-            const state = loadSyncedState(brewRoot);
-            state.codexMcp = true;
-            saveSyncedState(state, brewRoot);
-            return [{ entryName, status: 'already_exists', path: configPath }];
+    if (state.kiroMcp) {
+        const kiroConfig = path_1.default.join(os_1.default.homedir(), '.kiro', 'settings', 'mcp.json');
+        let kiroEntryPresent = false;
+        try {
+            kiroEntryPresent = JSON.parse(fs_1.default.readFileSync(kiroConfig, 'utf-8'))?.mcpServers?.agentbrew?.command === 'agentbrew';
         }
+        catch { }
+        if (!kiroEntryPresent)
+            state.kiroMcp = false;
     }
-    catch { }
-    // Remove any stale entry (e.g. wrong command) before re-adding to avoid duplicate TOML table headers
-    const cleaned = _removeTomlSection(raw, 'mcp_servers.agentbrew');
-    const sep = cleaned.length > 0 && !cleaned.endsWith('\n') ? '\n' : '';
-    const newContent = cleaned + sep + '\n[mcp_servers.agentbrew]\ncommand = "agentbrew"\n';
-    try {
-        fs_1.default.writeFileSync(configPath, newContent, 'utf-8');
-        const state = loadSyncedState(brewRoot);
-        state.codexMcp = true;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'linked', path: configPath }];
-    }
-    catch (e) {
-        return [{ entryName, status: 'error', note: e.message }];
-    }
-}
-/**
- * Removes the agentbrew entry from ~/.codex/config.toml.
- * Leaves the file in place; other sections are untouched.
- */
-function unsyncMcpServerFromCodex(brewRoot) {
-    const state = loadSyncedState(brewRoot);
-    if (!state.codexMcp)
-        return [];
-    const configPath = path_1.default.join(os_1.default.homedir(), '.codex', 'config.toml');
-    const entryName = 'agentbrew (Codex MCP)';
-    let raw;
-    try {
-        raw = fs_1.default.readFileSync(configPath, 'utf-8');
-    }
-    catch {
-        state.codexMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'skipped', note: 'Not found' }];
-    }
-    const cleaned = _removeTomlSection(raw, 'mcp_servers.agentbrew');
-    if (cleaned === raw) {
-        state.codexMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'skipped', note: 'Not found' }];
-    }
-    try {
-        const trimmed = cleaned.trimEnd();
-        fs_1.default.writeFileSync(configPath, trimmed ? trimmed + '\n' : '', 'utf-8');
-        state.codexMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'removed', path: configPath }];
-    }
-    catch (e) {
-        return [{ entryName, status: 'error', note: e.message }];
-    }
+    (0, sync_state_1.saveSyncedState)(state, brewRoot);
+    return results;
 }
 // ─── Cursor ──────────────────────────────────────────────────────────────────
 function buildCursorSkillsIndex(skills) {
@@ -638,11 +353,11 @@ function syncSkillsToCursor(skills, brewRoot) {
     fs_1.default.mkdirSync(rulesDir, { recursive: true });
     const indexPath = path_1.default.join(rulesDir, CURSOR_SKILLS_INDEX_FILE);
     const content = buildCursorSkillsIndex(skills);
-    const state = loadSyncedState(brewRoot);
+    const state = (0, sync_state_1.loadSyncedState)(brewRoot);
     try {
         fs_1.default.writeFileSync(indexPath, content, 'utf-8');
         state.cursor = true;
-        saveSyncedState(state, brewRoot);
+        (0, sync_state_1.saveSyncedState)(state, brewRoot);
         return [{ entryName: CURSOR_SKILLS_INDEX_FILE, status: 'linked', path: indexPath }];
     }
     catch (e) {
@@ -653,7 +368,7 @@ function syncSkillsToCursor(skills, brewRoot) {
  * Removes the AgentBrew skills index file from ~/.cursor/rules/.
  */
 function unsyncSkillsFromCursor(brewRoot) {
-    const state = loadSyncedState(brewRoot);
+    const state = (0, sync_state_1.loadSyncedState)(brewRoot);
     if (!state.cursor)
         return [];
     const indexPath = path_1.default.join(os_1.default.homedir(), '.cursor', 'rules', CURSOR_SKILLS_INDEX_FILE);
@@ -662,322 +377,43 @@ function unsyncSkillsFromCursor(brewRoot) {
         fs_1.default.lstatSync(indexPath);
         exists = true;
     }
-    catch { }
+    catch (e) {
+        if (e.code !== 'ENOENT')
+            logger_1.Logger.warn(`Unexpected error checking ${CURSOR_SKILLS_INDEX_FILE}: ${e.message}`);
+    }
     if (!exists) {
         state.cursor = false;
-        saveSyncedState(state, brewRoot);
+        (0, sync_state_1.saveSyncedState)(state, brewRoot);
         return [{ entryName: CURSOR_SKILLS_INDEX_FILE, status: 'skipped', note: 'Not found' }];
     }
     try {
         fs_1.default.rmSync(indexPath, { force: true });
         state.cursor = false;
-        saveSyncedState(state, brewRoot);
+        (0, sync_state_1.saveSyncedState)(state, brewRoot);
         return [{ entryName: CURSOR_SKILLS_INDEX_FILE, status: 'removed', path: indexPath }];
     }
     catch (e) {
         return [{ entryName: CURSOR_SKILLS_INDEX_FILE, status: 'error', note: e.message }];
     }
 }
-// ─── Kiro ────────────────────────────────────────────────────────────────────
-/**
- * Symlinks each skill directory into ~/.kiro/skills/<pkgName>-<skillName>
- * so Kiro can discover them as invocable skills.
- */
-function syncSkillsToKiro(skills, brewRoot) {
-    const kiroDir = path_1.default.join(os_1.default.homedir(), '.kiro');
-    if (!fs_1.default.existsSync(kiroDir))
-        return [];
-    const skillsDir = path_1.default.join(kiroDir, 'skills');
-    fs_1.default.mkdirSync(skillsDir, { recursive: true });
-    const state = loadSyncedState(brewRoot);
-    return symlinkSkills(skills, skillsDir, state, 'kiro', brewRoot);
-}
-/**
- * Removes all Kiro skill symlinks previously created by syncSkillsToKiro.
- */
-function unsyncSkillsFromKiro(brewRoot) {
-    const skillsDir = path_1.default.join(os_1.default.homedir(), '.kiro', 'skills');
-    const state = loadSyncedState(brewRoot);
-    const results = removeTrackedSymlinks(state.kiro, skillsDir);
-    state.kiro = [];
-    saveSyncedState(state, brewRoot);
-    return results;
-}
-// ─── Kiro MCP server registration ────────────────────────────────────────────
-const KIRO_MCP_ENTRY = 'agentbrew';
-/**
- * Adds agentbrew to ~/.kiro/settings/mcp.json so Kiro can discover MCP tools directly.
- * Merges into any existing config without disturbing other servers.
- */
-function syncMcpServerToKiro(brewRoot) {
-    const kiroDir = path_1.default.join(os_1.default.homedir(), '.kiro');
-    if (!fs_1.default.existsSync(kiroDir))
-        return [];
-    const settingsDir = path_1.default.join(kiroDir, 'settings');
-    fs_1.default.mkdirSync(settingsDir, { recursive: true });
-    const mcpJsonPath = path_1.default.join(settingsDir, 'mcp.json');
-    const entryName = 'agentbrew (Kiro MCP)';
-    let config = {};
-    try {
-        config = JSON.parse(fs_1.default.readFileSync(mcpJsonPath, 'utf-8'));
-    }
-    catch { }
-    const mcpServers = config.mcpServers ?? {};
-    const existing = mcpServers[KIRO_MCP_ENTRY];
-    if (existing?.command === 'agentbrew') {
-        const state = loadSyncedState(brewRoot);
-        state.kiroMcp = true;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'already_exists', path: mcpJsonPath }];
-    }
-    config.mcpServers = { ...mcpServers, [KIRO_MCP_ENTRY]: { command: 'agentbrew' } };
-    try {
-        fs_1.default.writeFileSync(mcpJsonPath, JSON.stringify(config, null, 2), 'utf-8');
-        const state = loadSyncedState(brewRoot);
-        state.kiroMcp = true;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'linked', path: mcpJsonPath }];
-    }
-    catch (e) {
-        return [{ entryName, status: 'error', note: e.message }];
-    }
-}
-/**
- * Removes the agentbrew entry from ~/.kiro/settings/mcp.json.
- * Leaves other servers intact; removes the file only if it becomes empty.
- */
-function unsyncMcpServerFromKiro(brewRoot) {
-    const state = loadSyncedState(brewRoot);
-    if (!state.kiroMcp)
-        return [];
-    const mcpJsonPath = path_1.default.join(os_1.default.homedir(), '.kiro', 'settings', 'mcp.json');
-    const entryName = 'agentbrew (Kiro MCP)';
-    let config = {};
-    try {
-        config = JSON.parse(fs_1.default.readFileSync(mcpJsonPath, 'utf-8'));
-    }
-    catch {
-        state.kiroMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'skipped', note: 'Not found' }];
-    }
-    if (!config.mcpServers?.[KIRO_MCP_ENTRY]) {
-        state.kiroMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'skipped', note: 'Not found' }];
-    }
-    delete config.mcpServers[KIRO_MCP_ENTRY];
-    if (Object.keys(config.mcpServers).length === 0)
-        delete config.mcpServers;
-    try {
-        if (Object.keys(config).length === 0) {
-            fs_1.default.rmSync(mcpJsonPath, { force: true });
-        }
-        else {
-            fs_1.default.writeFileSync(mcpJsonPath, JSON.stringify(config, null, 2), 'utf-8');
-        }
-        state.kiroMcp = false;
-        saveSyncedState(state, brewRoot);
-        return [{ entryName, status: 'removed', path: mcpJsonPath }];
-    }
-    catch (e) {
-        return [{ entryName, status: 'error', note: e.message }];
-    }
-}
-// ─── Instruction sync (unchanged) ───────────────────────────────────────────
-exports.MARKER_START = '<!-- agentbrew:shared:start -->';
-exports.MARKER_END = '<!-- agentbrew:shared:end -->';
-exports.INSTRUCTIONS_FILE = 'INSTRUCTIONS.md';
-const EXAMPLE_INSTRUCTIONS = `# AgentBrew Shared Instructions
-
-These instructions are shared across all your AI agents via AgentBrew.
-Edit this file and run \`agentbrew sync\` to push updates to all agent configs.
-
-## Example: API Usage Policy
-- Always use Context7 (the \`context7\` MCP tool) to fetch live API documentation
-  before writing code that calls an external library. This prevents using stale or
-  hallucinated function signatures.
-
-## Notes
-- This file is global. For project-specific context, add it directly to the
-  project's CLAUDE.md or GEMINI.md — or reference project docs from within this file.
-`;
-function getDefaultTargets() {
-    const home = os_1.default.homedir();
-    return [
-        {
-            name: 'Claude Code',
-            configPath: path_1.default.join(home, '.claude', 'CLAUDE.md'),
-            isFileOwned: false,
-        },
-        {
-            name: 'Gemini CLI',
-            configPath: path_1.default.join(home, '.gemini', 'GEMINI.md'),
-            isFileOwned: false,
-        },
-        {
-            name: 'OpenAI Codex CLI',
-            // Codex CLI's canonical instruction file is AGENTS.md (instructions.md is a legacy fallback)
-            configPath: path_1.default.join(home, '.codex', 'AGENTS.md'),
-            isFileOwned: false,
-        },
-        {
-            name: 'Cursor',
-            // Cursor "User Rules" directory (Cursor 0.47+): each .md file in this dir is a global rule.
-            // We own this specific file entirely — no markers needed.
-            configPath: path_1.default.join(home, '.cursor', 'rules', 'agentbrew-shared.md'),
-            isFileOwned: true,
-            agentRootDir: path_1.default.join(home, '.cursor'),
-        },
-        {
-            name: 'Windsurf',
-            configPath: path_1.default.join(home, '.codeium', 'windsurf', 'memories', 'global_rules.md'),
-            isFileOwned: false,
-        },
-        {
-            name: 'Kiro',
-            // Steering files in ~/.kiro/steering/ are auto-loaded in every Kiro interaction.
-            // We own this file entirely and must place the frontmatter at the very top.
-            configPath: path_1.default.join(home, '.kiro', 'steering', 'agentbrew-shared.md'),
-            isFileOwned: true,
-            frontmatter: '---\ninclusion: always\n---',
-            agentRootDir: path_1.default.join(home, '.kiro'),
-        },
-    ];
-}
-function getInstructionsPath(brewRoot) {
-    return path_1.default.join(brewRoot ?? (0, config_1.getBrewRoot)(), exports.INSTRUCTIONS_FILE);
-}
-function buildInjectedSection(content) {
-    const warning = `> ⚠️ Managed by AgentBrew. Edit \`~/.agentbrew/INSTRUCTIONS.md\` and run \`agentbrew sync\` to update.`;
-    return `${exports.MARKER_START}\n${warning}\n\n${content.trim()}\n${exports.MARKER_END}`;
-}
-/**
- * Injects (or updates) the agentbrew section in a file.
- * Creates the file and any parent directories if they don't exist.
- */
-function injectIntoFile(filePath, content) {
-    const section = buildInjectedSection(content);
-    if (!fs_1.default.existsSync(filePath)) {
-        fs_1.default.mkdirSync(path_1.default.dirname(filePath), { recursive: true });
-        fs_1.default.writeFileSync(filePath, section + '\n', 'utf-8');
-        return 'created';
-    }
-    const existing = fs_1.default.readFileSync(filePath, 'utf-8');
-    const startIdx = existing.indexOf(exports.MARKER_START);
-    const endIdx = existing.indexOf(exports.MARKER_END);
-    if (startIdx !== -1 && endIdx !== -1) {
-        const replaced = existing.substring(0, startIdx) +
-            section +
-            existing.substring(endIdx + exports.MARKER_END.length);
-        if (replaced === existing)
-            return 'unchanged';
-        fs_1.default.writeFileSync(filePath, replaced, 'utf-8');
-        return 'updated';
-    }
-    // No markers yet — append to end
-    const appended = existing.trimEnd() + '\n\n' + section + '\n';
-    fs_1.default.writeFileSync(filePath, appended, 'utf-8');
-    return 'updated';
-}
-/**
- * Removes the agentbrew section from a file, leaving surrounding content intact.
- */
-function removeFromFile(filePath) {
-    if (!fs_1.default.existsSync(filePath))
-        return 'not_found';
-    const existing = fs_1.default.readFileSync(filePath, 'utf-8');
-    const startIdx = existing.indexOf(exports.MARKER_START);
-    const endIdx = existing.indexOf(exports.MARKER_END);
-    if (startIdx === -1 || endIdx === -1)
-        return 'no_section';
-    const before = existing.substring(0, startIdx).trimEnd();
-    const after = existing.substring(endIdx + exports.MARKER_END.length).trimStart();
-    let result = before;
-    if (after)
-        result += '\n\n' + after;
-    result = result.trimEnd() + '\n';
-    fs_1.default.writeFileSync(filePath, result, 'utf-8');
-    return 'removed';
-}
-/**
- * Syncs ~/.agentbrew/INSTRUCTIONS.md into each target agent's global config file.
- * Accepts an optional `targets` override (used in tests).
- */
-function syncInstructions(targets, brewRoot) {
-    const instructionsPath = getInstructionsPath(brewRoot);
-    if (!fs_1.default.existsSync(instructionsPath)) {
-        fs_1.default.mkdirSync(path_1.default.dirname(instructionsPath), { recursive: true });
-        fs_1.default.writeFileSync(instructionsPath, EXAMPLE_INSTRUCTIONS, 'utf-8');
-        return [];
-    }
-    const content = fs_1.default.readFileSync(instructionsPath, 'utf-8');
-    const resolvedTargets = targets ?? getDefaultTargets();
-    const results = [];
-    for (const target of resolvedTargets) {
-        if (target.configPath === null) {
-            results.push({ agent: target.name, status: 'manual', note: target.manualInstructions });
-            continue;
-        }
-        if (target.isFileOwned) {
-            // Skip if the agent's root directory doesn't exist (agent not installed).
-            // Without this guard, mkdirSync below would create the directory unconditionally.
-            if (target.agentRootDir && !fs_1.default.existsSync(target.agentRootDir)) {
-                results.push({ agent: target.name, status: 'skipped', note: 'Agent not installed (config directory not found)' });
-                continue;
-            }
-            // We own this file entirely — write raw content, no markers needed.
-            // unsync deletes the file; there is no surrounding user content to delimit around.
-            const header = `> ⚠️ Managed by AgentBrew. Edit \`~/.agentbrew/INSTRUCTIONS.md\` and run \`agentbrew sync\` to update.\n`;
-            const prefix = target.frontmatter ? target.frontmatter + '\n' : '';
-            const fileContent = prefix + header + '\n' + content.trim() + '\n';
-            fs_1.default.mkdirSync(path_1.default.dirname(target.configPath), { recursive: true });
-            const existing = fs_1.default.existsSync(target.configPath)
-                ? fs_1.default.readFileSync(target.configPath, 'utf-8')
-                : null;
-            if (existing === fileContent) {
-                results.push({ agent: target.name, status: 'unchanged', path: target.configPath });
-            }
-            else {
-                fs_1.default.writeFileSync(target.configPath, fileContent, 'utf-8');
-                results.push({ agent: target.name, status: existing !== null ? 'updated' : 'created', path: target.configPath });
-            }
-            continue;
-        }
-        // Skip agents that aren't installed (config parent dir absent)
-        if (!fs_1.default.existsSync(path_1.default.dirname(target.configPath))) {
-            results.push({ agent: target.name, status: 'skipped', note: 'Agent not installed (config directory not found)' });
-            continue;
-        }
-        const status = injectIntoFile(target.configPath, content);
-        results.push({ agent: target.name, status, path: target.configPath });
-    }
-    return results;
-}
-/**
- * Removes the agentbrew section from all target agent config files.
- */
-function unsyncInstructions(targets) {
-    const resolvedTargets = targets ?? getDefaultTargets();
-    const results = [];
-    for (const target of resolvedTargets) {
-        if (target.configPath === null) {
-            results.push({ agent: target.name, status: 'manual', note: 'Remove manually from your agent UI settings.' });
-            continue;
-        }
-        if (target.isFileOwned) {
-            if (fs_1.default.existsSync(target.configPath)) {
-                fs_1.default.rmSync(target.configPath);
-                results.push({ agent: target.name, status: 'removed', path: target.configPath });
-            }
-            else {
-                results.push({ agent: target.name, status: 'not_found', path: target.configPath });
-            }
-            continue;
-        }
-        const status = removeFromFile(target.configPath);
-        results.push({ agent: target.name, status, path: target.configPath });
-    }
-    return results;
-}
+// ─── MCP registration ────────────────────────────────────────────────────────
+var sync_mcp_1 = require("./sync-mcp");
+Object.defineProperty(exports, "syncMcpServerToCursor", { enumerable: true, get: function () { return sync_mcp_1.syncMcpServerToCursor; } });
+Object.defineProperty(exports, "unsyncMcpServerFromCursor", { enumerable: true, get: function () { return sync_mcp_1.unsyncMcpServerFromCursor; } });
+Object.defineProperty(exports, "syncMcpServerToCodex", { enumerable: true, get: function () { return sync_mcp_1.syncMcpServerToCodex; } });
+Object.defineProperty(exports, "unsyncMcpServerFromCodex", { enumerable: true, get: function () { return sync_mcp_1.unsyncMcpServerFromCodex; } });
+Object.defineProperty(exports, "syncMcpServerToKiro", { enumerable: true, get: function () { return sync_mcp_1.syncMcpServerToKiro; } });
+Object.defineProperty(exports, "unsyncMcpServerFromKiro", { enumerable: true, get: function () { return sync_mcp_1.unsyncMcpServerFromKiro; } });
+// ─── Instruction sync ────────────────────────────────────────────────────────
+var sync_instructions_1 = require("./sync-instructions");
+Object.defineProperty(exports, "MARKER_START", { enumerable: true, get: function () { return sync_instructions_1.MARKER_START; } });
+Object.defineProperty(exports, "MARKER_END", { enumerable: true, get: function () { return sync_instructions_1.MARKER_END; } });
+Object.defineProperty(exports, "INSTRUCTIONS_FILE", { enumerable: true, get: function () { return sync_instructions_1.INSTRUCTIONS_FILE; } });
+Object.defineProperty(exports, "getDefaultTargets", { enumerable: true, get: function () { return sync_instructions_1.getDefaultTargets; } });
+Object.defineProperty(exports, "getInstructionsPath", { enumerable: true, get: function () { return sync_instructions_1.getInstructionsPath; } });
+Object.defineProperty(exports, "buildInjectedSection", { enumerable: true, get: function () { return sync_instructions_1.buildInjectedSection; } });
+Object.defineProperty(exports, "injectIntoFile", { enumerable: true, get: function () { return sync_instructions_1.injectIntoFile; } });
+Object.defineProperty(exports, "removeFromFile", { enumerable: true, get: function () { return sync_instructions_1.removeFromFile; } });
+Object.defineProperty(exports, "syncInstructions", { enumerable: true, get: function () { return sync_instructions_1.syncInstructions; } });
+Object.defineProperty(exports, "unsyncInstructions", { enumerable: true, get: function () { return sync_instructions_1.unsyncInstructions; } });
 //# sourceMappingURL=sync.js.map