fix: remove types: ['node'] from generated tsconfig

[Copilot]

The generated .svelte-kit/tsconfig.json hardcodes types: ['node'], which produces TS2688 when @types/node isn't installed. This silently causes svelte-check --incremental to skip standalone .ts files (e.g. +server.ts, utility modules), missing type errors that non-incremental mode catches.

• Remove types: ['node'] from the generated compilerOptions in write_tsconfig.js

Without types, TypeScript falls back to its default behavior: auto-include all @types/* packages present in node_modules. Users with @types/node installed see no change; users without it no longer get a phantom error that breaks incremental checking.

EDIT: I might want to add a failing test for this

[changeset-bot]

🦋 Changeset detected

Latest commit: aa61d1f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@sveltejs/kit	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[dummdidumm]

I believe in TS 6/7 the default is the empty array so this would break it for users of that version. We might have to check whether the types exist and add the array or not depending on the outcome

[teemingc]

I believe in TS 6/7 the default is the empty array so this would break it for users of that version. We might have to check whether the types exist and add the array or not depending on the outcome

That’s true. But, we didn’t have this before moving to TypeScript 6, and those who chose to upgrade would have had to deal with a bunch of breaking changes anyway. Therefore, I’m wondering if it’s better for users to add the types array themselves in the user config. It would be a little weird if only node was automatically added.