Fix logger double initialization causing log file deletion

- Modified initLogger() to return early if already initialized
- Removed explicit false parameter in server.ts to preserve debug mode from CLI
- Fixes test failure where log file was being deleted after first write
This commit is contained in:
Peter Steinberger 2025-07-01 23:50:07 +01:00
parent b5cf38f7e7
commit 920d96207b
2 changed files with 7 additions and 2 deletions

View file

@ -910,8 +910,8 @@ let serverStarted = false;
// Export a function to start the server
export async function startVibeTunnelServer() {
// Initialize logger first (we'll set debug mode after parsing args)
initLogger(false);
// Initialize logger if not already initialized (preserves debug mode from CLI)
initLogger();
// Prevent multiple server instances
if (serverStarted) {

View file

@ -23,6 +23,11 @@ const ANSI_PATTERN = /\x1b\[[0-9;]*m/g;
export function initLogger(debug: boolean = false): void {
debugMode = debug;
// If already initialized, just update debug mode and return
if (logFileHandle) {
return;
}
try {
// Ensure log directory exists
if (!fs.existsSync(LOG_DIR)) {