@sylphx/flow 1.2.0 → 1.3.0

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @sylphx/flow
 
+## 1.3.0
+
+### Minor Changes
+
+- Enhanced --sync with MCP registry checking:
+  - Detect servers not in Flow registry (removed or custom)
+  - Interactive selection for removal
+  - Clean removal from .mcp.json
+  - Flow: sync templates → check MCP → remove selected
+
+## 1.2.1
+
+### Patch Changes
+
+- Apply MEP principles to workspace documentation rule:
+  - Condensed from verbose instructions to condition→action format
+  - Removed step-by-step teaching and command examples
+  - Embedded verification in directives
+  - 31% reduction while maintaining clarity
+
 ## 1.2.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sylphx/flow",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "AI-powered development workflow automation with autonomous loop mode and smart configuration",
   "type": "module",
   "bin": {

package/src/commands/flow-command.ts

@@ -405,6 +405,20 @@ async function executeSetupPhase(prompt: string | undefined, options: FlowOption
 
         const deletedCount = await executeSyncDelete(manifest);
         console.log(chalk.green(`\n✓ Deleted ${deletedCount} files\n`));
+
+        // Check MCP servers
+        const { checkMCPServers, showNonRegistryServers, selectServersToRemove, removeMCPServers } = await import('../utils/sync-utils.js');
+        const nonRegistryServers = await checkMCPServers(process.cwd());
+
+        if (nonRegistryServers.length > 0) {
+          showNonRegistryServers(nonRegistryServers);
+          const serversToRemove = await selectServersToRemove(nonRegistryServers);
+
+          if (serversToRemove.length > 0) {
+            const removedCount = await removeMCPServers(process.cwd(), serversToRemove);
+            console.log(chalk.green(`\n✓ Removed ${removedCount} MCP server(s)\n`));
+          }
+        }
       } else if (!options.sync) {
         const targetId = await selectAndValidateTarget(initOptions);
         selectedTarget = targetId;
@@ -720,6 +734,20 @@ async function executeFlowOnce(prompt: string | undefined, options: FlowOptions)
 
         const deletedCount = await executeSyncDelete(manifest);
         console.log(chalk.green(`\n✓ Deleted ${deletedCount} files\n`));
+
+        // Check MCP servers
+        const { checkMCPServers, showNonRegistryServers, selectServersToRemove, removeMCPServers } = await import('../utils/sync-utils.js');
+        const nonRegistryServers = await checkMCPServers(process.cwd());
+
+        if (nonRegistryServers.length > 0) {
+          showNonRegistryServers(nonRegistryServers);
+          const serversToRemove = await selectServersToRemove(nonRegistryServers);
+
+          if (serversToRemove.length > 0) {
+            const removedCount = await removeMCPServers(process.cwd(), serversToRemove);
+            console.log(chalk.green(`\n✓ Removed ${removedCount} MCP server(s)\n`));
+          }
+        }
       } else {
         // Select and validate target (will use existing in repair mode, or prompt if needed)
         const targetId = await selectAndValidateTarget(initOptions);

package/src/utils/sync-utils.ts

@@ -2,6 +2,7 @@ import fs from 'node:fs';
 import path from 'node:path';
 import chalk from 'chalk';
 import type { Target } from '../types.js';
+import { MCP_SERVER_REGISTRY } from '../config/servers.js';
 
 /**
  * Files to delete during sync for each target
@@ -157,3 +158,108 @@ export async function confirmSync(): Promise<boolean> {
   ]);
   return confirm;
 }
+
+/**
+ * Check MCP servers - find servers not in Flow registry
+ */
+export async function checkMCPServers(cwd: string): Promise<string[]> {
+  const mcpPath = path.join(cwd, '.mcp.json');
+
+  if (!fs.existsSync(mcpPath)) {
+    return [];
+  }
+
+  try {
+    const content = await fs.promises.readFile(mcpPath, 'utf-8');
+    const mcpConfig = JSON.parse(content);
+
+    if (!mcpConfig.mcpServers) {
+      return [];
+    }
+
+    const installedServers = Object.keys(mcpConfig.mcpServers);
+    const registryServers = Object.keys(MCP_SERVER_REGISTRY);
+
+    // Find servers not in registry
+    return installedServers.filter(id => !registryServers.includes(id));
+  } catch (error) {
+    console.warn(chalk.yellow('⚠ Failed to read .mcp.json'));
+    return [];
+  }
+}
+
+/**
+ * Show non-registry servers
+ */
+export function showNonRegistryServers(servers: string[]): void {
+  if (servers.length === 0) {
+    return;
+  }
+
+  console.log(chalk.cyan.bold('\n📋 MCP Registry Check\n'));
+  console.log(chalk.yellow('⚠️  以下 MCP servers 唔係 Flow registry 入面:\n'));
+
+  servers.forEach(server => {
+    console.log(chalk.dim(`    - ${server}`));
+  });
+
+  console.log(chalk.dim('\n可能原因:'));
+  console.log(chalk.dim('  1. Flow registry 已移除'));
+  console.log(chalk.dim('  2. 你自己手動安裝\n'));
+}
+
+/**
+ * Select servers to remove
+ */
+export async function selectServersToRemove(servers: string[]): Promise<string[]> {
+  const { default: inquirer } = await import('inquirer');
+  const { selected } = await inquirer.prompt([
+    {
+      type: 'checkbox',
+      name: 'selected',
+      message: '選擇要刪除既 servers:',
+      choices: servers.map(s => ({ name: s, value: s })),
+    },
+  ]);
+  return selected;
+}
+
+/**
+ * Remove MCP servers from .mcp.json
+ */
+export async function removeMCPServers(cwd: string, serversToRemove: string[]): Promise<number> {
+  if (serversToRemove.length === 0) {
+    return 0;
+  }
+
+  const mcpPath = path.join(cwd, '.mcp.json');
+
+  try {
+    const content = await fs.promises.readFile(mcpPath, 'utf-8');
+    const mcpConfig = JSON.parse(content);
+
+    if (!mcpConfig.mcpServers) {
+      return 0;
+    }
+
+    let removedCount = 0;
+    for (const server of serversToRemove) {
+      if (mcpConfig.mcpServers[server]) {
+        delete mcpConfig.mcpServers[server];
+        removedCount++;
+      }
+    }
+
+    // Write back
+    await fs.promises.writeFile(
+      mcpPath,
+      JSON.stringify(mcpConfig, null, 2) + '\n',
+      'utf-8'
+    );
+
+    return removedCount;
+  } catch (error) {
+    console.warn(chalk.yellow('⚠ Failed to update .mcp.json'));
+    return 0;
+  }
+}