Commit 538edb3
authored
feat: add MCP server to Docker Compose files (#1208)
* feat: add MCP server to Docker Compose files
Add mcp-server service to dev and demo stacks. Dev stack builds from
source, exposes port 8091, and connects to meridian:50051 with a dev
API key. Demo stack pulls a versioned image, requires MCP_API_KEY, and
runs internal-only (proxied via Caddy).
* fix: use correct MERIDIAN_API_KEY env var in demo compose
The MCP server reads MERIDIAN_API_KEY (not MCP_API_KEY) as defined in
services/mcp-server/internal/auth/auth.go. Align demo compose with
dev compose and the application's expected env var name.
* fix: address CodeRabbit feedback on demo compose mcp-server config
- Promote MCP_BASE_URL to fail-fast (:?) to prevent silent SSE
misconfiguration; the server needs its own public URL for SSE
handshakes and OAuth redirect URIs
- Fix Caddyfile comment to reference actual endpoints (/sse, /message)
instead of incorrect /mcp/* wildcard
* docs: clarify OAuth env vars required when MCP_OAUTH_ENABLED=true
Add inline comments to MCP_OAUTH_CLIENT_ID and MCP_OAUTH_REDIRECT_URI
in the demo compose to prevent silent misconfiguration when OAuth is
enabled without the necessary companion variables.
---------
Co-authored-by: Ben Coombs <bjcoombs@users.noreply.github.com>1 parent c30900d commit 538edb3
2 files changed
Lines changed: 54 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | | - | |
6 | | - | |
7 | | - | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
8 | 9 | | |
9 | 10 | | |
10 | 11 | | |
| |||
108 | 109 | | |
109 | 110 | | |
110 | 111 | | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
111 | 140 | | |
112 | 141 | | |
113 | 142 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
16 | 17 | | |
17 | 18 | | |
18 | 19 | | |
| 20 | + | |
19 | 21 | | |
20 | 22 | | |
21 | 23 | | |
| |||
99 | 101 | | |
100 | 102 | | |
101 | 103 | | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
102 | 123 | | |
103 | 124 | | |
104 | 125 | | |
0 commit comments