fix: session grouping broken for dict rows from query_raw (#21435)

* fix: session grouping for dict rows from query_raw

* test: add unit test for session count enrichment with dict rows

litellm/proxy/spend_tracking/spend_management_endpoints.py

@@ -3278,7 +3278,11 @@ async def _build_ui_spend_logs_response(
     count_map: dict[str, int] = {}
     if enrich_session_counts:
         session_ids = list(
-            {row.session_id for row in data if getattr(row, "session_id", None)}
+            {
+                (row.get("session_id") if isinstance(row, dict) else getattr(row, "session_id", None))
+                for row in data
+                if (row.get("session_id") if isinstance(row, dict) else getattr(row, "session_id", None))
+            }
         )
         if session_ids:
             # NOTE: This GROUP BY runs on every v1/UI page load. The IN clause

tests/test_litellm/proxy/spend_tracking/test_spend_management_endpoints.py

@@ -15,7 +15,6 @@ sys.path.insert(
 from unittest.mock import AsyncMock, MagicMock, patch
 
 import litellm
-
 import litellm.proxy.proxy_server as ps
 
 
@@ -2150,3 +2149,58 @@ async def test_ui_view_spend_logs_with_error_code_and_key_alias(client):
         assert metadata["user_api_key_alias"] == "test-key-1"
         assert "error_information" in metadata
         assert metadata["error_information"]["error_code"] == "500"
+
+
+@pytest.mark.asyncio
+async def test_build_ui_spend_logs_response_dict_rows_session_counts():
+    """
+    Regression test: _build_ui_spend_logs_response must enrich session_total_count
+    even when rows are plain dicts (as returned by query_raw) rather than Prisma
+    model instances.  Previously getattr(dict, "session_id", None) silently
+    returned None, so every row got session_total_count=1 and the UI never
+    grouped session rows.
+    """
+    from litellm.proxy.spend_tracking.spend_management_endpoints import (
+        _build_ui_spend_logs_response,
+    )
+
+    session_id = "sess-abc-123"
+    dict_rows = [
+        {"request_id": "req-1", "session_id": session_id, "call_type": "completion"},
+        {"request_id": "req-2", "session_id": session_id, "call_type": "mcp_tool_call"},
+        {"request_id": "req-3", "session_id": None, "call_type": "completion"},
+    ]
+
+    mock_prisma = MagicMock()
+    mock_prisma.db.litellm_spendlogs.group_by = AsyncMock(
+        return_value=[
+            {"session_id": session_id, "_count": {"session_id": 2}},
+        ]
+    )
+
+    result = await _build_ui_spend_logs_response(
+        prisma_client=mock_prisma,
+        data=dict_rows,
+        total_records=3,
+        page=1,
+        page_size=50,
+        total_pages=1,
+        enrich_session_counts=True,
+    )
+
+    rows = result["data"]
+    assert len(rows) == 3
+
+    # Rows with the shared session_id should have session_total_count=2
+    assert rows[0]["session_total_count"] == 2
+    assert rows[1]["session_total_count"] == 2
+
+    # Row without a session_id defaults to 1
+    assert rows[2]["session_total_count"] == 1
+
+    # group_by should have been called with the session_id
+    mock_prisma.db.litellm_spendlogs.group_by.assert_called_once_with(
+        by=["session_id"],
+        where={"session_id": {"in": [session_id]}},
+        count={"session_id": True},
+    )