fix(types): include web_search_call.action.sources in ResponseIncludable #1649
+96
−11
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes being requested
Adds missing type support for the documented include value to resolve TypeScript compile errors without affecting runtime behavior.
Core changes:
Test coverage:
Additional context & links
Bug: #1645
This addresses TypeScript errors when using the documented include 'web_search_call.action.sources' in the Responses API.
Problem:
The OpenAPI description listed the value, but the enum omitted it. The SDK’s generated types mirror the enum, so the literal was missing and caused the TS compile
failures.
Solution:
Include the missing literal in ResponseIncludable and add focused tests to prevent regressions. This is a minimal, type-only change with no runtime impact.
Impact:
✅ I understand that this repository is auto-generated, and my pull request may not be merged