Skip to content

fix(mcp): derive gateway WebSocket URL from template port - #351

Open
Aphroq wants to merge 1 commit into
trustgraph-ai:masterfrom
Aphroq:fix/mcp-gateway-websocket-url
Open

fix(mcp): derive gateway WebSocket URL from template port#351
Aphroq wants to merge 1 commit into
trustgraph-ai:masterfrom
Aphroq:fix/mcp-gateway-websocket-url

Conversation

@Aphroq

@Aphroq Aphroq commented Aug 24, 2026

Copy link
Copy Markdown

Summary

Pass the generated API Gateway address explicitly to the optional MCP server in template 2.8.

TrustGraph 2.8 changed the MCP server built-in gateway default to ws://api-gateway:8888/api/v1/socket, while the 2.8 deployment template still starts and exposes the API Gateway on configurable port 8088. Generated deployments therefore start successfully but the MCP server connects to the wrong internal port when its first tool is invoked.

Template 2.7 is not changed because its MCP server default is already 8088.

Changes

  • derive the MCP gateway WebSocket URL from the existing api-gateway-port parameter
  • pass it through the existing --websocket-url option
  • add a 2.8 deployment contract test covering the default port and a custom override

No new configuration field or hard-coded duplicate port is introduced.

Validation

  • focused MCP deployment contract: 2 passed
  • unit and feature suite: 168 passed, 4 skipped, 2 expected failures

The test builds a full 2.8 Compose resource graph and asserts that the API Gateway port and MCP Server WebSocket URL resolve from the same configured value.

@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown

Contributor License Agreement ✅

All contributors have signed the CLA. Thank you!

@Aphroq
Aphroq force-pushed the fix/mcp-gateway-websocket-url branch from d44282b to 0e081bf Compare August 25, 2026 00:14
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.

1 participant