-
Notifications
You must be signed in to change notification settings - Fork 914
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.
| }: { | ||
| path: string; | ||
| resourceId: string; | ||
| resourceId?: string; |
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 mentioned this in Slack before, I think we can make this optional here because down the chain it's typed like that, too
| threadId: input.threadId, | ||
| memoryConfig: { | ||
| workingMemory: { | ||
| scope: "thread", |
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.
| }, | ||
| }, | ||
| { | ||
| // To use this, run "pnpm mastra:dev" in a separate terminal window |
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 don't believe this is valid. I'm pretty sure the mastra:dev script would run the mastra-agent-local agent listed below, but I'm not entirely sure since that's whats bundled into the dojo (and not listed in the exports). This script entry's existence may be entirely invalid.
Regardless, the agent with id mastra is actually located elsewhere in the repo, in integrations/mastra/typescript/examples
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.
As I understand it the local agent one imports the mastra class, the remote one initializes a MastraClient which needs an URL. I got it working like that so that’s why I added the comment
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.