Skip to content

fix(google): omit Vertex function call ids#17030

Open
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:iroiro147/ai/vertex-tool-id-15891
Open

fix(google): omit Vertex function call ids#17030
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:iroiro147/ai/vertex-tool-id-15891

Conversation

@iroiro147

Copy link
Copy Markdown

Summary

  • omit ordinary Gemini functionCall.id and functionResponse.id fields when converting messages for Vertex-like providers
  • keep the existing tool call/result IDs for the Google Gemini provider path
  • add regression coverage for Vertex request conversion and Google-provider preservation

Fixes #15891.

Tests

  • pnpm --filter @ai-sdk/google exec vitest run --config vitest.node.config.js src/convert-to-google-messages.test.ts (57 passed)
  • pnpm --filter @ai-sdk/google test:node (691 passed)
  • pnpm --filter @ai-sdk/google test:edge (691 passed)
  • pnpm --filter @ai-sdk/google type-check
  • pnpm exec oxfmt --check packages/google/src/convert-to-google-messages.ts packages/google/src/convert-to-google-messages.test.ts .changeset/vertex-tool-call-ids.md
  • pnpm exec oxlint packages/google/src/convert-to-google-messages.ts packages/google/src/convert-to-google-messages.test.ts
  • git diff --check
  • pnpm changeset status --since origin/main

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@ai-sdk/google-vertex — Tool call/result messages include id field rejected by Gemini API (400 on function_call / function_response)

1 participant