primer_view_components 0.0.79 → 0.0.80

data/app/components/primer/button_component.rb

@@ -65,12 +65,10 @@ module Primer
     renders_one :tooltip, lambda { |**system_arguments|
       raise ArgumentError, "Buttons with a tooltip must have a unique `id` set on the `Button`." if @id.blank? && !Rails.env.production?
 
-      @system_arguments = system_arguments
-
-      @system_arguments[:for_id] = @id
-      @system_arguments[:type] ||= :description
+      system_arguments[:for_id] = @id
+      system_arguments[:type] ||= :description
 
-      Primer::Alpha::Tooltip.new(**@system_arguments)
+      Primer::Alpha::Tooltip.new(**system_arguments)
     }
 
     # @example Schemes

data/app/components/primer/link_component.rb

@@ -24,12 +24,10 @@ module Primer
     renders_one :tooltip, lambda { |**system_arguments|
       raise ArgumentError, "Links with a tooltip must have a unique `id` set on the `LinkComponent`." if @id.blank? && !Rails.env.production?
 
-      @system_arguments = system_arguments
-
-      @system_arguments[:for_id] = @id
-      @system_arguments[:type] ||= :description
+      system_arguments[:for_id] = @id
+      system_arguments[:type] ||= :description
 
-      Primer::Alpha::Tooltip.new(**@system_arguments)
+      Primer::Alpha::Tooltip.new(**system_arguments)
     }
 
     # @example Default

data/app/components/primer/tab_container_component.d.ts

@@ -1 +1 @@
-gimport '@github/tab-container-element';
+import '@github/tab-container-element';

data/lib/primer/view_components/linters/disallow_action_list.rb

@@ -0,0 +1,73 @@
+# frozen_string_literal: true
+
+require "pathname"
+require_relative "helpers/rubocop_helpers"
+
+module ERBLint
+  module Linters
+    # Finds usages of ActionList CSS classes.
+    class DisallowActionList < Linter
+      include ERBLint::LinterRegistry
+      include TagTreeHelpers
+
+      class ConfigSchema < LinterConfig
+        property :ignore_files, accepts: array_of?(String), default: -> { [] }
+      end
+      self.config_schema = ConfigSchema
+
+      def run(processed_source)
+        return if ignored?(processed_source.filename)
+
+        class_regex = /ActionList[\w-]*/
+        tags, * = build_tag_tree(processed_source)
+
+        tags.each do |tag|
+          next if tag.closing?
+
+          classes =
+            if (class_attrib = tag.attributes["class"])
+              loc = class_attrib.value_node.loc
+              loc.source_buffer.source[loc.begin_pos...loc.end_pos]
+            else
+              ""
+            end
+
+          indices = [].tap do |results|
+            classes.scan(class_regex) do
+              results << Regexp.last_match.offset(0)
+            end
+          end
+
+          next if indices.empty?
+
+          indices.each do |(start_idx, end_idx)|
+            new_loc = class_attrib.value_node.loc.with(
+              begin_pos: class_attrib.value_node.loc.begin_pos + start_idx,
+              end_pos: class_attrib.value_node.loc.begin_pos + end_idx
+            )
+
+            add_offense(
+              new_loc,
+              "ActionList classes are only designed to be used by Primer View Components and " \
+                "should be considered private. Please reach out in the #primer-rails Slack channel."
+            )
+          end
+        end
+      end
+
+      private
+
+      def ignored?(filename)
+        filename = Pathname(filename)
+
+        begin
+          filename = filename.relative_path_from(Pathname(Dir.getwd))
+        rescue ArgumentError
+          # raised if the filename does not have Dir.getwd as a prefix
+        end
+
+        @config.ignore_files.any? { |pattern| filename.fnmatch?(pattern) }
+      end
+    end
+  end
+end

data/lib/primer/view_components/version.rb

@@ -5,7 +5,7 @@ module Primer
     module VERSION
       MAJOR = 0
       MINOR = 0
-      PATCH = 79
+      PATCH = 80
 
       STRING = [MAJOR, MINOR, PATCH].join(".")
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: primer_view_components
 version: !ruby/object:Gem::Version
-  version: 0.0.79
+  version: 0.0.80
 platform: ruby
 authors:
 - GitHub Open Source
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-09 00:00:00.000000000 Z
+date: 2022-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionview
@@ -536,6 +536,7 @@ files:
 - lib/primer/view_components/linters/button_component_migration_counter.rb
 - lib/primer/view_components/linters/clipboard_copy_component_migration_counter.rb
 - lib/primer/view_components/linters/close_button_component_migration_counter.rb
+- lib/primer/view_components/linters/disallow_action_list.rb
 - lib/primer/view_components/linters/flash_migration_counter.rb
 - lib/primer/view_components/linters/helpers/rubocop_helpers.rb
 - lib/primer/view_components/linters/label_component_migration_counter.rb
@@ -594,7 +595,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.32
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: ViewComponents for the Primer Design System