-
Notifications
You must be signed in to change notification settings - Fork 931
feat!(mastra): Support 1.0.0-beta #685
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?
Conversation
| `, | ||
| model: openai("gpt-4o-mini"), | ||
| model: "openai/gpt-4o-mini", |
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.
Mastra now has its own model router so we can remove AI SDK usage
| tools: { weatherTool, browserTool }, | ||
| memory: new Memory({ | ||
| storage: new LibSQLStore({ | ||
| id: "mastra-cli-example-db", |
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.
| execute: async ({ context }) => { | ||
| open(context.url); | ||
| return `Browsed ${context.url}`; | ||
| execute: async (inputData) => { |
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.
| "@ag-ui/client": ">=0.0.40", | ||
| "@copilotkit/runtime": "^1.10.5", | ||
| "@mastra/core": ">=0.20.1", | ||
| "zod": "^3.25.67" |
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.
zod is a peerDep of @mastra/core so not required to place it here, too
| ExperimentalEmptyAdapter, | ||
| } from "@copilotkit/runtime"; | ||
| import { RuntimeContext } from "@mastra/core/runtime-context"; | ||
| import { RequestContext } from "@mastra/core/request-context"; |
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 memory = await this.agent.getMemory(); | ||
| if (memory) { | ||
| const workingMemory = await memory.getWorkingMemory({ | ||
| resourceId: this.resourceId, |
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.
Everything is 'resource' scoped now, so we need to pass it here: https://mastra.ai/guides/v1/migrations/upgrade-to-v1/memory#default-memory-scope-from-thread-to-resource
That's why I made it required everywhere
|
@tylerslaton Should be all fixed 👍 |
|
@LekoArts we published a beta |
|
Thanks! I'll give it a try tomorrow 👍 |
Description
At Mastra we're working towards a v1.0.0 release and recently we released a beta version of that. Users can now install
@betaversions of Mastra packages and try out the new changes. This version bumps also comes with breaking changes which also affect third-party packages like@ag-ui/mastra.In our shared Slack channel I communicated with @tylerslaton so far about this and I'm open to handling however you prefer. Here's what I think we should do:
@latest)@ag-ui/mastra, tooIn this PR I've updated all examples, too, as they use the local version and would not work otherwise. It's probably easiest to create a release based off from this branch/PR and not merge the PR until Mastra v1. But if you want to do it in another way, let me know.