From 2f9b3baa56e3e975f2da755d49df177a6454f354 Mon Sep 17 00:00:00 2001 From: Aravind Date: Fri, 18 Sep 2026 18:56:46 +0530 Subject: [PATCH] chore(ts-migration): remove the migration-only TypeScript configuration MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `src` holds 321 TypeScript files and no JavaScript, so four settings that existed only to let the two coexist have nothing left to act on. Each was removed on its own and validated before the next. allowJs / checkJs Removed together, because they are coupled: `checkJs` without `allowJs` is TS5052, a configuration error rather than a code one. Verified first that nothing under `src` imports a `.js` module. The program still resolves the same 321 files and still reports 0 errors. This also closes a door. With `allowJs` on, a `.js` file added under `src` would be compiled and bundled silently; now it is a resolution failure, which is the right outcome for a codebase that has finished migrating. jsconfig.json Deleted. It had been inert since `tsconfig.json` appeared — TypeScript ignores a jsconfig wherever a tsconfig sits beside it — and it still carried the old `types: []` and the three stale excludes. A second, unread copy of the options is an invitation to edit the wrong file. components.json `"tsx": false` -> `true`, so `npx shadcn add` emits TSX. No runtime effect; it would have quietly reintroduced `.jsx` into a repository that has none. `README.md` documented `npm run typecheck` as `tsc -p ./jsconfig.json` with `checkJs`, which named a file that no longer exists. Corrected, along with a stale assertion count in the same table (835, against 1691 today). No file under `src` changed, so there is nothing for a per-file emitted-JS comparison to compare; the bundle hash is the check that matters and it is unmoved. typecheck 0 errors, 321 files in the program lint exit 0 npm test 1684/1691, the same 7 failures owliver unchanged; baselines still pinned, not recaptured build exit 0, bundle 74d17e2d… identical Left in place deliberately, reported rather than removed: `scripts/ssr-resolve.mjs`. Its own comment says it can go once `src` holds no `.js`, but that is only half the condition — the 203 literal `.js` paths inside `skill-check.mjs` would all have to be renamed first, and that is a large diff against the file that guards this migration. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_01HBG1wnuRfJKCstGB8Fekr8 --- README.md | 4 ++-- components.json | 2 +- jsconfig.json | 21 --------------------- tsconfig.json | 38 +++++++++++++++++++------------------- 4 files changed, 22 insertions(+), 43 deletions(-) delete mode 100644 jsconfig.json diff --git a/README.md b/README.md index 5179187..0b8039b 100644 --- a/README.md +++ b/README.md @@ -179,8 +179,8 @@ Toasts go through `ds/toast` (a wrapper over `react-hot-toast`), rendered by `