Add `"types": ["node"]` in `tsconfig.json` when needed

[hyunbinseo]

Describe the bug

When pnpm check is ran in the default minimal template:

pnpm dlx sv@0.15.3 create --template minimal --types ts --install pnpm ./

This warning is logged, which is valid since @types/node is not installed.

d:\_\tsconfig.json:1:1
Warn: Cannot find type definition file for 'node'.
  The file is in the program because:
    Entry point of type library 'node' specified in compilerOptions

Why not move "types": ["node"], to the tsconfig.json generated by the SV CLI?

- .svelte-kit\tsconfig.json
+ tsconfig.json

e.g. When ESLint add-on is selected which relies on node:path, this can be done:

1. @types/node is installed
2. "types": ["node"], is added

Reproduction

pnpm dlx sv@0.15.3 create --template minimal --types ts --install pnpm ./
pnpm check

pnpm i @types/node -D
pnpm check

Logs

System Info

System:
  OS: Windows 11 10.0.26200
  CPU: (8) x64 Intel(R) Core(TM) Ultra 7 258V
  Memory: 2.85 GB / 31.48 GB
Binaries:
  Node: 24.15.0 - C:\Users\hyunb\AppData\Local\fnm_multishells\28376_1778829182710\node.EXE
  Yarn: 1.22.22 - C:\Users\hyunb\AppData\Local\fnm_multishells\28376_1778829182710\yarn.CMD
  npm: 11.12.1 - C:\Users\hyunb\AppData\Local\fnm_multishells\28376_1778829182710\npm.CMD
  pnpm: 11.1.2 - C:\Users\hyunb\AppData\Local\fnm_multishells\28376_1778829182710\pnpm.CMD
  Deno: 2.4.0 - C:\Users\hyunb\.deno\bin\deno.EXE
Browsers:
  Chrome: 148.0.7778.97
  Edge: Chromium (140.0.3485.54)
  Firefox: 150.0.3 - C:\Program Files\Mozilla Firefox\firefox.exe
npmPackages:
  @sveltejs/adapter-auto: ^7.0.1 => 7.0.1
  @sveltejs/kit: ^2.57.0 => 2.59.1
  @sveltejs/vite-plugin-svelte: ^7.0.0 => 7.1.2
  svelte: ^5.55.2 => 5.55.5
  vite: ^8.0.7 => 8.0.12

Severity

annoyance

Additional Information

No response

[hyunbinseo]

@teemingc are you suggesting that @types/node should be installed by default? The issue body was intended for SvelteKit.

I really think @types/node should be installed only if needed, especially since it can mismatch the actual runtime version.

e.g. I am using Node.js 26 - @types/node does not exist yet.

[teemingc]

We plan to remove it from the generated config in kit. It’s up to users to install it themselves although the cli template should include it where needed

[hyunbinseo]

This was what I was hoping for:

We plan to remove it from the generated config in kit.

I guess there should be a follow up issue in the CLI - which can be this. Will update the title.

[teemingc]

related PR sveltejs/kit#15709

[sacrosanctic]

When should this be included, when we use adapter-node?

[teemingc]

adapter-node, vercel, netlify. Maybe adapter-static?

[sacrosanctic]

might be fixed by sveltejs/kit#16458

[teemingc]

Does it add the node type automatically? It shouldn’t because some apps may not use it