fix: address PR review feedback for agents service disabling
- Fix misleading profile documentation at top of docker-compose.yml
- Add AGENTS_ENABLED flag for cleaner agent service handling
- Make AGENTS_SERVICE_URL configurable via environment variable
- Prevent noisy connection errors when agents service isn't running
This provides a cleaner way to disable the agents service and allows
the application to skip agent wiring when AGENTS_ENABLED=false.
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
parent
45df79de9a
commit
ab56dd4844
@ -1,12 +1,9 @@
|
||||
# Docker Compose Profiles Strategy:
|
||||
# - No profile: All services start by default when running 'docker compose up'
|
||||
# - "backend": Starts only backend services (server, mcp, agents) for hybrid development
|
||||
# - "frontend": Starts only the frontend UI service
|
||||
# - "full": Starts all services for complete Docker deployment (same as no profile)
|
||||
# Use --profile flag to control which services start:
|
||||
# docker compose up # All services (default behavior)
|
||||
# docker compose --profile backend up # Backend only for hybrid dev
|
||||
# docker compose --profile full up # Everything in Docker (same as default)
|
||||
# Docker Compose profiles:
|
||||
# - Default (no profile): Starts archon-server, archon-mcp, and archon-frontend
|
||||
# - Agents are opt-in: archon-agents starts only with the "agents" profile
|
||||
# Usage:
|
||||
# docker compose up # Starts server, mcp, frontend (agents disabled)
|
||||
# docker compose --profile agents up -d # Also starts archon-agents
|
||||
|
||||
services:
|
||||
# Server Service (FastAPI + Socket.IO + Crawling)
|
||||
@ -30,6 +27,7 @@ services:
|
||||
- ARCHON_SERVER_PORT=${ARCHON_SERVER_PORT:-8181}
|
||||
- ARCHON_MCP_PORT=${ARCHON_MCP_PORT:-8051}
|
||||
- ARCHON_AGENTS_PORT=${ARCHON_AGENTS_PORT:-8052}
|
||||
- AGENTS_ENABLED=${AGENTS_ENABLED:-false}
|
||||
networks:
|
||||
- app-network
|
||||
volumes:
|
||||
@ -83,7 +81,8 @@ services:
|
||||
- LOG_LEVEL=${LOG_LEVEL:-INFO}
|
||||
# MCP needs to know where to find other services
|
||||
- API_SERVICE_URL=http://archon-server:${ARCHON_SERVER_PORT:-8181}
|
||||
- AGENTS_SERVICE_URL=http://archon-agents:${ARCHON_AGENTS_PORT:-8052}
|
||||
- AGENTS_ENABLED=${AGENTS_ENABLED:-false}
|
||||
- AGENTS_SERVICE_URL=${AGENTS_SERVICE_URL:-http://archon-agents:${ARCHON_AGENTS_PORT:-8052}}
|
||||
- ARCHON_MCP_PORT=${ARCHON_MCP_PORT:-8051}
|
||||
- ARCHON_SERVER_PORT=${ARCHON_SERVER_PORT:-8181}
|
||||
- ARCHON_AGENTS_PORT=${ARCHON_AGENTS_PORT:-8052}
|
||||
|
||||
Loading…
Reference in New Issue
Block a user