From f69952a5dab6da70f87001e125521993059533b0 Mon Sep 17 00:00:00 2001 From: Marc Mintel Date: Mon, 2 Feb 2026 17:43:54 +0100 Subject: [PATCH] refactor: Enable pino-pretty transport exclusively for development environments. --- lib/services/logging/pino-logger-service.ts | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/lib/services/logging/pino-logger-service.ts b/lib/services/logging/pino-logger-service.ts index e4cb5132..36ceabb6 100644 --- a/lib/services/logging/pino-logger-service.ts +++ b/lib/services/logging/pino-logger-service.ts @@ -12,20 +12,19 @@ export class PinoLoggerService implements LoggerService { // In Next.js, especially in the Edge runtime or during instrumentation, // pino transports (which use worker threads) can cause issues. // We disable transport in production and during instrumentation. - const useTransport = !config.isProduction && typeof window === 'undefined'; + const useTransport = config.isDevelopment && typeof window === 'undefined'; this.logger = pino({ name: name || 'app', level: config.logging.level, - transport: - useTransport - ? { - target: 'pino-pretty', - options: { - colorize: true, - }, - } - : undefined, + transport: useTransport + ? { + target: 'pino-pretty', + options: { + colorize: true, + }, + } + : undefined, }); } }