refactor(frame-manager): extract WhatsApp sync into lifecycle hooks (#6)

* refactor(frame-manager): extract WhatsApp sync into lifecycle hooks

- Create new frame-lifecycle-hooks.ts module for extensible frame events
- Remove WhatsApp-specific code from FrameManager (was 20 lines of tight coupling)
- FrameManager now calls frameLifecycleHooks.triggerClose() instead
- WhatsApp sync registers itself as a hook via registerWhatsAppSyncHook()
- Benefits:
  - FrameManager no longer knows about WhatsApp
  - Other integrations can subscribe to frame events
  - Better testability and separation of concerns
  - Fire-and-forget hooks don't block frame operations

https://claude.ai/code/session_014ojs76858uGzWb6Hrbs4VG

* refactor(errors): consolidate error codes and handler into single module

- Merge duplicate ErrorCode enums into src/core/errors/index.ts
- Add new error codes: AUTH_*, FS_*, GIT_*, NET_* categories
- Add getUserFriendlyMessage() for user-facing error messages
- Move ErrorHandler class with retry/circuit breaker to errors module
- Move validation utilities (validateInput, validateEmail, validatePath)
- error-handler.ts now re-exports from errors for backwards compatibility

Benefits:
- Single source of truth for all error codes
- Consistent error handling across CLI and core
- No more conflicting StackMemoryError class definitions
- Better organization with categorized error codes

https://claude.ai/code/session_014ojs76858uGzWb6Hrbs4VG

* chore: update package-lock.json

https://claude.ai/code/session_014ojs76858uGzWb6Hrbs4VG

* refactor(frame-manager): add lifecycle hooks and missing methods to RefactoredFrameManager

- Add frameLifecycleHooks integration to RefactoredFrameManager.closeFrame()
- Add missing methods for feature parity with legacy FrameManager:
  - setQueryMode() - change query mode for frame retrieval
  - getRecentFrames() - get recent frames with metadata
  - addContext() - add context metadata to current frame
  - deleteFrame() - delete frame completely (for handoffs)
  - extractTagsFromFrame() - extract tags for categorization
  - calculateFrameImportance() - calculate frame importance
- Add setQueryMode() and removeFrame() to FrameStack

This brings RefactoredFrameManager to feature parity with legacy FrameManager,
enabling eventual removal of the legacy implementation.

https://claude.ai/code/session_014ojs76858uGzWb6Hrbs4VG

* refactor(context): migrate all imports to use index.js and fix type exports

- Update all direct imports from frame-manager.ts to use index.js
- Update type-only imports to use frame-types.ts
- Fix type exports to use `export type` for pure type definitions
  (required because esbuild strips type-only modules at runtime)
- Update tests to use new import paths:
  - FrameManager now refers to RefactoredFrameManager
  - LegacyFrameManager available for backwards compatibility tests

All code now imports FrameManager from index.js, which provides
the RefactoredFrameManager as the primary implementation.

https://claude.ai/code/session_014ojs76858uGzWb6Hrbs4VG

* refactor(context): remove legacy FrameManager and consolidate imports

- Delete frame-manager.ts (1,391 lines) - legacy implementation
- Update all imports to use index.js centralized exports
- Fix updateParentFrame to properly update parent_frame_id and depth
- Add validation for non-existent parent frames
- Support parent_frame_id and depth updates in FrameDatabase
- Rewrite cycle detection tests to use unified FrameManager

https://claude.ai/code/session_014ojs76858uGzWb6Hrbs4VG

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: jonathanpeterwu

package-lock.json

@@ -4,7 +4,7 @@
  */
 
 import { bench, describe } from 'vitest';
-import { FrameManager } from '../../core/context/frame-manager.js';
+import { FrameManager } from '../../core/context/index.js';
 import Database from 'better-sqlite3';
 import { join } from 'path';
 import { mkdtempSync, rmSync } from 'fs';

src/__tests__/generated/frame-manager.bench.ts

@@ -4,7 +4,7 @@
  */
 
 import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { FrameManager } from '../../core/context/frame-manager.js';
+import { FrameManager } from '../../core/context/index.js';
 import Database from 'better-sqlite3';
 import { join } from 'path';
 import { mkdtempSync, rmSync } from 'fs';

src/__tests__/generated/frame-manager.generated.test.ts

@@ -3,7 +3,7 @@
  * Creates realistic test data for integration testing
  */
 
-import type { Frame, Event } from '../../../core/context/frame-manager.js';
+import type { Frame, Event } from '../../../core/context/index.js';
 
 export interface TestFrameOptions {
   count?: number;

src/__tests__/integration/fixtures/test-data-generator.ts

@@ -4,7 +4,7 @@
  */
 
 import { SQLiteAdapter, SQLiteConfig } from '../../../core/database/sqlite-adapter.js';
-import { FrameManager } from '../../../core/context/frame-manager.js';
+import { FrameManager } from '../../../core/context/index.js';
 import { SharedContextLayer } from '../../../core/context/shared-context-layer.js';
 import { ContextBridge } from '../../../core/context/context-bridge.js';
 import { ContextRetriever } from '../../../core/retrieval/context-retriever.js';

src/__tests__/integration/helpers/test-environment.ts

@@ -15,7 +15,7 @@ import {
   TaskPriority,
 } from '../../features/tasks/linear-task-manager.js';
 import { logger } from '../../core/monitoring/logger.js';
-import { FrameManager } from '../../core/context/frame-manager.js';
+import { FrameManager } from '../../core/context/index.js';
 import { TaskError, ErrorCode } from '../../core/errors/index.js';
 
 export interface AgentTaskSession {

src/agents/core/agent-task-manager.ts

@@ -12,7 +12,7 @@ import * as path from 'path';
 import Database from 'better-sqlite3';
 import { existsSync } from 'fs';
 import { ClearSurvival } from '../../core/session/clear-survival.js';
-import { FrameManager } from '../../core/context/frame-manager.js';
+import { FrameManager } from '../../core/context/index.js';
 import { HandoffGenerator } from '../../core/session/handoff-generator.js';
 import { sessionManager } from '../../core/session/session-manager.js';
 import { getEnv, getOptionalEnv } from '../../utils/env.js';

src/cli/commands/clear.ts

@@ -7,7 +7,7 @@ import { Command } from 'commander';
 import Database from 'better-sqlite3';
 import { join } from 'path';
 import { existsSync } from 'fs';
-import { FrameManager, FrameType } from '../../core/context/frame-manager.js';
+import { FrameManager, FrameType } from '../../core/context/index.js';
 import { createContextRehydrateCommand } from './context-rehydrate.js';
 
 // Type-safe environment variable access

src/cli/commands/context.ts

@@ -6,7 +6,7 @@
 import chalk from 'chalk';
 import Table from 'cli-table3';
 import { SessionManager } from '../../core/session/session-manager.js';
-import { FrameManager } from '../../core/context/frame-manager.js';
+import { FrameManager } from '../../core/context/index.js';
 import Database from 'better-sqlite3';
 import { join } from 'path';
 import { existsSync } from 'fs';

src/cli/commands/dashboard.ts

@@ -8,7 +8,7 @@ import Database from 'better-sqlite3';
 import { join } from 'path';
 import { existsSync } from 'fs';
 import chalk from 'chalk';
-import { FrameManager } from '../../core/context/frame-manager.js';
+import { FrameManager } from '../../core/context/index.js';
 import { LLMContextRetrieval } from '../../core/retrieval/index.js';
 import { DiscoveryHandlers } from '../../integrations/mcp/handlers/discovery-handlers.js';