@html-eslint/eslint-plugin 0.45.0 → 0.46.1

package/lib/configs/recommended.js

@@ -1,29 +1,39 @@
-/** @satisfies {import('eslint').Linter.FlatConfig} */
-const recommended = {
-  rules: {
-    "@html-eslint/require-lang": "error",
-    "@html-eslint/require-img-alt": "error",
-    "@html-eslint/require-doctype": "error",
-    "@html-eslint/require-title": "error",
-    "@html-eslint/no-multiple-h1": "error",
-    "@html-eslint/no-extra-spacing-attrs": "error",
-    "@html-eslint/attrs-newline": "error",
-    "@html-eslint/element-newline": [
-      "error",
-      {
-        inline: [`$inline`],
-      },
-    ],
-    "@html-eslint/no-duplicate-id": "error",
-    "@html-eslint/indent": "error",
-    "@html-eslint/require-li-container": "error",
-    "@html-eslint/quotes": "error",
-    "@html-eslint/no-obsolete-tags": "error",
-    "@html-eslint/require-closing-tags": "error",
-    "@html-eslint/no-duplicate-attrs": "error",
-    "@html-eslint/use-baseline": "error",
-    "@html-eslint/no-duplicate-in-head": "error",
-  },
+/** @satisfies {import('eslint').Linter.Config['rules']} */
+const recommendedRules = {
+  "html/require-lang": "error",
+  "html/require-img-alt": "error",
+  "html/require-doctype": "error",
+  "html/require-title": "error",
+  "html/no-multiple-h1": "error",
+  "html/no-extra-spacing-attrs": "error",
+  "html/attrs-newline": "error",
+  "html/element-newline": [
+    "error",
+    {
+      inline: [`$inline`],
+    },
+  ],
+  "html/no-duplicate-id": "error",
+  "html/indent": "error",
+  "html/require-li-container": "error",
+  "html/quotes": "error",
+  "html/no-obsolete-tags": "error",
+  "html/require-closing-tags": "error",
+  "html/no-duplicate-attrs": "error",
+  "html/use-baseline": "error",
+  "html/no-duplicate-in-head": "error",
 };
 
-module.exports = recommended;
+const recommendedLegacyRules = Object.entries(recommendedRules).reduce(
+  (acc, [key, value]) => {
+    acc[key.replace("html/", "@html-eslint/")] = value;
+    return acc;
+  },
+  /** @type {Record<string, typeof recommendedRules[keyof typeof recommendedRules]>} */
+  ({})
+);
+
+module.exports = {
+  recommendedRules,
+  recommendedLegacyRules,
+};

package/lib/index.js

@@ -1,9 +1,11 @@
 const rules = require("./rules");
-const recommended = require("./configs/recommended");
-const parser = require("@html-eslint/parser");
+const {
+  recommendedRules,
+  recommendedLegacyRules,
+} = require("./configs/recommended");
 const { HTMLLanguage } = require("./languages/html-language");
 const { name, version } = require("../package.json");
-
+const parser = require("@html-eslint/parser");
 /**
  * @import { ESLint } from "eslint";
  */
@@ -16,9 +18,18 @@ const plugin = {
     name,
     version,
   },
+  languages: {
+    html: new HTMLLanguage(),
+  },
+  rules,
   configs: {
-    recommended,
-
+    recommended: {
+      rules: recommendedRules,
+      plugins: {},
+    },
+    ["recommended-legacy"]: {
+      rules: recommendedLegacyRules,
+    },
     "flat/recommended": {
       plugins: {
         /** @type {ESLint.Plugin} */
@@ -26,17 +37,17 @@ const plugin = {
           return plugin;
         },
       },
-
       languageOptions: {
         parser,
       },
-      rules: recommended.rules,
+      rules: recommendedLegacyRules,
     },
   },
-  languages: {
-    html: new HTMLLanguage(),
-  },
-  rules,
 };
 
+{
+  // @ts-ignore
+  plugin.configs.recommended.plugins.html = plugin;
+}
+
 module.exports = plugin;

package/lib/rules/lowercase.js

@@ -1,5 +1,10 @@
 /**
- * @import {Tag, StyleTag, ScriptTag} from "@html-eslint/types";
+ * @import {
+ *   Tag,
+ *   StyleTag,
+ *   ScriptTag,
+ *   Doctype,
+ * } from "@html-eslint/types";
  * @import {RuleModule} from "../types";
  */
 
@@ -128,6 +133,48 @@ module.exports = {
       }
     }
 
+    /**
+     * @param {Doctype} doctype
+     */
+    function checkDoctype(doctype) {
+      if (doctype.open.value !== doctype.open.value.toLowerCase()) {
+        context.report({
+          node: doctype.open,
+          messageId: MESSAGE_IDS.UNEXPECTED,
+          data: {
+            name: doctype.open.value.slice(1),
+          },
+          fix(fixer) {
+            return fixer.replaceTextRange(doctype.open.range, "<!doctype");
+          },
+        });
+      }
+      if (doctype.attributes && doctype.attributes.length) {
+        doctype.attributes.forEach((attribute) => {
+          if (
+            attribute.value &&
+            attribute.value.value !== attribute.value.value.toLowerCase()
+          ) {
+            context.report({
+              node: attribute.value,
+              messageId: MESSAGE_IDS.UNEXPECTED,
+              data: {
+                name: attribute.value.value,
+              },
+              fix(fixer) {
+                return fixer.replaceText(
+                  // @ts-ignore
+                  attribute.value,
+                  // @ts-ignore
+                  attribute.value.value.toLowerCase()
+                );
+              },
+            });
+          }
+        });
+      }
+    }
+
     return createVisitors(context, {
       Tag(node) {
         if (node.name.toLocaleLowerCase() === "svg") {
@@ -142,6 +189,7 @@ module.exports = {
       },
       StyleTag: check,
       ScriptTag: check,
+      Doctype: checkDoctype,
     });
   },
 };

package/lib/rules/sort-attrs.js

@@ -24,7 +24,7 @@ module.exports = {
     type: "code",
 
     docs: {
-      description: "Enforce attributes alphabetical sorting",
+      description: "Enforce priority and alphabetical sorting of attributes",
       category: RULE_CATEGORY.STYLE,
       recommended: false,
       url: getRuleUrl("sort-attrs"),
@@ -60,7 +60,8 @@ module.exports = {
       },
     ],
     messages: {
-      [MESSAGE_IDS.UNSORTED]: "Attributes should be sorted alphabetically",
+      [MESSAGE_IDS.UNSORTED]:
+        "Attributes should be sorted by priority and alphabetically",
     },
   },
   create(context) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@html-eslint/eslint-plugin",
-  "version": "0.45.0",
+  "version": "0.46.1",
   "type": "commonjs",
   "description": "ESLint plugin for HTML",
   "author": "yeonjuan",
@@ -40,16 +40,16 @@
   ],
   "dependencies": {
     "@eslint/plugin-kit": "^0.3.1",
-    "@html-eslint/parser": "^0.45.0",
-    "@html-eslint/template-parser": "^0.45.0",
-    "@html-eslint/template-syntax-parser": "^0.45.0"
+    "@html-eslint/parser": "^0.46.0",
+    "@html-eslint/template-parser": "^0.46.0",
+    "@html-eslint/template-syntax-parser": "^0.46.0"
   },
   "peerDependencies": {
     "eslint": "^8.0.0 || ^9.0.0"
   },
   "devDependencies": {
     "@eslint/core": "^0.14.0",
-    "@html-eslint/types": "^0.45.0",
+    "@html-eslint/types": "^0.46.0",
     "@types/estree": "^0.0.47",
     "es-html-parser": "0.3.0",
     "eslint": "^9.27.0",
@@ -59,5 +59,5 @@
   "engines": {
     "node": "^18.18.0 || ^20.9.0 || >=21.1.0"
   },
-  "gitHead": "39d3cc1773a4d999ba348ded5903d753a3f7d12c"
+  "gitHead": "30d2b2e4c8b4b69fade92dbe37cb2a178500207c"
 }

package/types/configs/recommended.d.ts

@@ -1,22 +1,26 @@
-export let rules: {
-    "@html-eslint/require-lang": "error";
-    "@html-eslint/require-img-alt": "error";
-    "@html-eslint/require-doctype": "error";
-    "@html-eslint/require-title": "error";
-    "@html-eslint/no-multiple-h1": "error";
-    "@html-eslint/no-extra-spacing-attrs": "error";
-    "@html-eslint/attrs-newline": "error";
-    "@html-eslint/element-newline": ["error", {
+/** @satisfies {import('eslint').Linter.Config['rules']} */
+export const recommendedRules: {
+    "html/require-lang": "error";
+    "html/require-img-alt": "error";
+    "html/require-doctype": "error";
+    "html/require-title": "error";
+    "html/no-multiple-h1": "error";
+    "html/no-extra-spacing-attrs": "error";
+    "html/attrs-newline": "error";
+    "html/element-newline": ["error", {
         inline: string[];
     }];
-    "@html-eslint/no-duplicate-id": "error";
-    "@html-eslint/indent": "error";
-    "@html-eslint/require-li-container": "error";
-    "@html-eslint/quotes": "error";
-    "@html-eslint/no-obsolete-tags": "error";
-    "@html-eslint/require-closing-tags": "error";
-    "@html-eslint/no-duplicate-attrs": "error";
-    "@html-eslint/use-baseline": "error";
-    "@html-eslint/no-duplicate-in-head": "error";
+    "html/no-duplicate-id": "error";
+    "html/indent": "error";
+    "html/require-li-container": "error";
+    "html/quotes": "error";
+    "html/no-obsolete-tags": "error";
+    "html/require-closing-tags": "error";
+    "html/no-duplicate-attrs": "error";
+    "html/use-baseline": "error";
+    "html/no-duplicate-in-head": "error";
 };
+export const recommendedLegacyRules: Record<string, "error" | ["error", {
+    inline: string[];
+}]>;
 //# sourceMappingURL=recommended.d.ts.map

package/types/configs/recommended.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"recommended.d.ts","sourceRoot":"","sources":["../../lib/configs/recommended.js"],"names":[],"mappings":""}
+{"version":3,"file":"recommended.d.ts","sourceRoot":"","sources":["../../lib/configs/recommended.js"],"names":[],"mappings":"AAAA,2DAA2D;AAC3D;;;;;;;;;;;;;;;;;;;;EAuBE;AAEF;;IAOE"}

package/types/index.d.ts

@@ -1,36 +1,46 @@
 import { name } from "../package.json";
 import { version } from "../package.json";
-import type { ESLint } from "eslint";
-import parser = require("@html-eslint/parser");
 import { HTMLLanguage } from "./languages/html-language";
 import rules = require("./rules");
+import type { ESLint } from "eslint";
+import parser = require("@html-eslint/parser");
 export declare namespace meta {
     export { name };
     export { version };
 }
+export declare namespace languages {
+    let html: HTMLLanguage;
+}
+export { rules };
 export declare let configs: {
     recommended: {
         rules: {
-            "@html-eslint/require-lang": "error";
-            "@html-eslint/require-img-alt": "error";
-            "@html-eslint/require-doctype": "error";
-            "@html-eslint/require-title": "error";
-            "@html-eslint/no-multiple-h1": "error";
-            "@html-eslint/no-extra-spacing-attrs": "error";
-            "@html-eslint/attrs-newline": "error";
-            "@html-eslint/element-newline": ["error", {
+            "html/require-lang": "error";
+            "html/require-img-alt": "error";
+            "html/require-doctype": "error";
+            "html/require-title": "error";
+            "html/no-multiple-h1": "error";
+            "html/no-extra-spacing-attrs": "error";
+            "html/attrs-newline": "error";
+            "html/element-newline": ["error", {
                 inline: string[];
             }];
-            "@html-eslint/no-duplicate-id": "error";
-            "@html-eslint/indent": "error";
-            "@html-eslint/require-li-container": "error";
-            "@html-eslint/quotes": "error";
-            "@html-eslint/no-obsolete-tags": "error";
-            "@html-eslint/require-closing-tags": "error";
-            "@html-eslint/no-duplicate-attrs": "error";
-            "@html-eslint/use-baseline": "error";
-            "@html-eslint/no-duplicate-in-head": "error";
+            "html/no-duplicate-id": "error";
+            "html/indent": "error";
+            "html/require-li-container": "error";
+            "html/quotes": "error";
+            "html/no-obsolete-tags": "error";
+            "html/require-closing-tags": "error";
+            "html/no-duplicate-attrs": "error";
+            "html/use-baseline": "error";
+            "html/no-duplicate-in-head": "error";
         };
+        plugins: {};
+    };
+    "recommended-legacy": {
+        rules: Record<string, "error" | ["error", {
+            inline: string[];
+        }]>;
     };
     "flat/recommended": {
         plugins: {
@@ -40,31 +50,9 @@ export declare let configs: {
         languageOptions: {
             parser: typeof parser;
         };
-        rules: {
-            "@html-eslint/require-lang": "error";
-            "@html-eslint/require-img-alt": "error";
-            "@html-eslint/require-doctype": "error";
-            "@html-eslint/require-title": "error";
-            "@html-eslint/no-multiple-h1": "error";
-            "@html-eslint/no-extra-spacing-attrs": "error";
-            "@html-eslint/attrs-newline": "error";
-            "@html-eslint/element-newline": ["error", {
-                inline: string[];
-            }];
-            "@html-eslint/no-duplicate-id": "error";
-            "@html-eslint/indent": "error";
-            "@html-eslint/require-li-container": "error";
-            "@html-eslint/quotes": "error";
-            "@html-eslint/no-obsolete-tags": "error";
-            "@html-eslint/require-closing-tags": "error";
-            "@html-eslint/no-duplicate-attrs": "error";
-            "@html-eslint/use-baseline": "error";
-            "@html-eslint/no-duplicate-in-head": "error";
-        };
+        rules: Record<string, "error" | ["error", {
+            inline: string[];
+        }]>;
     };
 };
-export declare namespace languages {
-    let html: HTMLLanguage;
-}
-export { rules };
 //# sourceMappingURL=index.d.ts.map

package/types/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../lib/index.js"],"names":[],"mappings":";;4BAO2B,QAAQ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;YAgB3B,4BAA4B;qCAAjB,aAAa"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../lib/index.js"],"names":[],"mappings":";;;;4BAS2B,QAAQ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;YAyB3B,4BAA4B;qCAAjB,aAAa"}

package/types/rules/lowercase.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"lowercase.d.ts","sourceRoot":"","sources":["../../lib/rules/lowercase.js"],"names":[],"mappings":"wBAgBU,WAAW,EAAE,CAAC;;gCAdK,UAAU"}
+{"version":3,"file":"lowercase.d.ts","sourceRoot":"","sources":["../../lib/rules/lowercase.js"],"names":[],"mappings":"wBAqBU,WAAW,EAAE,CAAC;;gCAdK,UAAU"}