shakapacker 10.1.0 → 10.2.0

data/lib/install/bin/diff-bundler-config

@@ -2,12 +2,9 @@
 # frozen_string_literal: true
 
 require "rbconfig"
+require "yaml"
 
-# Keep helper logic in sync across:
-# - lib/install/bin/shakapacker-config
-# - lib/install/bin/diff-bundler-config
-# - spec/dummy/bin/shakapacker-config
-# - package/configExporter/cli.ts (createBinStub).
+# This binstub is managed by Shakapacker. Delete it and rerun the install or init command to regenerate it.
 def shakapacker_app_root
   candidate = File.expand_path("..", __dir__)
   return candidate if File.exist?(File.join(candidate, "Gemfile"))
@@ -28,9 +25,11 @@ def shakapacker_executable_candidates(executable)
 end
 
 def shakapacker_find_executable(executable)
-  ENV.fetch("PATH", "").split(File::PATH_SEPARATOR).each do |path|
+  ENV.fetch("PATH", "").split(File::PATH_SEPARATOR, -1).each do |path|
+    search_path = path.empty? ? Dir.pwd : path
+
     shakapacker_executable_candidates(executable).each do |candidate|
-      executable_path = File.join(path, candidate)
+      executable_path = File.join(search_path, candidate)
       return executable_path if File.file?(executable_path) && File.executable?(executable_path)
     end
   end
@@ -51,22 +50,94 @@ def shakapacker_node_env
   %w[development test].include?(ENV["RAILS_ENV"]) ? "development" : "production"
 end
 
+def shakapacker_load_yaml_file(path)
+  YAML.load_file(path, aliases: true)
+rescue ArgumentError
+  YAML.load_file(path)
+end
+
+def shakapacker_source_path(app_root)
+  config_path = ENV["SHAKAPACKER_CONFIG"] || File.join("config", "shakapacker.yml")
+  config_path = File.expand_path(config_path, app_root)
+  return nil unless File.file?(config_path)
+
+  config = shakapacker_load_yaml_file(config_path)
+  return nil unless config.is_a?(Hash)
+
+  env = ENV["RAILS_ENV"] || ENV["RACK_ENV"] || "development"
+  env_config = config[env] || config["production"]
+  return nil unless env_config.is_a?(Hash)
+
+  source_path = env_config["source_path"]
+  source_path if source_path.is_a?(String)
+rescue Psych::Exception, SystemCallError, ArgumentError
+  nil
+end
+
+def shakapacker_path_within?(path, parent)
+  path == parent || path.start_with?("#{parent}#{File::SEPARATOR}")
+end
+
+def shakapacker_package_root_marker?(path)
+  %w[
+    package.json
+    bun.lockb
+    pnpm-lock.yaml
+    yarn.lock
+    package-lock.json
+    node_modules
+  ].any? { |entry| File.exist?(File.join(path, entry)) }
+end
+
+def shakapacker_client_root(app_root)
+  source_path = shakapacker_source_path(app_root)
+  return app_root unless source_path && !source_path.empty?
+
+  app_root = File.expand_path(app_root)
+  current = File.expand_path(source_path, app_root)
+  return app_root unless shakapacker_path_within?(current, app_root)
+
+  loop do
+    return current if shakapacker_package_root_marker?(current)
+    break if current == app_root
+
+    parent = File.dirname(current)
+    break if parent == current
+
+    current = parent
+  end
+
+  app_root
+end
+
+def shakapacker_package_script_path(package_root, package_script)
+  File.join(
+    package_root,
+    "node_modules",
+    "shakapacker",
+    "package",
+    "bin",
+    package_script
+  )
+end
+
+def shakapacker_package_script_paths(app_root, client_root, package_script)
+  [client_root, app_root].uniq.map do |package_root|
+    shakapacker_package_script_path(package_root, package_script)
+  end
+end
+
 ENV["RAILS_ENV"] ||= ENV["RACK_ENV"] || "development"
 ENV["NODE_ENV"] ||= shakapacker_node_env
 
 app_root = shakapacker_app_root
+client_root = shakapacker_client_root(app_root)
 node_bin = shakapacker_node_binary
-script_path = File.join(
-  app_root,
-  "node_modules",
-  "shakapacker",
-  "package",
-  "bin",
-  "diff-bundler-config.cjs"
-)
-
-unless File.file?(script_path)
-  warn "[Shakapacker] Could not find #{script_path}. Run your package manager install command and try again."
+script_paths = shakapacker_package_script_paths(app_root, client_root, "diff-bundler-config.cjs")
+script_path = script_paths.find { |path| File.file?(path) }
+
+unless script_path
+  warn "[Shakapacker] Could not find #{script_paths.join(' or ')}. Run your package manager install command and try again."
   exit 1
 end
 

data/lib/install/bin/shakapacker-config

@@ -2,12 +2,9 @@
 # frozen_string_literal: true
 
 require "rbconfig"
+require "yaml"
 
-# Keep helper logic in sync across:
-# - lib/install/bin/shakapacker-config
-# - lib/install/bin/diff-bundler-config
-# - spec/dummy/bin/shakapacker-config
-# - package/configExporter/cli.ts (createBinStub).
+# This binstub is managed by Shakapacker. Delete it and rerun the install or init command to regenerate it.
 def shakapacker_app_root
   candidate = File.expand_path("..", __dir__)
   return candidate if File.exist?(File.join(candidate, "Gemfile"))
@@ -28,9 +25,11 @@ def shakapacker_executable_candidates(executable)
 end
 
 def shakapacker_find_executable(executable)
-  ENV.fetch("PATH", "").split(File::PATH_SEPARATOR).each do |path|
+  ENV.fetch("PATH", "").split(File::PATH_SEPARATOR, -1).each do |path|
+    search_path = path.empty? ? Dir.pwd : path
+
     shakapacker_executable_candidates(executable).each do |candidate|
-      executable_path = File.join(path, candidate)
+      executable_path = File.join(search_path, candidate)
       return executable_path if File.file?(executable_path) && File.executable?(executable_path)
     end
   end
@@ -51,22 +50,94 @@ def shakapacker_node_env
   %w[development test].include?(ENV["RAILS_ENV"]) ? "development" : "production"
 end
 
+def shakapacker_load_yaml_file(path)
+  YAML.load_file(path, aliases: true)
+rescue ArgumentError
+  YAML.load_file(path)
+end
+
+def shakapacker_source_path(app_root)
+  config_path = ENV["SHAKAPACKER_CONFIG"] || File.join("config", "shakapacker.yml")
+  config_path = File.expand_path(config_path, app_root)
+  return nil unless File.file?(config_path)
+
+  config = shakapacker_load_yaml_file(config_path)
+  return nil unless config.is_a?(Hash)
+
+  env = ENV["RAILS_ENV"] || ENV["RACK_ENV"] || "development"
+  env_config = config[env] || config["production"]
+  return nil unless env_config.is_a?(Hash)
+
+  source_path = env_config["source_path"]
+  source_path if source_path.is_a?(String)
+rescue Psych::Exception, SystemCallError, ArgumentError
+  nil
+end
+
+def shakapacker_path_within?(path, parent)
+  path == parent || path.start_with?("#{parent}#{File::SEPARATOR}")
+end
+
+def shakapacker_package_root_marker?(path)
+  %w[
+    package.json
+    bun.lockb
+    pnpm-lock.yaml
+    yarn.lock
+    package-lock.json
+    node_modules
+  ].any? { |entry| File.exist?(File.join(path, entry)) }
+end
+
+def shakapacker_client_root(app_root)
+  source_path = shakapacker_source_path(app_root)
+  return app_root unless source_path && !source_path.empty?
+
+  app_root = File.expand_path(app_root)
+  current = File.expand_path(source_path, app_root)
+  return app_root unless shakapacker_path_within?(current, app_root)
+
+  loop do
+    return current if shakapacker_package_root_marker?(current)
+    break if current == app_root
+
+    parent = File.dirname(current)
+    break if parent == current
+
+    current = parent
+  end
+
+  app_root
+end
+
+def shakapacker_package_script_path(package_root, package_script)
+  File.join(
+    package_root,
+    "node_modules",
+    "shakapacker",
+    "package",
+    "bin",
+    package_script
+  )
+end
+
+def shakapacker_package_script_paths(app_root, client_root, package_script)
+  [client_root, app_root].uniq.map do |package_root|
+    shakapacker_package_script_path(package_root, package_script)
+  end
+end
+
 ENV["RAILS_ENV"] ||= ENV["RACK_ENV"] || "development"
 ENV["NODE_ENV"] ||= shakapacker_node_env
 
 app_root = shakapacker_app_root
+client_root = shakapacker_client_root(app_root)
 node_bin = shakapacker_node_binary
-script_path = File.join(
-  app_root,
-  "node_modules",
-  "shakapacker",
-  "package",
-  "bin",
-  "shakapacker-config.cjs"
-)
-
-unless File.file?(script_path)
-  warn "[Shakapacker] Could not find #{script_path}. Run your package manager install command and try again."
+script_paths = shakapacker_package_script_paths(app_root, client_root, "shakapacker-config.cjs")
+script_path = script_paths.find { |path| File.file?(path) }
+
+unless script_path
+  warn "[Shakapacker] Could not find #{script_paths.join(' or ')}. Run your package manager install command and try again."
   exit 1
 end
 

data/lib/install/config/shakapacker.yml

@@ -35,6 +35,12 @@ default: &default
   public_output_path: packs
   cache_path: tmp/shakapacker
   webpack_compile_output: true
+
+  # Extra command-line flags passed to webpack/rspack when compiling.
+  # Do not include "--", Shakapacker wrapper flags, help/version flags, watch flags/forms, or managed flags.
+  # Example: ["--progress", "--fail-on-warnings"]
+  webpack_compile_flags: []
+
   # See https://github.com/shakacode/shakapacker#deployment
   shakapacker_precompile: true
 
@@ -59,7 +65,7 @@ default: &default
   javascript_transpiler: "swc"
 
   # Select assets bundler to use
-  # Available options: 'webpack' (default) or 'rspack'
+  # Available options: 'webpack' or 'rspack'
   assets_bundler: "webpack"
 
   # Path to the directory containing webpack/rspack config files

data/lib/install/package.json

@@ -1,8 +1,9 @@
 {
   "rspack": {
-    "@rspack/cli": "^1.0.0 || ^2.0.0-0",
-    "@rspack/core": "^1.0.0 || ^2.0.0-0",
-    "rspack-manifest-plugin": "^5.0.0"
+    "@rspack/cli": "^2.0.0",
+    "@rspack/core": "^2.0.0",
+    "@rspack/dev-server": "^2.0.0",
+    "rspack-manifest-plugin": "^5.2.2"
   },
   "webpack": {
     "mini-css-extract-plugin": "^2.0.0",
@@ -17,6 +18,7 @@
   "common": {
     "compression-webpack-plugin": "^9.0.0 || ^10.0.0 || ^11.0.0 || ^12.0.0",
     "css-loader": "^6.0.0 || ^7.0.0",
+    "sass": "^1.50.0",
     "sass-loader": "^13.0.0 || ^14.0.0 || ^15.0.0 || ^16.0.0",
     "style-loader": "^3.0.0 || ^4.0.0"
   },

data/lib/install/template.rb

@@ -15,6 +15,31 @@ require "json"
 @package_json ||= PackageJson.new
 install_dir = File.expand_path(File.dirname(__FILE__))
 
+# Read the existing app's bundler (if any) before copy_file can overwrite it, so a
+# re-install can default to that bundler instead of silently switching it.
+config_path = Rails.root.join("config/shakapacker.yml")
+shakapacker_config_preexisting = config_path.exist?
+existing_assets_bundler =
+  File.read(config_path)[/assets_bundler:\s*"([^"]+)"/, 1] if shakapacker_config_preexisting
+
+# New installs default to rspack; an existing app keeps its current bundler on a
+# re-install unless overridden by the env var/argument or a FORCE overwrite (see
+# Env.resolve_assets_bundler for the precedence rules). The bundled shakapacker.yml
+# ships "webpack" for backward compatibility and is rewritten below when the chosen
+# bundler differs.
+assets_bundler = Shakapacker::Install::Env.resolve_assets_bundler(
+  env_value: ENV["SHAKAPACKER_ASSETS_BUNDLER"],
+  existing_bundler: existing_assets_bundler,
+  force: @conflict_option[:force]
+)
+
+# Fail fast on a misspelled SHAKAPACKER_ASSETS_BUNDLER instead of failing later
+# with a confusing missing-config-directory or peer-lookup error.
+unless Shakapacker::Install::Env::VALID_BUNDLERS.include?(assets_bundler)
+  say "❌ Unknown bundler '#{assets_bundler}'. Set SHAKAPACKER_ASSETS_BUNDLER to one of: #{Shakapacker::Install::Env::VALID_BUNDLERS.join(", ")}.", :red
+  exit 1
+end
+
 # Installation strategy:
 # - USE_BABEL_PACKAGES installs both babel AND swc for compatibility
 # - Otherwise install only the specified transpiler
@@ -35,7 +60,6 @@ else
 end
 
 # Copy config file
-shakapacker_config_preexisting = Rails.root.join("config/shakapacker.yml").exist?
 copy_file "#{install_dir}/config/shakapacker.yml", "config/shakapacker.yml", @conflict_option
 
 # Update config to match the selected transpiler
@@ -46,14 +70,59 @@ if Shakapacker::Install::Env.update_transpiler_config?(
   config_preexisting: shakapacker_config_preexisting
 )
   gsub_file "config/shakapacker.yml", 'javascript_transpiler: "swc"', "javascript_transpiler: \"#{@transpiler_to_install}\""
-  say "   📝 Updated config/shakapacker.yml to use #{@transpiler_to_install} transpiler", :green
+  # Unlike the bundler rewrite below (which only runs on installer-owned files and
+  # aborts on a miss), this can also run against a pre-existing user config whose
+  # transpiler line may legitimately differ from the shipped "swc" literal. A no-op
+  # there is not an error, so only claim success when the value actually landed.
+  if File.read(config_path).include?("javascript_transpiler: \"#{@transpiler_to_install}\"")
+    say "   📝 Updated config/shakapacker.yml to use #{@transpiler_to_install} transpiler", :green
+  end
+end
+
+# Update config to match the selected bundler (see update_assets_bundler_config?
+# for when the installer rewrites the shipped "webpack" default vs. preserves an
+# existing config).
+if Shakapacker::Install::Env.update_assets_bundler_config?(
+  assets_bundler_to_install: assets_bundler,
+  conflict_option: @conflict_option,
+  config_preexisting: shakapacker_config_preexisting
+)
+  gsub_file "config/shakapacker.yml", 'assets_bundler: "webpack"', "assets_bundler: \"#{assets_bundler}\""
+  # gsub_file silently no-ops if the shipped literal is ever reformatted, so verify
+  # the value landed. Abort rather than continue, since the installer is about to set
+  # up the chosen bundler's dependencies and config, and a mismatched bundler value
+  # would produce a broken install. This runs before any dependencies are installed.
+  if File.read(config_path).include?("assets_bundler: \"#{assets_bundler}\"")
+    say "   📝 Updated config/shakapacker.yml to use #{assets_bundler} bundler", :green
+  else
+    say "❌ Could not set assets_bundler to \"#{assets_bundler}\" in config/shakapacker.yml " \
+        "— the expected 'assets_bundler: \"webpack\"' line was not found. Aborting so the " \
+        "install doesn't proceed with a mismatched bundler config.", :red
+    exit 1
+  end
+else
+  # The bundler config was left as-is (an existing app's config is preserved unless
+  # FORCE overwrites it). Report the value present, and warn if it differs from the
+  # bundler whose dependencies/config are being installed — usually when a bundler is
+  # requested explicitly (env var or task argument) against a preserved config, but
+  # also when a preserved config holds an unrecognized bundler and resolve_assets_bundler
+  # falls back to rspack. Either case would otherwise be a silent mismatch. Only act
+  # when we can read the value back — never guess, or the message could contradict the file.
+  retained_bundler = File.read(config_path)[/assets_bundler:\s*"([^"]+)"/, 1]
+  if retained_bundler && retained_bundler != assets_bundler
+    say "⚠️  Installing #{assets_bundler} dependencies, but config/shakapacker.yml keeps " \
+        "assets_bundler: \"#{retained_bundler}\". To switch an existing app's bundler, run " \
+        "`bin/rake shakapacker:switch_bundler #{assets_bundler} -- --install-deps`, " \
+        "or re-run the installer with FORCE=true to overwrite the config.", :yellow
+  elsif retained_bundler
+    say "   📝 Keeping assets_bundler: \"#{retained_bundler}\" in config/shakapacker.yml", :green
+  end
 end
 
 # Detect TypeScript usage
 # Auto-detect from tsconfig.json or explicit via SHAKAPACKER_USE_TYPESCRIPT env var
 @use_typescript = File.exist?(Rails.root.join("tsconfig.json")) ||
   Shakapacker::Install::Env.truthy_env?("SHAKAPACKER_USE_TYPESCRIPT")
-assets_bundler = ENV["SHAKAPACKER_ASSETS_BUNDLER"] || "webpack"
 config_extension = @use_typescript ? "ts" : "js"
 
 say "Copying #{assets_bundler} core config (#{config_extension.upcase})"
@@ -224,9 +293,12 @@ Dir.chdir(Rails.root) do
   end
 
   # Inline fetch_peer_dependencies and fetch_common_dependencies
-  peers = PackageJson.read(install_dir).fetch(ENV["SHAKAPACKER_ASSETS_BUNDLER"] || "webpack")
+  peers = PackageJson.read(install_dir).fetch(assets_bundler)
   common_deps = Shakapacker::Install::Env.truthy_env?("SKIP_COMMON_LOADERS") ? {} : PackageJson.read(install_dir).fetch("common")
-  peers = peers.merge(common_deps)
+  peers = common_deps.merge(peers)
+  if assets_bundler == "rspack" && common_deps.key?("css-loader")
+    peers["css-loader"] = "^7.1.4"
+  end
 
   # Add transpiler-specific dependencies based on detected/configured transpiler
   # Inline the logic here since methods can't be called before they're defined in Rails templates
@@ -255,7 +327,9 @@ Dir.chdir(Rails.root) do
     peers = peers.merge(esbuild_deps)
   end
 
-  dev_dependency_packages = ["webpack-dev-server"]
+  # Lists both dev servers for classification only; just the one matching the chosen
+  # bundler appears in `peers`, so only that package is actually installed.
+  dev_dependency_packages = ["webpack-dev-server", "@rspack/dev-server"]
 
   dependencies_to_add = []
   dev_dependencies_to_add = []
@@ -290,12 +364,17 @@ Dir.chdir(Rails.root) do
     exit 1
   end
 
-  say "Installing webpack-dev-server for live reloading as a development dependency"
-  begin
-    @package_json.manager.add!(dev_dependencies_to_add, type: :dev)
-  rescue PackageJson::Error
-    say "Shakapacker installation failed 😭 See above for details.", :red
-    exit 1
+  if dev_dependencies_to_add.any?
+    # Strip the trailing @version, keeping the package name; the regex drops only
+    # the last @-segment so scoped names (e.g. @rspack/dev-server) survive.
+    dev_dependency_names = dev_dependencies_to_add.map { |entry| entry.sub(/@[^@]+\z/, "") }
+    say "Installing development dependencies: #{dev_dependency_names.join(", ")}"
+    begin
+      @package_json.manager.add!(dev_dependencies_to_add, type: :dev)
+    rescue PackageJson::Error
+      say "Shakapacker installation failed 😭 See above for details.", :red
+      exit 1
+    end
   end
 
   # Configure babel preset in package.json if babel is being used

data/lib/shakapacker/base_strategy.rb

@@ -1,7 +1,7 @@
 module Shakapacker
   class BaseStrategy
-    def initialize
-      @config = Shakapacker.config
+    def initialize(instance = Shakapacker.instance)
+      @instance = instance
     end
 
     def after_compile_hook
@@ -10,7 +10,13 @@ module Shakapacker
 
     private
 
-      attr_reader :config
+      def config
+        @instance.config
+      end
+
+      def env
+        @instance.env
+      end
 
       def default_watched_paths
         [
@@ -18,7 +24,7 @@ module Shakapacker
           "#{config.source_path}{,/**/*}",
           "package.json", "package-lock.json", "yarn.lock",
           "pnpm-lock.yaml", "bun.lockb",
-          "config/webpack{,/**/*}"
+          "config/{webpack,rspack}{,/**/*}"
         ].freeze
       end
   end

data/lib/shakapacker/bundler_switcher.rb

@@ -19,12 +19,13 @@ module Shakapacker
       prod: %w[webpack-merge]
     }.freeze
 
-    # Default dependencies for each bundler (package names only, no versions)
+    # Default dependencies for each bundler. Rspack entries include install-time
+    # version ranges; package_names strips them before removal and display.
     # Note: Excludes independent/optional dependencies like @swc/core, swc-loader (user-configured
     # transpilers)
     DEFAULT_RSPACK_DEPS = {
-      dev: %w[@rspack/cli @rspack/plugin-react-refresh],
-      prod: %w[@rspack/core rspack-manifest-plugin]
+      dev: %w[@rspack/cli@^2.0.0 @rspack/dev-server@^2.0.0 @rspack/plugin-react-refresh@^2.0.0],
+      prod: %w[@rspack/core@^2.0.0 rspack-manifest-plugin@^5.2.2]
     }.freeze
 
     DEFAULT_WEBPACK_DEPS = {
@@ -267,8 +268,8 @@ module Shakapacker
           # Show what will be removed (only when switching and not no_uninstall)
           if switching && !no_uninstall && (!deps_to_remove[:dev].empty? || !deps_to_remove[:prod].empty?)
             puts "   🗑️  Removing:"
-            deps_to_remove[:dev].each { |dep| puts "      - #{dep} (dev)" }
-            deps_to_remove[:prod].each { |dep| puts "      - #{dep} (prod)" }
+            package_names(deps_to_remove[:dev]).each { |dep| puts "      - #{dep} (dev)" }
+            package_names(deps_to_remove[:prod]).each { |dep| puts "      - #{dep} (prod)" }
             puts ""
           elsif switching && no_uninstall
             puts "   ⏭️  Skipping uninstall (--no-uninstall)"
@@ -302,7 +303,7 @@ module Shakapacker
 
         # Combine dev and prod dependencies into a single list for removal
         # Package managers remove packages from both dependencies and devDependencies sections if present
-        all_deps = deps[:dev] + deps[:prod]
+        all_deps = package_names(deps[:dev] + deps[:prod])
 
         unless all_deps.empty?
           unless package_json.manager.remove(all_deps)
@@ -381,6 +382,11 @@ module Shakapacker
       end
 
       def print_uninstall_commands(package_manager, deps)
+        deps = {
+          dev: package_names(deps[:dev]),
+          prod: package_names(deps[:prod])
+        }
+
         case package_manager
         when "yarn"
           puts "   yarn remove #{deps[:dev].join(' ')}" unless deps[:dev].empty?
@@ -397,6 +403,16 @@ module Shakapacker
         end
       end
 
+      def package_names(dependencies)
+        dependencies.map do |dependency|
+          if dependency.start_with?("@")
+            dependency.count("@") > 1 ? dependency.sub(/@[^@]+\z/, "") : dependency
+          else
+            dependency.sub(/@[^@]+\z/, "")
+          end
+        end
+      end
+
       # Load YAML config file with Ruby version compatibility
       # Ruby 3.1+ supports aliases: keyword, older versions need YAML.safe_load
       def load_yaml_config(path)

data/lib/shakapacker/compiler.rb

@@ -5,6 +5,9 @@ require "shellwords"
 require_relative "compiler_strategy"
 
 class Shakapacker::Compiler
+  SpawnFailure = Class.new(StandardError)
+  private_constant :SpawnFailure
+
   # Additional environment variables that the compiler is being run with
   # Shakapacker::Compiler.env['FRONTEND_API_KEY'] = 'your_secret_key'
   cattr_accessor(:env) { {} }
@@ -36,9 +39,16 @@ class Shakapacker::Compiler
     else
       acquire_ipc_lock do
         run_precompile_hook if should_run_precompile_hook?
-        run_webpack.tap do |success|
-          after_compile_hook
+        spawn_failed = false
+        begin
+          success = run_webpack
+        rescue SpawnFailure
+          spawn_failed = true
+          success = false
         end
+
+        after_compile_hook unless spawn_failed
+        success
       end
     end
   end
@@ -83,8 +93,9 @@ class Shakapacker::Compiler
       config.root_path.join("tmp/shakapacker.lock")
     end
 
+    # Returns one executable path for array-form Open3, not a shell command snippet.
     def optional_ruby_runner
-      first_line = File.readlines(bin_shakapacker_path).first.chomp
+      first_line = File.readlines(bin_shakapacker_path).first&.chomp || ""
       /ruby/.match?(first_line) ? RbConfig.ruby : ""
     end
 
@@ -176,11 +187,21 @@ class Shakapacker::Compiler
     def run_webpack
       logger.info "Compiling..."
 
-      stdout, stderr, status = Open3.capture3(
-        webpack_env,
-        "#{optional_ruby_runner} '#{bin_shakapacker_path}'",
-        chdir: File.expand_path(config.root_path)
-      )
+      # Fetch compile flags before the spawn rescue so configuration errors stay explicit.
+      compile_flags = config.webpack_compile_flags
+
+      begin
+        command = shakapacker_command(compile_flags)
+        stdout, stderr, status = Open3.capture3(
+          webpack_env,
+          *command,
+          chdir: File.expand_path(config.root_path)
+        )
+      rescue Errno::EACCES, Errno::ENOENT, Errno::ENOEXEC, Errno::EPERM, Errno::ENOTDIR => e
+        logger.error "\nCOMPILATION FAILED:\n#{e.class}: #{e.message}"
+        show_doctor_hint_once
+        raise SpawnFailure
+      end
 
       if status.success?
         logger.info "Compiled all packs in #{config.public_output_path}"
@@ -211,6 +232,15 @@ class Shakapacker::Compiler
       config.root_path.join("bin/shakapacker")
     end
 
+    def shakapacker_command(compile_flags)
+      runner = optional_ruby_runner
+      bin_path = bin_shakapacker_path.to_s
+      flags_part = compile_flags.any? ? ["--", *compile_flags] : []
+
+      # Use the [cmd, argv0] form so Open3 never routes a lone binstub path through the shell.
+      runner.empty? ? [[bin_path, bin_path], *flags_part] : [runner, bin_path, *flags_part]
+    end
+
     # Fires only after a failed compile, so users in a healthy loop never see the tip.
     def show_doctor_hint_once
       return if self.class.doctor_hint_shown

data/lib/shakapacker/compiler_strategy.rb

@@ -3,14 +3,14 @@ require_relative "digest_strategy"
 
 module Shakapacker
   class CompilerStrategy
-    def self.from_config
-      strategy_from_config = Shakapacker.config.compiler_strategy
+    def self.from_config(instance = Shakapacker.instance)
+      strategy_from_config = instance.config.compiler_strategy
 
       case strategy_from_config
       when "mtime"
-        Shakapacker::MtimeStrategy.new
+        Shakapacker::MtimeStrategy.new(instance)
       when "digest"
-        Shakapacker::DigestStrategy.new
+        Shakapacker::DigestStrategy.new(instance)
       else
         raise "Unknown strategy '#{strategy_from_config}'. " \
               "Available options are 'mtime' and 'digest'."