@sylphx/flow 1.4.0 → 1.4.2

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # @sylphx/flow
 
+## 1.4.2
+
+### Patch Changes
+
+- Add visible deletion output during sync:
+  - Show each file being deleted with checkmark
+  - Display MCP servers being removed
+  - Clear visual feedback of the full sync process
+  - Users can now see exactly what's happening
+
+## 1.4.1
+
+### Patch Changes
+
+- Fix rules scanning showing all project markdown files:
+  - Skip rules scanning for Claude Code (rules embedded in agent files)
+  - Only scan when target has explicit rulesFile config
+  - Prevent scanning entire project directory
+
 ## 1.4.0
 
 ### Minor Changes

package/package.json

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

package/src/utils/sync-utils.ts

@@ -90,15 +90,28 @@ export async function buildSyncManifest(cwd: string, target: Target): Promise<Sy
     }
   }
 
-  // Rules files - check individual files
-  const rulesDir = path.dirname(target.config.rulesFile || '');
-  if (rulesDir && fs.existsSync(path.join(cwd, rulesDir))) {
-    const files = fs.readdirSync(path.join(cwd, rulesDir), { withFileTypes: true });
-    const ruleFiles = files
-      .filter((f) => f.isFile() && f.name.endsWith('.md'))
-      .map((f) => path.join(cwd, rulesDir, f.name));
-
-    manifest.rules = categorizeFiles(ruleFiles, FLOW_RULES);
+  // Rules files - only for targets with separate rules directory
+  // Claude Code has rules in agent files, so skip
+  if (target.config.rulesFile) {
+    const rulesPath = path.join(cwd, target.config.rulesFile);
+
+    // Check if it's a directory or file
+    if (fs.existsSync(rulesPath)) {
+      const stat = fs.statSync(rulesPath);
+
+      if (stat.isDirectory()) {
+        // Scan directory for rule files
+        const files = fs.readdirSync(rulesPath, { withFileTypes: true });
+        const ruleFiles = files
+          .filter((f) => f.isFile() && f.name.endsWith('.md'))
+          .map((f) => path.join(rulesPath, f.name));
+
+        manifest.rules = categorizeFiles(ruleFiles, FLOW_RULES);
+      } else {
+        // Single rules file - check if it matches Flow templates
+        manifest.rules = categorizeFiles([rulesPath], FLOW_RULES);
+      }
+    }
   }
 
   // MCP servers
@@ -370,14 +383,17 @@ export async function executeSyncDelete(
   let templatesDeleted = 0;
   let unknownsDeleted = 0;
 
+  console.log(chalk.cyan('\n🗑️  Deleting files...\n'));
+
   // Delete Flow templates
   for (const file of flowFiles) {
     try {
       await fs.promises.unlink(file);
+      console.log(chalk.dim(`  ✓ Deleted: ${path.basename(file)}`));
       templatesDeleted++;
     } catch (error) {
       if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
-        console.warn(chalk.yellow(`⚠ Failed to delete: ${file}`));
+        console.warn(chalk.yellow(`  ⚠ Failed to delete: ${path.basename(file)}`));
       }
     }
   }
@@ -389,14 +405,17 @@ export async function executeSyncDelete(
 
     try {
       await fs.promises.unlink(file);
+      console.log(chalk.dim(`  ✓ Deleted: ${path.basename(file)}`));
       unknownsDeleted++;
     } catch (error) {
       if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
-        console.warn(chalk.yellow(`⚠ Failed to delete: ${file}`));
+        console.warn(chalk.yellow(`  ⚠ Failed to delete: ${path.basename(file)}`));
       }
     }
   }
 
+  console.log('');
+
   return { templates: templatesDeleted, unknowns: unknownsDeleted };
 }
 
@@ -438,6 +457,7 @@ export async function removeMCPServers(cwd: string, serversToRemove: string[]):
     for (const server of serversToRemove) {
       if (mcpConfig.mcpServers[server]) {
         delete mcpConfig.mcpServers[server];
+        console.log(chalk.dim(`  ✓ Removed MCP: ${server}`));
         removedCount++;
       }
     }