fix: migrations runned at container mount

Author: HunteRoi

src/services/PostgresDatabaseService.ts

@@ -206,7 +206,7 @@ export default class PostgresDatabaseService implements IDatabaseService {
     async #runMigrations(): Promise<void> {
         await this.#runMigration("User soft delete", async () => {
             await this.#database.query(
-                "ALTER TABLE Users ADD IF NOT EXISTS isDeleted timestamp;",
+                "ALTER TABLE Users ADD COLUMN IF NOT EXISTS isDeleted timestamp;",
             );
         });
     }
@@ -216,12 +216,9 @@ export default class PostgresDatabaseService implements IDatabaseService {
             "SELECT * FROM Migrations WHERE name = $1",
             [name],
         );
-        const migration = [...result].pop();
-        if (!migration) {
-            await this.#database.query(
-                "INSERT INTO Migrations (name) VALUES($1);",
-                [name],
-            );
+        this.#logger.verbose(`Running migration "${name}" with pre-query returning ${JSON.stringify(result)}`);
+        if (result && result.rows.length === 0) {
+            await this.#database.query('INSERT INTO Migrations (name) VALUES($1);', [name]);
             await callback();
         }
     }