@eslinted/defaults 21.1.0 → 21.1.2

package/.github/workflows/RELEASE.yml

@@ -18,10 +18,10 @@ jobs:
     steps:
       - name: Checkout
         id: checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v6
       - name: Install Node
         id: node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v6
         with:
           node-version: 24
           registry-url: https://registry.npmjs.org

package/package.json

@@ -3,7 +3,7 @@
   "$help": "https://docs.npmjs.com/cli/configuring-npm/package-json",
   "$schema": "https://json.schemastore.org/package",
   "name": "@eslinted/defaults",
-  "version": "21.1.0",
+  "version": "21.1.2",
   "repository": "github:jimmy-zhening-luo/linted-defaults",
   "description": "Default scopes for `linted`",
   "keywords": [],

package/src/rules/css.ts

@@ -1,6 +1,6 @@
 import { State } from "./_state";
 
-// DOC: https://github.com/eslint/css/tree/main/docs/rules
+//@doc https://github.com/eslint/css/tree/main/docs/rules
 export default {
   rules: {
     "css/font-family-fallbacks": State.WARN,
@@ -11,12 +11,12 @@ export default {
     "css/no-invalid-at-rule-placement": State.ON,
     "css/no-invalid-at-rules": State.ON,
     "css/no-invalid-named-grid-areas": State.ON,
-    // "css/no-invalid-properties": State.OFF /* BUG: this is still way too broken, it doens't recognize actual rules. in future if it recognizes actual rules, turn on allow unknown values btw */,
+    // "css/no-invalid-properties": State.OFF /* @BUG this is still way too broken, it doens't recognize actual rules. in future if it recognizes actual rules, turn on allow unknown values btw */,
     "css/no-unmatchable-selectors": State.WARN,
     "css/prefer-logical-properties": State.WARN,
     "css/relative-font-units": State.WARN,
     // "css/selector-complexity": State.OFF /* @preference: or rather too lazy to configure because its default state is literally infinitely permissable */,
     "css/use-baseline": State.WARN,
-    // "css/use-layers": State.OFF /* not recommended by docs: https://github.com/eslint/css?tab=readme-ov-file#rules | BUG: ESLint config inspector: css/use-layers: Invalid rule has no description [INVALID] */,
+    // "css/use-layers": State.OFF /* not recommended by docs: https://github.com/eslint/css?tab=readme-ov-file#rules | @BUG ESLint config inspector: css/use-layers: Invalid rule has no description [INVALID] */,
   } as const,
 };

package/src/rules/html.ts

@@ -1,10 +1,10 @@
 import { State } from "./_state";
 
-// DOC: https://html-eslint.org/docs/rules
+//@doc https://html-eslint.org/docs/rules
 export default {
   rules: {
-    // #region BEST PRACTICE
-    // DOC: https://html-eslint.org/docs/rules#best-practice
+    //#region BEST PRACTICE
+    //@doc https://html-eslint.org/docs/rules#best-practice
     "html/css-no-empty-blocks": State.WARN,
     "html/head-order": State.ON,
     // "html/max-element-depth": State.OFF /* @preference: simply do not care */,
@@ -33,19 +33,19 @@ export default {
     "html/require-li-container": State.WARN,
     "html/require-meta-charset": State.WARN,
     "html/use-baseline": State.WARN,
-    // #endregion
+    //#endregion
 
-    // #region SEO
-    // DOC: https://html-eslint.org/docs/rules#seo
+    //#region SEO
+    //@doc https://html-eslint.org/docs/rules#seo
     "html/no-multiple-h1": State.WARN,
     "html/require-lang": State.WARN,
     // "html/require-meta-description": State.OFF /* @CONFLICT: breaks Svelte which leaves meta descriptions to route leaves */,
     // "html/require-open-graph-protocol": State.OFF /* @CONFLICT: idk what this is, also it probably breaks Svelte */,
     // "html/require-title": State.OFF /* @CONFLICT: breaks Svelte which leaves title to route leaves */,
-    // #endregion
+    //#endregion
 
-    // #region ACCESSIBILITY
-    // DOC: https://html-eslint.org/docs/rules#accessibility
+    //#region ACCESSIBILITY
+    //@doc https://html-eslint.org/docs/rules#accessibility
     "html/no-abstract-roles": State.WARN,
     "html/no-accesskey-attrs": State.WARN,
     "html/no-aria-hidden-body": State.WARN,
@@ -62,10 +62,10 @@ export default {
     "html/require-img-alt": State.WARN,
     "html/require-input-label": State.WARN,
     "html/require-meta-viewport": State.WARN,
-    // #endregion
+    //#endregion
 
-    // #region STYLE
-    // DOC: https://html-eslint.org/docs/rules#style
+    //#region STYLE
+    //@doc https://html-eslint.org/docs/rules#style
     // "html/attrs-newline": State.OFF,
     // "html/element-newline": State.OFF,
     // "html/id-naming-convention": State.OFF,
@@ -77,6 +77,6 @@ export default {
     // "html/no-trailing-spaces": State.OFF,
     // "html/quotes": State.OFF,
     // "html/sort-attrs": State.OFF,
-    // #endregion
+    //#endregion
   } as const,
 };

package/src/rules/js.ts

@@ -1,15 +1,15 @@
 import { State } from "./_state";
 
-// DOC: https://eslint.org/docs/latest/rules/
+//@doc https://eslint.org/docs/latest/rules/
 export default {
   rules: {
-    // #region PROBLEMS
-    // DOC: https://eslint.org/docs/latest/rules/#possible-problems
+    //#region PROBLEMS
+    //@doc https://eslint.org/docs/latest/rules/#possible-problems
     "array-callback-return": [
       State.ON,
       {
         checkForEach: true,
-        allowVoid: true /* INFO: only works if checkForEach */,
+        allowVoid: true /* @info only works if checkForEach */,
       },
     ],
     "constructor-super": State.ON /* @tsc */,
@@ -27,7 +27,7 @@ export default {
       {
         checkLoops: "all",
       },
-    ] /* BUG: incorrect option description (claims "all" is ok): https://eslint.org/docs/latest/rules/no-constant-condition */,
+    ] /* @BUG incorrect option description (claims "all" is ok): https://eslint.org/docs/latest/rules/no-constant-condition */,
     "no-constructor-return": State.ON,
     "no-control-regex": State.ON,
     "no-debugger": State.ON,
@@ -114,9 +114,9 @@ export default {
         requireStringLiterals: true,
       },
     ],
-    // #endregion
+    //#endregion
 
-    // #region SUGGESTIONS
+    //#region SUGGESTIONS
     // [ https://eslint.org/docs/latest/rules/#suggestions ]
     "accessor-pairs": [
       State.ON,
@@ -144,7 +144,7 @@ export default {
     // "func-name-matching": State.OFF,
     // "func-names": State.OFF,
     // "func-style": State.OFF /* supports TypeScript */,
-    // "grouped-accessor-pairs": State.OFF /* BUG: requires adjacency, not as described in documentation */,
+    // "grouped-accessor-pairs": State.OFF /* @BUG requires adjacency, not as described in documentation */,
     // "guard-for-in": State.OFF /* @preference: not helpful because using for-in on non-literal objects is bad practice and this rule doesn't guard against that anyway, while adding a pointless check to known object literals */,
     // "id-denylist": State.OFF,
     // "id-length": State.OFF,
@@ -338,15 +338,15 @@ export default {
     // "symbol-description": State.OFF,
     "vars-on-top": State.WARN,
     yoda: State.WARN,
-    // #endregion
+    //#endregion
 
-    // #region FORMATTING
+    //#region FORMATTING
     // [ https://eslint.org/docs/latest/rules/#layout--formatting ]
     "unicode-bom": State.WARN,
-    // #endregion
+    //#endregion
 
-    // #region STYLISTIC
-    // DOC: https://eslint.style/rules
+    //#region STYLISTIC
+    //@doc https://eslint.style/rules
 
     "stylistic/eol-last": State.WARN,
     "stylistic/linebreak-style": State.WARN,
@@ -394,7 +394,7 @@ export default {
     "stylistic/function-paren-newline": [
       State.WARN,
       "multiline-arguments",
-    ] /* BUG: https://github.com/eslint-stylistic/eslint-stylistic/issues/290 */,
+    ] /* @BUG https://github.com/eslint-stylistic/eslint-stylistic/issues/290 */,
     "stylistic/generator-star-spacing": State.WARN,
     "stylistic/implicit-arrow-linebreak": State.WARN,
     "stylistic/indent": [
@@ -488,7 +488,7 @@ export default {
       {
         max: 1,
         maxBOF: 0,
-        maxEOF: 0 /* INFO: combine with `eol-last` to ensure file still ends with a single line-break (eol character) */,
+        maxEOF: 0 /* @info combine with `eol-last` to ensure file still ends with a single line-break (eol character) */,
       },
     ],
     "stylistic/no-tabs": State.WARN,
@@ -529,7 +529,7 @@ export default {
         allowSingleLineBlocks: true,
       },
     ],
-    // "stylistic/padding-line-between-statements": State.OFF /* BUG: when in editor, always removes too many lines and ends up crushing two lines into one statement, which then cannot be autofixed */,
+    // "stylistic/padding-line-between-statements": State.OFF /* @BUG when in editor, always removes too many lines and ends up crushing two lines into one statement, which then cannot be autofixed */,
     "stylistic/quote-props": [
       State.WARN,
       "as-needed",
@@ -596,7 +596,7 @@ export default {
             after: true,
           },
         },
-      } /* BUG: non-overriden rule affects all type annotations (arrows, "as" keyword, "satisfies" keyword) except colons */,
+      } /* @BUG non-overriden rule affects all type annotations (arrows, "as" keyword, "satisfies" keyword) except colons */,
     ],
     "stylistic/type-generic-spacing": State.WARN,
     "stylistic/type-named-tuple-spacing": State.WARN,

package/src/rules/json.ts

@@ -1,6 +1,6 @@
 import { State } from "./_state";
 
-// DOC: https://ota-meshi.github.io/eslint-plugin-jsonc/rules/
+//@doc https://ota-meshi.github.io/eslint-plugin-jsonc/rules/
 export default {
   rules: {
     // "json/no-duplicate-keys": State.OFF /* jsonc/no-dupe-keys */,
@@ -32,7 +32,7 @@ export default {
     // "jsonc/sort-keys": State.OFF /* ESLint: json/sort-keys */,
     // "jsonc/valid-json-number": State.OFF,
     // "jsonc/vue-custom-block/no-parsing-error": State.OFF,
-    // "jsonc/array-bracket-newline": State.OFF /* BUG: DO NOT CONFIGURE */,
+    // "jsonc/array-bracket-newline": State.OFF /* @BUG DO NOT CONFIGURE */,
     "jsonc/array-bracket-spacing": State.WARN,
     "jsonc/array-element-newline": [
       State.WARN,

package/src/rules/svelte.ts

@@ -1,11 +1,11 @@
 import { State } from "./_state";
 
-// DOC: https://github.com/sveltejs/eslint-plugin-svelte/blob/ca37fbbe82d01019946e12eeb1b57c2a5736c018/packages/eslint-plugin-svelte/src/configs/base.ts
+//@doc https://github.com/sveltejs/eslint-plugin-svelte/blob/ca37fbbe82d01019946e12eeb1b57c2a5736c018/packages/eslint-plugin-svelte/src/configs/base.ts
 export default {
   rules: {
-    // #region DISABLE
+    //#region DISABLE
     // ESLint core rules known to cause problems with `.svelte`.
-    "no-inner-declarations": State.OFF /* The AST generated by svelte-eslint-parser will false positive because the root node of the script is not `Program`. INFO: also extended by svelte  */,
+    "no-inner-declarations": State.OFF /* The AST generated by svelte-eslint-parser will false positive because the root node of the script is not `Program`. @info also extended by svelte  */,
     "no-self-assign": State.OFF /* Self assign is one of way to update reactive value in Svelte. */,
     "no-unused-vars": State.OFF /* breaks use of svelte global */,
     "ts/no-unused-vars": State.OFF /* breaks use of svelte global */,
@@ -17,20 +17,20 @@ export default {
     "stylistic/member-delimiter-style": State.OFF,
     "stylistic/object-curly-spacing": State.OFF,
     "stylistic/semi": State.OFF,
-    // #endregion
+    //#endregion
 
-    // #region EXTEND
-    // DOC: http://sveltejs.github.io/eslint-plugin-svelte/rules/#extension-rules
+    //#region EXTEND
+    //@doc http://sveltejs.github.io/eslint-plugin-svelte/rules/#extension-rules
     "svelte/no-inner-declarations": [
       State.ON,
       "both",
     ] /* same options: no-inner-declarations */,
     /* @OVERRIDE */ "stylistic/no-trailing-spaces": State.OFF /* replaces `no-trailing-spaces` */,
     // "svelte/no-trailing-spaces": State.OFF,
-    // #endregion
+    //#endregion
 
-    // #region ERRORS
-    // DOC: http://sveltejs.github.io/eslint-plugin-svelte/rules/#possible-errors
+    //#region ERRORS
+    //@doc http://sveltejs.github.io/eslint-plugin-svelte/rules/#possible-errors
     "svelte/infinite-reactive-loop": State.ON,
     "svelte/no-dom-manipulating": State.ON,
     "svelte/no-dupe-else-if-blocks": State.ON,
@@ -50,16 +50,16 @@ export default {
     "svelte/require-store-reactive-access": State.ON,
     "svelte/valid-compile": State.ON,
     "svelte/valid-style-parse": State.ON,
-    // #endregion
+    //#endregion
 
-    // #region SECURITY
-    // DOC: http://sveltejs.github.io/eslint-plugin-svelte/rules/#security-vulnerability
+    //#region SECURITY
+    //@doc http://sveltejs.github.io/eslint-plugin-svelte/rules/#security-vulnerability
     "svelte/no-at-html-tags": State.ON,
     "svelte/no-target-blank": State.ON,
-    // #endregion
+    //#endregion
 
-    // #region BEST PRACTICES
-    // DOC: http://sveltejs.github.io/eslint-plugin-svelte/rules/#best-practices
+    //#region BEST PRACTICES
+    //@doc http://sveltejs.github.io/eslint-plugin-svelte/rules/#best-practices
     "svelte/block-lang": [
       State.WARN,
       {
@@ -77,7 +77,7 @@ export default {
     "svelte/no-reactive-literals": State.ON,
     "svelte/no-svelte-internal": State.ON,
     "svelte/no-unnecessary-state-wrap": State.ON,
-    // "svelte/no-unused-class-name": State.OFF /* incompatible: breaks if css class is located in another file; BUG: ESLint error if array empty */,
+    // "svelte/no-unused-class-name": State.OFF /* incompatible: breaks if css class is located in another file; @BUG ESLint error if array empty */,
     "svelte/no-unused-props": [
       State.WARN,
       {
@@ -100,10 +100,10 @@ export default {
     "svelte/require-optimized-style-attribute": State.ON,
     "svelte/require-stores-init": State.ON,
     "svelte/valid-each-key": State.ON,
-    // #endregion
+    //#endregion
 
-    // #region STYLE
-    // DOC: http://sveltejs.github.io/eslint-plugin-svelte/rules/#stylistic-issues
+    //#region STYLE
+    //@doc http://sveltejs.github.io/eslint-plugin-svelte/rules/#stylistic-issues
     "svelte/consistent-selector-style": [
       State.WARN,
       {
@@ -114,7 +114,7 @@ export default {
     // "svelte/first-attribute-linebreak": State.OFF,
     // "svelte/html-closing-bracket-new-line": State.OFF,
     // "svelte/html-closing-bracket-spacing": State.OFF,
-    // "svelte/html-quotes": State.OFF /* BUG: incompatible with object props, which should not be stringified */,
+    // "svelte/html-quotes": State.OFF /* @BUG incompatible with object props, which should not be stringified */,
     // "svelte/html-self-closing": State.OFF,
     // "svelte/indent": State.OFF,
     // "svelte/max-attributes-per-line": State.OFF,
@@ -134,23 +134,23 @@ export default {
     "svelte/shorthand-directive": State.WARN,
     // "svelte/sort-attributes": State.OFF,
     // "svelte/spaced-html-comment": State.OFF,
-    // #endregion
+    //#endregion
 
-    // #region SVELTE-KIT
-    // DOC: http://sveltejs.github.io/eslint-plugin-svelte/rules/#svelte-kit
+    //#region SVELTE-KIT
+    //@doc http://sveltejs.github.io/eslint-plugin-svelte/rules/#svelte-kit
     "svelte/no-export-load-in-svelte-module-in-kit-pages": State.ON,
     "svelte/no-navigation-without-resolve": State.ON,
     "svelte/valid-prop-names-in-kit-pages": State.ON,
-    // #endregion
+    //#endregion
 
-    // #region REQUIRED
-    // DOC: http://sveltejs.github.io/eslint-plugin-svelte/rules/#system
+    //#region REQUIRED
+    //@doc http://sveltejs.github.io/eslint-plugin-svelte/rules/#system
     "svelte/comment-directive": State.ON,
     "svelte/system": State.ON,
-    // #endregion
+    //#endregion
 
     //#region Future (HTML)
-    // DOC: https://html-eslint.org/docs/svelte/rules
+    //@doc https://html-eslint.org/docs/svelte/rules
     // "html/svelte/class-spacing": State.OFF,
     // "html/svelte/use-baseline": State.WARN,
     // "html/svelte/no-obsolete-tags": State.WARN,

package/src/rules/ts.ts

@@ -25,8 +25,8 @@ export default {
     "no-unsafe-negation": State.OFF,
     //#endregion
 
-    // #region EXTEND
-    // DOC: https://typescript-eslint.io/rules/?=extension-xdeprecated#rules
+    //#region EXTEND
+    //@doc https://typescript-eslint.io/rules/?=extension-xdeprecated#rules
     /* @OVERRIDE */ "dot-notation": State.OFF,
     "ts/dot-notation": [
       State.WARN,
@@ -62,10 +62,10 @@ export default {
     ],
     /* @OVERRIDE */ "require-await": State.OFF,
     "ts/require-await": State.ON,
-    // #endregion
+    //#endregion
 
-    // #region ...
-    // DOC: https://typescript-eslint.io/rules/?=xextension-xdeprecated#rules
+    //#region ...
+    //@doc https://typescript-eslint.io/rules/?=xextension-xdeprecated#rules
 
     // "ts/adjacent-overload-signatures": State.OFF,
     // "ts/array-type": State.OFF /* @preference: ideally, I'd like to don't-care mutable arrays while requiring readonly [] syntax for immutable arrays, but there is no way to do that with the current rule options */,
@@ -242,7 +242,7 @@ export default {
         allowRegExp: false,
       },
     ],
-    "ts/return-await": State.ON /* BUG: doc incorrectly states that this extends the DEPRECATED (since 8.46) no-return-await: https://typescript-eslint.io/rules/return-await */,
+    "ts/return-await": State.ON /* @BUG doc incorrectly states that this extends the DEPRECATED (since 8.46) no-return-await: https://typescript-eslint.io/rules/return-await */,
     "ts/strict-boolean-expressions": [
       State.ON,
       {
@@ -268,8 +268,8 @@ export default {
         ignoreStatic: true,
       },
     ],
-    // "ts/unified-signatures": State.OFF /* BUG: https://github.com/typescript-eslint/typescript-eslint/issues/11732 */,
+    // "ts/unified-signatures": State.OFF /* @BUG https://github.com/typescript-eslint/typescript-eslint/issues/11732 */,
     "ts/use-unknown-in-catch-callback-variable": State.ON,
-    // #endregion
+    //#endregion
   } as const,
 };

package/src/rules/yml.ts

@@ -1,6 +1,6 @@
 import { State } from "./_state";
 
-// DOC: https://ota-meshi.github.io/eslint-plugin-yml/rules/#yaml-rules
+//@doc https://ota-meshi.github.io/eslint-plugin-yml/rules/#yaml-rules
 export default {
   rules: {
     //#region Custom