Conversation
Signed-off-by: Tyler Slaton <tyler@copilotkit.ai>
tylerslaton
had a problem deploying
to
tyler/add-a2ui - app PR #13
Render
Failure
tylerslaton
left a comment
tylerslaton
left a comment
There was a problem hiding this comment.
@ranst91 some comments for you as you take a look into this!
| // new MCPAppsMiddleware({ | ||
| // mcpServers: [ | ||
| // { | ||
| // type: "http", | ||
| // url: | ||
| // process.env.MCP_SERVER_URL || "https://mcp.excalidraw.com", | ||
| // serverId: "example_mcp_app", | ||
| // }, | ||
| // ], | ||
| // }), |
There was a problem hiding this comment.
With the fix to multiple middlewares this will allow us to uncomment this out.
There was a problem hiding this comment.
Should also use the new runtime properties for this instead of the middlewares at all.
There was a problem hiding this comment.
I want this to a bit more built-in. Like, we could include a default theme in the renderer that we let the user replace - something like that.
| "@copilotkit/a2ui-renderer": "^1.52.1", | ||
| "@copilotkit/react-core": "1.52.1", | ||
| "@copilotkit/react-ui": "1.52.1", | ||
| "@copilotkit/runtime": "1.52.1", |
There was a problem hiding this comment.
Likely needs updating to w/e version fixes the other issues
| import { createA2UIMessageRenderer } from "@copilotkit/a2ui-renderer"; | ||
| import { theme } from "@/lib/a2ui-theme.css"; | ||
|
|
||
| const A2UIMessageRenderer = createA2UIMessageRenderer({ theme }); | ||
| const activityRenderers = [A2UIMessageRenderer]; |
There was a problem hiding this comment.
Ideally we need none of this code and it is instead hot-swappable. The default should be A2UI "just works" if your agent returns it. The middleware takes care of most of this, if we add that into runtime it'll just work by default.
The other end is the UI, if we make this the default (with a disable) then things become quite simple.
1 participant
No description provided.