Fix architecture as well as FunctionResponse class implementation to correctly define ID and name return types as string or null.

Author: felixarntz

docs/ARCHITECTURE.md

@@ -687,8 +687,8 @@ direction LR
 
     namespace AiClientNamespace.Tools.DTO {
         class FunctionCall {
-            +getId() string
-            +getName() string
+            +getId() ?string
+            +getName() ?string
             +getArgs() array< string, mixed >
             +getJsonSchema() array< string, mixed >$
         }
@@ -699,8 +699,8 @@ direction LR
             +getJsonSchema() array< string, mixed >$
         }
         class FunctionResponse {
-            +getId() string
-            +getName() string
+            +getId() ?string
+            +getName() ?string
             +getResponse() mixed
             +getJsonSchema() array< string, mixed >$
         }

src/Tools/DTO/FunctionCall.php

@@ -65,7 +65,7 @@ public function __construct(?string $id = null, ?string $name = null, array $arg
      *
      * @since n.e.x.t
      *
-     * @return string|null The unique identifier.
+     * @return string|null The function call ID.
      */
     public function getId(): ?string
     {

src/Tools/DTO/FunctionResponse.php

@@ -59,9 +59,9 @@ public function __construct(string $id, string $name, $response)
      *
      * @since n.e.x.t
      *
-     * @return string The function call ID.
+     * @return string|null The function call ID.
      */
-    public function getId(): string
+    public function getId(): ?string
     {
         return $this->id;
     }
@@ -71,9 +71,9 @@ public function getId(): string
      *
      * @since n.e.x.t
      *
-     * @return string The function name.
+     * @return string|null The function name.
      */
-    public function getName(): string
+    public function getName(): ?string
     {
         return $this->name;
     }