linted 16.12.1 → 16.12.2-rc.1

package/src/rules/json.ts

@@ -0,0 +1,4 @@
+import EnableX from "./json/enable_x.js";
+import Enable from "./json/enable.js";
+
+export default [EnableX, Enable] as const;

package/src/rules/jsonc/override.ts

@@ -0,0 +1,16 @@
+import { override, level, state } from "../_strings.js";
+
+const { ERROR, OFF } = level,
+{ AlwaysMultiline } = state;
+
+export default [
+  override,
+  {
+    // https://ota-meshi.github.io/eslint-plugin-jsonc/rules/#jsonc-rules
+    "jsonc/no-comments": OFF /* comments ARE allowed in JSONC */,
+    "jsonc/comma-dangle": [
+      ERROR,
+      AlwaysMultiline,
+    ] /* trailing commas ARE allowed in JSONC */,
+  },
+] as const;

package/src/rules/jsonc.ts

@@ -0,0 +1,4 @@
+import Json from "./json.js";
+import OverrideJson from "./jsonc/override.js";
+
+export default [...Json, OverrideJson] as const;

package/src/rules/md/enable.ts

@@ -0,0 +1,15 @@
+import { enable, level } from "../_strings.js";
+
+const { ERROR } = level;
+
+export default [
+  enable,
+  {
+    "markdownlint/md003": [
+      ERROR,
+      {
+        style: "atx",
+      },
+    ],
+  },
+] as const;

package/src/rules/md.ts

@@ -0,0 +1,3 @@
+import Enable from "./md/enable.js";
+
+export default [Enable] as const;

package/src/rules/mocha/enable.ts

@@ -0,0 +1,3 @@
+import { enable } from "../_strings.js";
+
+export default [enable, {}] as const;

package/src/rules/mocha/enable_recommended.ts

@@ -0,0 +1,9 @@
+import { recommended } from "../_strings.js";
+import mocha from "eslint-plugin-mocha";
+
+// REMOVE shared config once manually configured
+export default [
+  recommended,
+  (mocha.configs.flat.recommended as Record<"rules", Record<string, "off">>)
+    .rules,
+] as const;

package/src/rules/mocha.ts

@@ -0,0 +1,4 @@
+import Recommended from "./mocha/enable_recommended.js";
+import Enable from "./html/enable.js";
+
+export default [Recommended, Enable] as const;

package/src/rules/svelte/disable.ts

@@ -0,0 +1,9 @@
+import { disable, OFF } from "../_strings.js";
+
+export default [
+  disable,
+  {
+    // http://sveltejs.github.io/eslint-plugin-svelte/rules/#extension-rules
+    "no-inner-declarations": OFF /* also disabled in SvelteDisableConflictJs per Svelte base config */,
+  },
+] as const;

package/src/rules/svelte/disable_js.ts

@@ -0,0 +1,11 @@
+import { OFF } from "../_strings.js";
+
+export default [
+  "disable-js",
+  {
+    // Recommended by Svelte base config
+    // http://github.com/sveltejs/eslint-plugin-svelte/tree/main/src/configs/flat
+    "no-inner-declarations": OFF /* ESLint Core */,
+    "no-self-assign": OFF /* ESLint Core */,
+  },
+] as const;

package/src/rules/svelte/disable_ts.ts

@@ -0,0 +1,9 @@
+import { OFF } from "../_strings.js";
+
+export default [
+  "disable-ts",
+  {
+    "no-unused-vars": OFF /* ESLint Core */,
+    "@typescript-eslint/no-unused-vars": OFF /* TS Extension */,
+  },
+] as const;

package/src/{base/rules/presets/svelte/Enable.ts → rules/svelte/enable.ts}

@@ -1,18 +1,16 @@
-import { Strings, type RuleEntry } from "../index.js";
+import { enable, level, state } from "../_strings.js";
 
-const {
-  Id: { Enable },
-  Level: { ERROR, OFF },
-  State: {
-    NEVER,
-    ALWAYS,
-    BELOW,
-    DOUBLE,
-  },
-} = Strings;
+const
+{ ERROR, OFF } = level,
+{
+  Never,
+  Always,
+  Below,
+  Double,
+} = state;
 
 export default [
-  Enable,
+  enable,
   {
     // #region Possible Errors
     // http://sveltejs.github.io/eslint-plugin-svelte/rules/#possible-errors
@@ -26,13 +24,28 @@ export default [
     "svelte/no-export-load-in-svelte-module-in-kit-pages": ERROR,
     "svelte/no-not-function-handler": ERROR,
     "svelte/no-object-in-text-mustaches": ERROR,
-    "svelte/no-reactive-reassign": [ERROR, { props: true }],
+    "svelte/no-reactive-reassign": [
+      ERROR,
+      {
+        props: true,
+      },
+    ],
     "svelte/no-shorthand-style-property-overrides": ERROR,
     "svelte/no-store-async": ERROR,
-    "svelte/no-unknown-style-directive-property": [ERROR, { ignorePrefixed: true }],
+    "svelte/no-unknown-style-directive-property": [
+      ERROR,
+      {
+        ignorePrefixed: true,
+      },
+    ],
     "svelte/require-store-callbacks-use-set-param": ERROR,
     "svelte/require-store-reactive-access": ERROR,
-    "svelte/valid-compile": [ERROR, { ignoreWarnings: false }],
+    "svelte/valid-compile": [
+      ERROR,
+      {
+        ignoreWarnings: false,
+      },
+    ],
     "svelte/valid-prop-names-in-kit-pages": ERROR,
 
     // #endregion
@@ -40,7 +53,13 @@ export default [
     // #region Security
     // http://sveltejs.github.io/eslint-plugin-svelte/rules/#security-vulnerability
     "svelte/no-at-html-tags": ERROR,
-    "svelte/no-target-blank": [ERROR, { allowReferrer: false, enforceDynamicLinks: ALWAYS }],
+    "svelte/no-target-blank": [
+      ERROR,
+      {
+        allowReferrer: false,
+        enforceDynamicLinks: Always,
+      },
+    ],
 
     // #endregion
 
@@ -52,20 +71,41 @@ export default [
         enforceScriptPresent: true,
         enforceStylePresent: false,
         script: ["ts"],
-        style: ["css", "scss"],
+        style: [
+          "css",
+          "scss",
+        ],
       },
     ], /* INVESTIGATE: */
-    "svelte/button-has-type": [ERROR, { button: true, submit: true, reset: true }],
+    "svelte/button-has-type": [
+      ERROR,
+      {
+        button: true,
+        submit: true,
+        reset: true,
+      },
+    ],
     "svelte/no-at-debug-tags": ERROR,
     "svelte/no-ignored-unsubscribe": ERROR,
     "svelte/no-immutable-reactive-statements": ERROR,
-    "svelte/no-inline-styles": [ERROR, { allowTransitions: false }],
+    "svelte/no-inline-styles": [
+      ERROR,
+      {
+        allowTransitions: false,
+      },
+    ],
     "svelte/no-reactive-functions": ERROR,
     "svelte/no-reactive-literals": ERROR,
     "svelte/no-svelte-internal": ERROR,
     "svelte/no-unused-class-name": ERROR, /* BUG: ESLint error if array empty */
     "svelte/no-unused-svelte-ignore": ERROR,
-    "svelte/no-useless-mustaches": [ERROR, { ignoreIncludesComment: false, ignoreStringEscape: false }],
+    "svelte/no-useless-mustaches": [
+      ERROR,
+      {
+        ignoreIncludesComment: false,
+        ignoreStringEscape: false,
+      },
+    ],
     "svelte/prefer-destructured-store-props": ERROR,
     "svelte/require-each-key": ERROR,
     "svelte/require-event-dispatcher-types": ERROR,
@@ -78,38 +118,91 @@ export default [
     // #region Style
     // http://sveltejs.github.io/eslint-plugin-svelte/rules/#stylistic-issues
     "svelte/derived-has-same-inputs-outputs": ERROR,
-    "svelte/first-attribute-linebreak": [ERROR, { multiline: BELOW, singleline: BELOW }],
-    "svelte/html-closing-bracket-spacing": [ERROR, { startTag: NEVER, endTag: NEVER, selfClosingTag: ALWAYS }],
-    "svelte/html-quotes": [ERROR, { prefer: DOUBLE, dynamic: { quoted: false, avoidInvalidUnquotedInHTML: true } }],
+    "svelte/first-attribute-linebreak": [
+      ERROR,
+      {
+        multiline: Below,
+        singleline: Below,
+      },
+    ],
+    "svelte/html-closing-bracket-spacing": [
+      ERROR,
+      {
+        startTag: Never,
+        endTag: Never,
+        selfClosingTag: Always,
+      },
+    ],
+    "svelte/html-quotes": [
+      ERROR,
+      {
+        prefer: Double,
+        dynamic: {
+          quoted: false,
+          avoidInvalidUnquotedInHTML: true,
+        },
+      },
+    ],
     "svelte/html-self-closing": [
       ERROR,
       {
-        "void": ALWAYS /* or NEVER or "ignore" */,
-        normal: ALWAYS /* or NEVER or "ignore" */,
-        component: ALWAYS /* or NEVER or "ignore" */,
-        svelte: ALWAYS /* or NEVER or "ignore" */,
+        "void": Always /* or NEVER or "ignore" */,
+        normal: Always /* or NEVER or "ignore" */,
+        component: Always /* or NEVER or "ignore" */,
+        svelte: Always /* or NEVER or "ignore" */,
+      },
+    ],
+    "svelte/indent": [
+      ERROR,
+      {
+        indent: 2,
+        switchCase: 1,
+        alignAttributesVertically: true,
+      },
+    ],
+    "svelte/max-attributes-per-line": [
+      ERROR,
+      {
+        multiline: 1,
+        singleline: 1,
       },
     ],
-    "svelte/indent": [ERROR, { indent: 2, switchCase: 1, alignAttributesVertically: true }],
-    "svelte/max-attributes-per-line": [ERROR, { multiline: 1, singleline: 1 }],
     "svelte/mustache-spacing": [
       ERROR,
       {
-        textExpressions: NEVER /* or ALWAYS */,
-        attributesAndProps: NEVER /* or ALWAYS */,
-        directiveExpressions: NEVER /* or ALWAYS */,
-        tags: { openingBrace: NEVER /* or ALWAYS */, closingBrace: NEVER /* or ALWAYS or "always-after-expression" */ },
+        textExpressions: Never /* or ALWAYS */,
+        attributesAndProps: Never /* or ALWAYS */,
+        directiveExpressions: Never /* or ALWAYS */,
+        tags: {
+          openingBrace: Never /* or ALWAYS */,
+          closingBrace: Never /* or ALWAYS or "always-after-expression" */,
+        },
       },
     ],
     "svelte/no-extra-reactive-curlies": ERROR,
     "svelte/no-restricted-html-elements": OFF,
     "svelte/no-spaces-around-equal-signs-in-attribute": ERROR,
-    "svelte/prefer-class-directive": [ERROR, { prefer: "empty" }],
+    "svelte/prefer-class-directive": [
+      ERROR,
+      {
+        prefer: "empty",
+      },
+    ],
     "svelte/prefer-style-directive": ERROR,
-    "svelte/shorthand-attribute": [ERROR, { prefer: ALWAYS }],
-    "svelte/shorthand-directive": [ERROR, { prefer: ALWAYS }],
-    "svelte/sort-attributes": OFF, // INVESTIGATE:
-    "svelte/spaced-html-comment": [ERROR, ALWAYS],
+    "svelte/shorthand-attribute": [
+      ERROR,
+      {
+        prefer: Always,
+      },
+    ],
+    "svelte/shorthand-directive": [
+      ERROR,
+      {
+        prefer: Always,
+      },
+    ],
+    "svelte/sort-attributes": OFF /* INVESTIGATE: */,
+    "svelte/spaced-html-comment": [ERROR, Always],
 
     // #endregion
 
@@ -121,10 +214,15 @@ export default [
 
     // #region System (REQUIRED)
     // http://sveltejs.github.io/eslint-plugin-svelte/rules/#system
-    "svelte/comment-directive": [ERROR, { reportUnusedDisableDirectives: true }],
+    "svelte/comment-directive": [
+      ERROR,
+      {
+        reportUnusedDisableDirectives: true,
+      },
+    ],
     "svelte/system": ERROR,
 
     // #endregion
 
   },
-] as const satisfies RuleEntry;
+] as const;

package/src/rules/svelte/enable_x.ts

@@ -0,0 +1,20 @@
+import { enable_x, level, state } from "../_strings.js";
+
+const
+{ ERROR, OFF } = level,
+{ Allow, Both } = state;
+
+export default [
+  enable_x,
+  {
+    // http://sveltejs.github.io/eslint-plugin-svelte/rules/#extension-rules
+    "svelte/no-inner-declarations": [
+      ERROR,
+      Both,
+      {
+        blockScopedFunctions: Allow,
+      },
+    ] /* Same options as in JsEnable */,
+    "svelte/no-trailing-spaces": OFF /* Only applies to HTML comments, which I don't intend to use: http://sveltejs.github.io/eslint-plugin-svelte/rules/no-trailing-spaces/ */,
+  },
+] as const;

package/src/rules/svelte.ts

@@ -0,0 +1,15 @@
+import TsRuleset from "./ts.js";
+import DisableJS from "./svelte/disable_js.js";
+import DisableTS from "./svelte/disable_ts.js";
+import DisableX from "./svelte/disable.js";
+import EnableX from "./svelte/enable_x.js";
+import Enable from "./svelte/enable.js";
+
+export default [
+  ...TsRuleset,
+  DisableJS,
+  DisableTS,
+  DisableX,
+  EnableX,
+  Enable,
+] as const;

package/src/{base/rules/presets/ts/DisableCompiler.ts → rules/ts/disable.ts}

@@ -1,10 +1,9 @@
-import { Strings, type RuleEntry } from "../index.js";
-
-const { Level: { OFF } } = Strings;
+import { disable, OFF } from "../_strings.js";
 
 export default [
-  "disable-compiler",
+  disable,
   {
+    // Handled by TypeScript compiler
     "constructor-super": OFF,
     "no-const-assign": OFF,
     "no-dupe-args": OFF,
@@ -20,4 +19,4 @@ export default [
     "consistent-return": OFF,
     "no-redeclare": OFF,
   },
-] as const satisfies RuleEntry;
+] as const;

package/src/{base/rules/presets/ts/DisableX.ts → rules/ts/disable_x.ts}

@@ -1,11 +1,9 @@
-import { Strings, type RuleEntry } from "../index.js";
-
-const { Id: { DisableX }, Level: { OFF } } = Strings;
+import { disable_x, OFF } from "../_strings.js";
 
 export default [
-  DisableX,
+  disable_x,
   {
-    // https://typescript-eslint.io/rules/?=extension-xdeprecated#rules ]
+    // https://typescript-eslint.io/rules/?=extension-xdeprecated#rules
     "class-methods-use-this": OFF,
     "consistent-return": OFF,
     "default-param-last": OFF,
@@ -31,4 +29,4 @@ export default [
     "prefer-promise-reject-errors": OFF,
     "require-await": OFF,
   },
-] as const satisfies RuleEntry;
+] as const;