Skip to content

ci(migrations): run deploy-migrations on Node 22 + allow manual dispatch#319

Merged
QSchlegel merged 1 commit into
preprodfrom
claude/fix-deploy-migrations-node22
Jun 17, 2026
Merged

ci(migrations): run deploy-migrations on Node 22 + allow manual dispatch#319
QSchlegel merged 1 commit into
preprodfrom
claude/fix-deploy-migrations-node22

Conversation

@QSchlegel

Copy link
Copy Markdown
Collaborator

Problem

The Deploy Database Migrations workflow runs npm ci on Node 18, but the repo now depends on prisma@7.8.0, which refuses to install on Node <20.19 via a preinstall engine gate:

Prisma only supports Node.js versions 20.19+, 22.12+, 24.0+.
current: node v18.20.8
npm error command sh -c node scripts/preinstall-entry.js

So npm ci exits 1 before prisma migrate deploy ever runs. The job has been failing on every push to main that touches prisma/migrations/**.

Production impact: the most recent main run (after #303 merged) failed this way, so the 20260614120000_add_proposal_tally migration was not applied — the ProposalTally table is missing in prod and the governance vote-tally router (#302) errors until it exists.

Fix

  • Bump setup-node to 22 (matches pr-checks, unit-tests, trpc-integration-tests, ci-smoke-preprod).
  • Add a workflow_dispatch trigger. The workflow otherwise only fires on a push under prisma/migrations/**; since the migration is already merged (no new file to push), workflow_dispatch is the only way to apply it once this fix reaches main.

After this lands on main

  1. Re-run / manually dispatch Deploy Database Migrations to apply the pending ProposalTally migration to prod.
  2. daily-balance-snapshots.yml also pins Node 18 but installs from the self-contained scripts/ dir (no root Prisma) — left untouched here.

🤖 Generated with Claude Code

The deploy-migrations workflow pinned Node 18, but the repo now depends on
prisma@7.8.0, which refuses to install on Node <20.19 (preinstall engine
gate). npm ci failed before `prisma migrate deploy` ever ran, so the
20260614120000_add_proposal_tally migration never reached production.

- Bump setup-node to 22, matching pr-checks/unit-tests/trpc/ci-smoke.
- Add workflow_dispatch so a pending migration can be applied without
  pushing a fresh file under prisma/migrations/** (this workflow only
  triggers on that path otherwise).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
multisig Building Building Preview, Comment Jun 17, 2026 12:34pm

Request Review

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