From d3c9e293d8e39874b29831b0bcbd6663abbde2d8 Mon Sep 17 00:00:00 2001 From: KoalaTrapLord <265133236+KoalaTrapLord@users.noreply.github.com> Date: Sun, 22 Mar 2026 12:01:53 -0400 Subject: [PATCH] fix: mutate COMMANDER_META in-place so cross-module refs stay synced --- models/state.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/models/state.py b/models/state.py index 2ec8ebd..f4c4000 100644 --- a/models/state.py +++ b/models/state.py @@ -75,17 +75,18 @@ BUILTIN_COMMANDERS: dict = { def load_commander_meta() -> None: """Load commander meta mapping from file or fall back to builtins.""" - global COMMANDER_META meta_path = Path(__file__).parent.parent / "commander-meta.json" if meta_path.exists(): try: with open(meta_path, "r", encoding="utf-8") as f: - COMMANDER_META = json.load(f) + COMMANDER_META.clear() + COMMANDER_META.update(json.load(f)) log.info(f" Meta: Loaded {len(COMMANDER_META)} commanders from {meta_path}") return except Exception as e: log.warning(f" WARNING: Failed to load commander-meta.json: {e}") - COMMANDER_META = BUILTIN_COMMANDERS + COMMANDER_META.clear() + COMMANDER_META.update(BUILTIN_COMMANDERS) log.info(f" Meta: {len(COMMANDER_META)} built-in commanders")