From 975dece061f94f2b415bb644bb76637f347d0e70 Mon Sep 17 00:00:00 2001 From: Anatoly Koptev Date: Fri, 6 Feb 2026 22:18:57 -0800 Subject: [PATCH] fix: downgrade AuthConfig partial-init log from WARNING to INFO MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit AuthConfig.validate_partial_initialization() logs a WARNING on every startup when some components (openai, graph_db) are not configured. This is noisy because partial configuration is a valid and common state — the individual from_local_env() methods already log appropriate warnings for actual initialization failures. Change the partial-init log to INFO level; keep WARNING only for the edge case where ALL components are None. Co-Authored-By: Claude Opus 4.6 --- src/memos/configs/mem_scheduler.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/memos/configs/mem_scheduler.py b/src/memos/configs/mem_scheduler.py index a28f3bdce..9807f42c3 100644 --- a/src/memos/configs/mem_scheduler.py +++ b/src/memos/configs/mem_scheduler.py @@ -250,8 +250,12 @@ def validate_partial_initialization(self) -> "AuthConfig": "All configuration components are None. This may indicate missing environment variables or configuration files." ) elif failed_components: - logger.warning( - f"Failed to initialize components: {', '.join(failed_components)}. Successfully initialized: {', '.join(initialized_components)}" + # Use info level: individual from_local_env() methods already log + # warnings for actual initialization failures. Components that are + # simply not configured (no env vars) are not errors. + logger.info( + f"Components not configured: {', '.join(failed_components)}. " + f"Successfully initialized: {', '.join(initialized_components)}" ) return self