Skip to content

fix(docker): include package directory in image#114

Merged
chigwell merged 1 commit intochigwell:mainfrom
JTJag:fix/docker-image-startup
Apr 29, 2026
Merged

fix(docker): include package directory in image#114
chigwell merged 1 commit intochigwell:mainfrom
JTJag:fix/docker-image-startup

Conversation

@JTJag
Copy link
Copy Markdown
Contributor

@JTJag JTJag commented Apr 29, 2026

Summary

Fix Docker runtime startup failures after upstream project restructuring.

Why

After recent upstream changes, container startup failed with:

  • ModuleNotFoundError: No module named 'telegram_mcp'
    because the package directory was not copied into the image.

With package copy fixed, startup then failed on env parsing due to empty API ID default:

  • ValueError: invalid literal for int() with base 10: ''

Validation

  • docker build -t telegram-mcp:smoke . passes.
  • Runtime no longer fails on missing package import.
  • Runtime now correctly proceeds to Telethon credential validation (requires real TELEGRAM_API_ID/TELEGRAM_API_HASH at run time).

@chigwell chigwell merged commit 104ebda into chigwell:main Apr 29, 2026
4 checks passed
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.

2 participants