@claude-flow/shared 3.0.0-alpha.7 → 3.0.0-alpha.8

package/.claude-flow/logs/headless/optimize_1777379306973_an4lmy_result.log

@@ -0,0 +1,166 @@
+[2026-04-28T12:29:34.705Z] RESULT
+============================================================
+{
+  "success": true,
+  "output": "I'll analyze the provided codebase files for performance issues across the requested categories.\n\n## Performance Analysis\n\n---\n\n### 1. N+1 IO Pattern — `session-manager.ts`\n\n**Problem:** `terminateAllSessions` calls `terminateSession` in a loop, each of which fires a separate `persistSessions()` disk write. N sessions = N file writes.\n\n```typescript\n// CURRENT — N async writes, each writing the full sessions.json\nasync terminateAllSessions(): Promise<void> {\n  const sessions = this.getActiveSessions();\n  const batchSize = 5;\n  for (let i = 0; i < sessions.length; i += batchSize) {\n    const batch = sessions.slice(i, i + batchSize);\n    await Promise.allSettled(batch.map(s => this.terminateSession(s.id)));\n    // ↑ each terminateSession calls persistSessions() — batched calls, but still N writes\n  }\n}\n```\n\n**Fix:** Suppress persistence during batch, flush once after:\n\n```typescript\nasync terminateAllSessions(): Promise<void> {\n  const sessions = this.getActiveSessions();\n  // Mark all terminated in-memory first (no I/O)\n  for (const session of sessions) {\n    session.status = 'terminated';\n    session.endTime = new Date();\n    this.eventBus.emit(SystemEventTypes.SESSION_TERMINATED, {\n      sessionId: session.id,\n      agentId: session.agentId,\n      duration: session.endTime.getTime() - session.startTime.getTime(),\n    });\n    this.sessionProfiles.delete(session.id);\n  }\n  // Single write after all mutations\n  await this.persistSessions();\n}\n```\n\n---\n\n### 2. No React Code Found\n\nThe codebase is entirely backend TypeScript. No React components or rendering logic is present — skipping this section.\n\n---\n\n### 3. Caching Opportunities\n\n#### 3a. `HealthMonitor.getStatus()` — no result cache\n\n**Problem:** External callers (e.g., `getComponentHealth`) trigger a full re-evaluation of all checks even though the interval already runs them. Under load, every MCP status request re-runs every check.\n\n```typescript\n// CURRENT — runs all checks on every call\nasync getStatus(): Promise<IHealthStatus> { /* runs all checks */ }\n\nasync getComponentHealth(name: string) {\n  const status = await this.getStatus(); // ← full re-evaluation\n  return status.components[name];\n}\n```\n\n**Fix:** Cache the last result with a short TTL:\n\n```typescript\nprivate lastStatus: IHealthStatus | undefined;\nprivate lastStatusAt = 0;\nprivate readonly STATUS_CACHE_MS = 5000;\n\nasync getStatus(): Promise<IHealthStatus> {\n  const now = Date.now();\n  if (this.lastStatus && now - this.lastStatusAt < this.STATUS_CACHE_MS) {\n    return this.lastStatus;\n  }\n  const status = await this.computeStatus(); // extracted method\n  this.lastStatus = status;\n  this.lastStatusAt = now;\n  return status;\n}\n```\n\n#### 3b. `ConfigLoader.findConfigFile` — repeated filesystem probes\n\n**Problem:** `findConfigFile` probes all `CONFIG_FILE_NAMES` on every `load()` call with no caching. Repeated calls hit the filesystem unnecessarily.\n\n```typescript\n// Simple memoization\nprivate configFileCache = new Map<string, string | null>();\n\nprivate async findConfigFile(directory: string): Promise<string | null> {\n  if (this.configFileCache.has(directory)) {\n    return this.configFileCache.get(directory) ?? null;\n  }\n  // ...existing probe logic...\n  this.configFileCache.set(directory, result);\n  return result;\n}\n```\n\n#### 3c. `TaskManager.getTasks` — linear scan on hot path\n\n**Problem:** Every status/type/agent filter call creates a new array from the Map and scans all tasks. For 1000+ tasks, this is expensive on polling paths.\n\n**Fix:** Maintain secondary indexes:\n\n```typescript\nprivate tasksByStatus = new Map<TaskStatus, Set<string>>();\nprivate tasksByAgent = new Map<string, Set<string>>();\n\n// In createTask, assignTask, completeTask etc., keep indexes updated.\n// getTasks then becomes:\ngetTasks(filter?: Partial<Pick<ITask, 'status' | 'type' | 'assignedAgent'>>): ITask[] {\n  if (filter?.status) {\n    const ids = this.tasksByStatus.get(filter.status) ?? new Set();\n    return Array.from(ids, id => this.tasks.get(id)!).filter(Boolean);\n  }\n  // ...etc\n}\n```\n\n---\n\n### 4. Memory Leaks\n\n#### 4a. `EventBus` — `subscriptions` Map never pruned\n\n**Problem:** `subscribe()` adds to both `this.handlers` and `this.subscriptions`. `off()` only removes from `handlers`. The `subscriptions` Map grows indefinitely.\n\n```typescript\n// CURRENT — off() doesn't touch this.subscriptions\noff(type: string, handler: IEventHandler): void {\n  const handlers = this.handlers.get(type);\n  if (handlers) {\n    handlers.delete(handler);\n    // ↑ subscriptions map still holds the entry\n  }\n}\n```\n\n**Fix:** Ensure `removeSubscription` is implemented and called consistently:\n\n```typescript\nprivate removeSubscription(id: string): void {\n  const sub = this.subscriptions.get(id);\n  if (!sub) return;\n  for (const type of sub.filter.types ?? ['*']) {\n    const handlers = this.handlers.get(type);\n    handlers?.delete(sub.handler);\n    if (handlers?.size === 0) this.handlers.delete(type);\n  }\n  this.subscriptions.delete(id);\n}\n```\n\n#### 4b. `TaskManager.tasks` Map — completed tasks never evicted\n\n**Problem:** `this.tasks` accumulates every task ever created. The `cleanup` method exists in the interface but is never called automatically. This is an unbounded grow.\n\n**Fix:** Add a periodic self-cleaning or call cleanup in `completeTask`/`failTask` for tasks older than a configurable retention window. At minimum, schedule it:\n\n```typescript\nconstructor(private eventBus: IEventBus, queue?: ITaskQueue) {\n  this.queue = queue ?? new TaskQueue();\n  // Evict completed/failed tasks older than 1 hour every 10 minutes\n  setInterval(() => {\n    const cutoff = new Date(Date.now() - 3_600_000);\n    this.cleanup(cutoff).catch(() => {});\n  }, 600_000).unref(); // .unref() so it doesn't prevent process exit\n}\n```\n\n#### 4c. `HealthMonitor.history` — repeated array re-allocation\n\n**Problem:** Trimming creates a new array every interval cycle, generating GC pressure at the rate of `checkInterval`.\n\n```typescript\n// CURRENT — new array on every trim\nthis.history = this.history.slice(-this.config.historyLimit);\n```\n\n**Fix:** Splice in-place or use a fixed circular buffer:\n\n```typescript\nprivate addToHistory(status: IHealthStatus): void {\n  this.history.push(status);\n  if (this.history.length > this.config.historyLimit) {\n    this.history.splice(0, this.history.length - this.config.historyLimit);\n  }\n}\n```\n\n---\n\n### 5. Redundant Computations\n\n#### 5a. `TaskQueue.enqueue` — O(n log n) sort on every insertion\n\n**Problem:** Calling `Array.sort` after each enqueue is O(n log n). For a queue that receives many tasks, this adds up fast.\n\n```typescript\n// CURRENT\nasync enqueue(task: ITask): Promise<void> {\n  this.tasks.push(task);\n  this.tasks.sort((a, b) => b.priority - a.priority); // O(n log n) every time\n}\n```\n\n**Fix:** Binary-search insertion to maintain sorted order at O(log n):\n\n```typescript\nasync enqueue(task: ITask): Promise<void> {\n  let lo = 0, hi = this.tasks.length;\n  while (lo < hi) {\n    const mid = (lo + hi) >>> 1;\n    if (this.tasks[mid].priority >= task.priority) lo = mid + 1;\n    else hi = mid;\n  }\n  this.tasks.splice(lo, 0, task);\n}\n```\n\n#### 5b. `EventBus.dispatchEvent` — Set allocated per event\n\n**Problem:** A new `Set<IEventHandler>` is created on every dispatched event just to deduplicate handlers from two sources.\n\n```typescript\n// CURRENT — allocates a Set for every event\nprivate dispatchEvent<T>(event: IEvent<T>): void {\n  const allHandlers = new Set<IEventHandler>(); // ← new allocation per event\n  // ...\n}\n```\n\n**Fix:** Iterate directly, checking for presence in the first set:\n\n```typescript\nprivate dispatchEvent<T>(event: IEvent<T>): void {\n  const typeHandlers = this.handlers.get(event.type);\n  const wildcardHandlers = this.handlers.get('*');\n\n  if (typeHandlers) {\n    for (const handler of typeHandlers) this.invokeHandler(event, handler);\n  }\n  if (wildcardHandlers) {\n    for (const handler of wildcardHandlers) {\n      if (!typeHandlers?.has(handler)) this.invokeHandler(event, handler);\n    }\n  }\n}\n```\n\n#### 5c. `TaskManager.getTasks` — multi-pass filter\n\n**Problem:** When multiple filter properties are specified, separate `.filter()` calls each allocate a new array.\n\n```typescript\n// CURRENT — up to 3 array allocations\nif (filter.status) tasks = tasks.filter(t => t.status === filter.status);\nif (filter.type) tasks = tasks.filter(t => t.type === filter.type);\nif (filter.assignedAgent) tasks = tasks.filter(t => t.assignedAgent === filter.assignedAgent);\n```\n\n**Fix:** Single-pass composite filter:\n\n```typescript\ngetTasks(filter?: Partial<Pick<ITask, 'status' | 'type' | 'assignedAgent'>>): ITask[] {\n  if (!filter) return Array.from(this.tasks.values());\n  const { status, type, assignedAgent } = filter;\n  const result: ITask[] = [];\n  for (const t of this.tasks.values()) {\n    if (status && t.status !== status) continue;\n    if (type && t.type !== type) continue;\n    if (assignedAgent && t.assignedAgent !== assignedAgent) continue;\n    result.push(t);\n  }\n  return result;\n}\n```\n\n---\n\n### Priority Summary\n\n| Severity | Issue | File |\n|---|---|---|\n| High | N+1 disk writes on `terminateAllSessions` | `session-manager.ts` |\n| High | `TaskManager.tasks` unbounded growth | `task-manager.ts` |\n| High | `EventBus.subscriptions` never pruned | `event-bus.ts` |\n| Medium | `HealthMonitor.getStatus` re-evaluates on every external call | `health-monitor.ts` |\n| Medium | `TaskQueue.enqueue` O(n log n) sort per insert | `task-manager.ts` |\n| Low | `dispatchEvent` per-event Set allocation | `event-bus.ts` |\n| Low | `getTasks` multi-pass filter | `task-manager.ts` |\n| Low | `history.slice` allocation per interval | `health-monitor.ts` |\n",
+  "parsedOutput": {
+    "sections": [
+      {
+        "title": "Performance Analysis",
+        "content": "\n---\n\n",
+        "level": 2
+      },
+      {
+        "title": "1. N+1 IO Pattern — `session-manager.ts`",
+        "content": "\n**Problem:** `terminateAllSessions` calls `terminateSession` in a loop, each of which fires a separate `persistSessions()` disk write. N sessions = N file writes.\n\n```typescript\n// CURRENT — N async writes, each writing the full sessions.json\nasync terminateAllSessions(): Promise<void> {\n  const sessions = this.getActiveSessions();\n  const batchSize = 5;\n  for (let i = 0; i < sessions.length; i += batchSize) {\n    const batch = sessions.slice(i, i + batchSize);\n    await Promise.allSettled(batch.map(s => this.terminateSession(s.id)));\n    // ↑ each terminateSession calls persistSessions() — batched calls, but still N writes\n  }\n}\n```\n\n**Fix:** Suppress persistence during batch, flush once after:\n\n```typescript\nasync terminateAllSessions(): Promise<void> {\n  const sessions = this.getActiveSessions();\n  // Mark all terminated in-memory first (no I/O)\n  for (const session of sessions) {\n    session.status = 'terminated';\n    session.endTime = new Date();\n    this.eventBus.emit(SystemEventTypes.SESSION_TERMINATED, {\n      sessionId: session.id,\n      agentId: session.agentId,\n      duration: session.endTime.getTime() - session.startTime.getTime(),\n    });\n    this.sessionProfiles.delete(session.id);\n  }\n  // Single write after all mutations\n  await this.persistSessions();\n}\n```\n\n---\n\n",
+        "level": 3
+      },
+      {
+        "title": "2. No React Code Found",
+        "content": "\nThe codebase is entirely backend TypeScript. No React components or rendering logic is present — skipping this section.\n\n---\n\n",
+        "level": 3
+      },
+      {
+        "title": "3. Caching Opportunities",
+        "content": "\n",
+        "level": 3
+      },
+      {
+        "title": "3a. `HealthMonitor.getStatus()` — no result cache",
+        "content": "\n**Problem:** External callers (e.g., `getComponentHealth`) trigger a full re-evaluation of all checks even though the interval already runs them. Under load, every MCP status request re-runs every check.\n\n```typescript\n// CURRENT — runs all checks on every call\nasync getStatus(): Promise<IHealthStatus> { /* runs all checks */ }\n\nasync getComponentHealth(name: string) {\n  const status = await this.getStatus(); // ← full re-evaluation\n  return status.components[name];\n}\n```\n\n**Fix:** Cache the last result with a short TTL:\n\n```typescript\nprivate lastStatus: IHealthStatus | undefined;\nprivate lastStatusAt = 0;\nprivate readonly STATUS_CACHE_MS = 5000;\n\nasync getStatus(): Promise<IHealthStatus> {\n  const now = Date.now();\n  if (this.lastStatus && now - this.lastStatusAt < this.STATUS_CACHE_MS) {\n    return this.lastStatus;\n  }\n  const status = await this.computeStatus(); // extracted method\n  this.lastStatus = status;\n  this.lastStatusAt = now;\n  return status;\n}\n```\n\n",
+        "level": 4
+      },
+      {
+        "title": "3b. `ConfigLoader.findConfigFile` — repeated filesystem probes",
+        "content": "\n**Problem:** `findConfigFile` probes all `CONFIG_FILE_NAMES` on every `load()` call with no caching. Repeated calls hit the filesystem unnecessarily.\n\n```typescript\n// Simple memoization\nprivate configFileCache = new Map<string, string | null>();\n\nprivate async findConfigFile(directory: string): Promise<string | null> {\n  if (this.configFileCache.has(directory)) {\n    return this.configFileCache.get(directory) ?? null;\n  }\n  // ...existing probe logic...\n  this.configFileCache.set(directory, result);\n  return result;\n}\n```\n\n",
+        "level": 4
+      },
+      {
+        "title": "3c. `TaskManager.getTasks` — linear scan on hot path",
+        "content": "\n**Problem:** Every status/type/agent filter call creates a new array from the Map and scans all tasks. For 1000+ tasks, this is expensive on polling paths.\n\n**Fix:** Maintain secondary indexes:\n\n```typescript\nprivate tasksByStatus = new Map<TaskStatus, Set<string>>();\nprivate tasksByAgent = new Map<string, Set<string>>();\n\n// In createTask, assignTask, completeTask etc., keep indexes updated.\n// getTasks then becomes:\ngetTasks(filter?: Partial<Pick<ITask, 'status' | 'type' | 'assignedAgent'>>): ITask[] {\n  if (filter?.status) {\n    const ids = this.tasksByStatus.get(filter.status) ?? new Set();\n    return Array.from(ids, id => this.tasks.get(id)!).filter(Boolean);\n  }\n  // ...etc\n}\n```\n\n---\n\n",
+        "level": 4
+      },
+      {
+        "title": "4. Memory Leaks",
+        "content": "\n",
+        "level": 3
+      },
+      {
+        "title": "4a. `EventBus` — `subscriptions` Map never pruned",
+        "content": "\n**Problem:** `subscribe()` adds to both `this.handlers` and `this.subscriptions`. `off()` only removes from `handlers`. The `subscriptions` Map grows indefinitely.\n\n```typescript\n// CURRENT — off() doesn't touch this.subscriptions\noff(type: string, handler: IEventHandler): void {\n  const handlers = this.handlers.get(type);\n  if (handlers) {\n    handlers.delete(handler);\n    // ↑ subscriptions map still holds the entry\n  }\n}\n```\n\n**Fix:** Ensure `removeSubscription` is implemented and called consistently:\n\n```typescript\nprivate removeSubscription(id: string): void {\n  const sub = this.subscriptions.get(id);\n  if (!sub) return;\n  for (const type of sub.filter.types ?? ['*']) {\n    const handlers = this.handlers.get(type);\n    handlers?.delete(sub.handler);\n    if (handlers?.size === 0) this.handlers.delete(type);\n  }\n  this.subscriptions.delete(id);\n}\n```\n\n",
+        "level": 4
+      },
+      {
+        "title": "4b. `TaskManager.tasks` Map — completed tasks never evicted",
+        "content": "\n**Problem:** `this.tasks` accumulates every task ever created. The `cleanup` method exists in the interface but is never called automatically. This is an unbounded grow.\n\n**Fix:** Add a periodic self-cleaning or call cleanup in `completeTask`/`failTask` for tasks older than a configurable retention window. At minimum, schedule it:\n\n```typescript\nconstructor(private eventBus: IEventBus, queue?: ITaskQueue) {\n  this.queue = queue ?? new TaskQueue();\n  // Evict completed/failed tasks older than 1 hour every 10 minutes\n  setInterval(() => {\n    const cutoff = new Date(Date.now() - 3_600_000);\n    this.cleanup(cutoff).catch(() => {});\n  }, 600_000).unref(); // .unref() so it doesn't prevent process exit\n}\n```\n\n",
+        "level": 4
+      },
+      {
+        "title": "4c. `HealthMonitor.history` — repeated array re-allocation",
+        "content": "\n**Problem:** Trimming creates a new array every interval cycle, generating GC pressure at the rate of `checkInterval`.\n\n```typescript\n// CURRENT — new array on every trim\nthis.history = this.history.slice(-this.config.historyLimit);\n```\n\n**Fix:** Splice in-place or use a fixed circular buffer:\n\n```typescript\nprivate addToHistory(status: IHealthStatus): void {\n  this.history.push(status);\n  if (this.history.length > this.config.historyLimit) {\n    this.history.splice(0, this.history.length - this.config.historyLimit);\n  }\n}\n```\n\n---\n\n",
+        "level": 4
+      },
+      {
+        "title": "5. Redundant Computations",
+        "content": "\n",
+        "level": 3
+      },
+      {
+        "title": "5a. `TaskQueue.enqueue` — O(n log n) sort on every insertion",
+        "content": "\n**Problem:** Calling `Array.sort` after each enqueue is O(n log n). For a queue that receives many tasks, this adds up fast.\n\n```typescript\n// CURRENT\nasync enqueue(task: ITask): Promise<void> {\n  this.tasks.push(task);\n  this.tasks.sort((a, b) => b.priority - a.priority); // O(n log n) every time\n}\n```\n\n**Fix:** Binary-search insertion to maintain sorted order at O(log n):\n\n```typescript\nasync enqueue(task: ITask): Promise<void> {\n  let lo = 0, hi = this.tasks.length;\n  while (lo < hi) {\n    const mid = (lo + hi) >>> 1;\n    if (this.tasks[mid].priority >= task.priority) lo = mid + 1;\n    else hi = mid;\n  }\n  this.tasks.splice(lo, 0, task);\n}\n```\n\n",
+        "level": 4
+      },
+      {
+        "title": "5b. `EventBus.dispatchEvent` — Set allocated per event",
+        "content": "\n**Problem:** A new `Set<IEventHandler>` is created on every dispatched event just to deduplicate handlers from two sources.\n\n```typescript\n// CURRENT — allocates a Set for every event\nprivate dispatchEvent<T>(event: IEvent<T>): void {\n  const allHandlers = new Set<IEventHandler>(); // ← new allocation per event\n  // ...\n}\n```\n\n**Fix:** Iterate directly, checking for presence in the first set:\n\n```typescript\nprivate dispatchEvent<T>(event: IEvent<T>): void {\n  const typeHandlers = this.handlers.get(event.type);\n  const wildcardHandlers = this.handlers.get('*');\n\n  if (typeHandlers) {\n    for (const handler of typeHandlers) this.invokeHandler(event, handler);\n  }\n  if (wildcardHandlers) {\n    for (const handler of wildcardHandlers) {\n      if (!typeHandlers?.has(handler)) this.invokeHandler(event, handler);\n    }\n  }\n}\n```\n\n",
+        "level": 4
+      },
+      {
+        "title": "5c. `TaskManager.getTasks` — multi-pass filter",
+        "content": "\n**Problem:** When multiple filter properties are specified, separate `.filter()` calls each allocate a new array.\n\n```typescript\n// CURRENT — up to 3 array allocations\nif (filter.status) tasks = tasks.filter(t => t.status === filter.status);\nif (filter.type) tasks = tasks.filter(t => t.type === filter.type);\nif (filter.assignedAgent) tasks = tasks.filter(t => t.assignedAgent === filter.assignedAgent);\n```\n\n**Fix:** Single-pass composite filter:\n\n```typescript\ngetTasks(filter?: Partial<Pick<ITask, 'status' | 'type' | 'assignedAgent'>>): ITask[] {\n  if (!filter) return Array.from(this.tasks.values());\n  const { status, type, assignedAgent } = filter;\n  const result: ITask[] = [];\n  for (const t of this.tasks.values()) {\n    if (status && t.status !== status) continue;\n    if (type && t.type !== type) continue;\n    if (assignedAgent && t.assignedAgent !== assignedAgent) continue;\n    result.push(t);\n  }\n  return result;\n}\n```\n\n---\n\n",
+        "level": 4
+      },
+      {
+        "title": "Priority Summary",
+        "content": "| Severity | Issue | File |\n|---|---|---|\n| High | N+1 disk writes on `terminateAllSessions` | `session-manager.ts` |\n| High | `TaskManager.tasks` unbounded growth | `task-manager.ts` |\n| High | `EventBus.subscriptions` never pruned | `event-bus.ts` |\n| Medium | `HealthMonitor.getStatus` re-evaluates on every external call | `health-monitor.ts` |\n| Medium | `TaskQueue.enqueue` O(n log n) sort per insert | `task-manager.ts` |\n| Low | `dispatchEvent` per-event Set allocation | `event-bus.ts` |\n| Low | `getTasks` multi-pass filter | `task-manager.ts` |\n| Low | `history.slice` allocation per interval | `health-monitor.ts` |",
+        "level": 3
+      }
+    ],
+    "codeBlocks": [
+      {
+        "language": "typescript",
+        "code": "// CURRENT — N async writes, each writing the full sessions.json\nasync terminateAllSessions(): Promise<void> {\n  const sessions = this.getActiveSessions();\n  const batchSize = 5;\n  for (let i = 0; i < sessions.length; i += batchSize) {\n    const batch = sessions.slice(i, i + batchSize);\n    await Promise.allSettled(batch.map(s => this.terminateSession(s.id)));\n    // ↑ each terminateSession calls persistSessions() — batched calls, but still N writes\n  }\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "async terminateAllSessions(): Promise<void> {\n  const sessions = this.getActiveSessions();\n  // Mark all terminated in-memory first (no I/O)\n  for (const session of sessions) {\n    session.status = 'terminated';\n    session.endTime = new Date();\n    this.eventBus.emit(SystemEventTypes.SESSION_TERMINATED, {\n      sessionId: session.id,\n      agentId: session.agentId,\n      duration: session.endTime.getTime() - session.startTime.getTime(),\n    });\n    this.sessionProfiles.delete(session.id);\n  }\n  // Single write after all mutations\n  await this.persistSessions();\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "// CURRENT — runs all checks on every call\nasync getStatus(): Promise<IHealthStatus> { /* runs all checks */ }\n\nasync getComponentHealth(name: string) {\n  const status = await this.getStatus(); // ← full re-evaluation\n  return status.components[name];\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "private lastStatus: IHealthStatus | undefined;\nprivate lastStatusAt = 0;\nprivate readonly STATUS_CACHE_MS = 5000;\n\nasync getStatus(): Promise<IHealthStatus> {\n  const now = Date.now();\n  if (this.lastStatus && now - this.lastStatusAt < this.STATUS_CACHE_MS) {\n    return this.lastStatus;\n  }\n  const status = await this.computeStatus(); // extracted method\n  this.lastStatus = status;\n  this.lastStatusAt = now;\n  return status;\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "// Simple memoization\nprivate configFileCache = new Map<string, string | null>();\n\nprivate async findConfigFile(directory: string): Promise<string | null> {\n  if (this.configFileCache.has(directory)) {\n    return this.configFileCache.get(directory) ?? null;\n  }\n  // ...existing probe logic...\n  this.configFileCache.set(directory, result);\n  return result;\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "private tasksByStatus = new Map<TaskStatus, Set<string>>();\nprivate tasksByAgent = new Map<string, Set<string>>();\n\n// In createTask, assignTask, completeTask etc., keep indexes updated.\n// getTasks then becomes:\ngetTasks(filter?: Partial<Pick<ITask, 'status' | 'type' | 'assignedAgent'>>): ITask[] {\n  if (filter?.status) {\n    const ids = this.tasksByStatus.get(filter.status) ?? new Set();\n    return Array.from(ids, id => this.tasks.get(id)!).filter(Boolean);\n  }\n  // ...etc\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "// CURRENT — off() doesn't touch this.subscriptions\noff(type: string, handler: IEventHandler): void {\n  const handlers = this.handlers.get(type);\n  if (handlers) {\n    handlers.delete(handler);\n    // ↑ subscriptions map still holds the entry\n  }\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "private removeSubscription(id: string): void {\n  const sub = this.subscriptions.get(id);\n  if (!sub) return;\n  for (const type of sub.filter.types ?? ['*']) {\n    const handlers = this.handlers.get(type);\n    handlers?.delete(sub.handler);\n    if (handlers?.size === 0) this.handlers.delete(type);\n  }\n  this.subscriptions.delete(id);\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "constructor(private eventBus: IEventBus, queue?: ITaskQueue) {\n  this.queue = queue ?? new TaskQueue();\n  // Evict completed/failed tasks older than 1 hour every 10 minutes\n  setInterval(() => {\n    const cutoff = new Date(Date.now() - 3_600_000);\n    this.cleanup(cutoff).catch(() => {});\n  }, 600_000).unref(); // .unref() so it doesn't prevent process exit\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "// CURRENT — new array on every trim\nthis.history = this.history.slice(-this.config.historyLimit);"
+      },
+      {
+        "language": "typescript",
+        "code": "private addToHistory(status: IHealthStatus): void {\n  this.history.push(status);\n  if (this.history.length > this.config.historyLimit) {\n    this.history.splice(0, this.history.length - this.config.historyLimit);\n  }\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "// CURRENT\nasync enqueue(task: ITask): Promise<void> {\n  this.tasks.push(task);\n  this.tasks.sort((a, b) => b.priority - a.priority); // O(n log n) every time\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "async enqueue(task: ITask): Promise<void> {\n  let lo = 0, hi = this.tasks.length;\n  while (lo < hi) {\n    const mid = (lo + hi) >>> 1;\n    if (this.tasks[mid].priority >= task.priority) lo = mid + 1;\n    else hi = mid;\n  }\n  this.tasks.splice(lo, 0, task);\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "// CURRENT — allocates a Set for every event\nprivate dispatchEvent<T>(event: IEvent<T>): void {\n  const allHandlers = new Set<IEventHandler>(); // ← new allocation per event\n  // ...\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "private dispatchEvent<T>(event: IEvent<T>): void {\n  const typeHandlers = this.handlers.get(event.type);\n  const wildcardHandlers = this.handlers.get('*');\n\n  if (typeHandlers) {\n    for (const handler of typeHandlers) this.invokeHandler(event, handler);\n  }\n  if (wildcardHandlers) {\n    for (const handler of wildcardHandlers) {\n      if (!typeHandlers?.has(handler)) this.invokeHandler(event, handler);\n    }\n  }\n}"
+      },
+      {
+        "language": "typescript",
+        "code": "// CURRENT — up to 3 array allocations\nif (filter.status) tasks = tasks.filter(t => t.status === filter.status);\nif (filter.type) tasks = tasks.filter(t => t.type === filter.type);\nif (filter.assignedAgent) tasks = tasks.filter(t => t.assignedAgent === filter.assignedAgent);"
+      },
+      {
+        "language": "typescript",
+        "code": "getTasks(filter?: Partial<Pick<ITask, 'status' | 'type' | 'assignedAgent'>>): ITask[] {\n  if (!filter) return Array.from(this.tasks.values());\n  const { status, type, assignedAgent } = filter;\n  const result: ITask[] = [];\n  for (const t of this.tasks.values()) {\n    if (status && t.status !== status) continue;\n    if (type && t.type !== type) continue;\n    if (assignedAgent && t.assignedAgent !== assignedAgent) continue;\n    result.push(t);\n  }\n  return result;\n}"
+      }
+    ]
+  },
+  "durationMs": 67731,
+  "model": "sonnet",
+  "sandboxMode": "permissive",
+  "workerType": "optimize",
+  "timestamp": "2026-04-28T12:29:34.704Z",
+  "executionId": "optimize_1777379306973_an4lmy"
+}