shakapacker 10.1.0 → 10.3.0

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'."

data/lib/shakapacker/configuration.rb

@@ -27,6 +27,41 @@ require "active_support/core_ext/hash/indifferent_access"
 #
 # @see https://github.com/shakacode/shakapacker/blob/main/docs/shakapacker.yml.md
 class Shakapacker::Configuration
+  # Shared Ruby source of truth for Shakapacker-owned Node flags; Runner aliases this constant.
+  SHAKAPACKER_NODE_FLAGS = %w[--debug-shakapacker --trace-deprecation --no-deprecation].freeze
+
+  SHAKAPACKER_RUNNER_COMMANDS = %w[help h --help -h --help=verbose version v --version -v info i].freeze
+  private_constant :SHAKAPACKER_RUNNER_COMMANDS
+
+  SHAKAPACKER_HELP_FLAG_PATTERN = /\A(?:--help|-h)(?:=.*)?\z/
+  private_constant :SHAKAPACKER_HELP_FLAG_PATTERN
+
+  SHAKAPACKER_WATCH_FLAGS = %w[--watch -w].freeze
+  private_constant :SHAKAPACKER_WATCH_FLAGS
+
+  SHAKAPACKER_WATCH_FLAG_PATTERN = /\A(?:--watch|-w)(?:=.*)?\z/
+  private_constant :SHAKAPACKER_WATCH_FLAG_PATTERN
+
+  SHAKAPACKER_MANAGED_COMPILE_FLAGS =
+    %w[--config -c --node-env --nodeEnv --bundler --build --init --list-builds].freeze
+  private_constant :SHAKAPACKER_MANAGED_COMPILE_FLAGS
+
+  SHAKAPACKER_MANAGED_COMPILE_FLAG_PATTERN =
+    /\A(?:--config|-c|--node-env|--nodeEnv|--bundler|--build|--init|--list-builds)(?:=.*)?\z/
+  private_constant :SHAKAPACKER_MANAGED_COMPILE_FLAG_PATTERN
+
+  IMPLICIT_SWC_BABEL_FALLBACK_WARNING =
+    "`javascript_transpiler` is not set in config/shakapacker.yml. " \
+    "Shakapacker defaults to SWC, but swc-loader is not installed and Babel was detected, so Babel will be used. " \
+    "Set `javascript_transpiler: babel` (or `swc`) explicitly to silence this message. " \
+    "See https://github.com/shakacode/shakapacker/blob/main/docs/transpiler-migration.md"
+  private_constant :IMPLICIT_SWC_BABEL_FALLBACK_WARNING
+
+  DISALLOWED_WEBPACK_COMPILE_FLAGS =
+    (SHAKAPACKER_NODE_FLAGS + SHAKAPACKER_RUNNER_COMMANDS + SHAKAPACKER_WATCH_FLAGS +
+      SHAKAPACKER_MANAGED_COMPILE_FLAGS).freeze
+  private_constant :DISALLOWED_WEBPACK_COMPILE_FLAGS
+
   class << self
     # Flag indicating whether Shakapacker is currently being installed
     # Used to suppress certain validations during installation
@@ -233,6 +268,25 @@ class Shakapacker::Configuration
     fetch(:webpack_compile_output)
   end
 
+  # Returns extra command-line flags passed to the webpack/rspack compile command
+  #
+  # @return [Array<String>] bundler CLI flags
+  def webpack_compile_flags
+    flags = fetch(:webpack_compile_flags)
+    return [] if flags.nil?
+
+    valid_flags = flags.is_a?(Array) && flags.all? { |flag| valid_webpack_compile_flag?(flag) }
+
+    unless valid_flags
+      disallowed_flags = DISALLOWED_WEBPACK_COMPILE_FLAGS.join(", ")
+      raise "Shakapacker configuration error: compile flags (webpack_compile_flags) must be an array of " \
+            "non-empty strings and must not include \"--\" or Shakapacker-specific " \
+            "wrapper/short-circuit/watch/managed flags (#{disallowed_flags})"
+    end
+
+    flags
+  end
+
   # Returns the compiler strategy for determining staleness
   #
   # Options:
@@ -310,24 +364,54 @@ class Shakapacker::Configuration
   # Resolution order:
   # 1. javascript_transpiler setting in config file
   # 2. webpack_loader setting in config file (deprecated)
-  # 3. Default based on bundler (swc for rspack, babel for webpack)
+  # 3. Bundled defaults
+  # 4. Babel fallback for implicit webpack/SWC defaults when swc-loader is missing
   #
   # Validates that the configured transpiler matches installed packages.
   #
   # @return [String] "babel", "swc", or "esbuild"
   def javascript_transpiler
-    # Show deprecation warning if using old 'webpack_loader' key
-    if data.has_key?(:webpack_loader) && !data.has_key?(:javascript_transpiler)
-      $stderr.puts "⚠️  DEPRECATION WARNING: The 'webpack_loader' configuration option is deprecated. Please use 'javascript_transpiler' instead as it better reflects its purpose of configuring JavaScript transpilation regardless of the bundler used."
-    end
+    return @javascript_transpiler if defined?(@javascript_transpiler)
+
+    @javascript_transpiler =
+      begin
+        javascript_transpiler_configured = config_value_configured?(:javascript_transpiler)
+        webpack_loader_configured = config_value_present?(:webpack_loader)
 
-    # Use explicit config if set, otherwise default based on bundler
-    transpiler = fetch(:javascript_transpiler) || fetch(:webpack_loader) || default_javascript_transpiler
+        # Show deprecation warning if using old 'webpack_loader' key
+        if webpack_loader_configured && !javascript_transpiler_configured
+          $stderr.puts "⚠️  DEPRECATION WARNING: The 'webpack_loader' configuration option is deprecated. Please use 'javascript_transpiler' instead as it better reflects its purpose of configuring JavaScript transpilation regardless of the bundler used."
+        end
 
-    # Validate transpiler configuration
-    validate_transpiler_configuration(transpiler) unless self.class.installing
+        # Use explicit config if set, otherwise default based on bundler
+        current_javascript_transpiler =
+          if javascript_transpiler_configured
+            fetch(:javascript_transpiler)
+          elsif data.has_key?(:javascript_transpiler)
+            defaults[:javascript_transpiler]
+          else
+            fetch(:javascript_transpiler)
+          end
+
+        transpiler =
+          if webpack_loader_configured && !javascript_transpiler_configured
+            fetch(:webpack_loader) || default_javascript_transpiler
+          else
+            current_javascript_transpiler || fetch(:webpack_loader) || default_javascript_transpiler
+          end
 
-    transpiler
+        if !javascript_transpiler_configured &&
+            !webpack_loader_configured &&
+            implicit_swc_to_babel_fallback?(transpiler)
+          $stderr.puts IMPLICIT_SWC_BABEL_FALLBACK_WARNING
+          transpiler = "babel"
+        end
+
+        # Validate transpiler configuration
+        validate_transpiler_configuration(transpiler) unless self.class.installing
+
+        transpiler
+      end
   end
 
   # Deprecated alias for {#javascript_transpiler}
@@ -399,47 +483,139 @@ class Shakapacker::Configuration
 
   private
 
+    def valid_webpack_compile_flag?(flag)
+      flag.is_a?(String) &&
+        !flag.empty? &&
+        flag != "--" &&
+        !DISALLOWED_WEBPACK_COMPILE_FLAGS.include?(flag) &&
+        !SHAKAPACKER_HELP_FLAG_PATTERN.match?(flag) &&
+        !SHAKAPACKER_WATCH_FLAG_PATTERN.match?(flag) &&
+        !SHAKAPACKER_MANAGED_COMPILE_FLAG_PATTERN.match?(flag)
+    end
+
     def default_javascript_transpiler
       # RSpack has built-in SWC support, use it by default
       rspack? ? "swc" : "babel"
     end
 
+    def implicit_swc_to_babel_fallback?(transpiler)
+      webpack? &&
+        transpiler == "swc" &&
+        !package_dependency?("swc-loader") &&
+        package_dependency?("babel-loader")
+    end
+
+    def package_dependency?(package_name)
+      declared_package_dependencies.key?(package_name)
+    end
+
+    def config_value_present?(key)
+      return false unless data.has_key?(key)
+
+      value = data[key]
+      value.is_a?(String) && !value.strip.empty?
+    end
+
+    def config_value_configured?(key)
+      return false unless data.has_key?(key)
+
+      value = data[key]
+      return false if value.nil?
+      return false if value.is_a?(String) && value.strip.empty?
+
+      true
+    end
+
+    def declared_package_dependencies
+      return @declared_package_dependencies if defined?(@declared_package_dependencies)
+
+      @declared_package_dependencies =
+        package_json_paths.reverse_each.each_with_object({}) do |path, dependencies|
+          package_json = parse_package_json(path)
+          next unless package_json
+
+          dependencies.merge!(installable_package_dependencies(package_json))
+        end
+    end
+
+    def installable_package_dependencies(package_json)
+      (package_json["optionalDependencies"] || {})
+        .merge(package_json["devDependencies"] || {})
+        .merge(package_json["dependencies"] || {})
+    end
+
+    def package_json_paths
+      @package_json_paths ||= package_root_paths
+        .map { |path| path.join("package.json") }
+        .select(&:exist?)
+    end
+
+    def parse_package_json(path)
+      JSON.parse(File.read(path))
+    rescue JSON::ParserError, SystemCallError
+      nil
+    end
+
+    def package_root_paths
+      @package_root_paths ||= [javascript_package_root_path, root_path].uniq
+    end
+
+    def javascript_package_root_path
+      @javascript_package_root_path ||= begin
+        resolved_source_path = source_path.expand_path
+        app_root = root_path.expand_path
+
+        if path_within?(resolved_source_path, app_root)
+          current = resolved_source_path
+          loop do
+            break current if current.join("package.json").exist?
+            break root_path if current == app_root
+
+            parent = current.dirname
+            break root_path if parent == current
+
+            current = parent
+          end
+        else
+          root_path
+        end
+      rescue StandardError
+        root_path
+      end
+    end
+
+    def path_within?(path, parent)
+      path.to_s == parent.to_s || path.to_s.start_with?("#{parent}#{File::SEPARATOR}")
+    end
+
     def validate_transpiler_configuration(transpiler)
       return unless ENV["NODE_ENV"] != "test" # Skip validation in test environment
 
       # Skip validation if transpiler is set to 'none' (custom webpack config)
       return if transpiler == "none"
 
-      # Check if package.json exists
-      package_json_path = root_path.join("package.json")
-      return unless package_json_path.exist?
+      all_deps = declared_package_dependencies
+      return if all_deps.empty?
 
-      begin
-        package_json = JSON.parse(File.read(package_json_path))
-        all_deps = (package_json["dependencies"] || {}).merge(package_json["devDependencies"] || {})
-
-        # Check for transpiler mismatch
-        has_babel = all_deps.keys.any? { |pkg| pkg.start_with?("@babel/", "babel-") }
-        has_swc = all_deps.keys.any? { |pkg| pkg.include?("swc") }
-        has_esbuild = all_deps.keys.any? { |pkg| pkg.include?("esbuild") }
-
-        case transpiler
-        when "babel"
-          if !has_babel && has_swc
-            warn_transpiler_mismatch("Babel", "SWC packages found but Babel is configured")
-          end
-        when "swc"
-          if !has_swc && has_babel
-            warn_transpiler_mismatch("SWC", "Babel packages found but SWC is configured")
-          end
-        when "esbuild"
-          if !has_esbuild && (has_babel || has_swc)
-            other = has_babel ? "Babel" : "SWC"
-            warn_transpiler_mismatch("esbuild", "#{other} packages found but esbuild is configured")
-          end
+      # Check for transpiler mismatch
+      has_babel = all_deps.keys.any? { |pkg| pkg.start_with?("@babel/", "babel-") }
+      has_swc = all_deps.keys.any? { |pkg| pkg.include?("swc") }
+      has_esbuild = all_deps.keys.any? { |pkg| pkg.include?("esbuild") }
+
+      case transpiler
+      when "babel"
+        if !has_babel && has_swc
+          warn_transpiler_mismatch("Babel", "SWC packages found but Babel is configured")
+        end
+      when "swc"
+        if !has_swc && has_babel
+          warn_transpiler_mismatch("SWC", "Babel packages found but SWC is configured")
+        end
+      when "esbuild"
+        if !has_esbuild && (has_babel || has_swc)
+          other = has_babel ? "Babel" : "SWC"
+          warn_transpiler_mismatch("esbuild", "#{other} packages found but esbuild is configured")
         end
-      rescue JSON::ParserError
-        # Ignore if package.json is malformed
       end
     end
 

data/lib/shakapacker/dev_server_runner.rb

@@ -9,11 +9,13 @@ require_relative "version"
 module Shakapacker
   class DevServerRunner < Shakapacker::Runner
     def self.run(argv)
+      runner_argv, passthrough_argv = split_passthrough_argv(argv)
+
       # Show Shakapacker help and exit (don't call bundler)
-      if argv.include?("--help") || argv.include?("-h")
+      if runner_argv.include?("--help") || runner_argv.include?("-h")
         print_help
         exit(0)
-      elsif argv.include?("--version") || argv.include?("-v")
+      elsif runner_argv.include?("--version") || runner_argv.include?("-v")
         print_version
         exit(0)
       end
@@ -21,9 +23,9 @@ module Shakapacker
       Shakapacker.ensure_node_env!
 
       # Check for --build flag
-      build_index = argv.index("--build")
+      build_index = runner_argv.index("--build")
       if build_index
-        build_name = argv[build_index + 1]
+        build_name = runner_argv[build_index + 1]
 
         unless build_name
           $stderr.puts "[Shakapacker] Error: --build requires a build name"
@@ -51,11 +53,11 @@ module Shakapacker
           end
 
           # Remove --build and build name from argv
-          remaining_argv = argv.dup
+          remaining_argv = runner_argv.dup
           remaining_argv.delete_at(build_index + 1)
           remaining_argv.delete_at(build_index)
 
-          run_with_build_config(remaining_argv, build_config)
+          run_with_build_config(remaining_argv, build_config, passthrough_argv)
           return
         rescue ArgumentError => e
           $stderr.puts "[Shakapacker] #{e.message}"
@@ -63,10 +65,10 @@ module Shakapacker
         end
       end
 
-      new(argv).run
+      new(runner_argv, nil, nil, passthrough_argv).run
     end
 
-    def self.run_with_build_config(argv, build_config)
+    def self.run_with_build_config(argv, build_config, passthrough_argv = [])
       Shakapacker.ensure_node_env!
 
       # Apply build config environment variables
@@ -84,7 +86,7 @@ module Shakapacker
       puts "[Shakapacker] Config file: #{build_config[:config_file]}" if build_config[:config_file]
 
       # Pass bundler override so Configuration.assets_bundler reflects the build
-      new(argv, build_config, build_config[:bundler]).run
+      new(argv, build_config, build_config[:bundler], passthrough_argv).run
     end
 
     def self.print_help
@@ -99,8 +101,14 @@ module Shakapacker
           -h, --help              Show this help message
           -v, --version           Show Shakapacker version
           --debug-shakapacker     Enable Node.js debugging (--inspect-brk)
+          --trace-deprecation     Show stack traces for Node.js deprecations
+          --no-deprecation        Silence Node.js deprecation warnings
           --build <name>          Run a specific build configuration
 
+          Put Shakapacker-specific options before --. Arguments after -- are
+          passed directly to the selected bundler, except --host and --port,
+          which remain managed by config/shakapacker.yml.
+
         Build configurations (config/shakapacker-builds.yml):
           bin/shakapacker-dev-server --build dev-hmr    # Run the 'dev-hmr' build
 
@@ -116,6 +124,7 @@ module Shakapacker
           bin/shakapacker-dev-server --no-hot           # Disable HMR
           bin/shakapacker-dev-server --open             # Open browser automatically
           bin/shakapacker-dev-server --debug-shakapacker # Debug with Node inspector
+          bin/shakapacker-dev-server --trace-deprecation # Show Node deprecation traces
 
       HELP
 
@@ -126,12 +135,13 @@ module Shakapacker
         Options managed by Shakapacker (configured in config/shakapacker.yml):
           --host                  Set from dev_server.host (default: localhost)
           --port                  Set from dev_server.port (default: 3035)
-          --https                 Set from dev_server.server (http or https)
+          --https                 Requires dev_server.server: https before forwarding
           --config                Set automatically to config/webpack/webpack.config.js
                                   or config/rspack/rspack.config.js
 
-        Note: CLI flags for --host, --port, and --https are NOT supported.
-        Configure these in config/shakapacker.yml instead.
+        Note: CLI flags for --host and --port are NOT supported.
+        Configure these in config/shakapacker.yml instead. --https is accepted
+        only when dev_server.server is already set to https.
       HELP
     end
 
@@ -260,15 +270,19 @@ module Shakapacker
       UNSUPPORTED_SWITCHES = %w[--host --port]
       private_constant :UNSUPPORTED_SWITCHES
       def detect_unsupported_switches!
-        unsupported_switches = UNSUPPORTED_SWITCHES & @argv
+        # Host/port stay config-owned even after -- because Shakapacker assembles the dev-server command.
+        unsupported_switches = UNSUPPORTED_SWITCHES & bundler_argv
         if unsupported_switches.any?
-          $stdout.puts "The following CLI switches are not supported by Shakapacker: #{unsupported_switches.join(' ')}. Please edit your command and try again."
-          exit!
+          log_output.puts(
+            "The following CLI switches are not supported by Shakapacker: #{unsupported_switches.join(' ')}. " \
+            "Please set dev_server.host or dev_server.port in shakapacker.yml instead."
+          )
+          exit_after_shakapacker_flag_error(1)
         end
 
-        if @argv.include?("--https") && !@https
-          $stdout.puts "--https requires that 'server' in shakapacker.yml is set to 'https'"
-          exit!
+        if bundler_argv.include?("--https") && !@https
+          log_output.puts "--https requires that 'server' in shakapacker.yml is set to 'https'"
+          exit_after_shakapacker_flag_error(1)
         end
       end
 
@@ -289,10 +303,21 @@ module Shakapacker
 
         cmd = build_cmd
 
+        detect_shakapacker_flags_in_passthrough!
+
+        # Shakapacker-owned Node flags must appear before --; passthrough args belong to the bundler.
         if @argv.delete("--debug-shakapacker")
           env["NODE_OPTIONS"] = "#{env["NODE_OPTIONS"]} --inspect-brk --trace-warnings"
         end
 
+        if @argv.delete("--trace-deprecation")
+          env["NODE_OPTIONS"] = "#{env["NODE_OPTIONS"]} --trace-deprecation"
+        end
+
+        if @argv.delete("--no-deprecation")
+          env["NODE_OPTIONS"] = "#{env["NODE_OPTIONS"]} --no-deprecation"
+        end
+
         # Add config file
         cmd += ["--config", @webpack_config]
 
@@ -307,7 +332,7 @@ module Shakapacker
           cmd += ["--hot"] if @hot && @hot != false
         end
 
-        cmd += @argv
+        cmd += bundler_argv
 
         Dir.chdir(@app_path) do
           exec(env, *cmd)

data/lib/shakapacker/digest_strategy.rb

@@ -30,7 +30,7 @@ module Shakapacker
       end
 
       def watched_files_digest
-        if Rails.env.development?
+        if env.development?
           warn <<~MSG.strip
           Shakapacker::Compiler - Slow setup for development
           Prepare JS assets with either:
@@ -46,7 +46,7 @@ module Shakapacker
         files = Dir[*expanded_paths].reject { |f| File.directory?(f) }
         file_ids = files.sort.map { |f| "#{File.basename(f)}/#{Digest::SHA1.file(f).hexdigest}" }
 
-        asset_host = Shakapacker.config.asset_host.to_s
+        asset_host = config.asset_host.to_s
         Digest::SHA1.hexdigest(file_ids.join("/").concat(asset_host))
       end
 
@@ -56,7 +56,7 @@ module Shakapacker
       end
 
       def compilation_digest_path
-        config.cache_path.join("last-compilation-digest-#{Shakapacker.env}")
+        config.cache_path.join("last-compilation-digest-#{env}")
       end
   end
 end