From: xangelo <me@xangelo.ca>
Date: Thu, 12 Jun 2025 17:01:00 +0000 (-0400)
Subject: fix: migrations should be run out of process
X-Git-Url: https://git.xangelo.ca/?a=commitdiff_plain;h=refs%2Fheads%2Fmain;p=river.git

fix: migrations should be run out of process
---

diff --git a/src/server.ts b/src/server.ts
index 2e8c19a..cad6dd8 100644
--- a/src/server.ts
+++ b/src/server.ts
@@ -13,12 +13,7 @@ const db = knex({
   debug: process.env.DEBUG === 'true',
   connection: {
     filename: join('./data.db')
-  },
-  migrations: {
-    directory: join(__dirname, '../migrations'),
-    extension: 'ts'
-  },
-  useNullAsDefault: true
+  }
 });
 const parser = new Parser();
 const app = express();
@@ -124,24 +119,6 @@ async function queryFeeds() {
   setTimeout(queryFeeds, 1000 * 60);
 }
 
-// Migration runner function
-async function runMigrations() {
-  try {
-    console.log('Running database migrations...');
-    const [batchNo, log] = await db.migrate.latest();
-
-    if (log.length === 0) {
-      console.log('Database is already up to date');
-    } else {
-      console.log(`Batch ${batchNo} run: ${log.length} migrations`);
-      log.forEach((migration) => console.log(`- ${migration}`));
-    }
-  } catch (error) {
-    console.error('Migration failed:', error);
-    process.exit(1);
-  }
-}
-
 app.use(express.static(join(__dirname, 'public')));
 app.use(bodyParser.urlencoded({ extended: true }));
 app.use(express.json());
@@ -350,10 +327,6 @@ app.get('/admin/feeds', isAdmin, async (req, res) => {
 
 // Start the application
 async function startApp() {
-  // Run migrations first
-  await runMigrations();
-
-  // Then start the server
   app.listen(process.env.API_PORT, () => {
     console.log(`Listening on port ${process.env.API_PORT}`);
     console.log(`Feed refresh rate: ${FEED_REFRESH_RATE}m`);