@herb-tools/linter 0.8.9 → 0.9.0

package/dist/parse-cache.js

@@ -0,0 +1,30 @@
+import { DEFAULT_PARSER_OPTIONS } from "@herb-tools/core";
+import { DEFAULT_LINTER_PARSER_OPTIONS } from "./types.js";
+export class ParseCache {
+    herb;
+    cache = new Map();
+    constructor(herb) {
+        this.herb = herb;
+    }
+    get(source, parserOptions = {}) {
+        const effectiveOptions = this.resolveOptions(parserOptions);
+        const key = source + JSON.stringify(effectiveOptions);
+        let result = this.cache.get(key);
+        if (!result) {
+            result = this.herb.parse(source, effectiveOptions);
+            this.cache.set(key, result);
+        }
+        return result;
+    }
+    clear() {
+        this.cache.clear();
+    }
+    resolveOptions(parserOptions) {
+        return {
+            ...DEFAULT_PARSER_OPTIONS,
+            ...DEFAULT_LINTER_PARSER_OPTIONS,
+            ...parserOptions
+        };
+    }
+}
+//# sourceMappingURL=parse-cache.js.map

package/dist/parse-cache.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"parse-cache.js","sourceRoot":"","sources":["../src/parse-cache.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,sBAAsB,EAAE,MAAM,kBAAkB,CAAA;AACzD,OAAO,EAAE,6BAA6B,EAAE,MAAM,YAAY,CAAA;AAI1D,MAAM,OAAO,UAAU;IACb,IAAI,CAAa;IACjB,KAAK,GAAG,IAAI,GAAG,EAAuB,CAAA;IAE9C,YAAY,IAAiB;QAC3B,IAAI,CAAC,IAAI,GAAG,IAAI,CAAA;IAClB,CAAC;IAED,GAAG,CAAC,MAAc,EAAE,gBAAwC,EAAE;QAC5D,MAAM,gBAAgB,GAAG,IAAI,CAAC,cAAc,CAAC,aAAa,CAAC,CAAA;QAC3D,MAAM,GAAG,GAAG,MAAM,GAAG,IAAI,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAA;QAErD,IAAI,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,CAAC,CAAA;QAEhC,IAAI,CAAC,MAAM,EAAE,CAAC;YACZ,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAAA;YAClD,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,EAAE,MAAM,CAAC,CAAA;QAC7B,CAAC;QAED,OAAO,MAAM,CAAA;IACf,CAAC;IAED,KAAK;QACH,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,CAAA;IACpB,CAAC;IAED,cAAc,CAAC,aAAqC;QAClD,OAAO;YACL,GAAG,sBAAsB;YACzB,GAAG,6BAA6B;YAChC,GAAG,aAAa;SACjB,CAAA;IACH,CAAC;CACF"}

package/dist/rules/actionview-no-silent-helper.js

@@ -0,0 +1,45 @@
+import { ParserRule } from "../types.js";
+import { BaseRuleVisitor } from "./rule-utils.js";
+import { isERBOpenTagNode, isERBOutputNode } from "@herb-tools/core";
+class ActionViewNoSilentHelperVisitor extends BaseRuleVisitor {
+    visitHTMLElementNode(node) {
+        this.checkActionViewHelper(node);
+        super.visitHTMLElementNode(node);
+    }
+    checkActionViewHelper(node) {
+        if (!node.element_source || node.element_source === "HTML")
+            return;
+        if (!isERBOpenTagNode(node.open_tag))
+            return;
+        if (isERBOutputNode(node.open_tag))
+            return;
+        const tagOpening = node.open_tag.tag_opening?.value;
+        if (!tagOpening)
+            return;
+        const helperName = node.element_source.includes("#")
+            ? node.element_source.split("#").pop()
+            : node.element_source;
+        this.addOffense(`Avoid using \`${tagOpening} %>\` with \`${helperName}\`. Use \`<%= %>\` to ensure the helper's output is rendered.`, node.open_tag.location);
+    }
+}
+export class ActionViewNoSilentHelperRule extends ParserRule {
+    static ruleName = "actionview-no-silent-helper";
+    get defaultConfig() {
+        return {
+            enabled: true,
+            severity: "error"
+        };
+    }
+    get parserOptions() {
+        return {
+            track_whitespace: true,
+            action_view_helpers: true,
+        };
+    }
+    check(result, context) {
+        const visitor = new ActionViewNoSilentHelperVisitor(this.ruleName, context);
+        visitor.visit(result.value);
+        return visitor.offenses;
+    }
+}
+//# sourceMappingURL=actionview-no-silent-helper.js.map

package/dist/rules/actionview-no-silent-helper.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"actionview-no-silent-helper.js","sourceRoot":"","sources":["../../src/rules/actionview-no-silent-helper.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AACjD,OAAO,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKpE,MAAM,+BAAgC,SAAQ,eAAe;IAC3D,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,qBAAqB,CAAC,IAAI,CAAC,CAAA;QAChC,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;IAEO,qBAAqB,CAAC,IAAqB;QACjD,IAAI,CAAC,IAAI,CAAC,cAAc,IAAI,IAAI,CAAC,cAAc,KAAK,MAAM;YAAE,OAAM;QAClE,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,QAAQ,CAAC;YAAE,OAAM;QAC5C,IAAI,eAAe,CAAC,IAAI,CAAC,QAAQ,CAAC;YAAE,OAAM;QAE1C,MAAM,UAAU,GAAG,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,KAAK,CAAA;QAEnD,IAAI,CAAC,UAAU;YAAE,OAAM;QAEvB,MAAM,UAAU,GAAG,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,GAAG,CAAC;YAClD,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE;YACtC,CAAC,CAAC,IAAI,CAAC,cAAc,CAAA;QAEvB,IAAI,CAAC,UAAU,CACb,iBAAiB,UAAU,gBAAgB,UAAU,+DAA+D,EACpH,IAAI,CAAC,QAAQ,CAAC,QAAQ,CACvB,CAAA;IACH,CAAC;CACF;AAED,MAAM,OAAO,4BAA6B,SAAQ,UAAU;IAC1D,MAAM,CAAC,QAAQ,GAAG,6BAA6B,CAAA;IAE/C,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,OAAO;SAClB,CAAA;IACH,CAAC;IAED,IAAI,aAAa;QACf,OAAO;YACL,gBAAgB,EAAE,IAAI;YACtB,mBAAmB,EAAE,IAAI;SAC1B,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,+BAA+B,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAE3E,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC"}

package/dist/{src/rules → rules}/erb-comment-syntax.js

@@ -16,7 +16,7 @@ class ERBCommentSyntaxVisitor extends BaseRuleVisitor {
 }
 export class ERBCommentSyntax extends ParserRule {
     static autocorrectable = true;
-    name = "erb-comment-syntax";
+    static ruleName = "erb-comment-syntax";
     get defaultConfig() {
         return {
             enabled: true,
@@ -24,7 +24,7 @@ export class ERBCommentSyntax extends ParserRule {
         };
     }
     check(result, context) {
-        const visitor = new ERBCommentSyntaxVisitor(this.name, context);
+        const visitor = new ERBCommentSyntaxVisitor(this.ruleName, context);
         visitor.visit(result.value);
         return visitor.offenses;
     }

package/dist/rules/erb-comment-syntax.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"erb-comment-syntax.js","sourceRoot":"","sources":["../../src/rules/erb-comment-syntax.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AACjD,OAAO,EAAE,UAAU,EAA+B,MAAM,aAAa,CAAA;AASrE,MAAM,uBAAwB,SAAQ,eAA+C;IACnF,mBAAmB,CAAC,IAAoB;QACtC,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,EAAE,KAAK,IAAI,EAAE,CAAA;QAEzC,IAAI,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC;YAC1B,MAAM,UAAU,GAAG,IAAI,CAAC,WAAW,EAAE,KAAK,CAAA;YAE1C,IAAI,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC;gBACrC,IAAI,CAAC,UAAU,CACb,4BAA4B,UAAU,2GAA2G,EACjJ,IAAI,CAAC,QAAQ,EACb,EAAE,IAAI,EAAE,CACT,CAAA;YACH,CAAC;iBAAM,CAAC;gBACN,IAAI,CAAC,UAAU,CACb,4BAA4B,UAAU,0EAA0E,EAChH,IAAI,CAAC,QAAQ,EACb,EAAE,IAAI,EAAE,CACT,CAAA;YACH,CAAC;QACH,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,gBAAiB,SAAQ,UAA0C;IAC9E,MAAM,CAAC,eAAe,GAAG,IAAI,CAAA;IAC7B,MAAM,CAAC,QAAQ,GAAG,oBAAoB,CAAA;IAEtC,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,OAAO;SAClB,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,uBAAuB,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAEnE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;IAED,OAAO,CAAC,OAAoD,EAAE,MAAmB,EAAE,QAA+B;QAChH,IAAI,CAAC,OAAO,CAAC,cAAc;YAAE,OAAO,IAAI,CAAA;QAExC,MAAM,EAAE,IAAI,EAAE,GAAG,OAAO,CAAC,cAAc,CAAA;QAEvC,IAAI,CAAC,IAAI,CAAC,WAAW;YAAE,OAAO,IAAI,CAAA;QAClC,IAAI,CAAC,IAAI,CAAC,OAAO;YAAE,OAAO,IAAI,CAAA;QAE9B,IAAI,CAAC,WAAW,CAAC,KAAK,GAAG,KAAK,CAAA;QAE9B,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC,KAAK,CAAA;QAClC,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAA;QAErC,IAAI,KAAK,EAAE,CAAC;YACV,IAAI,CAAC,OAAO,CAAC,KAAK,GAAG,OAAO,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAA;QACzD,CAAC;QAED,OAAO,MAAM,CAAA;IACf,CAAC"}

package/dist/{src/rules → rules}/erb-no-case-node-children.js

@@ -36,7 +36,7 @@ class ERBNoCaseNodeChildrenVisitor extends BaseRuleVisitor {
     }
 }
 export class ERBNoCaseNodeChildrenRule extends ParserRule {
-    name = "erb-no-case-node-children";
+    static ruleName = "erb-no-case-node-children";
     get defaultConfig() {
         return {
             enabled: true,
@@ -44,7 +44,7 @@ export class ERBNoCaseNodeChildrenRule extends ParserRule {
         };
     }
     check(result, context) {
-        const visitor = new ERBNoCaseNodeChildrenVisitor(this.name, context);
+        const visitor = new ERBNoCaseNodeChildrenVisitor(this.ruleName, context);
         visitor.visit(result.value);
         return visitor.offenses;
     }

package/dist/rules/erb-no-case-node-children.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"erb-no-case-node-children.js","sourceRoot":"","sources":["../../src/rules/erb-no-case-node-children.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AACjD,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,gBAAgB,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAA;AAC5G,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAA;AAKrD,MAAM,4BAA6B,SAAQ,eAAe;IACxD,gBAAgB,CAAC,IAAiB;QAChC,IAAI,CAAC,qBAAqB,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;QACxC,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;IAC5B,CAAC;IAED,qBAAqB,CAAC,IAAsB;QAC1C,IAAI,CAAC,qBAAqB,CAAC,IAAI,EAAE,IAAI,CAAC,CAAA;QACtC,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;IAC5B,CAAC;IAEO,qBAAqB,CAAC,IAAoC,EAAE,IAAY;QAC9E,IAAI,CAAC,IAAI,CAAC,QAAQ,IAAI,IAAI,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC;YAAE,OAAM;QAExD,MAAM,QAAQ,GAAG,eAAe,CAAC,YAAY,CAAC,IAAI,CAAC,CAAA;QACnD,MAAM,cAAc,GAAG,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,CAAA;QAC3C,MAAM,aAAa,GAAG,cAAc,IAAI,SAAS,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,eAAe,CAAC,YAAY,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,MAAM,IAAI,SAAS,CAAA;QAEtI,KAAK,MAAM,KAAK,IAAI,IAAI,CAAC,QAAQ,EAAE,CAAC;YAClC,IAAI,CAAC,IAAI,CAAC,gBAAgB,CAAC,KAAK,CAAC,EAAE,CAAC;gBAClC,MAAM,SAAS,GAAG,eAAe,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,CAAA;gBAErD,IAAI,CAAC,UAAU,CACb,kBAAkB,SAAS,gBAAgB,QAAQ,YAAY,aAAa,sEAAsE,EAClJ,KAAK,CAAC,QAAQ,CACf,CAAA;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAEO,gBAAgB,CAAC,IAAU;QACjC,IAAI,gBAAgB,CAAC,IAAI,CAAC;YAAE,OAAO,IAAI,CAAA;QACvC,IAAI,aAAa,CAAC,IAAI,CAAC;YAAE,OAAO,IAAI,CAAA;QAEpC,IAAI,aAAa,CAAC,IAAI,CAAC,IAAI,cAAc,CAAC,IAAI,CAAC,EAAE,CAAC;YAChD,OAAO,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QACnC,CAAC;QAED,OAAO,KAAK,CAAA;IACd,CAAC;CACF;AAED,MAAM,OAAO,yBAA0B,SAAQ,UAAU;IACvD,MAAM,CAAC,QAAQ,GAAG,2BAA2B,CAAA;IAE7C,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,OAAO;SAClB,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,4BAA4B,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAExE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC"}

package/dist/rules/erb-no-conditional-html-element.js

@@ -0,0 +1,38 @@
+import dedent from "dedent";
+import { ParserRule } from "../types.js";
+import { BaseRuleVisitor } from "./rule-utils.js";
+class ERBNoConditionalHTMLElementRuleVisitor extends BaseRuleVisitor {
+    visitHTMLConditionalElementNode(node) {
+        const tagName = node.tag_name?.value || "element";
+        const condition = node.condition || "condition";
+        const suggestion = dedent `
+      Consider using a \`capture\` block instead:
+
+        <% content = capture do %>
+          ... your content here ...
+        <% end %>
+
+        <%= ${condition} ? content_tag(:${tagName}, content) : content %>
+    `;
+        this.addOffense(dedent `
+        Avoid opening and closing \`<${tagName}>\` tags in separate conditional blocks with the same condition. \
+        This pattern is difficult to read and maintain. ${suggestion}
+      `, node.location);
+        this.visitChildNodes(node);
+    }
+}
+export class ERBNoConditionalHTMLElementRule extends ParserRule {
+    static ruleName = "erb-no-conditional-html-element";
+    get defaultConfig() {
+        return {
+            enabled: true,
+            severity: "error"
+        };
+    }
+    check(result, context) {
+        const visitor = new ERBNoConditionalHTMLElementRuleVisitor(this.ruleName, context);
+        visitor.visit(result.value);
+        return visitor.offenses;
+    }
+}
+//# sourceMappingURL=erb-no-conditional-html-element.js.map

package/dist/rules/erb-no-conditional-html-element.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"erb-no-conditional-html-element.js","sourceRoot":"","sources":["../../src/rules/erb-no-conditional-html-element.ts"],"names":[],"mappings":"AAAA,OAAO,MAAM,MAAM,QAAQ,CAAA;AAE3B,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAKjD,MAAM,sCAAuC,SAAQ,eAAe;IAClE,+BAA+B,CAAC,IAAgC;QAC9D,MAAM,OAAO,GAAG,IAAI,CAAC,QAAQ,EAAE,KAAK,IAAI,SAAS,CAAA;QACjD,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,IAAI,WAAW,CAAA;QAE/C,MAAM,UAAU,GAAG,MAAM,CAAA;;;;;;;cAOf,SAAS,mBAAmB,OAAO;KAC5C,CAAA;QAED,IAAI,CAAC,UAAU,CACb,MAAM,CAAA;uCAC2B,OAAO;0DACY,UAAU;OAC7D,EACD,IAAI,CAAC,QAAQ,CACd,CAAA;QAED,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;IAC5B,CAAC;CACF;AAED,MAAM,OAAO,+BAAgC,SAAQ,UAAU;IAC7D,MAAM,CAAC,QAAQ,GAAG,iCAAiC,CAAA;IAEnD,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,OAAO;SAClB,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,sCAAsC,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAElF,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC"}

package/dist/rules/erb-no-conditional-open-tag.js

@@ -0,0 +1,24 @@
+import { ParserRule } from "../types.js";
+import { BaseRuleVisitor } from "./rule-utils.js";
+class ERBNoConditionalOpenTagRuleVisitor extends BaseRuleVisitor {
+    visitHTMLConditionalOpenTagNode(node) {
+        const tagName = node.tag_name?.value || "element";
+        this.addOffense(`Avoid using ERB conditionals to split the open and closing tag of \`<${tagName}>\` element.`, node.location);
+        this.visitChildNodes(node);
+    }
+}
+export class ERBNoConditionalOpenTagRule extends ParserRule {
+    static ruleName = "erb-no-conditional-open-tag";
+    get defaultConfig() {
+        return {
+            enabled: true,
+            severity: "error"
+        };
+    }
+    check(result, context) {
+        const visitor = new ERBNoConditionalOpenTagRuleVisitor(this.ruleName, context);
+        visitor.visit(result.value);
+        return visitor.offenses;
+    }
+}
+//# sourceMappingURL=erb-no-conditional-open-tag.js.map

package/dist/rules/erb-no-conditional-open-tag.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"erb-no-conditional-open-tag.js","sourceRoot":"","sources":["../../src/rules/erb-no-conditional-open-tag.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAKjD,MAAM,kCAAmC,SAAQ,eAAe;IAC9D,+BAA+B,CAAC,IAAgC;QAC9D,MAAM,OAAO,GAAG,IAAI,CAAC,QAAQ,EAAE,KAAK,IAAI,SAAS,CAAA;QAEjD,IAAI,CAAC,UAAU,CACb,wEAAwE,OAAO,cAAc,EAC7F,IAAI,CAAC,QAAQ,CACd,CAAA;QAED,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;IAC5B,CAAC;CACF;AAED,MAAM,OAAO,2BAA4B,SAAQ,UAAU;IACzD,MAAM,CAAC,QAAQ,GAAG,6BAA6B,CAAA;IAE/C,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,OAAO;SAClB,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,kCAAkC,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAE9E,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC"}

package/dist/rules/erb-no-duplicate-branch-elements.js

@@ -0,0 +1,245 @@
+import { ParserRule } from "../types.js";
+import { BaseRuleVisitor } from "./rule-utils.js";
+import { IdentityPrinter } from "@herb-tools/printer";
+import { isHTMLElementNode, isERBIfNode, isERBElseNode, isERBUnlessNode, isERBCaseNode, isERBWhenNode, isEquivalentElement, isPureWhitespaceNode, findParentArray, removeNodeFromArray, replaceNodeWithBody, createLiteral, HTMLElementNode, Location, } from "@herb-tools/core";
+function getSignificantNodes(statements) {
+    return statements.filter(node => !isPureWhitespaceNode(node));
+}
+function allEquivalentElements(nodes) {
+    if (nodes.length < 2)
+        return false;
+    if (!nodes.every(node => isHTMLElementNode(node)))
+        return false;
+    const first = nodes[0];
+    return nodes.slice(1).every(node => isEquivalentElement(first, node));
+}
+function collectBranchesFromIf(node) {
+    const branches = [];
+    let current = node.subsequent;
+    branches.push(node.statements);
+    while (current) {
+        if (isERBElseNode(current)) {
+            branches.push(current.statements);
+            return branches;
+        }
+        if (isERBIfNode(current)) {
+            branches.push(current.statements);
+            current = current.subsequent;
+        }
+        else {
+            break;
+        }
+    }
+    return null;
+}
+function collectBranchesFromUnless(node) {
+    if (!node.else_clause)
+        return null;
+    return [node.statements, node.else_clause.statements];
+}
+function collectBranchesFromCase(node) {
+    if (!node.else_clause)
+        return null;
+    const branches = [];
+    for (const condition of node.conditions) {
+        if (isERBWhenNode(condition)) {
+            branches.push(condition.statements);
+        }
+    }
+    branches.push(node.else_clause.statements);
+    return branches;
+}
+function collectBranches(node) {
+    if (isERBIfNode(node))
+        return collectBranchesFromIf(node);
+    if (isERBUnlessNode(node))
+        return collectBranchesFromUnless(node);
+    if (isERBCaseNode(node))
+        return collectBranchesFromCase(node);
+    return null;
+}
+function findCommonPrefixCount(branches, minLength) {
+    let count = 0;
+    for (let index = 0; index < minLength; index++) {
+        const nodesAtIndex = branches.map(branch => branch[index]);
+        if (allEquivalentElements(nodesAtIndex)) {
+            count++;
+        }
+        else {
+            break;
+        }
+    }
+    return count;
+}
+function findCommonSuffixCount(branches, minLength, prefixCount) {
+    let count = 0;
+    for (let offset = 0; offset < minLength - prefixCount; offset++) {
+        const nodesAtOffset = branches.map(branch => branch[branch.length - 1 - offset]);
+        if (allEquivalentElements(nodesAtOffset)) {
+            count++;
+        }
+        else {
+            break;
+        }
+    }
+    return count;
+}
+function createWrapper(template, body) {
+    return new HTMLElementNode({
+        type: "AST_HTML_ELEMENT_NODE",
+        open_tag: template.open_tag,
+        tag_name: template.tag_name,
+        body,
+        close_tag: template.close_tag,
+        is_void: template.is_void,
+        element_source: template.element_source,
+        location: Location.zero,
+        errors: [],
+    });
+}
+class ERBNoDuplicateBranchElementsVisitor extends BaseRuleVisitor {
+    processedIfNodes = new Set();
+    visitERBIfNode(node) {
+        if (this.processedIfNodes.has(node)) {
+            this.visitChildNodes(node);
+            return;
+        }
+        this.checkConditionalNode(node);
+        this.visitChildNodes(node);
+    }
+    visitERBUnlessNode(node) {
+        this.checkConditionalNode(node);
+        this.visitChildNodes(node);
+    }
+    visitERBCaseNode(node) {
+        this.checkConditionalNode(node);
+        this.visitChildNodes(node);
+    }
+    checkConditionalNode(node) {
+        const branches = collectBranches(node);
+        if (!branches)
+            return;
+        if (isERBIfNode(node)) {
+            this.markSubsequentIfNodesAsProcessed(node);
+        }
+        const state = { isFirstOffense: true };
+        this.checkBranches(branches, node, state);
+    }
+    markSubsequentIfNodesAsProcessed(node) {
+        let current = node.subsequent;
+        while (current) {
+            if (isERBIfNode(current)) {
+                this.processedIfNodes.add(current);
+                current = current.subsequent;
+            }
+            else {
+                break;
+            }
+        }
+    }
+    checkBranches(branches, conditionalNode, state) {
+        const significantBranches = branches.map(getSignificantNodes);
+        if (significantBranches.some(branch => branch.length === 0))
+            return;
+        const minLength = Math.min(...significantBranches.map(branch => branch.length));
+        const prefixCount = findCommonPrefixCount(significantBranches, minLength);
+        const suffixCount = findCommonSuffixCount(significantBranches, minLength, prefixCount);
+        for (let index = 0; index < prefixCount; index++) {
+            const elements = significantBranches.map(branch => branch[index]);
+            this.reportAndRecurse(elements, conditionalNode, state);
+        }
+        for (let offset = 0; offset < suffixCount; offset++) {
+            const elements = significantBranches.map(branch => branch[branch.length - 1 - offset]);
+            this.reportAndRecurse(elements, conditionalNode, state);
+        }
+    }
+    reportAndRecurse(elements, conditionalNode, state) {
+        const bodies = elements.map(element => element.body);
+        const bodiesMatch = elements.every(element => IdentityPrinter.print(element) === IdentityPrinter.print(elements[0]));
+        for (const element of elements) {
+            const printed = IdentityPrinter.print(element.open_tag);
+            const autofixContext = state.isFirstOffense
+                ? { node: conditionalNode }
+                : undefined;
+            this.addOffense(`The \`${printed}\` element is duplicated across all branches of this conditional and can be moved outside.`, bodiesMatch ? element.location : (element?.open_tag?.location || element.location), autofixContext);
+            state.isFirstOffense = false;
+        }
+        if (!bodiesMatch && bodies.every(body => body.length > 0)) {
+            this.checkBranches(bodies, conditionalNode, state);
+        }
+    }
+}
+export class ERBNoDuplicateBranchElementsRule extends ParserRule {
+    static ruleName = "erb-no-duplicate-branch-elements";
+    static autocorrectable = true;
+    static reindentAfterAutofix = true;
+    get defaultConfig() {
+        return {
+            enabled: true,
+            severity: "warning",
+        };
+    }
+    check(result, context) {
+        const visitor = new ERBNoDuplicateBranchElementsVisitor(this.ruleName, context);
+        visitor.visit(result.value);
+        return visitor.offenses;
+    }
+    autofix(offense, result) {
+        if (!offense.autofixContext)
+            return null;
+        const conditionalNode = offense.autofixContext.node;
+        const branches = collectBranches(conditionalNode);
+        if (!branches)
+            return null;
+        const significantBranches = branches.map(getSignificantNodes);
+        if (significantBranches.some(branch => branch.length === 0))
+            return null;
+        const minLength = Math.min(...significantBranches.map(branch => branch.length));
+        const prefixCount = findCommonPrefixCount(significantBranches, minLength);
+        const suffixCount = findCommonSuffixCount(significantBranches, minLength, prefixCount);
+        if (prefixCount === 0 && suffixCount === 0)
+            return null;
+        const parentInfo = findParentArray(result.value, conditionalNode);
+        if (!parentInfo)
+            return null;
+        let { array: parentArray, index: conditionalIndex } = parentInfo;
+        let hasWrapped = false;
+        const hoistElement = (elements, position) => {
+            const bodiesMatch = elements.every(element => IdentityPrinter.print(element) === IdentityPrinter.print(elements[0]));
+            if (bodiesMatch) {
+                for (let i = 0; i < branches.length; i++) {
+                    removeNodeFromArray(branches[i], elements[i]);
+                }
+                if (position === "before") {
+                    parentArray.splice(conditionalIndex, 0, elements[0]);
+                    conditionalIndex++;
+                }
+                else {
+                    parentArray.splice(conditionalIndex + 1, 0, elements[0]);
+                }
+            }
+            else {
+                if (hasWrapped)
+                    return;
+                for (let i = 0; i < branches.length; i++) {
+                    replaceNodeWithBody(branches[i], elements[i]);
+                }
+                const wrapper = createWrapper(elements[0], [createLiteral("\n"), conditionalNode, createLiteral("\n")]);
+                parentArray[conditionalIndex] = wrapper;
+                parentArray = wrapper.body;
+                conditionalIndex = 1;
+                hasWrapped = true;
+            }
+        };
+        for (let index = 0; index < prefixCount; index++) {
+            const elements = significantBranches.map(branch => branch[index]);
+            hoistElement(elements, "before");
+        }
+        for (let offset = 0; offset < suffixCount; offset++) {
+            const elements = significantBranches.map(branch => branch[branch.length - 1 - offset]);
+            hoistElement(elements, "after");
+        }
+        return result;
+    }
+}
+//# sourceMappingURL=erb-no-duplicate-branch-elements.js.map

package/dist/rules/erb-no-duplicate-branch-elements.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"erb-no-duplicate-branch-elements.js","sourceRoot":"","sources":["../../src/rules/erb-no-duplicate-branch-elements.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AACjD,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAA;AAErD,OAAO,EACL,iBAAiB,EACjB,WAAW,EACX,aAAa,EACb,eAAe,EACf,aAAa,EACb,aAAa,EACb,mBAAmB,EACnB,oBAAoB,EACpB,eAAe,EACf,mBAAmB,EACnB,mBAAmB,EACnB,aAAa,EACb,eAAe,EACf,QAAQ,GACT,MAAM,kBAAkB,CAAA;AAYzB,SAAS,mBAAmB,CAAC,UAAkB;IAC7C,OAAO,UAAU,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAC,CAAA;AAC/D,CAAC;AAED,SAAS,qBAAqB,CAAC,KAAa;IAC1C,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,KAAK,CAAA;IAClC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;QAAE,OAAO,KAAK,CAAA;IAE/D,MAAM,KAAK,GAAG,KAAK,CAAC,CAAC,CAAC,CAAA;IAEtB,OAAO,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,mBAAmB,CAAC,KAAK,EAAE,IAAuB,CAAC,CAAC,CAAA;AAC1F,CAAC;AAED,SAAS,qBAAqB,CAAC,IAAe;IAC5C,MAAM,QAAQ,GAAa,EAAE,CAAA;IAC7B,IAAI,OAAO,GAAmC,IAAI,CAAC,UAAU,CAAA;IAE7D,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;IAE9B,OAAO,OAAO,EAAE,CAAC;QACf,IAAI,aAAa,CAAC,OAAO,CAAC,EAAE,CAAC;YAC3B,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAA;YACjC,OAAO,QAAQ,CAAA;QACjB,CAAC;QAED,IAAI,WAAW,CAAC,OAAO,CAAC,EAAE,CAAC;YACzB,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAA;YACjC,OAAO,GAAG,OAAO,CAAC,UAAU,CAAA;QAC9B,CAAC;aAAM,CAAC;YACN,MAAK;QACP,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAA;AACb,CAAC;AAED,SAAS,yBAAyB,CAAC,IAAmB;IACpD,IAAI,CAAC,IAAI,CAAC,WAAW;QAAE,OAAO,IAAI,CAAA;IAElC,OAAO,CAAC,IAAI,CAAC,UAAU,EAAE,IAAI,CAAC,WAAW,CAAC,UAAU,CAAC,CAAA;AACvD,CAAC;AAED,SAAS,uBAAuB,CAAC,IAAiB;IAChD,IAAI,CAAC,IAAI,CAAC,WAAW;QAAE,OAAO,IAAI,CAAA;IAElC,MAAM,QAAQ,GAAa,EAAE,CAAA;IAE7B,KAAK,MAAM,SAAS,IAAI,IAAI,CAAC,UAAU,EAAE,CAAC;QACxC,IAAI,aAAa,CAAC,SAAS,CAAC,EAAE,CAAC;YAC7B,QAAQ,CAAC,IAAI,CAAC,SAAS,CAAC,UAAU,CAAC,CAAA;QACrC,CAAC;IACH,CAAC;IAED,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC,UAAU,CAAC,CAAA;IAE1C,OAAO,QAAQ,CAAA;AACjB,CAAC;AAED,SAAS,eAAe,CAAC,IAAqB;IAC5C,IAAI,WAAW,CAAC,IAAI,CAAC;QAAE,OAAO,qBAAqB,CAAC,IAAI,CAAC,CAAA;IACzD,IAAI,eAAe,CAAC,IAAI,CAAC;QAAE,OAAO,yBAAyB,CAAC,IAAI,CAAC,CAAA;IACjE,IAAI,aAAa,CAAC,IAAI,CAAC;QAAE,OAAO,uBAAuB,CAAC,IAAI,CAAC,CAAA;IAE7D,OAAO,IAAI,CAAA;AACb,CAAC;AAED,SAAS,qBAAqB,CAAC,QAAkB,EAAE,SAAiB;IAClE,IAAI,KAAK,GAAG,CAAC,CAAA;IAEb,KAAK,IAAI,KAAK,GAAG,CAAC,EAAE,KAAK,GAAG,SAAS,EAAE,KAAK,EAAE,EAAE,CAAC;QAC/C,MAAM,YAAY,GAAG,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAA;QAE1D,IAAI,qBAAqB,CAAC,YAAY,CAAC,EAAE,CAAC;YACxC,KAAK,EAAE,CAAA;QACT,CAAC;aAAM,CAAC;YACN,MAAK;QACP,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAA;AACd,CAAC;AAED,SAAS,qBAAqB,CAAC,QAAkB,EAAE,SAAiB,EAAE,WAAmB;IACvF,IAAI,KAAK,GAAG,CAAC,CAAA;IAEb,KAAK,IAAI,MAAM,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,GAAG,WAAW,EAAE,MAAM,EAAE,EAAE,CAAC;QAChE,MAAM,aAAa,GAAG,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,GAAG,MAAM,CAAC,CAAC,CAAA;QAEhF,IAAI,qBAAqB,CAAC,aAAa,CAAC,EAAE,CAAC;YACzC,KAAK,EAAE,CAAA;QACT,CAAC;aAAM,CAAC;YACN,MAAK;QACP,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAA;AACd,CAAC;AAED,SAAS,aAAa,CAAC,QAAyB,EAAE,IAAY;IAC5D,OAAO,IAAI,eAAe,CAAC;QACzB,IAAI,EAAE,uBAAuB;QAC7B,QAAQ,EAAE,QAAQ,CAAC,QAAQ;QAC3B,QAAQ,EAAE,QAAQ,CAAC,QAAQ;QAC3B,IAAI;QACJ,SAAS,EAAE,QAAQ,CAAC,SAAS;QAC7B,OAAO,EAAE,QAAQ,CAAC,OAAO;QACzB,cAAc,EAAE,QAAQ,CAAC,cAAc;QACvC,QAAQ,EAAE,QAAQ,CAAC,IAAI;QACvB,MAAM,EAAE,EAAE;KACX,CAAC,CAAA;AACJ,CAAC;AAED,MAAM,mCAAoC,SAAQ,eAA8C;IACtF,gBAAgB,GAAG,IAAI,GAAG,EAAQ,CAAA;IAE1C,cAAc,CAAC,IAAe;QAC5B,IAAI,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;YACpC,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;YAC1B,OAAM;QACR,CAAC;QAED,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;QAC/B,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;IAC5B,CAAC;IAED,kBAAkB,CAAC,IAAmB;QACpC,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;QAC/B,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;IAC5B,CAAC;IAED,gBAAgB,CAAC,IAAiB;QAChC,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;QAC/B,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;IAC5B,CAAC;IAEO,oBAAoB,CAAC,IAAqB;QAChD,MAAM,QAAQ,GAAG,eAAe,CAAC,IAAI,CAAC,CAAA;QACtC,IAAI,CAAC,QAAQ;YAAE,OAAM;QAErB,IAAI,WAAW,CAAC,IAAI,CAAC,EAAE,CAAC;YACtB,IAAI,CAAC,gCAAgC,CAAC,IAAI,CAAC,CAAA;QAC7C,CAAC;QAED,MAAM,KAAK,GAAG,EAAE,cAAc,EAAE,IAAI,EAAE,CAAA;QACtC,IAAI,CAAC,aAAa,CAAC,QAAQ,EAAE,IAAI,EAAE,KAAK,CAAC,CAAA;IAC3C,CAAC;IAEO,gCAAgC,CAAC,IAAe;QACtD,IAAI,OAAO,GAAmC,IAAI,CAAC,UAAU,CAAA;QAE7D,OAAO,OAAO,EAAE,CAAC;YACf,IAAI,WAAW,CAAC,OAAO,CAAC,EAAE,CAAC;gBACzB,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;gBAClC,OAAO,GAAG,OAAO,CAAC,UAAU,CAAA;YAC9B,CAAC;iBAAM,CAAC;gBACN,MAAK;YACP,CAAC;QACH,CAAC;IACH,CAAC;IAEO,aAAa,CAAC,QAAkB,EAAE,eAAgC,EAAE,KAAkC;QAC5G,MAAM,mBAAmB,GAAG,QAAQ,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAA;QAC7D,IAAI,mBAAmB,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC;YAAE,OAAM;QAEnE,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,CAAC,GAAG,mBAAmB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAA;QAC/E,MAAM,WAAW,GAAG,qBAAqB,CAAC,mBAAmB,EAAE,SAAS,CAAC,CAAA;QACzE,MAAM,WAAW,GAAG,qBAAqB,CAAC,mBAAmB,EAAE,SAAS,EAAE,WAAW,CAAC,CAAA;QAEtF,KAAK,IAAI,KAAK,GAAG,CAAC,EAAE,KAAK,GAAG,WAAW,EAAE,KAAK,EAAE,EAAE,CAAC;YACjD,MAAM,QAAQ,GAAG,mBAAmB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,CAAoB,CAAC,CAAA;YACpF,IAAI,CAAC,gBAAgB,CAAC,QAAQ,EAAE,eAAe,EAAE,KAAK,CAAC,CAAA;QACzD,CAAC;QAED,KAAK,IAAI,MAAM,GAAG,CAAC,EAAE,MAAM,GAAG,WAAW,EAAE,MAAM,EAAE,EAAE,CAAC;YACpD,MAAM,QAAQ,GAAG,mBAAmB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,GAAG,MAAM,CAAoB,CAAC,CAAA;YACzG,IAAI,CAAC,gBAAgB,CAAC,QAAQ,EAAE,eAAe,EAAE,KAAK,CAAC,CAAA;QACzD,CAAC;IACH,CAAC;IAEO,gBAAgB,CAAC,QAA2B,EAAE,eAAgC,EAAE,KAAkC;QACxH,MAAM,MAAM,GAAG,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAA;QACpD,MAAM,WAAW,GAAG,QAAQ,CAAC,KAAK,CAAC,OAAO,CAAC,EAAE,CAAC,eAAe,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,eAAe,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,CAAA;QAEpH,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,MAAM,OAAO,GAAG,eAAe,CAAC,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAA;YACvD,MAAM,cAAc,GAAG,KAAK,CAAC,cAAc;gBACzC,CAAC,CAAC,EAAE,IAAI,EAAE,eAA2C,EAAE;gBACvD,CAAC,CAAC,SAAS,CAAA;YAEb,IAAI,CAAC,UAAU,CACb,SAAS,OAAO,4FAA4F,EAC5G,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,EAAE,QAAQ,EAAE,QAAQ,IAAI,OAAO,CAAC,QAAQ,CAAC,EAClF,cAAc,CACf,CAAA;YAED,KAAK,CAAC,cAAc,GAAG,KAAK,CAAA;QAC9B,CAAC;QAED,IAAI,CAAC,WAAW,IAAI,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,EAAE,CAAC;YAC1D,IAAI,CAAC,aAAa,CAAC,MAAM,EAAE,eAAe,EAAE,KAAK,CAAC,CAAA;QACpD,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,gCAAiC,SAAQ,UAAyC;IAC7F,MAAM,CAAC,QAAQ,GAAG,kCAAkC,CAAA;IACpD,MAAM,CAAC,eAAe,GAAG,IAAI,CAAA;IAC7B,MAAM,CAAC,oBAAoB,GAAG,IAAI,CAAA;IAElC,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,SAAS;SACpB,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,mCAAmC,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAE/E,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;IAED,OAAO,CAAC,OAAmD,EAAE,MAAmB;QAC9E,IAAI,CAAC,OAAO,CAAC,cAAc;YAAE,OAAO,IAAI,CAAA;QAExC,MAAM,eAAe,GAAG,OAAO,CAAC,cAAc,CAAC,IAAI,CAAA;QACnD,MAAM,QAAQ,GAAG,eAAe,CAAC,eAAkC,CAAC,CAAA;QACpE,IAAI,CAAC,QAAQ;YAAE,OAAO,IAAI,CAAA;QAE1B,MAAM,mBAAmB,GAAG,QAAQ,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAA;QAC7D,IAAI,mBAAmB,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC;YAAE,OAAO,IAAI,CAAA;QAExE,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,CAAC,GAAG,mBAAmB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAA;QAC/E,MAAM,WAAW,GAAG,qBAAqB,CAAC,mBAAmB,EAAE,SAAS,CAAC,CAAA;QACzE,MAAM,WAAW,GAAG,qBAAqB,CAAC,mBAAmB,EAAE,SAAS,EAAE,WAAW,CAAC,CAAA;QAEtF,IAAI,WAAW,KAAK,CAAC,IAAI,WAAW,KAAK,CAAC;YAAE,OAAO,IAAI,CAAA;QAEvD,MAAM,UAAU,GAAG,eAAe,CAAC,MAAM,CAAC,KAAK,EAAE,eAAkC,CAAC,CAAA;QACpF,IAAI,CAAC,UAAU;YAAE,OAAO,IAAI,CAAA;QAE5B,IAAI,EAAE,KAAK,EAAE,WAAW,EAAE,KAAK,EAAE,gBAAgB,EAAE,GAAG,UAAU,CAAA;QAChE,IAAI,UAAU,GAAG,KAAK,CAAA;QAEtB,MAAM,YAAY,GAAG,CAAC,QAA2B,EAAE,QAA4B,EAAQ,EAAE;YACvF,MAAM,WAAW,GAAG,QAAQ,CAAC,KAAK,CAAC,OAAO,CAAC,EAAE,CAAC,eAAe,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,eAAe,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,CAAA;YAEpH,IAAI,WAAW,EAAE,CAAC;gBAChB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,QAAQ,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;oBACzC,mBAAmB,CAAC,QAAQ,CAAC,CAAC,CAAW,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAA;gBACzD,CAAC;gBAED,IAAI,QAAQ,KAAK,QAAQ,EAAE,CAAC;oBAC1B,WAAW,CAAC,MAAM,CAAC,gBAAgB,EAAE,CAAC,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAA;oBACpD,gBAAgB,EAAE,CAAA;gBACpB,CAAC;qBAAM,CAAC;oBACN,WAAW,CAAC,MAAM,CAAC,gBAAgB,GAAG,CAAC,EAAE,CAAC,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAA;gBAC1D,CAAC;YACH,CAAC;iBAAM,CAAC;gBACN,IAAI,UAAU;oBAAE,OAAM;gBAEtB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,QAAQ,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;oBACzC,mBAAmB,CAAC,QAAQ,CAAC,CAAC,CAAW,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAA;gBACzD,CAAC;gBAED,MAAM,OAAO,GAAG,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,EAAE,eAAkC,EAAE,aAAa,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;gBAE1H,WAAW,CAAC,gBAAgB,CAAC,GAAG,OAAO,CAAA;gBACvC,WAAW,GAAG,OAAO,CAAC,IAAc,CAAA;gBACpC,gBAAgB,GAAG,CAAC,CAAA;gBACpB,UAAU,GAAG,IAAI,CAAA;YACnB,CAAC;QACH,CAAC,CAAA;QAED,KAAK,IAAI,KAAK,GAAG,CAAC,EAAE,KAAK,GAAG,WAAW,EAAE,KAAK,EAAE,EAAE,CAAC;YACjD,MAAM,QAAQ,GAAG,mBAAmB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,CAAoB,CAAC,CAAA;YACpF,YAAY,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAA;QAClC,CAAC;QAED,KAAK,IAAI,MAAM,GAAG,CAAC,EAAE,MAAM,GAAG,WAAW,EAAE,MAAM,EAAE,EAAE,CAAC;YACpD,MAAM,QAAQ,GAAG,mBAAmB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,GAAG,MAAM,CAAoB,CAAC,CAAA;YACzG,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QACjC,CAAC;QAED,OAAO,MAAM,CAAA;IACf,CAAC"}

package/dist/{src/rules → rules}/erb-no-empty-tags.js

@@ -14,7 +14,7 @@ class ERBNoEmptyTagsVisitor extends BaseRuleVisitor {
     }
 }
 export class ERBNoEmptyTagsRule extends ParserRule {
-    name = "erb-no-empty-tags";
+    static ruleName = "erb-no-empty-tags";
     get defaultConfig() {
         return {
             enabled: true,
@@ -22,7 +22,7 @@ export class ERBNoEmptyTagsRule extends ParserRule {
         };
     }
     check(result, context) {
-        const visitor = new ERBNoEmptyTagsVisitor(this.name, context);
+        const visitor = new ERBNoEmptyTagsVisitor(this.ruleName, context);
         visitor.visit(result.value);
         return visitor.offenses;
     }

package/dist/rules/erb-no-empty-tags.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"erb-no-empty-tags.js","sourceRoot":"","sources":["../../src/rules/erb-no-empty-tags.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAEjD,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AAIxC,MAAM,qBAAsB,SAAQ,eAAe;IACjD,mBAAmB,CAAC,IAAoB;QACtC,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAA;QAE1B,MAAM,EAAE,OAAO,EAAE,WAAW,EAAE,GAAG,IAAI,CAAA;QAErC,IAAI,CAAC,OAAO;YAAE,OAAM;QACpB,IAAI,WAAW,EAAE,KAAK,KAAK,EAAE;YAAE,OAAM;QACrC,IAAI,OAAO,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC;YAAE,OAAM;QAE3C,IAAI,CAAC,UAAU,CACb,oEAAoE,EACpE,IAAI,CAAC,QAAQ,CACd,CAAA;IACH,CAAC;CACF;AAED,MAAM,OAAO,kBAAmB,SAAQ,UAAU;IAChD,MAAM,CAAC,QAAQ,GAAG,mBAAmB,CAAA;IAErC,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,OAAO;SAClB,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,qBAAqB,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAEjE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC"}

package/dist/{src/rules → rules}/erb-no-extra-newline.js

@@ -1,23 +1,6 @@
-import { BaseSourceRuleVisitor } from "./rule-utils.js";
+import { Location } from "@herb-tools/core";
+import { BaseSourceRuleVisitor, positionFromOffset } from "./rule-utils.js";
 import { SourceRule } from "../types.js";
-import { Location, Position } from "@herb-tools/core";
-function positionFromOffset(source, offset) {
-    let line = 1;
-    let column = 0;
-    let currentOffset = 0;
-    for (let i = 0; i < source.length && currentOffset < offset; i++) {
-        const char = source[i];
-        currentOffset++;
-        if (char === "\n") {
-            line++;
-            column = 0;
-        }
-        else {
-            column++;
-        }
-    }
-    return new Position(line, column);
-}
 class ERBNoExtraNewLineVisitor extends BaseSourceRuleVisitor {
     visitSource(source) {
         if (source.length === 0)
@@ -41,7 +24,7 @@ class ERBNoExtraNewLineVisitor extends BaseSourceRuleVisitor {
 }
 export class ERBNoExtraNewLineRule extends SourceRule {
     static autocorrectable = true;
-    name = "erb-no-extra-newline";
+    static ruleName = "erb-no-extra-newline";
     get defaultConfig() {
         return {
             enabled: true,
@@ -49,7 +32,7 @@ export class ERBNoExtraNewLineRule extends SourceRule {
         };
     }
     check(source, context) {
-        const visitor = new ERBNoExtraNewLineVisitor(this.name, context);
+        const visitor = new ERBNoExtraNewLineVisitor(this.ruleName, context);
         visitor.visit(source);
         return visitor.offenses;
     }

package/dist/rules/erb-no-extra-newline.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"erb-no-extra-newline.js","sourceRoot":"","sources":["../../src/rules/erb-no-extra-newline.ts"],"names":[],"mappings":"AAAA,OAAO,EAAa,QAAQ,EAAE,MAAM,kBAAkB,CAAA;AAEtD,OAAO,EAAE,qBAAqB,EAAE,kBAAkB,EAAE,MAAM,iBAAiB,CAAA;AAC3E,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AAQxC,MAAM,wBAAyB,SAAQ,qBAAsD;IACjF,WAAW,CAAC,MAAc;QAClC,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;YAAE,OAAM;QAE/B,MAAM,KAAK,GAAG,SAAS,CAAA;QAEvB,IAAI,KAA6B,CAAA;QAEjC,OAAO,CAAC,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YAC7C,MAAM,WAAW,GAAG,KAAK,CAAC,KAAK,GAAG,CAAC,CAAA;YACnC,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAA;YAC/C,MAAM,KAAK,GAAG,kBAAkB,CAAC,MAAM,EAAE,WAAW,CAAC,CAAA;YACrD,MAAM,GAAG,GAAG,kBAAkB,CAAC,MAAM,EAAE,SAAS,CAAC,CAAA;YACjD,MAAM,QAAQ,GAAG,IAAI,QAAQ,CAAC,KAAK,EAAE,GAAG,CAAC,CAAA;YAEzC,MAAM,UAAU,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAA;YAEtC,IAAI,CAAC,UAAU,CACb,qCAAqC,UAAU,UAAU,UAAU,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,kDAAkD,EAC9I,QAAQ,EACR;gBACE,IAAI,EAAE,IAAmB;gBACzB,WAAW;gBACX,SAAS;aACV,CACF,CAAA;QACH,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,qBAAsB,SAAQ,UAAU;IACnD,MAAM,CAAC,eAAe,GAAG,IAAI,CAAA;IAC7B,MAAM,CAAC,QAAQ,GAAG,sBAAsB,CAAA;IAExC,IAAI,aAAa;QACf,OAAO;YACL,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE,OAAO;SAClB,CAAA;IACH,CAAC;IAED,KAAK,CAAC,MAAc,EAAE,OAA8B;QAClD,MAAM,OAAO,GAAG,IAAI,wBAAwB,CAAC,IAAI,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;QAEpE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,CAAA;QAErB,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;IAED,OAAO,CAAC,OAAqD,EAAE,MAAc,EAAE,QAA+B;QAC5G,IAAI,CAAC,OAAO,CAAC,cAAc;YAAE,OAAO,IAAI,CAAA;QAExC,MAAM,EAAE,WAAW,EAAE,SAAS,EAAE,GAAG,OAAO,CAAC,cAAc,CAAA;QAEzD,MAAM,MAAM,GAAG,MAAM,CAAC,SAAS,CAAC,CAAC,EAAE,WAAW,CAAC,CAAA;QAC/C,MAAM,KAAK,GAAG,MAAM,CAAC,SAAS,CAAC,SAAS,CAAC,CAAA;QAEzC,OAAO,MAAM,GAAG,KAAK,CAAA;IACvB,CAAC"}

package/dist/{src/rules → rules}/erb-no-extra-whitespace-inside-tags.js

@@ -11,16 +11,39 @@ class ERBNoExtraWhitespaceInsideTagsVisitor extends BaseRuleVisitor {
         if (this.hasExtraLeadingWhitespace(value)) {
             this.reportWhitespace(node, openTag, closeTag, value, "start", 0, `Remove extra whitespace after \`${openTag.value}\`.`, "after-open");
         }
-        if (openTag.value === "<%#" && value.startsWith("=") && value.length > 1) {
-            const afterEquals = value.substring(1);
-            if (afterEquals.match(/^\s{2,}/) && !afterEquals.startsWith("  \n") && !afterEquals.startsWith("\n")) {
-                this.reportWhitespace(node, openTag, closeTag, value, "start", 1, `Remove extra whitespace after \`<%#=\`.`, "after-comment-equals");
+        if (openTag.value === "<%#") {
+            const prefix = this.getCommentedTagPrefix(value);
+            if (prefix) {
+                const afterPrefix = value.substring(prefix.length);
+                const tag = `<%#${prefix}`;
+                const hasExtraWhitespace = afterPrefix.match(/^\s{2,}/) && !afterPrefix.startsWith("  \n") && !afterPrefix.startsWith("\n");
+                if (hasExtraWhitespace) {
+                    this.reportWhitespace(node, openTag, closeTag, value, "start", prefix.length, `Remove extra whitespace after \`${tag}\`. This looks like a temporarily commented ERB tag.`, "after-comment-equals", "info");
+                }
+                else {
+                    this.addOffense(`\`${tag}\` looks like a temporarily commented ERB tag.`, openTag.location, { node, openTag, closeTag, content: value, fixType: "after-comment-equals", unsafe: true }, "info");
+                }
             }
         }
         if (this.hasExtraTrailingWhitespace(value)) {
             this.reportWhitespace(node, openTag, closeTag, value, "end", 0, `Remove extra whitespace before \`${closeTag.value}\`.`, "before-close");
         }
     }
+    getCommentedTagPrefix(content) {
+        if (content.startsWith("graphql"))
+            return "graphql";
+        if (content.startsWith("%="))
+            return "%=";
+        if (content.startsWith("=="))
+            return "==";
+        if (content.startsWith("%"))
+            return "%";
+        if (content.startsWith("="))
+            return "=";
+        if (content.startsWith("-"))
+            return "-";
+        return null;
+    }
     hasExtraLeadingWhitespace(content) {
         return content.startsWith("  ") && !content.startsWith("  \n");
     }
@@ -41,20 +64,21 @@ class ERBNoExtraWhitespaceInsideTagsVisitor extends BaseRuleVisitor {
             return Location.from(contentLocation.end.line, contentLocation.end.column - length, contentLocation.end.line, contentLocation.end.column);
         }
     }
-    reportWhitespace(node, openTag, closeTag, content, position, offset, message, fixType) {
+    reportWhitespace(node, openTag, closeTag, content, position, offset, message, fixType, severity, unsafe) {
         const location = this.getWhitespaceLocation(node, content, position, offset);
         this.addOffense(message, location, {
             node,
             openTag,
             closeTag,
             content,
-            fixType
-        });
+            fixType,
+            unsafe,
+        }, severity);
     }
 }
 export class ERBNoExtraWhitespaceRule extends ParserRule {
     static autocorrectable = true;
-    name = "erb-no-extra-whitespace-inside-tags";
+    static ruleName = "erb-no-extra-whitespace-inside-tags";
     get defaultConfig() {
         return {
             enabled: true,
@@ -62,7 +86,7 @@ export class ERBNoExtraWhitespaceRule extends ParserRule {
         };
     }
     check(result, context) {
-        const visitor = new ERBNoExtraWhitespaceInsideTagsVisitor(this.name, context);
+        const visitor = new ERBNoExtraWhitespaceInsideTagsVisitor(this.ruleName, context);
         visitor.visit(result.value);
         return visitor.offenses;
     }
@@ -80,12 +104,14 @@ export class ERBNoExtraWhitespaceRule extends ParserRule {
             case "after-open":
                 node.content.value = content.replace(/^\s{2,}/, " ");
                 break;
-            case "after-comment-equals":
-                if (content.startsWith("=")) {
-                    const afterEquals = content.substring(1);
-                    node.content.value = "= " + afterEquals.replace(/^\s{2,}/, "");
+            case "after-comment-equals": {
+                const prefix = content.startsWith("graphql") ? "graphql" : content.startsWith("%=") ? "%=" : content.startsWith("==") ? "==" : content.startsWith("%") ? "%" : content.startsWith("=") ? "=" : content.startsWith("-") ? "-" : null;
+                if (prefix) {
+                    const afterPrefix = content.substring(prefix.length);
+                    node.content.value = prefix + " " + afterPrefix.replace(/^\s{2,}/, "");
                 }
                 break;
+            }
             default:
                 return null;
         }