From 349ea26d8b42198710040f82592f6ba63faa5433 Mon Sep 17 00:00:00 2001 From: "dayuan.jiang" Date: Sun, 11 Oct 2026 14:56:40 +0900 Subject: [PATCH] fix(mcp-server): review fixes for the shell's build tsc -p shell checked nothing: the inherited exclude dropped every shell file; the shell's tsconfig now excludes node_modules only. Tailwind scans app-toaster.tsx too, so the sync notices look like the web app's toasts. The bundle keeps its dependencies' license comments (esbuild appends them at the end). postcss, which build-shell.mjs loads from the root, is a declared devDependency. --- package-lock.json | 1 + package.json | 1 + packages/mcp-server/scripts/build-shell.mjs | 1 - packages/mcp-server/shell/shell.css | 1 + packages/mcp-server/shell/tsconfig.json | 3 ++- 5 files changed, 5 insertions(+), 2 deletions(-) diff --git a/package-lock.json b/package-lock.json index 46b3bb8b..dc5c3477 100644 --- a/package-lock.json +++ b/package-lock.json @@ -100,6 +100,7 @@ "husky": "^9.1.7", "jsdom": "^27.4.0", "lint-staged": "^16.2.7", + "postcss": "^8.5.6", "shx": "^0.4.0", "tailwindcss": "^4", "typescript": "^5", diff --git a/package.json b/package.json index c987c36e..7865c467 100644 --- a/package.json +++ b/package.json @@ -135,6 +135,7 @@ "husky": "^9.1.7", "jsdom": "^27.4.0", "lint-staged": "^16.2.7", + "postcss": "^8.5.6", "shx": "^0.4.0", "tailwindcss": "^4", "typescript": "^5", diff --git a/packages/mcp-server/scripts/build-shell.mjs b/packages/mcp-server/scripts/build-shell.mjs index 64d0c3bf..562c3133 100644 --- a/packages/mcp-server/scripts/build-shell.mjs +++ b/packages/mcp-server/scripts/build-shell.mjs @@ -35,7 +35,6 @@ const result = await build({ target: "es2022", platform: "browser", jsx: "automatic", - legalComments: "none", metafile: true, absWorkingDir: root, alias: { diff --git a/packages/mcp-server/shell/shell.css b/packages/mcp-server/shell/shell.css index 70ac7dd1..8eec509e 100644 --- a/packages/mcp-server/shell/shell.css +++ b/packages/mcp-server/shell/shell.css @@ -8,6 +8,7 @@ @source "../../../components/canvas"; @source "../../../components/ui"; @source "../../../components/ai-elements"; +@source "../../../components/workspace/app-toaster.tsx"; /* System fonts stand in for the web fonts the Next.js layout loads */ :root { diff --git a/packages/mcp-server/shell/tsconfig.json b/packages/mcp-server/shell/tsconfig.json index dcf2cc39..91aae0f6 100644 --- a/packages/mcp-server/shell/tsconfig.json +++ b/packages/mcp-server/shell/tsconfig.json @@ -5,5 +5,6 @@ "incremental": false, "plugins": [] }, - "include": ["./**/*.ts", "./**/*.tsx"] + "include": ["./**/*.ts", "./**/*.tsx"], + "exclude": ["node_modules"] }