From 4284ec6eba0e1c9635c12ac685d113d8b351c53a Mon Sep 17 00:00:00 2001 From: Ed_ Date: Wed, 10 Jun 2026 09:03:12 -0400 Subject: [PATCH] fix(controller): remove 'persona_manager' from _LAZY_MANAGER_DEFAULTS --- src/app_controller.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/app_controller.py b/src/app_controller.py index 1d35b11f..363ebca7 100644 --- a/src/app_controller.py +++ b/src/app_controller.py @@ -1261,12 +1261,12 @@ class AppController: # Manager attributes that are initialized by init_state() but are absent # on a bare AppController() (which some tests construct). Return None # for these so test code that references them without calling init_state - # does not crash. hasattr() still returns False for non-mocked access - # paths because callers wrap in try/except for AttributeError when they - # need to distinguish "lazy" from "absent". + # does not crash. NOTE: callers that need to distinguish "lazy" from + # "absent" must use try/except AttributeError explicitly; hasattr() + # returns True because __getattr__ returns None (a valid attribute + # value). _LAZY_MANAGER_DEFAULTS = { "context_preset_manager", - "persona_manager", "tool_preset_manager", "preset_manager", "vendor_state",