lint-rules-alvin 1.1.4 → 1.2.0

package/eslint/configs/index.d.ts

@@ -9,4 +9,5 @@ export declare const json: Linter.Config[];
 export declare const markdown: Linter.Config;
 export declare const reactHooks: Linter.Config;
 export declare const stylistic: Linter.Config;
+export declare const sveltekit: Linter.Config[];
 export declare const typescript: Linter.Config;

package/eslint/configs/index.js

@@ -7,4 +7,5 @@ export { json } from './json.js';
 export { markdown } from './markdown.js';
 export { reactHooks } from './reactHooks.js';
 export { stylistic } from './stylistic.js';
+export { sveltekit } from './sveltekit.js';
 export { typescript } from './typescript.js';

package/eslint/configs/sveltekit.js

@@ -0,0 +1,209 @@
+import eslintPluginSvelte from 'eslint-plugin-svelte';
+
+/**
+ * The ESLint `stylistic` config. Extends `configs.recommended` and overrides all rules.
+ *
+ * @type {import('eslint').Linter.Config[]}
+ */
+export const sveltekit = [
+	...eslintPluginSvelte.configs['flat/base'],
+	{
+		files: [
+			'**/*.svelte',
+			'**/*.svelte.ts',
+			'**/*.svelte.js'
+		],
+		rules: {
+
+			// Svelte-specific
+			'svelte/infinite-reactive-loop': 'warn',
+			'svelte/no-dom-manupulating': 'warn',
+			'svelte/no-dupe-else-if-blocks': 'error',
+			'svelte/no-dupe-else-on-directives': 'error',
+			'svelte/no-dupe-style-properties': 'error',
+			'svelte/no-dupe-use-directives': 'error',
+			'svelte/no-not-function-handler': 'error',
+			'svelte/no-object-in-text-mustaches': 'error',
+			'svelte/no-raw-special-elements': 'error',
+			'svelte/no-reactive-reassign': 'error',
+			'svelte/no-shorthand-style-property-overrides': 'warn',
+			'svelte/no-store-async': 'error',
+			'svelte/no-top-level-browser-globals': 'error',
+			'svelte/no-unknown-style-directive-property': 'error',
+			'svelte/prefer-svelte-reactivity': 'error',
+			'svelte/require-store-callbacks-use-set-param': 'error',
+			'svelte/require-store-reactive-access': 'error',
+			'svelte/valid-compile': 'off',
+			'svelte/valid-style-parse': 'error',
+
+			// Security
+			'svelte/no-at-html-tags': 'warn',
+			'svelte/no-target-blank': 'error',
+
+			// Best practices
+			'svelte/block-lang': [
+				'error',
+				{
+					enforceScriptPresent: false,
+					enforceStylePresent: false,
+					script: [
+						'ts',
+						'js'
+					]
+				}
+			],
+			'svelte/button-has-type': 'off',
+			'svelte/no-add-event-listener': 'error',
+			'svelte/no-at-debug-tags': 'warn',
+			'svelte/no-ignored-unsubscribe': 'warn',
+			'svelte/no-immutable-reactive-statements': 'error',
+			'svelte/no-inline-styles': 'error',
+			'svelte/no-inspect': 'warn',
+			'svelte/no-reactive-functions': 'error',
+			'svelte/no-reactive-literals': 'error',
+			'svelte/no-svelte-internal': 'error',
+			'svelte/no-unnecessary-state-wrap': 'error',
+			'svelte/no-unused-class-name': 'error',
+			'svelte/no-unused-props': 'error',
+			'svelte/no-unused-svelte-ignore': 'error',
+			'svelte/no-useless-children-snippet': 'error',
+			'svelte/no-useless-mustaches': 'error',
+			'prefer-const': 'off',
+			'svelte/prefer-const': 'error',
+			'svelte/prefer-destructed-store-props': 'error',
+			'svelte/prefer-writable-derived': 'error',
+			'svelte/require-each-key': 'error',
+			'svelte/require-event-dispatcher-types': 'error',
+			'svelte/require-optimized-style-attribute': 'error',
+			'svelte/require-store-init': 'error',
+			'svelte/valid-each-key': 'error',
+
+			// Stylistic
+			'svelte/consistent-selector-style': 'off',
+			'svelte/derived-has-same-inputs-outputs': 'error',
+			'svelte/first-attribute-linebreak': [
+				'error',
+				{
+					singleline: 'beside',
+					multiline: 'below'
+				}
+			],
+			'svelte/html-closing-bracket-new-line': [
+				'error',
+				{
+					singleline: 'never',
+					multiline: 'always',
+					selfClosingTag: {
+						singleline: 'never',
+						multiline: 'always'
+					}
+				}
+			],
+			'svelte/html-closing-bracket-spacing': [
+				'error',
+				{
+					startTag: 'never',
+					endTag: 'never',
+					selfClosingTag: 'always'
+				}
+			],
+			'svelte/html-quotes': [
+				'error',
+				{
+					prefer: 'double',
+					dynamic: {
+						quoted: false,
+						avoidInvalidUnquotedInHTML: false
+					}
+				}
+			],
+			'svelte/html-self-closing': [
+				'error',
+				'default'
+			],
+			'indent': 'off', // eslint-disable-line @stylistic/quote-props
+			'@stylistic/indent': 'off',
+			'svelte/indent': [
+				'error',
+				{
+					indent: 'tab',
+					switchCase: 1,
+					alignAttributesVertically: false
+				}
+			],
+			'svelte/max-attributes-per-line': [
+				'error',
+				{
+					singleline: 1,
+					multiline: 1
+				}
+			],
+			'svelte/mustache-spacing': [
+				'error',
+				{
+					textExpressions: 'never',
+					attributesAndProps: 'never',
+					directiveExpressions: 'never',
+					tags: {
+						openingBrace: 'never',
+						closingBrace: 'never'
+					}
+				}
+			],
+			'svelte/no-extra-reactive-curlies': 'error',
+			'svelte/no-restricted-html-elements': 'off', // Allow the user to override
+			'svelte/no-spaces-around-equal-signs-in-attribute': 'error',
+			'svelte/prefer-class-directive': 'error',
+			'svelte/prefer-style-directive': 'error',
+			'svelte/require-event-prefix': 'off',
+			'svelte/shorthand-attribute': [
+				'error',
+				{ prefer: 'always' }
+			],
+			'svelte/shorthand-directive': [
+				'off',
+				{ prefer: 'never' }
+			],
+			'svelte/sort-attributes': 'error',
+			'svelte/spaced-html-comment': [
+				'error',
+				'always'
+			],
+
+			// Extensions
+			'svelte/no-inner-declarations': [
+				'error',
+				'functions',
+				{ blockScopedFunctions: 'allow' }
+			],
+			'no-trailing-spaces': 'off',
+			'svelte/no-trailing-spaces': [
+				'error',
+				{
+					skipBlankLines: false,
+					ignoreComments: false
+				}
+			],
+
+			// SvelteKit
+			'svelte/no-export-load-in-svelte-module-in-kit-pages': 'error',
+			'svelte/no-navigation-without-resolve': [
+				'error',
+				{
+					ignoreGoto: false,
+					ignoreLinks: false,
+					ignorePushState: false,
+					ignoreReplaceState: false
+				}
+			],
+			'svelte/valid-prop-names-in-kit-pages': 'error',
+
+			// System
+			'svelte/comment-directive': [
+				'error',
+				{ reportUnusedDisableDirectives: true }
+			],
+			'svelte/system': 'error' // Keep this on!
+		}
+	}
+];

package/eslint/presets/index.d.ts

@@ -1,14 +1,11 @@
 import type { Linter } from 'eslint';
 
 /**
- * ESLint preset for Astro/React with JavaScript.
- *
- * This package uses ESLint's flat config format, so the preset is an array
- * of flat-config items.
+ * ESLint preset for Astro/React with TypeScript.
  */
-export declare const astroReact: readonly Linter.Config[];
+export declare const astroReactTs: readonly Linter.Config[];
 
 /**
- * ESLint preset for Astro/React with TypeScript.
+ * ESLint preset for SvelteKit with TypeScript.
  */
-export declare const astroReactTs: readonly Linter.Config[];
+export declare const sveltekitTs: readonly Linter.Config[];

package/eslint/presets/index.js

@@ -1,2 +1,2 @@
-export { astroReact } from './astroReact';
-export { astroReactTs } from './astroReactTs';
+export { astroReactTs } from './astroReactTs.js';
+export { sveltekitTs } from './sveltekitTs.js';

package/eslint/presets/sveltekitTs.js

@@ -0,0 +1,25 @@
+import { defineConfig } from 'eslint/config';
+import { importX as eslintPluginImportX } from 'eslint-plugin-import-x';
+import {
+	base,
+	importX,
+	sveltekit,
+	stylistic,
+	custom,
+	typescript
+} from '../configs/index.js';
+
+/**
+ * The `ESLint` PocketBase/SvelteKit config with typescript.
+ */
+export const sveltekitTs = defineConfig(
+	base,
+	sveltekit,
+	typescript,
+	{
+		...eslintPluginImportX.configs['flat/typescript'],
+		...importX
+	},
+	stylistic,
+	custom
+);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lint-rules-alvin",
-  "version": "1.1.4",
+  "version": "1.2.0",
   "description": "My own personal linting ruleset for a bunch of different plugins. Includes a few custom rules. Used in a few of my repos.",
   "keywords": [
     "eslint"
@@ -43,7 +43,9 @@
     "eslint-plugin-jsonc": "^2.21.0",
     "eslint-plugin-markdown": "^5.1.0",
     "eslint-plugin-react-hooks": "^7.0.0",
+    "eslint-plugin-svelte": "^3.12.4",
     "globals": "^16.4.0",
+    "svelte": "^5.40.2",
     "typescript": "^5.9.3",
     "typescript-eslint": "^8.46.1"
   },
@@ -59,6 +61,9 @@
     "eslint-plugin-jsonc": "^2.21.0",
     "eslint-plugin-markdown": "^5.1.0",
     "eslint-plugin-react-hooks": "^7.0.0",
+    "eslint-plugin-svelte": "^3.12.4",
+    "globals": "^16.4.0",
+    "svelte": "^5.40.2",
     "typescript": "^5.9.3",
     "typescript-eslint": "^8.46.0"
   },
@@ -96,6 +101,15 @@
     "eslint-plugin-react-hooks": {
       "optional": true
     },
+    "eslint-plugin-svelte": {
+      "optional": true
+    },
+    "globals": {
+      "optional": true
+    },
+    "svelte": {
+      "optional": true
+    },
     "typescript": {
       "optional": true
     },

package/eslint/presets/astroReact.js

@@ -1,34 +0,0 @@
-import { defineConfig } from 'eslint/config';
-import { importX as eslintPluginImportX } from 'eslint-plugin-import-x';
-import {
-	base,
-	astro,
-	importX,
-	reactHooks,
-	stylistic,
-	custom
-} from '../configs/index.js';
-
-/**
- * The `ESLint` Astro/React config with typescript.
- */
-export const astroReact = defineConfig(
-	base,
-	astro,
-	{
-		...eslintPluginImportX.configs['flat/react'],
-		...importX
-	},
-	reactHooks,
-	stylistic,
-	custom,
-	{
-		name: 'eslint-plugin-astro-stylistic-override',
-		files: [ '**/*.astro' ],
-		rules: {
-			'@stylistic/jsx-one-expression-per-line': 'off',
-			'@stylistic/jsx-curly-brace-presence': 'off',
-			'import-x/unambiguous': 'off'
-		}
-	}
-);