Remove webpacker support and require Shakapacker 8.2+

- Delete rakelib/webpacker_examples.rake and webpacker example generation
- Remove adapt_for_older_shakapacker_generator.rb (Shakapacker 6/7 compatibility)
- Remove webpacker binaries from dummy app (bin/webpacker, bin/webpacker-dev-server)
- Simplify PackerUtils to only support Shakapacker 8.2+ (remove webpacker logic)
- Update all tests to use Shakapacker instead of Webpacker mocks
- Remove webpacker references from rake tasks and configuration
- Fix pack_generator_spec.rb to use targeted stubs and proper output expectations

BREAKING CHANGE: Webpacker is no longer supported. Shakapacker 8.2+ is now required.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: justin808

lib/generators/react_on_rails/adapt_for_older_shakapacker_generator.rb

@@ -3,40 +3,27 @@
 module ReactOnRails
   module PackerUtils
     def self.using_packer?
-      using_shakapacker_const? || using_webpacker_const?
+      using_shakapacker_const?
     end
 
     def self.using_shakapacker_const?
       return @using_shakapacker_const if defined?(@using_shakapacker_const)
 
       @using_shakapacker_const = ReactOnRails::Utils.gem_available?("shakapacker") &&
-                                 shakapacker_version_requirement_met?("7.0.0")
-    end
-
-    def self.using_webpacker_const?
-      return @using_webpacker_const if defined?(@using_webpacker_const)
-
-      @using_webpacker_const = (ReactOnRails::Utils.gem_available?("shakapacker") &&
-                          shakapacker_version_as_array[0] <= 6) ||
-                               ReactOnRails::Utils.gem_available?("webpacker")
+                                 shakapacker_version_requirement_met?("8.2.0")
     end
 
     def self.packer_type
       return "shakapacker" if using_shakapacker_const?
-      return "webpacker" if using_webpacker_const?
 
       nil
     end
 
     def self.packer
       return nil unless using_packer?
 
-      if using_shakapacker_const?
-        require "shakapacker"
-        return ::Shakapacker
-      end
-      require "webpacker"
-      ::Webpacker
+      require "shakapacker"
+      ::Shakapacker
     end
 
     def self.dev_server_running?
@@ -106,7 +93,6 @@ def self.asset_uri_from_packer(asset_name)
     end
 
     def self.precompile?
-      return ::Webpacker.config.webpacker_precompile? if using_webpacker_const?
       return ::Shakapacker.config.shakapacker_precompile? if using_shakapacker_const?
 
       false

lib/react_on_rails/packer_utils.rb

@@ -10,7 +10,7 @@ module ReactOnRails
   module TaskHelpers
     class ExampleType
       def self.all
-        @all ||= { webpacker_examples: [], shakapacker_examples: [] }
+        @all ||= { shakapacker_examples: [] }
       end
 
       attr_reader :packer_type, :name, :generator_options

rakelib/example_type.rb

@@ -50,7 +50,6 @@ task :release, %i[gem_version dry_run tools_install] do |_t, args|
 
   # Having the examples prevents publishing
   Rake::Task["shakapacker_examples:clobber"].invoke
-  Rake::Task["webpacker_examples:clobber"].invoke
   # Delete any react_on_rails.gemspec except the root one
   sh_in_dir(gem_root, "find . -mindepth 2 -name 'react_on_rails.gemspec' -delete")
 

rakelib/release.rake

@@ -16,7 +16,6 @@ namespace :run_rspec do
   examples_config = symbolize_keys(YAML.safe_load_file(examples_config_file))
   examples_config[:example_type_data].each do |example_type_data|
     ExampleType.new(packer_type: "shakapacker_examples", **symbolize_keys(example_type_data))
-    ExampleType.new(packer_type: "webpacker_examples", **symbolize_keys(example_type_data))
   end
 
   spec_dummy_dir = File.join("spec", "dummy")
@@ -38,15 +37,6 @@ namespace :run_rspec do
                  command_name: "dummy_no_turbolinks")
   end
 
-  # Dynamically define Rake tasks for each example app found in the examples directory
-  ExampleType.all[:webpacker_examples].each do |example_type|
-    puts "Creating #{example_type.rspec_task_name} task"
-    desc "Runs RSpec for #{example_type.name_pretty} only"
-    task example_type.rspec_task_name_short => example_type.gen_task_name do
-      run_tests_in(File.join(examples_dir, example_type.name)) # have to use relative path
-    end
-  end
-
   # Dynamically define Rake tasks for each example app found in the examples directory
   ExampleType.all[:shakapacker_examples].each do |example_type|
     puts "Creating #{example_type.rspec_task_name} task"
@@ -56,11 +46,6 @@ namespace :run_rspec do
     end
   end
 
-  desc "Runs Rspec for webpacker example apps only"
-  task webpacker_examples: "webpacker_examples:gen_all" do
-    ExampleType.all[:webpacker_examples].each { |example_type| Rake::Task[example_type.rspec_task_name].invoke }
-  end
-
   desc "Runs Rspec for shakapacker example apps only"
   task shakapacker_examples: "shakapacker_examples:gen_all" do
     ExampleType.all[:shakapacker_examples].each { |example_type| Rake::Task[example_type.rspec_task_name].invoke }

rakelib/run_rspec.rake

@@ -32,7 +32,7 @@ module ReactOnRails
           .and_return(packer_public_output_path)
       end
 
-      it "does not throw if the generated assets dir is blank with webpacker" do
+      it "does not throw if the generated assets dir is blank with shakapacker" do
         expect do
           ReactOnRails.configure do |config|
             config.generated_assets_dir = ""
@@ -76,45 +76,7 @@ module ReactOnRails
     end
 
     describe ".build_production_command" do
-      context "when using Shakapacker 6", if: ReactOnRails::PackerUtils.packer_type != "shakapacker" do
-        it "fails when \"shakapacker_precompile\" is truly and \"build_production_command\" is truly" do
-          allow(Webpacker).to receive_message_chain("config.webpacker_precompile?")
-            .and_return(true)
-          expect do
-            ReactOnRails.configure do |config|
-              config.build_production_command = "RAILS_ENV=production NODE_ENV=production bin/shakapacker"
-            end
-          end.to raise_error(ReactOnRails::Error, /webpacker_precompile: false/)
-        end
-
-        it "doesn't fail when \"shakapacker_precompile\" is falsy and \"build_production_command\" is truly" do
-          allow(Webpacker).to receive_message_chain("config.webpacker_precompile?")
-            .and_return(false)
-          expect do
-            ReactOnRails.configure do |config|
-              config.build_production_command = "RAILS_ENV=production NODE_ENV=production bin/shakapacker"
-            end
-          end.not_to raise_error
-        end
-
-        it "doesn't fail when \"shakapacker_precompile\" is truly and \"build_production_command\" is falsy" do
-          allow(Webpacker).to receive_message_chain("config.webpacker_precompile?")
-            .and_return(true)
-          expect do
-            ReactOnRails.configure {} # rubocop:disable-line Lint/EmptyBlock
-          end.not_to raise_error
-        end
-
-        it "doesn't fail when \"shakapacker_precompile\" is falsy and \"build_production_command\" is falsy" do
-          allow(Webpacker).to receive_message_chain("config.webpacker_precompile?")
-            .and_return(false)
-          expect do
-            ReactOnRails.configure {} # rubocop:disable-line Lint/EmptyBlock
-          end.not_to raise_error
-        end
-      end
-
-      context "when using Shakapacker 8", if: ReactOnRails::PackerUtils.packer_type == "shakapacker" do
+      context "when using Shakapacker 8" do
         it "fails when \"shakapacker_precompile\" is truly and \"build_production_command\" is truly" do
           allow(Shakapacker).to receive_message_chain("config.shakapacker_precompile?")
             .and_return(true)

rakelib/webpacker_examples.rake

@@ -5,35 +5,23 @@
 
 RSpec.describe ReactOnRails::Dev::PackGenerator do
   describe ".generate" do
-    before do
-      # Stub the global system method to prevent actual command execution
-      allow_any_instance_of(Object).to receive(:system).and_return(true)
-    end
-
     it "runs pack generation successfully in verbose mode" do
       command = "bundle exec rake react_on_rails:generate_packs"
-      allow_any_instance_of(Object).to receive(:system).with(command).and_return(true)
+      allow(described_class).to receive(:system).with(command).and_return(true)
 
-      expect { described_class.generate(verbose: true) }.to output.to_stdout_from_any_process
+      expect { described_class.generate(verbose: true) }.to output(/📦 Generating React on Rails packs.../).to_stdout_from_any_process
     end
 
     it "runs pack generation successfully in quiet mode" do
       command = "bundle exec rake react_on_rails:generate_packs > /dev/null 2>&1"
-      allow_any_instance_of(Object).to receive(:system).with(command).and_return(true)
+      allow(described_class).to receive(:system).with(command).and_return(true)
 
-      # Suppress the print and puts output for quiet mode
-      allow_any_instance_of(Object).to receive(:print)
-      allow_any_instance_of(Object).to receive(:puts)
-
-      expect { described_class.generate(verbose: false) }.not_to output.to_stdout_from_any_process
+      expect { described_class.generate(verbose: false) }.to output(/📦 Generating packs\.\.\. ✅/).to_stdout_from_any_process
     end
 
     it "exits with error when pack generation fails" do
       command = "bundle exec rake react_on_rails:generate_packs > /dev/null 2>&1"
-      allow_any_instance_of(Object).to receive(:system).with(command).and_return(false)
-
-      # Suppress the puts output
-      allow_any_instance_of(Object).to receive(:puts)
+      allow(described_class).to receive(:system).with(command).and_return(false)
 
       expect { described_class.generate(verbose: false) }.to raise_error(SystemExit)
     end