@html-eslint/eslint-plugin 0.41.0-alpha.0 → 0.42.0

package/lib/rules/require-img-alt.js

@@ -2,7 +2,7 @@
  * @typedef { import("@html-eslint/types").Tag } Tag
  *
  * @typedef {Object} Option
- * @property {string[]} [Option.substitute]
+ * @property {string[]} [substitute]
  *
  * @typedef { import("../types").RuleModule<[Option]> } RuleModule
  */
@@ -13,6 +13,7 @@ const { getRuleUrl } = require("./utils/rule");
 
 const MESSAGE_IDS = {
   MISSING_ALT: "missingAlt",
+  INSERT_ALT: "insertAlt",
 };
 
 /**
@@ -20,7 +21,7 @@ const MESSAGE_IDS = {
  */
 module.exports = {
   meta: {
-    type: "code",
+    type: "suggestion",
 
     docs: {
       description: "Require `alt` attribute at `<img>` tag",
@@ -30,6 +31,7 @@ module.exports = {
     },
 
     fixable: null,
+    hasSuggestions: true,
     schema: [
       {
         type: "object",
@@ -45,6 +47,7 @@ module.exports = {
     ],
     messages: {
       [MESSAGE_IDS.MISSING_ALT]: "Missing `alt` attribute at `<img>` tag",
+      [MESSAGE_IDS.INSERT_ALT]: 'Insert `alt=""` at `<img>` tag',
     },
   },
 
@@ -60,13 +63,27 @@ module.exports = {
         if (node.name !== "img") {
           return;
         }
-        if (!hasAltAttrAndValue(node, substitute)) {
+
+        const hasAlt = hasValidAltOrSubstitute(node, substitute);
+        const hasSubstituteOption = substitute.length > 0;
+
+        if (!hasAlt) {
           context.report({
             loc: {
               start: node.openStart.loc.start,
               end: node.openEnd.loc.end,
             },
             messageId: MESSAGE_IDS.MISSING_ALT,
+            suggest: hasSubstituteOption
+              ? null
+              : [
+                  {
+                    messageId: MESSAGE_IDS.INSERT_ALT,
+                    fix(fixer) {
+                      return fixer.insertTextBefore(node.openEnd, ' alt=""');
+                    },
+                  },
+                ],
           });
         }
       },
@@ -77,16 +94,22 @@ module.exports = {
 /**
  * @param {Tag} node
  * @param {string[]} substitute
- * @returns
+ * @returns {boolean}
  */
-function hasAltAttrAndValue(node, substitute = []) {
-  return node.attributes.some((attr) => {
-    if (attr.key && attr.value) {
-      return (
-        (attr.key.value === "alt" || substitute.includes(attr.key.value)) &&
-        typeof attr.value.value === "string"
-      );
+function hasValidAltOrSubstitute(node, substitute) {
+  let hasAnyAlt = false;
+
+  for (const attr of node.attributes) {
+    if (attr.key && attr.key.value) {
+      const isAltAttr = attr.key.value === "alt";
+      const isSubstituteAttr = substitute.includes(attr.key.value);
+
+      if (isAltAttr || isSubstituteAttr) {
+        hasAnyAlt = true;
+        break;
+      }
     }
-    return false;
-  });
+  }
+
+  return hasAnyAlt;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@html-eslint/eslint-plugin",
-  "version": "0.41.0-alpha.0",
+  "version": "0.42.0",
   "type": "commonjs",
   "description": "ESLint plugin for HTML",
   "author": "yeonjuan",
@@ -40,16 +40,16 @@
   ],
   "dependencies": {
     "@eslint/plugin-kit": "^0.3.1",
-    "@html-eslint/parser": "^0.41.0-alpha.0",
-    "@html-eslint/template-parser": "^0.41.0-alpha.0",
-    "@html-eslint/template-syntax-parser": "^0.41.0-alpha.0"
+    "@html-eslint/parser": "^0.42.0",
+    "@html-eslint/template-parser": "^0.42.0",
+    "@html-eslint/template-syntax-parser": "^0.42.0"
   },
   "peerDependencies": {
     "eslint": "^8.0.0 || ^9.0.0"
   },
   "devDependencies": {
     "@eslint/core": "^0.14.0",
-    "@html-eslint/types": "^0.41.0-alpha.0",
+    "@html-eslint/types": "^0.42.0",
     "@types/estree": "^0.0.47",
     "es-html-parser": "0.2.0",
     "eslint": "^9.27.0",
@@ -59,5 +59,5 @@
   "engines": {
     "node": "^18.18.0 || ^20.9.0 || >=21.1.0"
   },
-  "gitHead": "a1d2d99f91d5adc86184ea15151b5c04bd53ea8d"
+  "gitHead": "30e536dad0fef8ea04b8ef383024f203a3f2655f"
 }

package/types/configs/recommended.d.ts

@@ -17,5 +17,6 @@ export let rules: {
     "@html-eslint/require-closing-tags": "error";
     "@html-eslint/no-duplicate-attrs": "error";
     "@html-eslint/use-baseline": "error";
+    "@html-eslint/no-duplicate-in-head": "error";
 };
 //# sourceMappingURL=recommended.d.ts.map

package/types/index.d.ts

@@ -29,6 +29,7 @@ export declare let configs: {
             "@html-eslint/require-closing-tags": "error";
             "@html-eslint/no-duplicate-attrs": "error";
             "@html-eslint/use-baseline": "error";
+            "@html-eslint/no-duplicate-in-head": "error";
         };
     };
     "flat/recommended": {
@@ -58,6 +59,7 @@ export declare let configs: {
             "@html-eslint/require-closing-tags": "error";
             "@html-eslint/no-duplicate-attrs": "error";
             "@html-eslint/use-baseline": "error";
+            "@html-eslint/no-duplicate-in-head": "error";
         };
     };
 };

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":";;4BAO2B,QAAQ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;YAgB3B,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":";;;wBAmBU,UAAU;;WAlBN,OAAO,oBAAoB,EAAE,GAAG;gBAChC,OAAO,oBAAoB,EAAE,QAAQ;iBACrC,OAAO,oBAAoB,EAAE,SAAS;kBACtC,OAAO,UAAU,EAAE,UAAU,CAAC,EAAE,CAAC"}
+{"version":3,"file":"lowercase.d.ts","sourceRoot":"","sources":["../../lib/rules/lowercase.js"],"names":[],"mappings":";;;wBAkBU,UAAU;;WAjBN,OAAO,oBAAoB,EAAE,GAAG;gBAChC,OAAO,oBAAoB,EAAE,QAAQ;iBACrC,OAAO,oBAAoB,EAAE,SAAS;kBACtC,OAAO,UAAU,EAAE,UAAU,CAAC,EAAE,CAAC"}

package/types/rules/no-aria-hidden-on-focusable.d.ts

@@ -0,0 +1,8 @@
+declare namespace _exports {
+    export { RuleModule, Tag };
+}
+declare const _exports: RuleModule;
+export = _exports;
+type RuleModule = import("../types").RuleModule<[]>;
+type Tag = import("@html-eslint/types").Tag;
+//# sourceMappingURL=no-aria-hidden-on-focusable.d.ts.map

package/types/rules/no-aria-hidden-on-focusable.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-aria-hidden-on-focusable.d.ts","sourceRoot":"","sources":["../../lib/rules/no-aria-hidden-on-focusable.js"],"names":[],"mappings":";;;wBA8BU,UAAU;;kBA7BN,OAAO,UAAU,EAAE,UAAU,CAAC,EAAE,CAAC;WACjC,OAAO,oBAAoB,EAAE,GAAG"}

package/types/rules/no-duplicate-in-head.d.ts

@@ -0,0 +1,11 @@
+declare namespace _exports {
+    export { Tag, StyleTag, ScriptTag, AttributeValue, RuleModule };
+}
+declare const _exports: RuleModule;
+export = _exports;
+type Tag = import("@html-eslint/types").Tag;
+type StyleTag = import("@html-eslint/types").StyleTag;
+type ScriptTag = import("@html-eslint/types").ScriptTag;
+type AttributeValue = import("@html-eslint/types").AttributeValue;
+type RuleModule = import("../types").RuleModule<[]>;
+//# sourceMappingURL=no-duplicate-in-head.d.ts.map

package/types/rules/no-duplicate-in-head.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-duplicate-in-head.d.ts","sourceRoot":"","sources":["../../lib/rules/no-duplicate-in-head.js"],"names":[],"mappings":";;;wBAgEU,UAAU;;WA/DN,OAAO,oBAAoB,EAAE,GAAG;gBAChC,OAAO,oBAAoB,EAAE,QAAQ;iBACrC,OAAO,oBAAoB,EAAE,SAAS;sBACtC,OAAO,oBAAoB,EAAE,cAAc;kBAC3C,OAAO,UAAU,EAAE,UAAU,CAAC,EAAE,CAAC"}

package/types/rules/no-empty-headings.d.ts

@@ -0,0 +1,9 @@
+declare namespace _exports {
+    export { RuleModule, Tag, Text };
+}
+declare const _exports: RuleModule;
+export = _exports;
+type RuleModule = import("../types").RuleModule<[]>;
+type Tag = import("@html-eslint/types").Tag;
+type Text = import("@html-eslint/types").Text;
+//# sourceMappingURL=no-empty-headings.d.ts.map

package/types/rules/no-empty-headings.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-empty-headings.d.ts","sourceRoot":"","sources":["../../lib/rules/no-empty-headings.js"],"names":[],"mappings":";;;wBA0EU,UAAU;;kBAzEN,OAAO,UAAU,EAAE,UAAU,CAAC,EAAE,CAAC;WACjC,OAAO,oBAAoB,EAAE,GAAG;YAChC,OAAO,oBAAoB,EAAE,IAAI"}

package/types/rules/no-invalid-entity.d.ts

@@ -0,0 +1,13 @@
+declare namespace _exports {
+    export { RuleModule, SuggestionReportDescriptor, Text, EntityData };
+}
+declare const _exports: RuleModule;
+export = _exports;
+type RuleModule = import("../types").RuleModule<[]>;
+type SuggestionReportDescriptor = import("../types").SuggestionReportDescriptor;
+type Text = import("@html-eslint/types").Text;
+type EntityData = {
+    codepoints: number[];
+    characters: string;
+};
+//# sourceMappingURL=no-invalid-entity.d.ts.map

package/types/rules/no-invalid-entity.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-invalid-entity.d.ts","sourceRoot":"","sources":["../../lib/rules/no-invalid-entity.js"],"names":[],"mappings":";;;wBAyBU,UAAU;;kBAxBN,OAAO,UAAU,EAAE,UAAU,CAAC,EAAE,CAAC;kCACjC,OAAO,UAAU,EAAE,0BAA0B;YAC7C,OAAO,oBAAoB,EAAE,IAAI;;gBAMjC,MAAM,EAAE;gBACR,MAAM"}

package/types/rules/require-img-alt.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"require-img-alt.d.ts","sourceRoot":"","sources":["../../lib/rules/require-img-alt.js"],"names":[],"mappings":";;;wBAkBU,UAAU;;WAjBN,OAAO,oBAAoB,EAAE,GAAG;;;;kBAKhC,OAAO,UAAU,EAAE,UAAU,CAAC,CAAC,MAAM,CAAC,CAAC"}
+{"version":3,"file":"require-img-alt.d.ts","sourceRoot":"","sources":["../../lib/rules/require-img-alt.js"],"names":[],"mappings":";;;wBAmBU,UAAU;;WAlBN,OAAO,oBAAoB,EAAE,GAAG;;;;kBAKhC,OAAO,UAAU,EAAE,UAAU,CAAC,CAAC,MAAM,CAAC,CAAC"}