-
Notifications
You must be signed in to change notification settings - Fork 3
Magnitude Template #57
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
Magnitude Template #57
Conversation
Created a magnitude.run template for create-kernel-app and updated the base CLI to include the new template. Confirmed the template files work by running the CLI locally.
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 b0d4df2...10de38b
Analysis
Based on the provided analysis, there appear to be no significant architectural issues with this PR. The analysis only contains positive feedback about the implementation following established patterns, proper integration, and good practices. The review mentions the template is TypeScript-only, which is noted as appropriate given Magnitude.run's current ecosystem, rather than being flagged as a limitation.
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
6 files reviewed | 0 comments | Review on Mesa | Edit Reviewer 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 b0d4df2...10de38b
Analysis
-
URL validation in the Magnitude template needs to be more robust, which could lead to runtime errors or security issues when users input malformed URLs.
-
The template contains hardcoded model references that limit configurability, potentially causing integration problems as AI models evolve or when users want to use alternative models.
-
Some error messages lack specificity, which could hinder debugging and troubleshooting for developers using the template.
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
6 files reviewed | 0 comments | Review on Mesa | Edit Reviewer Settings
| // Navigate and extract via Magnitude | ||
| ////////////////////////////////////// | ||
| await agent.act( | ||
| `Go to ${target}. Explore the page by scrolling down the page twice. Narrate key steps.` |
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.
I wonder if this should be in the payload since that is most likely how people will use web agents? This could be a more common patter, not entirely sure though.
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.
I set it up this way so someone can quickly see Magnitude’s benefits without needing a specific task that highlights each feature (e.g. Extract URLs, scroll, etc.). I agree this isn’t the most common pattern or likely a workflow users would have, but it shows off the benefits well IMO and it’s easy to overwrite if needed.
Description
Created a magnitude.run template for create-kernel-app and updated the base CLI to include the new template. Confirmed the template files work by running the CLI locally.
Implementation Checklist
Testing
Docs
Visual Proof
TL;DR
Adds a new
magnitudeproject template to thecreate-kernel-appCLI tool.Why we made these changes
To provide developers with a pre-configured starting point for building applications with
magnitude.run, simplifying the setup process and showcasing a best-practice project structure.What changed?
magnitudetemplate: A new project template was added undertemplates/typescript/magnitude/, including a basicpackage.json,tsconfig.json, and a sampleindex.ts.magnitudeas a selectable option when generating a new project.Description generated by Mesa. Update settings