fix: tsconfig includes should cover svelte.config.js

[benmccann]

I was unable to setup https://typescript-eslint.io/rules/no-misused-promises/ in a SvelteKit project without this. I might have been able to tweak the esconfig settings to get it to work, but either way it seemed like the svelte.config.js should be there if the vite.config.js is there

[changeset-bot]

🦋 Changeset detected

Latest commit: 01a3a7c

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]

Looks good to me but can't help but wonder why we didn't do it previously - did we omit it on purpose to not pull in certain global type definitions which pollute the namespace? I'm not sure.

[benmccann]

I just went through the file history and can't see that we ever removed it, so I'm going to assume it was just overlooked.

[thenbe]

I was unable to setup typescript-eslint.io/rules/no-misused-promises in a SvelteKit project without this.

I've encountered similar(?) weirdness while trying to add type-aware linting to this example repo. In my case, the current workaround is to ensure the $DIR in pnpm eslint $DIR has an immediate child that ends in .svelte. Otherwise type-aware linting fails for all svelte files.

I'm still having trouble identifying the root cause.