primer_view_components 0.0.55 → 0.0.59

data/lib/primer/classify.rb

@@ -2,8 +2,6 @@
 
 require_relative "classify/cache"
 require_relative "classify/flex"
-require_relative "classify/functional_background_colors"
-require_relative "classify/functional_border_colors"
 require_relative "classify/grid"
 require_relative "classify/utilities"
 require_relative "classify/validation"
@@ -14,7 +12,6 @@ module Primer
     # Keys where we can simply translate { key: value } into ".key-value"
     CONCAT_KEYS = %i[text box_shadow].freeze
 
-    BG_KEY = :bg
     TEXT_KEYS = %i[font_family font_style font_weight text_align text_transform].freeze
     BOX_SHADOW_KEY = :box_shadow
     CONTAINER_KEY = :container
@@ -69,7 +66,6 @@ module Primer
       }
     }.freeze
     BORDER_KEY = :border
-    BORDER_COLOR_KEY = :border_color
     BORDER_MARGIN_KEYS = %i[border_top border_bottom border_left border_right].freeze
     BORDER_RADIUS_KEY = :border_radius
     TYPOGRAPHY_KEYS = [:font_size].freeze
@@ -84,9 +80,7 @@ module Primer
       Primer::Classify::Grid::KEYS +
       [
         BORDER_KEY,
-        BORDER_COLOR_KEY,
         BORDER_RADIUS_KEY,
-        BG_KEY,
         BOX_SHADOW_KEY,
         CONTAINER_KEY
       ]
@@ -94,23 +88,25 @@ module Primer
 
     class << self
       def call(classes: "", style: nil, **args)
-        extracted_results = extract_hash(args)
+        extract_css_attrs(args).tap do |extracted_results|
+          classes = +"#{validated_class_names(classes)} #{extracted_results[:class]}"
+          classes.strip!
+          extracted_results[:class] = presence(classes)
 
-        extracted_results[:class] = [
-          validated_class_names(classes),
-          extracted_results.delete(:classes)
-        ].compact.join(" ").strip.presence
-
-        extracted_results[:style] = [
-          extracted_results.delete(:styles),
-          style
-        ].compact.join.presence
-
-        extracted_results
+          styles = "#{extracted_results[:style]}#{style}"
+          extracted_results[:style] = presence(styles)
+        end
       end
 
       private
 
+      # do this instead of using Rails' presence/blank?, which are a lot slower
+      def presence(obj)
+        # rubocop:disable Rails/Blank
+        !obj || obj.empty? ? nil : obj
+        # rubocop:enable Rails/Blank
+      end
+
       def validated_class_names(classes)
         return if classes.blank?
 
@@ -120,7 +116,11 @@ module Primer
               memo << class_name if Primer::Classify::Validation.invalid?(class_name)
             end
 
-          raise ArgumentError, "Use System Arguments (https://primer.style/view-components/system-arguments) instead of Primer CSS class #{'name'.pluralize(invalid_class_names.length)} #{invalid_class_names.to_sentence}. This warning will not be raised in production. Set PRIMER_WARNINGS_DISABLED=1 to disable this warning." if invalid_class_names.any?
+          if invalid_class_names.any?
+            raise ArgumentError, "Use System Arguments (https://primer.style/view-components/system-arguments) "\
+              "instead of Primer CSS class #{'name'.pluralize(invalid_class_names.length)} #{invalid_class_names.to_sentence}. "\
+              "This warning will not be raised in production. Set PRIMER_WARNINGS_DISABLED=1 to disable this warning."
+          end
         end
 
         classes
@@ -133,80 +133,88 @@ module Primer
       #
       # styles_hash - A hash with utility keys that mimic the interface used by https://github.com/primer/components
       #
-      # Returns a string of Primer CSS class names to be added to an HTML class attribute
+      # Returns a string of Primer CSS class names and style attributes to be added to an HTML tag.
       #
       # Example usage:
-      # extract_hash({ mt: 4, py: 2 }) => "mt-4 py-2"
-      def extract_hash(styles_hash)
-        memo = { classes: [], styles: +"" }
-        styles_hash.each do |key, value|
-          next unless VALID_KEYS.include?(key)
+      # extract_css_attrs({ mt: 4, py: 2 }) => "mt-4 py-2"
+      def extract_css_attrs(styles_hash)
+        classes = []
+        styles = +""
 
+        styles_hash.each do |key, value|
           if value.is_a?(Array)
             raise ArgumentError, "#{key} does not support responsive values" unless RESPONSIVE_KEYS.include?(key) || Primer::Classify::Utilities.supported_key?(key)
 
             value.each_with_index do |val, index|
-              Primer::Classify::Cache.read(memo, key, val, BREAKPOINTS[index]) || extract_value(memo, key, val, BREAKPOINTS[index])
+              extract_one_css_attr(classes, styles, key, val, BREAKPOINTS[index])
             end
           else
-            Primer::Classify::Cache.read(memo, key, value, BREAKPOINTS[0]) || extract_value(memo, key, value, BREAKPOINTS[0])
+            extract_one_css_attr(classes, styles, key, value, BREAKPOINTS[0])
           end
         end
 
-        memo[:classes] = memo[:classes].join(" ")
+        {
+          class: classes.join(" "),
+          style: styles
+        }
+      end
+
+      def extract_one_css_attr(classes, styles, key, val, breakpoint)
+        found_classes = Primer::Classify::Cache.instance.fetch(breakpoint, key, val) do
+          classes_from(key, val, breakpoint)
+        end
+
+        classes << found_classes if found_classes
 
-        memo
+        found_styles = styles_from(key, val, breakpoint)
+        styles << found_styles if found_styles
       end
 
-      def extract_value(memo, key, val, breakpoint)
+      def styles_from(key, val, _breakpoint)
+        # Turn this into an if/else like classes_from when we have more branches.
+        # Could be that way now, but it makes Rubocop unhappy.
+      end
+
+      def classes_from(key, val, breakpoint)
         return if val.nil? || val == ""
 
         if Primer::Classify::Utilities.supported_key?(key)
-          memo[:classes] << Primer::Classify::Utilities.classname(key, val, breakpoint)
+          Primer::Classify::Utilities.classname(key, val, breakpoint)
         elsif BOOLEAN_MAPPINGS.key?(key)
-          BOOLEAN_MAPPINGS[key][:mappings].each do |m|
-            memo[:classes] << m[:css_class] if m[:value] == val && m[:css_class].present?
+          bools = BOOLEAN_MAPPINGS[key][:mappings].each_with_object([]) do |m, memo|
+            memo << m[:css_class] if m[:value] == val && m[:css_class].present?
           end
-        elsif key == BG_KEY
-          if val.to_s.start_with?("#")
-            memo[:styles] << "background-color: #{val};"
+          bools.empty? ? nil : bools.join(" ")
+        elsif key == BORDER_KEY
+          if val == true
+            "border"
           else
-            memo[:classes] << Primer::Classify::FunctionalBackgroundColors.color(val)
+            "border-#{val.to_s.dasherize}"
           end
-        elsif key == BORDER_KEY
-          border_value = if val == true
-                           "border"
-                         else
-                           "border-#{val.to_s.dasherize}"
-                         end
-
-          memo[:classes] << border_value
-        elsif key == BORDER_COLOR_KEY
-          memo[:classes] << Primer::Classify::FunctionalBorderColors.color(val)
         elsif BORDER_MARGIN_KEYS.include?(key)
-          memo[:classes] << "#{key.to_s.dasherize}-#{val}"
+          "#{key.to_s.dasherize}-#{val}"
         elsif key == BORDER_RADIUS_KEY
-          memo[:classes] << "rounded-#{val}"
+          "rounded-#{val}"
         elsif Primer::Classify::Flex::KEYS.include?(key)
-          memo[:classes] << Primer::Classify::Flex.classes(key, val, breakpoint)
+          Primer::Classify::Flex.classes(key, val, breakpoint)
         elsif Primer::Classify::Grid::KEYS.include?(key)
-          memo[:classes] << Primer::Classify::Grid.classes(key, val, breakpoint)
+          Primer::Classify::Grid.classes(key, val, breakpoint)
         elsif TEXT_KEYS.include?(key)
-          memo[:classes] << "text-#{val.to_s.dasherize}"
+          "text-#{val.to_s.dasherize}"
         elsif TYPOGRAPHY_KEYS.include?(key)
-          memo[:classes] << if val == :small || val == :normal
-                              "text-#{val.to_s.dasherize}"
-                            else
-                              "f#{val.to_s.dasherize}"
-                            end
+          if val == :small || val == :normal
+            "text-#{val.to_s.dasherize}"
+          else
+            "f#{val.to_s.dasherize}"
+          end
         elsif key == BOX_SHADOW_KEY
-          memo[:classes] << if val == true
-                              "color-shadow-small"
-                            elsif val == :none || val.blank?
-                              "box-shadow-none"
-                            else
-                              "color-shadow-#{val.to_s.dasherize}"
-                            end
+          if val == true
+            "color-shadow-small"
+          elsif val == :none || val.blank?
+            "box-shadow-none"
+          else
+            "color-shadow-#{val.to_s.dasherize}"
+          end
         end
       end
 
@@ -215,6 +223,6 @@ module Primer
       end
     end
 
-    Cache.preload!
+    Cache.instance.preload!
   end
 end

data/lib/primer/view_components/engine.rb

@@ -7,14 +7,13 @@ module Primer
     # :nodoc:
     class Engine < ::Rails::Engine
       isolate_namespace Primer::ViewComponents
-      config.autoload_once_paths = %W[
+      config.eager_load_paths = %W[
         #{root}/app/components
         #{root}/app/lib
       ]
 
       config.primer_view_components = ActiveSupport::OrderedOptions.new
 
-      config.primer_view_components.force_functional_colors = true
       config.primer_view_components.force_system_arguments = false
       config.primer_view_components.silence_deprecations = false
 

data/lib/primer/view_components/linters/argument_mappers/button.rb

@@ -23,12 +23,13 @@ module ERBLint
           component: "Primer::BaseButton",
           constant: "TYPE_OPTIONS"
         ).freeze
+
         DEFAULT_TAG = Primer::ViewComponents::Constants.get(
           component: "Primer::BaseButton",
           constant: "DEFAULT_TAG"
         ).freeze
 
-        ATTRIBUTES = %w[disabled type].freeze
+        ATTRIBUTES = %w[disabled type href name value tabindex].freeze
 
         def attribute_to_args(attribute)
           attr_name = attribute.name
@@ -43,6 +44,8 @@ module ERBLint
             raise ConversionError, "Button component does not support type \"#{attribute.value}\"" unless TYPE_OPTIONS.include?(attribute.value)
 
             { type: ":#{attribute.value}" }
+          else
+            { attr_name.to_sym => erb_helper.convert(attribute) }
           end
         end
 

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

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require_relative "base_linter"
+
+module ERBLint
+  module Linters
+    # Counts the number of times a HTML breadcrumbs is used instead of the component.
+    class BreadcrumbsComponentMigrationCounter < BaseLinter
+      MESSAGE = "We are migrating breadcrumbs to use [Primer::Breadcrumbs](https://primer.style/view-components/components/beta/breadcrumbs), please try to use that instead of raw HTML."
+      CLASSES = %w[breadcrumb-item breadcrumb-item-selected].freeze
+      TAGS = %w[li].freeze
+    end
+  end
+end

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

@@ -22,6 +22,11 @@ module ERBLint
         # Hash children indicates that there are tags in the content.
         return nil if tag_tree[:children].first.is_a?(Hash)
 
+        content = tag_tree[:children].first
+
+        # Don't accept content with ERB blocks
+        return nil if content.type != :text || content.children&.any? { |n| n.try(:type) == :erb }
+
         ARGUMENT_MAPPER.new(tag).to_s
       rescue ArgumentMappers::ConversionError
         nil

data/lib/primer/view_components/version.rb

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

data/lib/rubocop/cop/primer/deprecated_arguments.rb

@@ -125,7 +125,12 @@ module RuboCop
             orange_6: nil,
             orange_7: nil,
             orange_8: nil,
-            orange_9: nil
+            orange_9: nil,
+            purple_light: nil,
+            purple: nil,
+            yellow_dark: nil,
+            orange: nil,
+            pink: nil
           },
           border_color: {
             gray: "border_color: :primary",

data/lib/rubocop/cop/primer/primer_octicon.rb

@@ -75,8 +75,11 @@ module RuboCop
             classes = classes(kwargs)
             size_attributes = transform_sizes(kwargs)
             args = arguments_as_string(node, size_attributes, classes)
-
-            corrector.replace(node.loc.expression, "primer_octicon(#{args})")
+            if node.dot?
+              corrector.replace(node.loc.expression, "#{node.receiver.source}.primer_octicon(#{args})")
+            else
+              corrector.replace(node.loc.expression, "primer_octicon(#{args})")
+            end
           end
         end
 
@@ -128,12 +131,29 @@ module RuboCop
           string_args = string_args_to_string(node)
 
           args = "#{args}, #{size_attributes_to_string(size_attributes)}" if size_args.present?
-          args = "#{args}, #{::Primer::Classify::Utilities.classes_to_args(classes)}" if classes.present?
+          args = "#{args}, #{utilities_args(classes)}" if classes.present?
           args = "#{args}, #{string_args}" if string_args.present?
 
           args
         end
 
+        def utilities_args(classes)
+          args = ::Primer::Classify::Utilities.classes_to_hash(classes)
+
+          color = case args[:color]
+                  when :text_white
+                    :text_white
+                  when :text_link
+                    :icon_info
+                  when Symbol
+                    args[:color].to_s.gsub("text_", "icon_").to_sym
+                  end
+
+          args[:color] = color if color
+
+          ::Primer::Classify::Utilities.hash_to_args(args)
+        end
+
         def size_attributes_to_string(size_attributes)
           # No arguments if they map to the default size
           return if size_attributes.blank? || size_attributes.values.all?(&:blank?)

data/lib/tasks/utilities.rake

@@ -11,6 +11,8 @@ namespace :utilities do
     # rubocop:disable Lint/ConstantDefinitionInBlock
     SUPPORTED_KEYS = %i[
       anim
+      color-bg
+      color-border
       color-icon
       color-text
       d

data/static/constants.json

@@ -109,7 +109,16 @@
     ]
   },
   "Primer::Beta::Breadcrumbs": {
-    "Item": "Primer::Beta::Breadcrumbs::Item"
+    "ARGS_DENYLIST": {
+      "[:p, :pt, :pb, :pr, :pl, :px, :py]": "Padding system arguments are not allowed on Breadcrumbs. Consider using margins instead.",
+      "[:font_size]": "Breadcrumbs do not support the font_size system argument."
+    },
+    "ARIA_LABEL": {
+      "label": "Breadcrumb"
+    },
+    "FONT_SIZE_MESSAGE": "Breadcrumbs do not support the font_size system argument.",
+    "Item": "Primer::Beta::Breadcrumbs::Item",
+    "PADDING_MESSAGE": "Padding system arguments are not allowed on Breadcrumbs. Consider using margins instead."
   },
   "Primer::Beta::Breadcrumbs::Item": {
   },

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: primer_view_components
 version: !ruby/object:Gem::Version
-  version: 0.0.55
+  version: 0.0.59
 platform: ruby
 authors:
 - GitHub Open Source
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-09 00:00:00.000000000 Z
+date: 2021-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionview
@@ -366,7 +366,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.9.25
-description: 
+description:
 email:
 - opensource+primer_view_components@github.com
 executables: []
@@ -389,6 +389,8 @@ files:
 - app/components/primer/alpha/underline_panels.rb
 - app/components/primer/auto_complete/auto_complete.d.ts
 - app/components/primer/auto_complete/auto_complete.js
+- app/components/primer/auto_complete/auto_component.d.ts
+- app/components/primer/auto_complete/auto_component.js
 - app/components/primer/base_button.rb
 - app/components/primer/base_component.rb
 - app/components/primer/beta/auto_complete.rb
@@ -503,9 +505,6 @@ files:
 - lib/primer/classify.rb
 - lib/primer/classify/cache.rb
 - lib/primer/classify/flex.rb
-- lib/primer/classify/functional_background_colors.rb
-- lib/primer/classify/functional_border_colors.rb
-- lib/primer/classify/functional_colors.rb
 - lib/primer/classify/grid.rb
 - lib/primer/classify/utilities.rb
 - lib/primer/classify/utilities.yml
@@ -525,6 +524,7 @@ files:
 - lib/primer/view_components/linters/argument_mappers/system_arguments.rb
 - lib/primer/view_components/linters/autocorrectable.rb
 - lib/primer/view_components/linters/base_linter.rb
+- lib/primer/view_components/linters/breadcrumbs_component_migration_counter.rb
 - 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
@@ -557,7 +557,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -573,7 +573,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: ViewComponents for the Primer Design System
 test_files: []

data/lib/primer/classify/functional_background_colors.rb

@@ -1,63 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "functional_colors"
-
-module Primer
-  class Classify
-    # Background specific functional colors
-    # https://primer-css-git-mkt-color-modes-docs-primer.vercel.app/css/support/v16-migration#background
-    class FunctionalBackgroundColors < FunctionalColors
-      FUNCTIONAL_OPTIONS = {
-        primary: :primary,
-        secondary: :secondary,
-        tertiary: :tertiary,
-        canvas: :canvas,
-        canvas_inset: :canvas_inset,
-        canvas_inverse: :canvas_inverse,
-        info: :info,
-        info_inverse: :info_inverse,
-        success: :success,
-        success_inverse: :success_inverse,
-        warning: :warning,
-        warning_inverse: :warning_inverse,
-        danger: :danger,
-        danger_inverse: :danger_inverse,
-        overlay: :overlay
-      }.freeze
-
-      MAPPINGS = {
-        white: FUNCTIONAL_OPTIONS[:primary],
-        gray_light: FUNCTIONAL_OPTIONS[:secondary],
-        gray: FUNCTIONAL_OPTIONS[:tertiary],
-        gray_dark: FUNCTIONAL_OPTIONS[:canvas_inverse],
-        blue_light: FUNCTIONAL_OPTIONS[:info],
-        blue: FUNCTIONAL_OPTIONS[:info_inverse],
-        green_light: FUNCTIONAL_OPTIONS[:success],
-        green: FUNCTIONAL_OPTIONS[:success_inverse],
-        yellow_light: FUNCTIONAL_OPTIONS[:warning],
-        yellow: FUNCTIONAL_OPTIONS[:warning_inverse],
-        red_light: FUNCTIONAL_OPTIONS[:danger],
-        red: FUNCTIONAL_OPTIONS[:danger_inverse]
-      }.freeze
-
-      OPTIONS = FUNCTIONAL_OPTIONS.values.freeze
-      OPTIONS_WITHOUT_MAPPINGS = [:purple_light, :purple, :yellow_dark, :orange, :pink].freeze
-      DEPRECATED_OPTIONS = [*MAPPINGS.keys, *OPTIONS_WITHOUT_MAPPINGS].freeze
-
-      class << self
-        def color(val)
-          functional_color(
-            key: "background",
-            value: val,
-            mappings: MAPPINGS,
-            non_functional_prefix: "bg",
-            functional_prefix: "color-bg",
-            number_prefix: "bg",
-            functional_options: OPTIONS,
-            options_without_mappigs: OPTIONS_WITHOUT_MAPPINGS
-          )
-        end
-      end
-    end
-  end
-end

data/lib/primer/classify/functional_border_colors.rb

@@ -1,52 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "functional_colors"
-
-module Primer
-  class Classify
-    # Border specific functional colors
-    # https://primer-css-git-mkt-color-modes-docs-primer.vercel.app/css/support/v16-migration#border
-    class FunctionalBorderColors < FunctionalColors
-      FUNCTIONAL_OPTIONS = {
-        primary: :primary,
-        secondary: :secondary,
-        tertiary: :tertiary,
-        info: :info,
-        success: :success,
-        warning: :warning,
-        danger: :danger,
-        inverse: :inverse
-      }.freeze
-
-      MAPPINGS = {
-        gray: FUNCTIONAL_OPTIONS[:primary],
-        gray_light: FUNCTIONAL_OPTIONS[:secondary],
-        gray_dark: FUNCTIONAL_OPTIONS[:tertiary],
-        blue: FUNCTIONAL_OPTIONS[:info],
-        green: FUNCTIONAL_OPTIONS[:success],
-        yellow: FUNCTIONAL_OPTIONS[:warning],
-        red: FUNCTIONAL_OPTIONS[:danger],
-        white: FUNCTIONAL_OPTIONS[:inverse]
-      }.freeze
-
-      OPTIONS = FUNCTIONAL_OPTIONS.values.freeze
-      OPTIONS_WITHOUT_MAPPINGS = [:gray_darker, :blue_light, :red_light, :purple, :black_fade, :white_fade].freeze
-      DEPRECATED_OPTIONS = [*MAPPINGS.keys, *OPTIONS_WITHOUT_MAPPINGS].freeze
-
-      class << self
-        def color(val)
-          functional_color(
-            key: "border",
-            value: val,
-            mappings: MAPPINGS,
-            non_functional_prefix: "border",
-            functional_prefix: "color-border",
-            number_prefix: "border",
-            functional_options: OPTIONS,
-            options_without_mappigs: OPTIONS_WITHOUT_MAPPINGS
-          )
-        end
-      end
-    end
-  end
-end

data/lib/primer/classify/functional_colors.rb

@@ -1,68 +0,0 @@
-# frozen_string_literal: true
-
-module Primer
-  class Classify
-    # https://primer-css-git-mkt-color-modes-docs-primer.vercel.app/css/support/v16-migration
-    class FunctionalColors
-      class << self
-        def color(val)
-          # Implemented by class' childrens.
-        end
-
-        private
-
-        # @param key [String|Symbol] Option name.
-        # @param value [String|Symbol] Option value.
-        # @param mappings [Hash] A `color` => `functional_color` mapping hash.
-        # @param non_functional_prefix [String] The prefix to use for the non-functional color classes. E.g. "text" would create "text-value".
-        # @param functional_prefix [String] The prefix to use for the functional color classes. E.g. "color-text" would create "color-text-value".
-        # @param number_prefix [String] The prefix to use for colors ending with number. E.g. "text" would create "text-value-1".
-        # @param functional_options [Array] All the acceptable functional values.
-        # @param options_without_mappigs [Array] Non functional values that don't have an associated functional color.
-        def functional_color(
-          key:,
-          value:,
-          mappings:,
-          non_functional_prefix:,
-          functional_options:,
-          functional_prefix: "",
-          number_prefix: "",
-          options_without_mappigs: []
-        )
-          sym_value = value.to_sym
-          dasherized_value = value.to_s.dasherize
-          # the value is a functional color
-          return "#{number_prefix}-#{dasherized_value}" if ends_with_number?(sym_value)
-          return "#{functional_prefix}-#{dasherized_value}" if functional_options.include?(sym_value)
-          # if the app still allows non functional colors
-          return "#{non_functional_prefix}-#{dasherized_value}" unless force_functional_colors?
-
-          if mappings.key?(sym_value) || options_without_mappigs.include?(sym_value)
-            functional_color = mappings[sym_value]
-            # colors without functional mapping stay the same
-            return "#{non_functional_prefix}-#{dasherized_value}" if functional_color.blank?
-
-            ActiveSupport::Deprecation.warn("#{key} #{value} is deprecated. Please use #{functional_color} instead.") unless Rails.env.production? || silence_deprecations?
-
-            return "#{functional_prefix}-#{functional_color.to_s.dasherize}"
-          end
-
-          raise ArgumentError, "#{key} #{value} does not exist." unless Rails.env.production?
-        end
-
-        def ends_with_number?(val)
-          char_code = val[-1].ord
-          char_code >= 48 && char_code <= 57
-        end
-
-        def force_functional_colors?
-          Rails.application.config.primer_view_components.force_functional_colors
-        end
-
-        def silence_deprecations?
-          Rails.application.config.primer_view_components.silence_deprecations
-        end
-      end
-    end
-  end
-end