Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions javascript/examples/vitest/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,11 @@
"vitest-mock-extended": "5.1.1"
},
"dependencies": {
"@ai-sdk/anthropic": "3.0.41",
"@ai-sdk/xai": "3.0.50",
"@ai-sdk/anthropic": "4.0.71",
"@ai-sdk/xai": "5.0.14",
"@langwatch/scenario": "workspace:*",
"@openai/agents": "^0.16.0",
"ai": "^6.0.0",
"ai": "^7.0.127",
"express": "^5.1.0",
"openai": "^6.9.1",
"vite": "^8.2.1",
Expand Down
8 changes: 4 additions & 4 deletions javascript/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,14 +56,14 @@
"packageManager": "pnpm@10.22.0",
"dependencies": {
"@ag-ui/core": "^0.0.58",
"@ai-sdk/openai": "^3.0.26",
"@ai-sdk/openai": "^4.0.83",
"@openai/agents": "^0.16.0",
"@opentelemetry/api": "^1.9.0",
"@opentelemetry/context-async-hooks": "2.10.0",
"@opentelemetry/sdk-node": "0.221.0",
"@opentelemetry/sdk-trace-base": "2.10.0",
"@opentelemetry/sdk-trace-node": "2.10.0",
"ai": "^6.0.0",
"ai": "^7.0.127",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 · Migrate the example contracts with AI SDK 7

  • What you're looking at. The root and example lock resolutions move to AI SDK 7 without migrating the Vitest example callers.

  • Why it's scary. Current CI fails in llm-provider-mocking.test.ts because GenerateTextResult now needs three type parameters, and in weather-agent.test.ts and tool-call-role-reversal.test.ts because ToolExecutionOptions now requires context. Main still has these callers. Update the examples and their tests in the same dependency migration; the required JavaScript check currently stops at typecheck. Evidence: https://github.com/langwatch/scenario/actions/runs/37373631308/job/111976898611

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 · Complete the AI SDK 7 caller migration

  • What you're looking at. The new head still upgrades ai to 7 without updating the example callers identified in the previous review. Main also retains those callers.

  • Why it's scary. Current CI typecheck fails at llm-provider-mocking.test.ts:54 because GenerateTextResult now needs three type arguments, and at tool-call-role-reversal.test.ts:45 and weather-agent.test.ts:47 because ToolExecutionOptions requires context. Migrate these examples with the dependency upgrade and rerun the required JavaScript check.

"chalk": "^5.6.2",
"ffmpeg-static": "5.3.0",
"fft.js": "4.0.4",
Expand All @@ -76,8 +76,8 @@
"zod": "^3.25.76"
},
"devDependencies": {
"@ai-sdk/amazon-bedrock": "^4.0.183",
"@ai-sdk/google": "^3.0.127",
"@ai-sdk/amazon-bedrock": "^5.0.105",
"@ai-sdk/google": "^4.0.87",
"@amiceli/vitest-cucumber": "^7.0.0",
"@elevenlabs/elevenlabs-js": "^2.51.0",
"@eslint/js": "^9.39.1",
Expand Down
Loading
Loading