@sylphx/flow 2.1.2 → 2.1.4

package/src/utils/display/notifications.ts

@@ -6,10 +6,14 @@
 import { playNotificationSound } from './audio-player.js';
 
 // Terminal notification with sound
-export function sendTerminalNotification(title: string, message: string, options?: {
-  sound?: boolean;
-  duration?: number;
-}) {
+export function sendTerminalNotification(
+  _title: string,
+  _message: string,
+  options?: {
+    sound?: boolean;
+    duration?: number;
+  }
+) {
   const { sound = true, duration = 3000 } = options || {};
 
   // Play system sound using cross-platform audio player
@@ -30,40 +34,45 @@ export function sendTerminalNotification(title: string, message: string, options
 }
 
 // OS-level notification using system APIs
-export async function sendOSNotification(title: string, message: string, options?: {
-  icon?: string;
-  urgency?: 'low' | 'normal' | 'critical';
-  sound?: boolean;
-  timeout?: number;
-}) {
+export async function sendOSNotification(
+  title: string,
+  message: string,
+  options?: {
+    icon?: string;
+    urgency?: 'low' | 'normal' | 'critical';
+    sound?: boolean;
+    timeout?: number;
+  }
+) {
   const {
     icon = '🌀', // Flow-themed spiral emoji for Sylphx Flow notifications
     urgency = 'normal',
     sound = true,
-    timeout = 5000
+    timeout = 5000,
   } = options || {};
-  
+
   try {
     if (process.platform === 'darwin') {
       // macOS: use osascript with simplified approach
-      const { spawn } = require('child_process');
-      
+      const { spawn } = require('node:child_process');
+
       await new Promise<void>((resolve, reject) => {
         // Simple notification without complex escaping
         const args = [
-          '-e', `display notification "${message.replace(/"/g, '\\"')}" with title "${title.replace(/"/g, '\\"')}"`
+          '-e',
+          `display notification "${message.replace(/"/g, '\\"')}" with title "${title.replace(/"/g, '\\"')}"`,
         ];
-        
-        const proc = spawn('osascript', args, { 
+
+        const proc = spawn('osascript', args, {
           stdio: 'pipe',
-          env: { ...process.env, PATH: '/usr/bin:/bin:/usr/sbin:/sbin' }
+          env: { ...process.env, PATH: '/usr/bin:/bin:/usr/sbin:/sbin' },
         });
-        
+
         let stderr = '';
         proc.stderr?.on('data', (data) => {
           stderr += data.toString();
         });
-        
+
         proc.on('exit', (code) => {
           if (code === 0) {
             // Play sound separately if needed using cross-platform audio player
@@ -79,28 +88,32 @@ export async function sendOSNotification(title: string, message: string, options
         });
         proc.on('error', reject);
       });
-      
     } else if (process.platform === 'linux') {
       // Linux: use notify-send
-      const { spawn } = require('child_process');
+      const { spawn } = require('node:child_process');
       await new Promise<void>((resolve, reject) => {
         const proc = spawn('notify-send', [
-          '--urgency', urgency,
-          '--icon', icon,
-          '--expire-time', timeout.toString(),
+          '--urgency',
+          urgency,
+          '--icon',
+          icon,
+          '--expire-time',
+          timeout.toString(),
           title,
-          message
+          message,
         ]);
         proc.on('exit', (code) => {
-          if (code === 0) resolve();
-          else reject(new Error(`notify-send failed with code ${code}`));
+          if (code === 0) {
+            resolve();
+          } else {
+            reject(new Error(`notify-send failed with code ${code}`));
+          }
         });
         proc.on('error', reject);
       });
-
     } else if (process.platform === 'win32') {
       // Windows: use PowerShell toast notifications
-      const { spawn } = require('child_process');
+      const { spawn } = require('node:child_process');
       const powershellScript = `
         Add-Type -AssemblyName System.Windows.Forms
         $notification = New-Object System.Windows.Forms.NotifyIcon
@@ -113,12 +126,15 @@ export async function sendOSNotification(title: string, message: string, options
         Start-Sleep -Milliseconds ${timeout + 1000}
         $notification.Dispose()
       `;
-      
+
       await new Promise<void>((resolve, reject) => {
         const proc = spawn('powershell', ['-Command', powershellScript]);
         proc.on('exit', (code) => {
-          if (code === 0) resolve();
-          else reject(new Error(`PowerShell failed with code ${code}`));
+          if (code === 0) {
+            resolve();
+          } else {
+            reject(new Error(`PowerShell failed with code ${code}`));
+          }
         });
         proc.on('error', reject);
       });
@@ -132,24 +148,20 @@ export async function sendOSNotification(title: string, message: string, options
 
 // Combined notification - sends both terminal and OS notifications
 export function sendNotification(
-  title: string, 
-  message: string, 
+  title: string,
+  message: string,
   options?: {
     osNotification?: boolean;
     terminalNotification?: boolean;
     sound?: boolean;
   }
 ) {
-  const {
-    osNotification = true,
-    terminalNotification = true,
-    sound = true
-  } = options || {};
-  
+  const { osNotification = true, terminalNotification = true, sound = true } = options || {};
+
   if (terminalNotification) {
     sendTerminalNotification(title, message, { sound });
   }
-  
+
   if (osNotification) {
     sendOSNotification(title, message, { sound });
   }
@@ -163,7 +175,8 @@ export async function checkNotificationSupport(): Promise<{
 }> {
   return {
     terminalSupported: true, // Always supported
-    osSupported: process.platform === 'darwin' || process.platform === 'linux' || process.platform === 'win32',
-    platform: process.platform
+    osSupported:
+      process.platform === 'darwin' || process.platform === 'linux' || process.platform === 'win32',
+    platform: process.platform,
   };
-}
+}

package/src/utils/display/status.ts

@@ -13,9 +13,7 @@ import { isVersionOutdated } from '../version.js';
 export async function showStatus(state: ProjectState): Promise<void> {
   console.log(chalk.cyan.bold('📊 Project Status\n'));
 
-  if (!state.initialized) {
-    console.log('  ' + chalk.yellow('⚠  Not initialized'));
-  } else {
+  if (state.initialized) {
     console.log(`  ${chalk.green('✓')} Initialized (Flow v${state.version || 'unknown'})`);
 
     if (state.target) {
@@ -26,21 +24,38 @@ export async function showStatus(state: ProjectState): Promise<void> {
     // Component status
     const components = state.components;
     console.log(`\n  ${chalk.cyan('Components:')}`);
-    console.log(`    Agents: ${components.agents.installed ? chalk.green(`✓ ${components.agents.count}`) : chalk.red('✗')}`);
-    console.log(`    Rules: ${components.rules.installed ? chalk.green(`✓ ${components.rules.count}`) : chalk.red('✗')}`);
+    console.log(
+      `    Agents: ${components.agents.installed ? chalk.green(`✓ ${components.agents.count}`) : chalk.red('✗')}`
+    );
+    console.log(
+      `    Rules: ${components.rules.installed ? chalk.green(`✓ ${components.rules.count}`) : chalk.red('✗')}`
+    );
     console.log(`    Hooks: ${components.hooks.installed ? chalk.green('✓') : chalk.red('✗')}`);
-    console.log(`    MCP: ${components.mcp.installed ? chalk.green(`✓ ${components.mcp.serverCount} servers`) : chalk.red('✗')}`);
-    console.log(`    Output styles: ${components.outputStyles.installed ? chalk.green('✓') : chalk.red('✗')}`);
-    console.log(`    Slash commands: ${components.slashCommands.installed ? chalk.green(`✓ ${components.slashCommands.count}`) : chalk.red('✗')}`);
+    console.log(
+      `    MCP: ${components.mcp.installed ? chalk.green(`✓ ${components.mcp.serverCount} servers`) : chalk.red('✗')}`
+    );
+    console.log(
+      `    Output styles: ${components.outputStyles.installed ? chalk.green('✓') : chalk.red('✗')}`
+    );
+    console.log(
+      `    Slash commands: ${components.slashCommands.installed ? chalk.green(`✓ ${components.slashCommands.count}`) : chalk.red('✗')}`
+    );
 
     // Outdated warnings
     if (state.outdated) {
-      console.log(`\n  ${chalk.yellow('⚠')} Flow version outdated: ${state.version} → ${state.latestVersion}`);
+      console.log(
+        `\n  ${chalk.yellow('⚠')} Flow version outdated: ${state.version} → ${state.latestVersion}`
+      );
     }
 
-    if (state.targetVersion && state.targetLatestVersion &&
-        isVersionOutdated(state.targetVersion, state.targetLatestVersion)) {
-      console.log(`  ${chalk.yellow('⚠')} ${state.target} update available: v${state.targetVersion} → v${state.targetLatestVersion}`);
+    if (
+      state.targetVersion &&
+      state.targetLatestVersion &&
+      isVersionOutdated(state.targetVersion, state.targetLatestVersion)
+    ) {
+      console.log(
+        `  ${chalk.yellow('⚠')} ${state.target} update available: v${state.targetVersion} → v${state.targetLatestVersion}`
+      );
     }
 
     if (state.lastUpdated) {
@@ -49,6 +64,8 @@ export async function showStatus(state: ProjectState): Promise<void> {
         console.log(`\n  ${chalk.yellow('⚠')} Last updated: ${days} days ago`);
       }
     }
+  } else {
+    console.log(`  ${chalk.yellow('⚠  Not initialized')}`);
   }
 
   console.log('');

package/src/utils/files/file-operations.ts

@@ -5,8 +5,8 @@
 
 import fs from 'node:fs/promises';
 import path from 'node:path';
-import { pathSecurity } from '../security/security.js';
 import { formatFileSize as formatFileSizeCore } from '../../core/formatting/bytes.js';
+import { pathSecurity } from '../security/security.js';
 
 export interface FileReadOptions {
   encoding?: BufferEncoding;

package/src/utils/files/sync-utils.ts

@@ -1,18 +1,21 @@
 import fs from 'node:fs';
 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 '../config/paths.js';
+import type { Target } from '../../types.js';
+import { getAgentsDir, getRulesDir, getSlashCommandsDir } from '../config/paths.js';
 
 /**
  * Scan directory for .md files and return basenames
  */
 function scanTemplateDir(dir: string): string[] {
-  if (!fs.existsSync(dir)) return [];
+  if (!fs.existsSync(dir)) {
+    return [];
+  }
 
   try {
-    return fs.readdirSync(dir, { withFileTypes: true })
+    return fs
+      .readdirSync(dir, { withFileTypes: true })
       .filter((f) => f.isFile() && f.name.endsWith('.md'))
       .map((f) => f.name);
   } catch {
@@ -31,9 +34,9 @@ const FLOW_RULES = scanTemplateDir(getRulesDir());
  * Categorized files for sync
  */
 interface CategorizedFiles {
-  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)
+  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)
 }
 
 /**
@@ -48,8 +51,8 @@ interface SyncManifest {
     notInRegistry: string[];
   };
   hooks: {
-    inConfig: string[];      // Hooks from config (will be synced)
-    orphaned: string[];      // Hooks that exist locally but not in config (ask user)
+    inConfig: string[]; // Hooks from config (will be synced)
+    orphaned: string[]; // Hooks that exist locally but not in config (ask user)
   };
   preserve: string[];
 }
@@ -151,14 +154,14 @@ export async function buildSyncManifest(cwd: string, target: Target): Promise<Sy
         const installedServers = Object.keys(mcpConfig.mcpServers);
         const registryServers = Object.keys(MCP_SERVER_REGISTRY);
 
-        manifest.mcpServers.inRegistry = installedServers.filter(id =>
+        manifest.mcpServers.inRegistry = installedServers.filter((id) =>
           registryServers.includes(id)
         );
-        manifest.mcpServers.notInRegistry = installedServers.filter(id =>
-          !registryServers.includes(id)
+        manifest.mcpServers.notInRegistry = installedServers.filter(
+          (id) => !registryServers.includes(id)
         );
       }
-    } catch (error) {
+    } catch (_error) {
       console.warn(chalk.yellow('⚠ Failed to read .mcp.json'));
     }
   }
@@ -178,25 +181,21 @@ export async function buildSyncManifest(cwd: string, target: Target): Promise<Sy
           // In the future, this could be read from Flow config
           const expectedHookTypes = ['Notification'];
 
-          manifest.hooks.inConfig = existingHookTypes.filter(type =>
+          manifest.hooks.inConfig = existingHookTypes.filter((type) =>
             expectedHookTypes.includes(type)
           );
-          manifest.hooks.orphaned = existingHookTypes.filter(type =>
-            !expectedHookTypes.includes(type)
+          manifest.hooks.orphaned = existingHookTypes.filter(
+            (type) => !expectedHookTypes.includes(type)
           );
         }
-      } catch (error) {
+      } catch (_error) {
         console.warn(chalk.yellow('⚠ Failed to read settings.json'));
       }
     }
   }
 
   // Files to preserve
-  manifest.preserve = [
-    '.sylphx-flow/',
-    '.secrets/',
-    target.config.configFile || '',
-  ]
+  manifest.preserve = ['.sylphx-flow/', '.secrets/', target.config.configFile || '']
     .filter(Boolean)
     .map((p) => path.join(cwd, p));
 
@@ -379,7 +378,9 @@ export interface SelectedToRemove {
 /**
  * Select unknown files to remove
  */
-export async function selectUnknownFilesToRemove(manifest: SyncManifest): Promise<SelectedToRemove> {
+export async function selectUnknownFilesToRemove(
+  manifest: SyncManifest
+): Promise<SelectedToRemove> {
   const unknownFiles: Array<{ name: string; value: string; type: string }> = [];
 
   // Collect all unknown files
@@ -463,10 +464,7 @@ export async function selectUnknownFilesToRemove(manifest: SyncManifest): Promis
 /**
  * Show final summary before execution
  */
-export function showFinalSummary(
-  manifest: SyncManifest,
-  selectedUnknowns: SelectedToRemove
-): void {
+export function showFinalSummary(manifest: SyncManifest, selectedUnknowns: SelectedToRemove): void {
   console.log(chalk.cyan.bold('\n━━━ 📋 Final Summary\n'));
 
   // Will delete + reinstall
@@ -490,7 +488,10 @@ export function showFinalSummary(
   }
 
   // Will remove (selected unknowns)
-  const totalToRemove = selectedUnknowns.files.length + selectedUnknowns.mcpServers.length + selectedUnknowns.hooks.length;
+  const totalToRemove =
+    selectedUnknowns.files.length +
+    selectedUnknowns.mcpServers.length +
+    selectedUnknowns.hooks.length;
   if (totalToRemove > 0) {
     console.log(chalk.red('Remove (selected):\n'));
     selectedUnknowns.files.forEach((file) => {
@@ -506,7 +507,11 @@ export function showFinalSummary(
   }
 
   // Will preserve
-  const allSelected = [...selectedUnknowns.files, ...selectedUnknowns.mcpServers, ...selectedUnknowns.hooks];
+  const allSelected = [
+    ...selectedUnknowns.files,
+    ...selectedUnknowns.mcpServers,
+    ...selectedUnknowns.hooks,
+  ];
   const preservedUnknowns = [
     ...manifest.agents.unknown,
     ...manifest.slashCommands.unknown,
@@ -618,14 +623,10 @@ export async function removeMCPServers(cwd: string, serversToRemove: string[]):
     }
 
     // Write back
-    await fs.promises.writeFile(
-      mcpPath,
-      JSON.stringify(mcpConfig, null, 2) + '\n',
-      'utf-8'
-    );
+    await fs.promises.writeFile(mcpPath, `${JSON.stringify(mcpConfig, null, 2)}\n`, 'utf-8');
 
     return removedCount;
-  } catch (error) {
+  } catch (_error) {
     console.warn(chalk.yellow('⚠ Failed to update .mcp.json'));
     return 0;
   }
@@ -659,14 +660,10 @@ export async function removeHooks(cwd: string, hooksToRemove: string[]): Promise
     }
 
     // Write back
-    await fs.promises.writeFile(
-      settingsPath,
-      JSON.stringify(settings, null, 2) + '\n',
-      'utf-8'
-    );
+    await fs.promises.writeFile(settingsPath, `${JSON.stringify(settings, null, 2)}\n`, 'utf-8');
 
     return removedCount;
-  } catch (error) {
+  } catch (_error) {
     console.warn(chalk.yellow('⚠ Failed to update settings.json'));
     return 0;
   }

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';
 
 /**
@@ -51,7 +51,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 {