Skip to content

feat: improve mcp tools based on endpoint#318

Merged
nicoloboschi merged 4 commits intomainfrom
fix-mcp-thing
Feb 8, 2026
Merged

feat: improve mcp tools based on endpoint#318
nicoloboschi merged 4 commits intomainfrom
fix-mcp-thing

Conversation

@nicoloboschi
Copy link
Collaborator

@nicoloboschi nicoloboschi commented Feb 6, 2026

fix #317

nicoloboschi added a commit that referenced this pull request Feb 6, 2026
- Add test_mcp_endpoint_routing.py to verify single-bank vs multi-bank tool exposure
- Verifies /mcp/ exposes all tools with bank_id parameters
- Verifies /mcp/{bank_id}/ only exposes scoped tools without bank_id parameters
- Regression test for issue #317

Related: #317, #318
- Add test_mcp_endpoint_routing.py to verify single-bank vs multi-bank tool exposure
- Verifies /mcp/ exposes all tools with bank_id parameters
- Verifies /mcp/{bank_id}/ only exposes scoped tools without bank_id parameters
- Regression test for issue #317

Related: #317, #318
Replace httpx AsyncClient SSE parsing with proper MCP StreamableHTTP
client. This correctly tests the MCP server using the actual protocol
that clients will use.

Fixes #317
@nicoloboschi nicoloboschi merged commit d90588b into main Feb 8, 2026
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MCP Server bank-specific URLs should expose bank-specific functionality

1 participant