@sylphx/flow 1.4.2 → 1.4.4

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # @sylphx/flow
 
+## 1.4.3
+
+### Patch Changes
+
+- Fix sync not actually updating files (CRITICAL):
+  - Installation was comparing content and skipping writes
+  - Even after deletion, files weren't updated if content "matched"
+  - Add force mode that always overwrites during sync
+  - Sync now properly updates all files regardless of content
+
 ## 1.4.2
 
 ### Patch Changes

package/package.json

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

package/src/commands/init-core.ts

@@ -168,7 +168,12 @@ export async function installComponents(
   if (target.setupAgents && options.agents !== false) {
     const agentSpinner = quiet ? null : ora({ text: 'Installing agents', color: 'cyan' }).start();
     try {
-      const agentResult = await target.setupAgents(process.cwd(), { ...options, quiet: true });
+      // DEBUG: Log force flag
+      if (!quiet && options.clear) {
+        console.log(`[DEBUG] Installing agents with force=${options.clear}`);
+      }
+
+      const agentResult = await target.setupAgents(process.cwd(), { ...options, quiet: true, force: options.clear });
       result.installed.agents = agentResult.count;
 
       if (agentSpinner) {
@@ -188,7 +193,7 @@ export async function installComponents(
   if (target.setupOutputStyles && options.outputStyles !== false) {
     const stylesSpinner = quiet ? null : ora({ text: 'Installing output styles', color: 'cyan' }).start();
     try {
-      const stylesResult = await target.setupOutputStyles(process.cwd(), { ...options, quiet: true });
+      const stylesResult = await target.setupOutputStyles(process.cwd(), { ...options, quiet: true, force: options.clear });
       result.installed.outputStyles = stylesResult.count;
 
       if (stylesSpinner) {
@@ -216,7 +221,7 @@ export async function installComponents(
   if (target.setupRules && options.rules !== false) {
     const rulesSpinner = quiet ? null : ora({ text: 'Installing rules', color: 'cyan' }).start();
     try {
-      const rulesResult = await target.setupRules(process.cwd(), { ...options, quiet: true });
+      const rulesResult = await target.setupRules(process.cwd(), { ...options, quiet: true, force: options.clear });
       result.installed.rules = rulesResult.count;
 
       if (rulesSpinner) {
@@ -247,7 +252,12 @@ export async function installComponents(
       color: 'cyan',
     }).start();
     try {
-      const commandsResult = await target.setupSlashCommands(process.cwd(), { ...options, quiet: true });
+      // DEBUG: Log force flag
+      if (!quiet && options.clear) {
+        console.log(`[DEBUG] Installing slash commands with force=${options.clear}`);
+      }
+
+      const commandsResult = await target.setupSlashCommands(process.cwd(), { ...options, quiet: true, force: options.clear });
       result.installed.slashCommands = commandsResult.count;
 
       if (commandsSpinner) {

package/src/core/installers/file-installer.ts

@@ -15,6 +15,8 @@ export interface InstallOptions extends CommonOptions {
   flatten?: boolean;
   /** Show progress messages */
   showProgress?: boolean;
+  /** Force overwrite without checking content (for sync) */
+  force?: boolean;
 }
 
 export type FileTransformFn = (content: string, sourcePath?: string) => Promise<string>;
@@ -122,22 +124,49 @@ export async function installToDirectory(
     let content = fs.readFileSync(sourcePath, 'utf8');
     content = await transform(content, file);
 
-    const localProcessed = localInfo ? await transform(localInfo.content, file) : '';
-    const contentChanged = !localInfo || localProcessed !== content;
+    // DEBUG: Log installation details
+    if (options.force && !options.quiet) {
+      console.log(
+        `[DEBUG] Installing ${file} - force=${options.force}, exists=${!!localInfo}, path=${targetPath}`
+      );
+    }
 
-    if (contentChanged) {
+    // Force mode: always overwrite without checking
+    if (options.force) {
       fs.writeFileSync(targetPath, content, 'utf8');
+
+      // DEBUG: Verify write
+      if (!options.quiet) {
+        const written = fs.existsSync(targetPath);
+        if (!written) {
+          console.warn(`[DEBUG] Failed to write: ${targetPath}`);
+        }
+      }
+
       results.push({
         file,
         status: localInfo ? 'updated' : 'added',
         action: localInfo ? 'Updated' : 'Created',
       });
     } else {
-      results.push({
-        file,
-        status: 'current',
-        action: 'Already current',
-      });
+      // Normal mode: check if content changed
+      const localProcessed = localInfo ? await transform(localInfo.content, file) : '';
+      const contentChanged = !localInfo || localProcessed !== content;
+
+      if (contentChanged) {
+        fs.writeFileSync(targetPath, content, 'utf8');
+        results.push({
+          file,
+          status: localInfo ? 'updated' : 'added',
+          action: localInfo ? 'Updated' : 'Created',
+        });
+      } else {
+        results.push({
+          file,
+          status: 'current',
+          action: 'Already current',
+        });
+      }
     }
   });
 

package/src/utils/sync-utils.ts

@@ -388,12 +388,25 @@ export async function executeSyncDelete(
   // Delete Flow templates
   for (const file of flowFiles) {
     try {
+      const existsBefore = fs.existsSync(file);
+      if (!existsBefore) {
+        console.log(chalk.dim(`  ⊘ Already deleted: ${path.basename(file)}`));
+        continue;
+      }
+
       await fs.promises.unlink(file);
-      console.log(chalk.dim(`  ✓ Deleted: ${path.basename(file)}`));
-      templatesDeleted++;
+
+      // Verify deletion
+      const existsAfter = fs.existsSync(file);
+      if (existsAfter) {
+        console.warn(chalk.yellow(`  ⚠ File still exists after deletion: ${path.basename(file)}`));
+      } else {
+        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: ${path.basename(file)}`));
+        console.warn(chalk.yellow(`  ⚠ Failed to delete: ${path.basename(file)} - ${error}`));
       }
     }
   }