fix(API): log staging commit errors via winston instead of stderr

TheLastCicada · TheLastCicada · commit 037424faf410 · 2026-04-20T15:21:19.000-07:00
The commit endpoint's catch block used console.trace, which writes to
stderr and never reaches the structured winston log files. This made
400 responses from POST /v1/staging/commit opaque in logs, forcing
operators to correlate with pm2/journald output to find the root
cause. Route the error through the existing logger so the reason
lands in combined.log and error.log alongside the request metadata.
diff --git a/src/controllers/staging.controller.js b/src/controllers/staging.controller.js
@@ -3,6 +3,8 @@ import _ from 'lodash';
 import { Sequelize } from 'sequelize';
 import { Staging } from '../models';
 
+import { logger } from '../config/logger.js';
+
 import {
   optionallyPaginatedResponse,
   paginationParams,
@@ -110,7 +112,9 @@ export const commit = async (req, res) => {
       success: true,
     });
   } catch (error) {
-    console.trace(error);
+    logger.error(`POST /v1/staging/commit failed: ${error.message}`, {
+      stack: error.stack,
+    });
     res.status(400).json({
       message: 'Error commiting staging table',
       error: error.message,
