Conversation
| @@ -0,0 +1,5 @@ | |||
| --- | |||
| '@ai-sdk/provider-utils': major | |||
There was a problem hiding this comment.
I used major since it's removing an API, but the pull request template calls explicitly for a patch type. Let me know what your preference is
There was a problem hiding this comment.
on v5 it does not matter, the release will be major regardless.
There was a problem hiding this comment.
Yes that's clear. But I think it might help with the release notes / upgrade guide when we flag breaking changes explicitly?
| ToolCall, | ||
| ToolCallUnion, | ||
| } from './tool-call'; | ||
| export type { CoreToolCallUnion, ToolCall, ToolCallUnion } from './tool-call'; |
There was a problem hiding this comment.
or was that in the other pr?
There was a problem hiding this comment.
yes that was removed in https://github.com/vercel/ai/pull/5768/files#diff-97e4c4e0b8b41041865409d1692cc8374efa62a36b8a50e736e799fb46829255L26, I'll merge v5 into this branch as it caused conflicts. I tried to split up the changes by packages for easier reviews
CoreToolCall and CoreToolResult types
Background
Towards #5765
Summary
Removes deprecated
CoreToolCallandCoreToolResulttypesTasks
Tests for the changes have been added (for bug fixes / features)Docs have been added / updated (for bug fixes / features)pnpm prettier-fixto fix any formatting issues