OpenAICUAClient.ts and AnthropicCUAClient.ts pass the whole clientOptions to the LLM constructor

[majkiw]

why

#643

what changed

OpenAICUAClient now respects baseURL passed in the options

[changeset-bot]

🦋 Changeset detected

Latest commit: 4fdfea8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@browserbasehq/stagehand	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[greptile-apps]

PR Summary

The changes update the OpenAICUAClient to read and apply the baseURL from clientOptions, aligning its behavior with AnthropicCUAClient.

• In /lib/agent/OpenAICUAClient.ts, baseURL is extracted from clientOptions if provided.
• The baseURL is injected into the clientOptions before initializing the OpenAI client.
• This adjustment ensures consistency with AnthropicCUAClient for comprehensive client configuration.

_{💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!}

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[kamath]

Thanks so much for opening this PR! Ideally this would take the entire clientOptions type -- happy to take that on unless you want to add that to this PR

[majkiw]

Thanks @kamath!
I thought I'd add that, too, but I ran into some backward compatibility questions.
Probably better if I just pass it to you then.

As I mentioned in the issue, ideally, we would be able to pass the whole llmClient to support custom instrumentation for better observability (just like for the Stagehand constructor).

But for now, to unblock our pilot implementation, just that baseURL would be very helpful 🙏

[majkiw]

This probably didn't work as it was setting baseUrl instead of baseURL