Skip to content

Commit fb7f12b

Browse files
committed
chore: remove dead ./react-with-deps export (copy-paste bug from #221)
The react-with-deps build entry was introduced in PR #221 (e078250) as a sibling to app-with-deps for CDN/unpkg no-bundler use. But its external array was duplicated without editing — it never dropped @modelcontextprotocol/sdk from externals, so it has shipped byte-identical to ./react for 10 releases (v0.3.1 → v1.2.0). After #534 merged, both entries use ['react', 'react-dom', ...PEER_EXTERNALS] — still identical, just smaller. Delete rather than fix: React apps always use a bundler; the CDN/unpkg use case (HTML-string apps like qr-server) doesn't apply to React hooks. Zero consumers verified (examples/docs/tests/plugins).
1 parent bf623f8 commit fb7f12b

2 files changed

Lines changed: 0 additions & 9 deletions

File tree

build.bun.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,6 @@ await Promise.all([
5252
outdir: "dist/src/react",
5353
external: ["react", "react-dom", ...PEER_EXTERNALS],
5454
}),
55-
buildJs("src/react/index.tsx", {
56-
outdir: "dist/src/react",
57-
external: ["react", "react-dom", ...PEER_EXTERNALS],
58-
naming: { entry: "react-with-deps.js" },
59-
}),
6055
buildJs("src/server/index.ts", {
6156
outdir: "dist/src/server",
6257
external: PEER_EXTERNALS,

package.json

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,6 @@
2626
"types": "./dist/src/react/index.d.ts",
2727
"default": "./dist/src/react/index.js"
2828
},
29-
"./react-with-deps": {
30-
"types": "./dist/src/react/index.d.ts",
31-
"default": "./dist/src/react/react-with-deps.js"
32-
},
3329
"./app-bridge": {
3430
"types": "./dist/src/app-bridge.d.ts",
3531
"default": "./dist/src/app-bridge.js"

0 commit comments

Comments
 (0)