Skip to content

Comments

fix(ci,backend,docker): ensure migration files present and included in CI and build outputs#24

Open
cto-new[bot] wants to merge 1 commit intodevelopfrom
fix/restore-instance-migration-1690280117923-add-checks
Open

fix(ci,backend,docker): ensure migration files present and included in CI and build outputs#24
cto-new[bot] wants to merge 1 commit intodevelopfrom
fix/restore-instance-migration-1690280117923-add-checks

Conversation

@cto-new
Copy link

@cto-new cto-new bot commented Oct 27, 2025

Summary

This PR ensures that database migration files are always present and included at build and in CI, preventing deployment and CI failures due to missing artifacts.

Details

  • Adds a check-migration-files script that verifies all expected migration files exist in the backend
  • Integrates the migration file check into all backend CI jobs (unit, e2e, migration)
  • Updates the Dockerfile to explicitly copy the migration/ directory and backend ormconfig.js into Docker images
  • Addresses the root cause of missing migration file errors in CI/Docker by enforcing early and automated checks
  • No breaking changes; improves reliability and reduces production drift risk

Warning: Task VM test is not passing, cto.new will perform much better if you fix the setup

…ed in builds

CI sometimes failed due to missing migration files in the backend or Docker context.
This change ensures that required migration artifacts are always present and
correctly included in both CI checks and release/Docker artifacts.

- Adds an explicit check-migration-files script to verify migration files exist
- Adds this check to all backend CI jobs for faster, clearer feedback if missing
- Dockerfile now explicitly copies migration directory and ormconfig.js to ensure
  migrations are present in Docker images/releases
- Minimizes future risk of DB drift by surfacing migration file absence early

No breaking changes; expected to reduce CI false-negatives and production risk.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants