@sylphx/flow 2.1.2 → 2.1.4

package/src/targets/claude-code.ts

@@ -7,10 +7,10 @@ import { 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 { CLIError } from '../utils/error-handler.js';
 import { getAgentsDir, getSlashCommandsDir } from '../utils/config/paths.js';
-import { sanitize } from '../utils/security/security.js';
 import { fileUtils, generateHelpText, pathUtils, yamlUtils } from '../utils/config/target-utils.js';
+import { CLIError } from '../utils/error-handler.js';
+import { sanitize } from '../utils/security/security.js';
 
 /**
  * Claude Code target - composition approach with all original functionality
@@ -24,6 +24,7 @@ export const claudeCodeTarget: Target = {
   isDefault: false,
 
   config: {
+    configDir: '.claude',
     agentDir: '.claude/agents',
     agentExtension: '.md',
     agentFormat: 'yaml-frontmatter',
@@ -218,8 +219,12 @@ Please begin your response with a comprehensive summary of all the instructions
     if (options.dryRun) {
       // Build the command for display
       const dryRunArgs = ['claude', '--dangerously-skip-permissions'];
-      if (options.print) dryRunArgs.push('-p');
-      if (options.continue) dryRunArgs.push('-c');
+      if (options.print) {
+        dryRunArgs.push('-p');
+      }
+      if (options.continue) {
+        dryRunArgs.push('-c');
+      }
       dryRunArgs.push('--system-prompt', '"<agent content>"');
       if (sanitizedUserPrompt.trim() !== '') {
         dryRunArgs.push(`"${sanitizedUserPrompt}"`);
@@ -297,14 +302,18 @@ Please begin your response with a comprehensive summary of all the instructions
           reject(error);
         });
       });
-    } catch (error: any) {
-      if (error.code === 'ENOENT') {
+    } catch (error: unknown) {
+      const err = error as NodeJS.ErrnoException & { code?: string | number };
+      if (err.code === 'ENOENT') {
         throw new CLIError('Claude Code not found. Please install it first.', 'CLAUDE_NOT_FOUND');
       }
-      if (error.code) {
-        throw new CLIError(`Claude Code exited with code ${error.code}`, 'CLAUDE_ERROR');
+      if (err.code) {
+        throw new CLIError(`Claude Code exited with code ${err.code}`, 'CLAUDE_ERROR');
       }
-      throw new CLIError(`Failed to execute Claude Code: ${error.message}`, 'CLAUDE_ERROR');
+      throw new CLIError(
+        `Failed to execute Claude Code: ${(error as Error).message}`,
+        'CLAUDE_ERROR'
+      );
     }
   },
 
@@ -333,8 +342,9 @@ Please begin your response with a comprehensive summary of all the instructions
       try {
         const content = await fsPromises.readFile(settingsPath, 'utf8');
         settings = JSON.parse(content);
-      } catch (error: any) {
-        if (error.code !== 'ENOENT') {
+      } catch (error: unknown) {
+        const err = error as NodeJS.ErrnoException;
+        if (err.code !== 'ENOENT') {
           throw error;
         }
         // File doesn't exist, will create new
@@ -376,7 +386,9 @@ Please begin your response with a comprehensive summary of all the instructions
    * Configure session and prompt hooks for system information display
    */
   async setupHooks(cwd: string, _options: CommonOptions): Promise<SetupResult> {
-    const { processSettings, generateHookCommands } = await import('./functional/claude-code-logic.js');
+    const { processSettings, generateHookCommands } = await import(
+      './functional/claude-code-logic.js'
+    );
     const { pathExists, createDirectory, readFile, writeFile } = await import(
       '../composables/functional/useFileSystem.js'
     );
@@ -428,7 +440,8 @@ Please begin your response with a comprehensive summary of all the instructions
 
   /**
    * Setup agents for Claude Code
-   * Install agents to .claude/agents/ directory with rules and output styles appended
+   * Install agents to .claude/agents/ directory with rules appended
+   * Output styles are applied dynamically at runtime based on user settings
    */
   async setupAgents(cwd: string, options: CommonOptions): Promise<SetupResult> {
     const { enhanceAgentContent } = await import('../utils/agent-enhancer.js');
@@ -445,8 +458,8 @@ Please begin your response with a comprehensive summary of all the instructions
         // Transform agent content (converts to Claude Code format, strips unsupported fields)
         const transformed = await this.transformAgentContent(content, undefined, sourcePath);
 
-        // Enhance with rules and output styles
-        const enhanced = await enhanceAgentContent(transformed, rules);
+        // Enhance with rules only (output styles are applied dynamically at runtime)
+        const enhanced = await enhanceAgentContent(transformed, rules, []);
 
         return enhanced;
       },

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,8 +12,9 @@
 
 import fs from 'node:fs/promises';
 import path from 'node:path';
-import { getOutputStylesDir, getRulesDir } 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';
 
 /**
  * Load and combine rules and output styles
@@ -57,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`);
       }
@@ -87,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`);
         }
       }
@@ -130,3 +131,110 @@ export async function enhanceAgentContent(
 
   return `${agentContent}\n\n---\n\n# Rules and Output Styles\n\n${rulesAndStyles}`;
 }
+
+/**
+ * Filter rules: intersection of agent's required rules and globally enabled rules
+ * @param agentRules - Rules defined in agent frontmatter
+ * @param enabledRules - Globally enabled rules from user settings
+ * @returns Intersection of both arrays (rules that are both required by agent AND enabled globally)
+ */
+function filterRules(agentRules?: string[], enabledRules?: string[]): string[] | undefined {
+  // If agent doesn't define rules, return undefined (will use default)
+  if (!agentRules || agentRules.length === 0) {
+    return undefined;
+  }
+
+  // If no global filter, use all agent rules
+  if (!enabledRules || enabledRules.length === 0) {
+    return agentRules;
+  }
+
+  // Return intersection: rules that are both in agent's list AND globally enabled
+  const filtered = agentRules.filter((rule) => enabledRules.includes(rule));
+  return filtered.length > 0 ? filtered : undefined;
+}
+
+/**
+ * Load agent content from various locations
+ * @param agentName - Name of the agent (without .md extension)
+ * @param agentFilePath - Optional specific file path to load from
+ * @param enabledRules - Globally enabled rules (filtered with agent's frontmatter rules)
+ * @param enabledOutputStyles - Optional array of enabled output style names
+ */
+export async function loadAgentContent(
+  agentName: string,
+  agentFilePath?: string,
+  enabledRules?: string[],
+  enabledOutputStyles?: string[]
+): Promise<string> {
+  try {
+    // If specific file path provided, load from there
+    if (agentFilePath) {
+      const content = await fs.readFile(path.resolve(agentFilePath), 'utf-8');
+      // Extract rules from agent frontmatter
+      const { metadata } = await yamlUtils.extractFrontMatter(content);
+      const agentRules = metadata.rules as string[] | undefined;
+      // Filter: intersection of agent's rules and globally enabled rules
+      const rulesToLoad = filterRules(agentRules, enabledRules);
+      // Enhance with filtered rules and enabled output styles
+      return await enhanceAgentContent(content, rulesToLoad, enabledOutputStyles);
+    }
+
+    // First try to load from .claude/agents/ directory (processed agents with rules already included)
+    const claudeAgentPath = path.join(process.cwd(), '.claude', 'agents', `${agentName}.md`);
+
+    try {
+      const content = await fs.readFile(claudeAgentPath, 'utf-8');
+      // Enhance with enabled output styles (rules are already included in the file)
+      return await enhanceAgentContent(content, [], enabledOutputStyles);
+    } catch (_error) {
+      // Try to load from local agents/ directory (user-defined agents)
+      const localAgentPath = path.join(process.cwd(), 'agents', `${agentName}.md`);
+
+      try {
+        const content = await fs.readFile(localAgentPath, 'utf-8');
+        // Extract rules from agent frontmatter
+        const { metadata } = await yamlUtils.extractFrontMatter(content);
+        const agentRules = metadata.rules as string[] | undefined;
+        // Filter: intersection of agent's rules and globally enabled rules
+        const rulesToLoad = filterRules(agentRules, enabledRules);
+        // Enhance with filtered rules and enabled output styles
+        return await enhanceAgentContent(content, rulesToLoad, enabledOutputStyles);
+      } catch (_error2) {
+        // Try to load from the package's agents directory
+        const packageAgentsDir = getAgentsDir();
+        const packageAgentPath = path.join(packageAgentsDir, `${agentName}.md`);
+
+        const content = await fs.readFile(packageAgentPath, 'utf-8');
+        // Extract rules from agent frontmatter
+        const { metadata } = await yamlUtils.extractFrontMatter(content);
+        const agentRules = metadata.rules as string[] | undefined;
+        // Filter: intersection of agent's rules and globally enabled rules
+        const rulesToLoad = filterRules(agentRules, enabledRules);
+        // Enhance with filtered rules and enabled output styles
+        return await enhanceAgentContent(content, rulesToLoad, enabledOutputStyles);
+      }
+    }
+  } catch (_error) {
+    throw new CLIError(
+      `Agent '${agentName}' not found${agentFilePath ? ` at ${agentFilePath}` : ''}`,
+      'AGENT_NOT_FOUND'
+    );
+  }
+}
+
+/**
+ * Extract agent instructions from agent content (strip YAML front matter)
+ */
+export function extractAgentInstructions(agentContent: string): string {
+  // Extract content after YAML front matter
+  const yamlFrontMatterRegex = /^---\s*\n[\s\S]*?\n---\s*\n/;
+  const match = agentContent.match(yamlFrontMatterRegex);
+
+  if (match) {
+    return agentContent.substring(match[0].length).trim();
+  }
+
+  // If no front matter, return the full content
+  return agentContent.trim();
+}

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 },