-
Notifications
You must be signed in to change notification settings - Fork 3
Docs: Add Notte Integration #91
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
Docs: Add Notte Integration #91
Conversation
Co-authored-by: null <>
|
Requesting review from @juecd who has experience with the following files modified in this PR:
|
Mesa DescriptionDescriptionAdds a new integration guide for using the Notte AI agent framework with Kernel's cloud browsers.
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 IssueFixes [Github issue link] [If this corresponds to a fix from another Kernel OSS repo, include this:] Fixes [Link to other repo] [Replace with actual issue link, e.g., Fixes https://github.com/username/repo/issues/123] Additional Notes[Any additional context, concerns, or notes for reviewers] 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 3e1d7f5...83df1c3
Analysis
-
The PR focuses on documentation only with no actual implementation code, which may make it difficult to verify if the documented integration patterns actually work as described.
-
While leveraging CDP (Chrome DevTools Protocol) provides integration capabilities, there's no mention of potential performance impacts or limitations when connecting Notte AI with Kernel's cloud browsers.
-
The documentation may need additional sections on troubleshooting common integration issues, as the current focus appears to be on the happy path implementation.
-
There's no indication of version compatibility requirements or potential future API changes that could impact the integration's stability.
Tip
⚡ Quick Actions
This review was generated by Mesa.
Actions:
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 | Review on Mesa | Edit Reviewer Settings
|
@tembo can you do the following:
from kernel import Kernel Load environment variables from .env fileload_dotenv() def main(): if name == "main":
|
Thanks for the requested changes! I've updated the Notte integration guide with your complete example, changed the installation command to use |
|
@tembo for the existing script conversion steps only, can you simplify and remove some of the actual logic to get the API keys (just write placeholders as you original did) + remove the try except logic. Try to closely mimic the shortness here: https://docs.notte.cc/features/sessions/cdp#use-external-session-provider-via-cdp using only the required pieces to get a Kernel browser up and running with Notte agent |
I've simplified the Notte integration script as requested, removing the error handling and API key logic in favor of placeholder values. The changes closely follow the minimalist approach of the Notte docs example, focusing only on the essential steps needed to get a Kernel browser working with the Notte agent. I've kept the code straightforward and brief throughout all sections. |
|
@tembo please remove the written benefit of headless browsers as we aren't using that |
Removed the "Headless execution" benefit line from the document as requested. The benefits section now focuses only on cloud infrastructure, scalability, and other relevant Kernel advantages without mentioning headless browsers. |
|
@tembo and remove the "notte-sdk" mention in the required SDKs, so just this: "uv pip install kernel" |
Thanks @dprevoznik - I've removed the "notte-sdk" mention in the installation command as requested, so it now only shows "uv pip install kernel". The Notte SDK would be installed separately by users according to their own requirements. |
Small edits
Remove headless param and use headful default
Description
Adds Notte integration to the Integration Guides section, detailing how to use Kernel's cloud browsers with Notte via CDP.
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 [Github issue link]
[If this corresponds to a fix from another Kernel OSS repo, include this:]
Fixes [Link to other repo]
[Replace with actual issue link, e.g., Fixes https://github.com/username/repo/issues/123]
Additional Notes
[Any additional context, concerns, or notes for reviewers]