refactor: simplifies Fiel and MessagePart toArray

Author: JasonTheAdams

src/Files/DTO/File.php

@@ -401,10 +401,12 @@ public function toArray(): array
             'mimeType' => $this->getMimeType(),
         ];
 
-        if ($this->fileType->isRemote() && $this->url !== null) {
+        if ($this->url !== null) {
             $data['url'] = $this->url;
         } elseif (!$this->fileType->isRemote() && $this->base64Data !== null) {
             $data['base64Data'] = $this->base64Data;
+        } else {
+            throw new RuntimeException('File requires either url or base64Data. This should not be a possible condition.');
         }
 
         return $data;

src/Messages/DTO/MessagePart.php

@@ -4,6 +4,8 @@
 
 namespace WordPress\AiClient\Messages\DTO;
 
+use InvalidArgumentException;
+use RuntimeException;
 use WordPress\AiClient\Common\AbstractDataValueObject;
 use WordPress\AiClient\Files\DTO\File;
 use WordPress\AiClient\Messages\Enums\MessagePartTypeEnum;
@@ -228,14 +230,16 @@ public function toArray(): array
     {
         $data = ['type' => $this->type->value];
 
-        if ($this->type->isText() && $this->text !== null) {
+        if ($this->text !== null) {
             $data['text'] = $this->text;
-        } elseif ($this->type->isFile() && $this->file !== null) {
+        } elseif ($this->file !== null) {
             $data['file'] = $this->file->toArray();
-        } elseif ($this->type->isFunctionCall() && $this->functionCall !== null) {
+        } elseif ($this->functionCall !== null) {
             $data['functionCall'] = $this->functionCall->toArray();
-        } elseif ($this->type->isFunctionResponse() && $this->functionResponse !== null) {
+        } elseif ($this->functionResponse !== null) {
             $data['functionResponse'] = $this->functionResponse->toArray();
+        } else {
+            throw new RuntimeException('MessagePart requires one of: text, file, functionCall, or functionResponse. This should not be a possible condition.');
         }
 
         return $data;