fix: wire runMigrations() into startup before initDatabase() #41

Merged
jason merged 1 commits from fix/wire-migrations-to-startup into master 2026-03-10 14:28:20 -05:00
Showing only changes of commit e800cb91f2 - Show all commits

View File

@@ -3,6 +3,7 @@ const cors = require('cors');
const helmet = require('helmet');
const path = require('path');
const fs = require('fs');
const { runMigrations } = require('./db/migrations');
const { initDatabase } = require('./db/init');
const { logStartupBanner } = require('./utils/startupLog');
@@ -18,6 +19,16 @@ const DATA_DIR = process.env.DATA_DIR || path.join(__dirname, '../data');
if (!fs.existsSync(dir)) fs.mkdirSync(dir, { recursive: true });
});
// Run migrations BEFORE initializing the DB connection used by routes
const DB_PATH = process.env.DB_PATH || path.join(__dirname, '../data/breedr.db');
console.log('Running database migrations...');
try {
runMigrations(DB_PATH);
} catch (err) {
console.error('Migration failed — aborting startup:', err.message);
process.exit(1);
}
// Init DB (path is managed internally by db/init.js)
console.log('Initializing database...');
initDatabase();
@@ -49,7 +60,7 @@ app.use('/api/settings', require('./routes/settings'));
if (process.env.NODE_ENV === 'production') {
const clientBuild = path.join(__dirname, '../client/dist');
app.use(express.static(clientBuild));
app.get(/^(?!\/(?: api|static|uploads)\/).*$/, (_req, res) => {
app.get(/^(?!\/(?:api|static|uploads)\/).*$/, (_req, res) => {
res.sendFile(path.join(clientBuild, 'index.html'));
});
}
@@ -64,7 +75,6 @@ app.use((err, _req, res, _next) => {
});
app.listen(PORT, '0.0.0.0', () => {
// Display comprehensive startup log
logStartupBanner({
appName: 'BREEDR',
port: PORT,
@@ -76,4 +86,4 @@ app.listen(PORT, '0.0.0.0', () => {
});
});
module.exports = app;
module.exports = app;