@sylphx/flow 2.1.3 → 2.1.5

package/src/utils/index.ts

@@ -3,57 +3,49 @@
  * Feature-based organization for better modularity
  */
 
+// ============================================================================
+// SHARED UTILITIES
+// ============================================================================
+export * from '../shared/index.js';
+// ============================================================================
+// AGENTS
+// ============================================================================
+export * from './agent-enhancer.js';
+export * from './config/mcp-config.js';
+export * from './config/paths.js';
 // ============================================================================
 // CONFIG & SETTINGS
 // ============================================================================
 export * from './config/settings.js';
-export * from './config/mcp-config.js';
 export * from './config/target-config.js';
 export * from './config/target-utils.js';
-export * from './config/paths.js';
-
 // ============================================================================
 // DISPLAY & OUTPUT
 // ============================================================================
 export * from './display/banner.js';
-export * from './display/status.js';
 export * from './display/cli-output.js';
 export * from './display/logger.js';
 export * from './display/notifications.js';
-
+export * from './display/status.js';
+// ============================================================================
+// ERROR HANDLING
+// ============================================================================
+export * from './error-handler.js';
 // ============================================================================
 // FILES & SYNC
 // ============================================================================
 export * from './files/file-operations.js';
 export * from './files/sync-utils.js';
-
 // ============================================================================
-// SECURITY
+// FUNCTIONAL PROGRAMMING
 // ============================================================================
-export * from './security/security.js';
+export * from './functional.js';
 export * from './security/secret-utils.js';
-
 // ============================================================================
-// ERROR HANDLING
+// SECURITY
 // ============================================================================
-export * from './error-handler.js';
-
+export * from './security/security.js';
 // ============================================================================
 // VERSIONING
 // ============================================================================
 export * from './version.js';
-
-// ============================================================================
-// AGENTS
-// ============================================================================
-export * from './agent-enhancer.js';
-
-// ============================================================================
-// FUNCTIONAL PROGRAMMING
-// ============================================================================
-export * from './functional.js';
-
-// ============================================================================
-// SHARED UTILITIES
-// ============================================================================
-export * from '../shared/index.js';

package/src/utils/package-manager-detector.ts

@@ -27,10 +27,18 @@ export function detectPackageManagerFromUserAgent(): PackageManager | null {
     return null;
   }
 
-  if (userAgent.includes('bun')) return 'bun';
-  if (userAgent.includes('pnpm')) return 'pnpm';
-  if (userAgent.includes('yarn')) return 'yarn';
-  if (userAgent.includes('npm')) return 'npm';
+  if (userAgent.includes('bun')) {
+    return 'bun';
+  }
+  if (userAgent.includes('pnpm')) {
+    return 'pnpm';
+  }
+  if (userAgent.includes('yarn')) {
+    return 'yarn';
+  }
+  if (userAgent.includes('npm')) {
+    return 'npm';
+  }
 
   return null;
 }
@@ -38,7 +46,9 @@ export function detectPackageManagerFromUserAgent(): PackageManager | null {
 /**
  * Detect package manager from lock files in directory
  */
-export function detectPackageManagerFromLockFiles(dir: string = process.cwd()): PackageManager | null {
+export function detectPackageManagerFromLockFiles(
+  dir: string = process.cwd()
+): PackageManager | null {
   const lockFiles: Record<PackageManager, string[]> = {
     bun: ['bun.lockb', 'bun.lock'],
     pnpm: ['pnpm-lock.yaml'],

package/src/utils/security/security.ts

@@ -209,11 +209,15 @@ export const commandSecurity = {
 
     try {
       return await execFileAsync(command, validatedArgs, secureOptions);
-    } catch (error: any) {
+    } catch (error: unknown) {
       // Sanitize error message to prevent information disclosure
-      const sanitizedError = new Error(`Command execution failed: ${command}`);
-      sanitizedError.code = error.code;
-      sanitizedError.signal = error.signal;
+      const err = error as NodeJS.ErrnoException & { signal?: string };
+      const sanitizedError = new Error(`Command execution failed: ${command}`) as Error & {
+        code?: string;
+        signal?: string;
+      };
+      sanitizedError.code = err.code;
+      sanitizedError.signal = err.signal;
       throw sanitizedError;
     }
   },

package/src/utils/target-selection.ts

@@ -5,7 +5,7 @@
 
 import chalk from 'chalk';
 import inquirer from 'inquirer';
-import { TargetInstaller } from '../services/target-installer.js';
+import type { TargetInstaller } from '../services/target-installer.js';
 import { handlePromptError } from './prompt-helpers.js';
 
 /**
@@ -15,7 +15,7 @@ export interface TargetChoice {
   /** Display name of the target */
   name: string;
   /** Target identifier */
-  value: 'claude-code' | 'opencode' | 'cursor';
+  value: 'claude-code' | 'opencode';
   /** Whether the target is currently installed */
   installed: boolean;
 }
@@ -37,11 +37,6 @@ export function buildAvailableTargets(installedTargets: string[]): TargetChoice[
       value: 'opencode',
       installed: installedTargets.includes('opencode'),
     },
-    {
-      name: 'Cursor',
-      value: 'cursor',
-      installed: installedTargets.includes('cursor'),
-    },
   ];
 }
 
@@ -51,7 +46,10 @@ export function buildAvailableTargets(installedTargets: string[]): TargetChoice[
  * @param context - Context where choice is displayed (affects status message)
  * @returns Formatted string with target name and installation status
  */
-export function formatTargetChoice(target: TargetChoice, context: 'execution' | 'settings'): string {
+export function formatTargetChoice(
+  target: TargetChoice,
+  context: 'execution' | 'settings'
+): string {
   const status = target.installed
     ? chalk.green(' ✓ installed')
     : context === 'execution'

package/src/utils/version.ts

@@ -34,9 +34,11 @@ export function isVersionOutdated(current: string, latest: string): boolean {
 /**
  * Parse version string into components
  */
-export function parseVersion(version: string): { major: number; minor: number; patch: number } | null {
+export function parseVersion(
+  version: string
+): { major: number; minor: number; patch: number } | null {
   const parts = version.split('.').map(Number);
-  if (parts.length < 3 || parts.some(isNaN)) {
+  if (parts.length < 3 || parts.some(Number.isNaN)) {
     return null;
   }
   return {

package/src/commands/flow-orchestrator.ts

@@ -1,328 +0,0 @@
-/**
- * Flow Orchestrator - Simplified flow management
- * Separates concerns and reduces complexity
- */
-
-import chalk from 'chalk';
-import type { FlowOptions } from './flow/types.js';
-import { StateDetector, type ProjectState } from '../core/state-detector.js';
-import { UpgradeManager } from '../core/upgrade-manager.js';
-import { targetManager } from '../core/target-manager.js';
-import { detectPackageManager, getUpgradeCommand } from '../utils/package-manager-detector.js';
-
-/**
- * Step 1: Check for available upgrades
- */
-export async function checkUpgrades(
-  state: ProjectState,
-  options: FlowOptions
-): Promise<void> {
-  if (options.initOnly || options.runOnly) return;
-
-  const upgradeManager = new UpgradeManager();
-  const updates = await upgradeManager.checkUpdates();
-  const packageManager = detectPackageManager();
-
-  // Check Flow upgrade
-  if (updates.flowUpdate && updates.flowVersion) {
-    const upgradeCmd = getUpgradeCommand('@sylphx/flow', packageManager);
-    console.log(
-      chalk.yellow(
-        `📦 Sylphx Flow update available: ${updates.flowVersion.current} → ${updates.flowVersion.latest}`
-      )
-    );
-    console.log(chalk.dim(`   Quick upgrade: ${chalk.cyan('sylphx-flow upgrade --auto')}`));
-    console.log(chalk.dim(`   Or run: ${chalk.cyan(upgradeCmd)}\n`));
-  }
-
-  // Check target upgrade
-  if (updates.targetUpdate && updates.targetVersion) {
-    console.log(
-      chalk.yellow(
-        `📦 Target update available: ${updates.targetVersion.current} → ${updates.targetVersion.latest}`
-      )
-    );
-    console.log(chalk.dim(`   Run: ${chalk.cyan('sylphx-flow upgrade --target --auto')}\n`));
-  }
-}
-
-
-/**
- * Step 2: Check component integrity and prompt for repair
- */
-export async function checkComponentIntegrity(
-  state: ProjectState,
-  options: FlowOptions
-): Promise<void> {
-  // Skip if not initialized or cleaning or init-only
-  if (!state.initialized || options.clean || options.initOnly) return;
-
-  // Skip in quick mode
-  if (options.quick) return;
-
-  // Find missing components (target-aware)
-  const missing: string[] = [];
-
-  // Agents are always required
-  if (!state.components.agents.installed) missing.push('agents');
-
-  // For OpenCode: check rules (separate AGENTS.md file)
-  // For Claude Code: rules are included in agent files, so skip this check
-  if (state.target === 'opencode' && !state.components.rules.installed) {
-    missing.push('rules');
-  }
-
-  // Hooks are optional - don't check
-  // Claude Code can have hooks in .claude/hooks/*.js but they're optional
-  // OpenCode doesn't have separate hooks
-
-  // MCP is optional now - many users don't use MCP
-  // if (!state.components.mcp.installed) missing.push('mcp');
-
-  // Output styles:
-  // - Claude Code: included in agent files, so skip check
-  // - OpenCode: included in AGENTS.md, so skip check
-
-  // Slash commands are optional
-  // if (!state.components.slashCommands.installed) missing.push('slash commands');
-
-  // If no missing components, we're good
-  if (missing.length === 0) return;
-
-  // Prompt user to repair
-  console.log(chalk.yellow(`\n⚠️  Missing components detected: ${missing.join(', ')}\n`));
-  const { default: inquirer } = await import('inquirer');
-  const { repair } = await inquirer.prompt([
-    {
-      type: 'confirm',
-      name: 'repair',
-      message: 'Install missing components now?',
-      default: true,
-    },
-  ]);
-
-  if (repair) {
-    // Set repair mode - will trigger component installation without full re-init
-    options.repair = true;
-    console.log(chalk.cyan('\n🔧 Repairing components...\n'));
-  } else {
-    console.log(chalk.dim('Skipping repair. Components may not work correctly.\n'));
-  }
-}
-
-/**
- * Step 2.5: Check sync status (new templates available)
- * Only checks for missing templates, ignores unknown files
- */
-export async function checkSyncStatus(
-  state: ProjectState,
-  options: FlowOptions
-): Promise<void> {
-  // Skip if not initialized, syncing, or init-only
-  if (!state.initialized || options.sync || options.initOnly) return;
-
-  // Skip in quick mode
-  if (options.quick) return;
-
-  // Need target to check sync status
-  if (!state.target) return;
-
-  try {
-    const { buildSyncManifest } = await import('../utils/files/sync-utils.js');
-    const target = targetManager.getTarget(state.target);
-
-    if (target._tag === 'None') return;
-
-    const manifest = await buildSyncManifest(process.cwd(), target.value);
-
-    // Count missing templates (new templates not installed locally)
-    const missingCount =
-      manifest.agents.missing.length +
-      manifest.slashCommands.missing.length +
-      manifest.rules.missing.length;
-
-    // Only prompt if there are missing templates
-    if (missingCount > 0) {
-      const missing: string[] = [];
-
-      if (manifest.agents.missing.length > 0) {
-        missing.push(`${manifest.agents.missing.length} agent${manifest.agents.missing.length > 1 ? 's' : ''}`);
-      }
-      if (manifest.slashCommands.missing.length > 0) {
-        missing.push(`${manifest.slashCommands.missing.length} command${manifest.slashCommands.missing.length > 1 ? 's' : ''}`);
-      }
-      if (manifest.rules.missing.length > 0) {
-        missing.push(`${manifest.rules.missing.length} rule${manifest.rules.missing.length > 1 ? 's' : ''}`);
-      }
-
-      console.log(chalk.yellow(`\n📦 New templates available: ${missing.join(', ')}\n`));
-      console.log(chalk.dim(`   Run ${chalk.cyan('sylphx-flow --sync')} to install new templates\n`));
-    }
-  } catch (error) {
-    // Silently ignore sync check errors - don't block execution
-  }
-}
-
-/**
- * Step 3: Handle target selection
- * Returns the selected target ID
- */
-export async function selectTarget(
-  state: ProjectState,
-  options: FlowOptions
-): Promise<string | undefined> {
-  // Force target selection when cleaning
-  if (options.clean) {
-    try {
-      const targetId = await targetManager.promptForTargetSelection();
-      console.log(chalk.green(`✅ Selected target: ${targetId}`));
-      return targetId;
-    } catch (error) {
-      // User cancelled with Ctrl+C - exit gracefully
-      if (error instanceof Error && error.name === 'ExitPromptError') {
-        console.log('\n');
-        process.exit(0);
-      }
-      throw error;
-    }
-  }
-
-  // Use existing target or option
-  return options.target || state.target;
-}
-
-/**
- * Step 3: Initialize project
- */
-export async function initializeProject(
-  targetId: string | undefined,
-  options: FlowOptions
-): Promise<void> {
-  if (options.runOnly && !options.clean) return;
-
-  console.log(chalk.cyan.bold('━ Initializing Project\n'));
-
-  const { runInit } = await import('./init-command.js');
-
-  const initOptions = {
-    target: targetId,
-    verbose: options.verbose,
-    dryRun: options.dryRun,
-    clear: options.clean || false,
-    mcp: options.mcp !== false,
-    agents: options.agents !== false,
-    rules: options.rules !== false,
-    outputStyles: options.outputStyles !== false,
-    slashCommands: options.slashCommands !== false,
-    hooks: options.hooks !== false,
-    helpOption: () => {},
-  };
-
-  try {
-    await runInit(initOptions);
-
-    if (!options.dryRun) {
-      console.log(chalk.green.bold('\n✓ Initialization complete\n'));
-    } else {
-      console.log(chalk.dim('\n✓ Dry run complete - skipping execution\n'));
-    }
-  } catch (error) {
-    console.error(chalk.red.bold('\n✗ Initialization failed:'), error);
-    throw error;
-  }
-}
-
-/**
- * Step 4: Launch target
- */
-export async function launchTarget(
-  targetId: string | undefined,
-  prompt: string | undefined,
-  options: FlowOptions,
-  state: ProjectState
-): Promise<void> {
-  if (options.initOnly) return;
-
-  // Resolve target
-  const resolvedTarget = await targetManager.resolveTarget({
-    target: targetId || state.target,
-    allowSelection: false,
-  });
-
-  console.log(chalk.cyan.bold(`━ Launching ${resolvedTarget}\n`));
-
-  // Check if target supports command execution
-  const { getTargetsWithCommandSupport } = await import('../config/targets.js');
-  const supportedTargets = getTargetsWithCommandSupport().map(t => t.id);
-
-  if (!supportedTargets.includes(resolvedTarget)) {
-    console.log(chalk.red.bold('✗ Unsupported target platform\n'));
-    console.log(
-      chalk.yellow(`Target '${resolvedTarget}' does not support agent execution.`)
-    );
-    console.log(chalk.cyan(`Supported platforms: ${supportedTargets.join(', ')}\n`));
-    throw new Error(`Unsupported target: ${resolvedTarget}`);
-  }
-
-  // Handle Claude Code specific setup
-  if (resolvedTarget === 'claude-code') {
-    await setupClaudeCode(options);
-  }
-
-  // Execute command
-  await executeCommand(resolvedTarget, prompt, options);
-}
-
-/**
- * Setup Claude Code (provider + agent selection)
- */
-async function setupClaudeCode(options: FlowOptions): Promise<void> {
-  const { SmartConfigService } = await import('../services/smart-config-service.js');
-  const { ConfigService } = await import('../services/config-service.js');
-
-  // Check if API keys are configured
-  if (!(await ConfigService.hasInitialSetup())) {
-    console.log(chalk.cyan('\n🔑 First-time setup for Claude Code:\n'));
-    await SmartConfigService.initialSetup();
-    console.log(chalk.green('\n✅ Claude Code setup complete!\n'));
-  }
-
-  // Select provider and agent
-  const runtimeChoices = await SmartConfigService.selectRuntimeChoices({
-    selectProvider: options.selectProvider,
-    selectAgent: options.selectAgent,
-    useDefaults: options.useDefaults,
-    provider: options.provider,
-    agent: options.agent,
-  });
-
-  // Setup environment
-  await SmartConfigService.setupEnvironment(runtimeChoices.provider!);
-
-  // Store selected agent
-  options.agent = runtimeChoices.agent;
-}
-
-/**
- * Execute the target command
- */
-async function executeCommand(
-  targetId: string,
-  prompt: string | undefined,
-  options: FlowOptions
-): Promise<void> {
-  const agent = options.agent || 'coder';
-  const verbose = options.verbose || false;
-
-  if (verbose || options.runOnly) {
-    console.log(`🤖 Agent: ${agent}`);
-    console.log(`🎯 Target: ${targetId}`);
-    if (prompt) {
-      console.log(`💬 Prompt: ${prompt}\n`);
-    }
-  }
-
-  // Run the command
-  const { runCommand } = await import('./run-command.js');
-  await runCommand({ target: targetId, agent, prompt, verbose });
-}

package/src/commands/init-command.ts

@@ -1,92 +0,0 @@
-import boxen from 'boxen';
-import chalk from 'chalk';
-import gradient from 'gradient-string';
-import {
-  selectAndValidateTarget,
-  previewDryRun,
-  installComponents,
-  type InitOptions,
-} from './init-core.js';
-
-/**
- * Legacy init with full UI - used by setup command for backward compatibility
- * The flow command uses init-core functions directly for better integration
- */
-export async function runInit(options: InitOptions): Promise<void> {
-  // Create ASCII art title
-  const title = `
-███████╗██╗   ██╗██╗     ██████╗ ██╗  ██╗██╗  ██╗    ███████╗██╗      ██████╗ ██╗    ██╗
-██╔════╝╚██╗ ██╔╝██║     ██╔══██╗██║  ██║╚██╗██╔╝    ██╔════╝██║     ██╔═══██╗██║    ██║
-███████╗ ╚████╔╝ ██║     ██████╔╝███████║ ╚███╔╝     █████╗  ██║     ██║   ██║██║ █╗ ██║
-╚════██║  ╚██╔╝  ██║     ██╔═══╝ ██╔══██║ ██╔██╗     ██╔══╝  ██║     ██║   ██║██║███╗██║
-███████║   ██║   ███████╗██║     ██║  ██║██╔╝ ██╗    ██║     ███████╗╚██████╔╝╚███╔███╔╝
-╚══════╝   ╚═╝   ╚══════╝╚═╝     ╚═╝  ╚═╝╚═╝  ╚═╝    ╚═╝     ╚══════╝ ╚═════╝  ╚══╝╚══╝
-`;
-
-  console.log(gradient(['cyan', 'blue'])(title));
-  console.log(chalk.dim.cyan('                          Project Initialization\n'));
-
-  // Select and validate target using core function
-  const targetId = await selectAndValidateTarget(options);
-
-  // Dry run preview
-  if (options.dryRun) {
-    console.log(
-      boxen(
-        chalk.yellow('⚠ Dry Run Mode') + chalk.dim('\nNo changes will be made to your project'),
-        {
-          padding: 1,
-          margin: { top: 0, bottom: 1, left: 0, right: 0 },
-          borderStyle: 'round',
-          borderColor: 'yellow',
-        }
-      )
-    );
-
-    await previewDryRun(targetId, options);
-
-    console.log(
-      '\n' +
-        boxen(chalk.green.bold('✓ Dry run complete'), {
-          padding: { top: 0, bottom: 0, left: 2, right: 2 },
-          margin: 0,
-          borderStyle: 'round',
-          borderColor: 'green',
-        }) +
-        '\n'
-    );
-    return;
-  }
-
-  console.log(chalk.cyan.bold('\n━━━ Installing Core Components ━━━\n'));
-
-  // Install components using core function
-  const result = await installComponents(targetId, options);
-
-  // Success summary
-  console.log(
-    '\n' +
-      boxen(
-        chalk.green.bold('✓ Setup complete!') +
-          '\n\n' +
-          chalk.dim(`Target: ${result.targetName}`) +
-          '\n\n' +
-          chalk.cyan('Ready to code with Sylphx Flow'),
-        {
-          padding: 1,
-          margin: 0,
-          borderStyle: 'round',
-          borderColor: 'green',
-        }
-      ) +
-      '\n'
-  );
-}
-
-/**
- * LEGACY: init command has been integrated into the flow command.
- * Use `flow --init-only` instead of standalone `init` command.
- *
- * This export is kept for backward compatibility but will be removed in future versions.
- * The runInit() function is the core implementation used by flow command.
- */