a11y-lint 0.10.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ecfe715cda47636bb8dc87f9cf3655c641d33d2e10cfd8471be73b2dd7336a1f
-  data.tar.gz: 818bb7d22993ac9036ac148d60adce32c2ef7ee342bc741bbd8e64d893414d2f
+  metadata.gz: e1c92e880f51310dfe123ffdbc872bc661a781371365c53dd1a1972e302dcc1d
+  data.tar.gz: dfb7186f222b4c87ffdc818e843907435eebda8e99cccdc945cf1f4d8b77708f
 SHA512:
-  metadata.gz: 3a0d354908cfc0c670db3556f0854e1e91f0486ae502ff6f93395a95923d1a6b7b456bd625ffb476ba8f7e8d3f8bea5f859ed4875e0cf172770be5e87da8517f
-  data.tar.gz: e0e5e582649edc60f0df35635af190beb49b2ab69c9da392e9ae31326918c6704962b5a5cd6d60f8e1bb8bf47474a9d4b2fdb18baf424f3013871a7d3cc17a10
+  metadata.gz: 0fde7a03bb0e20294c451a57da6aa487fd7aa6cf6b3bde0d41e6b7a323ededf15c626e6b06e7dd55f37b109a0371cc24b5708fd5528f959bb96a6269642152f8
+  data.tar.gz: b43f87086fdabf16d139639d169be80cadea3da8c61b09e7a92cc66f4c5d59011dce8b7433307877b652b4b85c2d4cb923c6ed9a95695094b035cbcdbe2d6d0a

data/.claude/rules/testing.md

@@ -49,4 +49,4 @@ globs: test/**/*.rb
 
   Each style must be tested for both the offense case and the "passes with fix" case.
 
-- Every rule must be tested against both the Slim and ERB pipelines.
+- Every rule must be tested against the Slim, ERB, and Phlex pipelines.

data/CHANGELOG.md

@@ -7,6 +7,28 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.11.0] - 2026-04-21
+
+### Added
+
+- `AreaMissingAlt` rule: detects `<area>` tags without `alt` attributes (WCAG 1.1.1)
+- `InputImageMissingAlt` rule: detects `<input type="image">` tags without `alt` attributes (WCAG 1.1.1)
+- `ImageSubmitTagMissingAlt` rule: detects `image_submit_tag` calls without an `alt` option (WCAG 1.1.1)
+- `InputMissingAutocomplete` rule: detects `<input>` elements without an `autocomplete` attribute (WCAG 1.3.5)
+- `AnchorMissingAccessibleName` rule: detects `<a>` elements without text content, `aria-label`, or a child image with `alt` (WCAG 4.1.2)
+- `ButtonMissingAccessibleName` rule: detects `<button>` elements without text content, `aria-label`, or a child image with `alt` (WCAG 4.1.2)
+
+### Changed
+
+- **Breaking:** Split `MissingAccessibleName` into `LinkToMissingAccessibleName` (for `link_to`/`external_link_to`) and `ButtonTagMissingAccessibleName` (for `button_tag`) to follow one-rule-per-helper convention
+- **Breaking:** Rename `A11y::Lint::Rule` to `A11y::Lint::NodeRule`. Custom rules that subclass the base class must update the parent class name
+- Auto-require all rule files instead of listing them individually
+- Node attribute storage now preserves string values, enabling rules to check attribute values (not just existence)
+
+### Fixed
+
+- `LinkToMissingAccessibleName` / `ButtonTagMissingAccessibleName`: no longer flag `link_to`/`button_tag` blocks containing an `image_tag` with a non-empty `alt` option
+
 ## [0.10.0] - 2026-04-15
 
 ### Changed

data/CLAUDE.md

@@ -32,6 +32,16 @@ a11y-lint is a Ruby gem (v0.1.0) for accessibility linting. It uses the `A11y::L
 - **Tests:** `test/` directory using Minitest; test helper at `test/test_helper.rb`
 - **Dummy app:** `test/fixtures/dummy_app/` — a fixture app with Slim/ERB/Phlex templates for end-to-end smoke testing before releases (`bundle exec a11y-lint test/fixtures/dummy_app`)
 
+## Rule Scoping Convention
+
+Each rule targets **one element or helper**, unless multiple elements share identical check logic (e.g., `<ul>` and `<ol>` in `ListInvalidChildren`).
+
+- **HTML element rules** are named after the element: `ImgMissingAlt`, `AreaMissingAlt`, `AnchorMissingAccessibleName`
+- **Rails helper rules** are named after the helper: `ImageTagMissingAlt`, `LinkToMissingAccessibleName`
+- HTML/helper pairs mirror each other (e.g., `ImgMissingAlt` / `ImageTagMissingAlt`)
+
+Do not bundle unrelated elements or helpers into a single rule just because they share a WCAG criterion.
+
 ## Code Style
 
 RuboCop is configured in `.rubocop.yml`:

data/lib/a11y/lint/block_inspection.rb

@@ -6,7 +6,7 @@ module A11y
     # Depends on the host class implementing
     # #block_has_text_children? and #block_body_codes.
     module BlockInspection
-      ICON_HELPERS = %w[inline_svg icon image_tag svg_icon].freeze
+      ICON_HELPERS = %w[inline_svg icon svg_icon].freeze
 
       def block_has_only_icon_helpers?
         return false if block_has_text_children?
@@ -14,15 +14,28 @@ module A11y
         codes = block_body_codes
         return true unless codes&.any?
 
-        codes.all? { |code| icon_helper_call?(code) }
+        codes.all? do |code|
+          icon_helper?(code) || decorative_image_tag?(code)
+        end
       end
 
       private
 
-      def icon_helper_call?(code)
+      def icon_helper?(code)
         call = RubyCode.new(code).call_node
         call && ICON_HELPERS.include?(call.method_name)
       end
+
+      # image_tag counts as decorative only when it lacks a non-empty
+      # alt. A non-empty alt provides the accessible name, matching how
+      # <a><img alt="Home"></a> is treated by
+      # AnchorMissingAccessibleName#child_image_has_alt?.
+      def decorative_image_tag?(code)
+        call = RubyCode.new(code).call_node
+        call &&
+          call.method_name == "image_tag" &&
+          !call.keyword_non_empty?(:alt)
+      end
     end
   end
 end

data/lib/a11y/lint/call_node.rb

@@ -30,6 +30,23 @@ module A11y
         end
       end
 
+      # True when the keyword is present AND its value is a non-empty
+      # string literal OR any non-string expression (dynamic — can't be
+      # statically proven empty, so treat as providing content).
+      # False when the key is absent or the value is an empty string
+      # literal.
+      def keyword_non_empty?(key)
+        return false unless (kw_hash = find_keyword_hash)
+
+        assoc = kw_hash.elements.find { |a| key_name(a) == key.to_s }
+        return false unless assoc
+
+        value = assoc.value
+        return !value.unescaped.empty? if value.is_a?(Prism::StringNode)
+
+        true
+      end
+
       def positional_args
         return [] unless @prism_node.arguments
 

data/lib/a11y/lint/cli.rb

@@ -99,7 +99,7 @@ module A11y
 
         Rules.constants.filter_map do |name|
           klass = Rules.const_get(name)
-          next unless klass.is_a?(Class) && klass < Rule
+          next unless klass.is_a?(Class) && klass < NodeRule
 
           klass if configuration.enabled?(klass.rule_name)
         end

data/lib/a11y/lint/erb_element_node.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    # Wraps a Nokogiri HTML element from an ERB template
+    # as a queryable node for lint rules.
+    class ErbElementNode
+      attr_reader :line
+
+      def initialize(nokogiri_node:, line:, has_erb_output: false)
+        @nokogiri_node = nokogiri_node
+        @line = line
+        @has_erb_output = has_erb_output
+      end
+
+      def tag_name
+        @nokogiri_node.name
+      end
+
+      def attribute?(name)
+        attributes.key?(name)
+      end
+
+      def attributes
+        @attributes ||=
+          @nokogiri_node
+          .attributes
+          .transform_values(&:value)
+      end
+
+      def call_node
+        nil
+      end
+
+      def ruby_code
+        nil
+      end
+
+      def block_body_codes
+        nil
+      end
+
+      def block_has_text_children?
+        false
+      end
+
+      def text_content?
+        @has_erb_output || !@nokogiri_node.text.strip.empty?
+      end
+
+      # Returns direct element children wrapped as ErbElementNode objects.
+      def children
+        @nokogiri_node.element_children.map do |child|
+          ErbElementNode.new(nokogiri_node: child, line: child.line)
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/erb_output_node.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    # Wraps an extracted ERB output tag (<%= ... %>)
+    # as a queryable node for lint rules.
+    class ErbOutputNode
+      include BlockInspection
+
+      attr_reader(:block_body_codes, :line, :ruby_code)
+
+      def initialize(
+        ruby_code:, line:,
+        block_body_codes: nil, block_has_text_children: false
+      )
+        @ruby_code = ruby_code
+        @line = line
+        @block_body_codes = block_body_codes
+        @block_has_text_children = block_has_text_children
+      end
+
+      def tag_name
+        nil
+      end
+
+      def attribute?(name)
+        attributes.key?(name)
+      end
+
+      def attributes
+        {}
+      end
+
+      def call_node
+        @call_node ||= RubyCode.new(ruby_code).call_node
+      end
+
+      def block_has_text_children?
+        @block_has_text_children
+      end
+
+      def children
+        []
+      end
+    end
+  end
+end

data/lib/a11y/lint/erb_runner.rb

@@ -8,6 +8,7 @@ module A11y
     class ErbRunner
       ERB_TAG = /<%.*?%>/m
       ERB_OUTPUT_TAG = /<%=\s*(.*?)\s*-?%>/m
+      ERB_OUTPUT_MARKER = "A11Y_LINT_ERB_OUTPUT"
       VOID_ELEMENTS = %w[
         area base br col embed hr img input
         link meta param source track wbr
@@ -32,19 +33,27 @@ module A11y
       attr_reader :rules
 
       def check_html_nodes(source)
-        html = source.gsub(ERB_TAG, "")
+        html = source.gsub(ERB_OUTPUT_TAG, ERB_OUTPUT_MARKER)
+        html = html.gsub(ERB_TAG, "")
         doc = Nokogiri::HTML4::DocumentFragment.parse(html)
 
         doc.traverse do |nokogiri_node|
           next unless nokogiri_node.element?
           next unless source_confirmed_element?(html, nokogiri_node.name)
 
-          check_node(
-            ErbNode.new(nokogiri_node: nokogiri_node, line: nokogiri_node.line)
-          )
+          node = build_erb_element_node(nokogiri_node)
+          check_node(node)
         end
       end
 
+      def build_erb_element_node(nokogiri_node)
+        ErbElementNode.new(
+          nokogiri_node: nokogiri_node,
+          line: nokogiri_node.line,
+          has_erb_output: nokogiri_node.text.include?(ERB_OUTPUT_MARKER)
+        )
+      end
+
       def source_confirmed_element?(html, tag_name)
         VOID_ELEMENTS.include?(tag_name) || html.include?("</#{tag_name}>")
       end
@@ -64,7 +73,7 @@ module A11y
         block_body_codes, block_has_text =
           extract_block_info(source, code, match_end)
 
-        ErbNode.new(
+        ErbOutputNode.new(
           ruby_code: code, line: line,
           block_body_codes: block_body_codes,
           block_has_text_children: block_has_text

data/lib/a11y/lint/{rule.rb → node_rule.rb}

@@ -2,8 +2,8 @@
 
 module A11y
   module Lint
-    # Base class for accessibility lint rules.
-    class Rule
+    # Base class for per-node accessibility lint rules.
+    class NodeRule
       def self.check(node)
         new(node).check
       end

data/lib/a11y/lint/phlex_node.rb

@@ -22,7 +22,8 @@ module A11y
         line:, tag_name: nil, attributes: {},
         call_node: nil, children: [],
         block_body_codes: nil,
-        block_has_text_children: false
+        block_has_text_children: false,
+        text_content: false
       )
         @tag_name = tag_name
         @attributes = attributes
@@ -31,6 +32,7 @@ module A11y
         @children = children
         @block_body_codes = block_body_codes
         @block_has_text_children = block_has_text_children
+        @text_content = text_content
       end
       # rubocop:enable Metrics/ParameterLists
 
@@ -46,13 +48,18 @@ module A11y
         @block_has_text_children
       end
 
-      def self.build_tag(call_node, children: [])
+      def text_content?
+        @text_content
+      end
+
+      def self.build_tag(call_node, children: [], text_content: false)
         name = call_node.name.to_s
         new(
           tag_name: html_tag_name(name),
           attributes: extract_attributes(call_node),
           line: call_node.location.start_line,
-          children: children
+          children: children,
+          text_content: text_content
         )
       end
 
@@ -74,7 +81,7 @@ module A11y
 
         kwarg_nodes(call_node).each_with_object({}) do |elem, h|
           key = kwarg_key(elem.key)
-          h[key] = true if key
+          h[key] = kwarg_value(elem.value) if key
         end
       end
 
@@ -91,7 +98,15 @@ module A11y
         end
       end
 
-      private_class_method :kwarg_key, :kwarg_nodes,
+      def self.kwarg_value(value_node)
+        case value_node
+        when Prism::StringNode then value_node.unescaped
+        when Prism::SymbolNode then value_node.value
+        else true
+        end
+      end
+
+      private_class_method :kwarg_key, :kwarg_nodes, :kwarg_value,
                            :extract_attributes
     end
   end

data/lib/a11y/lint/phlex_runner.rb

@@ -46,7 +46,10 @@ module A11y
 
       def check_tag(node)
         children = collect_block_children(node.block)
-        check_node(PhlexNode.build_tag(node, children:))
+        has_text = tag_block_has_text?(node.block, children)
+        check_node(
+          PhlexNode.build_tag(node, children:, text_content: has_text)
+        )
       end
 
       def check_helper(node)
@@ -80,8 +83,25 @@ module A11y
 
       def gather_tag_child(child, result)
         kids = collect_block_children(child.block)
-        result << PhlexNode.build_tag(child, children: kids)
-        check_node(result.last)
+        has_text = tag_block_has_text?(child.block, kids)
+        tag = PhlexNode.build_tag(
+          child, children: kids, text_content: has_text
+        )
+        result << tag
+        check_node(tag)
+      end
+
+      def tag_block_has_text?(block, children)
+        return false unless block.is_a?(Prism::BlockNode)
+
+        scan_for_text(block) || children.any?(&:text_content?)
+      end
+
+      def scan_for_text(node)
+        node.child_nodes.compact.any? do |child|
+          text_call?(child) || child.is_a?(Prism::YieldNode) ||
+            (!receiverless_call?(child) && scan_for_text(child))
+        end
       end
 
       def analyze_helper_block(call_node)
@@ -116,6 +136,10 @@ module A11y
         receiverless_call?(node) && PhlexNode.html_tag?(node.name.to_s)
       end
 
+      def text_call?(node)
+        receiverless_call?(node) && node.name.to_s == "plain"
+      end
+
       def receiverless_call?(node)
         node.is_a?(Prism::CallNode) && node.receiver.nil?
       end

data/lib/a11y/lint/phlex_tags.rb

@@ -14,6 +14,7 @@ module A11y
         %w[
           a
           abbr
+          area
           address
           article
           aside

data/lib/a11y/lint/rules/perceivable/area_missing_alt.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    module Rules
+      # Checks that area tags include an alt attribute (WCAG 1.1.1).
+      # https://www.w3.org/WAI/WCAG21/Techniques/html/H24
+      class AreaMissingAlt < NodeRule
+        def check
+          return unless an_area_without_an_alt_attribute?
+
+          "area tag is missing an alt attribute (WCAG 1.1.1)"
+        end
+
+        private
+
+        def an_area_without_an_alt_attribute?
+          node.tag_name == "area" && !node.attribute?("alt")
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/rules/perceivable/image_submit_tag_missing_alt.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    module Rules
+      # Checks that image_submit_tag calls include an alt option (WCAG 1.1.1).
+      # https://www.w3.org/WAI/WCAG21/Techniques/html/H36
+      class ImageSubmitTagMissingAlt < NodeRule
+        def check
+          return if no_offense?
+
+          "image_submit_tag is missing an alt option (WCAG 1.1.1)"
+        end
+
+        private
+
+        def no_offense?
+          !image_submit_tag || image_submit_tag.keyword?(:alt)
+        end
+
+        def image_submit_tag
+          @image_submit_tag ||= node.call_node&.find("image_submit_tag")
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/rules/perceivable/image_tag_missing_alt.rb

@@ -4,7 +4,7 @@ module A11y
   module Lint
     module Rules
       # Checks that image_tag calls include an alt option (WCAG 1.1.1).
-      class ImageTagMissingAlt < Rule
+      class ImageTagMissingAlt < NodeRule
         def check
           return if no_offense?
 

data/lib/a11y/lint/rules/perceivable/img_missing_alt.rb

@@ -4,7 +4,7 @@ module A11y
   module Lint
     module Rules
       # Checks that img tags include an alt attribute (WCAG 1.1.1).
-      class ImgMissingAlt < Rule
+      class ImgMissingAlt < NodeRule
         def check
           return unless an_image_without_an_alt_attribute?
 

data/lib/a11y/lint/rules/perceivable/input_image_missing_alt.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    module Rules
+      # Checks that <input type="image"> tags include an alt
+      # attribute (WCAG 1.1.1).
+      # https://www.w3.org/WAI/WCAG21/Techniques/html/H36
+      class InputImageMissingAlt < NodeRule
+        def check
+          return unless an_input_image_without_an_alt_attribute?
+
+          "input type=\"image\" is missing an alt attribute (WCAG 1.1.1)"
+        end
+
+        private
+
+        def an_input_image_without_an_alt_attribute?
+          node.tag_name == "input" &&
+            node.attributes["type"] == "image" &&
+            !node.attribute?("alt")
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/rules/perceivable/input_missing_autocomplete.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    module Rules
+      # Checks that <input> elements include an autocomplete
+      # attribute (WCAG 1.3.5).
+      # https://www.w3.org/WAI/WCAG21/Understanding/identify-input-purpose.html
+      class InputMissingAutocomplete < NodeRule
+        EXCLUDED_TYPES = %w[
+          button
+          checkbox
+          file
+          hidden
+          image
+          radio
+          reset
+          submit
+        ].freeze
+
+        def check
+          return unless an_input_missing_autocomplete?
+
+          "input is missing an autocomplete attribute (WCAG 1.3.5)"
+        end
+
+        private
+
+        def an_input_missing_autocomplete?
+          node.tag_name == "input" &&
+            !EXCLUDED_TYPES.include?(node.attributes["type"]) &&
+            !node.attribute?("autocomplete")
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/rules/perceivable/list_invalid_children.rb

@@ -5,7 +5,7 @@ module A11y
     module Rules
       # Checks that <ul> and <ol> only directly contain <li>, <script>,
       # or <template> elements (WCAG 1.3.1).
-      class ListInvalidChildren < Rule
+      class ListInvalidChildren < NodeRule
         LIST_TAGS = %w[ul ol].freeze
         ALLOWED_CHILDREN = %w[li script template].freeze
 

data/lib/a11y/lint/rules/robust/anchor_missing_accessible_name.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    module Rules
+      # Checks that <a> elements with no text content or meaningful
+      # child elements include an aria-label (WCAG 4.1.2).
+      class AnchorMissingAccessibleName < NodeRule
+        def check
+          return if no_offense?
+
+          "a tag is missing an accessible name " \
+            "requires an aria-label (WCAG 4.1.2)"
+        end
+
+        private
+
+        def no_offense?
+          node.tag_name != "a" ||
+            aria_label? ||
+            node.text_content? ||
+            child_image_has_alt?
+        end
+
+        def aria_label?
+          node.attribute?("aria-label") || node.attribute?("aria_label")
+        end
+
+        def child_image_has_alt?
+          node.children.any? do |child|
+            child.tag_name == "img" && non_empty_alt?(child)
+          end
+        end
+
+        def non_empty_alt?(child)
+          alt = child.attributes["alt"]
+          alt.is_a?(String) && !alt.strip.empty?
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/rules/robust/button_missing_accessible_name.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    module Rules
+      # Checks that <button> elements with no text content or meaningful
+      # child elements include an aria-label (WCAG 4.1.2).
+      class ButtonMissingAccessibleName < NodeRule
+        def check
+          return if no_offense?
+
+          "button tag is missing an accessible name " \
+            "requires an aria-label (WCAG 4.1.2)"
+        end
+
+        private
+
+        def no_offense?
+          node.tag_name != "button" ||
+            aria_label? ||
+            node.text_content? ||
+            child_image_has_alt?
+        end
+
+        def aria_label?
+          node.attribute?("aria-label") || node.attribute?("aria_label")
+        end
+
+        def child_image_has_alt?
+          node.children.any? do |child|
+            child.tag_name == "img" && non_empty_alt?(child)
+          end
+        end
+
+        def non_empty_alt?(child)
+          alt = child.attributes["alt"]
+          alt.is_a?(String) && !alt.strip.empty?
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/rules/robust/button_tag_missing_accessible_name.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+module A11y
+  module Lint
+    module Rules
+      # Checks that button_tag calls with empty text or block content
+      # include an aria-label (WCAG 4.1.2).
+      class ButtonTagMissingAccessibleName < NodeRule
+        def check
+          return if no_offense?
+
+          offense_message
+        end
+
+        private
+
+        def no_offense?
+          !helper_call ||
+            aria_label? ||
+            !(helper_call.first_positional_arg_empty_string? ||
+              (helper_call.block? && node.block_has_only_icon_helpers?))
+        end
+
+        def aria_label?
+          helper_call.keyword?(:aria, :label) ||
+            helper_call.keyword?(:"aria-label")
+        end
+
+        def helper_call
+          @helper_call ||= begin
+            call = node.call_node
+            call if call && call.method_name == "button_tag"
+          end
+        end
+
+        def offense_message
+          <<~MSG.strip
+            button_tag missing an accessible name \
+            requires an aria-label (WCAG 4.1.2)
+          MSG
+        end
+      end
+    end
+  end
+end

data/lib/a11y/lint/rules/robust/{missing_accessible_name.rb → link_to_missing_accessible_name.rb}

@@ -3,10 +3,10 @@
 module A11y
   module Lint
     module Rules
-      # Checks that link_to, external_link_to, and button_tag calls with
-      # empty text or block content include an aria-label (WCAG 4.1.2).
-      class MissingAccessibleName < Rule
-        METHODS = %w[link_to external_link_to button_tag].freeze
+      # Checks that link_to and external_link_to calls with empty text or
+      # block content include an aria-label (WCAG 4.1.2).
+      class LinkToMissingAccessibleName < NodeRule
+        METHODS = %w[link_to external_link_to].freeze
 
         def check
           return if no_offense?

data/lib/a11y/lint/slim_node.rb

@@ -62,7 +62,16 @@ module A11y
       def block_has_text_children?
         return false unless slim_output?
 
-        text_content?(@sexp[4])
+        block_text_content?(@sexp[4])
+      end
+
+      # Returns true when the HTML element body contains visible text
+      # or dynamic output (i.e. content that could provide an accessible
+      # name). Only meaningful for HTML tag nodes.
+      def text_content?
+        return false unless html_tag?
+
+        text_or_output?(@sexp[4])
       end
 
       private
@@ -90,11 +99,18 @@ module A11y
         sexp.is_a?(Array) && sexp[0] == :slim && sexp[1] == :output
       end
 
-      def text_content?(sexp)
+      def block_text_content?(sexp)
         return false unless sexp.is_a?(Array)
         return true if slim_text_sexp?(sexp) || html_tag_sexp?(sexp)
 
-        sexp.any? { |child| text_content?(child) }
+        sexp.any? { |child| block_text_content?(child) }
+      end
+
+      def text_or_output?(sexp)
+        return false unless sexp.is_a?(Array)
+        return true if slim_text_sexp?(sexp) || slim_output_sexp?(sexp)
+
+        sexp.any? { |child| text_or_output?(child) }
       end
 
       def slim_text_sexp?(sexp)
@@ -122,7 +138,19 @@ module A11y
         return {} unless html_attributes?
 
         sexp_attributes[2..].each_with_object({}) do |attr_sexp, result|
-          result[attr_sexp[2]] = true if html_attribute?(attr_sexp)
+          next unless html_attribute?(attr_sexp)
+
+          result[attr_sexp[2]] = static_value(attr_sexp[3])
+        end
+      end
+
+      def static_value(value_sexp)
+        if value_sexp.is_a?(Array) && value_sexp[0] == :escape &&
+           value_sexp[2].is_a?(Array) &&
+           value_sexp[2][0] == :slim && value_sexp[2][1] == :interpolate
+          value_sexp[2][2]
+        else
+          true
         end
       end
 

data/lib/a11y/lint/version.rb

@@ -2,6 +2,6 @@
 
 module A11y
   module Lint
-    VERSION = "0.10.0"
+    VERSION = "0.11.0"
   end
 end

data/lib/a11y/lint.rb

@@ -8,15 +8,13 @@ require_relative "lint/call_node"
 require_relative "lint/ruby_code"
 require_relative "lint/block_inspection"
 require_relative "lint/slim_node"
-require_relative "lint/erb_node"
+require_relative "lint/erb_element_node"
+require_relative "lint/erb_output_node"
 require_relative "lint/phlex_tags"
 require_relative "lint/phlex_node"
 require_relative "lint/configuration"
-require_relative "lint/rule"
-require_relative "lint/rules/perceivable/image_tag_missing_alt"
-require_relative "lint/rules/perceivable/img_missing_alt"
-require_relative "lint/rules/perceivable/list_invalid_children"
-require_relative "lint/rules/robust/missing_accessible_name"
+require_relative "lint/node_rule"
+Dir[File.join(__dir__, "lint", "rules", "**", "*.rb")].each { |f| require f }
 require_relative "lint/slim_runner"
 require_relative "lint/erb_runner"
 require_relative "lint/phlex_runner"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: a11y-lint
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Abdullah Hashim
@@ -59,19 +59,27 @@ files:
 - lib/a11y/lint/call_node.rb
 - lib/a11y/lint/cli.rb
 - lib/a11y/lint/configuration.rb
-- lib/a11y/lint/erb_node.rb
+- lib/a11y/lint/erb_element_node.rb
+- lib/a11y/lint/erb_output_node.rb
 - lib/a11y/lint/erb_runner.rb
 - lib/a11y/lint/errors.rb
+- lib/a11y/lint/node_rule.rb
 - lib/a11y/lint/offense.rb
 - lib/a11y/lint/phlex_node.rb
 - lib/a11y/lint/phlex_runner.rb
 - lib/a11y/lint/phlex_tags.rb
 - lib/a11y/lint/ruby_code.rb
-- lib/a11y/lint/rule.rb
+- lib/a11y/lint/rules/perceivable/area_missing_alt.rb
+- lib/a11y/lint/rules/perceivable/image_submit_tag_missing_alt.rb
 - lib/a11y/lint/rules/perceivable/image_tag_missing_alt.rb
 - lib/a11y/lint/rules/perceivable/img_missing_alt.rb
+- lib/a11y/lint/rules/perceivable/input_image_missing_alt.rb
+- lib/a11y/lint/rules/perceivable/input_missing_autocomplete.rb
 - lib/a11y/lint/rules/perceivable/list_invalid_children.rb
-- lib/a11y/lint/rules/robust/missing_accessible_name.rb
+- lib/a11y/lint/rules/robust/anchor_missing_accessible_name.rb
+- lib/a11y/lint/rules/robust/button_missing_accessible_name.rb
+- lib/a11y/lint/rules/robust/button_tag_missing_accessible_name.rb
+- lib/a11y/lint/rules/robust/link_to_missing_accessible_name.rb
 - lib/a11y/lint/slim_node.rb
 - lib/a11y/lint/slim_runner.rb
 - lib/a11y/lint/version.rb

data/lib/a11y/lint/erb_node.rb

@@ -1,70 +0,0 @@
-# frozen_string_literal: true
-
-module A11y
-  module Lint
-    # Wraps a Nokogiri node or extracted ERB output tag
-    # as a queryable node for lint rules.
-    class ErbNode
-      include BlockInspection
-
-      attr_reader(:block_body_codes, :line, :ruby_code)
-
-      def initialize(
-        line:,
-        block_body_codes: nil,
-        block_has_text_children: false,
-        nokogiri_node: nil,
-        ruby_code: nil
-      )
-        @line = line
-        @block_body_codes = block_body_codes
-        @block_has_text_children = block_has_text_children
-        @nokogiri_node = nokogiri_node
-        @ruby_code = ruby_code
-      end
-
-      def tag_name
-        nokogiri_node&.name
-      end
-
-      def attribute?(name)
-        attributes.key?(name)
-      end
-
-      def attributes
-        @attributes ||= extract_attributes
-      end
-
-      def call_node
-        @call_node ||= RubyCode.new(ruby_code).call_node
-      end
-
-      def block_has_text_children?
-        @block_has_text_children
-      end
-
-      # Returns direct element children wrapped as ErbNode objects.
-      def children
-        return [] unless nokogiri_node
-
-        nokogiri_node.element_children.map do |child|
-          ErbNode.new(nokogiri_node: child, line: child.line)
-        end
-      end
-
-      private
-
-      attr_reader(:nokogiri_node)
-
-      def extract_attributes
-        return {} unless nokogiri_node
-
-        nokogiri_node
-          .attributes
-          .each_with_object({}) do |(name, _attr), result|
-            result[name] = true
-        end
-      end
-    end
-  end
-end