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/index.js CHANGED Viewed

@@ -1,4 +1,4 @@
-import { getNodesBeforePosition, getNodesAfterPosition, filterNodes, ERBContentNode, isERBOutputNode, Visitor, isToken, isParseResult, getStaticAttributeName, hasDynamicAttributeName as hasDynamicAttributeName$1, getCombinedAttributeName, hasStaticContent, getStaticContentFromNodes, Location, hasERBOutput, isEffectivelyStatic, getValidatableStaticContent, isERBNode, isWhitespaceNode, isCommentNode, isLiteralNode, isHTMLTextNode, Position, filterERBContentNodes, isNode, LiteralNode, didyoumean, filterLiteralNodes, Token, getTagName as getTagName$1, isHTMLElementNode, HTMLCloseTagNode, isHTMLOpenTagNode, WhitespaceNode, filterWhitespaceNodes, HTMLOpenTagNode } from '@herb-tools/core';
+import { getNodesBeforePosition, getNodesAfterPosition, filterNodes, ERBContentNode, isERBOutputNode, Visitor, isToken, isParseResult, getStaticAttributeName, hasDynamicAttributeName as hasDynamicAttributeName$1, getCombinedAttributeName, hasStaticContent, getStaticContentFromNodes, Location, hasERBOutput, isEffectivelyStatic, getValidatableStaticContent, isERBNode, isWhitespaceNode, isCommentNode, isLiteralNode, isHTMLTextNode, Position, filterERBContentNodes, isNode, LiteralNode, didyoumean, filterLiteralNodes, Token, getTagName as getTagName$1, isHTMLElementNode, HTMLCloseTagNode, isHTMLOpenTagNode, WhitespaceNode, filterWhitespaceNodes, HTMLOpenTagNode, isERBCommentNode } from '@herb-tools/core';
 class PrintContext {
     output = "";
@@ -3053,6 +3053,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;
 }
 /**
@@ -5152,8 +5154,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");
     }
@@ -6663,6 +6670,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;
@@ -6873,6 +6920,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");