fix(D3b): set_model also requires callable derive; reset inside try (Greptile)

- Greptile: runtime_checkable only checks the derive attribute exists
  (and not even callability pre-3.12); also require callable(model.derive)
  so a non-callable derive is rejected at swap time.
- Greptile: move the leading MoodRegistry._reset() inside the try for
  consistency with the test_mood.py fixture pattern.

Author: chiruu12

src/hive/agents/mood.py

@@ -82,10 +82,12 @@ def __init__(self) -> None:
         self._model: MoodModel = CircumplexMood()
 
     def set_model(self, model: MoodModel) -> None:
-        # MoodModel is runtime-checkable, so this catches a misconfigured model
-        # at swap time rather than with an AttributeError on the next derive().
-        if not isinstance(model, MoodModel):
-            raise TypeError("mood model must implement MoodModel (a derive(...) method)")
+        # MoodModel is runtime-checkable but only checks that a `derive` attribute
+        # exists (pre-3.12 doesn't even check callability), so also require it be
+        # callable -- catching a misconfigured model at swap time rather than with
+        # an error on the next derive(). Signature validation is left to runtime.
+        if not isinstance(model, MoodModel) or not callable(model.derive):
+            raise TypeError("mood model must implement MoodModel (a callable derive(...) method)")
         self._model = model
 
     def derive(self, happiness: float, suffering_load: float, in_crisis: bool) -> MoodState:

tests/test_narrative_in_prompt.py

@@ -120,9 +120,9 @@ async def test_pursuit_prompt_includes_mood(hive_dir: Path) -> None:
     from hive.runtime.persona import Persona
 
     # Isolate the global MoodRegistry so a model swapped by another test can't
-    # change the derived label this test asserts on.
-    MoodRegistry._reset()
+    # change the derived label this test asserts on (reset before and after).
     try:
+        MoodRegistry._reset()
         store = HiveStore(hive_dir / "hive.db")
         await store.initialize()
         agent = AgentState(