mirror of
https://github.com/rennf93/roboco.git
synced 2026-08-03 07:23:24 +02:00
fix(agent): pin uv to baked /app/.venv so MCP/SDK servers start instantly (#179)
Every agent MCP server is launched as `uv run python -m roboco.mcp.<server>` (via the orchestrator-generated mcp-config.json) and the SDK server via `uv run python -m roboco.agent_sdk.server` (sdk-startup-hook.sh) — both with cwd = the agent's WORKSPACE, not /app. `uv run` then resolves a cwd-relative `.venv` (≠ the image's baked /app/.venv), ignores VIRTUAL_ENV with a warning, and RE-SYNCS the full dependency set (torch/lancedb/pyarrow/scipy, ~350MB) into a fresh venv on every spawn. A warm host uv wheel cache masks this (fast re-resolve from cached wheels — earlier runs this session opened PR #26/#28/#29 fine). On a COLD cache (first spawn after an image rebuild — exactly when deploying new fixes) the download takes minutes, the MCP servers never register, and the agent burns its whole budget with "No such tool available: mcp__roboco-*" before reaping. Observed this session: be-dev-1 never claimed; /tmp/sdk-server.log showed the live torch/lancedb download + the `VIRTUAL_ENV ... will be ignored` warning. Fix: set UV_PROJECT_ENVIRONMENT=/app/.venv in (1) every MCP server's env in the generated mcp-config.json (one place — shared mcp_env dict) and (2) the SDK startup hook. uv then reuses the pre-baked image venv instantly, regardless of cwd or cache state. Not a regression from this session's code (none of #172b/#175/#176/#177/#178 touched the launch/venv path — verified); a pre-existing launch-cwd fragility that rebuilding to deploy exposed. Test: _generate_mcp_config asserts every server env pins UV_PROJECT_ENVIRONMENT=/app/.venv. make quality green.
This commit is contained in:
@@ -12,6 +12,17 @@ LOG_FILE="/tmp/sdk-server.log"
|
||||
BRIEFING_FILE="/app/briefing.md"
|
||||
PRECOMPACT_FILE="/tmp/roboco-precompact-${AGENT_ID}.md"
|
||||
|
||||
# #179: this hook runs with cwd = the agent's workspace, so a bare
|
||||
# `uv run` resolves a cwd-relative `.venv` (≠ the baked /app/.venv),
|
||||
# ignores VIRTUAL_ENV with a warning, and RE-SYNCS the full dependency
|
||||
# set (torch/lancedb/pyarrow/scipy, ~350MB) into a fresh venv. On a cold
|
||||
# uv wheel cache (first spawn after an image rebuild) that download takes
|
||||
# minutes and the SDK/MCP layer never comes up before the agent reaps.
|
||||
# Pin uv to the pre-baked image venv so it starts instantly regardless
|
||||
# of cwd. (The orchestrator sets the same var in every MCP server's env
|
||||
# in the generated mcp-config.json — keep both in sync.)
|
||||
export UV_PROJECT_ENVIRONMENT=/app/.venv
|
||||
|
||||
# --- SDK bring-up ---------------------------------------------------------
|
||||
if ! curl -sf "http://localhost:${SDK_PORT}/health" >/dev/null 2>&1; then
|
||||
echo "[SDK] Starting for agent ${AGENT_ID} on port ${SDK_PORT}..."
|
||||
|
||||
Reference in New Issue
Block a user