@sylphx/flow 2.1.3 → 2.1.4

package/src/targets/opencode.ts

@@ -3,14 +3,14 @@ import path from 'node:path';
 import chalk from 'chalk';
 import { getRulesPath, ruleFileExists } from '../config/rules.js';
 import { MCP_SERVER_REGISTRY } from '../config/servers.js';
-import { installToDirectory, installFile } from '../core/installers/file-installer.js';
+import { installFile, installToDirectory } from '../core/installers/file-installer.js';
 import { createMCPInstaller } from '../core/installers/mcp-installer.js';
 import type { AgentMetadata } from '../types/target-config.types.js';
 import type { CommonOptions, MCPServerConfigUnion, SetupResult, Target } from '../types.js';
 import { getAgentsDir, getOutputStylesDir, getSlashCommandsDir } from '../utils/config/paths.js';
-import { secretUtils } from '../utils/security/secret-utils.js';
 import { fileUtils, generateHelpText, yamlUtils } from '../utils/config/target-utils.js';
 import { CLIError } from '../utils/error-handler.js';
+import { secretUtils } from '../utils/security/secret-utils.js';
 
 /**
  * OpenCode target - composition approach with all original functionality
@@ -24,6 +24,7 @@ export const opencodeTarget: Target = {
   isDefault: true,
 
   config: {
+    configDir: '.opencode',
     agentDir: '.opencode/agent',
     agentExtension: '.md',
     agentFormat: 'yaml-frontmatter',
@@ -63,7 +64,12 @@ export const opencodeTarget: Target = {
 
     // If additional metadata is provided, merge it (but exclude unsupported fields)
     if (metadata) {
-      const { name: additionalName, mode: additionalMode, rules: additionalRules, ...additionalCleanMetadata } = metadata;
+      const {
+        name: additionalName,
+        mode: additionalMode,
+        rules: additionalRules,
+        ...additionalCleanMetadata
+      } = metadata;
       const mergedMetadata = { ...cleanMetadata, ...additionalCleanMetadata };
       return yamlUtils.addFrontMatter(baseContent, mergedMetadata);
     }
@@ -397,9 +403,15 @@ export const opencodeTarget: Target = {
    * Execute OpenCode CLI
    */
   async executeCommand(
-    systemPrompt: string,
+    _systemPrompt: string,
     userPrompt: string,
-    options: { verbose?: boolean; dryRun?: boolean; print?: boolean; continue?: boolean; agent?: string } = {}
+    options: {
+      verbose?: boolean;
+      dryRun?: boolean;
+      print?: boolean;
+      continue?: boolean;
+      agent?: string;
+    } = {}
   ): Promise<void> {
     if (options.dryRun) {
       // Build the command for display
@@ -492,7 +504,6 @@ export const opencodeTarget: Target = {
           }
         });
       });
-
     } catch (error) {
       if (error instanceof Error) {
         throw new CLIError(`Failed to execute OpenCode: ${error.message}`, 'OPENCODE_ERROR');

package/src/types/cli.types.ts

@@ -82,6 +82,6 @@ export interface RunCommandOptions {
   agent?: string;
   agentFile?: string;
   prompt?: string;
-  print?: boolean;      // Headless print mode
-  continue?: boolean;   // Continue previous conversation
+  print?: boolean; // Headless print mode
+  continue?: boolean; // Continue previous conversation
 }

package/src/types/provider.types.ts

@@ -10,13 +10,7 @@
  * Provider IDs
  * All supported AI providers
  */
-export type ProviderId =
-  | 'anthropic'
-  | 'openai'
-  | 'google'
-  | 'openrouter'
-  | 'claude-code'
-  | 'zai';
+export type ProviderId = 'anthropic' | 'openai' | 'google' | 'openrouter' | 'claude-code' | 'zai';
 
 /**
  * Provider configuration value

package/src/types/session.types.ts

@@ -49,7 +49,7 @@ export type MessagePart =
   | {
       type: 'error';
       error: string;
-      status: 'completed';  // Errors are immediately completed
+      status: 'completed'; // Errors are immediately completed
     };
 
 /**
@@ -94,8 +94,8 @@ export interface TokenUsage {
  * - content: Shown in UI AND sent to LLM
  */
 export interface MessageMetadata {
-  cpu?: string;         // CPU usage at creation time (e.g., "45.3% (8 cores)")
-  memory?: string;      // Memory usage at creation time (e.g., "4.2GB/16.0GB")
+  cpu?: string; // CPU usage at creation time (e.g., "45.3% (8 cores)")
+  memory?: string; // Memory usage at creation time (e.g., "4.2GB/16.0GB")
   // Future: add more fields as needed (sessionId, requestId, modelVersion, etc.)
 }
 
@@ -136,14 +136,14 @@ export interface MessageMetadata {
  */
 export interface SessionMessage {
   role: 'user' | 'assistant';
-  content: MessagePart[];  // UI display (without system status)
+  content: MessagePart[]; // UI display (without system status)
   timestamp: number;
-  status?: 'active' | 'completed' | 'error' | 'abort';  // Message lifecycle state (default: 'completed')
-  metadata?: MessageMetadata;  // System info for LLM context (not shown in UI)
-  todoSnapshot?: Todo[];   // Full todo state at message creation time (for rewind + LLM context)
+  status?: 'active' | 'completed' | 'error' | 'abort'; // Message lifecycle state (default: 'completed')
+  metadata?: MessageMetadata; // System info for LLM context (not shown in UI)
+  todoSnapshot?: Todo[]; // Full todo state at message creation time (for rewind + LLM context)
   attachments?: FileAttachment[];
-  usage?: TokenUsage;          // For UI/monitoring, not sent to LLM
-  finishReason?: string;       // For flow control (stop/tool-calls/length/error), not sent to LLM
+  usage?: TokenUsage; // For UI/monitoring, not sent to LLM
+  finishReason?: string; // For flow control (stop/tool-calls/length/error), not sent to LLM
 }
 
 /**
@@ -203,8 +203,8 @@ export interface Session {
   provider: ProviderId;
   model: string;
   messages: SessionMessage[];
-  todos: Todo[];         // Per-session todo list (not global!)
-  nextTodoId: number;    // Next todo ID for this session (starts at 1)
+  todos: Todo[]; // Per-session todo list (not global!)
+  nextTodoId: number; // Next todo ID for this session (starts at 1)
 
   // Note: Streaming state derived from message.status, not stored here
   // To check if streaming: messages.some(m => m.status === 'active')

package/src/types/target.types.ts

@@ -4,13 +4,15 @@
  */
 
 import type { CommonOptions, SetupResult } from './common.types.js';
-import type { MCPServerConfigFlags, MCPServerConfigUnion } from './mcp.types.js';
+import type { MCPServerConfigUnion } from './mcp.types.js';
 
 /**
  * Target-specific configuration
  * Defines how agents, configs, and other artifacts are structured for each target
  */
 export interface TargetConfig {
+  /** Base configuration directory (e.g., '.claude', '.opencode') */
+  configDir: string;
   /** Directory where agents are installed */
   agentDir: string;
   /** File extension for agent files */

package/src/types/todo.types.ts

@@ -10,7 +10,7 @@ export interface Todo {
   content: string;
   status: TodoStatus;
   activeForm: string; // Present continuous form (e.g., "Building feature X")
-  ordering: number;   // For custom ordering (higher = earlier in list)
+  ordering: number; // For custom ordering (higher = earlier in list)
 }
 
 export interface TodoUpdate {

package/src/types.ts

@@ -12,9 +12,9 @@
 
 // Re-export all types for backward compatibility
 export type {
+  CommandConfig,
   CommandHandler,
   CommandOptions,
-  CommandConfig,
   RunCommandOptions,
 } from './types/cli.types.js';
 export type {

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

@@ -1,14 +1,14 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { afterEach, beforeEach, describe, expect, it } from 'vitest';
 import {
-  detectPackageManagerFromUserAgent,
-  detectPackageManagerFromLockFiles,
   detectPackageManager,
+  detectPackageManagerFromLockFiles,
+  detectPackageManagerFromUserAgent,
   getPackageManagerInfo,
   getUpgradeCommand,
 } from '../package-manager-detector';
-import fs from 'node:fs';
-import path from 'node:path';
-import os from 'node:os';
 
 describe('Package Manager Detection', () => {
   const originalEnv = process.env;

package/src/utils/agent-enhancer.ts

@@ -12,7 +12,7 @@
 
 import fs from 'node:fs/promises';
 import path from 'node:path';
-import { getOutputStylesDir, getRulesDir, getAgentsDir } from './config/paths.js';
+import { getAgentsDir, getOutputStylesDir, getRulesDir } from './config/paths.js';
 import { yamlUtils } from './config/target-utils.js';
 import { CLIError } from './error-handler.js';
 
@@ -58,7 +58,7 @@ async function loadRules(ruleNames?: string[]): Promise<string> {
         // Strip YAML front matter
         const stripped = await yamlUtils.stripFrontMatter(content);
         sections.push(stripped);
-      } catch (error) {
+      } catch (_error) {
         // Log warning if rule file not found, but continue with other rules
         console.warn(`Warning: Rule file not found: ${ruleName}.md`);
       }
@@ -88,7 +88,7 @@ async function loadOutputStyles(styleNames?: string[]): Promise<string> {
           const content = await fs.readFile(filePath, 'utf8');
           const stripped = await yamlUtils.stripFrontMatter(content);
           sections.push(stripped);
-        } catch (error) {
+        } catch (_error) {
           console.warn(`Warning: Output style file not found: ${styleName}.md`);
         }
       }
@@ -150,7 +150,7 @@ function filterRules(agentRules?: string[], enabledRules?: string[]): string[] |
   }
 
   // Return intersection: rules that are both in agent's list AND globally enabled
-  const filtered = agentRules.filter(rule => enabledRules.includes(rule));
+  const filtered = agentRules.filter((rule) => enabledRules.includes(rule));
   return filtered.length > 0 ? filtered : undefined;
 }
 

package/src/utils/config/paths.ts

@@ -39,7 +39,9 @@ export function findPackageRoot(context?: string): string {
     }
 
     const parentDir = path.dirname(currentDir);
-    if (parentDir === currentDir) break; // reached filesystem root
+    if (parentDir === currentDir) {
+      break; // reached filesystem root
+    }
     currentDir = parentDir;
   }
 

package/src/utils/config/target-utils.ts

@@ -288,7 +288,7 @@ export const pathUtils = {
     // Try to extract from content title
     const titleMatch = content.match(/^#\s+(.+?)(?:\s+Agent)?$/m);
     if (titleMatch) {
-      const title = titleMatch[1]!.trim().toLowerCase();
+      const title = titleMatch[1]?.trim().toLowerCase();
       const kebabTitle = title.replace(/\s+/g, '-').replace(/[^a-z0-9-]/g, '');
       return kebabTitle + (kebabTitle.includes('agent') ? '' : '-agent');
     }
@@ -319,7 +319,7 @@ export const pathUtils = {
   extractDescription(content: string): string {
     const firstParagraph = content.match(/^#\s+.+?\n\n(.+?)(?:\n\n|\n#|$)/s);
     if (firstParagraph) {
-      return firstParagraph[1]!.trim().replace(/\n+/g, ' ');
+      return firstParagraph[1]?.trim().replace(/\n+/g, ' ');
     }
     return 'Development agent for specialized tasks';
   },

package/src/utils/display/banner.ts

@@ -3,8 +3,8 @@
  * Welcome messages and branding
  */
 
-import chalk from 'chalk';
 import boxen from 'boxen';
+import chalk from 'chalk';
 
 /**
  * Display welcome banner
@@ -13,7 +13,7 @@ export function showWelcome(): void {
   console.log(
     boxen(
       `${chalk.cyan.bold('Sylphx Flow')} ${chalk.dim('- AI-Powered Development Framework')}\n` +
-      `${chalk.dim('Auto-initialization • Smart upgrades • One-click launch')}`,
+        `${chalk.dim('Auto-initialization • Smart upgrades • One-click launch')}`,
       {
         padding: 1,
         margin: { bottom: 1 },

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;