[Dineshsuriya D (droideronline)]

Motivation & Context

FoundryAgent accepts a timeout, but FoundryChatClient and RawFoundryChatClient do not, so there is no supported way to set an HTTP timeout on them. Callers have to reassign client.client after construction.

Description & Review Guide

• What are the major changes?
  • Add timeout: float | None = None to RawFoundryChatClient.__init__ and FoundryChatClient.__init__.
  • When set, apply it with openai_client.with_options(timeout=timeout) on the client returned by project_client.get_openai_client(), the same approach FoundryAgent uses.
  • Add unit tests for both the timeout and no-timeout paths.
• What is the impact of these changes? None unless timeout is passed. The default keeps the OpenAI SDK behavior.
• What do you want reviewers to focus on? Whether applying the timeout via with_options after get_openai_client() is the right place, consistent with FoundryAgent.

Related Issue

Fixes #9698

AI Assistance

• No material AI assistance was used.
• This is an AI-assisted contribution. I reviewed, understood, and verified all submitted content and accept responsibility for it.

AI assistance details: implementation, tests, and this PR description.

Contribution Checklist

• The code builds clean without any errors or warnings
• All unit tests pass, and I have added new tests where possible
• The PR follows the Contribution Guidelines
• This PR links to an agreed issue with no competing open PR, or the Related Issue section documents a trivial-change or repository-automation exception.
• This is not a breaking change. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[Eduard van Valkenburg (eavanvalkenburg)]

/review

[github-actions]

MAF Automated Review — Iteration 1

Result: No findings
Scope: full PR (1 commit(s)): 814361287216
Model: gpt-5.6-sol

Overview

The PR adds an optional keyword-only timeout to both Foundry chat-client constructors and applies it by cloning the OpenAI client with with_options, matching the existing FoundryAgent contract. The default path preserves the original client, and focused tests cover both configured and omitted timeout behavior. No publishable correctness, compatibility, architecture, or security issue was established.

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
No publishable findings remained after source verification for this scope.