From aff960f025e1e8e7ba73f35e02f2604c9e203e58 Mon Sep 17 00:00:00 2001 From: Danil Silantyev Date: Sat, 4 Jul 2026 12:53:18 +0700 Subject: [PATCH] chore(tsconfig): drop stale dashboard build comment and vestigial exclude MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The exclude comment described src/dashboard as "the Svelte browser bundle — compiled separately by scripts/build-dashboard-ui.mjs ... covered by tsconfig.dashboard.json for editor + CI type-checking." None of that is true: - src/dashboard/ is server-rendered HTML (htmx + Alpine, both inlined by scripts/vendor-ui.mjs) — not Svelte, and there is no esbuild-svelte step - scripts/build-dashboard-ui.mjs and tsconfig.dashboard.json do not exist - CI (.github/workflows/ci.yml) runs only the root typecheck/test/build The exclude itself was a no-op: src/dashboard.ts (which IS in the program) imports ./dashboard/fragments.js and ./dashboard/types.js, so tsc already type-checks and emits dist/dashboard/* transitively. Verified: removing the exclude leaves typecheck green and dist/dashboard/* emits identically with or without it. Dropped both the false comment and the dead exclude entry. --- tsconfig.json | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/tsconfig.json b/tsconfig.json index d47376a..de853e6 100644 --- a/tsconfig.json +++ b/tsconfig.json @@ -25,12 +25,6 @@ "exclude": [ "node_modules", "dist", - "legacy", - // src/dashboard/ is the Svelte browser bundle — compiled separately by - // scripts/build-dashboard-ui.mjs with the DOM lib + esbuild-svelte. The - // root tsconfig targets Node + WebWorker (no `document` / `localStorage`), - // so type-checking dashboard sources here would always fail. They're still - // covered by `tsconfig.dashboard.json` for editor + CI type-checking. - "src/dashboard" + "legacy" ] }