Hermes
7bc2a207f3
After the DC-005 module reorganization (41 files moved into src/ subdirs),
138 test suites failed because the refactor script's path-rewrite logic
missed three categories:
1. Files inside src/ doing 'require("./src/...")' — should be 'require("../...")'
2. Files in src/X/Y/ doing 'require("../../../src/...")' — should be 'require("../../...")'
3. Test files in __tests__/ with leftover 'require("../../../src/...")' paths
Root cause: the original refactor script ran before all files were moved,
so it computed relative paths against stale filesystem state.
Result:
- 30/30 test suites pass
- 879/879 tests pass (was: 18/30 suites, 614/687 tests)
Also fixed:
- routes/apps/restore.js: wrong responses import path
- routes/*/*.js: '../../src/utilities/X' → '../src/utilities/X' (depth 2 routes)
78 lines
2.7 KiB
JavaScript
78 lines
2.7 KiB
JavaScript
const express = require('express');
|
|
const fs = require('fs');
|
|
const fsp = require('fs').promises;
|
|
const { exists } = require('../src/utilities/fs-helpers');
|
|
const { paginate, parsePaginationParams } = require('../src/utilities/pagination');
|
|
const { success } = require('../src/utils/responses');
|
|
|
|
/**
|
|
* Error logs routes factory
|
|
* @param {Object} deps - Explicit dependencies
|
|
* @param {string} deps.ERROR_LOG_FILE - Path to error log file
|
|
* @param {Object} deps.auditLogger - Audit logger instance
|
|
* @param {Function} deps.asyncHandler - Async route handler wrapper
|
|
* @returns {express.Router}
|
|
*/
|
|
module.exports = function({ ERROR_LOG_FILE, auditLogger, asyncHandler }) {
|
|
const router = express.Router();
|
|
|
|
// Get error logs
|
|
router.get('/error-logs', asyncHandler(async (req, res) => {
|
|
if (!await exists(ERROR_LOG_FILE)) {
|
|
return success(res, { logs: [] });
|
|
}
|
|
|
|
const logContent = await fsp.readFile(ERROR_LOG_FILE, 'utf8');
|
|
const logEntries = logContent.split('='.repeat(80)).filter(entry => entry.trim());
|
|
|
|
const logs = logEntries.map(entry => {
|
|
const lines = entry.trim().split('\n');
|
|
const firstLine = lines[0] || '';
|
|
const match = firstLine.match(/\[(.*?)\] (.*?): (.*)/);
|
|
|
|
if (match) {
|
|
return {
|
|
timestamp: match[1],
|
|
context: match[2],
|
|
error: match[3]
|
|
};
|
|
}
|
|
return null;
|
|
}).filter(Boolean);
|
|
|
|
success(res, { logs: logs.slice(-50).reverse() });
|
|
}, 'error-logs-get'));
|
|
|
|
// Clear error logs
|
|
router.delete('/error-logs', asyncHandler(async (req, res) => {
|
|
if (await exists(ERROR_LOG_FILE)) {
|
|
await fsp.writeFile(ERROR_LOG_FILE, '');
|
|
}
|
|
success(res, { message: 'Error logs cleared' });
|
|
}, 'error-logs-clear'));
|
|
|
|
// Audit log
|
|
router.get('/audit-logs', asyncHandler(async (req, res) => {
|
|
const paginationParams = parsePaginationParams(req.query);
|
|
const action = req.query.action || '';
|
|
if (paginationParams) {
|
|
// When paginating, fetch all matching entries and let pagination slice
|
|
const entries = await auditLogger.query({ limit: Number.MAX_SAFE_INTEGER, offset: 0, action });
|
|
const result = paginate(entries, paginationParams);
|
|
success(res, { entries: result.data, pagination: result.pagination });
|
|
} else {
|
|
const limit = parseInt(req.query.limit) || 50;
|
|
const offset = parseInt(req.query.offset) || 0;
|
|
const entries = await auditLogger.query({ limit, offset, action });
|
|
success(res, { entries });
|
|
}
|
|
}, 'audit-log'));
|
|
|
|
router.delete('/audit-logs', asyncHandler(async (req, res) => {
|
|
await auditLogger.clear();
|
|
success(res, { message: 'Audit log cleared' });
|
|
}, 'audit-log-clear'));
|
|
|
|
return router;
|
|
};
|