fix(OpenAI) - Add index to CreateStreamedResponseToolCall

[clementmas]

What:

• Bug Fix
• New Feature

Description:

When a response is streaming tool calls, we have to know the index of the delta to recompose the JSON arguments properly.

See tool calls chunk combination code

foreach ($toolCallsDelta ?? [] as $toolCallDelta) {
    if (! isset($toolCalls[$toolCallDelta->index])) {
        // Initialize the tool call when first encountered
        $toolCalls[$toolCallDelta->index] = [
            'id' => $toolCallDelta->id,
            'type' => $toolCallDelta->type,
            'function' => [
                'name' => $toolCallDelta->function->name ?? '',
                'arguments' => $toolCallDelta->function->arguments ?? '',
            ],
        ];

        continue;
    }

    // Append to arguments for subsequent chunks
    if (isset($toolCallDelta->function->arguments)) {
        $toolCalls[$toolCallDelta->index]['function']['arguments'] .= $toolCallDelta->function->arguments;
    }
}

I don't see any other way to do this except not use any streaming which doesn't provide an acceptable UX.

I'm open to suggestions if I missed something.

[iBotPeaches]

Could you make 2 changes? then good to go

1. Add the missing property to the toArray function.
2. Update a test to assert the index property can be pulled off the object.

[iBotPeaches]

src/Responses/Chat/CreateStreamedResponseToolCall.php ............. pa39 96%  

Odd that file does not report as 100% covered anymore. You may have to run the coverage report locally and see what line its complaining about.

[clementmas]

I fixed the missing type on the callback fn ($value) => $value !== null. I need it to make sure index = 0 is not removed from the array.

[iBotPeaches]

thanks!