jekyll_plugin_support 0.8.6 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6bf93dd0ea77ecf7c70380afdddc398824f3e1e27226ed3b8e996e8170a15443
-  data.tar.gz: 32e434ab647fcfc81e674a54fb676b1b17450ceef5c38f0f51c1a5f59588f666
+  metadata.gz: cb346f630f3f3d54d602c2c130ead054765b3f1fcd5d2a13550a82d850ce6c7f
+  data.tar.gz: 50a247655dbdb650eaeabb6163843b2d9f2c1ec47c69609f43b0e22df10d96c7
 SHA512:
-  metadata.gz: 2c254100e325c89dfd874e7403d83c5e3b10a808de8d987748fca7ef955d92dba2a90157868d7dd525953661968eb142a208bafc0015dbf99f357b1bf255dcf0
-  data.tar.gz: ae9cf29baf54d74c957bea4ff126edfe49ddcebb3ba327b2ae7729ef2e783fbe6f1df73a0a635f865a8f8fcbd00b33ebbc3e94ba949c892480a33543a31d21d4
+  metadata.gz: c3277bc67de6688e3c4af585940a26bd641d157a54be0757a34aae364454fc5bfe952cb5e7834d8dc57eda4e6957ce1c1adcf603915d3bff442e7a88e52552ca
+  data.tar.gz: f9e68b39c8de0fa100d720601b59e2a052a8fb3de9746ccb7a37fa28752bb43171827dd325f7b1e6bda920561716feed78a69e477d1fc022832ff43ce5d9234a

data/.rubocop.yml

@@ -34,14 +34,14 @@ Metrics/AbcSize:
 
 Metrics/BlockLength:
   Exclude:
-    - jekyll_all_collections.gemspec
+    - jekyll_plugin_support.gemspec
   Max: 30
 
 Metrics/CyclomaticComplexity:
   Max: 25
 
 Metrics/MethodLength:
-  Max: 30
+  Max: 50
 
 Metrics/PerceivedComplexity:
   Max: 25
@@ -49,14 +49,17 @@ Metrics/PerceivedComplexity:
 Naming/FileName:
   Exclude:
     - Rakefile
+    - "*.md"
 
 RSpec/ExampleLength:
   Max: 30
 
-RSpec/FilePath:
+RSpec/SpecFilePathFormat:
   Enabled: false
   IgnoreMethods: true
-  SpecSuffixOnly: true
+
+RSpec/SpecFilePathSuffix:
+  Enabled: false
 
 RSpec/IndexedLet:
   Enabled: false

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Change Log
 
+## 1.0.0 / 2024-07-23
+
+* Added the `redef_without_warning` method so tag and block plugins can be subclassed.
+* Put most of the functionality into the `JekyllSupport` module.
+* Made tag and block code more consistent
+
 
 ## 0.8.6 / 2024-06-11
 

data/README.md

@@ -55,9 +55,9 @@ add the following line to your Jekyll plugin’s `Gemfile`.
 
 ```ruby
 group :jekyll_plugins do
-  ...
+  # ...
   gem 'jekyll_plugin_support', '>= 0.8.0'
-  ...
+  # ...
 end
 ```
 
@@ -65,9 +65,9 @@ Otherwise, if your custom plugin will be packaged into a gem, add the following
 
 ```ruby
 Gem::Specification.new do |spec|
-  ...
+  # ...
   spec.add_dependency 'jekyll_plugin_support', '>= 0.8.0'
-  ...
+  # ...
 end
 ```
 
@@ -516,7 +516,7 @@ without tokenization, and you expect that the plugin might be invoked with large
 derive your plugin from `JekyllBlockNoArgParsing` or `JekyllTagNoArgParsing`.
 
 
-## Subclass Attribution
+## Attribution
 
 `JekyllTag` and `JekyllBlock` subclasses of `jekyll_plugin_support` can utilize the `attribution`
 option if they are published as a gem.
@@ -584,6 +584,21 @@ An alternative attribution string can be specified properties can be output usin
 {% my_tag attribution="Generated by the #{name} #{version} Jekyll plugin, written by #{author} #{date}" %}
 ```
 
+## Subclassing
+
+Jekyll plugins created using `jekyll_plugin_support` are implemented as Ruby classes.
+If you would like to create a version of an existing Jekyll plugin, you will need to subclass the plugin.
+In order to do that, you will need to override the plugin name and version, which are defined as constants.
+
+`Jekyll_plugin_support` provides a method that allows
+a constant to be redefined, called `redef_without_warning`.
+Use it in a subclass like this:
+
+```ruby
+redef_without_warning :PLUGIN_NAME, 'my_plugin'.freeze
+redef_without_warning :VERSION, '0.1.0'.freeze
+```
+
 
 ## Development
 

data/lib/block/jekyll_plugin_support_block.rb

@@ -5,9 +5,6 @@ module JekyllSupport
   class JekyllBlock < Liquid::Block
     attr_reader :argument_string, :helper, :line_number, :logger, :page, :site, :text
 
-    include JekyllSupportErrorHandling
-    extend JekyllSupportErrorHandling
-
     # See https://github.com/Shopify/liquid/wiki/Liquid-for-Programmers#create-your-own-tags
     # @param tag_name [String] the name of the tag, which we usually know.
     # @param argument_string [String] the arguments passed to the tag, as a single string.
@@ -26,7 +23,7 @@ module JekyllSupport
       @helper = JekyllPluginHelper.new tag_name, markup, @logger, respond_to?(:no_arg_parsing)
 
       @error_name = "#{tag_name.camelcase(:upper)}Error"
-      Jekyll::CustomError.factory @error_name
+      JekyllSupport::CustomError.factory @error_name
     end
 
     # Liquid::Block subclasses do not render if there is no content within the tag
@@ -49,6 +46,8 @@ module JekyllSupport
 
       @config = @site.config
       @tag_config = @config[@tag_name]
+      @jps = @config['jekyll_plugin_support']
+      @pry_on_standard_error = @jps['pry_on_standard_error'] || false if @jps
 
       set_error_context
 

data/lib/block/jekyll_plugin_support_block_noarg.rb

@@ -4,9 +4,6 @@ module JekyllSupport
   class JekyllBlockNoArgParsing < JekyllBlock
     attr_reader :argument_string, :helper, :line_number, :logger, :page, :site
 
-    include JekyllSupportErrorHandling
-    extend JekyllSupportErrorHandling
-
     def initialize(tag_name, argument_string, parse_context)
       class << self
         include NoArgParsing

data/lib/error/jekyll_custom_error.rb

@@ -1,7 +1,7 @@
 require 'facets/string/camelcase'
 require 'facets/string/snakecase'
 
-module Jekyll
+module JekyllSupport
   # Use like this:
   # CustomError.new(:MyError, 'blah', 'asdf')
   class CustomError < StandardError

data/lib/error/jekyll_plugin_error_handling.rb

@@ -1,4 +1,4 @@
-module JekyllSupportErrorHandling
+module JekyllSupport
   attr_reader :logger, :page
 
   # If a Jekyll plugin needs to crash exit, and stop Jekyll, call this method.

data/lib/generator/jekyll_plugin_support_generator.rb

@@ -7,9 +7,6 @@ module JekyllSupport
   class JekyllGenerator < Jekyll::Generator
     attr_reader :helper, :line_number, :logger, :site
 
-    include JekyllSupportErrorHandling
-    extend JekyllSupportErrorHandling
-
     # Method prescribed by the Jekyll plugin lifecycle.
     # Defines @config, @envs, @mode and @site
     # @return [void]
@@ -17,7 +14,7 @@ module JekyllSupport
       @logger ||= PluginMetaLogger.instance.new_logger(self, PluginMetaLogger.instance.config)
 
       @error_name = "#{self.class.name}Error"
-      # Jekyll::CustomError.factory @error_name
+      # JekyllSupport::CustomError.factory @error_name
 
       @site   = site
       @config = @site.config

data/lib/helper/jekyll_plugin_helper.rb

@@ -4,107 +4,109 @@ require 'shellwords'
 require_relative 'jekyll_plugin_helper_class'
 require_relative 'jekyll_plugin_helper_attribution'
 
-class JekyllPluginHelper
-  attr_accessor :liquid_context
-  attr_reader :argv, :attribution, :keys_values, :logger, :markup, :no_arg_parsing, :params, :tag_name,
-              :argv_original, :excerpt_caller, :keys_values_original, :params_original, :jpsh_subclass_caller
-
-  # See https://github.com/Shopify/liquid/wiki/Liquid-for-Programmers#create-your-own-tags
-  # @param tag_name [String] the name of the tag, which we already know.
-  # @param argument_string [String] the arguments from the tag, as a single string.
-  # @param parse_context [Liquid::ParseContext] hash that stores Liquid options.
-  #        By default it has two keys: :locale and :line_numbers, the first is a Liquid::I18n object, and the second,
-  #        a boolean parameter that determines if error messages should display the line number the error occurred.
-  #        This argument is used mostly to display localized error messages on Liquid built-in Tags and Filters.
-  #        See https://github.com/Shopify/liquid/wiki/Liquid-for-Programmers#create-your-own-tags
-  # @return [void]
-  def initialize(tag_name, markup, logger, no_arg_parsing)
-    @tag_name = tag_name
-    @logger = logger
-    @no_arg_parsing = no_arg_parsing
-    @markup = markup
-  rescue StandardError => e
-    e.shorten_backtrace
-    @logger.error { e.message }
-  end
+module JekyllSupport
+  class JekyllPluginHelper
+    attr_accessor :liquid_context
+    attr_reader :argv, :attribution, :keys_values, :logger, :markup, :no_arg_parsing, :params, :tag_name,
+                :argv_original, :excerpt_caller, :keys_values_original, :params_original, :jpsh_subclass_caller
+
+    # See https://github.com/Shopify/liquid/wiki/Liquid-for-Programmers#create-your-own-tags
+    # @param tag_name [String] the name of the tag, which we already know.
+    # @param argument_string [String] the arguments from the tag, as a single string.
+    # @param parse_context [Liquid::ParseContext] hash that stores Liquid options.
+    #        By default it has two keys: :locale and :line_numbers, the first is a Liquid::I18n object, and the second,
+    #        a boolean parameter that determines if error messages should display the line number the error occurred.
+    #        This argument is used mostly to display localized error messages on Liquid built-in Tags and Filters.
+    #        See https://github.com/Shopify/liquid/wiki/Liquid-for-Programmers#create-your-own-tags
+    # @return [void]
+    def initialize(tag_name, markup, logger, no_arg_parsing)
+      @tag_name = tag_name
+      @logger = logger
+      @no_arg_parsing = no_arg_parsing
+      @markup = markup
+    rescue StandardError => e
+      e.shorten_backtrace
+      @logger.error { e.message }
+    end
 
-  # @return undefined if parameter was specified, removes it from the available tokens and returns value
-  def parameter_specified?(name, delete_param: true)
-    return false if @keys_values.to_s.empty?
+    # @return undefined if parameter was specified, removes it from the available tokens and returns value
+    def parameter_specified?(name, delete_param: true)
+      return false if @keys_values.to_s.empty?
 
-    key = name
-    key = name.to_sym if @keys_values&.first&.first.instance_of?(Symbol)
-    value = @keys_values[key]
-    delete_parameter(name) if delete_param
-    value
-  end
+      key = name
+      key = name.to_sym if @keys_values&.first&.first.instance_of?(Symbol)
+      value = @keys_values[key]
+      delete_parameter(name) if delete_param
+      value
+    end
 
-  def reinitialize(markup)
-    # @keys_values was a Hash[Symbol, String|Boolean] but now it is Hash[String, String|Boolean]
-    @markup = markup
-    if @no_arg_parsing
-      define_singleton_method(:argv) { warn_fetch :argv }
-      define_singleton_method(:keys_values) { warn_fetch :keys_values }
-      define_singleton_method(:params) { warn_fetch :params }
-      define_singleton_method(:parameter_specified?) { |_name| warn_parse(:parameter_specified?) }
-      define_singleton_method(:delete_parameter) { |_name| warn_parse(:delete_parameter) }
-      @attribution = false
-    else
-      parse markup
-      @attribution = parameter_specified?('attribution') || false
-      @logger.debug { "@keys_values='#{@keys_values}'" }
+    def reinitialize(markup)
+      # @keys_values was a Hash[Symbol, String|Boolean] but now it is Hash[String, String|Boolean]
+      @markup = markup
+      if @no_arg_parsing
+        define_singleton_method(:argv) { warn_fetch :argv }
+        define_singleton_method(:keys_values) { warn_fetch :keys_values }
+        define_singleton_method(:params) { warn_fetch :params }
+        define_singleton_method(:parameter_specified?) { |_name| warn_parse(:parameter_specified?) }
+        define_singleton_method(:delete_parameter) { |_name| warn_parse(:delete_parameter) }
+        @attribution = false
+      else
+        parse markup
+        @attribution = parameter_specified?('attribution') || false
+        @logger.debug { "@keys_values='#{@keys_values}'" }
+      end
     end
-  end
 
-  # Call this method to return the remaining markup after `parameter_specified?` has been invoked.
-  def remaining_markup
-    @argv&.join(' ')
-  end
+    # Call this method to return the remaining markup after `parameter_specified?` has been invoked.
+    def remaining_markup
+      @argv&.join(' ')
+    end
 
-  def remaining_markup_original
-    @argv_original&.join(' ')
-  end
+    def remaining_markup_original
+      @argv_original&.join(' ')
+    end
 
-  def warn_fetch(variable)
-    abort "Error: Argument parsing was suppressed, but an attempt to obtain the value of #{variable} was made"
-  end
+    def warn_fetch(variable)
+      abort "Error: Argument parsing was suppressed, but an attempt to obtain the value of #{variable} was made"
+    end
 
-  def warn_parse(meth)
-    abort "Error: Argument parsing was suppressed, but an attempt to invoke #{meth} was made"
-  end
+    def warn_parse(meth)
+      abort "Error: Argument parsing was suppressed, but an attempt to invoke #{meth} was made"
+    end
 
-  def delete_parameter(key)
-    return if @keys_values.empty? || @params.nil?
+    def delete_parameter(key)
+      return if @keys_values.empty? || @params.nil?
 
-    @params.delete(key)
-    @argv.delete_if { |x| x == key or x.start_with?("#{key}=") }
-    @keys_values.delete(key)
+      @params.delete(key)
+      @argv.delete_if { |x| x == key or x.start_with?("#{key}=") }
+      @keys_values.delete(key)
 
-    @params_original.delete(key)
-    @argv_original.delete_if { |x| x == key or x.start_with?("#{key}=") }
-    @keys_values_original.delete(key)
-  end
+      @params_original.delete(key)
+      @argv_original.delete_if { |x| x == key or x.start_with?("#{key}=") }
+      @keys_values_original.delete(key)
+    end
 
-  private
+    private
 
-  def page
-    @liquid_context.registers[:page]
-  end
+    def page
+      @liquid_context.registers[:page]
+    end
 
-  def parse(markup)
-    @argv_original = Shellwords.split(markup)
-    @keys_values_original = KeyValueParser
-                            .new({}, { array_values: false, normalize_keys: false, separator: /=/ })
-                            .parse(@argv_original)
-    @params_original = @keys_values_original unless respond_to?(:no_arg_parsing) && no_arg_parsing
+    def parse(markup)
+      @argv_original = Shellwords.split(markup)
+      @keys_values_original = KeyValueParser
+                              .new({}, { array_values: false, normalize_keys: false, separator: /=/ })
+                              .parse(@argv_original)
+      @params_original = @keys_values_original unless respond_to?(:no_arg_parsing) && no_arg_parsing
 
-    @argv = Shellwords.split(self.class.expand_env(markup, logger))
-    @keys_values = KeyValueParser
-                   .new({}, { array_values: false, normalize_keys: false, separator: /=/ })
-                   .parse(@argv)
+      @argv = Shellwords.split(self.class.expand_env(markup, logger))
+      @keys_values = KeyValueParser
+                     .new({}, { array_values: false, normalize_keys: false, separator: /=/ })
+                     .parse(@argv)
 
-    return if respond_to?(:no_arg_parsing) && no_arg_parsing
+      return if respond_to?(:no_arg_parsing) && no_arg_parsing
 
-    @params = @keys_values # TODO: @keys_values should be deleted
+      @params = @keys_values # TODO: @keys_values should be deleted
+    end
   end
 end

data/lib/helper/jekyll_plugin_helper_attribution.rb

@@ -1,72 +1,74 @@
-# Attribution aspect of JekyllPluginHelper
-class JekyllPluginHelper
-  # @param file must be a fully qualified file name that points to a file within a gem.
-  # @return Gem::Specification of gem that file points into, or nil if not called from a gem
-  # See https://stackoverflow.com/a/75890279/553865
-  def self.current_spec(file)
-    abort 'JekyllPluginHelper::current_spec: file is nil' if file.nil?
-    return nil unless File.exist?(file)
+module JekyllSupport
+  # Attribution aspect of JekyllPluginHelper
+  class JekyllPluginHelper
+    # @param file must be a fully qualified file name that points to a file within a gem.
+    # @return Gem::Specification of gem that file points into, or nil if not called from a gem
+    # See https://stackoverflow.com/a/75890279/553865
+    def self.current_spec(file)
+      abort 'JekyllPluginHelper::current_spec: file is nil' if file.nil?
+      return nil unless File.exist?(file)
 
-    searcher = if Gem::Specification.respond_to?(:find)
-                 Gem::Specification
-               elsif Gem.respond_to?(:searcher)
-                 Gem.searcher.init_gemspecs
-               end
+      searcher = if Gem::Specification.respond_to?(:find)
+                   Gem::Specification
+                 elsif Gem.respond_to?(:searcher)
+                   Gem.searcher.init_gemspecs
+                 end
 
-    searcher&.find do |spec|
-      file.start_with? spec.full_gem_path
+      searcher&.find do |spec|
+        file.start_with? spec.full_gem_path
+      end
     end
-  end
 
-  def attribute
-    return unless @current_gem
+    def attribute
+      return unless @current_gem
 
-    <<~END_OUTPUT
-      <div id="jps_attribute_#{rand(999_999)}" class="jps_attribute">
-        <div>
-          <a href="#{@homepage}" target="_blank" rel="nofollow">
-            #{attribution_string}
-          </a>
+      <<~END_OUTPUT
+        <div id="jps_attribute_#{rand(999_999)}" class="jps_attribute">
+          <div>
+            <a href="#{@homepage}" target="_blank" rel="nofollow">
+              #{attribution_string}
+            </a>
+          </div>
         </div>
-      </div>
-    END_OUTPUT
-  end
+      END_OUTPUT
+    end
 
-  def default_attribution
-    authors = @authors&.join(', ')
-    result = "Generated by the \#{@name} v\#{@version} Jekyll plugin"
-    result << ", written by #{authors}" if authors
-    result << " \#{@published_date}" if @published_date
-    result << '.'
-    result
-  end
+    def default_attribution
+      authors = @authors&.join(', ')
+      result = "Generated by the \#{@name} v\#{@version} Jekyll plugin"
+      result << ", written by #{authors}" if authors
+      result << " \#{@published_date}" if @published_date
+      result << '.'
+      result
+    end
 
-  # Sets @current_gem if file points at a uniquely named file within a gem.
-  # @param file must be a fully qualified file name in a gem, for example: __FILE__
-  def gem_file(file)
-    @current_gem = JekyllPluginHelper.current_spec file
-    @logger.debug "No gem found for '#{file} was found." unless @current_gem
-    annotate_globals if @attribution && @current_gem
-  end
+    # Sets @current_gem if file points at a uniquely named file within a gem.
+    # @param file must be a fully qualified file name in a gem, for example: __FILE__
+    def gem_file(file)
+      @current_gem = JekyllPluginHelper.current_spec file
+      @logger.debug "No gem found for '#{file} was found." unless @current_gem
+      annotate_globals if @attribution && @current_gem
+    end
 
-  private
+    private
 
-  def annotate_globals
-    return unless @current_gem
+    def annotate_globals
+      return unless @current_gem
 
-    @name           = @current_gem.name
-    @authors        = @current_gem.authors
-    @homepage       = @current_gem.homepage
-    @published_date = @current_gem.date.to_date.to_s
-    @version        = @current_gem.version
-  end
+      @name           = @current_gem.name
+      @authors        = @current_gem.authors
+      @homepage       = @current_gem.homepage
+      @published_date = @current_gem.date.to_date.to_s
+      @version        = @current_gem.version
+    end
 
-  def attribution_string
-    string = if @attribution == true
-               default_attribution
-             else
-               @attribution
-             end
-    String.interpolate { string }
+    def attribution_string
+      string = if @attribution == true
+                 default_attribution
+               else
+                 @attribution
+               end
+      String.interpolate { string }
+    end
   end
 end

data/lib/helper/jekyll_plugin_helper_class.rb

@@ -2,80 +2,82 @@ require 'facets/string/camelcase'
 require 'facets/string/snakecase'
 require 'yaml'
 
-# Class methods for JekyllPluginHelper
-class JekyllPluginHelper
-  # Expand an environment variable reference
-  def self.expand_env(str, logger = nil, die_if_undefined: false)
-    str&.gsub(/\$([a-zA-Z_][a-zA-Z0-9_]*)|\${\g<1>}/) do
-      envar = Regexp.last_match(1)
-      unless ENV.key? envar
-        msg = "jekyll_plugin_support error: environment variable #{envar} is undefined"
-        raise JekyllPluginSupportError, msg.red, [] if die_if_undefined
+module JekyllSupport
+  # Class methods for JekyllPluginHelper
+  class JekyllPluginHelper
+    # Expand an environment variable reference
+    def self.expand_env(str, logger = nil, die_if_undefined: false)
+      str&.gsub(/\$([a-zA-Z_][a-zA-Z0-9_]*)|\${\g<1>}/) do
+        envar = Regexp.last_match(1)
+        unless ENV.key? envar
+          msg = "jekyll_plugin_support error: environment variable #{envar} is undefined"
+          raise JekyllPluginSupportError, msg.red, [] if die_if_undefined
 
-        if logger
-          logger.warn msg
-        else
-          puts msg.red
+          if logger
+            logger.warn msg
+          else
+            puts msg.red
+          end
         end
+        ENV.fetch(envar, nil)
       end
-      ENV.fetch(envar, nil)
     end
-  end
 
-  def self.generate_message(klass, tag_name, version)
-    error_name_stub = klass.name.include?('::') ? klass.name.split('::')[1] : klass.name
-    error_ruby_class_name = "#{error_name_stub.camelcase(:upper)}Error"
-    config_die_key = "die_on_#{error_ruby_class_name.snakecase}"
-    error_css_class_name = error_ruby_class_name.split('::').last.snakecase
-    config = YAML.load_file('_config.yml')
-    tag_config = config[tag_name]
-    tag_config_msg = if tag_config.nil?
-                       <<~END_MSG
-                         _config.yml does not contain configuration information for this plugin.
-                           You could add a section containing default values by specifying a section for the tag name,
-                           and an entry whose name starts with `die_on_`, followed by a snake_case version of the error name.
+    def self.generate_message(klass, tag_name, version)
+      error_name_stub = klass.name.include?('::') ? klass.name.split('::')[1] : klass.name
+      error_ruby_class_name = "#{error_name_stub.camelcase(:upper)}Error"
+      config_die_key = "die_on_#{error_ruby_class_name.snakecase}"
+      error_css_class_name = error_ruby_class_name.split('::').last.snakecase
+      config = YAML.load_file('_config.yml')
+      tag_config = config[tag_name]
+      tag_config_msg = if tag_config.nil?
+                         <<~END_MSG
+                           _config.yml does not contain configuration information for this plugin.
+                             You could add a section containing default values by specifying a section for the tag name,
+                             and an entry whose name starts with `die_on_`, followed by a snake_case version of the error name.
 
-                             #{tag_name}:
-                               #{config_die_key}: false
-                       END_MSG
-                     else
-                       <<~END_MSG
-                         _config.yml contains the following configuration for this plugin:
-                           #{tag_config}
-                       END_MSG
-                     end
+                               #{tag_name}:
+                                 #{config_die_key}: false
+                         END_MSG
+                       else
+                         <<~END_MSG
+                           _config.yml contains the following configuration for this plugin:
+                             #{tag_config}
+                         END_MSG
+                       end
 
-    <<~END_MSG
-      Loaded plugin #{tag_name} v#{version}. It has:
-        Error class: #{error_ruby_class_name}
-        CSS class for error messages: #{error_css_class_name}
+      <<~END_MSG
+        Loaded plugin #{tag_name} v#{version}. It has:
+          Error class: #{error_ruby_class_name}
+          CSS class for error messages: #{error_css_class_name}
 
-        #{tag_config_msg}
-    END_MSG
-  end
+          #{tag_config_msg}
+      END_MSG
+    end
 
-  def self.register(klass, tag_name)
-    abort("Error: The #{tag_name} plugin does not define VERSION") \
-      unless klass.const_defined?(:VERSION)
+    def self.register(klass, tag_name)
+      abort("Error: The #{tag_name} plugin does not define VERSION") \
+        unless klass.const_defined?(:VERSION)
 
-    version = klass.const_get(:VERSION)
+      version = klass.const_get(:VERSION)
 
-    abort("Error: The #{tag_name} plugin is not an instance of JekyllSupport::JekyllBlock or JekyllSupport::JekyllTag") \
-      unless klass.instance_of?(Class) &&
-             (klass.ancestors.include?(JekyllSupport::JekyllBlock) ||
-              klass.ancestors.include?(JekyllSupport::JekyllTag))
+      abort("Error: The #{tag_name} plugin is not an instance of JekyllSupport::JekyllBlock or JekyllSupport::JekyllTag") \
+        unless klass.instance_of?(Class) &&
+               (klass.ancestors.include?(JekyllSupport::JekyllBlock) ||
+                klass.ancestors.include?(JekyllSupport::JekyllTag))
 
-    Liquid::Template.register_tag(tag_name, klass)
-    msg = generate_message(klass, tag_name, version)
-    PluginMetaLogger.instance.info { msg }
-  end
+      Liquid::Template.register_tag(tag_name, klass)
+      msg = generate_message(klass, tag_name, version)
+      PluginMetaLogger.instance.info { msg }
+    end
 
-  def self.remove_html_tags(string)
-    string.gsub(/<[^>]*>/, '').strip
-  end
+    def self.remove_html_tags(string)
+      string.gsub(/<[^>]*>/, '').strip
+    end
 
-  # strip leading and trailing quotes if present
-  def self.remove_quotes(string)
-    string.strip.gsub(/\A'|\A"|'\Z|"\Z/, '').strip if string
+    # strip leading and trailing quotes if present
+    def self.remove_quotes(string)
+      string.strip.gsub(/\A'|\A"|'\Z|"\Z/, '').strip if string
+    end
   end
 end

data/lib/jekyll_plugin_support/jekyll_plugin_support_class.rb

@@ -22,7 +22,7 @@ module JekyllSupport
 
   # @return a new StandardError subclass containing the shorten_backtrace method
   def define_error
-    Class.new Jekyll::CustomError
+    Class.new JekyllSupport::CustomError
   end
   module_function :define_error
 

data/lib/jekyll_plugin_support/version.rb

@@ -1,3 +1,3 @@
 module JekyllPluginSupportVersion
-  VERSION = '0.8.6'.freeze
+  VERSION = '1.0.0'.freeze
 end

data/lib/jekyll_plugin_support.rb

@@ -8,6 +8,13 @@ def require_directory(dir)
   end
 end
 
+module JekyllSupport
+  def self.redef_without_warning(const, value)
+    send(:remove_const, const) if const_defined?(const)
+    const_set const, value
+  end
+end
+
 module NoArgParsing
   attr_accessor :no_arg_parsing
 

data/lib/tag/jekyll_plugin_support_tag.rb

@@ -6,9 +6,6 @@ module JekyllSupport
   class JekyllTag < Liquid::Tag
     attr_reader :argument_string, :helper, :line_number, :logger, :page, :site
 
-    include JekyllSupportErrorHandling
-    extend JekyllSupportErrorHandling
-
     # See https://github.com/Shopify/liquid/wiki/Liquid-for-Programmers#create-your-own-tags
     # @param tag_name [String] the name of the tag, which we usually know.
     # @param argument_string [String] the arguments passed to the tag, as a single string.
@@ -29,7 +26,7 @@ module JekyllSupport
       @helper = JekyllPluginHelper.new(tag_name, @argument_string, @logger, respond_to?(:no_arg_parsing))
 
       @error_name = "#{tag_name.camelcase(:upper)}Error"
-      Jekyll::CustomError.factory @error_name
+      JekyllSupport::CustomError.factory @error_name
     end
 
     # Method prescribed by the Jekyll plugin lifecycle.
@@ -55,10 +52,11 @@ module JekyllSupport
       @paginator = @envs[:paginator]
       @theme     = @envs[:theme]
 
-      @mode = @config['env']&.key?('JEKYLL_ENV') ? @config['env']['JEKYLL_ENV'] : 'development'
+      env = @config['env']
+      @mode = env&.key?('JEKYLL_ENV') ? env['JEKYLL_ENV'] : 'development'
 
       markup = JekyllSupport.lookup_liquid_variables liquid_context, @argument_string
-      @helper.reinitialize markup
+      @helper.reinitialize markup.strip
 
       render_impl
     rescue StandardError => e

data/lib/tag/jekyll_plugin_support_tag_noarg.rb

@@ -4,9 +4,6 @@ module JekyllSupport
   class JekyllTagNoArgParsing < JekyllTag
     attr_reader :argument_string, :helper, :line_number, :logger, :page, :site
 
-    include JekyllSupportErrorHandling
-    extend JekyllSupportErrorHandling
-
     def initialize(tag_name, argument_string, parse_context)
       class << self
         include NoArgParsing

data/spec/custom_error_spec.rb

@@ -17,7 +17,7 @@ class CustomErrorSpec
     raise AnError, 'Oops'
   rescue AnError => e
     puts "Caught AnError: #{e.message}"
-  rescue Jekyll::CustomError => e
+  rescue JekyllSupport::CustomError => e
     puts "Caught CustomError: #{e.message}"
   end
 
@@ -31,7 +31,7 @@ class CustomErrorSpec
     end
   end
 
-  RSpec.describe Jekyll::CustomError do
+  RSpec.describe JekyllSupport::CustomError do
     it 'can create custom errors' do
       expect { raise AnError, 'Oops' }.to raise_error(AnError)
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll_plugin_support
 version: !ruby/object:Gem::Version
-  version: 0.8.6
+  version: 1.0.0
 platform: ruby
 authors:
 - Mike Slinn
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-05-12 00:00:00.000000000 Z
+date: 2024-07-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: facets
@@ -140,7 +140,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.10
+rubygems_version: 3.5.16
 signing_key:
 specification_version: 4
 summary: Provides a framework for writing and testing Jekyll plugins