Centralize API response parsing with ParsesApiResponse trait and fix null handling

- Introduced `ParsesApiResponse` trait to DRY up `parseResponse` logic
- Refactored `QueryBuilder` and `ApiModel` to use the new trait, removing duplication
- Enhanced `parseResponse` to handle:
  - `HttpResponse` (Laravel HTTP client)
  - `JsonResponse`
  - arrays, `Collection`, JSON strings, and generic objects with `toArray`
- Added null‐response checks to prevent `method_exists` on null and fixed test failures for missing data

Author: KroderDev

CHANGELOG.md

@@ -10,6 +10,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Query builder with `where()->get()` support for remote models.
 - Smart error handling based on the request's expected format.
 - Error propagation improvements with tests for all status codes.
+- Introduced `ParsesApiResponse` trait to centralize API response parsing logic.
+
+### Changed
+- Refactored `QueryBuilder` and `ApiModel` to use the new `ParsesApiResponse` trait, removing duplicate `parseResponse` implementations.
+
 
 ## [0.4.1] - 2025-07-10
 ### Added

src/Models/Model.php

@@ -9,10 +9,12 @@
 use Kroderdev\LaravelMicroserviceCore\Contracts\ApiGatewayClientInterface;
 use Kroderdev\LaravelMicroserviceCore\Interfaces\ApiModelContract;
 use Kroderdev\LaravelMicroserviceCore\Traits\ApiModelTrait;
+use Kroderdev\LaravelMicroserviceCore\Traits\ParsesApiResponse;
 
 abstract class Model extends BaseModel implements ApiModelContract
 {
     use ApiModelTrait;
+    use ParsesApiResponse;
 
     /**
      * Endpoint for this model. Defaults to plural kebab of the class name.
@@ -150,28 +152,4 @@ public function refresh(): static
 
         return $this;
     }
-
-    /**
-     * Normalize different response types to array.
-     */
-    protected static function parseResponse($response): array
-    {
-        if ($response === null) {
-            return [];
-        }
-
-        if (is_array($response)) {
-            return $response;
-        }
-
-        if ($response instanceof Collection) {
-            return $response->toArray();
-        }
-
-        if (method_exists($response, 'json')) {
-            return $response->json();
-        }
-
-        return (array) $response;
-    }
 }

src/Models/QueryBuilder.php

@@ -3,9 +3,12 @@
 namespace Kroderdev\LaravelMicroserviceCore\Models;
 
 use Illuminate\Support\Collection;
+use Kroderdev\LaravelMicroserviceCore\Traits\ParsesApiResponse;
 
 class QueryBuilder
 {
+    use ParsesApiResponse;
+
     protected string $model;
 
     protected array $filters = [];
@@ -31,25 +34,4 @@ public function get($columns = ['*']): Collection
 
         return $model::fromCollection($data['data'] ?? $data);
     }
-
-    protected function parseResponse($response): array
-    {
-        if ($response === null) {
-            return [];
-        }
-
-        if (is_array($response)) {
-            return $response;
-        }
-
-        if ($response instanceof Collection) {
-            return $response->toArray();
-        }
-
-        if (method_exists($response, 'json')) {
-            return $response->json();
-        }
-
-        return (array) $response;
-    }
 }

src/Traits/ParsesApiResponse.php

@@ -0,0 +1,60 @@
+<?php
+
+namespace Kroderdev\LaravelMicroserviceCore\Traits;
+
+use Illuminate\Http\Client\Response as HttpResponse;
+use Illuminate\Http\JsonResponse;
+use Illuminate\Support\Collection;
+
+trait ParsesApiResponse
+{
+    /**
+     * Normalize different response types to array.
+     *
+     * @param mixed $response
+     * @return array
+     */
+    protected static function parseResponse(mixed $response): array
+    {
+        if (is_null($response)) {
+            return [];
+        }
+
+        if ($response instanceof HttpResponse) {
+            $response->throw();
+            $decoded = $response->json();
+            return is_array($decoded) ? $decoded : [];
+        }
+
+        if ($response instanceof JsonResponse) {
+            $decoded = $response->getData(true);
+            return is_array($decoded) ? $decoded : [];
+        }
+
+        if (is_array($response)) {
+            return $response;
+        }
+
+        if ($response instanceof Collection) {
+            return $response->toArray();
+        }
+
+        if (is_string($response)) {
+            $decoded = json_decode($response, true);
+            return (json_last_error() === JSON_ERROR_NONE && is_array($decoded))
+                ? $decoded
+                : [];
+        }
+
+        if (method_exists($response, 'toArray')) {
+            try {
+                $array = $response->toArray();
+                return is_array($array) ? $array : [];
+            } catch (\Throwable $e) {
+                return [];
+            }
+        }
+
+        return [];
+    }
+}