@proteinjs/db 1.34.0 → 1.34.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@proteinjs/db",
-  "version": "1.34.0",
+  "version": "1.34.1",
   "main": "./dist/generated/index.js",
   "types": "./dist/generated/index.d.ts",
   "exports": {
@@ -66,5 +66,5 @@
     "ts-jest": "29.1.1",
     "typescript": "5.2.2"
   },
-  "gitHead": "7a301275151aee8040fd52e15e3fe950a6863783"
+  "gitHead": "a5a56218a5757e47576cbe79858c9aacc6026816"
 }

package/src/MigrationRunner.ts

@@ -206,7 +206,7 @@ export class MigrationRunner implements MigrationRunnerService {
     const db = getRunDb();
     migration.status = 'running';
     migration.startTime = moment();
-    await db.update(migrationTable, migration);
+    await db.update(migrationTable, this.definedFields(migration));
     this.logger.info({ message: `Running migration (${migration.id}) ${migration.description}` });
     try {
       migration.output = await migration.run();
@@ -222,12 +222,31 @@ export class MigrationRunner implements MigrationRunnerService {
       migration.endTime = moment();
     }
     migration.duration = this.duration(migration.startTime, migration.endTime);
-    await db.update(migrationTable, migration);
+    await db.update(migrationTable, this.definedFields(migration));
     this.logger.info({
       message: `[${migration.status}] (${migration.duration}) Finished running migration (${migration.id}) ${migration.description}`,
     });
   }
 
+  /**
+   * The run-state payload with `undefined`-valued fields OMITTED. Several of the record's fields
+   * are legitimately absent depending on the run (`output` for a void `run()`, `failureMessage`/
+   * `failureStack` for a non-Error throw), but the migration object carries them as explicit
+   * `undefined` assignments — and `RecordSerializer` rejects any payload field holding `undefined`
+   * (never a partial write), which would strand the row at 'running' status with the run's real
+   * outcome lost. Absent means omitted, never undefined — for EVERY optional field of the payload,
+   * not per-field.
+   */
+  private definedFields(migration: Migration): Partial<Migration> {
+    const payload: Partial<Migration> = {};
+    for (const [field, value] of Object.entries(migration)) {
+      if (value !== undefined) {
+        (payload as any)[field] = value;
+      }
+    }
+    return payload;
+  }
+
   private resolveMigration(migrationTable: Table<Migration>, id: string): Migration {
     const migration = new SourceRecordRepo().getSourceRecord<Migration>(migrationTable.name, id);
     if (!migration) {