feat(plugin-tools): improve error serialization and let each storage plugin serialize errors themselves

packages/mysql/src/index.ts

@@ -20,7 +20,7 @@ import {
   type GenerateMigrationFunction,
   type GeneratorPlugin,
 } from '@emigrate/plugin-tools/types';
-import { getTimestampPrefix, sanitizeMigrationName } from '@emigrate/plugin-tools';
+import { getTimestampPrefix, sanitizeMigrationName, serializeError } from '@emigrate/plugin-tools';
 
 const defaultTable = 'migrations';
 
@@ -213,7 +213,7 @@ export const createMysqlStorage = ({ table = defaultTable, connection }: MysqlSt
               status: row.status,
               date: new Date(row.date),
               // FIXME: Migrate the migrations table to support the error column
-              error: row.status === 'failed' ? new Error('Unknown error reason') : undefined,
+              error: row.status === 'failed' ? serializeError(new Error('Unknown error reason')) : undefined,
             };
           }
         },

packages/plugin-tools/src/index.ts

@@ -11,12 +11,16 @@ import {
 } from './types.js';
 
 export const serializeError = (error: Error): SerializedError => {
-  return {
+  const properties: Record<string, unknown> = {
     name: error.name,
-    message: error.message,
-    stack: error.stack,
-    cause: error.cause instanceof Error ? serializeError(error.cause) : error.cause,
   };
+
+  for (const key of Object.getOwnPropertyNames(error)) {
+    const value = error[key as keyof Error];
+    properties[key] = value instanceof Error ? serializeError(value) : value;
+  }
+
+  return properties as SerializedError;
 };
 
 export const isGeneratorPlugin = (plugin: any): plugin is GeneratorPlugin => {

packages/plugin-tools/src/types.ts

@@ -5,7 +5,8 @@ export type StringOrModule<T> = string | T | (() => Awaitable<T>) | (() => Await
 export type MigrationStatus = 'failed' | 'done' | 'pending';
 
 export type SerializedError = {
-  name: string;
+  [key: string]: unknown;
+  name?: string;
   message: string;
   stack?: string;
   cause?: unknown;
@@ -73,7 +74,7 @@ export type Storage = {
    * @param migration The name of the migration that should be marked as failed.
    * @param error The error that caused the migration to fail.
    */
-  onError(migration: MigrationMetadataFinished, error: SerializedError): Promise<void>;
+  onError(migration: MigrationMetadataFinished, error: Error): Promise<void>;
   /**
    * Called when the command is finished or aborted (e.g. by a SIGTERM or SIGINT signal).
    *

packages/storage-fs/src/index.ts

@@ -1,6 +1,7 @@
 import fs from 'node:fs/promises';
 import path from 'node:path';
 import process from 'node:process';
+import { serializeError } from '@emigrate/plugin-tools';
 import { type SerializedError, type EmigrateStorage, type MigrationStatus } from '@emigrate/plugin-tools/types';
 
 export type StorageFsOptions = {
@@ -26,7 +27,7 @@ export default function storageFs({ filename }: StorageFsOptions): EmigrateStora
 
   let lastUpdate: Promise<void> = Promise.resolve();
 
-  const update = async (migration: string, status: MigrationStatus, error?: SerializedError) => {
+  const update = async (migration: string, status: MigrationStatus, error?: Error) => {
     lastUpdate = lastUpdate.then(async () => {
       const history = await read();
 
@@ -35,7 +36,7 @@ export default function storageFs({ filename }: StorageFsOptions): EmigrateStora
         [migration]: {
           status,
           date: new Date().toISOString(),
-          error,
+          error: error ? serializeError(error) : undefined,
         },
       };