Skip to content

fix(core): preserve AI SDK provider metadata#35812

Closed
kitlangton wants to merge 3 commits into
v2from
llm-reproducers
Closed

fix(core): preserve AI SDK provider metadata#35812
kitlangton wants to merge 3 commits into
v2from
llm-reproducers

Conversation

@kitlangton

@kitlangton kitlangton commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • preserve content-level provider metadata when lowering canonical LLM messages into AI SDK prompts
  • cover reasoning and tool-call replay metadata in the compatibility adapter

This keeps provider-specific continuation data such as thought signatures and provider item IDs available during history replay.

Testing

  • bun run test test/aisdk.test.ts from packages/core
  • bun typecheck from packages/core
  • pre-push workspace typecheck: 31 successful, 31 total

@kitlangton kitlangton changed the title fix(core): preserve provider adapter metadata fix(core): preserve AI SDK provider metadata Jul 8, 2026
@opencode-agent

opencode-agent Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Audit note: #35817 overlaps this provider-metadata preservation path and is broader (namespace preservation across SessionRunner collection/replay plus AI SDK lowering). Please clarify whether #35817 builds on this PR or supersedes it before either merges.

@kitlangton

Copy link
Copy Markdown
Contributor Author

Superseded by #35817, which is already merged into and preserves the provider metadata namespace while covering the same replay path. Rebasing this branch onto current leaves no diff. The focused AI SDK test and Core typecheck pass on the merged implementation.

@kitlangton kitlangton closed this Jul 8, 2026
@kitlangton
kitlangton deleted the llm-reproducers branch July 8, 2026 14:31
@kitlangton

Copy link
Copy Markdown
Contributor Author

Correction: superseded by #35817, which is already merged into v2 and preserves the provider metadata namespace while covering the same replay path. Rebasing this branch onto current v2 leaves no diff. The focused AI SDK test and Core typecheck pass on the merged implementation.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant