Skip to content

chore(provider-utils): move ToolResultContent to provider-utils#6121

Merged
samdenty merged 2 commits intov5from
v5-provider-utils-cleanup
May 2, 2025
Merged

chore(provider-utils): move ToolResultContent to provider-utils#6121
samdenty merged 2 commits intov5from
v5-provider-utils-cleanup

Conversation

@samdenty
Copy link
Contributor

@samdenty samdenty commented May 2, 2025

Background

After a lot of cross-referencing, the only cleanup we need is to unify this interface into provider-utils

Summary

Move ToolResultContent to provider-utils

import { z } from 'zod';

// Copied from ai package
type ExecuteFunction<PARAMETERS, RESULT> =
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is ExecuteFunction not copied in some fashion?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nope not anymore

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

couldn't find a reference, tried searching multiple ways

@lgrammel
Copy link
Collaborator

lgrammel commented May 2, 2025

  • needs a changeset bc things are moved crosspackage
  • summary section and cleanup of description would be helpful for review and commit history

@samdenty samdenty merged commit a571d6e into v5 May 2, 2025
8 checks passed
@samdenty samdenty deleted the v5-provider-utils-cleanup branch May 2, 2025 15:33
samdenty added a commit that referenced this pull request May 5, 2025
* origin/v5: (23 commits)
  chore (ai): remove StreamData and mergeStreams (#6150)
  chore (build): fix typecheck setup (#6151)
  chore (ai): remove IdGenerator duplication (#6147)
  chore (ai): remove content from ui messages (#6145)
  Version Packages (canary) (#6115)
  feat (provider): support reasoning tokens, cached input tokens, total token in usage information (#6140)
  test (ai): use `mock-provider` instead of `openai` to match used `MockImageModelV2` (#6132)
  fix(providers): always use optional instead of mix of nullish for providerOptions (#6122)
  chore (ai): remove sendExtraMessageFields (#6120)
  chore(provider-utils): move ToolResultContent to provider-utils (#6121)
  feat(embed-many): respect supportsParallelCalls & concurrency (#6108)
  chore (provider-utils): return IdGenerator interface (#6118)
  chore (provider): improve jsdoc comments (#6117)
  chore (ai): remove outdated typecasts (#6116)
  chore (ai): remove useChat keepLastMessageOnError (#6114)
  fix(providers/openai): logprobs for stream alongside completion model (#6091)
  chore (ai): move maxSteps into UseChatOptions (#6113)
  docs (ai): add notes on schema .optional() (#6111)
  chore: update references to use ai-sdk.dev (#6076) (#6107)
  Version Packages (canary) (#6069)
  ...
iteratetograceness pushed a commit to iteratetograceness/ai that referenced this pull request May 9, 2025
…el#6121)

## Background

After a lot of cross-referencing, the only cleanup we need is to unify
this interface into provider-utils

## Summary

Move ToolResultContent to provider-utils
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.

3 participants