fix Prisma migrate

keystonejs · Aug 9, 2024 · 5bd6b44 · 5bd6b44
1 parent 3f4d613
commit 5bd6b44
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/.changeset/fix-prisma-migrate.md b/.changeset/fix-prisma-migrate.md
@@ -0,0 +1,5 @@
+---
+'@keystone-6/core': minor
+---
+
+Fix Prisma errors not showing when a Prisma migration errors internally
diff --git a/packages/core/src/lib/migrations.ts b/packages/core/src/lib/migrations.ts
@@ -1,4 +1,3 @@
-import { type ChildProcess } from 'node:child_process'
 import { toSchemasContainer } from '@prisma/internals'
 
 // @ts-expect-error
@@ -62,9 +61,9 @@ export async function withMigrate<T> (
       }
     })
   } finally {
-    const closePromise = new Promise<void>(resolve => {
-      const { child } = migrate.engine as { child: ChildProcess }
-      child.once('exit', () => resolve())
+    const closePromise = new Promise<void>(async (resolve) => {
+      await migrate.engine.initPromise
+      migrate.engine.child.once('exit', () => resolve())
     })
     migrate.stop()
     await closePromise