diff --git a/docker-compose.yml b/docker-compose.yml index f23db168..9fc33f49 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -14,20 +14,17 @@ services: env_file: - .env environment: - CLINICAL_ASSISTANT_MCP_URL: http://mcp:8000/mcp + # clinical-mcp, not mcp: this host runs several MCP servers and the bare + # name said nothing about which. Same container, added alias. + CLINICAL_ASSISTANT_MCP_URL: http://clinical-mcp:8000/mcp REDIS_URL: redis://ped-ai-redis:6379 LOKI_URL: http://monitoring-loki:3100 - # Deliberate: ped-ai talks to LiteLLM over the container network, not the - # public hostname. Clinical traffic then never leaves the host and does - # not depend on Caddy, public DNS or edge TLS being up. The latency - # difference is ~19ms on calls that take hundreds of ms, so speed is not - # the reason — not leaving the box is. - # - # OpenBao still stores https://llm.danvics.com for this key and Compose - # env wins, so the vault value is dead. Change it in the vault to match, - # or this line will keep looking like the accidental override it used to - # be. The app's AppRole is read-only, so that has to be done by hand. - LITELLM_API_BASE: http://litellm:4000 + # LITELLM_API_BASE is intentionally not set here: OpenBao supplies + # https://llm.danvics.com and Compose env would override it. The public + # hostname is the deliberate choice for consistency across the estate. + # The cost is not speed (~19ms on calls taking hundreds) — it is that + # AI calls now depend on Caddy, public DNS and edge TLS being up. + # To pin ped-ai to the container network instead, set it here. TTS_PROVIDER: litellm LITELLM_TTS_MODEL: local-kokoro-tts LITELLM_TTS_VOICE: sherpa/kokoro:am_adam