Skip to content

fix: avoid injecting a non-serializable TypeScript config callback - #17512

Merged
teemingc merged 4 commits into
mainfrom
triage/kit-17511-eslint-cache
Oct 21, 2026
Merged

teemingc merged 4 commits into
mainfrom
triage/kit-17511-eslint-cache

Conversation

@svelte-triage-bot

Copy link
Copy Markdown
Contributor
  • Kit injects an identity typescript.config callback even when omitted, preventing ESLint from serializing the loaded configuration with --cache.
  • Configuration flattening in #16752 exposed this default; #16458 already made transforms optional. Remove the redundant callback rather than special-casing ESLint serialization.
  • Preserve explicit callbacks, validation/deprecation diagnostics, and service-worker configuration separation. Cover cached linting and mutation/replacement callbacks; user-supplied non-serializable functions remain subject to ESLint’s restrictions.

Fixes #17511

@pkg-svelte-dev

pkg-svelte-dev Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from 0bfe122:

pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/0bfe1227960863d52e3d53d0f9317f4cc41e67b9

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/kit/pr/17512

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

@teemingc
teemingc marked this pull request as ready for review October 21, 2026 21:02
@changeset-bot

changeset-bot Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0bfe122

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

@teemingc

Copy link
Copy Markdown
Member

Removed some tests. We don’t need that many for something most people have removed and we plan to remove too

@teemingc
teemingc merged commit 186388b into sveltejs:main Oct 21, 2026
36 of 38 checks passed
@github-actions github-actions Bot mentioned this pull request Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Svelte Kit 3 is not working with eslint when we pass --cache to eslint

1 participant