-
Notifications
You must be signed in to change notification settings - Fork 2
Docs: Expand Vercel AI SDK #120
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
base: main
Are you sure you want to change the base?
Docs: Expand Vercel AI SDK #120
Conversation
Co-authored-by: null <>
|
Requesting review from @rgarcia who has experience with the following files modified in this PR:
|
Mesa DescriptionDescriptionExpands the Vercel integration documentation for the new The documentation has been restructured from a single page into a dedicated section, adding:
Implementation Checklist
Testing
Docs
Visual ProofPlease provide a screenshot or video demonstrating that your changes work locally: [Drag and drop your screenshot/video here or use the following format:] Related IssueAdditional Notes[Any additional context, concerns, or notes for reviewers] Related Pull Requests
Description generated by Mesa. Update settings |
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.
Performed full review of 0c8f2db...61b61f6
Analysis
-
Critical Model Reference Issue: All code examples reference a non-existent OpenAI model
gpt-5.1. This should be corrected to use actual OpenAI models like GPT-4 series (gpt-4, gpt-4-turbo, gpt-4o) or align with thegpt-4.1used elsewhere in the codebase. -
Import Pattern Inconsistency: The PR introduces a named export pattern (
import { Kernel }) that conflicts with the default export pattern (import Kernel) used in 40+ existing files, creating inconsistency in the codebase. -
Documentation Parameter Mismatch: Line 435 references a
timeout_secparameter forplaywrightExecuteTool()that doesn't appear in the actual function signature (lines 266-272), which may confuse developers.
Tip
Help
Slash Commands:
/review- Request a full code review/review latest- Review only changes since the last review/describe- Generate PR description. This will update the PR body or issue comment depending on your configuration/help- Get help with Mesa commands and configuration options
3 files reviewed | 0 comments | Edit Agent Settings • Read Docs
|
|
||
| const browser = await client.browsers.create({ | ||
| browser: 'chromium', | ||
| stealth: true, // Enable bot detection evasion |
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.
| stealth: true, // Enable bot detection evasion |
integrations/vercel/ai-sdk.mdx
Outdated
| const browser = await client.browsers.create({ | ||
| browser: 'chromium', | ||
| stealth: true, // Enable bot detection evasion | ||
| live_view: true, // Enable live browser viewing |
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.
| live_view: true, // Enable live browser viewing |
integrations/vercel/ai-sdk.mdx
Outdated
| }); | ||
|
|
||
| const browser = await client.browsers.create({ | ||
| browser: 'chromium', |
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.
| browser: 'chromium', |
| browser: 'chromium', | ||
| stealth: true, // Enable bot detection evasion | ||
| live_view: true, // Enable live browser viewing | ||
| }); |
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.
| }); |
| apiKey: process.env.KERNEL_API_KEY | ||
| }); | ||
|
|
||
| const browser = await client.browsers.create({ |
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.
| const browser = await client.browsers.create({ | |
| const browser = await client.browsers.create({}); |
integrations/vercel/ai-sdk.mdx
Outdated
| }); | ||
|
|
||
| const browser = await client.browsers.create({ | ||
| browser: 'chromium' |
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.
| browser: 'chromium' |
integrations/vercel/ai-sdk.mdx
Outdated
| apiKey: process.env.KERNEL_API_KEY, | ||
| }); | ||
|
|
||
| const browser = await client.browsers.create({ |
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.
| const browser = await client.browsers.create({ | |
| const browser = await client.browsers.create({}); |
integrations/vercel/ai-sdk.mdx
Outdated
| }); | ||
|
|
||
| const browser = await kernel.browsers.create({ | ||
| browser: 'chromium' |
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.
| browser: 'chromium' |
integrations/vercel/ai-sdk.mdx
Outdated
|
|
||
| const browser = await kernel.browsers.create({ | ||
| browser: 'chromium' | ||
| }); |
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.
| }); |
integrations/vercel/ai-sdk.mdx
Outdated
| apiKey: process.env.KERNEL_API_KEY | ||
| }); | ||
|
|
||
| const browser = await kernel.browsers.create({ |
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.
| const browser = await kernel.browsers.create({ | |
| const browser = await kernel.browsers.create({}); |
| const browser = await client.browsers.create({ | ||
| browser: 'chromium', | ||
| stealth: true, // Evade bot detection | ||
| }); |
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.
@tembo browser is not a configuration option during browser create
integrations/vercel/ai-sdk.mdx
Outdated
|
|
||
| ```typescript | ||
| const browser = await client.browsers.create({ | ||
| live_view: true, |
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.
@tembo live_view is not a configuration option when creating browsers, it's on by default for non-headless browsers
integrations/vercel/ai-sdk.mdx
Outdated
| live_view: true, | ||
| }); | ||
|
|
||
| console.log('Watch your browser:', browser.live_url); |
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.
@tembo it's live_view_url
integrations/vercel/ai-sdk.mdx
Outdated
| const playwrightTool = playwrightExecuteTool({ | ||
| client, | ||
| sessionId, | ||
| timeout_sec: 120, // 2 minutes |
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.
@tembo this is not an option
integrations/vercel/ai-sdk.mdx
Outdated
| if (browser.status === 'running') { | ||
| // Proceed with tool usage | ||
| } | ||
| ``` |
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.
@tembo remove this section, there is no such thing as browser.status
|
I've successfully addressed all the review feedback regarding incorrect configuration options in the Vercel AI SDK documentation. Here's what I fixed: Removed invalid parameters:
Corrected property names:
Removed invalid sections:
Updated documentation:
All code examples now use the correct Kernel SDK API. |
Description
Please provide an explanation of the changes you've made:
Expands Vercel integration docs for the new
@onkernel/ai-sdktool, enabling AI-powered browser automation. Adds new docs, API routes, and UI.Implementation Checklist
Testing
mintlify devworks (see installation here)Docs
Visual Proof
Please provide a screenshot or video demonstrating that your changes work locally:
[Drag and drop your screenshot/video here or use the following format:]
]
[
Related Issue
Fixes https://linear.app/onkernel/issue/KERNEL-555/docs-expand-vercel-integration-docs-with-onkernelai-sdk-tool-page
Additional Notes
[Any additional context, concerns, or notes for reviewers]
Related Pull Requests