agent-relay 2.3.13 → 2.3.15

package/packages/wrapper/src/client.ts

@@ -1,1166 +0,0 @@
-/**
- * Relay Client
- * Connects to the daemon and handles message sending/receiving.
- *
- * @deprecated **MIGRATION REQUIRED** - This module will be removed in a future version.
- *
- * ## Migration Path
- *
- * Replace imports from `@agent-relay/wrapper` with `@agent-relay/sdk`:
- *
- * ```typescript
- * // BEFORE (deprecated)
- * import { RelayClient } from '@agent-relay/wrapper';
- *
- * // AFTER (recommended)
- * import { RelayClient } from '@agent-relay/sdk';
- * ```
- *
- * The `@agent-relay/sdk` provides the same API with:
- * - Identical method signatures
- * - Same configuration options
- * - Compatible event handlers
- *
- * ## Timeline
- *
- * - Current: Deprecation warning on first use
- * - Next minor: TypeScript deprecation errors
- * - Next major: This module will be removed
- *
- * ## Internal Use Only
- *
- * This client is retained only for internal daemon/wrapper integration.
- * External consumers should always use `@agent-relay/sdk`.
- *
- * Optimizations:
- * - Monotonic ID generation (faster than UUID)
- * - Write coalescing (batch socket writes)
- * - Circular dedup cache (O(1) eviction)
- */
-
-import net from 'node:net';
-import { randomUUID } from 'node:crypto';
-import { generateId } from './id-generator.js';
-// Import types from SDK (re-exported via protocol for compatibility)
-import {
-  type Envelope,
-  type HelloPayload,
-  type WelcomePayload,
-  type SendPayload,
-  type SendMeta,
-  type SendEnvelope,
-  type DeliverEnvelope,
-  type AckPayload,
-  type ErrorPayload,
-  type PayloadKind,
-  type SpeakOnTrigger,
-  type LogPayload,
-  type EntityType,
-  type SpawnPayload,
-  type SpawnResultPayload,
-  type ReleasePayload,
-  type ReleaseResultPayload,
-  PROTOCOL_VERSION,
-} from '@agent-relay/protocol/types';
-import {
-  type SpawnResult,
-  type ReleaseResult,
-  toSpawnResult,
-  toReleaseResult,
-} from '@agent-relay/utils/client-helpers';
-
-// Re-export types for consumers
-export type { SpawnResult, ReleaseResult };
-import type {
-  ChannelMessagePayload,
-  ChannelJoinEnvelope,
-  ChannelLeaveEnvelope,
-  ChannelMessageEnvelope,
-  MessageAttachment,
-} from '@agent-relay/protocol/channels';
-import { encodeFrameLegacy, FrameParser } from '@agent-relay/protocol/framing';
-import { DEFAULT_SOCKET_PATH } from '@agent-relay/config/relay-config';
-
-export type ClientState = 'DISCONNECTED' | 'CONNECTING' | 'HANDSHAKING' | 'READY' | 'BACKOFF';
-
-export interface SyncOptions {
-  timeoutMs?: number;
-  kind?: PayloadKind;
-  data?: Record<string, unknown>;
-  thread?: string;
-}
-
-export interface ClientConfig {
-  socketPath: string;
-  agentName: string;
-  /** Entity type: 'agent' (default) or 'user' for human users */
-  entityType?: EntityType;
-  /** Optional CLI identifier to surface to the dashboard */
-  cli?: string;
-  /** Optional program identifier (e.g., 'claude', 'gpt-4o') */
-  program?: string;
-  /** Optional model identifier (e.g., 'claude-3-opus-2024-xx') */
-  model?: string;
-  /** Optional task description for registry/dashboard */
-  task?: string;
-  /** Optional working directory to surface in registry/dashboard */
-  workingDirectory?: string;
-  /** Display name for human users */
-  displayName?: string;
-  /** Avatar URL for human users */
-  avatarUrl?: string;
-  /** Suppress client-side console logging */
-  quiet?: boolean;
-  reconnect: boolean;
-  maxReconnectAttempts: number;
-  reconnectDelayMs: number;
-  reconnectMaxDelayMs: number;
-  /** Internal flag to suppress deprecation warning (for wrapper internal use only) */
-  _internal?: boolean;
-}
-
-const DEFAULT_CLIENT_CONFIG: ClientConfig = {
-  socketPath: DEFAULT_SOCKET_PATH,
-  agentName: 'agent',
-  cli: undefined,
-  quiet: false,
-  reconnect: true,
-  maxReconnectAttempts: 10,
-  reconnectDelayMs: 100,
-  reconnectMaxDelayMs: 30000,
-};
-
-/**
- * Circular buffer for O(1) deduplication with bounded memory.
- */
-class CircularDedupeCache {
-  private ids: Set<string> = new Set();
-  private ring: string[];
-  private head = 0;
-  private readonly capacity: number;
-
-  constructor(capacity = 2000) {
-    this.capacity = capacity;
-    this.ring = new Array(capacity);
-  }
-
-  /** Returns true if duplicate (already seen) */
-  check(id: string): boolean {
-    if (this.ids.has(id)) return true;
-
-    // Evict oldest if at capacity
-    if (this.ids.size >= this.capacity) {
-      const oldest = this.ring[this.head];
-      if (oldest) this.ids.delete(oldest);
-    }
-
-    // Add new ID
-    this.ring[this.head] = id;
-    this.ids.add(id);
-    this.head = (this.head + 1) % this.capacity;
-
-    return false;
-  }
-
-  clear(): void {
-    this.ids.clear();
-    this.ring = new Array(this.capacity);
-    this.head = 0;
-  }
-}
-
-/**
- * @deprecated Use `RelayClient` from `@agent-relay/sdk` instead.
- * This class will be removed in a future major version.
- */
-export class RelayClient {
-  /** Track if deprecation warning has been shown (warn once per process) */
-  private static _deprecationWarningShown = false;
-
-  private config: ClientConfig;
-  private socket?: net.Socket;
-  private parser: FrameParser;
-
-  private _state: ClientState = 'DISCONNECTED';
-  private sessionId?: string;
-  private resumeToken?: string;
-  private reconnectAttempts = 0;
-  private reconnectDelay: number;
-  private reconnectTimer?: NodeJS.Timeout;
-  private _destroyed = false;
-
-  // Circular dedup cache (O(1) eviction vs O(n) array shift)
-  private dedupeCache = new CircularDedupeCache(2000);
-
-  // Write coalescing: batch multiple writes into single syscall
-  private writeQueue: Buffer[] = [];
-  private writeScheduled = false;
-
-  private pendingSyncAcks: Map<string, { resolve: (ack: AckPayload) => void; reject: (err: Error) => void; timeoutHandle: NodeJS.Timeout }> = new Map();
-
-  // Event handlers
-  /**
-   * Handler for incoming messages.
-   * @param from - The sender agent name
-   * @param payload - The message payload
-   * @param messageId - Unique message ID
-   * @param meta - Optional message metadata
-   * @param originalTo - Original 'to' field from sender (e.g., '*' for broadcasts)
-   */
-  onMessage?: (from: string, payload: SendPayload, messageId: string, meta?: SendMeta, originalTo?: string) => void;
-  /**
-   * Callback for channel messages.
-   * @param from - Sender name
-   * @param channel - Channel name
-   * @param body - Message content
-   * @param envelope - Full envelope for additional data
-   */
-  onChannelMessage?: (from: string, channel: string, body: string, envelope: Envelope<ChannelMessagePayload>) => void;
-  onStateChange?: (state: ClientState) => void;
-  onError?: (error: Error) => void;
-
-  constructor(config: Partial<ClientConfig> = {}) {
-    // Show deprecation warning once per process (skip for internal wrapper usage)
-    if (!RelayClient._deprecationWarningShown && !config._internal) {
-      RelayClient._deprecationWarningShown = true;
-      console.warn(
-        '\x1b[33m[DEPRECATION WARNING]\x1b[0m RelayClient from @agent-relay/wrapper is deprecated.\n' +
-        '  Migrate to @agent-relay/sdk:\n' +
-        '    import { RelayClient } from \'@agent-relay/sdk\';\n' +
-        '  This module will be removed in a future major version.'
-      );
-    }
-
-    this.config = { ...DEFAULT_CLIENT_CONFIG, ...config };
-    this.parser = new FrameParser();
-    this.parser.setLegacyMode(true); // Use 4-byte header for backwards compatibility
-    this.reconnectDelay = this.config.reconnectDelayMs;
-  }
-
-  get state(): ClientState {
-    return this._state;
-  }
-
-  get agentName(): string {
-    return this.config.agentName;
-  }
-
-  /** Get the session ID assigned by the server */
-  get currentSessionId(): string | undefined {
-    return this.sessionId;
-  }
-
-  /**
-   * Connect to the relay daemon.
-   */
-  connect(): Promise<void> {
-    if (this._state !== 'DISCONNECTED' && this._state !== 'BACKOFF') {
-      return Promise.resolve();
-    }
-
-    return new Promise((resolve, reject) => {
-      let settled = false;
-      const settleResolve = (): void => {
-        if (settled) return;
-        settled = true;
-        resolve();
-      };
-      const settleReject = (err: Error): void => {
-        if (settled) return;
-        settled = true;
-        reject(err);
-      };
-
-      this.setState('CONNECTING');
-
-      this.socket = net.createConnection(this.config.socketPath, () => {
-        this.setState('HANDSHAKING');
-        this.sendHello();
-      });
-
-      this.socket.on('data', (data) => this.handleData(data));
-
-      this.socket.on('close', () => {
-        this.handleDisconnect();
-      });
-
-      this.socket.on('error', (err) => {
-        if (this._state === 'CONNECTING') {
-          settleReject(err);
-        }
-        this.handleError(err);
-      });
-
-      // Wait for WELCOME
-      const checkReady = setInterval(() => {
-        if (this._state === 'READY') {
-          clearInterval(checkReady);
-          clearTimeout(timeout);
-          settleResolve();
-        }
-      }, 10);
-
-      // Timeout
-      const timeout = setTimeout(() => {
-        if (this._state !== 'READY') {
-          clearInterval(checkReady);
-          this.socket?.destroy();
-          settleReject(new Error('Connection timeout'));
-        }
-      }, 5000);
-    });
-  }
-
-  /**
-   * Disconnect from the relay daemon.
-   */
-  disconnect(): void {
-    if (this.reconnectTimer) {
-      clearTimeout(this.reconnectTimer);
-      this.reconnectTimer = undefined;
-    }
-
-    if (this.socket) {
-      this.send({
-        v: PROTOCOL_VERSION,
-        type: 'BYE',
-        id: generateId(),
-        ts: Date.now(),
-        payload: {},
-      });
-      this.socket.end();
-      this.socket = undefined;
-    }
-
-    this.setState('DISCONNECTED');
-  }
-
-  /**
-   * Permanently destroy the client. Disconnects and prevents any reconnection.
-   */
-  destroy(): void {
-    this._destroyed = true;
-    this.disconnect();
-  }
-
-  /**
-   * Send a message to another agent.
-   * @param to - Target agent name or '*' for broadcast
-   * @param body - Message body
-   * @param kind - Message type (default: 'message')
-   * @param data - Optional structured data
-   * @param thread - Optional thread ID for grouping related messages
-   * @param meta - Optional message metadata (importance, replyTo, etc.)
-   */
-  sendMessage(to: string, body: string, kind: PayloadKind = 'message', data?: Record<string, unknown>, thread?: string, meta?: SendMeta): boolean {
-    if (this._state !== 'READY') {
-      return false;
-    }
-
-    const envelope: SendEnvelope = {
-      v: PROTOCOL_VERSION,
-      type: 'SEND',
-      id: generateId(),
-      ts: Date.now(),
-      to,
-      payload: {
-        kind,
-        body,
-        data,
-        thread,
-      },
-      payload_meta: meta,
-    };
-
-    return this.send(envelope);
-  }
-
-  /**
-   * Send an ACK for a delivered message.
-   */
-  sendAck(payload: AckPayload): boolean {
-    if (this._state !== 'READY') {
-      return false;
-    }
-
-    const envelope: Envelope<AckPayload> = {
-      v: PROTOCOL_VERSION,
-      type: 'ACK',
-      id: generateId(),
-      ts: Date.now(),
-      payload,
-    };
-
-    return this.send(envelope);
-  }
-
-  /**
-   * Send a message and wait for a correlated ACK response.
-   */
-  async sendAndWait(to: string, body: string, options: SyncOptions = {}): Promise<AckPayload> {
-    if (this._state !== 'READY') {
-      throw new Error('Client not ready');
-    }
-
-    const correlationId = randomUUID();
-    const timeoutMs = options.timeoutMs ?? 30000;
-    const kind = options.kind ?? 'message';
-
-    return new Promise<AckPayload>((resolve, reject) => {
-      const timeoutHandle = setTimeout(() => {
-        this.pendingSyncAcks.delete(correlationId);
-        reject(new Error(`ACK timeout after ${timeoutMs}ms`));
-      }, timeoutMs);
-
-      this.pendingSyncAcks.set(correlationId, { resolve, reject, timeoutHandle });
-
-      const envelope: SendEnvelope = {
-        v: PROTOCOL_VERSION,
-        type: 'SEND',
-        id: generateId(),
-        ts: Date.now(),
-        to,
-        payload: {
-          kind,
-          body,
-          data: options.data,
-          thread: options.thread,
-        },
-        payload_meta: {
-          sync: {
-            correlationId,
-            timeoutMs,
-            blocking: true,
-          },
-        },
-      };
-
-      const sent = this.send(envelope);
-      if (!sent) {
-        clearTimeout(timeoutHandle);
-        this.pendingSyncAcks.delete(correlationId);
-        reject(new Error('Failed to send message'));
-      }
-    });
-  }
-
-  /**
-   * Broadcast a message to all agents.
-   */
-  broadcast(body: string, kind: PayloadKind = 'message', data?: Record<string, unknown>): boolean {
-    return this.sendMessage('*', body, kind, data);
-  }
-
-  // =============================================================================
-  // Channel Operations
-  // =============================================================================
-
-  /**
-   * Join a channel.
-   * @param channel - Channel name (e.g., '#general', 'dm:alice:bob')
-   * @param displayName - Optional display name for this member
-   */
-  joinChannel(channel: string, displayName?: string): boolean {
-    if (this._state !== 'READY') {
-      return false;
-    }
-
-    const envelope: ChannelJoinEnvelope = {
-      v: PROTOCOL_VERSION,
-      type: 'CHANNEL_JOIN',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        channel,
-        displayName,
-      },
-    };
-
-    return this.send(envelope);
-  }
-
-  /**
-   * Admin join: Add any member to a channel (does not require member to be connected).
-   * Used by dashboard to sync channel memberships for agents.
-   * @param channel - Channel name (e.g., '#general')
-   * @param member - Name of the member to add
-   */
-  adminJoinChannel(channel: string, member: string): boolean {
-    if (this._state !== 'READY') {
-      return false;
-    }
-
-    const envelope: ChannelJoinEnvelope = {
-      v: PROTOCOL_VERSION,
-      type: 'CHANNEL_JOIN',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        channel,
-        member, // Admin mode: specify member to add
-      },
-    };
-
-    return this.send(envelope);
-  }
-
-  /**
-   * Leave a channel.
-   * @param channel - Channel name to leave
-   * @param reason - Optional reason for leaving
-   */
-  leaveChannel(channel: string, reason?: string): boolean {
-    if (this._state !== 'READY') return false;
-
-    const envelope: ChannelLeaveEnvelope = {
-      v: PROTOCOL_VERSION,
-      type: 'CHANNEL_LEAVE',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        channel,
-        reason,
-      },
-    };
-
-    return this.send(envelope);
-  }
-
-  /**
-   * Admin remove: Remove any member from a channel (does not require member to be connected).
-   * Used by dashboard to remove channel members.
-   * @param channel - Channel name (e.g., '#general')
-   * @param member - Name of the member to remove
-   */
-  adminRemoveMember(channel: string, member: string): boolean {
-    if (this._state !== 'READY') {
-      return false;
-    }
-
-    const envelope: ChannelLeaveEnvelope = {
-      v: PROTOCOL_VERSION,
-      type: 'CHANNEL_LEAVE',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        channel,
-        member, // Admin mode: specify member to remove
-      },
-    };
-
-    return this.send(envelope);
-  }
-
-  /**
-   * Send a message to a channel.
-   * @param channel - Channel name
-   * @param body - Message content
-   * @param options - Optional thread, mentions, attachments
-   */
-  sendChannelMessage(
-    channel: string,
-    body: string,
-    options?: {
-      thread?: string;
-      mentions?: string[];
-      attachments?: MessageAttachment[];
-      data?: Record<string, unknown>;
-    }
-  ): boolean {
-    if (this._state !== 'READY') {
-      return false;
-    }
-
-    const envelope: ChannelMessageEnvelope = {
-      v: PROTOCOL_VERSION,
-      type: 'CHANNEL_MESSAGE',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        channel,
-        body,
-        thread: options?.thread,
-        mentions: options?.mentions,
-        attachments: options?.attachments,
-        data: options?.data,
-      },
-    };
-
-    return this.send(envelope);
-  }
-
-  /**
-   * Subscribe to a topic.
-   */
-  subscribe(topic: string): boolean {
-    if (this._state !== 'READY') return false;
-
-    return this.send({
-      v: PROTOCOL_VERSION,
-      type: 'SUBSCRIBE',
-      id: generateId(),
-      ts: Date.now(),
-      topic,
-      payload: {},
-    });
-  }
-
-  /**
-   * Unsubscribe from a topic.
-   */
-  unsubscribe(topic: string): boolean {
-    if (this._state !== 'READY') return false;
-
-    return this.send({
-      v: PROTOCOL_VERSION,
-      type: 'UNSUBSCRIBE',
-      id: generateId(),
-      ts: Date.now(),
-      topic,
-      payload: {},
-    });
-  }
-
-  /**
-   * Bind this agent as a shadow to a primary agent.
-   * As a shadow, this agent will receive copies of messages to/from the primary.
-   * @param primaryAgent - The agent to shadow
-   * @param options - Shadow configuration options
-   */
-  bindAsShadow(
-    primaryAgent: string,
-    options: {
-      /** When this shadow should speak (default: ['EXPLICIT_ASK']) */
-      speakOn?: SpeakOnTrigger[];
-      /** Receive copies of messages TO the primary (default: true) */
-      receiveIncoming?: boolean;
-      /** Receive copies of messages FROM the primary (default: true) */
-      receiveOutgoing?: boolean;
-    } = {}
-  ): boolean {
-    if (this._state !== 'READY') return false;
-
-    return this.send({
-      v: PROTOCOL_VERSION,
-      type: 'SHADOW_BIND',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        primaryAgent,
-        speakOn: options.speakOn,
-        receiveIncoming: options.receiveIncoming,
-        receiveOutgoing: options.receiveOutgoing,
-      },
-    });
-  }
-
-  /**
-   * Unbind this agent from a primary agent (stop shadowing).
-   * @param primaryAgent - The agent to stop shadowing
-   */
-  unbindAsShadow(primaryAgent: string): boolean {
-    if (this._state !== 'READY') return false;
-
-    return this.send({
-      v: PROTOCOL_VERSION,
-      type: 'SHADOW_UNBIND',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        primaryAgent,
-      },
-    });
-  }
-
-
-  /**
-   * Send log/output data to the daemon for dashboard streaming.
-   * Used by daemon-connected agents (not spawned workers) to stream their output.
-   * @param data - The log/output data to send
-   * @returns true if sent successfully, false otherwise
-   */
-  sendLog(data: string): boolean {
-    if (this._state !== 'READY') {
-      return false;
-    }
-
-    const envelope: Envelope<LogPayload> = {
-      v: PROTOCOL_VERSION,
-      type: 'LOG',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        data,
-        timestamp: Date.now(),
-      },
-    };
-
-    return this.send(envelope);
-  }
-
-  // =============================================================================
-  // Spawn/Release Operations
-  // =============================================================================
-
-  /**
-   * Spawn a worker agent via the daemon.
-   * @param options - Spawn configuration
-   * @returns Promise resolving to spawn result
-   */
-  async spawn(options: {
-    name: string;
-    cli: string;
-    task: string;
-    model?: string;
-    cwd?: string;
-    team?: string;
-    interactive?: boolean;
-    spawnerName?: string;
-    userId?: string;
-    includeWorkflowConventions?: boolean;
-    shadowMode?: 'subagent' | 'process';
-    shadowOf?: string;
-    shadowAgent?: string;
-    shadowTriggers?: SpeakOnTrigger[];
-    shadowSpeakOn?: SpeakOnTrigger[];
-  }): Promise<SpawnResult> {
-    if (this._state !== 'READY') {
-      return { success: false, error: 'Client not ready' };
-    }
-
-    try {
-      const payload: SpawnPayload = {
-        name: options.name,
-        cli: options.cli,
-        task: options.task,
-        model: options.model,
-        cwd: options.cwd,
-        team: options.team,
-        interactive: options.interactive,
-        spawnerName: options.spawnerName || this.config.agentName,
-        userId: options.userId,
-        includeWorkflowConventions: options.includeWorkflowConventions,
-        shadowMode: options.shadowMode,
-        shadowOf: options.shadowOf,
-        shadowAgent: options.shadowAgent,
-        shadowTriggers: options.shadowTriggers,
-        shadowSpeakOn: options.shadowSpeakOn,
-      };
-
-      const result = await this.requestResponse<SpawnResultPayload>(
-        'SPAWN',
-        payload as unknown as Record<string, unknown>,
-        60000 // 60 second timeout for spawn
-      );
-
-      return toSpawnResult(result);
-    } catch (e) {
-      return { success: false, error: e instanceof Error ? e.message : String(e) };
-    }
-  }
-
-  /**
-   * Release (terminate) a worker agent via the daemon.
-   * @param name - Name of the agent to release
-   * @param reason - Optional reason for release
-   * @returns Promise resolving to release result
-   */
-  async release(name: string, reason?: string): Promise<ReleaseResult> {
-    if (this._state !== 'READY') {
-      return { success: false, error: 'Client not ready' };
-    }
-
-    try {
-      const payload: ReleasePayload = { name, reason };
-
-      const result = await this.requestResponse<ReleaseResultPayload>(
-        'RELEASE',
-        payload as unknown as Record<string, unknown>,
-        10000 // 10 second timeout for release
-      );
-
-      return toReleaseResult(result);
-    } catch (e) {
-      return { success: false, error: e instanceof Error ? e.message : String(e) };
-    }
-  }
-
-  /**
-   * Send an envelope and wait for a matching response.
-   * Used for request/response operations like spawn/release.
-   */
-  private requestResponse<T>(
-    type: string,
-    payload: Record<string, unknown>,
-    timeoutMs: number
-  ): Promise<T> {
-    return new Promise<T>((resolve, reject) => {
-      if (!this.socket) {
-        reject(new Error('Not connected'));
-        return;
-      }
-
-      const requestId = generateId();
-      let timeoutHandle: NodeJS.Timeout;
-      let responseHandler: (envelope: Envelope) => void;
-
-      const cleanup = (): void => {
-        clearTimeout(timeoutHandle);
-        // Remove the handler from pending response handlers
-        this.pendingResponses.delete(requestId);
-      };
-
-      timeoutHandle = setTimeout(() => {
-        cleanup();
-        reject(new Error(`Request timeout after ${timeoutMs}ms`));
-      }, timeoutMs);
-
-      responseHandler = (envelope: Envelope): void => {
-        // Match by replyTo or id
-        const replyTo = (envelope.payload as { replyTo?: string })?.replyTo;
-        if (replyTo === requestId || envelope.id === requestId) {
-          cleanup();
-
-          if (envelope.type === 'ERROR') {
-            const errorPayload = envelope.payload as { message?: string; code?: string };
-            reject(new Error(errorPayload.message || errorPayload.code || 'Unknown error'));
-          } else {
-            resolve(envelope.payload as T);
-          }
-        }
-      };
-
-      // Register handler for this request
-      this.pendingResponses.set(requestId, responseHandler);
-
-      // Send the request envelope
-      const envelope: Envelope = {
-        v: PROTOCOL_VERSION,
-        type: type as Envelope['type'],
-        id: requestId,
-        ts: Date.now(),
-        payload,
-      };
-
-      this.send(envelope);
-    });
-  }
-
-  // Map to track pending request/response handlers
-  private pendingResponses: Map<string, (envelope: Envelope) => void> = new Map();
-
-  private setState(state: ClientState): void {
-    this._state = state;
-    if (this.onStateChange) {
-      this.onStateChange(state);
-    }
-  }
-
-  private sendHello(): void {
-    const hello: Envelope<HelloPayload> = {
-      v: PROTOCOL_VERSION,
-      type: 'HELLO',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        agent: this.config.agentName,
-        entityType: this.config.entityType,
-        cli: this.config.cli,
-        program: this.config.program,
-        model: this.config.model,
-        task: this.config.task,
-        workingDirectory: this.config.workingDirectory,
-        displayName: this.config.displayName,
-        avatarUrl: this.config.avatarUrl,
-        capabilities: {
-          ack: true,
-          resume: true,
-          max_inflight: 256,
-          supports_topics: true,
-        },
-        session: this.resumeToken ? { resume_token: this.resumeToken } : undefined,
-      },
-    };
-
-    this.send(hello);
-  }
-
-  private send(envelope: Envelope): boolean {
-    if (!this.socket) return false;
-
-    try {
-      const frame = encodeFrameLegacy(envelope);
-      this.writeQueue.push(frame);
-
-      // Coalesce writes: schedule flush on next tick if not already scheduled
-      if (!this.writeScheduled) {
-        this.writeScheduled = true;
-        setImmediate(() => this.flushWrites());
-      }
-      return true;
-    } catch (err) {
-      this.handleError(err as Error);
-      return false;
-    }
-  }
-
-  /**
-   * Flush all queued writes in a single syscall.
-   */
-  private flushWrites(): void {
-    this.writeScheduled = false;
-    if (this.writeQueue.length === 0 || !this.socket) return;
-
-    if (this.writeQueue.length === 1) {
-      // Single frame - write directly (no concat needed)
-      this.socket.write(this.writeQueue[0]);
-    } else {
-      // Multiple frames - batch into single write
-      this.socket.write(Buffer.concat(this.writeQueue));
-    }
-    this.writeQueue = [];
-  }
-
-  private handleData(data: Buffer): void {
-    try {
-      const frames = this.parser.push(data);
-      for (const frame of frames) {
-        this.processFrame(frame);
-      }
-    } catch (err) {
-      this.handleError(err as Error);
-    }
-  }
-
-  private processFrame(envelope: Envelope): void {
-    // Check for pending request/response handlers first
-    // This handles SPAWN_RESULT, RELEASE_RESULT, and ERROR responses
-    const replyTo = (envelope.payload as { replyTo?: string })?.replyTo;
-    if (replyTo && this.pendingResponses.has(replyTo)) {
-      const handler = this.pendingResponses.get(replyTo);
-      if (handler) {
-        handler(envelope);
-        return;
-      }
-    }
-
-    switch (envelope.type) {
-      case 'WELCOME':
-        this.handleWelcome(envelope as Envelope<WelcomePayload>);
-        break;
-
-      case 'DELIVER':
-        this.handleDeliver(envelope as DeliverEnvelope);
-        break;
-
-      case 'CHANNEL_MESSAGE':
-        this.handleChannelMessage(envelope as Envelope<ChannelMessagePayload> & { from?: string });
-        break;
-
-      case 'PING':
-        this.handlePing(envelope);
-        break;
-
-      case 'ACK':
-        this.handleAck(envelope as Envelope<AckPayload>);
-        break;
-
-      case 'ERROR':
-        this.handleErrorFrame(envelope as Envelope<ErrorPayload>);
-        break;
-
-      case 'SPAWN_RESULT':
-      case 'RELEASE_RESULT':
-        // These should be handled by pending response handlers above
-        // If we get here, it's an orphaned response (no handler waiting)
-        break;
-
-      case 'BUSY':
-        console.warn('[client] Server busy, backing off');
-        break;
-    }
-  }
-
-  private handleWelcome(envelope: Envelope<WelcomePayload>): void {
-    this.sessionId = envelope.payload.session_id;
-    this.resumeToken = envelope.payload.resume_token;
-    this.reconnectAttempts = 0;
-    this.reconnectDelay = this.config.reconnectDelayMs;
-    this.setState('READY');
-    if (!this.config.quiet) {
-      console.log(`[client] Connected as ${this.config.agentName} (session: ${this.sessionId})`);
-    }
-  }
-
-  private handleDeliver(envelope: DeliverEnvelope): void {
-    // Send ACK
-    this.send({
-      v: PROTOCOL_VERSION,
-      type: 'ACK',
-      id: generateId(),
-      ts: Date.now(),
-      payload: {
-        ack_id: envelope.id,
-        seq: envelope.delivery.seq,
-      },
-    });
-
-    const duplicate = this.markDelivered(envelope.id);
-    if (duplicate) {
-      return;
-    }
-
-    // Notify handler
-    // Pass originalTo from delivery info so handlers know if this was a broadcast
-    if (this.onMessage && envelope.from) {
-      this.onMessage(envelope.from, envelope.payload, envelope.id, envelope.payload_meta, envelope.delivery.originalTo);
-    }
-  }
-
-  private handleAck(envelope: Envelope<AckPayload>): void {
-    const correlationId = envelope.payload.correlationId;
-    if (!correlationId) return;
-
-    const pending = this.pendingSyncAcks.get(correlationId);
-    if (!pending) return;
-
-    clearTimeout(pending.timeoutHandle);
-    this.pendingSyncAcks.delete(correlationId);
-    pending.resolve(envelope.payload);
-  }
-
-  private handleChannelMessage(envelope: Envelope<ChannelMessagePayload> & { from?: string }): void {
-    if (!this.config.quiet) {
-      console.log(`[client] handleChannelMessage: from=${envelope.from}, channel=${envelope.payload.channel}`);
-    }
-
-    const duplicate = this.markDelivered(envelope.id);
-    if (duplicate) {
-      if (!this.config.quiet) {
-        console.log(`[client] handleChannelMessage: duplicate message ${envelope.id}, skipping`);
-      }
-      return;
-    }
-
-    // Notify channel message handler
-    if (this.onChannelMessage && envelope.from) {
-      if (!this.config.quiet) {
-        console.log(`[client] Calling onChannelMessage callback`);
-      }
-      this.onChannelMessage(
-        envelope.from,
-        envelope.payload.channel,
-        envelope.payload.body,
-        envelope as Envelope<ChannelMessagePayload>
-      );
-    } else if (!this.config.quiet) {
-      console.log(`[client] No onChannelMessage handler set (handler=${!!this.onChannelMessage}, from=${envelope.from})`);
-    }
-
-    // Also call onMessage for backwards compatibility
-    // Convert to SendPayload format (channel is passed as 5th argument, not in payload)
-    if (this.onMessage && envelope.from) {
-      const sendPayload: SendPayload = {
-        kind: 'message',
-        body: envelope.payload.body,
-        data: {
-          _isChannelMessage: true,
-          _channel: envelope.payload.channel,
-          _mentions: envelope.payload.mentions,
-        },
-        thread: envelope.payload.thread,
-      };
-      this.onMessage(envelope.from, sendPayload, envelope.id, undefined, envelope.payload.channel);
-    }
-  }
-
-  private handlePing(envelope: Envelope): void {
-    this.send({
-      v: PROTOCOL_VERSION,
-      type: 'PONG',
-      id: generateId(),
-      ts: Date.now(),
-      payload: (envelope.payload as { nonce?: string }) ?? {},
-    });
-  }
-
-  private handleErrorFrame(envelope: Envelope<ErrorPayload>): void {
-    console.error('[client] Server error:', envelope.payload);
-
-    if (envelope.payload.code === 'RESUME_TOO_OLD') {
-      if (this.resumeToken) {
-        console.warn('[client] Resume token rejected, clearing and requesting new session');
-      }
-      // Clear resume token so next HELLO starts a fresh session instead of looping on an invalid token
-      this.resumeToken = undefined;
-      this.sessionId = undefined;
-    }
-  }
-
-  private handleDisconnect(): void {
-    this.parser.reset();
-    this.socket = undefined;
-    this.rejectPendingSyncAcks(new Error('Disconnected while awaiting ACK'));
-
-    // Don't reconnect if permanently destroyed
-    if (this._destroyed) {
-      this.setState('DISCONNECTED');
-      return;
-    }
-
-    if (this.config.reconnect && this.reconnectAttempts < this.config.maxReconnectAttempts) {
-      this.scheduleReconnect();
-    } else {
-      this.setState('DISCONNECTED');
-      if (this.reconnectAttempts >= this.config.maxReconnectAttempts) {
-        console.error(
-          `[client] Max reconnect attempts reached (${this.config.maxReconnectAttempts}), giving up`
-        );
-      }
-    }
-  }
-
-  private handleError(error: Error): void {
-    console.error('[client] Error:', error.message);
-    if (this.onError) {
-      this.onError(error);
-    }
-  }
-
-  private rejectPendingSyncAcks(error: Error): void {
-    for (const [correlationId, pending] of this.pendingSyncAcks.entries()) {
-      clearTimeout(pending.timeoutHandle);
-      pending.reject(error);
-      this.pendingSyncAcks.delete(correlationId);
-    }
-  }
-
-  private scheduleReconnect(): void {
-    this.setState('BACKOFF');
-    this.reconnectAttempts++;
-
-    // Exponential backoff with jitter
-    const jitter = Math.random() * 0.3 + 0.85; // 0.85 - 1.15
-    const delay = Math.min(this.reconnectDelay * jitter, this.config.reconnectMaxDelayMs);
-    this.reconnectDelay *= 2;
-
-    if (!this.config.quiet) {
-      console.log(`[client] Reconnecting in ${Math.round(delay)}ms (attempt ${this.reconnectAttempts})`);
-    }
-
-    this.reconnectTimer = setTimeout(() => {
-      this.connect().catch(() => {
-        // Will trigger another reconnect
-      });
-    }, delay);
-  }
-
-  /**
-   * Check if message was already delivered (deduplication).
-   * Uses circular buffer for O(1) eviction.
-   * @returns true if the message has already been seen.
-   */
-  private markDelivered(id: string): boolean {
-    return this.dedupeCache.check(id);
-  }
-}