fix: guard whisper_detail error path against non-JSON and non-dict responses

Align error handling with whisper_status pattern: handle empty body,
non-JSON responses, and pass status_code as separate exception argument.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: johnyrahul

src/unstract/llmwhisperer/client_v2.py

@@ -352,8 +352,16 @@ def whisper_detail(self, whisper_hash: str) -> Any:
         req = requests.Request("GET", url, headers=self.headers, params=params)
         prepared = req.prepare()
         response = self._send_request(prepared)
-            err = json.loads(response.text)
-            err["status_code"] = response.status_code
+        if response.status_code != 200:
+            if not (response.text or "").strip():
+                raise LLMWhispererClientException("API error: empty response body", response.status_code)
+            try:
+                err = json.loads(response.text)
+            except json.JSONDecodeError as e:
+                response_preview = response.text[:500] + "..." if len(response.text) > 500 else response.text
+                raise LLMWhispererClientException(
+                    f"API error: non-JSON response - {response_preview}", response.status_code
+                ) from e
             raise LLMWhispererClientException(err, response.status_code)
         return json.loads(response.text)
 

tests/unit/client_v2_test.py

@@ -75,7 +75,7 @@ def test_whisper_detail_not_found(mocker: MockerFixture, client_v2: LLMWhisperer
 
     error = exc_info.value.error_message()
     assert error["message"] == "Record not found"
-    assert error["status_code"] == 400
+    assert exc_info.value.status_code == 400
     mock_send.assert_called_once()