Start the one-container image without a Bot it does not contain - #107
Merged
davidmckayv merged 7 commits intoAug 21, 2026
Conversation
jeonjw85
requested review from
MikeRyanDev,
davidmckayv,
guidovizoso and
tylerslaton
as code owners
August 21, 2026 20:33
MANAGED_AGENT_AG_UI_URL is no longer required. A URL with no token still refuses to start; a leftover token with no URL is ignored, so a one-container image can boot from a laptop .env.
The shipped Risk Analyst points at MANAGED_AGENT_AG_UI_URL. When that is unset the endpoint is empty, the coworker is left out of the roster, and channels drop the reference rather than failing to load.
…naged Bot Product-created coworkers used to fall back to the managed URL. Without one that fallback is gone, so create and duplicate say so instead of storing an agent that cannot answer.
scripts/start.sh still points at agent-langgraph for a local stack. The URL stays in the script rather than .env, so docker run --env-file .env does not inherit a host the image does not contain.
The image does not carry agent-langgraph. Requiring a URL against 127.0.0.1:4201 was the same lie the healthcheck used to tell.
The snapshot cache already lives in Postgres, so the one-replica warning is also gone.
jeonjw85
force-pushed
the
fix/managed-agent-optional
branch
from
August 21, 2026 20:37
9f49dac to
03c443d
Compare
# Conflicts: # CHANGELOG.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this changes
The one-container image does not carry
agent-botoragent-langgraph.MANAGED_AGENT_AG_UI_URLwas required and defaulted tolocalhost:4201, so the shipped Risk Analyst coworker was registered against a host that was not there and every conversation with it failed. The image looked deployed.The URL is no longer required. Unset, the server starts, the package omits that coworker, and creating one without its own endpoint is refused rather than pointed at nothing. A leftover
MANAGED_AGENT_TOKENfromscripts/start.shis ignored. A URL with no token still refuses to start.scripts/start.shstill points a laptop atagent-langgraph. The URL stays in the script rather than.env, sodocker run --env-file .envdoes not inherit it.A
.envcopied from an older.env.examplestill hasMANAGED_AGENT_AG_UI_URL=http://localhost:4201/ag-ui. Unset it before running the image.The deployment doc no longer says to run one replica because the snapshot cache is in process memory. That moved to Postgres in #90.
Where it runs
Boundary and audit
ManagedAgentUnavailableError, mapped to 400. No new audit class; nothing was acted on.Changelog
Unreleased→UpgradingandFixed. The one-replica warning under Deployment is also gone.Proof
bun test server/tests/config.test.ts server/tests/tenant-package.test.ts server/tests/agent-profile-store.integration.test.ts server/tests/runtime-agents.integration.test.ts: 86 pass, 0 fail.bun run --filter server typecheckandbun run --filter app typecheck: clean.bun x biome format/lintclean on the changed TypeScript. Pre-existingnoTemplateCurlyInStringwarnings intenant-package.test.tsare unchanged.Test plan
docker runwith noMANAGED_AGENT_AG_UI_URL. Expect the image to answer, and Risk Analyst not on the roster.MANAGED_AGENT_TOKENand no URL. Expect the same.bash scripts/start.shon a laptop. Expect Risk Analyst present and talking toagent-langgraph.