Skip to content

Commit dd8c7e5

Browse files
committed
fix(prefer-const): remove $state from excluded runes
1 parent e7d2856 commit dd8c7e5

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

packages/eslint-plugin-svelte/src/rules/prefer-const.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
import { createRule } from '../utils';
21
import type { TSESTree } from '@typescript-eslint/types';
32

3+
import { createRule } from '../utils/index.js';
44
import { defineWrapperListener, getCoreRule } from '../utils/eslint-core.js';
55

66
const coreRule = getCoreRule('prefer-const');
@@ -31,7 +31,7 @@ function shouldSkipDeclaration(declaration: TSESTree.Expression | null) {
3131
return false;
3232
}
3333

34-
if (callee.type === 'Identifier' && ['$state', '$props', '$derived'].includes(callee.name)) {
34+
if (callee.type === 'Identifier' && ['$props', '$derived'].includes(callee.name)) {
3535
return true;
3636
}
3737

packages/eslint-plugin-svelte/tests/fixtures/rules/prefer-const/invalid/test01-errors.yaml

+4
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,10 @@
22
line: 3
33
column: 6
44
suggestions: null
5+
- message: "'state' is never reassigned. Use 'const' instead."
6+
line: 4
7+
column: 6
8+
suggestions: null
59
- message: "'doubled' is never reassigned. Use 'const' instead."
610
line: 6
711
column: 6

packages/eslint-plugin-svelte/tests/fixtures/rules/prefer-const/invalid/test01-output.svelte

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<script>
22
let { prop1, prop2 } = $props();
33
const zero = 0;
4-
let state = $state(0);
4+
const state = $state(0);
55
let raw = $state.raw(0);
66
const doubled = state * 2;
77
let derived = $derived(state * 2);

0 commit comments

Comments
 (0)