fix(viewer): hotfix missing message_utils in image#201
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe viewer publish workflow now runs when ChangesDocker publish inputs and dev gating
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Carries the viewer-image hotfix (#201) so the standalone viewer container starts again (it was missing src/message_utils.py). Also tightens viewer rebuild triggers (pyproject.toml/uv.lock, src/__init__.py) and drops the unused requirements-viewer.txt path filter.
|
Merged and shipped in v7.17.1 — thank you, @charys117! 🙏 Great catch: the standalone viewer image was missing |
Summary
Hotfix for the current viewer image startup failure.
The current viewer container can fail to start because
Dockerfile.viewerdoes not copysrc/message_utils.pyinto the image, while the viewer startup path now imports it throughsrc.db.adapter.This PR adds the missing file to the viewer image and updates the viewer image publish path filters so future changes to this dependency rebuild the viewer image.
Changes
src/message_utils.pyinto the standalone viewer image.src/message_utils.pyto the viewer Docker publish workflow path filters.src/realtime.pyto the same path filters because it is also copied byDockerfile.viewer.Validation
Tested with a fork GHCR build workflow:
python -c "import src.web.main"Workflow run:
Summary by CodeRabbit