mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
fix(mcp-apps-renderer): address review on the React-pilot PR
Bridge-free /activity (review blocker): the `./activity` entry pulled ext-apps' LATEST_PROTOCOL_VERSION through constants.ts, statically dragging the App SDK + zod/v4 (~+55 kB gzip) into react-core's eager chunk. Move MCP_APPS_PROTOCOL_VERSION to the bridge side (session.ts), sourced from the /app-bridge subpath the session already imports - single source of truth, no hand-maintained literal, zero extra cost. dist/activity.mjs and dist/constants.mjs now carry no @modelcontextprotocol edge. Keep react-core's zod peer at >=3.25 (review blocker): the floor moved down to the package, it did not go away; ext-apps + the MCP SDK both hard-require it, so >=3.0.0 is an ERESOLVE install failure once an MCP App renders. Drop the <copilotkit-mcp-app> web component from this PR: nothing consumes it here (the React adapter builds its own iframe) and it carried two open defects. It lands with the Vue/Angular adapters, its first real consumers. Removes its files, exports, tsdown entry, and the now-unused lit dependency. Also: add content.serverId to the bind effect deps; delete drained thread entries from the request queue's maps (bounded growth); register the package in static_compat.yml (+ a compat-check script) and static_bundle_size.yml; publint repository.url -> git+https. Manifest regenerated.
This commit is contained in:
@@ -1072,46 +1072,6 @@
|
||||
"path": "packages/mcp-apps-renderer/package.json",
|
||||
"selector": "exports[\"./package.json\"]"
|
||||
}
|
||||
},
|
||||
{
|
||||
"importPath": "@copilotkit/mcp-apps-renderer/web-components",
|
||||
"exportKey": "./web-components",
|
||||
"kind": "code",
|
||||
"conditions": {
|
||||
"import": {
|
||||
"types": "./dist/web-components/index.d.mts",
|
||||
"default": "./dist/web-components/index.mjs"
|
||||
},
|
||||
"require": {
|
||||
"types": "./dist/web-components/index.d.cts",
|
||||
"default": "./dist/web-components/index.cjs"
|
||||
}
|
||||
},
|
||||
"provenance": {
|
||||
"kind": "package-json",
|
||||
"path": "packages/mcp-apps-renderer/package.json",
|
||||
"selector": "exports[\"./web-components\"]"
|
||||
}
|
||||
},
|
||||
{
|
||||
"importPath": "@copilotkit/mcp-apps-renderer/web-components/define",
|
||||
"exportKey": "./web-components/define",
|
||||
"kind": "code",
|
||||
"conditions": {
|
||||
"import": {
|
||||
"types": "./dist/web-components/define.d.mts",
|
||||
"default": "./dist/web-components/define.mjs"
|
||||
},
|
||||
"require": {
|
||||
"types": "./dist/web-components/define.d.cts",
|
||||
"default": "./dist/web-components/define.cjs"
|
||||
}
|
||||
},
|
||||
"provenance": {
|
||||
"kind": "package-json",
|
||||
"path": "packages/mcp-apps-renderer/package.json",
|
||||
"selector": "exports[\"./web-components/define\"]"
|
||||
}
|
||||
}
|
||||
],
|
||||
"compatibility": {
|
||||
@@ -1233,7 +1193,7 @@
|
||||
"peerDependencies": {
|
||||
"react": "^18 || ^19 || ^19.0.0-rc",
|
||||
"react-dom": "^18 || ^19 || ^19.0.0-rc",
|
||||
"zod": ">=3.0.0"
|
||||
"zod": ">=3.25"
|
||||
},
|
||||
"provenance": [
|
||||
{
|
||||
|
||||
Reference in New Issue
Block a user