react_on_rails 13.3.5 → 14.0.0

data/lib/generators/react_on_rails/generator_messages.rb

@@ -37,5 +37,45 @@ module GeneratorMessages
     def clear
       @output = []
     end
+
+    def helpful_message_after_installation
+      <<~MSG
+
+        What to do next:
+
+          - See the documentation on https://github.com/shakacode/shakapacker#webpack-configuration
+            for how to customize the default webpack configuration.
+
+          - Include your webpack assets to your application layout.
+
+              <%= javascript_pack_tag 'hello-world-bundle' %>
+
+          - To start Rails server run:
+
+              ./bin/dev # Running with HMR
+
+            or
+
+              ./bin/dev-static # Running with statically created bundles, without HMR
+
+          - To server render, change this line app/views/hello_world/index.html.erb to
+            `prerender: true` to see server rendering (right click on page and select "view source").
+
+              <%= react_component("HelloWorldApp", props: @hello_world_props, prerender: true) %>
+
+        Alternative steps to run the app:
+
+          - Run `rails s` to start the Rails server.
+
+          - Run bin/shakapacker-dev-server to start the Webpack dev server for compilation of Webpack
+            assets as soon as you save. This default setup with the dev server does not work
+            for server rendering
+
+          - Visit http://localhost:3000/hello_world and see your React On Rails app running!
+
+          - To turn on HMR, edit config/shakapacker.yml and set HMR to true. Restart the rails server
+            and bin/shakapacker-dev-server. Or use Procfile.dev.
+      MSG
+    end
   end
 end

data/lib/generators/react_on_rails/install_generator.rb

@@ -29,6 +29,7 @@ module ReactOnRails
         if installation_prerequisites_met? || options.ignore_warnings?
           invoke_generators
           add_bin_scripts
+          add_post_install_message
         else
           error = "react_on_rails generator prerequisites not met!"
           GeneratorMessages.add_error(error)
@@ -52,6 +53,8 @@ module ReactOnRails
         else
           invoke "react_on_rails:react_no_redux"
         end
+
+        invoke "react_on_rails:adapt_for_older_shakapacker" unless using_shakapacker_7?
       end
 
       # NOTE: other requirements for existing files such as .gitignore or application.
@@ -89,6 +92,24 @@ module ReactOnRails
 
         File.chmod(0o755, *files_to_become_excutable)
       end
+
+      def add_post_install_message
+        message = GeneratorMessages.helpful_message_after_installation
+        unless using_shakapacker_7?
+          message = message.gsub("config/shakapacker", "config/webpacker")
+          message = message.gsub("bin/shakapacker", "bin/webpacker")
+        end
+
+        GeneratorMessages.add_info(message)
+      end
+
+      def using_shakapacker_7?
+        shakapacker_gem = Gem::Specification.find_by_name("shakapacker")
+        shakapacker_gem.version.segments.first == 7
+      rescue Gem::MissingSpecError
+        # In case using Webpacker
+        false
+      end
     end
   end
 end

data/lib/generators/react_on_rails/templates/base/base/Procfile.dev

@@ -1,5 +1,5 @@
 # Procfile for development using HMR
 # You can run these commands in separate shells
 rails: bundle exec rails s -p 3000
-wp-client: bin/webpacker-dev-server
-wp-server: SERVER_BUNDLE_ONLY=yes bin/webpacker --watch
+wp-client: bin/shakapacker-dev-server
+wp-server: SERVER_BUNDLE_ONLY=yes bin/shakapacker --watch

data/lib/generators/react_on_rails/templates/base/base/Procfile.dev-static

@@ -5,5 +5,5 @@ web: rails s -p 3000
 # When making frequent changes to client side assets, you will prefer building webpack assets
 # upon saving rather than when you refresh your browser page.
 # Note, if using React on Rails localization you will need to run
-# `bundle exec rake react_on_rails:locale` before you run bin/webpacker
-webpack: sh -c 'rm -rf public/packs/* || true && bin/webpacker -w'
+# `bundle exec rake react_on_rails:locale` before you run bin/shakapacker
+webpack: sh -c 'rm -rf public/packs/* || true && bin/shakapacker -w'

data/lib/generators/react_on_rails/templates/base/base/config/initializers/react_on_rails.rb

@@ -24,8 +24,8 @@ ReactOnRails.configure do |config|
   # to automatically refresh your webpack assets on every test run.
   #
   # Alternately, you can remove the `ReactOnRails::TestHelper.configure_rspec_to_compile_assets`
-  # and set the config/webpacker.yml option for test to true.
-  config.build_test_command = "RAILS_ENV=test bin/webpacker"
+  # and set the config/shakapacker.yml option for test to true.
+  config.build_test_command = "RAILS_ENV=test bin/shakapacker"
 
   ################################################################################
   ################################################################################

data/lib/generators/react_on_rails/templates/base/base/config/{webpacker.yml → shakapacker.yml}

@@ -1,11 +1,11 @@
-# Note: You must restart bin/webpacker-dev-server for changes to take effect
+# Note: You must restart bin/shakapacker-dev-server for changes to take effect
 
 default: &default
   source_path: app/javascript
   source_entry_path: packs
   public_root_path: public
   public_output_path: packs
-  cache_path: tmp/webpacker
+  cache_path: tmp/shakapacker
   webpack_compile_output: true
 
   # Additional paths webpack should lookup modules
@@ -17,7 +17,7 @@ default: &default
 
 development:
   <<: *default
-  # This is false since we're running `bin/webpacker -w` in Procfile.dev-setic
+  # This is false since we're running `bin/shakapacker -w` in Procfile.dev-static
   compile: false
 
   # Reference: https://webpack.js.org/configuration/dev-server/

data/lib/generators/react_on_rails/templates/base/base/config/webpack/commonWebpackConfig.js.tt

@@ -1,7 +1,9 @@
 <%= add_documentation_reference(config[:message], "// https://github.com/shakacode/react_on_rails_demo_ssr_hmr/blob/master/config/webpack/commonWebpackConfig.js") %>
 
 // Common configuration applying to client and server configuration
-const { webpackConfig: baseClientWebpackConfig, merge } = require('shakapacker');
+const { generateWebpackConfig, merge } = require('shakapacker');
+
+const baseClientWebpackConfig = generateWebpackConfig();
 
 const commonOptions = {
   resolve: {

data/lib/generators/react_on_rails/templates/dev_tests/spec/rails_helper.rb

@@ -40,7 +40,7 @@ end
 # ActiveRecord::Migration.maintain_test_schema!
 
 RSpec.configure do |config|
-  config.before(:each, type: :system, js: true) do
+  config.before(:each, :js, type: :system) do
     driven_by :selenium_chrome
   end
 
@@ -49,7 +49,7 @@ RSpec.configure do |config|
   ReactOnRails::TestHelper.configure_rspec_to_compile_assets(config)
 
   # Remove this line if you"re not using ActiveRecord or ActiveRecord fixtures
-  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+  config.fixture_path = "#{Rails.root}/spec/fixtures"
 
   # If you"re not using ActiveRecord, or you"d prefer not to run each of your
   # examples within a transaction, remove the following line or assign false

data/lib/generators/react_on_rails/templates/dev_tests/spec/system/hello_world_spec.rb

@@ -2,7 +2,7 @@
 
 require_relative "../rails_helper"
 
-describe "Hello World", js: true do
+describe "Hello World", :js do
   it "the hello world example works" do
     visit "/hello_world"
     expect(heading).to have_text("Hello World")

data/lib/react_on_rails/configuration.rb

@@ -137,35 +137,23 @@ module ReactOnRails
     end
 
     def adjust_precompile_task
-      skip_react_on_rails_precompile = %w[no false n f].include?(ENV["REACT_ON_RAILS_PRECOMPILE"])
+      skip_react_on_rails_precompile = %w[no false n f].include?(ENV.fetch("REACT_ON_RAILS_PRECOMPILE", nil))
 
       return if skip_react_on_rails_precompile || build_production_command.blank?
 
-      if Webpacker.config.webpacker_precompile?
-        msg = <<~MSG
-
-          React on Rails and Shakapacker error in configuration!
-          In order to use config/react_on_rails.rb config.build_production_command,
-          you must edit config/webpacker.yml to include this value in the default configuration:
-          'webpacker_precompile: false'
-
-          Alternatively, remove the config/react_on_rails.rb config.build_production_command and the
-          default bin/webpacker script will be used for assets:precompile.
-
-        MSG
-        raise ReactOnRails::Error, msg
-      end
+      raise(ReactOnRails::Error, compile_command_conflict_message) if shakapacker_precompile?
 
       precompile_tasks = lambda {
         Rake::Task["react_on_rails:generate_packs"].invoke
         Rake::Task["react_on_rails:assets:webpack"].invoke
-        puts "Invoking task webpacker:clean from React on Rails"
 
         # VERSIONS is per the shakacode/shakapacker clean method definition.
         # We set it very big so that it is not used, and then clean just
         # removes files older than 1 hour.
         versions = 100_000
-        Rake::Task["webpacker:clean"].invoke(versions)
+
+        puts "Invoking task #{shakapacker_clean_task} from React on Rails"
+        Rake::Task[shakapacker_clean_task].invoke(versions)
       }
 
       if Rake::Task.task_defined?("assets:precompile")
@@ -187,7 +175,7 @@ module ReactOnRails
       webpacker_public_output_path = ReactOnRails::WebpackerUtils.webpacker_public_output_path
 
       if File.expand_path(generated_assets_dir) == webpacker_public_output_path.to_s
-        Rails.logger.warn("You specified generated_assets_dir in `config/initializers/react_on_rails.rb` "\
+        Rails.logger.warn("You specified generated_assets_dir in `config/initializers/react_on_rails.rb` " \
                           "with Webpacker. Remove this line from your configuration file.")
       else
         msg = <<~MSG
@@ -225,20 +213,20 @@ module ReactOnRails
 
       if ReactOnRails::WebpackerUtils.using_webpacker?
         webpacker_public_output_path = ReactOnRails::WebpackerUtils.webpacker_public_output_path
-        Rails.logger.warn "Error configuring config/initializers/react_on_rails. Define neither the "\
-                          "generated_assets_dirs no the generated_assets_dir when using Webpacker. This is defined by "\
-                          "public_output_path specified in webpacker.yml = #{webpacker_public_output_path}."
+        Rails.logger.warn "Error configuring config/initializers/react_on_rails. Define neither the " \
+                          "generated_assets_dirs no the generated_assets_dir when using Webpacker. This is defined " \
+                          "by public_output_path specified in webpacker.yml = #{webpacker_public_output_path}."
         return
       end
 
-      Rails.logger.warn "[DEPRECATION] ReactOnRails: Use config.generated_assets_dir rather than "\
+      Rails.logger.warn "[DEPRECATION] ReactOnRails: Use config.generated_assets_dir rather than " \
                         "generated_assets_dirs"
       if generated_assets_dir.blank?
         self.generated_assets_dir = generated_assets_dirs
       else
-        Rails.logger.warn "[DEPRECATION] ReactOnRails. You have both generated_assets_dirs and "\
-                          "generated_assets_dir defined. Define ONLY generated_assets_dir if NOT using Webpacker"\
-                          " and define neither if using Webpacker"
+        Rails.logger.warn "[DEPRECATION] ReactOnRails. You have both generated_assets_dirs and " \
+                          "generated_assets_dir defined. Define ONLY generated_assets_dir if NOT using Webpacker " \
+                          "and define neither if using Webpacker"
       end
     end
 
@@ -266,6 +254,32 @@ module ReactOnRails
 
       raise ReactOnRails::Error, msg
     end
+
+    def shakapacker_precompile?
+      return Webpacker.config.webpacker_precompile? if ReactOnRails::WebpackerUtils.using_shakapacker_6?
+
+      Webpacker.config.shakapacker_precompile?
+    end
+
+    def shakapacker_clean_task
+      ReactOnRails::WebpackerUtils.using_shakapacker_6? ? "webpacker:clean" : "shakapacker:clean"
+    end
+
+    def compile_command_conflict_message
+      packer = ReactOnRails::WebpackerUtils.using_shakapacker_6? ? "webpacker" : "shakapacker"
+
+      <<~MSG
+
+        React on Rails and Shakapacker error in configuration!
+        In order to use config/react_on_rails.rb config.build_production_command,
+        you must edit config/#{packer}.yml to include this value in the default configuration:
+        '#{packer}_precompile: false'
+
+        Alternatively, remove the config/react_on_rails.rb config.build_production_command and the
+        default bin/#{packer} script will be used for assets:precompile.
+
+      MSG
+    end
   end
 end
 # rubocop:enable Metrics/ClassLength

data/lib/react_on_rails/git_utils.rb

@@ -4,13 +4,13 @@ require "English"
 
 module ReactOnRails
   module GitUtils
-    def self.uncommitted_changes?(message_handler)
+    def self.uncommitted_changes?(message_handler, git_installed = true)
       return false if ENV["COVERAGE"] == "true"
 
       status = `git status --porcelain`
-      return false if $CHILD_STATUS.success? && status.empty?
+      return false if git_installed && status&.empty?
 
-      error = if $CHILD_STATUS.success?
+      error = if git_installed
                 "You have uncommitted code. Please commit or stash your changes before continuing"
               else
                 "You do not have Git installed. Please install Git, and commit your changes before continuing"

data/lib/react_on_rails/helper.rb

@@ -155,7 +155,7 @@ module ReactOnRails
       if defer
         @registered_stores_defer_render ||= []
         @registered_stores_defer_render << redux_store_data
-        "YOU SHOULD NOT SEE THIS ON YOUR VIEW -- Uses as a code block, like <% redux_store %> "\
+        "YOU SHOULD NOT SEE THIS ON YOUR VIEW -- Uses as a code block, like <% redux_store %> " \
           "and not <%= redux store %>"
       else
         @registered_stores ||= []
@@ -245,7 +245,7 @@ module ReactOnRails
       return "{}" if hash_or_string.nil?
 
       unless hash_or_string.is_a?(String) || hash_or_string.is_a?(Hash)
-        raise ReactOnRails::Error, "#{__method__} only accepts String or Hash as argument "\
+        raise ReactOnRails::Error, "#{__method__} only accepts String or Hash as argument " \
                                    "(#{hash_or_string.class} given)."
       end
 
@@ -313,20 +313,25 @@ module ReactOnRails
       @rails_context.merge(serverSide: server_side)
     end
 
-    def load_pack_for_generated_component(component_name)
-      ReactOnRails::WebpackerUtils.raise_nested_entries_disabled unless ReactOnRails::WebpackerUtils.nested_entries?
+    def load_pack_for_generated_component(react_component_name, render_options)
+      return unless render_options.auto_load_bundle
 
-      append_javascript_pack_tag("generated/#{component_name}",
+      ReactOnRails::WebpackerUtils.raise_nested_entries_disabled unless ReactOnRails::WebpackerUtils.nested_entries?
+      if Rails.env.development?
+        is_component_pack_present = File.exist?(generated_components_pack_path(react_component_name))
+        raise_missing_autoloaded_bundle(react_component_name) unless is_component_pack_present
+      end
+      append_javascript_pack_tag("generated/#{react_component_name}",
                                  defer: ReactOnRails.configuration.defer_generated_component_packs)
-      append_stylesheet_pack_tag("generated/#{component_name}")
+      append_stylesheet_pack_tag("generated/#{react_component_name}")
     end
 
     # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity
 
     private
 
-    def generated_components_pack_path(component_name)
-      "#{ReactOnRails::WebpackerUtils.webpacker_source_entry_path}/generated/#{component_name}.js"
+    def generated_components_pack_path(react_component_name)
+      "#{ReactOnRails::WebpackerUtils.webpacker_source_entry_path}/generated/#{react_component_name}.js"
     end
 
     def build_react_component_result_for_server_rendered_string(
@@ -439,11 +444,7 @@ module ReactOnRails
       # Create the HTML rendering part
       result = server_rendered_react_component(render_options)
 
-      if render_options.auto_load_bundle
-        is_component_pack_present = File.exist?(generated_components_pack_path(react_component_name))
-
-        load_pack_for_generated_component(react_component_name) if is_component_pack_present
-      end
+      load_pack_for_generated_component(react_component_name, render_options)
 
       {
         render_options: render_options,
@@ -564,6 +565,17 @@ module ReactOnRails
       instrument_method :react_component, type: "ReactOnRails", name: "react_component"
       instrument_method :react_component_hash, type: "ReactOnRails", name: "react_component_hash"
     end
+
+    def raise_missing_autoloaded_bundle(react_component_name)
+      msg = <<~MSG
+        **ERROR** ReactOnRails: Component "#{react_component_name}" is configured as "auto_load_bundle: true"
+        but the generated component entrypoint, which should have been at #{generated_components_pack_path(react_component_name)},
+        is missing. You might want to check that this component is in a directory named "#{ReactOnRails.configuration.components_subdirectory}"
+        & that "bundle exec rake react_on_rails:generate_packs" has been run.
+      MSG
+
+      raise ReactOnRails::Error, msg
+    end
   end
 end
 # rubocop:enable Metrics/ModuleLength

data/lib/react_on_rails/json_output.rb

@@ -4,25 +4,8 @@ require "active_support/core_ext/string/output_safety"
 
 module ReactOnRails
   class JsonOutput
-    ESCAPE_REPLACEMENT = {
-      "&" => '\u0026',
-      ">" => '\u003e',
-      "<" => '\u003c',
-      "\u2028" => '\u2028',
-      "\u2029" => '\u2029'
-    }.freeze
-    ESCAPE_REGEXP = /[\u2028\u2029&><]/u.freeze
-
     def self.escape(json)
-      return escape_without_erb_util(json) if Utils.rails_version_less_than_4_1_1
-
       ERB::Util.json_escape(json)
     end
-
-    def self.escape_without_erb_util(json)
-      # https://github.com/rails/rails/blob/60257141462137331387d0e34931555cf0720886/activesupport/lib/active_support/core_ext/string/output_safety.rb#L113
-
-      json.to_s.gsub(ESCAPE_REGEXP, ESCAPE_REPLACEMENT)
-    end
   end
 end

data/lib/react_on_rails/locales/base.rb

@@ -100,14 +100,14 @@ module ReactOnRails
 
       def convert
         file_names.each do |name|
-          template = send("template_#{name}")
+          template = send(:"template_#{name}")
           path = file(name)
           generate_file(template, path)
         end
       end
 
       def generate_file(template, path)
-        result = ERB.new(template).result()
+        result = ERB.new(template).result
         File.write(path, result)
       end
 
@@ -138,7 +138,7 @@ module ReactOnRails
       def flatten(translations)
         translations.each_with_object({}) do |(k, v), h|
           if v.is_a? Hash
-            flatten(v).map { |hk, hv| h["#{k}.#{hk}".to_sym] = hv }
+            flatten(v).map { |hk, hv| h[:"#{k}.#{hk}"] = hv }
           elsif v.is_a?(String)
             h[k] = v.gsub("%{", "{")
           elsif !v.is_a?(Array)
@@ -157,7 +157,7 @@ module ReactOnRails
         <<-JS.strip_heredoc
           import { defineMessages } from 'react-intl';
 
-          const defaultLocale = \'#{default_locale}\';
+          const defaultLocale = '#{default_locale}';
 
           const defaultMessages = defineMessages(#{@defaults});
 

data/lib/react_on_rails/locales/to_js.rb

@@ -21,7 +21,7 @@ module ReactOnRails
         <<-JS.strip_heredoc
           import { defineMessages } from 'react-intl';
 
-          const defaultLocale = \'#{default_locale}\';
+          const defaultLocale = '#{default_locale}';
 
           const defaultMessages = defineMessages(#{@defaults});
 

data/lib/react_on_rails/packs_generator.rb

@@ -5,7 +5,7 @@ require "fileutils"
 module ReactOnRails
   # rubocop:disable Metrics/ClassLength
   class PacksGenerator
-    CONTAINS_CLIENT_OR_SERVER_REGEX = /\.(server|client)($|\.)/.freeze
+    CONTAINS_CLIENT_OR_SERVER_REGEX = /\.(server|client)($|\.)/
     MINIMUM_SHAKAPACKER_VERSION = [6, 5, 1].freeze
 
     def self.instance
@@ -13,6 +13,8 @@ module ReactOnRails
     end
 
     def generate_packs_if_stale
+      return unless ReactOnRails.configuration.auto_load_bundle
+
       are_generated_files_present_and_up_to_date = Dir.exist?(generated_packs_directory_path) &&
                                                    File.exist?(generated_server_bundle_file_path) &&
                                                    !stale_or_missing_packs?
@@ -153,9 +155,7 @@ module ReactOnRails
     end
 
     def common_component_to_path
-      common_components_paths = Dir.glob("#{components_search_path}/*").reject do |f|
-        CONTAINS_CLIENT_OR_SERVER_REGEX.match?(f)
-      end
+      common_components_paths = Dir.glob("#{components_search_path}/*").grep_v(CONTAINS_CLIENT_OR_SERVER_REGEX)
       component_name_to_path(common_components_paths)
     end
 

data/lib/react_on_rails/react_component/render_options.rb

@@ -35,7 +35,7 @@ module ReactOnRails
                                                                           props.clone)
           end
 
-          raise ReactOnRails::Error, "ReactOnRails: your rendering_props_extension module is missing the "\
+          raise ReactOnRails::Error, "ReactOnRails: your rendering_props_extension module is missing the " \
                                      "required adjust_props_for_client_side_hydration method & can not be used"
         end
         props

data/lib/react_on_rails/server_rendering_pool/ruby_embedded_java_script.rb

@@ -7,7 +7,6 @@ module ReactOnRails
   module ServerRenderingPool
     # rubocop:disable Metrics/ClassLength
     class RubyEmbeddedJavaScript
-      # rubocop:enable Metrics/ClassLength
       class << self
         def reset_pool
           options = {
@@ -113,9 +112,7 @@ module ReactOnRails
 
         def eval_js(js_code, _render_options)
           @js_context_pool.with do |js_context|
-            result = js_context.eval(js_code)
-            js_context.eval("console.history = []")
-            result
+            js_context.eval(js_code)
           end
         end
 
@@ -127,8 +124,8 @@ module ReactOnRails
             File.read(server_js_file)
           end
         rescue StandardError => e
-          msg = "You specified server rendering JS file: #{server_js_file}, but it cannot be "\
-                "read. You may set the server_bundle_js_file in your configuration to be \"\" to "\
+          msg = "You specified server rendering JS file: #{server_js_file}, but it cannot be " \
+                "read. You may set the server_bundle_js_file in your configuration to be \"\" to " \
                 "avoid this warning.\nError is: #{e}"
           raise ReactOnRails::Error, msg
         end
@@ -147,15 +144,15 @@ module ReactOnRails
           begin
             if ReactOnRails.configuration.trace
               Rails.logger.info do
-                "[react_on_rails] Created JavaScript context with file "\
+                "[react_on_rails] Created JavaScript context with file " \
                   "#{ReactOnRails::Utils.server_bundle_js_file_path}"
               end
             end
             ExecJS.compile(base_js_code)
           rescue StandardError => e
-            msg = "ERROR when compiling base_js_code! "\
-                  "See file #{file_name} to "\
-                  "correlate line numbers of error. Error is\n\n#{e.message}"\
+            msg = "ERROR when compiling base_js_code! " \
+                  "See file #{file_name} to " \
+                  "correlate line numbers of error. Error is\n\n#{e.message}" \
                   "\n\n#{e.backtrace.join("\n")}"
             Rails.logger.error(msg)
             trace_js_code_used("Error when compiling JavaScript code for the context.", base_js_code,
@@ -194,8 +191,8 @@ module ReactOnRails
 
         def undefined_for_exec_js_logging(function_name)
           if ReactOnRails.configuration.trace
-            "console.error('[React on Rails Rendering] #{function_name} is not defined for server rendering.');\n"\
-              "  console.error(getStackTrace().join('\\n'));"
+            "console.error('[React on Rails Rendering] #{function_name} is not defined for server rendering.');\n  " \
+              "console.error(getStackTrace().join('\\n'));"
           else
             ""
           end
@@ -237,6 +234,7 @@ module ReactOnRails
           raise ReactOnRails::Error, msg
         end
       end
+      # rubocop:enable Metrics/ClassLength
     end
   end
 end

data/lib/react_on_rails/test_helper/webpack_assets_status_checker.rb

@@ -62,10 +62,10 @@ module ReactOnRails
           else
             file_list = make_file_list(make_globs(generated_assets_full_path)).to_ary
             puts "V" * 80
-            puts "Please define config.webpack_generated_files (array) so the test helper knows "\
-                 "which files are required. If you are using webpacker, you typically need to only "\
+            puts "Please define config.webpack_generated_files (array) so the test helper knows " \
+                 "which files are required. If you are using webpacker, you typically need to only " \
                  "include 'manifest.json'."
-            puts "Detected the possible following files to check for webpack compilation in "\
+            puts "Detected the possible following files to check for webpack compilation in " \
                  "#{generated_assets_full_path}"
             puts file_list.join("\n")
             puts "^" * 80

data/lib/react_on_rails/test_helper.rb

@@ -80,8 +80,8 @@ module ReactOnRails
 
         unless @printed_once
           puts
-          puts "====> React On Rails: Checking files in "\
-               "#{webpack_assets_status_checker.generated_assets_full_path} for "\
+          puts "====> React On Rails: Checking files in " \
+               "#{webpack_assets_status_checker.generated_assets_full_path} for " \
                "outdated/missing bundles based on source_path #{source_path}"
           puts
           @printed_once = true

data/lib/react_on_rails/utils.rb

@@ -118,12 +118,6 @@ module ReactOnRails
       end
     end
 
-    # rubocop:disable Naming/VariableNumber
-    def self.rails_version_less_than_4_1_1
-      rails_version_less_than("4.1.1")
-    end
-    # rubocop:enable Naming/VariableNumber
-
     module Required
       def required(arg_name)
         raise ReactOnRails::Error, "#{arg_name} is required"
@@ -195,13 +189,13 @@ module ReactOnRails
       to_remove = str.length - max_length
       lstrip = (to_remove / 2.0).ceil
       rstrip = to_remove - lstrip
-      str[0..(midpoint - lstrip - 1)] + TRUNCATION_FILLER + str[(midpoint + rstrip)..-1]
+      str[0..(midpoint - lstrip - 1)] + TRUNCATION_FILLER + str[(midpoint + rstrip)..]
     end
 
     def self.find_most_recent_mtime(files)
       files.reduce(1.year.ago) do |newest_time, file|
         mt = File.mtime(file)
-        mt > newest_time ? mt : newest_time
+        [mt, newest_time].max
       end
     end
 

data/lib/react_on_rails/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ReactOnRails
-  VERSION = "13.3.5"
+  VERSION = "14.0.0"
 end

data/lib/react_on_rails/version_checker.rb

@@ -6,7 +6,7 @@ module ReactOnRails
   class VersionChecker
     attr_reader :node_package_version
 
-    MAJOR_MINOR_PATCH_VERSION_REGEX = /(\d+)\.(\d+)\.(\d+)/.freeze
+    MAJOR_MINOR_PATCH_VERSION_REGEX = /(\d+)\.(\d+)\.(\d+)/
 
     def self.build
       new(NodePackageVersion.build)
@@ -52,7 +52,7 @@ module ReactOnRails
     end
 
     def raise_node_semver_version_warning
-      msg = "**ERROR** ReactOnRails: Your node package version for react-on-rails contains a "\
+      msg = "**ERROR** ReactOnRails: Your node package version for react-on-rails contains a " \
             "^ or ~\n#{common_error_msg}"
       raise ReactOnRails::Error, msg
     end