Skip to content

feat: add shared MCP connection manager#6534

Merged
TimothyZhang7 merged 3 commits intoaden-hive:mainfrom
VasuBansal7576:codex/mcp-connection-manager-6348-draft
Mar 18, 2026
Merged

feat: add shared MCP connection manager#6534
TimothyZhang7 merged 3 commits intoaden-hive:mainfrom
VasuBansal7576:codex/mcp-connection-manager-6348-draft

Conversation

@VasuBansal7576
Copy link
Copy Markdown
Contributor

Summary

  • add a process-wide MCPConnectionManager with refcounted acquire/release, reconnect, health checks, and cleanup
  • add opt-in ToolRegistry integration that uses the shared pool while preserving the default direct MCPClient path
  • add unit tests for reuse, refcounting, concurrent access, cleanup, reconnect, health checks, and registry integration

Testing

  • uv run pytest core/tests/test_mcp_connection_manager.py core/tests/test_tool_registry.py
  • uv run pytest core/tests

Closes #6348

@TimothyZhang7 TimothyZhang7 added bounty:large Bounty: new feature, perf work, architecture docs (75 pts) bounty:medium Bounty: bug fix, tests, guides, CLI improvements (30 pts) and removed bounty:large Bounty: new feature, perf work, architecture docs (75 pts) labels Mar 18, 2026
@TimothyZhang7 TimothyZhang7 merged commit 6fdaa9e into aden-hive:main Mar 18, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bounty:medium Bounty: bug fix, tests, guides, CLI improvements (30 pts)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MCP Registry] MCPConnectionManager — shared connection pool with refcounting

2 participants