Conversation
|
Hey! Your PR title Please update it to start with one of:
Where See CONTRIBUTING.md for details. |
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
eee4be4 to
5e29c0a
Compare
f2a5f45 to
444f174
Compare
444f174 to
17707ea
Compare
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
|
This contribution is still actively maintained and remains the prerequisite for pingdotgg/t3code#10726. I rebased it onto the current Current validation passes: 47 focused tests, the repository pre-push typecheck across all 30 participating packages, the four directly affected package typechecks, targeted oxlint with 0 errors, Prettier, and The closure was the automated age/reaction cleanup rather than a code or review failure. Could a maintainer please reopen the PR for review? |
|
Replacement PR: #49317. The implementation was rebased onto current |
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.
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, andpackages/opencodegit diff --checkScreenshots / recordings
Not applicable; this is session metadata only.
Checklist