-
-
Notifications
You must be signed in to change notification settings - Fork 463
feat: ai-sdk v6 #894
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: ai-sdk v6 #894
Conversation
🦋 Changeset detectedLatest commit: 5408cc6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 26 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This comment has been minimized.
This comment has been minimized.
Deploying voltagent with
|
| Latest commit: |
5408cc6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://34220443.voltagent.pages.dev |
| Branch Preview URL: | https://feat-ai-sdk-v6.voltagent.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3 issues found across 156 files
Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed.
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="examples/with-planagents/package.json">
<violation number="1" location="examples/with-planagents/package.json:9">
P3: Dependency versions are inconsistent with other examples. Consider updating `@voltagent/core` to `^1.5.2` and `@voltagent/libsql` to `^1.1.0` to match other examples in the repository.</violation>
</file>
<file name="examples/with-planagents/src/index.ts">
<violation number="1" location="examples/with-planagents/src/index.ts:74">
P2: Using `void` suppresses TypeScript warnings but doesn't handle errors. If `runSummaryDemo()` fails (file not found, API errors), it will cause an unhandled promise rejection. Consider adding error handling.</violation>
</file>
<file name="examples/with-planagents/src/tools.ts">
<violation number="1" location="examples/with-planagents/src/tools.ts:37">
P2: Missing error handling for `tavilySearch.invoke()`. This API call can throw errors (network failures, rate limits, API errors) that will propagate unhandled. Wrap in try-catch for consistent error handling, similar to the pattern in `examples/with-tavily-search/src/tools.ts`.</violation>
</file>
Reply to cubic to teach it or ask questions. Tag @cubic-dev-ai to re-run a review.
| "@ai-sdk/openai": "^3.0.0", | ||
| "@langchain/tavily": "^0.1.0", | ||
| "@voltagent/cli": "^0.1.17", | ||
| "@voltagent/core": "^1.5.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P3: Dependency versions are inconsistent with other examples. Consider updating @voltagent/core to ^1.5.2 and @voltagent/libsql to ^1.1.0 to match other examples in the repository.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At examples/with-planagents/package.json, line 9:
<comment>Dependency versions are inconsistent with other examples. Consider updating `@voltagent/core` to `^1.5.2` and `@voltagent/libsql` to `^1.1.0` to match other examples in the repository.</comment>
<file context>
@@ -0,0 +1,29 @@
+ "@ai-sdk/openai": "^3.0.0",
+ "@langchain/tavily": "^0.1.0",
+ "@voltagent/cli": "^0.1.17",
+ "@voltagent/core": "^1.5.1",
+ "@voltagent/libsql": "^1.0.14",
+ "@voltagent/logger": "^1.0.4",
</file context>
| ); | ||
| }; | ||
|
|
||
| void runSummaryDemo(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: Using void suppresses TypeScript warnings but doesn't handle errors. If runSummaryDemo() fails (file not found, API errors), it will cause an unhandled promise rejection. Consider adding error handling.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At examples/with-planagents/src/index.ts, line 74:
<comment>Using `void` suppresses TypeScript warnings but doesn't handle errors. If `runSummaryDemo()` fails (file not found, API errors), it will cause an unhandled promise rejection. Consider adding error handling.</comment>
<file context>
@@ -0,0 +1,74 @@
+ );
+};
+
+void runSummaryDemo();
</file context>
| topic, | ||
| }); | ||
|
|
||
| return await tavilySearch.invoke({ query }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: Missing error handling for tavilySearch.invoke(). This API call can throw errors (network failures, rate limits, API errors) that will propagate unhandled. Wrap in try-catch for consistent error handling, similar to the pattern in examples/with-tavily-search/src/tools.ts.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At examples/with-planagents/src/tools.ts, line 37:
<comment>Missing error handling for `tavilySearch.invoke()`. This API call can throw errors (network failures, rate limits, API errors) that will propagate unhandled. Wrap in try-catch for consistent error handling, similar to the pattern in `examples/with-tavily-search/src/tools.ts`.</comment>
<file context>
@@ -0,0 +1,39 @@
+ topic,
+ });
+
+ return await tavilySearch.invoke({ query });
+ },
+});
</file context>
PR Checklist
Please check if your PR fulfills the following requirements:
Bugs / Features
What is the current behavior?
What is the new behavior?
fixes (issue)
Notes for reviewers
Summary by cubic
Upgrade VoltAgent to AI SDK v6 across core and examples. Aligns APIs, updates tests/docs, adds a PlanAgents quickstart, tool approval, and automatic summarization with better observability.
New Features
Migration
Written for commit 5408cc6. Summary will update automatically on new commits.