Skip to content

bridge: restore uvicorn entrypoint accidentally removed in #113#114

Merged
BrettKinny merged 1 commit into
mainfrom
fix-bridge-uvicorn-entrypoint
May 23, 2026
Merged

bridge: restore uvicorn entrypoint accidentally removed in #113#114
BrettKinny merged 1 commit into
mainfrom
fix-bridge-uvicorn-entrypoint

Conversation

@BrettKinny
Copy link
Copy Markdown
Owner

Hotfix. #113 ripped the if __name__ == "__main__": uvicorn.run(...) block at the bottom of bridge.py along with the legacy voice path. The container CMD is python bridge.py, so without that block the script runs FastAPI module-level wiring then exits silently. Symptom: container restart-loops with "Prometheus /metrics mounted" as the last log every ~2s.

Caught on the very first redeploy after #113 merged. Refs #111.

The #111 rip ripped the `if __name__ == "__main__": uvicorn.run(...)`
block at the bottom of bridge.py along with the legacy voice/ACP code.
Without it, the container CMD `python bridge.py` runs the module-level
FastAPI wiring then exits silently — the container restart-looped
with "Prometheus /metrics mounted" as its last log line every couple
of seconds.

Add it back. Reads PORT (default 8081, matches the Dockerfile env
default) + DOTTY_BRIDGE_HOST (default 0.0.0.0). Caught immediately on
the first redeploy after #113 merged.

Refs #111

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 23, 2026 13:58
@BrettKinny BrettKinny merged commit f87018a into main May 23, 2026
6 of 8 checks passed
@BrettKinny BrettKinny deleted the fix-bridge-uvicorn-entrypoint branch May 23, 2026 13:59
@BrettKinny BrettKinny review requested due to automatic review settings May 23, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant