Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add top-level ResponsesClient factory on OpenAIClient#1143

Merged
joseharriaga merged 11 commits into
mainfrom
copilot/openai001-add-get-responses-client
May 1, 2026
Merged

Add top-level ResponsesClient factory on OpenAIClient#1143
joseharriaga merged 11 commits into
mainfrom
copilot/openai001-add-get-responses-client

Conversation

Copilot AI commented Apr 30, 2026

Copy link
Copy Markdown
Contributor
  • Inspect generation/API export scripts and repo guidance
  • Run baseline build
  • Regenerate library code with scripts/Invoke-CodeGen.ps1
  • Regenerate API listings with scripts/Export-Api.ps1
  • Review generated diffs
  • Run targeted validation
  • Run final validation

Copilot AI and others added 8 commits April 30, 2026 21:55
Copilot AI requested a review from joseharriaga April 30, 2026 22:38
@joseharriaga joseharriaga marked this pull request as ready for review April 30, 2026 22:47
Copilot AI review requested due to automatic review settings April 30, 2026 22:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a top-level ResponsesClient factory to OpenAIClient so callers can create a ResponsesClient while reusing the existing OpenAIClient pipeline/configuration (including endpoint and pipeline customizations).

Changes:

  • Add OpenAIClient.GetResponsesClient() and a private derived TopLevelResponsesClient bridge to access ResponsesClient’s protected internal pipeline constructor across assemblies.
  • Add smoke tests verifying ResponsesClient can be created from OpenAIClient and that endpoint/pipeline options persist through the factory.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
OpenAI/src/Custom/OpenAIClient.cs Adds GetResponsesClient() factory and the TopLevelResponsesClient constructor-bridge to reuse the existing pipeline.
tests/Responses/ResponsesSmokeTests.cs Adds smoke tests covering factory creation and option persistence via a mock transport and pipeline policy.

Comment thread OpenAI/src/Custom/OpenAIClient.cs
Comment thread OpenAI/src/Custom/OpenAIClient.cs
Comment thread OpenAI/src/Custom/OpenAIClient.cs
Copilot AI requested review from Copilot and removed request for Copilot May 1, 2026 22:29
@joseharriaga joseharriaga merged commit fcf157c into main May 1, 2026
4 checks passed
@joseharriaga joseharriaga deleted the copilot/openai001-add-get-responses-client branch May 1, 2026 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants