fix: Respect BASIC_MEMORY_LOG_LEVEL and BASIC_MEMORY_CONSOLE_LOGGING environment variables

[jope-bm]

Summary

Fixes #255 - Console logging was ignoring BASIC_MEMORY_CONSOLE_LOGGING=false and log_level=ERROR settings.

Root Cause

The logging setup function was not properly reading environment variables before falling back to config file settings. This was due to a timing issue where logging setup occurred during module import before environment variables could be processed through Pydantic's settings system.

Changes Made

Environment Variable Support:

• ✅ BASIC_MEMORY_LOG_LEVEL environment variable is now properly read and used
• ✅ BASIC_MEMORY_CONSOLE_LOGGING now accepts multiple truthy values: "true", "1", "yes", "on" (case-insensitive)
• ✅ Environment variables take precedence over config file settings as expected

Technical Implementation:

• Modified setup_basic_memory_logging() in src/basic_memory/config.py
• Added direct environment variable reading before config fallback
• Enhanced boolean parsing for console logging setting

Testing

Verified that the following configurations now work correctly:

# Disable console logging entirely
BASIC_MEMORY_CONSOLE_LOGGING=false BASIC_MEMORY_LOG_LEVEL=ERROR basic-memory command

# Enable console logging with ERROR level only
BASIC_MEMORY_CONSOLE_LOGGING=true BASIC_MEMORY_LOG_LEVEL=ERROR basic-memory command

# Alternative truthy values
BASIC_MEMORY_CONSOLE_LOGGING=1 basic-memory command
BASIC_MEMORY_CONSOLE_LOGGING=yes basic-memory command

Before vs After

Before:

• ❌ BASIC_MEMORY_LOG_LEVEL ignored completely
• ❌ Console shows INFO/DEBUG logs regardless of settings
• ❌ Only accepts exactly "true" for console logging

After:

• ✅ BASIC_MEMORY_LOG_LEVEL=ERROR correctly filters to ERROR+ only
• ✅ BASIC_MEMORY_CONSOLE_LOGGING=false properly disables console output
• ✅ Accepts "true", "1", "yes", "on" for console logging

Test plan

• Environment variables properly override config file settings
• Log level filtering works correctly (ERROR only shows ERROR/CRITICAL)
• Console logging can be disabled via environment variable
• Multiple truthy values accepted for console logging
• Existing functionality preserved (config-based tests still pass)
• No regressions in related functionality

🤖 Generated with Claude Code

[phernandez]

I always hate logging.

[CLAassistant]

All committers have signed the CLA.