feat(logging): suppress error/warn logs during graceful shutdown

Add global shutdown flag to prevent error/warn logs from being emitted during graceful shutdown. This provides cleaner output when handling SIGINT/Ctrl+C signals.

Author: moazbuilds

src/agents/monitoring/cleanup.ts

@@ -1,6 +1,7 @@
 import { AgentMonitorService } from './monitor.js';
 import { AgentLoggerService } from './logger.js';
 import * as logger from '../../shared/logging/logger.js';
+import { setShuttingDown } from '../../shared/logging/logger.js';
 import { killAllActiveProcesses } from '../../infra/process/spawn.js';
 
 /**
@@ -132,6 +133,9 @@ export class MonitoringCleanup {
     // Second Ctrl+C (after debounce): Stop workflow and exit
     logger.debug(`[${source}] Second Ctrl+C detected after ${timeSinceFirst}ms - stopping workflow and exiting`);
 
+    // Suppress all error/warn logs during graceful shutdown
+    setShuttingDown(true);
+
     // Emit workflow:skip to abort the currently running step
     (process as NodeJS.EventEmitter).emit('workflow:skip');
 
@@ -153,6 +157,9 @@ export class MonitoringCleanup {
   private static async handleSignal(signal: string, message: string): Promise<void> {
     logger.debug(`Received ${signal}: ${message}`);
 
+    // Suppress all error/warn logs during graceful shutdown
+    setShuttingDown(true);
+
     // Kill all active child processes before cleanup
     logger.debug('Killing all active child processes...');
     killAllActiveProcesses();

src/shared/logging/logger.ts

@@ -17,6 +17,20 @@ type LogLevel = keyof typeof LOG_LEVELS;
 
 let debugLogStream: fs.WriteStream | null = null;
 
+/**
+ * Global shutdown flag - when true, error/warn logs are suppressed
+ * Set this when handling SIGINT/Ctrl+C for clean exit
+ */
+let isShuttingDown = false;
+
+export function setShuttingDown(value: boolean): void {
+  isShuttingDown = value;
+}
+
+export function getShuttingDown(): boolean {
+  return isShuttingDown;
+}
+
 function resolveRequestedLevel(): string {
   const explicit = process.env.LOG_LEVEL;
   if (explicit && explicit.trim()) {
@@ -80,6 +94,7 @@ export function info(message: string, ...args: unknown[]): void {
 }
 
 export function warn(message: string, ...args: unknown[]): void {
+  if (isShuttingDown) return; // Suppress during graceful shutdown
   if (shouldLog('warn')) {
     // Write directly to stderr to bypass console hijacking
     const formatted = formatMessage(`[WARN] ${message}`, ...args);
@@ -88,6 +103,7 @@ export function warn(message: string, ...args: unknown[]): void {
 }
 
 export function error(message: string, ...args: unknown[]): void {
+  if (isShuttingDown) return; // Suppress during graceful shutdown
   if (shouldLog('error')) {
     // Write directly to stderr to bypass console hijacking
     const formatted = formatMessage(`[ERROR] ${message}`, ...args);