fix(eslint): account for all config formats

[sacrosanctic]

I spotted the problem when looking at https://github.com/hyunbinseo/svelte-kit-templates/blob/b200ae76d7bb2c61e6e695c4d90912666d39256c/javascript/eslint.config.js. I had to test eslint by hand.

We should add a test, could someone lend me hand?

That repo was quite helpful, maybe we can expand on the templates we take snapshots for?

• missing type

[changeset-bot]

🦋 Changeset detected

Latest commit: 4074d63

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

This PR includes changesets to release 1 package

Name	Type
sv	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

[svelte-docs-bot]

Preview: https://svelte-dev-git-preview-cli-994-svelte.vercel.app/

[pkg-pr-new]

Open in StackBlitz

npx https://pkg.pr.new/svelte-migrate@994

npx https://pkg.pr.new/sv@994

npx https://pkg.pr.new/@sveltejs/sv-utils@994

commit: 4074d63

[manuel3108]

Which problem exactly are you trying to solve in this file? https://github.com/hyunbinseo/svelte-kit-templates/blob/b200ae76d7bb2c61e6e695c4d90912666d39256c/javascript/eslint.config.js I don't think im following currently.

Don't mind the comments below, it's just so that we don't forget about them.

[sacrosanctic]

Which problem exactly are you trying to solve in this file?

@manuel3108

prettier and svelte.configs.prettier are supposed to be right after svelte.configs.recommended.

The original only partially supported the following cases, [], defineConfig(), and defineConfig([]). It now also handles svelte.configs.recommended whether it is spread or not.

[sacrosanctic]

@jycouet

Added some of the changes from #943 to lighten the load there.

Is the CI telling me i need to make that change in the svelte.dev repo?

Should we have more snapshots?

[jycouet]

I didn't get what is the root cause of this PR?

[sacrosanctic]

Not too familiar with the AST, cannot move the comment inside the object.

[jycouet]

Not too familiar with the AST, cannot move the comment inside the object.

Nah, I don't manage too. esrap manage before or after node, but not inside. (I think it's not needed, it's ok)
I updated with 1 existing rule. WDYT? (if you have a better rule idea, that's good for me too)

Let me know

[sacrosanctic]

LGTM