test: cover actions fallback and missing action validation

Agent-Logs-Url: https://github.com/openai-php/client/sessions/18ddc838-e520-4048-879d-4a1218823f7a

Co-authored-by: iBotPeaches <611784+iBotPeaches@users.noreply.github.com>

Author: Copilot

src/Responses/Responses/Output/OutputComputerToolCall.php

@@ -63,7 +63,16 @@ private function __construct(
      */
     public static function from(array $attributes): self
     {
-        $actionAttributes = $attributes['action'] ?? ($attributes['actions'][0] ?? []);
+        $actionAttributes = [];
+        if (isset($attributes['action'])) {
+            $actionAttributes = $attributes['action'];
+        } elseif (isset($attributes['actions']) && isset($attributes['actions'][0])) {
+            $actionAttributes = $attributes['actions'][0];
+        }
+
+        if (! isset($actionAttributes['type'])) {
+            throw new \InvalidArgumentException('Missing required computer action payload.');
+        }
 
         $action = match ($actionAttributes['type']) {
             'click' => Click::from($actionAttributes),

tests/Responses/Responses/Output/OutputComputerToolCall.php

@@ -54,3 +54,45 @@
         ])
         ->not->toHaveKey('actions');
 });
+
+test('from with multiple actions uses the first action', function () {
+    $payload = outputComputerToolCall();
+    unset($payload['action']);
+    $payload['actions'] = [
+        ['type' => 'screenshot'],
+        ['type' => 'wait'],
+    ];
+
+    $response = OutputComputerToolCall::from($payload);
+
+    expect($response->action)
+        ->toBeInstanceOf(OutputComputerActionScreenshot::class);
+});
+
+test('from without action payload throws exception', function () {
+    $payload = outputComputerToolCall();
+    unset($payload['action']);
+    $payload['actions'] = [];
+
+    expect(fn (): OutputComputerToolCall => OutputComputerToolCall::from($payload))
+        ->toThrow(InvalidArgumentException::class, 'Missing required computer action payload.');
+});
+
+test('from without action and actions keys throws exception', function () {
+    $payload = outputComputerToolCall();
+    unset($payload['action'], $payload['actions']);
+
+    expect(fn (): OutputComputerToolCall => OutputComputerToolCall::from($payload))
+        ->toThrow(InvalidArgumentException::class, 'Missing required computer action payload.');
+});
+
+test('from with malformed actions payload throws exception', function () {
+    $payload = outputComputerToolCall();
+    unset($payload['action']);
+    $payload['actions'] = [
+        ['x' => 1],
+    ];
+
+    expect(fn (): OutputComputerToolCall => OutputComputerToolCall::from($payload))
+        ->toThrow(InvalidArgumentException::class, 'Missing required computer action payload.');
+});