diff --git a/gateway/config.py b/gateway/config.py index 2b187c521..0cf8fdfa9 100644 --- a/gateway/config.py +++ b/gateway/config.py @@ -97,10 +97,11 @@ class SessionResetPolicy: @classmethod def from_dict(cls, data: Dict[str, Any]) -> "SessionResetPolicy": # Handle both missing keys and explicit null values (YAML null → None) + mode = data.get("mode") at_hour = data.get("at_hour") idle_minutes = data.get("idle_minutes") return cls( - mode=data.get("mode", "both"), + mode=mode if mode is not None else "both", at_hour=at_hour if at_hour is not None else 4, idle_minutes=idle_minutes if idle_minutes is not None else 1440, ) diff --git a/tests/gateway/test_config.py b/tests/gateway/test_config.py index c604ee521..d23147d81 100644 --- a/tests/gateway/test_config.py +++ b/tests/gateway/test_config.py @@ -83,6 +83,14 @@ class TestSessionResetPolicy: assert policy.at_hour == 4 assert policy.idle_minutes == 1440 + def test_from_dict_treats_null_values_as_defaults(self): + restored = SessionResetPolicy.from_dict( + {"mode": None, "at_hour": None, "idle_minutes": None} + ) + assert restored.mode == "both" + assert restored.at_hour == 4 + assert restored.idle_minutes == 1440 + class TestGatewayConfigRoundtrip: def test_full_roundtrip(self):