Conversation
|
The following comment was made by an LLM, it may be inaccurate: The searches reveal one potentially related PR that may have addressed similar concerns: Related PR:
The current PR #49317 is a replacement for #42433 (which was closed by automated cleanup) and specifically addresses issue #42420. The search results confirm this is the only open PR with this exact title and related keywords, so there are no direct duplicate PRs in the system. |
|
Thanks for flagging #45008. There is substantial overlap, but the contracts differ: #45008 adds a broader |
3def058 to
23d3d24
Compare
|
Rebased onto current |
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
|
This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window. Feel free to open a new pull request that follows our guidelines. |
|
GitHub would not reopen this automation-closed PR. The same validated head now continues in #49571 with the correctly formatted compliance template. |
Replaces #42433, which was closed by the repository's automated age/reaction cleanup rather than a code or review failure.
Issue for this PR
Closes #42420
Related to #26091, but intentionally narrower: this keeps the AI SDK's structured model ID instead of arbitrary response headers.
This is also the upstream prerequisite for pingdotgg/t3code#10726.
Type of change
What does this PR do?
OpenCode currently discards
response.modelIdwhen adapting AI SDK stream events. This carries that value on the correspondingstep-finishpart and exposes it as optionalmodelIDin the session schema and generated SDK type. Response headers are not retained.How did you verify your code works?
packages/llm,packages/schema,packages/sdk, andpackages/opencodegit diff --checkThe replacement branch is rebased onto current
devat23d3d240344abcf9bc5eb72ab55eeaa6c5da6e8e.Screenshots / recordings
Not applicable; this is session metadata only.
Checklist