fix: wire runMigrations() into startup before initDatabase() #41
@@ -3,6 +3,7 @@ const cors = require('cors');
|
|||||||
const helmet = require('helmet');
|
const helmet = require('helmet');
|
||||||
const path = require('path');
|
const path = require('path');
|
||||||
const fs = require('fs');
|
const fs = require('fs');
|
||||||
|
const { runMigrations } = require('./db/migrations');
|
||||||
const { initDatabase } = require('./db/init');
|
const { initDatabase } = require('./db/init');
|
||||||
const { logStartupBanner } = require('./utils/startupLog');
|
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 });
|
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)
|
// Init DB (path is managed internally by db/init.js)
|
||||||
console.log('Initializing database...');
|
console.log('Initializing database...');
|
||||||
initDatabase();
|
initDatabase();
|
||||||
@@ -49,7 +60,7 @@ app.use('/api/settings', require('./routes/settings'));
|
|||||||
if (process.env.NODE_ENV === 'production') {
|
if (process.env.NODE_ENV === 'production') {
|
||||||
const clientBuild = path.join(__dirname, '../client/dist');
|
const clientBuild = path.join(__dirname, '../client/dist');
|
||||||
app.use(express.static(clientBuild));
|
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'));
|
res.sendFile(path.join(clientBuild, 'index.html'));
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
@@ -64,7 +75,6 @@ app.use((err, _req, res, _next) => {
|
|||||||
});
|
});
|
||||||
|
|
||||||
app.listen(PORT, '0.0.0.0', () => {
|
app.listen(PORT, '0.0.0.0', () => {
|
||||||
// Display comprehensive startup log
|
|
||||||
logStartupBanner({
|
logStartupBanner({
|
||||||
appName: 'BREEDR',
|
appName: 'BREEDR',
|
||||||
port: PORT,
|
port: PORT,
|
||||||
@@ -76,4 +86,4 @@ app.listen(PORT, '0.0.0.0', () => {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
module.exports = app;
|
module.exports = app;
|
||||||
|
|||||||
Reference in New Issue
Block a user