@sylphx/flow 1.2.0 → 1.3.1

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # @sylphx/flow
 
+## 1.3.1
+
+### Patch Changes
+
+- Redesign sync flow for better clarity:
+  - Remove duplicate config files in preserved list
+  - Show MCP check in preview upfront (not after confirmation)
+  - Combined preview: templates + MCP servers + preserved files
+  - Clear sections with emojis for easy scanning
+
+## 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.1",
   "description": "AI-powered development workflow automation with autonomous loop mode and smart configuration",
   "type": "module",
   "bin": {

package/src/commands/flow-command.ts

@@ -380,7 +380,7 @@ async function executeSetupPhase(prompt: string | undefined, options: FlowOption
 
       // Handle sync mode - delete template files first
       if (options.sync && !options.dryRun) {
-        const { buildSyncManifest, showSyncPreview, confirmSync, executeSyncDelete } = await import('../utils/sync-utils.js');
+        const { buildSyncManifest, showSyncPreview, confirmSync, executeSyncDelete, checkMCPServers, selectServersToRemove, removeMCPServers } = await import('../utils/sync-utils.js');
 
         // Need target to build manifest
         const targetId = await selectAndValidateTarget(initOptions);
@@ -394,8 +394,11 @@ async function executeSetupPhase(prompt: string | undefined, options: FlowOption
         const target = targetOption.value;
         const manifest = await buildSyncManifest(process.cwd(), target);
 
+        // Check MCP servers before showing preview
+        const nonRegistryServers = await checkMCPServers(process.cwd());
+
         console.log(chalk.cyan.bold('━━━ 🔄 Synchronizing Files\n'));
-        showSyncPreview(manifest, process.cwd());
+        showSyncPreview(manifest, process.cwd(), nonRegistryServers);
 
         const confirmed = await confirmSync();
         if (!confirmed) {
@@ -403,8 +406,19 @@ async function executeSetupPhase(prompt: string | undefined, options: FlowOption
           process.exit(0);
         }
 
+        // Delete templates
         const deletedCount = await executeSyncDelete(manifest);
-        console.log(chalk.green(`\n✓ Deleted ${deletedCount} files\n`));
+        console.log(chalk.green(`\n✓ Deleted ${deletedCount} template files\n`));
+
+        // Handle MCP servers if any found
+        if (nonRegistryServers.length > 0) {
+          const serversToRemove = await selectServersToRemove(nonRegistryServers);
+
+          if (serversToRemove.length > 0) {
+            const removedCount = await removeMCPServers(process.cwd(), serversToRemove);
+            console.log(chalk.green(`✓ Removed ${removedCount} MCP server(s)\n`));
+          }
+        }
       } else if (!options.sync) {
         const targetId = await selectAndValidateTarget(initOptions);
         selectedTarget = targetId;
@@ -695,7 +709,7 @@ async function executeFlowOnce(prompt: string | undefined, options: FlowOptions)
 
       // Handle sync mode - delete template files first
       if (options.sync && !options.dryRun) {
-        const { buildSyncManifest, showSyncPreview, confirmSync, executeSyncDelete } = await import('../utils/sync-utils.js');
+        const { buildSyncManifest, showSyncPreview, confirmSync, executeSyncDelete, checkMCPServers, selectServersToRemove, removeMCPServers } = await import('../utils/sync-utils.js');
 
         // Need target to build manifest
         const targetId = await selectAndValidateTarget(initOptions);
@@ -709,8 +723,11 @@ async function executeFlowOnce(prompt: string | undefined, options: FlowOptions)
         const target = targetOption.value;
         const manifest = await buildSyncManifest(process.cwd(), target);
 
+        // Check MCP servers before showing preview
+        const nonRegistryServers = await checkMCPServers(process.cwd());
+
         console.log(chalk.cyan.bold('━━━ 🔄 Synchronizing Files\n'));
-        showSyncPreview(manifest, process.cwd());
+        showSyncPreview(manifest, process.cwd(), nonRegistryServers);
 
         const confirmed = await confirmSync();
         if (!confirmed) {
@@ -718,8 +735,19 @@ async function executeFlowOnce(prompt: string | undefined, options: FlowOptions)
           process.exit(0);
         }
 
+        // Delete templates
         const deletedCount = await executeSyncDelete(manifest);
-        console.log(chalk.green(`\n✓ Deleted ${deletedCount} files\n`));
+        console.log(chalk.green(`\n✓ Deleted ${deletedCount} template files\n`));
+
+        // Handle MCP servers if any found
+        if (nonRegistryServers.length > 0) {
+          const serversToRemove = await selectServersToRemove(nonRegistryServers);
+
+          if (serversToRemove.length > 0) {
+            const removedCount = await removeMCPServers(process.cwd(), serversToRemove);
+            console.log(chalk.green(`✓ 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
@@ -59,8 +60,6 @@ export async function buildSyncManifest(cwd: string, target: Target): Promise<Sy
     '.sylphx-flow/',
     '.secrets/',
     target.config.configFile || '',
-    '.mcp.json',
-    'opencode.jsonc',
   ]
     .filter(Boolean)
     .map((p) => path.join(cwd, p));
@@ -71,46 +70,64 @@ export async function buildSyncManifest(cwd: string, target: Target): Promise<Sy
 /**
  * Show sync preview - what will be deleted
  */
-export function showSyncPreview(manifest: SyncManifest, cwd: string): void {
+export function showSyncPreview(
+  manifest: SyncManifest,
+  cwd: string,
+  nonRegistryServers: string[]
+): void {
   console.log(chalk.cyan.bold('📋 Sync Preview\n'));
-  console.log(chalk.dim('The following files will be deleted and re-installed:\n'));
 
+  // Template files section
   const allFiles = [...manifest.agents, ...manifest.slashCommands, ...manifest.rules];
 
-  if (allFiles.length === 0) {
-    console.log(chalk.yellow('  No template files found\n'));
-    return;
-  }
+  if (allFiles.length > 0) {
+    console.log(chalk.yellow('🔄 Templates (delete + reinstall):\n'));
 
-  // Group by type
-  if (manifest.agents.length > 0) {
-    console.log(chalk.cyan('  Agents:'));
-    manifest.agents.forEach((file) => {
-      const relative = path.relative(cwd, file);
-      console.log(chalk.dim(`    - ${relative}`));
-    });
-    console.log('');
-  }
+    if (manifest.agents.length > 0) {
+      console.log(chalk.dim('  Agents:'));
+      manifest.agents.forEach((file) => {
+        const relative = path.relative(cwd, file);
+        console.log(chalk.dim(`    - ${relative}`));
+      });
+      console.log('');
+    }
 
-  if (manifest.slashCommands.length > 0) {
-    console.log(chalk.cyan('  Slash Commands:'));
-    manifest.slashCommands.forEach((file) => {
-      const relative = path.relative(cwd, file);
-      console.log(chalk.dim(`    - ${relative}`));
-    });
-    console.log('');
+    if (manifest.slashCommands.length > 0) {
+      console.log(chalk.dim('  Slash Commands:'));
+      manifest.slashCommands.forEach((file) => {
+        const relative = path.relative(cwd, file);
+        console.log(chalk.dim(`    - ${relative}`));
+      });
+      console.log('');
+    }
+
+    if (manifest.rules.length > 0) {
+      console.log(chalk.dim('  Rules:'));
+      manifest.rules.forEach((file) => {
+        const relative = path.relative(cwd, file);
+        console.log(chalk.dim(`    - ${relative}`));
+      });
+      console.log('');
+    }
+  } else {
+    console.log(chalk.yellow('🔄 Templates: None found\n'));
   }
 
-  if (manifest.rules.length > 0) {
-    console.log(chalk.cyan('  Rules:'));
-    manifest.rules.forEach((file) => {
-      const relative = path.relative(cwd, file);
-      console.log(chalk.dim(`    - ${relative}`));
+  // MCP servers section
+  if (nonRegistryServers.length > 0) {
+    console.log(chalk.yellow('🔍 MCP Servers (not in registry):\n'));
+    nonRegistryServers.forEach((server) => {
+      console.log(chalk.dim(`    - ${server}`));
     });
-    console.log('');
+    console.log(chalk.dim('\n  Possible reasons:'));
+    console.log(chalk.dim('    1. Removed from Flow registry'));
+    console.log(chalk.dim('    2. Custom installation\n'));
+  } else {
+    console.log(chalk.green('✓ MCP Servers: All in registry\n'));
   }
 
-  console.log(chalk.green('✓ Preserved:'));
+  // Preserved files section
+  console.log(chalk.green('✓ Preserved:\n'));
   manifest.preserve.forEach((file) => {
     const relative = path.relative(cwd, file);
     if (fs.existsSync(file)) {
@@ -157,3 +174,88 @@ 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 [];
+  }
+}
+
+/**
+ * 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;
+  }
+}