Skip to content

fix(docker): preload messaging gateway deps#26394

Closed
vaddisrinivas wants to merge 1 commit into
NousResearch:mainfrom
vaddisrinivas:codex/fix-docker-messaging-deps
Closed

fix(docker): preload messaging gateway deps#26394
vaddisrinivas wants to merge 1 commit into
NousResearch:mainfrom
vaddisrinivas:codex/fix-docker-messaging-deps

Conversation

@vaddisrinivas

@vaddisrinivas vaddisrinivas commented May 15, 2026

Copy link
Copy Markdown
Contributor

What

  • Install the [messaging] extra in the published Docker image alongside [all].
  • Keep lazy_deps.py available for remaining optional platform packages and future pin bumps.
  • Add a Dockerfile contract test so gateway messaging deps stay preloaded.

Why

The official Docker image should start Telegram/Discord gateway adapters without relying on first-boot lazy installation. This addresses the missing python-telegram-bot failure reported in #24698.

Fixes #24698.

How to test

  • uv run --with pytest --with pytest-xdist python -m pytest tests/tools/test_dockerfile_pid1_reaping.py tests/test_project_metadata.py -q
  • uv run --with pytest --with pytest-xdist python -m pytest tests/tools/test_dockerfile_node_modules_perms.py -q
  • uv lock --check
  • uv run python scripts/check-windows-footguns.py --all
  • git diff --check

Platforms tested

  • macOS local checkout for tests above.
  • Full Docker image build not run locally.

@vaddisrinivas vaddisrinivas force-pushed the codex/fix-docker-messaging-deps branch from eb54a7a to 47f4a72 Compare May 15, 2026 14:28
@daimon-nous daimon-nous Bot added type/bug Something isn't working P2 Medium — degraded but workaround exists area/docker Docker image, Compose, packaging labels May 15, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Merged via #27558 with your authorship preserved via rebase-merge. Commit a2cc30544 is on main. Thanks for the fix!

@teknium1 teknium1 closed this May 17, 2026
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docker Docker image, Compose, packaging P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: python-telegram-bot not installed in latest Docker image (v0.13.0)

2 participants