Rim/codtracker-js
Rim/codtracker-js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

fix(logger.js): properly log user activity and api activity separately in node and bundled environments

Browse Source
This commit is contained in:
Rim 2025-04-01 22:16:53 -04:00
parent 50f338917c
commit 4a81f0411f
1 changed files with 46 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

69
src/js/logger.js
View File

@ -3,32 +3,51 @@ const fs = require('fs');
const path = require('path'); const path = require('path');
class Logger { class Logger {
constructor(options = {}) { constructor(options = {}) {
// Dynamically determine the base directory
const isPackaged = process.pkg !== undefined;
let baseDir;
if (isPackaged) {
// If running as a packaged executable, use the directory where the executable is located
baseDir = path.dirname(process.execPath);
} else {
// In development, use the project root directory
baseDir = process.cwd();
}
this.options = { this.options = {
logToConsole: options.logToConsole !== false, logToConsole: options.logToConsole !== false,
logToFile: options.logToFile || false, logToFile: options.logToFile || false,
logDirectory: options.logDirectory || path.join(__dirname, '..', '..', 'logs'), // Use the determined base directory + logs
userActivityLogFile: options.userActivityLogFile || 'user-activity.log', logDirectory: options.logDirectory || path.join(baseDir, 'logs'),
apiLogFile: options.apiLogFile || 'api.log', userActivityLogFile: options.userActivityLogFile || 'user-activity.log',
// Log levels: debug, info, warn, error apiLogFile: options.apiLogFile || 'api.log',
minLevel: options.minLevel || 'info' minLevel: options.minLevel || 'info'
}; };
// Create log directory if it doesn't exist and logging to file is enabled // Create log directory if it doesn't exist and logging to file is enabled
if (this.options.logToFile) { if (this.options.logToFile) {
if (!fs.existsSync(this.options.logDirectory)) { try {
fs.mkdirSync(this.options.logDirectory, { recursive: true }); if (!fs.existsSync(this.options.logDirectory)) {
} fs.mkdirSync(this.options.logDirectory, { recursive: true });
}
} catch (err) {
console.error(`Failed to create log directory: ${err.message}`);
// Fall back to logging in the same directory as the executable if directory creation fails
this.options.logDirectory = baseDir;
console.log(`Falling back to logging in: ${this.options.logDirectory}`);
}
} }
// Log levels and their priorities // Log levels and their priorities
this.levels = { this.levels = {
debug: 0, debug: 0,
info: 1, info: 1,
warn: 2, warn: 2,
error: 3 error: 3
}; };
} }
shouldLog(level) { shouldLog(level) {
return this.levels[level] >= this.levels[this.options.minLevel]; return this.levels[level] >= this.levels[this.options.minLevel];
@ -51,16 +70,20 @@ class Logger {
writeToFile(content, isUserActivity = false) { writeToFile(content, isUserActivity = false) {
if (!this.options.logToFile) return; if (!this.options.logToFile) return;
const logFile = isUserActivity const logFile = isUserActivity
? path.join(this.options.logDirectory, this.options.userActivityLogFile) ? path.join(this.options.logDirectory, this.options.userActivityLogFile)
: path.join(this.options.logDirectory, this.options.apiLogFile); : path.join(this.options.logDirectory, this.options.apiLogFile);
fs.appendFile(logFile, content + '\n', (err) => { try {
if (err) { fs.appendFileSync(logFile, content + '\n');
console.error(`Error writing to log file: ${err.message}`); } catch (err) {
console.error(`Error writing to log file: ${err.message}`);
// Optionally fall back to console logging if file writing fails
if (this.options.logToConsole) {
console.error(`Failed to write to log file, logging to console instead: ${content}`);
} }
}); }
} }
debug(message, data = {}) { debug(message, data = {}) {