feat: DH-22113: MCP sessions by bmingles · Pull Request #321 · deephaven/vscode-deephaven

DH-21947_mcp-data-tool-fixes/task-03: Update unit tests in tableUtils.spec.ts to verify rollup table support. Current tests only mock basic Table objects. Add test cases that: 1) Mock a rollup tabl…

bmingles

committed

DH-21947_mcp-data-tool-fixes/task-04: Verify MCP data tools work with rollup tables. Three MCP tools use getTableOrError: getTableData.ts (line 104), getTableStats.ts (line 81), getColumnStats.ts (…

bmingles

committed

Fixed type errors affecting unit tests (#DH-21947)

bmingles

committed

use fetchVariableDefinitionByPredicate (#DH-21947)

bmingles

committed

DH-21947_mcp-sessions/task-01: Refactor SdkMcpServer creation from constructor to factory method. Currently, McpServer constructor creates ONE SdkMcpServer instance that gets reused (causing race c…

bmingles

committed

DH-21947_mcp-sessions/task-02: Add session storage infrastructure to McpServer class. Need to store both transports and server instances per session, plus import required utilities for session mana…

bmingles

committed

DH-21947_mcp-sessions/task-03: Implement session-based HTTP request handler in start() method. Replace the current stateless pattern (new transport per request) with stateful session management. Ex…

bmingles

committed

DH-21947_mcp-sessions/task-04: Implement session cleanup in transport.onclose handler. When a session disconnects, need to clean up both the transport AND the server instance from their respective …

bmingles

committed

DH-21947_mcp-sessions/task-05: Add graceful shutdown logic to stop() method. When the MCP server is stopping (extension deactivation, server restart, etc), need to close all active sessions cleanly…

bmingles

committed

DH-21947_mcp-sessions/task-06: Test session-based implementation with manual verification. Need to verify that: (1) parallel requests work correctly without race conditions, (2) sessions persist ac…

bmingles

committed

DH-21947_mcp-sessions/task-08: Update MCP documentation to reflect session-based architecture. Add notes about session support, implications for clients, and troubleshooting session-related issues.…

bmingles

committed

DH-21947_mcp-sessions/task-09: Clean up project: move migration plan and architecture docs back to docs/ folder for long-term reference. These documents provide valuable context for understanding t…

bmingles

committed

Cleaned up session support in MCP server (#DH-21947)

bmingles

committed

ran docs formatter and deleted planning doc (#DH-21947)

bmingles

committed

cleaned up diagrams (#DH-21947)

bmingles

committed

Cleaned up mcp.md (#DH-21947)

bmingles

committed

Fixed a race condition bug with DHC connections (#DH-21947)

bmingles

committed

Updated comment (#DH-21947)

bmingles

committed

Removed stray line that appears to have been a merge artifact (#DH-22113)

bmingles

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: DH-22113: MCP sessions#321

feat: DH-22113: MCP sessions#321
bmingles wants to merge 19 commits into
mainfrom
DH-22113_mcp-sessions

Commits on Jun 9, 2026

DH-21947_mcp-data-tool-fixes/task-03: Update unit tests in tableUtils.spec.ts to verify rollup table support. Current tests only mock basic Table objects. Add test cases that: 1) Mock a rollup tabl…

DH-21947_mcp-data-tool-fixes/task-04: Verify MCP data tools work with rollup tables. Three MCP tools use getTableOrError: getTableData.ts (line 104), getTableStats.ts (line 81), getColumnStats.ts (…

Fixed type errors affecting unit tests (#DH-21947)

use fetchVariableDefinitionByPredicate (#DH-21947)

DH-21947_mcp-sessions/task-01: Refactor SdkMcpServer creation from constructor to factory method. Currently, McpServer constructor creates ONE SdkMcpServer instance that gets reused (causing race c…

DH-21947_mcp-sessions/task-02: Add session storage infrastructure to McpServer class. Need to store both transports and server instances per session, plus import required utilities for session mana…

DH-21947_mcp-sessions/task-03: Implement session-based HTTP request handler in start() method. Replace the current stateless pattern (new transport per request) with stateful session management. Ex…

DH-21947_mcp-sessions/task-04: Implement session cleanup in transport.onclose handler. When a session disconnects, need to clean up both the transport AND the server instance from their respective …

DH-21947_mcp-sessions/task-05: Add graceful shutdown logic to stop() method. When the MCP server is stopping (extension deactivation, server restart, etc), need to close all active sessions cleanly…

DH-21947_mcp-sessions/task-06: Test session-based implementation with manual verification. Need to verify that: (1) parallel requests work correctly without race conditions, (2) sessions persist ac…

DH-21947_mcp-sessions/task-08: Update MCP documentation to reflect session-based architecture. Add notes about session support, implications for clients, and troubleshooting session-related issues.…

DH-21947_mcp-sessions/task-09: Clean up project: move migration plan and architecture docs back to docs/ folder for long-term reference. These documents provide valuable context for understanding t…

Cleaned up session support in MCP server (#DH-21947)

ran docs formatter and deleted planning doc (#DH-21947)

cleaned up diagrams (#DH-21947)

Cleaned up mcp.md (#DH-21947)

Fixed a race condition bug with DHC connections (#DH-21947)

Updated comment (#DH-21947)

Removed stray line that appears to have been a merge artifact (#DH-22113)