Merge pull request #190 from kalibr-ai/fix/model-attribution-outcome-reporting

fix: kalibr_model_used on response + re-resolve tenant_id in singleton

Author: devonakelley

kalibr/intelligence.py

@@ -677,7 +677,7 @@ def __exit__(self, *args):
 
 def _get_intelligence_client() -> KalibrIntelligence:
     """Get or create the singleton intelligence client.
-    
+
     Thread-safe singleton pattern using double-checked locking.
     """
     global _intelligence_client
@@ -686,6 +686,11 @@ def _get_intelligence_client() -> KalibrIntelligence:
             # Double-check inside lock to prevent race condition
             if _intelligence_client is None:
                 _intelligence_client = KalibrIntelligence()
+    # Re-resolve if tenant_id is empty (env var may have been set after construction)
+    if not _intelligence_client.tenant_id:
+        _, tenant_id = resolve_credentials()
+        if tenant_id:
+            _intelligence_client.tenant_id = tenant_id
     return _intelligence_client
 
 

kalibr/router.py

@@ -894,6 +894,7 @@ def _heal_dispatch(m_id: str, msgs: List[Dict], system_prompt: Optional[str] = N
                     response.kalibr_healed = bool(heal_result.get("healed"))
                     response.kalibr_heal_count = heal_result.get("heal_count", 0)
                     response.kalibr_models_tried = heal_result.get("models_tried") or []
+                    response.kalibr_model_used = used_model
                     return response
 
                 # Heal loop exhausted all paths — fire failure telemetry.
@@ -1071,6 +1072,7 @@ def _heal_dispatch(m_id: str, msgs: List[Dict], system_prompt: Optional[str] = N
 
                     # Add trace_id to response for explicit linkage
                     response.kalibr_trace_id = trace_id
+                    response.kalibr_model_used = candidate_model
                     return response
 
                 except Exception as e: