npm - @herb-tools/linter - Versions diffs - 0.8.0 → 0.8.2 - Mend

@herb-tools/linter 0.8.0 → 0.8.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (33) hide show

package/README.md +42 -2
package/dist/herb-lint.js +88 -17
package/dist/herb-lint.js.map +1 -1
package/dist/index.cjs +58 -0
package/dist/index.cjs.map +1 -1
package/dist/index.js +59 -1
package/dist/index.js.map +1 -1
package/dist/loader.cjs +70 -8
package/dist/loader.cjs.map +1 -1
package/dist/loader.js +70 -8
package/dist/loader.js.map +1 -1
package/dist/package.json +7 -7
package/dist/src/herb-disable-comment-utils.js.map +1 -1
package/dist/src/linter-ignore.js +42 -0
package/dist/src/linter-ignore.js.map +1 -0
package/dist/src/linter.js +12 -0
package/dist/src/linter.js.map +1 -1
package/dist/src/rules/html-head-only-elements.js +6 -1
package/dist/src/rules/html-head-only-elements.js.map +1 -1
package/dist/src/rules/rule-utils.js +2 -0
package/dist/src/rules/rule-utils.js.map +1 -1
package/dist/tsconfig.tsbuildinfo +1 -1
package/dist/types/linter-ignore.d.ts +12 -0
package/dist/types/rules/rule-utils.d.ts +1 -1
package/dist/types/src/linter-ignore.d.ts +12 -0
package/dist/types/src/rules/rule-utils.d.ts +1 -1
package/docs/rules/html-head-only-elements.md +24 -2
package/package.json +7 -7
package/src/herb-disable-comment-utils.ts +3 -0
package/src/linter-ignore.ts +50 -0
package/src/linter.ts +14 -0
package/src/rules/html-head-only-elements.ts +6 -1
package/src/rules/rule-utils.ts +3 -1

package/dist/loader.js CHANGED Viewed

@@ -165,7 +165,7 @@ class Token {
 }
 // NOTE: This file is generated by the templates/template.rb script and should not
-// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.0/templates/javascript/packages/core/src/errors.ts.erb
+// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.2/templates/javascript/packages/core/src/errors.ts.erb
 class HerbError {
     type;
     message;
@@ -695,7 +695,7 @@ function convertToUTF8(string) {
 }
 // NOTE: This file is generated by the templates/template.rb script and should not
-// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.0/templates/javascript/packages/core/src/nodes.ts.erb
+// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.2/templates/javascript/packages/core/src/nodes.ts.erb
 class Node {
     type;
     location;
@@ -2929,7 +2929,7 @@ class ParseResult extends Result {
 }
 // NOTE: This file is generated by the templates/template.rb script and should not
-// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.0/templates/javascript/packages/core/src/node-type-guards.ts.erb
+// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.2/templates/javascript/packages/core/src/node-type-guards.ts.erb
 /**
  * Type guard functions for AST nodes.
  * These functions provide type checking by combining both instanceof
@@ -3319,10 +3319,14 @@ function isERBOutputNode(node) {
     return ["<%=", "<%=="].includes(node.tag_opening?.value);
 }
 /**
- * Checks if a node is a non-output ERB node (control flow: <% %>)
+ * Checks if a node is a ERB comment node (control flow: <%# %>)
  */
-function isERBControlFlowNode(node) {
-    return isAnyOf(node, ERBIfNode, ERBUnlessNode, ERBBlockNode, ERBCaseNode, ERBCaseMatchNode, ERBWhileNode, ERBForNode, ERBBeginNode);
+function isERBCommentNode(node) {
+    if (!isERBNode(node))
+        return false;
+    if (!node.tag_opening?.value)
+        return false;
+    return node.tag_opening?.value === "<%#" || (node.tag_opening?.value !== "<%#" && (node.content?.value || "").trimStart().startsWith("#"));
 }
 /**
  * Checks if an array of nodes contains any ERB content nodes
@@ -3441,7 +3445,7 @@ function getTagName$1(node) {
  * Check if a node is a comment (HTML comment or ERB comment)
  */
 function isCommentNode(node) {
-    return isNode(node, HTMLCommentNode) || (isERBNode(node) && !isERBControlFlowNode(node));
+    return isHTMLCommentNode(node) || isERBCommentNode(node);
 }
 /**
  * Compares two positions to determine if the first comes before the second
@@ -3628,7 +3632,7 @@ function didyoumean(input, list, threshold) {
 }
 // NOTE: This file is generated by the templates/template.rb script and should not
-// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.0/templates/javascript/packages/core/src/visitor.ts.erb
+// be modified manually. See /Users/marcoroth/Development/herb-release-0.8.2/templates/javascript/packages/core/src/visitor.ts.erb
 class Visitor {
     visit(node) {
         if (!node)
@@ -6843,6 +6847,8 @@ function findAttributeByName(attributes, attributeName) {
  * Checks if a tag has a specific attribute
  */
 function hasAttribute(node, attributeName) {
+    if (!node)
+        return false;
     return getAttribute(node, attributeName) !== null;
 }
 /**
@@ -8942,8 +8948,13 @@ class HeadOnlyElementsVisitor extends BaseRuleVisitor {
             return;
         if (tagName === "title" && this.insideSVG)
             return;
+        if (tagName === "meta" && this.hasItempropAttribute(node))
+            return;
         this.addOffense(`Element \`<${tagName}>\` must be placed inside the \`<head>\` tag.`, node.location);
     }
+    hasItempropAttribute(node) {
+        return hasAttribute(node.open_tag, "itemprop");
+    }
     get insideHead() {
         return this.elementStack.includes("head");
     }
@@ -10453,6 +10464,46 @@ const rules = [
     ParserNoErrorsRule,
 ];
+const HERB_LINTER_PREFIX = "herb:linter";
+const HERB_LINTER_IGNORE_PREFIX = `${HERB_LINTER_PREFIX} ignore`;
+/**
+ * Check if an ERB content node is a herb:linter ignore comment.
+ *
+ * @param node - The ERB content node to check
+ * @returns true if this is a linter ignore directive
+ */
+function isHerbLinterIgnoreComment(node) {
+    if (!isERBCommentNode(node))
+        return false;
+    const content = node?.content?.value || "";
+    return content.trim() === HERB_LINTER_IGNORE_PREFIX;
+}
+/**
+ * Check if the document contains a herb:linter ignore directive anywhere.
+ */
+function hasLinterIgnoreDirective(parseResult) {
+    if (parseResult.failed)
+        return false;
+    const detector = new LinterIgnoreDetector();
+    detector.visit(parseResult.value);
+    return detector.hasIgnoreDirective;
+}
+/**
+ * Visitor that detects if the AST contains a herb:linter ignore directive.
+ */
+class LinterIgnoreDetector extends Visitor {
+    hasIgnoreDirective = false;
+    visitERBContentNode(node) {
+        if (isHerbLinterIgnoreComment(node)) {
+            this.hasIgnoreDirective = true;
+            return;
+        }
+        if (this.hasIgnoreDirective)
+            return;
+        this.visitChildNodes(node);
+    }
+}
 class Linter {
     rules;
     allAvailableRules;
@@ -10663,6 +10714,17 @@ class Linter {
         let ignoredCount = 0;
         let wouldBeIgnoredCount = 0;
         const parseResult = this.herb.parse(source, { track_whitespace: true });
+        // Check for file-level ignore directive using visitor
+        if (hasLinterIgnoreDirective(parseResult)) {
+            return {
+                offenses: [],
+                errors: 0,
+                warnings: 0,
+                info: 0,
+                hints: 0,
+                ignored: 0
+            };
+        }
         const lexResult = this.herb.lex(source);
         const hasParserErrors = parseResult.recursiveErrors().length > 0;
         const sourceLines = source.split("\n");