@sylphx/flow 1.7.0 → 1.8.1

package/src/core/connection-pool.ts

@@ -1,390 +0,0 @@
-/**
- * Database Connection Pool
- *
- * Manages a pool of database connections for better performance and resource management
- */
-
-export interface ConnectionConfig {
-  maxConnections?: number;
-  minConnections?: number;
-  acquireTimeout?: number;
-  idleTimeout?: number;
-  maxLifetime?: number;
-  healthCheckInterval?: number;
-}
-
-export interface Connection<T = any> {
-  id: string;
-  instance: T;
-  createdAt: number;
-  lastUsed: number;
-  isInUse: boolean;
-  isHealthy: boolean;
-}
-
-export interface ConnectionPoolInstance<T> {
-  acquire(): Promise<T>;
-  release(connectionInstance: T): Promise<void>;
-  getStats(): {
-    totalConnections: number;
-    activeConnections: number;
-    idleConnections: number;
-    unhealthyConnections: number;
-    maxConnections?: number;
-    minConnections?: number;
-  };
-  dispose(): Promise<void>;
-}
-
-/**
- * Create a connection pool for database connections
- */
-export function createConnectionPool<T>(
-  createConnection: () => Promise<T>,
-  destroyConnection: (connection: T) => Promise<void>,
-  healthCheck: (connection: T) => Promise<boolean>,
-  configInput: ConnectionConfig = {}
-): ConnectionPoolInstance<T> {
-  // Closure-based state
-  const connections = new Map<string, Connection<T>>();
-  let availableConnections: string[] = [];
-  let connectionCount = 0;
-  let healthCheckTimer: NodeJS.Timeout | undefined;
-  let isDisposing = false;
-
-  const config: Required<ConnectionConfig> = {
-    maxConnections: 10,
-    minConnections: 2,
-    acquireTimeout: 30000,
-    idleTimeout: 300000,
-    maxLifetime: 3600000,
-    healthCheckInterval: 60000,
-    ...configInput,
-  };
-
-  /**
-   * Generate a unique connection ID
-   */
-  const generateConnectionId = (): string => {
-    return `conn_${Date.now()}_${Math.random().toString(36).substr(2, 9)}`;
-  };
-
-  /**
-   * Check if a connection is valid
-   */
-  const isConnectionValid = (connection: Connection<T>): boolean => {
-    const now = Date.now();
-
-    // Check age
-    if (now - connection.createdAt > config.maxLifetime) {
-      return false;
-    }
-
-    // Check if it's idle for too long (but only if not in use)
-    if (!connection.isInUse && now - connection.lastUsed > config.idleTimeout) {
-      return false;
-    }
-
-    // Check health
-    return connection.isHealthy;
-  };
-
-  /**
-   * Create a new connection
-   */
-  const createNewConnection = async (): Promise<T> => {
-    const startTime = Date.now();
-    const connectionInstance = await createConnection();
-    const createTime = Date.now() - startTime;
-
-    const connection: Connection<T> = {
-      id: generateConnectionId(),
-      instance: connectionInstance,
-      createdAt: Date.now(),
-      lastUsed: Date.now(),
-      isInUse: true,
-      isHealthy: true,
-    };
-
-    connections.set(connection.id, connection);
-    connectionCount++;
-
-    console.debug(
-      `New connection created in ${createTime}ms, total connections: ${connectionCount}`
-    );
-    return connectionInstance;
-  };
-
-  /**
-   * Wait for a connection to become available
-   */
-  const waitForConnection = async (): Promise<T> => {
-    const timeout = config.acquireTimeout;
-    const startTime = Date.now();
-
-    return new Promise((resolve, reject) => {
-      const checkInterval = setInterval(() => {
-        if (isDisposing) {
-          clearInterval(checkInterval);
-          reject(new Error('Connection pool is disposing'));
-          return;
-        }
-
-        if (availableConnections.length > 0) {
-          clearInterval(checkInterval);
-          const connectionId = availableConnections.shift()!;
-          const connection = connections.get(connectionId)!;
-
-          if (isConnectionValid(connection)) {
-            connection.isInUse = true;
-            connection.lastUsed = Date.now();
-            resolve(connection.instance);
-          } else {
-            // Remove invalid connection and try again
-            connections.delete(connectionId);
-            connectionCount--;
-            destroyConnection(connection.instance).catch(console.error);
-            return;
-          }
-        }
-
-        // Check timeout
-        if (Date.now() - startTime > timeout) {
-          clearInterval(checkInterval);
-          reject(new Error('Connection acquire timeout'));
-        }
-      }, 100);
-    });
-  };
-
-  /**
-   * Maintain minimum connections
-   */
-  const maintainMinConnections = async (): Promise<void> => {
-    const minConnections = config.minConnections;
-    const availableCount = availableConnections.length;
-
-    if (availableCount < minConnections && connectionCount < config.maxConnections) {
-      const needed = minConnections - availableCount;
-      for (let i = 0; i < needed; i++) {
-        createNewConnection()
-          .then((connection) => release(connection))
-          .catch((error) => console.error('Failed to maintain minimum connection:', error));
-      }
-    }
-  };
-
-  /**
-   * Start health check timer
-   */
-  const startHealthCheck = (): void => {
-    const interval = config.healthCheckInterval;
-
-    healthCheckTimer = setInterval(async () => {
-      if (isDisposing) {
-        return;
-      }
-
-      for (const [id, connection] of connections) {
-        try {
-          const isHealthy = await healthCheck(connection.instance);
-
-          if (isHealthy) {
-            connection.isHealthy = true;
-          } else {
-            connection.isHealthy = false;
-            console.warn(`Connection ${id} failed health check`);
-
-            // Remove unhealthy connection if not in use
-            if (!connection.isInUse) {
-              connections.delete(id);
-              connectionCount--;
-              const index = availableConnections.indexOf(id);
-              if (index > -1) {
-                availableConnections.splice(index, 1);
-              }
-              await destroyConnection(connection.instance);
-            }
-          }
-        } catch (error) {
-          console.error(`Health check failed for connection ${id}:`, error);
-          connection.isHealthy = false;
-        }
-      }
-
-      // Maintain minimum connections
-      await maintainMinConnections();
-    }, interval);
-  };
-
-  /**
-   * Initialize minimum connections
-   */
-  const initializeMinConnections = async (): Promise<void> => {
-    const minConnections = config.minConnections;
-    const promises = [];
-
-    for (let i = 0; i < minConnections; i++) {
-      promises.push(
-        createNewConnection()
-          .then((connection) => release(connection))
-          .catch((error) => console.error('Failed to initialize minimum connection:', error))
-      );
-    }
-
-    await Promise.all(promises);
-  };
-
-  /**
-   * Acquire a connection from the pool
-   */
-  const acquire = async (): Promise<T> => {
-    if (isDisposing) {
-      throw new Error('Connection pool is disposing');
-    }
-
-    // Try to get an available connection
-    while (availableConnections.length > 0) {
-      const connectionId = availableConnections.shift()!;
-      const connection = connections.get(connectionId)!;
-
-      if (isConnectionValid(connection)) {
-        connection.isInUse = true;
-        connection.lastUsed = Date.now();
-        return connection.instance;
-      }
-      // Remove invalid connection
-      connections.delete(connectionId);
-      connectionCount--;
-      await destroyConnection(connection.instance);
-    }
-
-    // Create new connection if under limit
-    if (connectionCount < config.maxConnections) {
-      return await createNewConnection();
-    }
-
-    // Wait for a connection to become available
-    return await waitForConnection();
-  };
-
-  /**
-   * Release a connection back to the pool
-   */
-  const release = async (connectionInstance: T): Promise<void> => {
-    for (const [id, connection] of connections) {
-      if (connection.instance === connectionInstance) {
-        connection.isInUse = false;
-        connection.lastUsed = Date.now();
-        availableConnections.push(id);
-        return;
-      }
-    }
-
-    // Connection not found in pool, destroy it
-    await destroyConnection(connectionInstance);
-  };
-
-  /**
-   * Get pool statistics
-   */
-  const getStats = () => {
-    const activeConnections = Array.from(connections.values()).filter((c) => c.isInUse).length;
-    const idleConnections = availableConnections.length;
-    const unhealthyConnections = Array.from(connections.values()).filter(
-      (c) => !c.isHealthy
-    ).length;
-
-    return {
-      totalConnections: connectionCount,
-      activeConnections,
-      idleConnections,
-      unhealthyConnections,
-      maxConnections: config.maxConnections,
-      minConnections: config.minConnections,
-    };
-  };
-
-  /**
-   * Close all connections and dispose the pool
-   */
-  const dispose = async (): Promise<void> => {
-    isDisposing = true;
-
-    if (healthCheckTimer) {
-      clearInterval(healthCheckTimer);
-      healthCheckTimer = undefined;
-    }
-
-    const destroyPromises = Array.from(connections.values()).map(async (connection) => {
-      try {
-        await destroyConnection(connection.instance);
-      } catch (error) {
-        console.error('Error destroying connection during pool disposal:', error);
-      }
-    });
-
-    await Promise.all(destroyPromises);
-
-    connections.clear();
-    availableConnections = [];
-    connectionCount = 0;
-
-    console.log('Connection pool disposed');
-  };
-
-  // Initialize pool
-  startHealthCheck();
-  initializeMinConnections();
-
-  return {
-    acquire,
-    release,
-    getStats,
-    dispose,
-  };
-}
-
-/**
- * @deprecated Use createConnectionPool() for new code
- */
-export class ConnectionPool<T> {
-  private instance: ConnectionPoolInstance<T>;
-
-  constructor(
-    createConnection: () => Promise<T>,
-    destroyConnection: (connection: T) => Promise<void>,
-    healthCheck: (connection: T) => Promise<boolean>,
-    config: ConnectionConfig = {}
-  ) {
-    this.instance = createConnectionPool(createConnection, destroyConnection, healthCheck, config);
-  }
-
-  async acquire(): Promise<T> {
-    return this.instance.acquire();
-  }
-
-  async release(connectionInstance: T): Promise<void> {
-    return this.instance.release(connectionInstance);
-  }
-
-  getStats() {
-    return this.instance.getStats();
-  }
-
-  async dispose(): Promise<void> {
-    return this.instance.dispose();
-  }
-}
-
-/**
- * Create a connection pool for database connections
- */
-export function createDatabaseConnectionPool(
-  createDbConnection: () => Promise<any>,
-  destroyDbConnection: (connection: any) => Promise<void>,
-  healthCheckFn: (connection: any) => Promise<boolean>,
-  config?: ConnectionConfig
-): ConnectionPoolInstance<any> {
-  return createConnectionPool(createDbConnection, destroyDbConnection, healthCheckFn, config);
-}

package/src/core/di-container.ts

@@ -1,155 +0,0 @@
-/**
- * Lightweight Dependency Injection Container
- *
- * A minimal DI container focusing on core services: database, logging, configuration
- * Uses constructor injection pattern for better testability
- */
-
-export type ServiceFactory<T> = () => T | Promise<T>;
-export type ServiceLifetime = 'singleton' | 'transient' | 'scoped';
-
-export interface ServiceDescriptor<T = any> {
-  factory: ServiceFactory<T>;
-  lifetime: ServiceLifetime;
-  instance?: T;
-  isResolved?: boolean;
-}
-
-export class DIContainer {
-  private services = new Map<string, ServiceDescriptor>();
-  private scopedInstances = new Map<string, any>();
-
-  /**
-   * Register a service with the container
-   */
-  register<T>(
-    token: string,
-    factory: ServiceFactory<T>,
-    lifetime: ServiceLifetime = 'singleton'
-  ): void {
-    this.services.set(token, {
-      factory,
-      lifetime,
-      isResolved: false,
-    });
-  }
-
-  /**
-   * Register a singleton instance
-   */
-  registerInstance<T>(token: string, instance: T): void {
-    this.services.set(token, {
-      factory: () => instance,
-      lifetime: 'singleton',
-      instance,
-      isResolved: true,
-    });
-  }
-
-  /**
-   * Resolve a service from the container
-   */
-  async resolve<T>(token: string): Promise<T> {
-    const descriptor = this.services.get(token);
-
-    if (!descriptor) {
-      throw new Error(`Service not registered: ${token}`);
-    }
-
-    switch (descriptor.lifetime) {
-      case 'singleton':
-        return this.resolveSingleton<T>(descriptor);
-      case 'transient':
-        return this.resolveTransient<T>(descriptor);
-      case 'scoped':
-        return this.resolveScoped<T>(token, descriptor);
-      default:
-        throw new Error(`Unsupported service lifetime: ${descriptor.lifetime}`);
-    }
-  }
-
-  /**
-   * Check if a service is registered
-   */
-  isRegistered(token: string): boolean {
-    return this.services.has(token);
-  }
-
-  /**
-   * Create a new scope for scoped services
-   */
-  createScope(): DIContainer {
-    const scope = new DIContainer();
-    // Copy all service descriptors but not instances
-    for (const [token, descriptor] of this.services) {
-      scope.services.set(token, { ...descriptor });
-    }
-    return scope;
-  }
-
-  /**
-   * Clear scoped instances (useful for request cleanup)
-   */
-  clearScope(): void {
-    this.scopedInstances.clear();
-  }
-
-  /**
-   * Dispose all singleton services that have dispose method
-   */
-  async dispose(): Promise<void> {
-    for (const descriptor of this.services.values()) {
-      if (descriptor.instance && typeof descriptor.instance.dispose === 'function') {
-        try {
-          await descriptor.instance.dispose();
-        } catch (error) {
-          console.error('Error disposing service:', error);
-        }
-      }
-    }
-    this.services.clear();
-    this.scopedInstances.clear();
-  }
-
-  private async resolveSingleton<T>(descriptor: ServiceDescriptor<T>): Promise<T> {
-    if (descriptor.isResolved && descriptor.instance) {
-      return descriptor.instance;
-    }
-
-    const instance = await descriptor.factory();
-    descriptor.instance = instance;
-    descriptor.isResolved = true;
-
-    return instance;
-  }
-
-  private async resolveTransient<T>(descriptor: ServiceDescriptor<T>): Promise<T> {
-    return await descriptor.factory();
-  }
-
-  private async resolveScoped<T>(token: string, descriptor: ServiceDescriptor<T>): Promise<T> {
-    if (this.scopedInstances.has(token)) {
-      return this.scopedInstances.get(token);
-    }
-
-    const instance = await descriptor.factory();
-    this.scopedInstances.set(token, instance);
-
-    return instance;
-  }
-}
-
-// Global container instance
-export const container = new DIContainer();
-
-// Service tokens constants
-export const SERVICE_TOKENS = {
-  DATABASE: 'database',
-  LOGGER: 'logger',
-  CONFIG: 'config',
-  MEMORY_STORAGE: 'memoryStorage',
-  SEARCH_SERVICE: 'searchService',
-  MCP_SERVICE: 'mcpService',
-  EMBEDDING_PROVIDER: 'embeddingProvider',
-  TARGET_MANAGER: 'targetManager',
-} as const;

package/src/core/headless-display.ts

@@ -1,96 +0,0 @@
-/**
- * Headless Display
- * Formatting and display logic for headless mode (non-TUI)
- */
-
-import chalk from 'chalk';
-
-/**
- * Format tool arguments for display
- */
-function formatArgs(args: unknown): string {
-  if (!args || typeof args !== 'object') {
-    return '';
-  }
-
-  const argsStr = Object.keys(args).length === 0
-    ? ''
-    : JSON.stringify(args, null, 2);
-
-  if (!argsStr) {
-    return '';
-  }
-
-  const lines = argsStr.split('\n');
-  const truncated = lines.length > 5
-    ? lines.slice(0, 5).join('\n') + chalk.dim('\n     … +' + (lines.length - 5) + ' lines')
-    : argsStr;
-
-  return truncated;
-}
-
-/**
- * Format tool result for display
- */
-function formatResult(result: unknown): string {
-  const resultStr = JSON.stringify(result, null, 2);
-  const lines = resultStr.split('\n');
-  const truncated = lines.length > 5
-    ? lines.slice(0, 5).join('\n') + chalk.dim('\n     … +' + (lines.length - 5) + ' lines')
-    : resultStr;
-
-  return truncated;
-}
-
-/**
- * Display callbacks for headless mode
- */
-export function createHeadlessDisplay(quiet: boolean) {
-  let hasOutput = false;
-
-  return {
-    onToolCall: (toolName: string, args: unknown) => {
-      if (quiet) return;
-
-      // Flush stdout to ensure proper ordering
-      if (hasOutput) {
-        process.stdout.write('\n');
-      }
-
-      const argsStr = formatArgs(args);
-      if (argsStr) {
-        process.stderr.write(`\n${chalk.green('⏺')} ${chalk.bold(toolName)}\n`);
-        process.stderr.write(chalk.dim(`  ⎿ ${argsStr.split('\n').join('\n     ')}\n`));
-      } else {
-        process.stderr.write(`\n${chalk.green('⏺')} ${chalk.bold(toolName)}\n`);
-      }
-    },
-
-    onToolResult: (toolName: string, result: unknown, duration: number) => {
-      if (quiet) return;
-
-      const resultStr = formatResult(result);
-      process.stderr.write(`${chalk.green('●')} ${chalk.bold(toolName)} ${chalk.dim(`(${duration}ms)`)}\n`);
-      process.stderr.write(chalk.dim(`  ⎿ ${resultStr.split('\n').join('\n     ')}\n\n`));
-    },
-
-    onTextDelta: (text: string) => {
-      if (!hasOutput) {
-        hasOutput = true;
-        // Add newline before first text output if we're not in quiet mode
-        if (!quiet) {
-          process.stdout.write('\n');
-        }
-      }
-      process.stdout.write(text);
-    },
-
-    onComplete: () => {
-      if (hasOutput) {
-        process.stdout.write('\n\n');
-      }
-    },
-
-    hasOutput: () => hasOutput,
-  };
-}

package/src/core/interfaces/index.ts

@@ -1,22 +0,0 @@
-/**
- * Core interfaces for dependency inversion
- * All abstractions in one place
- *
- * DESIGN RATIONALE:
- * - Single source of truth for contracts
- * - Enables dependency injection
- * - Facilitates testing
- * - Clear separation of interface and implementation
- */
-
-// Re-export commonly used types from existing interfaces
-export type {
-  CoreService,
-  FileProcessor,
-  IDatabaseConnection,
-  ILogger,
-  InitializationOptions,
-  TargetManager,
-} from '../interfaces.js';
-export * from './repository.interface.js';
-export * from './service.interface.js';