Skip to content

Add CI workflow to verify database migrations sequentially - #919

Open
Daluworks wants to merge 1 commit into
Core-Foundry:mainfrom
Daluworks:main
Open

Daluworks wants to merge 1 commit into
Core-Foundry:mainfrom
Daluworks:main

Conversation

@Daluworks

Copy link
Copy Markdown

Overview

Adds a new GitHub Actions CI workflow (.github/workflows/ci.yml) that verifies database migrations can be applied sequentially on a clean SQLite database. The workflow ensures that pending migrations cause the CI to fail, then applies all migrations and runs the listener test suite. This prevents migration-related issues from reaching production.

Related Issue

Closes #865

Changes

  • .github/workflows/ci.yml – New CI workflow file with steps to checkout, set up Node.js, install dependencies, check migrations, apply migrations, and run type‑checking and tests.
  • Updated CI configuration references in documentation (if any).

Verification

# Verify migration check works on a fresh DB
cd listener && npm run check-migrations && npm run migrate
# Run full test suite
cd listener && npm run typecheck && npm test

The CI pipeline runs these commands automatically on pushes and pull requests.

How to Test

  1. Clone the repository and checkout the main branch.
  2. Ensure you have Node.js v22 installed.
  3. Run the commands above locally to confirm migrations apply without errors.
  4. Open a PR that modifies a migration file; the CI should fail if check-migrations detects pending migrations.

Checklist

  • Branch is up to date with main
  • Tests added/updated and all pass locally
  • cargo fmt --all run (if Rust changes)
  • npm run lint passes (if TypeScript changes)
  • Documentation updated if behavior changed

@drips-wave

drips-wave Bot commented Sep 30, 2026

Copy link
Copy Markdown

@Daluworks Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

Add CI Migration Compatibility Test

1 participant