From 8b7451330f547d469529c0653c90b7474fb0207f Mon Sep 17 00:00:00 2001
From: Trent Larson <trent@trentlarson.com>
Date: Mon, 26 May 2025 15:50:37 -0600
Subject: [PATCH] remove possibility of failing a migration script and then
 succeeding on later ones

---
 src/services/migrationService.ts | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/src/services/migrationService.ts b/src/services/migrationService.ts
index b82b8d4e..d571f1ba 100644
--- a/src/services/migrationService.ts
+++ b/src/services/migrationService.ts
@@ -54,16 +54,11 @@ export class MigrationService {
     // Run pending migrations in order
     for (const migration of this.migrations) {
       if (!executedMigrations.has(migration.name)) {
-        try {
-          await sqlExec(migration.sql);
-          await sqlExec("INSERT INTO migrations (name) VALUES (?)", [
-            migration.name,
-          ]);
-          logger.log(`Migration ${migration.name} executed successfully`);
-        } catch (error) {
-          logger.error(`Error executing migration ${migration.name}:`, error);
-          throw error;
-        }
+        await sqlExec(migration.sql);
+        await sqlExec("INSERT INTO migrations (name) VALUES (?)", [
+          migration.name,
+        ]);
+        logger.log(`Migration ${migration.name} executed successfully`);
       }
     }
   }