lint-rules-alvin 1.1.4 → 1.2.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/eslint/configs/index.d.ts +1 -0
- package/eslint/configs/index.js +1 -0
- package/eslint/configs/sveltekit.js +209 -0
- package/eslint/presets/index.d.ts +4 -7
- package/eslint/presets/index.js +2 -2
- package/eslint/presets/sveltekitTs.js +30 -0
- package/package.json +21 -10
- package/eslint/presets/astroReact.js +0 -34
|
@@ -9,4 +9,5 @@ export declare const json: Linter.Config[];
|
|
|
9
9
|
export declare const markdown: Linter.Config;
|
|
10
10
|
export declare const reactHooks: Linter.Config;
|
|
11
11
|
export declare const stylistic: Linter.Config;
|
|
12
|
+
export declare const sveltekit: Linter.Config[];
|
|
12
13
|
export declare const typescript: Linter.Config;
|
package/eslint/configs/index.js
CHANGED
|
@@ -0,0 +1,209 @@
|
|
|
1
|
+
import eslintPluginSvelte from 'eslint-plugin-svelte';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* The ESLint `stylistic` config. Extends `configs.recommended` and overrides all rules.
|
|
5
|
+
*
|
|
6
|
+
* @type {import('eslint').Linter.Config[]}
|
|
7
|
+
*/
|
|
8
|
+
export const sveltekit = [
|
|
9
|
+
...eslintPluginSvelte.configs['flat/base'],
|
|
10
|
+
{
|
|
11
|
+
files: [
|
|
12
|
+
'**/*.svelte',
|
|
13
|
+
'**/*.svelte.ts',
|
|
14
|
+
'**/*.svelte.js'
|
|
15
|
+
],
|
|
16
|
+
rules: {
|
|
17
|
+
|
|
18
|
+
// Svelte-specific
|
|
19
|
+
'svelte/infinite-reactive-loop': 'warn',
|
|
20
|
+
'svelte/no-dom-manipulating': 'warn',
|
|
21
|
+
'svelte/no-dupe-else-if-blocks': 'error',
|
|
22
|
+
'svelte/no-dupe-on-directives': 'error',
|
|
23
|
+
'svelte/no-dupe-style-properties': 'error',
|
|
24
|
+
'svelte/no-dupe-use-directives': 'error',
|
|
25
|
+
'svelte/no-not-function-handler': 'error',
|
|
26
|
+
'svelte/no-object-in-text-mustaches': 'error',
|
|
27
|
+
'svelte/no-raw-special-elements': 'error',
|
|
28
|
+
'svelte/no-reactive-reassign': 'error',
|
|
29
|
+
'svelte/no-shorthand-style-property-overrides': 'warn',
|
|
30
|
+
'svelte/no-store-async': 'error',
|
|
31
|
+
'svelte/no-top-level-browser-globals': 'error',
|
|
32
|
+
'svelte/no-unknown-style-directive-property': 'error',
|
|
33
|
+
'svelte/prefer-svelte-reactivity': 'error',
|
|
34
|
+
'svelte/require-store-callbacks-use-set-param': 'error',
|
|
35
|
+
'svelte/require-store-reactive-access': 'error',
|
|
36
|
+
'svelte/valid-compile': 'off',
|
|
37
|
+
'svelte/valid-style-parse': 'error',
|
|
38
|
+
|
|
39
|
+
// Security
|
|
40
|
+
'svelte/no-at-html-tags': 'warn',
|
|
41
|
+
'svelte/no-target-blank': 'error',
|
|
42
|
+
|
|
43
|
+
// Best practices
|
|
44
|
+
'svelte/block-lang': [
|
|
45
|
+
'error',
|
|
46
|
+
{
|
|
47
|
+
enforceScriptPresent: false,
|
|
48
|
+
enforceStylePresent: false,
|
|
49
|
+
script: [
|
|
50
|
+
'ts',
|
|
51
|
+
'js'
|
|
52
|
+
]
|
|
53
|
+
}
|
|
54
|
+
],
|
|
55
|
+
'svelte/button-has-type': 'off',
|
|
56
|
+
'svelte/no-add-event-listener': 'error',
|
|
57
|
+
'svelte/no-at-debug-tags': 'warn',
|
|
58
|
+
'svelte/no-ignored-unsubscribe': 'warn',
|
|
59
|
+
'svelte/no-immutable-reactive-statements': 'error',
|
|
60
|
+
'svelte/no-inline-styles': 'error',
|
|
61
|
+
'svelte/no-inspect': 'warn',
|
|
62
|
+
'svelte/no-reactive-functions': 'error',
|
|
63
|
+
'svelte/no-reactive-literals': 'error',
|
|
64
|
+
'svelte/no-svelte-internal': 'error',
|
|
65
|
+
'svelte/no-unnecessary-state-wrap': 'error',
|
|
66
|
+
'svelte/no-unused-class-name': 'off', // TailwindCSS doesn't work with this rule apparently
|
|
67
|
+
'svelte/no-unused-props': 'error',
|
|
68
|
+
'svelte/no-unused-svelte-ignore': 'error',
|
|
69
|
+
'svelte/no-useless-children-snippet': 'error',
|
|
70
|
+
'svelte/no-useless-mustaches': 'error',
|
|
71
|
+
'prefer-const': 'off',
|
|
72
|
+
'svelte/prefer-const': 'error',
|
|
73
|
+
'svelte/prefer-destructured-store-props': 'error',
|
|
74
|
+
'svelte/prefer-writable-derived': 'error',
|
|
75
|
+
'svelte/require-each-key': 'error',
|
|
76
|
+
'svelte/require-event-dispatcher-types': 'error',
|
|
77
|
+
'svelte/require-optimized-style-attribute': 'error',
|
|
78
|
+
'svelte/require-stores-init': 'error',
|
|
79
|
+
'svelte/valid-each-key': 'error',
|
|
80
|
+
|
|
81
|
+
// Stylistic
|
|
82
|
+
'svelte/consistent-selector-style': 'off',
|
|
83
|
+
'svelte/derived-has-same-inputs-outputs': 'error',
|
|
84
|
+
'svelte/first-attribute-linebreak': [
|
|
85
|
+
'error',
|
|
86
|
+
{
|
|
87
|
+
singleline: 'beside',
|
|
88
|
+
multiline: 'below'
|
|
89
|
+
}
|
|
90
|
+
],
|
|
91
|
+
'svelte/html-closing-bracket-new-line': [
|
|
92
|
+
'error',
|
|
93
|
+
{
|
|
94
|
+
singleline: 'never',
|
|
95
|
+
multiline: 'always',
|
|
96
|
+
selfClosingTag: {
|
|
97
|
+
singleline: 'never',
|
|
98
|
+
multiline: 'always'
|
|
99
|
+
}
|
|
100
|
+
}
|
|
101
|
+
],
|
|
102
|
+
'svelte/html-closing-bracket-spacing': [
|
|
103
|
+
'error',
|
|
104
|
+
{
|
|
105
|
+
startTag: 'never',
|
|
106
|
+
endTag: 'never',
|
|
107
|
+
selfClosingTag: 'always'
|
|
108
|
+
}
|
|
109
|
+
],
|
|
110
|
+
'svelte/html-quotes': [
|
|
111
|
+
'error',
|
|
112
|
+
{
|
|
113
|
+
prefer: 'double',
|
|
114
|
+
dynamic: {
|
|
115
|
+
quoted: false,
|
|
116
|
+
avoidInvalidUnquotedInHTML: false
|
|
117
|
+
}
|
|
118
|
+
}
|
|
119
|
+
],
|
|
120
|
+
'svelte/html-self-closing': [
|
|
121
|
+
'error',
|
|
122
|
+
'default'
|
|
123
|
+
],
|
|
124
|
+
'indent': 'off', // eslint-disable-line @stylistic/quote-props
|
|
125
|
+
'@stylistic/indent': 'off',
|
|
126
|
+
'svelte/indent': [
|
|
127
|
+
'error',
|
|
128
|
+
{
|
|
129
|
+
indent: 'tab',
|
|
130
|
+
switchCase: 1,
|
|
131
|
+
alignAttributesVertically: false
|
|
132
|
+
}
|
|
133
|
+
],
|
|
134
|
+
'svelte/max-attributes-per-line': [
|
|
135
|
+
'error',
|
|
136
|
+
{
|
|
137
|
+
singleline: 1,
|
|
138
|
+
multiline: 1
|
|
139
|
+
}
|
|
140
|
+
],
|
|
141
|
+
'svelte/mustache-spacing': [
|
|
142
|
+
'error',
|
|
143
|
+
{
|
|
144
|
+
textExpressions: 'never',
|
|
145
|
+
attributesAndProps: 'never',
|
|
146
|
+
directiveExpressions: 'never',
|
|
147
|
+
tags: {
|
|
148
|
+
openingBrace: 'never',
|
|
149
|
+
closingBrace: 'never'
|
|
150
|
+
}
|
|
151
|
+
}
|
|
152
|
+
],
|
|
153
|
+
'svelte/no-extra-reactive-curlies': 'error',
|
|
154
|
+
'svelte/no-restricted-html-elements': 'off', // Allow the user to override
|
|
155
|
+
'svelte/no-spaces-around-equal-signs-in-attribute': 'error',
|
|
156
|
+
'svelte/prefer-class-directive': 'error',
|
|
157
|
+
'svelte/prefer-style-directive': 'error',
|
|
158
|
+
'svelte/require-event-prefix': 'off',
|
|
159
|
+
'svelte/shorthand-attribute': [
|
|
160
|
+
'error',
|
|
161
|
+
{ prefer: 'always' }
|
|
162
|
+
],
|
|
163
|
+
'svelte/shorthand-directive': [
|
|
164
|
+
'off',
|
|
165
|
+
{ prefer: 'never' }
|
|
166
|
+
],
|
|
167
|
+
'svelte/sort-attributes': 'error',
|
|
168
|
+
'svelte/spaced-html-comment': [
|
|
169
|
+
'error',
|
|
170
|
+
'always'
|
|
171
|
+
],
|
|
172
|
+
|
|
173
|
+
// Extensions
|
|
174
|
+
'svelte/no-inner-declarations': [
|
|
175
|
+
'error',
|
|
176
|
+
'functions',
|
|
177
|
+
{ blockScopedFunctions: 'allow' }
|
|
178
|
+
],
|
|
179
|
+
'no-trailing-spaces': 'off',
|
|
180
|
+
'svelte/no-trailing-spaces': [
|
|
181
|
+
'error',
|
|
182
|
+
{
|
|
183
|
+
skipBlankLines: false,
|
|
184
|
+
ignoreComments: false
|
|
185
|
+
}
|
|
186
|
+
],
|
|
187
|
+
|
|
188
|
+
// SvelteKit
|
|
189
|
+
'svelte/no-export-load-in-svelte-module-in-kit-pages': 'error',
|
|
190
|
+
'svelte/no-navigation-without-resolve': [
|
|
191
|
+
'error',
|
|
192
|
+
{
|
|
193
|
+
ignoreGoto: false,
|
|
194
|
+
ignoreLinks: false,
|
|
195
|
+
ignorePushState: false,
|
|
196
|
+
ignoreReplaceState: false
|
|
197
|
+
}
|
|
198
|
+
],
|
|
199
|
+
'svelte/valid-prop-names-in-kit-pages': 'error',
|
|
200
|
+
|
|
201
|
+
// System
|
|
202
|
+
'svelte/comment-directive': [
|
|
203
|
+
'error',
|
|
204
|
+
{ reportUnusedDisableDirectives: true }
|
|
205
|
+
],
|
|
206
|
+
'svelte/system': 'error' // Keep this on!
|
|
207
|
+
}
|
|
208
|
+
}
|
|
209
|
+
];
|
|
@@ -1,14 +1,11 @@
|
|
|
1
1
|
import type { Linter } from 'eslint';
|
|
2
2
|
|
|
3
3
|
/**
|
|
4
|
-
* ESLint preset for Astro/React with
|
|
5
|
-
*
|
|
6
|
-
* This package uses ESLint's flat config format, so the preset is an array
|
|
7
|
-
* of flat-config items.
|
|
4
|
+
* ESLint preset for Astro/React with TypeScript.
|
|
8
5
|
*/
|
|
9
|
-
export declare const
|
|
6
|
+
export declare const astroReactTs: readonly Linter.Config[];
|
|
10
7
|
|
|
11
8
|
/**
|
|
12
|
-
* ESLint preset for
|
|
9
|
+
* ESLint preset for SvelteKit with TypeScript.
|
|
13
10
|
*/
|
|
14
|
-
export declare const
|
|
11
|
+
export declare const sveltekitTs: readonly Linter.Config[];
|
package/eslint/presets/index.js
CHANGED
|
@@ -1,2 +1,2 @@
|
|
|
1
|
-
export {
|
|
2
|
-
export {
|
|
1
|
+
export { astroReactTs } from './astroReactTs.js';
|
|
2
|
+
export { sveltekitTs } from './sveltekitTs.js';
|
|
@@ -0,0 +1,30 @@
|
|
|
1
|
+
import { defineConfig } from 'eslint/config';
|
|
2
|
+
import { importX as eslintPluginImportX } from 'eslint-plugin-import-x';
|
|
3
|
+
import {
|
|
4
|
+
base,
|
|
5
|
+
importX,
|
|
6
|
+
sveltekit,
|
|
7
|
+
stylistic,
|
|
8
|
+
custom,
|
|
9
|
+
typescript
|
|
10
|
+
} from '../configs/index.js';
|
|
11
|
+
|
|
12
|
+
/**
|
|
13
|
+
* The `ESLint` PocketBase/SvelteKit config with typescript.
|
|
14
|
+
*/
|
|
15
|
+
export const sveltekitTs = defineConfig(
|
|
16
|
+
base,
|
|
17
|
+
typescript,
|
|
18
|
+
sveltekit,
|
|
19
|
+
{
|
|
20
|
+
...eslintPluginImportX.configs['flat/typescript'],
|
|
21
|
+
...importX
|
|
22
|
+
},
|
|
23
|
+
stylistic,
|
|
24
|
+
custom,
|
|
25
|
+
{
|
|
26
|
+
name: 'sveltekit-override',
|
|
27
|
+
files: [ '**/*.svelte' ],
|
|
28
|
+
rules: { 'import-x/unambiguous': 'off' }
|
|
29
|
+
}
|
|
30
|
+
);
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "lint-rules-alvin",
|
|
3
|
-
"version": "1.1
|
|
3
|
+
"version": "1.2.1",
|
|
4
4
|
"description": "My own personal linting ruleset for a bunch of different plugins. Includes a few custom rules. Used in a few of my repos.",
|
|
5
5
|
"keywords": [
|
|
6
6
|
"eslint"
|
|
@@ -13,19 +13,16 @@
|
|
|
13
13
|
"author": "Alvin Raul Szőke",
|
|
14
14
|
"type": "module",
|
|
15
15
|
"exports": {
|
|
16
|
-
"./eslint":
|
|
17
|
-
|
|
18
|
-
"types": "./eslint/presets/index.d.ts"
|
|
19
|
-
},
|
|
20
|
-
"./eslint/configs": "./eslint/configs/index.js",
|
|
16
|
+
"./eslint/presets/astroReactTs": "./eslint/presets/astroReactTs.js",
|
|
17
|
+
"./eslint/presets/sveltekitTs": "./eslint/presets/sveltekitTs.js",
|
|
21
18
|
"./eslint/configs/astro": "./eslint/configs/astro.js",
|
|
22
|
-
"./eslint/configs/astro_ts": "./eslint/configs/
|
|
19
|
+
"./eslint/configs/astro_ts": "./eslint/configs/astroTs.js",
|
|
23
20
|
"./eslint/configs/base": "./eslint/configs/base.js",
|
|
24
|
-
"./eslint/configs/import_x": "./eslint/configs/
|
|
25
|
-
"./eslint/configs/import_x_ts": "./eslint/configs/
|
|
21
|
+
"./eslint/configs/import_x": "./eslint/configs/importX.js",
|
|
22
|
+
"./eslint/configs/import_x_ts": "./eslint/configs/importXTs.js",
|
|
26
23
|
"./eslint/configs/json": "./eslint/configs/json.js",
|
|
27
24
|
"./eslint/configs/markdown": "./eslint/configs/markdown.js",
|
|
28
|
-
"./eslint/configs/react_hooks": "./eslint/configs/
|
|
25
|
+
"./eslint/configs/react_hooks": "./eslint/configs/reactHooks.js",
|
|
29
26
|
"./eslint/configs/stylistic": "./eslint/configs/stylistic.js",
|
|
30
27
|
"./eslint/configs/typescript": "./eslint/configs/typescript.js"
|
|
31
28
|
},
|
|
@@ -43,7 +40,9 @@
|
|
|
43
40
|
"eslint-plugin-jsonc": "^2.21.0",
|
|
44
41
|
"eslint-plugin-markdown": "^5.1.0",
|
|
45
42
|
"eslint-plugin-react-hooks": "^7.0.0",
|
|
43
|
+
"eslint-plugin-svelte": "^3.12.4",
|
|
46
44
|
"globals": "^16.4.0",
|
|
45
|
+
"svelte": "^5.40.2",
|
|
47
46
|
"typescript": "^5.9.3",
|
|
48
47
|
"typescript-eslint": "^8.46.1"
|
|
49
48
|
},
|
|
@@ -59,6 +58,9 @@
|
|
|
59
58
|
"eslint-plugin-jsonc": "^2.21.0",
|
|
60
59
|
"eslint-plugin-markdown": "^5.1.0",
|
|
61
60
|
"eslint-plugin-react-hooks": "^7.0.0",
|
|
61
|
+
"eslint-plugin-svelte": "^3.12.4",
|
|
62
|
+
"globals": "^16.4.0",
|
|
63
|
+
"svelte": "^5.40.2",
|
|
62
64
|
"typescript": "^5.9.3",
|
|
63
65
|
"typescript-eslint": "^8.46.0"
|
|
64
66
|
},
|
|
@@ -96,6 +98,15 @@
|
|
|
96
98
|
"eslint-plugin-react-hooks": {
|
|
97
99
|
"optional": true
|
|
98
100
|
},
|
|
101
|
+
"eslint-plugin-svelte": {
|
|
102
|
+
"optional": true
|
|
103
|
+
},
|
|
104
|
+
"globals": {
|
|
105
|
+
"optional": true
|
|
106
|
+
},
|
|
107
|
+
"svelte": {
|
|
108
|
+
"optional": true
|
|
109
|
+
},
|
|
99
110
|
"typescript": {
|
|
100
111
|
"optional": true
|
|
101
112
|
},
|
|
@@ -1,34 +0,0 @@
|
|
|
1
|
-
import { defineConfig } from 'eslint/config';
|
|
2
|
-
import { importX as eslintPluginImportX } from 'eslint-plugin-import-x';
|
|
3
|
-
import {
|
|
4
|
-
base,
|
|
5
|
-
astro,
|
|
6
|
-
importX,
|
|
7
|
-
reactHooks,
|
|
8
|
-
stylistic,
|
|
9
|
-
custom
|
|
10
|
-
} from '../configs/index.js';
|
|
11
|
-
|
|
12
|
-
/**
|
|
13
|
-
* The `ESLint` Astro/React config with typescript.
|
|
14
|
-
*/
|
|
15
|
-
export const astroReact = defineConfig(
|
|
16
|
-
base,
|
|
17
|
-
astro,
|
|
18
|
-
{
|
|
19
|
-
...eslintPluginImportX.configs['flat/react'],
|
|
20
|
-
...importX
|
|
21
|
-
},
|
|
22
|
-
reactHooks,
|
|
23
|
-
stylistic,
|
|
24
|
-
custom,
|
|
25
|
-
{
|
|
26
|
-
name: 'eslint-plugin-astro-stylistic-override',
|
|
27
|
-
files: [ '**/*.astro' ],
|
|
28
|
-
rules: {
|
|
29
|
-
'@stylistic/jsx-one-expression-per-line': 'off',
|
|
30
|
-
'@stylistic/jsx-curly-brace-presence': 'off',
|
|
31
|
-
'import-x/unambiguous': 'off'
|
|
32
|
-
}
|
|
33
|
-
}
|
|
34
|
-
);
|