agent-switchboard 0.2.3 → 0.3.0

package/dist/index.js

@@ -10,7 +10,6 @@ import { confirm } from '@inquirer/prompts';
 import chalk from 'chalk';
 import { Command } from 'commander';
 import ora from 'ora';
-import { getAgentById } from './agents/registry.js';
 import { distributeCommands } from './commands/distribution.js';
 import { importCommandFromFile } from './commands/importer.js';
 import { buildCommandInventory } from './commands/inventory.js';
@@ -21,12 +20,11 @@ import { getAgentsDir, getAgentsHome, getClaudeDir, getCodexDir, getCommandsDir,
 import { loadSwitchboardConfig, loadSwitchboardConfigWithLayers, } from './config/switchboard-config.js';
 import { distributeHooks } from './hooks/distribution.js';
 import { loadHookLibrary } from './hooks/library.js';
-import { ensureLibraryDirectories, writeFileSecure } from './library/fs.js';
+import { copyDirRecursive, ensureLibraryDirectories, isDir, isFile, listFilesRecursively, writeFileSecure, } from './library/fs.js';
 import { addLocalSource, addRemoteSource, getSources, inferSourceName, isGitUrl, parseGitUrl, removeSource, updateRemoteSources, validateSourcePath, } from './library/sources.js';
 import { loadLibraryStateSection, loadMcpEnabledState, saveMcpEnabledState, } from './library/state.js';
 import { readMarketplace } from './marketplace/reader.js';
 import { buildPluginIndex } from './plugins/index.js';
-import { RULE_SUPPORTED_AGENTS } from './rules/agents.js';
 import { composeActiveRules } from './rules/composer.js';
 import { distributeRules, listIndirectAgents, listPerFileAgents, listUnsupportedAgents, } from './rules/distribution.js';
 import { buildRuleInventory } from './rules/inventory.js';
@@ -38,6 +36,8 @@ import { buildSkillInventory } from './skills/inventory.js';
 import { distributeSubagents } from './subagents/distribution.js';
 import { importSubagentFromFile } from './subagents/importer.js';
 import { buildSubagentInventory } from './subagents/inventory.js';
+import { initTargets } from './targets/init.js';
+import { getTargetById, getTargetsForSection } from './targets/registry.js';
 import { showCommandSelector } from './ui/command-ui.js';
 import { showHookSelector } from './ui/hook-ui.js';
 import { showMcpServerUI } from './ui/mcp-ui.js';
@@ -75,6 +75,7 @@ program
         const scope = resolveScope(options);
         const loadOptions = scopeToLoadOptions(scope);
         const { config, layers } = loadSwitchboardConfigWithLayers(loadOptions);
+        await initTargets(config);
         console.log(chalk.yellow('⚠ Sync overwrites agent config without diff.'));
         console.log();
         if (options.update !== false) {
@@ -110,26 +111,14 @@ program
         console.log(chalk.blue('Inventory:'));
         {
             const sections = ['mcp', 'rules', 'commands', 'agents', 'skills', 'hooks'];
-            // Keep in sync with platform lists in each distribution module
-            const sectionPlatforms = {
-                mcp: [
-                    'claude-code',
-                    'claude-desktop',
-                    'codex',
-                    'cursor',
-                    'gemini',
-                    'opencode',
-                    'trae',
-                    'trae-cn',
-                ],
-                rules: ['claude-code', 'codex', 'cursor', 'gemini', 'opencode', 'trae', 'trae-cn'],
-                commands: ['claude-code', 'codex', 'cursor', 'gemini', 'opencode'],
-                agents: ['claude-code', 'opencode', 'cursor'],
-                skills: cursorSkillsDeduped
-                    ? ['claude-code', 'codex', 'gemini', 'opencode', 'trae', 'trae-cn']
-                    : ['claude-code', 'codex', 'gemini', 'opencode', 'cursor', 'trae', 'trae-cn'],
-                hooks: ['claude-code'],
-            };
+            const sectionPlatforms = {};
+            for (const s of sections) {
+                let ids = getTargetsForSection(s).map((t) => t.id);
+                if (s === 'skills' && cursorSkillsDeduped) {
+                    ids = ids.filter((id) => id !== 'cursor');
+                }
+                sectionPlatforms[s] = ids;
+            }
             const termWidth = process.stdout.columns || 80;
             const maxSectionLen = Math.max(...sections.map((s) => s.length));
             const maxCountLen = Math.max(...sections.map((s) => `(${config[s].enabled.length})`.length));
@@ -194,9 +183,7 @@ program
         // Show enabled plugins summary
         {
             const pluginIndex = buildPluginIndex();
-            const enabledPluginRefs = Object.entries(config.plugins.enabled)
-                .filter(([, v]) => v === true)
-                .map(([k]) => k);
+            const enabledPluginRefs = config.plugins.enabled;
             if (enabledPluginRefs.length > 0) {
                 const names = enabledPluginRefs
                     .map((pid) => {
@@ -295,22 +282,6 @@ program
         process.exit(1);
     }
 });
-function isDir(p) {
-    try {
-        return fs.existsSync(p) && fs.statSync(p).isDirectory();
-    }
-    catch {
-        return false;
-    }
-}
-function isFile(p) {
-    try {
-        return fs.existsSync(p) && fs.statSync(p).isFile();
-    }
-    catch {
-        return false;
-    }
-}
 function resolveScope(input) {
     if (!input)
         return undefined;
@@ -377,26 +348,6 @@ function defaultSkillSourceDir(platform) {
             return path.join(getCursorDir(), 'skills');
     }
 }
-function listFilesRecursively(root, filterExts) {
-    const out = [];
-    const exts = new Set(filterExts.map((e) => e.toLowerCase()));
-    const walk = (dir) => {
-        const entries = fs.readdirSync(dir, { withFileTypes: true });
-        for (const entry of entries) {
-            const abs = path.join(dir, entry.name);
-            if (entry.isDirectory()) {
-                walk(abs);
-            }
-            else if (entry.isFile()) {
-                const ext = path.extname(entry.name).toLowerCase();
-                if (exts.has(ext))
-                    out.push(abs);
-            }
-        }
-    };
-    walk(root);
-    return out;
-}
 async function confirmOverwrite(filePath, force) {
     if (!fs.existsSync(filePath))
         return true;
@@ -404,29 +355,6 @@ async function confirmOverwrite(filePath, force) {
         return true;
     return await confirm({ message: `File exists: ${filePath}. Overwrite?`, default: false });
 }
-function copyDirRecursive(src, dest) {
-    fs.mkdirSync(dest, { recursive: true });
-    const entries = fs.readdirSync(src, { withFileTypes: true });
-    for (const entry of entries) {
-        const srcPath = path.join(src, entry.name);
-        const destPath = path.join(dest, entry.name);
-        if (entry.isDirectory()) {
-            copyDirRecursive(srcPath, destPath);
-        }
-        else {
-            fs.copyFileSync(srcPath, destPath);
-            // Preserve executable permissions
-            try {
-                const mode = fs.statSync(srcPath).mode;
-                if (mode & 0o111)
-                    fs.chmodSync(destPath, mode & 0o777);
-            }
-            catch {
-                // Ignore
-            }
-        }
-    }
-}
 /**
  * Extract hooks from Claude Code's ~/.claude/settings.json and import as a
  * bundle into ~/.asb/hooks/. Copies referenced script files if they exist
@@ -662,7 +590,7 @@ ruleCommand
         console.log();
         printAgentSyncStatus({
             agentSync: inventory.state.agentSync,
-            agents: RULE_SUPPORTED_AGENTS,
+            agents: getTargetsForSection('rules').map((t) => t.id),
         });
         const unsupportedAgents = listUnsupportedAgents();
         if (unsupportedAgents.length > 0) {
@@ -689,6 +617,7 @@ ruleCommand.action(async (options) => {
     try {
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
+        await initTargets(config);
         const selection = await showRuleSelector({ scope, pageSize: config.ui.page_size });
         if (!selection) {
             return;
@@ -777,6 +706,7 @@ commandRoot.action(async (options) => {
     try {
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
+        await initTargets(config);
         const selection = await showCommandSelector({ scope, pageSize: config.ui.page_size });
         if (!selection)
             return;
@@ -926,6 +856,7 @@ agentRoot.action(async (options) => {
     try {
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
+        await initTargets(config);
         const selection = await showSubagentSelector({ scope, pageSize: config.ui.page_size });
         if (!selection)
             return;
@@ -1073,6 +1004,7 @@ skillRoot.action(async (options) => {
     try {
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
+        await initTargets(config);
         const selection = await showSkillSelector({ scope, pageSize: config.ui.page_size });
         if (!selection)
             return;
@@ -1374,6 +1306,7 @@ hookRoot
 async function applyToAgents(scope, enabledServerNames, options) {
     const mcpConfig = loadMcpConfigWithPlugins();
     const switchboardConfig = loadSwitchboardConfig(scopeToLoadOptions(scope));
+    await initTargets(switchboardConfig);
     const useSpinner = options?.useSpinner ?? true;
     const results = [];
     if (switchboardConfig.applications.active.length === 0) {
@@ -1404,7 +1337,18 @@ async function applyToAgents(scope, enabledServerNames, options) {
             const activeSet = new Set(agentActiveServers);
             const enabledServers = Object.fromEntries(Object.entries(mcpConfig.mcpServers).filter(([name]) => activeSet.has(name)));
             const configToApply = { mcpServers: enabledServers };
-            const agent = getAgentById(agentId);
+            const target = getTargetById(agentId);
+            if (!target?.mcp) {
+                persist(chalk.yellow('⚠'), `${chalk.cyan(agentId)} - no MCP handler (skipped)`);
+                results.push({
+                    application: agentId,
+                    filePath: '(unknown)',
+                    status: 'skipped',
+                    reason: 'no MCP handler',
+                });
+                continue;
+            }
+            const mcpHandler = target.mcp;
             const readFileSafe = (p) => {
                 try {
                     return fs.readFileSync(p, 'utf-8');
@@ -1413,11 +1357,10 @@ async function applyToAgents(scope, enabledServerNames, options) {
                     return null;
                 }
             };
-            // Use project-level config when --project is specified
-            if (scope?.project && agent.applyProjectConfig) {
-                const projectPath = agent.projectConfigPath?.(scope.project) ?? 'project config';
+            if (scope?.project && mcpHandler.applyProjectConfig) {
+                const projectPath = mcpHandler.projectConfigPath?.(scope.project) ?? 'project config';
                 const before = readFileSafe(projectPath);
-                agent.applyProjectConfig(scope.project, configToApply);
+                mcpHandler.applyProjectConfig(scope.project, configToApply);
                 const after = readFileSafe(projectPath);
                 const changed = before !== after;
                 persist(chalk.green('✓'), `${chalk.cyan(agentId)} ${chalk.dim(shortenPath(projectPath))}`);
@@ -1429,9 +1372,9 @@ async function applyToAgents(scope, enabledServerNames, options) {
                 });
             }
             else {
-                const configPath = agent.configPath();
+                const configPath = mcpHandler.configPath();
                 const before = readFileSafe(configPath);
-                agent.applyConfig(configToApply);
+                mcpHandler.applyConfig(configToApply);
                 const after = readFileSafe(configPath);
                 const changed = before !== after;
                 persist(chalk.green('✓'), `${chalk.cyan(agentId)} ${chalk.dim(shortenPath(configPath))}`);
@@ -1527,21 +1470,22 @@ pluginRoot
         const index = buildPluginIndex();
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
-        const enabledMap = config.plugins.enabled;
+        const enabledList = config.plugins.enabled;
+        const enabledSet = new Set(enabledList);
         if (options.json) {
             console.log(JSON.stringify(index.plugins.map((p) => {
                 const ref = p.meta.sourceKind === 'marketplace' ? `${p.id}@${p.meta.sourceName}` : p.id;
                 return {
                     id: p.id,
                     ref,
-                    enabled: enabledMap[ref] ?? null,
+                    enabled: enabledSet.has(ref),
                     ...p.meta,
                     components: Object.fromEntries(Object.entries(p.components).map(([k, v]) => [k, v.length])),
                 };
             }), null, 2));
             return;
         }
-        printPluginList(index.plugins, enabledMap);
+        printPluginList(index.plugins, enabledList);
     }
     catch (error) {
         if (error instanceof Error) {
@@ -1578,7 +1522,7 @@ function pluginEnableAction(id, options) {
     try {
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
-        if (config.plugins.enabled[id] === true) {
+        if (config.plugins.enabled.includes(id)) {
             console.log(chalk.yellow(`⚠ Plugin "${id}" is already enabled.`));
             return;
         }
@@ -1592,7 +1536,7 @@ function pluginEnableAction(id, options) {
             ...layer,
             plugins: {
                 ...(layer.plugins ?? {}),
-                enabled: { ...(layer.plugins?.enabled ?? {}), [id]: true },
+                enabled: [...(layer.plugins?.enabled ?? []), id],
             },
         }), scopeToLoadOptions(scope));
         console.log(chalk.green(`✓ Plugin "${id}" enabled.`));
@@ -1608,19 +1552,15 @@ function pluginDisableAction(id, options) {
     try {
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
-        if (config.plugins.enabled[id] === undefined) {
-            console.log(chalk.yellow(`⚠ Plugin "${id}" is not installed.`));
-            return;
-        }
-        if (config.plugins.enabled[id] === false) {
-            console.log(chalk.yellow(`⚠ Plugin "${id}" is already disabled.`));
+        if (!config.plugins.enabled.includes(id)) {
+            console.log(chalk.yellow(`⚠ Plugin "${id}" is not enabled.`));
             return;
         }
         updateConfigLayer((layer) => ({
             ...layer,
             plugins: {
                 ...(layer.plugins ?? {}),
-                enabled: { ...(layer.plugins?.enabled ?? {}), [id]: false },
+                enabled: (layer.plugins?.enabled ?? []).filter((x) => x !== id),
             },
         }), scopeToLoadOptions(scope));
         console.log(chalk.green(`✓ Plugin "${id}" disabled.`));
@@ -1636,18 +1576,17 @@ function pluginUninstallAction(id, options) {
     try {
         const scope = resolveScope(options);
         const config = loadSwitchboardConfig(scopeToLoadOptions(scope));
-        if (config.plugins.enabled[id] === undefined) {
-            console.log(chalk.yellow(`⚠ Plugin "${id}" is not installed.`));
+        if (!config.plugins.enabled.includes(id)) {
+            console.log(chalk.yellow(`⚠ Plugin "${id}" is not enabled.`));
             return;
         }
-        updateConfigLayer((layer) => {
-            const updated = { ...(layer.plugins?.enabled ?? {}) };
-            delete updated[id];
-            return {
-                ...layer,
-                plugins: { ...(layer.plugins ?? {}), enabled: updated },
-            };
-        }, scopeToLoadOptions(scope));
+        updateConfigLayer((layer) => ({
+            ...layer,
+            plugins: {
+                ...(layer.plugins ?? {}),
+                enabled: (layer.plugins?.enabled ?? []).filter((x) => x !== id),
+            },
+        }), scopeToLoadOptions(scope));
         console.log(chalk.green(`✓ Plugin "${id}" uninstalled.`));
     }
     catch (error) {
@@ -1661,27 +1600,25 @@ const pluginScopeOpts = [
     ['-p, --profile <name>', 'Profile configuration to use'],
     ['--project <path>', 'Project directory containing .asb.toml'],
 ];
-const enableCmd = pluginRoot
-    .command('enable <id>')
-    .description('Enable a plugin (set plugins.enabled to true)');
+const enableCmd = pluginRoot.command('enable <id>').description('Add a plugin to the enabled list');
 for (const [flag, desc] of pluginScopeOpts)
     enableCmd.option(flag, desc);
 enableCmd.action(pluginEnableAction);
 const installCmd = pluginRoot
     .command('install <id>')
-    .description('Enable a plugin (alias for enable)');
+    .description('Add a plugin to the enabled list (alias for enable)');
 for (const [flag, desc] of pluginScopeOpts)
     installCmd.option(flag, desc);
 installCmd.action(pluginEnableAction);
 const disableCmd = pluginRoot
     .command('disable <id>')
-    .description('Disable a plugin (set plugins.enabled to false)');
+    .description('Remove a plugin from the enabled list');
 for (const [flag, desc] of pluginScopeOpts)
     disableCmd.option(flag, desc);
 disableCmd.action(pluginDisableAction);
 const uninstallCmd = pluginRoot
     .command('uninstall <id>')
-    .description('Remove a plugin from plugins.enabled');
+    .description('Remove a plugin from the enabled list (alias for disable)');
 for (const [flag, desc] of pluginScopeOpts)
     uninstallCmd.option(flag, desc);
 uninstallCmd.action(pluginUninstallAction);
@@ -1915,7 +1852,7 @@ program
     .action(() => {
     console.error(chalk.red('✗ `asb source` has been removed.\n' +
         '  Sources are now managed under `asb plugin marketplace`.\n' +
-        '  Config has moved from [library.sources] to [plugins.sources].\n' +
+        '  Config has moved from [library.sources] to [plugins].\n' +
         '  Run `asb plugin marketplace --help` for usage.'));
     process.exit(1);
 });