From f42cbac4659e03b51beae2fa21714d7d9ea8b471 Mon Sep 17 00:00:00 2001 From: Zbigniew Sobiecki Date: Mon, 16 Feb 2026 11:17:31 +0000 Subject: [PATCH] fix: make LOG_LEVEL env var respected by llmist agent logger The llmist agent logger in agents/utils/setup.ts only checked LLMIST_LOG_LEVEL, ignoring the standard LOG_LEVEL env var used everywhere else. This caused agent debug logs to be invisible in Docker worker containers even when LOG_LEVEL=debug was set. - agents/utils/setup.ts: fall through to LOG_LEVEL when LLMIST_LOG_LEVEL is not set (preserves backwards compat) - index.ts, worker-entry.ts: remove redundant || 'info' fallback since env.ts already defaults LOG_LEVEL to 'info' - config/env.ts: tighten loadEnvConfigSafe return type so logLevel and port are non-optional (only databaseUrl is truly optional) Co-Authored-By: Claude Opus 4.6 --- src/agents/utils/setup.ts | 3 ++- src/config/env.ts | 2 +- src/index.ts | 2 +- src/worker-entry.ts | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/agents/utils/setup.ts b/src/agents/utils/setup.ts index ee49cd8e..fa717b1f 100644 --- a/src/agents/utils/setup.ts +++ b/src/agents/utils/setup.ts @@ -18,7 +18,8 @@ export const LOG_LEVELS: Record = { }; export function getLogLevel(): number { - const level = process.env.LLMIST_LOG_LEVEL?.toLowerCase() || 'debug'; + const level = + process.env.LLMIST_LOG_LEVEL?.toLowerCase() || process.env.LOG_LEVEL?.toLowerCase() || 'debug'; return LOG_LEVELS[level] ?? LOG_LEVELS.debug; } diff --git a/src/config/env.ts b/src/config/env.ts index 04811242..41dffffd 100644 --- a/src/config/env.ts +++ b/src/config/env.ts @@ -24,7 +24,7 @@ export function loadEnvConfig(): EnvConfig { }; } -export function loadEnvConfigSafe(): Partial { +export function loadEnvConfigSafe(): Omit & { databaseUrl?: string } { return { port: Number.parseInt(getEnvOrDefault('PORT', '3000'), 10), logLevel: getEnvOrDefault('LOG_LEVEL', 'info'), diff --git a/src/index.ts b/src/index.ts index dac1350a..82f247f9 100644 --- a/src/index.ts +++ b/src/index.ts @@ -13,7 +13,7 @@ import { logger, setLogLevel, startFreshMachineTimer } from './utils/index.js'; async function main(): Promise { // Load environment config const envConfig = loadEnvConfigSafe(); - setLogLevel(envConfig.logLevel || 'info'); + setLogLevel(envConfig.logLevel); logger.info('Starting Cascade server', { port: envConfig.port }); diff --git a/src/worker-entry.ts b/src/worker-entry.ts index c6632fce..76e86fe4 100644 --- a/src/worker-entry.ts +++ b/src/worker-entry.ts @@ -64,7 +64,7 @@ async function main(): Promise { // Load environment config const envConfig = loadEnvConfigSafe(); - setLogLevel(envConfig.logLevel || 'info'); + setLogLevel(envConfig.logLevel); logger.info('[Worker] Starting job', { jobId, jobType });