view_component_css_dsl 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 494769078a576e9a1f7d3be9b2819ed2c93bf148da58154c68356c01410d9e8f
-  data.tar.gz: 529f35969651591c73f2c21012ab828c84305f94df5317f612fc4dc13b75cc39
+  metadata.gz: 8881683c0d397b134f9fc656eda692df513d6bd3160c23ef397a212f51cd5eee
+  data.tar.gz: 58f07649276631f545c464dc758df825252ef1aeecd03524266528a886a3d9ba
 SHA512:
-  metadata.gz: 365ce4701b070176cfda8d7a7dcb8005011326d3b888315ce0d37ba80cd2026bc8b5e3dcf8833747911e1ad5e5ca4d752ec28cf6dfa6b058240b847950bd1ca3
-  data.tar.gz: fc2ff970cc016c1fe8c788c622af0894c8d0c701afa9861fc40bc34c7926c3d2c19912c3d51287cf43da6d68c826fb4f4163e75ccb1714b874fb7c0621ff48ac
+  metadata.gz: 202032a8ca4d109eab58eb325d5fd3f022479068654e4ab9d801ec515461f41d8d7fc359a4995bde466db2ccfcafc0ddde3fdcce8efe0f98924e8c3cd2b6af30
+  data.tar.gz: d3b1162aaac4717d3aef5b26e42b615ad3a642deb80fb386cf53f50538210e876271bd94e40df7f19302182893800b867582fb2cc34c16365c24805e4f350618

data/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 
 
+## [0.1.5] - 2026-06-09
+
+### Added
+
+- ViewComponentCssDsl::Verifier cross_declaration_conflicts check — warns when a class declared in one place (e.g. a base `leading-snug`) is silently dropped because a *different* declaration merged on top of it (e.g. a size axis's `text-sm`, since Tailwind font-size utilities also set line-height). Suppresses intentional same-family overrides (`p-2` → `p-8`) and surfaces only cross-family drops.
+- Verifier cross_declaration_conflicts check warning on classes silently dropped when separate DSL declarations merge (168a79e)
+
+### Fixed
+
+- CHANGELOG.md retains the full release history (old entries were trimmed on every version bump) (793d53e)
+
 ## [0.1.4] - 2026-06-04
 
 ### Added
@@ -17,3 +28,22 @@
 ### Breaking
 
 - `hidden` is now treated as part of the display-class conflict group. Strings like `"flex hidden"` collapse to `"hidden"` rather than keeping both. Previous behavior was non-standard — `tailwind-merge` (JS) and `tailwind_merge` (Ruby) have always treated `hidden` as a display utility. For JS-toggle visibility patterns, use the HTML5 `hidden` attribute (e.g., `attribute hidden: -> { … }` or pass `hidden: true` as an html_attr) and toggle it with `element.toggleAttribute('hidden')` instead of relying on the class merger.
+
+## [0.1.2] - 2026-05-15
+
+### Added
+
+- data, aria, and attribute DSL declarators for first-class HTML attribute declarations (cf37050)
+
+## [0.1.1] - 2026-05-15
+
+### Changed
+
+- `view_component` dependency pinned to `~> 4.0` (was `>= 4.0`) — RubyGems-recommended SemVer-aware constraint
+- Minimum Ruby version raised to 3.2 (was 3.1), matching the floor for `view_component >= 4.0`
+
+## [0.1.0] - 2026-05-15
+
+### Added
+
+- Initial release. Extracted from SOFware/forge.

data/README.md

@@ -462,12 +462,13 @@ Axis, method, and proc rules are appended, not overridden.
 
 The DSL's worst failure modes are silent: a typo'd or hallucinated Tailwind class produces no CSS at all under JIT, a self-conflicting declaration quietly drops a class, and a rule referencing a missing method only raises at render time on the code path that hits it. `ViewComponentCssDsl::Verifier` catches all of these statically — fast enough to run on every edit.
 
-`verify(component)` returns `Finding` structs (`component`, `check`, `severity`, `message`). Six checks run:
+`verify(component)` returns `Finding` structs (`component`, `check`, `severity`, `message`). Seven checks run:
 
 | Check | Asserts | Catches |
 | --- | --- | --- |
 | `class_validity` | Every declared class exists in the compiled Tailwind output | Typos, hallucinated classes, theme values that don't exist |
 | `self_conflicts` | No declaration conflicts with itself | `css "block flex"` silently dropping `block` |
+| `cross_declaration_conflicts` | No class declared in one place is silently dropped when a different declaration merges on top of it | A base `leading-snug` that a size axis's `text-sm` overrides (font-size utilities also set line-height) |
 | `method_rules` | Every Symbol in `css`/`data`/`aria`/`attribute` rules resolves to a method | Render-time `NoMethodError`s |
 | `axes_settable` | Every axis has an initialize param or `@ivar` assignment | Variant rules that can never fire |
 | `variant_matrix` | `#css` builds cleanly for every axis-value combination | Anything the static checks miss, without rendering |

data/lib/view_component_css_dsl/verifier.rb

@@ -6,13 +6,17 @@ require "view_component_css_dsl"
 # itself can't surface until render time (or surfaces silently). Designed to be
 # fast enough to run on every edit.
 #
-# The six checks:
+# The seven checks:
 #
 #   class_validity  - every declared class exists in the compiled Tailwind output;
 #                     catches typos, hallucinated classes, and theme values that
 #                     don't exist (requires known_classes:)
 #   self_conflicts  - no declaration conflicts with itself; catches e.g.
 #                     css "block flex" silently dropping "block"
+#   cross_declaration_conflicts - no class declared in one place is silently
+#                     dropped when a different declaration merges on top of it;
+#                     catches e.g. a base leading-snug that a size axis's text-sm
+#                     overrides (font-size utilities also set line-height)
 #   method_rules    - every Symbol in css/data/aria/attribute rules resolves to a
 #                     method; catches render-time NoMethodErrors
 #   axes_settable   - every axis has an initialize param or @ivar assignment;
@@ -59,6 +63,7 @@ class ViewComponentCssDsl::Verifier
   def verify(component)
     check_class_validity(component) +
       check_self_conflicts(component) +
+      check_cross_declaration_conflicts(component) +
       check_method_rules(component) +
       check_axes_settable(component) +
       check_variant_matrix(component) +
@@ -106,6 +111,32 @@ class ViewComponentCssDsl::Verifier
     end
   end
 
+  # A class declared in one place can be silently dropped when a *different*
+  # declaration is merged on top of it — the blind spot check_self_conflicts
+  # (one declaration at a time) and check_variant_matrix (exceptions only) both
+  # miss. The footgun: a base `leading-snug` that a size axis's `text-sm`
+  # overrides, because Tailwind font-size utilities also set line-height.
+  #
+  # Reported as warnings, not errors: a cross-declaration drop is often
+  # intentional — a variant overriding a base default. We suppress same-family
+  # overrides (p-2 -> p-8) and surface only drops whose winning class is a
+  # different utility family (leading-snug dropped by text-sm), which is almost
+  # always a surprise.
+  def check_cross_declaration_conflicts(component)
+    axis_combinations(component).flat_map do |combo|
+      tokens = contributing_tokens(component, combo)
+      next [] if tokens.size < 2
+
+      survivors = component.smart_merge(tokens.map { |t| t[:class] }.join(" ")).split
+      dropped_conflicts(component, tokens, survivors).map do |dropped, winner|
+        finding(component, :cross_declaration_conflicts, :warning,
+          "#{dropped[:label]}: \"#{dropped[:class]}\" is silently dropped when " \
+          "merged with \"#{winner[:class]}\" from #{winner[:label]} (both set " \
+          "the same CSS property)")
+      end
+    end
+  end
+
   # Symbols in css/data/aria/attribute rules are method references; each must
   # resolve on the component. The DSL only raises for these at render time.
   def check_method_rules(component)
@@ -227,6 +258,65 @@ class ViewComponentCssDsl::Verifier
       component.private_method_defined?(method_name)
   end
 
+  ##################################################################################
+  # Cross-declaration conflicts
+  ##################################################################################
+
+  # {class:, label:} for every class the static declarations contribute for this
+  # combo, in merge order: base declarations first, then matching axis rules.
+  def contributing_tokens(component, combo)
+    declarations =
+      base_declarations(component) + matching_axis_declarations(component, combo)
+    declarations.flat_map do |label, styles|
+      styles.split.map { |cls| {class: cls, label:} }
+    end
+  end
+
+  def base_declarations(component)
+    component._css_base_declarations.map { |styles| ["css \"#{styles}\"", styles] }
+  end
+
+  def matching_axis_declarations(component, combo)
+    component._css_axis_rules.filter_map do |rule|
+      matches = rule[:axes].all? { |axis, value| combo[axis] == value }
+      [axis_label(rule[:axes]), rule[:styles]] if matches
+    end
+  end
+
+  # [dropped_token, winning_token] pairs the merge silently removed. A class is
+  # dropped when it's absent from survivors; its winner is the later class that
+  # displaced it (found by pairwise re-merge). Only cross-declaration,
+  # cross-family pairs are returned — same-declaration drops belong to
+  # check_self_conflicts, same-family drops are intentional overrides.
+  def dropped_conflicts(component, tokens, survivors)
+    tokens.each_with_index.filter_map do |token, index|
+      next if survivors.include?(token[:class])
+
+      winner = winning_token(component, tokens, index)
+      next unless winner
+      next if winner[:label] == token[:label]
+      next if utility_family(winner[:class]) == utility_family(token[:class])
+
+      [token, winner]
+    end
+  end
+
+  # The later token that displaces tokens[index]: the first subsequent token
+  # that, merged after it, wins outright.
+  def winning_token(component, tokens, index)
+    dropped = tokens[index][:class]
+    tokens.drop(index + 1).find do |candidate|
+      merged = component.smart_merge("#{dropped} #{candidate[:class]}").split
+      merged == [candidate[:class]]
+    end
+  end
+
+  # The utility family of a Tailwind class, ignoring variant prefixes and
+  # negativity: hover:-mt-2 -> "mt", leading-snug -> "leading", text-sm -> "text".
+  def utility_family(token)
+    token.split(":").last.delete_prefix("-").split("-").first
+  end
+
   ##################################################################################
   # Axis settability
   ##################################################################################

data/lib/view_component_css_dsl/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ViewComponentCssDsl
-  VERSION = "0.1.4"
+  VERSION = "0.1.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: view_component_css_dsl
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - Jeff Lange