fix(OpenAI): augment support for Response API - MCP Connectors

Author: iBotPeaches

src/Responses/Responses/Output/OutputMcpCall.php

@@ -48,7 +48,10 @@ public static function from(array $attributes): self
             arguments: $attributes['arguments'],
             name: $attributes['name'],
             approvalRequestId: $attributes['approval_request_id'],
-            error: $attributes['error'],
+            // TODO - Why is this coming back as an array?
+            error: is_array($attributes['error'])
+            ? json_encode($attributes['error'])
+            : $attributes['error'],
             output: $attributes['output'],
         );
     }

src/Responses/Responses/Tool/RemoteMcpTool.php

@@ -11,7 +11,7 @@
 /**
  * @phpstan-import-type McpToolNamesFilterType from McpToolNamesFilter
  *
- * @phpstan-type RemoteMcpToolType array{type: 'mcp', server_label: string, server_url: string, require_approval: 'never'|'always'|array<'never'|'always', McpToolNamesFilterType>|null, allowed_tools: array<int, string>|McpToolNamesFilterType|null, headers: array<string, string>|null}
+ * @phpstan-type RemoteMcpToolType array{type: 'mcp', server_label: string, authorization: string|null, connector_id?: string|null, server_url: string|null, require_approval: 'never'|'always'|array<'never'|'always', McpToolNamesFilterType>|null, allowed_tools: array<int, string>|McpToolNamesFilterType|null, headers: array<string, string>|null, server_description?: string|null}
  *
  * @implements ResponseContract<RemoteMcpToolType>
  */
@@ -33,10 +33,13 @@ final class RemoteMcpTool implements ResponseContract
     private function __construct(
         public readonly string $type,
         public readonly string $serverLabel,
-        public readonly string $serverUrl,
+        public readonly ?string $serverUrl = null,
         public readonly string|array|null $requireApproval = null,
         public readonly array|McpToolNamesFilter|null $allowedTools = null,
         public readonly ?array $headers = null,
+        public readonly ?string $connectorId = null,
+        public readonly ?string $authorization = null,
+        public readonly ?string $serverDescription = null,
     ) {}
 
     /**
@@ -59,10 +62,13 @@ public static function from(array $attributes): self
         return new self(
             type: $attributes['type'],
             serverLabel: $attributes['server_label'],
-            serverUrl: $attributes['server_url'],
+            serverUrl: $attributes['server_url'] ?? null,
             requireApproval: $requireApproval,
             allowedTools: $allowedTools, // @phpstan-ignore-line
             headers: $attributes['headers'] ?? null,
+            connectorId: $attributes['connector_id'] ?? null,
+            authorization: $attributes['authorization'] ?? null,
+            serverDescription: $attributes['server_description'] ?? null,
         );
     }
 
@@ -90,6 +96,9 @@ public function toArray(): array
             'require_approval' => $requireApproval,
             'allowed_tools' => $allowedTools,
             'headers' => $this->headers,
+            'connector_id' => $this->connectorId,
+            'authorization' => $this->authorization,
+            'server_description' => $this->serverDescription,
         ];
     }
 }