Skip to content

fix: docker build backend context - #94

Merged
tspicer merged 1 commit into
mainfrom
codex/fix-docker-build-backend
Jul 5, 2026
Merged

fix: docker build backend context#94
tspicer merged 1 commit into
mainfrom
codex/fix-docker-build-backend

Conversation

@tspicer

@tspicer tspicer commented Jul 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Include build_package.py in the Docker build context and copy it before the project install layer.
  • Add CI workflow regression checks for the Dockerfile and .dockerignore expectations.
  • Refresh the lockfile after the FastMCP/code-mode dependency update so uv sync --frozen remains valid.

Root Cause

pyproject.toml uses build_package.py as an in-tree PEP 517 backend, but the Docker build context excluded that file and the Dockerfile did not copy it before uv sync built the project. The project install layer therefore failed with ModuleNotFoundError: No module named 'build_package'.

Validation

  • uv lock --check
  • uv run ruff check --fix tests/unit/test_ci_workflows.py
  • uv run pytest tests/unit/test_ci_workflows.py tests/unit/test_stage_wheel_resources.py
  • docker build -t amazon-ads-mcp:latest .

@tspicer tspicer changed the title [codex] fix Docker build backend context fix: Docker build backend context Jul 5, 2026
@tspicer tspicer changed the title fix: Docker build backend context fix: docker build backend context Jul 5, 2026
@tspicer
tspicer marked this pull request as ready for review July 5, 2026 13:06
@tspicer
tspicer merged commit c439b11 into main Jul 5, 2026
10 of 12 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.

1 participant