Skip to content

Conversation

@crickman
Copy link
Contributor

@crickman crickman commented Dec 12, 2025

Expose InternalItemContentInputImage.ImageUrl as public property: ResponseContentPart.InputImageUrl

Added smoke validation all public ResponseContentPart usage and serialization

closes #875

@jsquire
Copy link
Collaborator

jsquire commented Dec 13, 2025

Hi @crickman. Thanks very much for your contribution and your interest in improving the OpenAI developer experience. As discussed in #875, I think the proposed change aligns with the local and upstream REST API specs and agree this should be public. We appreciate your thorough and comprehensive approach to test validation. The implementation looks good to me.

I'm going to ask @joseharriaga to make the final call, as he'll have a more complete understanding of the historical context.

@jsquire jsquire added the area: responses This item is related to Responses label Dec 13, 2025
@christothes
Copy link
Collaborator

Hi @crickman - Please run the Export-API script to update the public API surface file.

@crickman
Copy link
Contributor Author

@christothes

Hi @crickman - Please run the Export-API script to update the public API surface file.

I'm getting this error:

&: C:\Users\crickman\source\repos\openai-fork\scripts\Export-Api.ps1:234
Line |
 234 |      & genapi @genapiArgs
     |        ~~~~~~
     | The term 'genapi' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Is there a module I need to install (type-spec related maybe)?

@christothes
Copy link
Collaborator

@christothes

Hi @crickman - Please run the Export-API script to update the public API surface file.

I'm getting this error:

&: C:\Users\crickman\source\repos\openai-fork\scripts\Export-Api.ps1:234
Line |
 234 |      & genapi @genapiArgs
     |        ~~~~~~
     | The term 'genapi' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Is there a module I need to install (type-spec related maybe)?

I think you can grab it from here https://github.com/dotnet/sdk/tree/main/src/Compatibility/GenAPI

@crickman
Copy link
Contributor Author

@christothes - Updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: responses This item is related to Responses community-contribution Community members are working on the issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] ImageUrl not exposed on ResponseContentPart for CreateInputImagePart

4 participants