From 9743fcd13ef99bd22baa3f14d81a52de91956857 Mon Sep 17 00:00:00 2001 From: jope-bm Date: Fri, 22 Aug 2025 07:58:19 -0600 Subject: [PATCH] fix: Respect BASIC_MEMORY_LOG_LEVEL and BASIC_MEMORY_CONSOLE_LOGGING environment variables (#264) Signed-off-by: Joe P Co-authored-by: Claude --- src/basic_memory/config.py | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/basic_memory/config.py b/src/basic_memory/config.py index 4f723134..25316997 100644 --- a/src/basic_memory/config.py +++ b/src/basic_memory/config.py @@ -351,15 +351,22 @@ def setup_basic_memory_logging(): # pragma: no cover # print("Skipping duplicate logging setup") return - # Check for console logging environment variable - console_logging = os.getenv("BASIC_MEMORY_CONSOLE_LOGGING", "false").lower() == "true" + # Check for console logging environment variable - accept more truthy values + console_logging_env = os.getenv("BASIC_MEMORY_CONSOLE_LOGGING", "false").lower() + console_logging = console_logging_env in ("true", "1", "yes", "on") + + # Check for log level environment variable first, fall back to config + log_level = os.getenv("BASIC_MEMORY_LOG_LEVEL") + if not log_level: + config_manager = ConfigManager() + log_level = config_manager.config.log_level config_manager = ConfigManager() config = get_project_config() setup_logging( env=config_manager.config.env, home_dir=user_home, # Use user home for logs - log_level=config_manager.config.log_level, + log_level=log_level, log_file=f"{DATA_DIR_NAME}/basic-memory-{process_name}.log", console=console_logging, )