fix: address PR review comments for httpx migration

- Remove unused error_message variable in exception handlers
- Restore error assertion in test_multiple_account_ids_mixed_success

Author: ryoppippi

stackone_ai/models.py

@@ -255,7 +255,6 @@ def execute(
             raise ValueError(error_message) from exc
         except httpx.HTTPStatusError as exc:
             status = "error"
-            error_message = str(exc)
             response_body = None
             if exc.response.text:
                 try:
@@ -269,7 +268,6 @@ def execute(
             ) from exc
         except httpx.RequestError as exc:
             status = "error"
-            error_message = str(exc)
             raise StackOneError(f"Request failed: {exc}") from exc
         finally:
             datetime.now(timezone.utc)

tests/test_feedback.py

@@ -200,6 +200,8 @@ def custom_side_effect(request: httpx.Request) -> httpx.Response:
 
         error_result = next(r for r in result["results"] if r["account_id"] == "acc_unauthorized")
         assert error_result["status"] == "error"
+        assert "error" in error_result
+        assert "401" in error_result["error"]
 
     def test_tool_integration(self) -> None:
         """Test that feedback tool integrates properly with toolset."""