Fix: Clarify HTTP client variable in factory example

[rema424]

Summary

• Fixed undefined $client variable reference in README factory example
• Explicitly defined the client variable for better clarity

Description

The README example for configuring the Gemini client factory contained a reference to an undefined $client variable within the withStreamHandler method. This change explicitly defines the $client variable when configuring withHttpClient to make the example clearer and prevent potential confusion for users.

Changes

• Added explicit variable assignment: $client = new \GuzzleHttp\Client([]) in the withHttpClient method call

Test plan

• Verified the example code is syntactically correct
• Confirmed the change improves documentation clarity

This is a documentation-only change that improves the code example's clarity.

[Plytas]

Thanks! But maybe we could use a different name for the variable like $guzzleClient as $client is already being defined above?

[Plytas]

Also to note. You might not need the withHttpClient() call if it's the same as default

client/src/Factory.php

Lines 146 to 148 in 84ffe3d

	if ($client instanceof GuzzleClient) {
	return fn (RequestInterface $request): ResponseInterface => $client->send($request, [RequestOptions::STREAM => true]);
	}

[rema424]

@Plytas I've improved the example by renaming the variable to $guzzleClient for clarity and adding a timeout configuration. What do you think?

    ->withHttpClient($guzzleClient = new \GuzzleHttp\Client(['timeout' => 30]))  // default: HTTP client found using PSR-18 HTTP Client Discovery
    ->withStreamHandler(fn(RequestInterface $request): ResponseInterface => $guzzleClient->send($request, [
        'stream' => true // Allows to provide a custom stream handler for the http client.
    ]))

[Plytas]

Awesome, thanks!