Skip to content

Latest commit

 

History

History
276 lines (205 loc) · 10.5 KB

File metadata and controls

276 lines (205 loc) · 10.5 KB

mcp-debugger Architecture Overview

⚠️ DRAFT DOCUMENTATION This documentation is based on the current mcp-debugger architecture and will be refined based on real-world adapter development feedback.

From Python-Specific to Multi-Language Platform

The mcp-debugger has undergone a major architectural transformation, evolving from a Python-specific debugging tool into a multi-language debugging platform. This refactoring introduced a clean adapter pattern that separates language-agnostic core functionality from language-specific implementations.

Key Architectural Components

1. Language-Agnostic Core

The core components handle session management, process lifecycle, and DAP communication without any language-specific knowledge:

  • SessionManager - Thin public facade over the session manager hierarchy (session-manager-core.tssession-manager-data.tssession-manager-operations.tssession-manager.ts)
  • ProxyManager - Manages DAP proxy processes
  • SessionStore - In-memory session storage

2. Debug Adapter Interface

The IDebugAdapter interface defines the contract that all language adapters must implement:

export interface IDebugAdapter extends EventEmitter {
  readonly language: DebugLanguage;
  readonly name: string;
  
  // Lifecycle
  initialize(): Promise<void>;
  dispose(): Promise<void>;
  
  // Environment validation
  validateEnvironment(): Promise<ValidationResult>;
  resolveExecutablePath(preferredPath?: string): Promise<string>;
  
  // DAP operations
  buildAdapterCommand(config: AdapterConfig): AdapterCommand;
  sendDapRequest<T>(command: string, args?: unknown): Promise<T>;
  handleDapEvent(event: DebugProtocol.Event): void;
  
  // ... 30+ methods total
}

3. Language Adapters

Each supported language implements the IDebugAdapter interface:

4. Adapter Registry

The AdapterRegistry manages available adapters through two mechanisms: explicit registration and dynamic loading. Factories can be pre-registered at startup, or loaded on demand via AdapterLoader when enableDynamicLoading is enabled (or MCP_CONTAINER=true):

// Explicit registration
registry.register('python', new PythonAdapterFactory());
registry.register('mock', new MockAdapterFactory());

// Dynamic loading happens automatically when create() is called
// for a language without a pre-registered factory
const adapter = await registry.create('go', config); // loads @debugmcp/adapter-go on demand

Data Flow Architecture

sequenceDiagram
    participant Client as MCP Client
    participant Server as MCP Server
    participant SM as SessionManager
    participant AR as AdapterRegistry
    participant Adapter as Language Adapter
    participant PM as ProxyManager
    participant DAP as Debug Adapter Process
    
    Client->>Server: create_debug_session(language='python')
    Server->>Server: Validate language via getSupportedLanguagesAsync()
    Server->>SM: createSession(language)
    SM->>SM: SessionStore.createSession(params)
    SM-->>Client: sessionInfo (adapter instance not yet created)

    Client->>Server: start_debugging(sessionId)
    Server->>SM: startDebugging()
    SM->>AR: create(language, config)
    AR->>Adapter: factory.create(config)
    SM->>Adapter: validateEnvironment()
    SM->>PM: proxyManagerFactory.create(adapter)
    PM->>Adapter: buildAdapterCommand()
    PM->>DAP: spawn debug adapter
    
    DAP-->>PM: DAP events
    PM-->>Adapter: handleDapEvent()
    Adapter-->>SM: emit events
    SM-->>Client: debugging updates
Loading

Key Design Decisions

1. Wrap, Don't Rewrite

The existing ProxyManager provides excellent process management. Rather than rewriting it, we inject adapters to handle language-specific concerns:

// Before: ProxyManager had Python-specific logic
class ProxyManager {
  private async spawnPythonDebugger() { /* ... */ }
}

// After: ProxyManager delegates to adapters
class ProxyManager {
  constructor(
    private adapter: IDebugAdapter | null,
    private proxyProcessLauncher: IProxyProcessLauncher,
    private fileSystem: IFileSystem,
    private logger: ILogger,
    runtimeEnv?: ProxyRuntimeEnvironment
  ) {}

  async start(config: ProxyConfig) {
    // adapter.buildAdapterCommand() provides the spawn command
    // adapter.prepareSpawnContext() sets up environment/args
    // ProxyManager spawns the proxy worker process, which in turn
    // spawns the debug adapter using the adapter-provided command
  }
}

2. Invariant Core, Variable Adapters

The core remains stable while adapters handle all language-specific variations:

  • Core handles: Process lifecycle, IPC, session state, DAP transport
  • Adapters handle: Executable discovery, command building, environment validation, language-specific quirks

3. Event-Driven Communication

Adapters extend EventEmitter for loose coupling:

adapter.on('stopped', (event) => {
  // Update UI, notify client
});

adapter.on('stateChanged', (oldState, newState) => {
  // Track adapter state transitions
});

Reality Check: Theory vs Practice

Path Handling Complexity

Theory: Adapters handle path translation cleanly Reality: Path handling is centralized in utility modules rather than delegated to individual debug adapters. SimpleFileChecker and resolvePathForRuntime in src/utils/ handle path resolution, file existence checks, and container-mode prefix logic. Path handling also occurs in proxy script discovery and absolute path resolution for breakpoints. The approach is to use centralized path utilities to avoid duplicating cross-platform logic, while leaving runtime-specific path interpretation to the OS and debug adapter.

State Management

Theory: Clean state transitions following VALID_TRANSITIONS Reality: Some adapters use simpler or looser state models than earlier expectations. The mock adapter policy still maintains explicit initialized/configuration state transitions via createInitialState(), updateStateOnCommand(), and updateStateOnEvent().

DAP Event Sequences

Theory: Simple event flow
Reality: Each debugger has quirks. See DAP Sequence Reference for real-world examples:

  • Node.js waits for debugger detach
  • Python handles sys.exit() specially
  • Event ordering matters: exitedterminated

Performance Characteristics

The adapter pattern adds minimal overhead:

Operation Before After Impact
Session Creation 100ms 105ms +5%
Breakpoint Setting <10ms <10ms None
Step Operations <50ms <50ms None
Memory per Session Baseline +~1MB Adapter instance

Testing Infrastructure

The refactoring improved testability:

  • 1200+ passing tests (100% success rate)
  • Mock adapter enables integration testing without external dependencies
  • Type safety throughout with TypeScript strict mode
  • Comprehensive test coverage for all components

Common Pitfalls

1. Forgetting Event Registration

Adapters must properly handle DAP events:

// ❌ Wrong: Forgetting to update state
handleDapEvent(event: DebugProtocol.Event): void {
  this.emit(event.event, event.body);
}

// ✅ Correct: Update internal state
// Note: Paused execution is primarily modeled at the session/proxy
// orchestration layer. At the adapter level, DEBUGGING indicates
// an active debug session (which may be paused or running).
handleDapEvent(event: DebugProtocol.Event): void {
  if (event.event === 'stopped') {
    this.currentThreadId = event.body?.threadId;
    this.transitionTo(AdapterState.DEBUGGING);
  }
  this.emit(event.event, event.body);
}

2. Incorrect State Transitions

The stopped event means PAUSED, not terminated:

// ❌ Wrong: Confusing stopped with terminated
if (event.event === 'stopped') {
  this.state = AdapterState.TERMINATED; // NO!
}

// ✅ Correct: Stopped = paused for debugging
if (event.event === 'stopped') {
  this.state = AdapterState.DEBUGGING; // Paused at breakpoint
}

3. Missing Terminated Event

Adapters that need to synthesize session-end events (e.g., a mock adapter or one whose underlying runtime does not emit terminated) should send both exited and terminated. When the real debug adapter process sends these events over DAP, ProxyManager forwards them automatically — adapters in that case do not need to emit them manually.

// ❌ Wrong: Only sending exited (when synthesizing events manually)
async endSession() {
  this.emit('exited', { exitCode: 0 });
}

// ✅ Correct: Send both exited and terminated (when synthesizing events manually)
async endSession(exitCode: number) {
  this.emit('exited', { exitCode });
  this.emit('terminated');
}

Next Steps

Version History

  • v0.21.0 - Ruby adapter, 8 adapters total
  • v0.19.0 - .NET/C# adapter, 7 language adapters total
  • v0.18.0 - Go adapter, Java adapter
  • v0.17.0 - Rust adapter
  • v0.16.0 - JavaScript adapter
  • v0.10.0 - Multi-language architecture with adapter pattern
  • v0.9.x - Python-specific implementation
  • v0.8.x - Initial MCP server implementation

This documentation reflects the state of mcp-debugger after the major refactoring completed in January 2025. For the refactoring history, see refactoring-summary.md.