fix(Gemini): correct list models call (openai-php#567)

* fix: Adjusts Model\RetrieveResponse to handle Gemini API calls

* fix: Adjusts PHPdoc to account for nullable 'created' field

Author: the-fermi-paradox

src/Responses/Models/ListResponse.php

@@ -12,12 +12,12 @@
 use OpenAI\Testing\Responses\Concerns\Fakeable;
 
 /**
- * @implements ResponseContract<array{object: string, data: array<int, array{id: string, object: string, created: int, owned_by: string}>}>
+ * @implements ResponseContract<array{object: string, data: array<int, array{id: string, object: string, created: ?int, owned_by: string}>}>
  */
 final class ListResponse implements ResponseContract, ResponseHasMetaInformationContract
 {
     /**
-     * @use ArrayAccessible<array{object: string, data: array<int, array{id: string, object: string, created: int, owned_by: string}>}>
+     * @use ArrayAccessible<array{object: string, data: array<int, array{id: string, object: string, created: ?int, owned_by: string}>}>
      */
     use ArrayAccessible;
 
@@ -36,7 +36,7 @@ private function __construct(
     /**
      * Acts as static factory, and returns a new Response instance.
      *
-     * @param  array{object: string, data: array<int, array{id: string, object: string, created: int, owned_by: string}>}  $attributes
+     * @param  array{object: string, data: array<int, array{id: string, object: string, created: ?int, owned_by: string}>}  $attributes
      */
     public static function from(array $attributes, MetaInformation $meta): self
     {

src/Responses/Models/RetrieveResponse.php

@@ -12,12 +12,12 @@
 use OpenAI\Testing\Responses\Concerns\Fakeable;
 
 /**
- * @implements ResponseContract<array{id: string, object: string, created: int, owned_by: string}>
+ * @implements ResponseContract<array{id: string, object: string, created: ?int, owned_by: string}>
  */
 final class RetrieveResponse implements ResponseContract, ResponseHasMetaInformationContract
 {
     /**
-     * @use ArrayAccessible<array{id: string, object: string, created: int, owned_by: string}>
+     * @use ArrayAccessible<array{id: string, object: string, created: ?int, owned_by: string}>
      */
     use ArrayAccessible;
 
@@ -27,22 +27,22 @@ final class RetrieveResponse implements ResponseContract, ResponseHasMetaInforma
     private function __construct(
         public readonly string $id,
         public readonly string $object,
-        public readonly int $created,
+        public readonly ?int $created,
         public readonly string $ownedBy,
         private readonly MetaInformation $meta,
     ) {}
 
     /**
      * Acts as static factory, and returns a new Response instance.
      *
-     * @param  array{id: string, object: string, created: int, owned_by: string}  $attributes
+     * @param  array{id: string, object: string, created: ?int, owned_by: string}  $attributes
      */
     public static function from(array $attributes, MetaInformation $meta): self
     {
         return new self(
             $attributes['id'],
             $attributes['object'],
-            $attributes['created'],
+            $attributes['created'] ?? null,
             $attributes['owned_by'],
             $meta,
         );

tests/Fixtures/Model.php

@@ -13,6 +13,18 @@ function model(): array
     ];
 }
 
+/**
+ * @return array<string, string>
+ */
+function googleModel(): array
+{
+    return [
+        'id' => 'text-davinci-003',
+        'object' => 'model',
+        'owned_by' => 'google',
+    ];
+}
+
 /**
  * @return array<string, mixed>
  */

tests/Responses/Models/RetrieveResponse.php

@@ -15,6 +15,17 @@
         ->meta()->toBeInstanceOf(MetaInformation::class);
 });
 
+test('from google', function () {
+    $result = RetrieveResponse::from(googleModel(), meta());
+
+    expect($result)
+        ->toBeInstanceOf(RetrieveResponse::class)
+        ->id->toBe('text-davinci-003')
+        ->object->toBe('model')
+        ->ownedBy->toBe('google')
+        ->meta()->toBeInstanceOf(MetaInformation::class);
+});
+
 test('as array accessible', function () {
     $result = RetrieveResponse::from(model(), meta());