feat: added the no-useless-children-snippet rule

[marekdedic]

Closes #909

[changeset-bot]

🦋 Changeset detected

Latest commit: 4a0df5a

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

This PR includes changesets to release 1 package

Name	Type
eslint-plugin-svelte	Minor

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

[marekdedic]

Blocked by #923 which causes the CI errors

[ota-meshi]

Thank you for this PR! The implementation looks good, but I think we need to fix CI errors.
Could you please put a _requirements.json with {"svelte": ">=5.0.0"} in test fixtures directories to exclude old Svelte tests?
Could you also fix the linting errors?

https://github.com/sveltejs/eslint-plugin-svelte/blob/main/packages/eslint-plugin-svelte/tests/fixtures/rules/no-inner-declarations/invalid/_requirements.json

[ota-meshi]

This PR is looks good, but I made this package ESM.
#932

Could you please merge the main branch and fix it by adding .js to the import statement of your implementation?

[marekdedic]

Done :)

[baseballyama]

In cases like this code, “Found an unnecessary children snippet.” appears, but this is incorrect. It would be better to only inspect inside Svelte components.

{#snippet children()}
  Hi!
{/snippet}

{@render children()}

https://svelte.dev/playground/hello-world?version=5.16.0#H4sIAAAAAAAAE23LywrCMBSE4Vep46aFovusdNd3sC5qM2IgnobkeCPk3SWgC8HtN_9kyHQlDAZ6vzSPJXrbtLROaTv0ODvPBHPI0FeoXQX039c-hE2602u105T4z-dFlKIJBnmdxIVAbeaL8zZS2q6MMurgVqPk7WetJHkXKZbxN0UP5VNhNN5YjuUNEZNJhcEAAAA=

[pkg-pr-new]

Open in Stackblitz

npm i https://pkg.pr.new/eslint-plugin-svelte@922

commit: 4a0df5a

[marekdedic]

@baseballyama Good catch, fixed & added it as a test :)