@sylphx/flow 2.1.0 → 2.1.1

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # @sylphx/flow
 
+## 2.1.1
+
+### Patch Changes
+
+- 8ae48d6: Fix singleFiles location and improve settings cleanup
+
+  **Bug Fixes:**
+
+  1. Fixed silent.md location bug - output style files were incorrectly written to project root instead of target config directory (.claude/ or .opencode/)
+
+  2. Enhanced clearUserSettings to ensure complete cleanup in replace mode:
+     - Now clears ALL user configuration including hooks, complete MCP config, rules, and singleFiles
+     - Removes entire MCP section (not just servers) to properly clear user hooks
+     - Added legacy cleanup to remove incorrectly placed files from project root
+
+  This fixes the issue where user's hooks and MCP configs were still affecting execution even in replace mode (non-merge mode).
+
 ## 2.1.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sylphx/flow",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "One CLI to rule them all. Unified orchestration layer for Claude Code, OpenCode, Cursor and all AI development tools. Auto-detection, auto-installation, auto-upgrade.",
   "type": "module",
   "bin": {

package/src/core/attach-manager.ts

@@ -420,7 +420,9 @@ ${rules}
   }
 
   /**
-   * Attach single files (CLAUDE.md, .cursorrules, etc.)
+   * Attach single files (output styles like silent.md)
+   * NOTE: These files are placed in the target config directory (.claude/ or .opencode/),
+   * NOT in the project root directory.
    */
   private async attachSingleFiles(
     projectPath: string,
@@ -428,33 +430,22 @@ ${rules}
     result: AttachResult,
     manifest: BackupManifest
   ): Promise<void> {
+    // Get target from manifest to determine correct directory
+    const target = manifest.target;
+    const targetDir = this.projectManager.getTargetConfigDir(projectPath, target);
+
     for (const file of singleFiles) {
-      const filePath = path.join(projectPath, file.path);
+      // Write to target config directory, not project root
+      const filePath = path.join(targetDir, file.path);
       const existed = existsSync(filePath);
 
       if (existed) {
-        // User has file, append Flow content
-        const userContent = await fs.readFile(filePath, 'utf-8');
-
-        // Check if already appended
-        if (userContent.includes('<!-- Sylphx Flow Enhancement -->')) {
-          continue;
-        }
-
-        const merged = `${userContent}
-
----
-
-**Sylphx Flow Enhancement:**
-
-${file.content}
-`;
-
-        await fs.writeFile(filePath, merged);
+        // User has file, overwrite with Flow content (backed up already)
+        await fs.writeFile(filePath, file.content);
 
         manifest.backup.singleFiles[file.path] = {
           existed: true,
-          originalSize: userContent.length,
+          originalSize: (await fs.readFile(filePath, 'utf-8')).length,
           flowContentAdded: true,
         };
       } else {

package/src/core/flow-executor.ts

@@ -169,6 +169,7 @@ export class FlowExecutor {
 
   /**
    * Clear user settings in replace mode
+   * This ensures a clean slate for Flow's configuration
    */
   private async clearUserSettings(
     projectPath: string,
@@ -188,7 +189,7 @@ export class FlowExecutor {
       ? { agents: 'agents', commands: 'commands' }
       : { agents: 'agent', commands: 'command' };
 
-    // Clear agents directory
+    // 1. Clear agents directory (including AGENTS.md rules file)
     const agentsDir = path.join(targetDir, dirs.agents);
     if (existsSync(agentsDir)) {
       const files = await fs.readdir(agentsDir);
@@ -197,7 +198,7 @@ export class FlowExecutor {
       }
     }
 
-    // Clear commands directory
+    // 2. Clear commands directory
     const commandsDir = path.join(targetDir, dirs.commands);
     if (existsSync(commandsDir)) {
       const files = await fs.readdir(commandsDir);
@@ -206,31 +207,69 @@ export class FlowExecutor {
       }
     }
 
-    // Clear MCP configuration
+    // 3. Clear hooks directory
+    const hooksDir = path.join(targetDir, 'hooks');
+    if (existsSync(hooksDir)) {
+      const files = await fs.readdir(hooksDir);
+      for (const file of files) {
+        await fs.unlink(path.join(hooksDir, file));
+      }
+    }
+
+    // 4. Clear MCP configuration completely
     const configPath = target === 'claude-code'
       ? path.join(targetDir, 'settings.json')
       : path.join(targetDir, '.mcp.json');
 
     if (existsSync(configPath)) {
-      // Clear MCP servers section only, keep other settings
-      const config = JSON.parse(await fs.readFile(configPath, 'utf-8'));
       if (target === 'claude-code') {
-        if (config.mcp?.servers) {
-          config.mcp.servers = {};
+        // For Claude Code, clear entire MCP section to remove all user config
+        const config = JSON.parse(await fs.readFile(configPath, 'utf-8'));
+        if (config.mcp) {
+          // Remove entire MCP configuration, not just servers
+          delete config.mcp;
           await fs.writeFile(configPath, JSON.stringify(config, null, 2));
         }
       } else {
-        // For opencode, clear the entire .mcp.json
+        // For OpenCode, clear the entire .mcp.json file
         await fs.writeFile(configPath, JSON.stringify({ servers: {} }, null, 2));
       }
     }
 
-    // Clear hooks directory if exists
-    const hooksDir = path.join(targetDir, 'hooks');
-    if (existsSync(hooksDir)) {
-      const files = await fs.readdir(hooksDir);
-      for (const file of files) {
-        await fs.unlink(path.join(hooksDir, file));
+    // 5. Clear AGENTS.md rules file (for OpenCode)
+    // Claude Code AGENTS.md is already handled in agents directory
+    if (target === 'opencode') {
+      const rulesPath = path.join(targetDir, 'AGENTS.md');
+      if (existsSync(rulesPath)) {
+        await fs.unlink(rulesPath);
+      }
+    }
+
+    // 6. Clear single files (output styles like silent.md)
+    // These are now in the target directory, not project root
+    const singleFiles = ['silent.md']; // Add other known single files here
+    for (const fileName of singleFiles) {
+      const filePath = path.join(targetDir, fileName);
+      if (existsSync(filePath)) {
+        await fs.unlink(filePath);
+      }
+    }
+
+    // 7. Clean up any Flow-created files in project root (legacy bug cleanup)
+    // This handles files that were incorrectly created in project root
+    const legacySingleFiles = ['silent.md'];
+    for (const fileName of legacySingleFiles) {
+      const filePath = path.join(projectPath, fileName);
+      if (existsSync(filePath)) {
+        // Only delete if it looks like a Flow-created file
+        try {
+          const content = await fs.readFile(filePath, 'utf-8');
+          if (content.includes('Sylphx Flow') || content.includes('Silent Execution Style')) {
+            await fs.unlink(filePath);
+          }
+        } catch {
+          // Ignore errors - file might not be readable
+        }
       }
     }
   }