@sylphx/flow 1.4.10 → 1.4.12

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # @sylphx/flow
 
+## 1.4.12
+
+### Patch Changes
+
+- d88d280: Show missing templates in sync preview:
+  - Added "Will install (new templates)" section
+  - Users can now see which templates will be newly installed
+  - Better visibility into what changes sync will make
+
+## 1.4.11
+
+### Patch Changes
+
+- 22ddfb9: Fix sync to dynamically scan templates instead of hardcoding (CRITICAL):
+  - Now scans assets/ directory at runtime for agents, slash commands, and rules
+  - Prevents sync from breaking when templates change
+  - Old commands (commit, context, explain, review, test) now correctly detected as unknown files
+  - New commands (cleanup, improve, polish, quality, release) properly recognized as Flow templates
+
 ## 1.4.10
 
 ### Patch Changes

package/package.json

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

package/src/utils/sync-utils.ts

@@ -3,20 +3,37 @@ import path from 'node:path';
 import chalk from 'chalk';
 import type { Target } from '../types.js';
 import { MCP_SERVER_REGISTRY } from '../config/servers.js';
+import { getAgentsDir, getSlashCommandsDir, getRulesDir } from './paths.js';
 
 /**
- * Flow template filenames (source of truth)
+ * Scan directory for .md files and return basenames
  */
-const FLOW_AGENTS = ['coder.md', 'orchestrator.md', 'reviewer.md', 'writer.md'];
-const FLOW_SLASH_COMMANDS = ['commit.md', 'context.md', 'explain.md', 'review.md', 'test.md'];
-const FLOW_RULES = ['code-standards.md', 'core.md'];
+function scanTemplateDir(dir: string): string[] {
+  if (!fs.existsSync(dir)) return [];
+
+  try {
+    return fs.readdirSync(dir, { withFileTypes: true })
+      .filter((f) => f.isFile() && f.name.endsWith('.md'))
+      .map((f) => f.name);
+  } catch {
+    return [];
+  }
+}
+
+/**
+ * Flow template filenames (scanned from assets at runtime)
+ */
+const FLOW_AGENTS = scanTemplateDir(getAgentsDir());
+const FLOW_SLASH_COMMANDS = scanTemplateDir(getSlashCommandsDir());
+const FLOW_RULES = scanTemplateDir(getRulesDir());
 
 /**
  * Categorized files for sync
  */
 interface CategorizedFiles {
-  inFlow: string[];      // Files that exist in Flow templates
+  inFlow: string[];      // Files that exist locally and in Flow templates (will reinstall)
   unknown: string[];     // Files not in Flow templates (custom or removed)
+  missing: string[];     // Flow templates that don't exist locally (will install)
 }
 
 /**
@@ -34,11 +51,15 @@ interface SyncManifest {
 }
 
 /**
- * Categorize files into Flow templates vs unknown
+ * Categorize files into Flow templates vs unknown, and detect missing templates
  */
 function categorizeFiles(files: string[], flowTemplates: string[]): CategorizedFiles {
   const inFlow: string[] = [];
   const unknown: string[] = [];
+  const existingBasenames = files.map((f) => path.basename(f));
+
+  // Find missing Flow templates (in flowTemplates but not in files)
+  const missing = flowTemplates.filter((template) => !existingBasenames.includes(template));
 
   for (const file of files) {
     const basename = path.basename(file);
@@ -49,7 +70,7 @@ function categorizeFiles(files: string[], flowTemplates: string[]): CategorizedF
     }
   }
 
-  return { inFlow, unknown };
+  return { inFlow, unknown, missing };
 }
 
 /**
@@ -57,9 +78,9 @@ function categorizeFiles(files: string[], flowTemplates: string[]): CategorizedF
  */
 export async function buildSyncManifest(cwd: string, target: Target): Promise<SyncManifest> {
   const manifest: SyncManifest = {
-    agents: { inFlow: [], unknown: [] },
-    slashCommands: { inFlow: [], unknown: [] },
-    rules: { inFlow: [], unknown: [] },
+    agents: { inFlow: [], unknown: [], missing: [] },
+    slashCommands: { inFlow: [], unknown: [], missing: [] },
+    rules: { inFlow: [], unknown: [], missing: [] },
     mcpServers: { inRegistry: [], notInRegistry: [] },
     preserve: [],
   };
@@ -198,6 +219,40 @@ export function showSyncPreview(manifest: SyncManifest, cwd: string): void {
     }
   }
 
+  // Missing templates section (will be installed)
+  const hasMissingFiles =
+    manifest.agents.missing.length > 0 ||
+    manifest.slashCommands.missing.length > 0 ||
+    manifest.rules.missing.length > 0;
+
+  if (hasMissingFiles) {
+    console.log(chalk.green('Will install (new templates):\n'));
+
+    if (manifest.agents.missing.length > 0) {
+      console.log(chalk.dim('  Agents:'));
+      manifest.agents.missing.forEach((file) => {
+        console.log(chalk.dim(`    + ${file}`));
+      });
+      console.log('');
+    }
+
+    if (manifest.slashCommands.missing.length > 0) {
+      console.log(chalk.dim('  Commands:'));
+      manifest.slashCommands.missing.forEach((file) => {
+        console.log(chalk.dim(`    + ${file}`));
+      });
+      console.log('');
+    }
+
+    if (manifest.rules.missing.length > 0) {
+      console.log(chalk.dim('  Rules:'));
+      manifest.rules.missing.forEach((file) => {
+        console.log(chalk.dim(`    + ${file}`));
+      });
+      console.log('');
+    }
+  }
+
   // Unknown files section
   const hasUnknownFiles =
     manifest.agents.unknown.length > 0 ||