RubyGems - shakapacker - Versions diffs - 8.4.0 → 9.0.0 - Mend

shakapacker 8.4.0 → 9.0.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (166) hide show

checksums.yaml +4 -4
data/.eslintignore +1 -0
data/.eslintrc.fast.js +40 -0
data/.eslintrc.js +48 -0
data/.github/STATUS.md +1 -0
data/.github/workflows/claude-code-review.yml +54 -0
data/.github/workflows/claude.yml +50 -0
data/.github/workflows/dummy.yml +8 -4
data/.github/workflows/generator.yml +17 -14
data/.github/workflows/node.yml +23 -1
data/.github/workflows/ruby.yml +11 -0
data/.github/workflows/test-bundlers.yml +170 -0
data/.gitignore +17 -0
data/.husky/pre-commit +2 -0
data/.npmignore +56 -0
data/.prettierignore +3 -0
data/.rubocop.yml +1 -0
data/.yalcignore +26 -0
data/CHANGELOG.md +156 -18
data/CLAUDE.md +29 -0
data/CONTRIBUTING.md +138 -20
data/Gemfile.lock +3 -3
data/README.md +130 -5
data/Rakefile +39 -4
data/TODO.md +50 -0
data/TODO_v9.md +87 -0
data/conductor-setup.sh +70 -0
data/conductor.json +7 -0
data/docs/cdn_setup.md +379 -0
data/docs/css-modules-export-mode.md +512 -0
data/docs/deployment.md +10 -1
data/docs/optional-peer-dependencies.md +198 -0
data/docs/peer-dependencies.md +60 -0
data/docs/rspack.md +190 -0
data/docs/rspack_migration_guide.md +202 -0
data/docs/transpiler-migration.md +188 -0
data/docs/transpiler-performance.md +179 -0
data/docs/troubleshooting.md +5 -0
data/docs/typescript-migration.md +378 -0
data/docs/typescript.md +99 -0
data/docs/using_esbuild_loader.md +3 -3
data/docs/using_swc_loader.md +5 -3
data/docs/v6_upgrade.md +10 -0
data/docs/v9_upgrade.md +413 -0
data/lib/install/bin/shakapacker +3 -5
data/lib/install/config/rspack/rspack.config.js +6 -0
data/lib/install/config/rspack/rspack.config.ts +7 -0
data/lib/install/config/shakapacker.yml +12 -2
data/lib/install/config/webpack/webpack.config.ts +7 -0
data/lib/install/package.json +38 -0
data/lib/install/template.rb +194 -44
data/lib/shakapacker/configuration.rb +141 -0
data/lib/shakapacker/dev_server_runner.rb +25 -5
data/lib/shakapacker/doctor.rb +844 -0
data/lib/shakapacker/manifest.rb +4 -2
data/lib/shakapacker/rspack_runner.rb +19 -0
data/lib/shakapacker/runner.rb +144 -4
data/lib/shakapacker/swc_migrator.rb +376 -0
data/lib/shakapacker/utils/manager.rb +2 -0
data/lib/shakapacker/version.rb +1 -1
data/lib/shakapacker/version_checker.rb +1 -1
data/lib/shakapacker/webpack_runner.rb +4 -42
data/lib/shakapacker.rb +2 -1
data/lib/tasks/shakapacker/doctor.rake +8 -0
data/lib/tasks/shakapacker/install.rake +12 -2
data/lib/tasks/shakapacker/migrate_to_swc.rake +13 -0
data/lib/tasks/shakapacker.rake +1 -0
data/package/.npmignore +4 -0
data/package/babel/preset.ts +56 -0
data/package/config.ts +175 -0
data/package/{dev_server.js → dev_server.ts} +8 -5
data/package/env.ts +92 -0
data/package/environments/base.ts +138 -0
data/package/environments/development.ts +90 -0
data/package/environments/production.ts +80 -0
data/package/environments/test.ts +53 -0
data/package/environments/types.ts +90 -0
data/package/esbuild/index.ts +42 -0
data/package/index.d.ts +3 -97
data/package/index.ts +52 -0
data/package/loaders.d.ts +28 -0
data/package/optimization/rspack.ts +36 -0
data/package/optimization/webpack.ts +57 -0
data/package/plugins/rspack.ts +103 -0
data/package/plugins/webpack.ts +62 -0
data/package/rspack/index.ts +64 -0
data/package/rules/{babel.js → babel.ts} +2 -2
data/package/rules/{coffee.js → coffee.ts} +1 -1
data/package/rules/css.ts +3 -0
data/package/rules/{erb.js → erb.ts} +1 -1
data/package/rules/esbuild.ts +10 -0
data/package/rules/file.ts +40 -0
data/package/rules/{jscommon.js → jscommon.ts} +4 -4
data/package/rules/{less.js → less.ts} +4 -4
data/package/rules/raw.ts +25 -0
data/package/rules/rspack.ts +176 -0
data/package/rules/{sass.js → sass.ts} +7 -3
data/package/rules/{stylus.js → stylus.ts} +4 -8
data/package/rules/swc.ts +10 -0
data/package/rules/{index.js → webpack.ts} +1 -1
data/package/swc/index.ts +54 -0
data/package/types/README.md +87 -0
data/package/types/index.ts +60 -0
data/package/types.ts +108 -0
data/package/utils/configPath.ts +6 -0
data/package/utils/debug.ts +49 -0
data/package/utils/defaultConfigPath.ts +4 -0
data/package/utils/errorCodes.ts +219 -0
data/package/utils/errorHelpers.ts +143 -0
data/package/utils/getStyleRule.ts +64 -0
data/package/utils/helpers.ts +85 -0
data/package/utils/{inliningCss.js → inliningCss.ts} +3 -3
data/package/utils/pathValidation.ts +139 -0
data/package/utils/requireOrError.ts +15 -0
data/package/utils/snakeToCamelCase.ts +5 -0
data/package/utils/typeGuards.ts +342 -0
data/package/utils/validateDependencies.ts +61 -0
data/package/webpack-types.d.ts +33 -0
data/package/webpackDevServerConfig.ts +117 -0
data/package.json +134 -9
data/scripts/remove-use-strict.js +45 -0
data/scripts/type-check-no-emit.js +27 -0
data/test/package/config.test.js +3 -0
data/test/package/env.test.js +42 -7
data/test/package/environments/base.test.js +5 -1
data/test/package/rules/babel.test.js +16 -0
data/test/package/rules/esbuild.test.js +1 -1
data/test/package/rules/raw.test.js +40 -7
data/test/package/rules/swc.test.js +1 -1
data/test/package/rules/webpack.test.js +35 -0
data/test/package/staging.test.js +4 -3
data/test/package/transpiler-defaults.test.js +127 -0
data/test/peer-dependencies.sh +85 -0
data/test/scripts/remove-use-strict.test.js +125 -0
data/test/typescript/build.test.js +118 -0
data/test/typescript/environments.test.js +107 -0
data/test/typescript/pathValidation.test.js +142 -0
data/test/typescript/securityValidation.test.js +182 -0
data/tools/README.md +124 -0
data/tools/css-modules-v9-codemod.js +179 -0
data/tsconfig.eslint.json +16 -0
data/tsconfig.json +38 -0
data/yarn.lock +2704 -767
metadata +111 -41
data/package/babel/preset.js +0 -48
data/package/config.js +0 -56
data/package/env.js +0 -48
data/package/environments/base.js +0 -171
data/package/environments/development.js +0 -13
data/package/environments/production.js +0 -88
data/package/environments/test.js +0 -3
data/package/esbuild/index.js +0 -40
data/package/index.js +0 -40
data/package/rules/css.js +0 -3
data/package/rules/esbuild.js +0 -10
data/package/rules/file.js +0 -29
data/package/rules/raw.js +0 -5
data/package/rules/swc.js +0 -10
data/package/swc/index.js +0 -50
data/package/utils/configPath.js +0 -4
data/package/utils/defaultConfigPath.js +0 -2
data/package/utils/getStyleRule.js +0 -40
data/package/utils/helpers.js +0 -62
data/package/utils/snakeToCamelCase.js +0 -5
data/package/webpackDevServerConfig.js +0 -71
data/test/package/rules/index.test.js +0 -16

data/lib/install/template.rb CHANGED Viewed

@@ -2,26 +2,72 @@ require "shakapacker/utils/misc"
 require "shakapacker/utils/manager"
 require "shakapacker/utils/version_syntax_converter"
 require "package_json"
+require "yaml"
+require "json"
 # Install Shakapacker
 force_option = ENV["FORCE"] ? { force: true } : {}
-copy_file "#{__dir__}/config/shakapacker.yml", "config/shakapacker.yml", force_option
-remove_file "#{__dir__}/package.json" if force_option[:force]
+# Initialize variables for use throughout the template
+# Using instance variable to avoid method definition issues in Rails templates
+@package_json ||= PackageJson.new
+install_dir = File.expand_path(File.dirname(__FILE__))
-say "Copying webpack core config"
-directory "#{__dir__}/config/webpack", "config/webpack", force_option
+# Installation strategy:
+# - USE_BABEL_PACKAGES installs both babel AND swc for compatibility
+# - Otherwise install only the specified transpiler
+if ENV["USE_BABEL_PACKAGES"] == "true" || ENV["USE_BABEL_PACKAGES"] == "1"
+  @transpiler_to_install = "babel"
+  say "📦 Installing Babel packages (USE_BABEL_PACKAGES is set)", :yellow
+  say "✨ Also installing SWC packages for default config compatibility", :green
+elsif ENV["JAVASCRIPT_TRANSPILER"]
+  @transpiler_to_install = ENV["JAVASCRIPT_TRANSPILER"]
+  say "📦 Installing #{@transpiler_to_install} packages", :blue
+else
+  # Default to swc (matches the default in shakapacker.yml)
+  @transpiler_to_install = "swc"
+  say "✨ Installing SWC packages (20x faster than Babel)", :green
+end
+# Copy config file
+copy_file "#{install_dir}/config/shakapacker.yml", "config/shakapacker.yml", force_option
+# Update config if USE_BABEL_PACKAGES is set to ensure babel is used at runtime
+if @transpiler_to_install == "babel" && !ENV["JAVASCRIPT_TRANSPILER"]
+  # When USE_BABEL_PACKAGES is set, update the config to use babel
+  gsub_file "config/shakapacker.yml", "javascript_transpiler: 'swc'", "javascript_transpiler: 'babel'"
+  say "   📝 Updated config/shakapacker.yml to use Babel transpiler", :green
+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")) ||
+  ENV["SHAKAPACKER_USE_TYPESCRIPT"] == "true"
+assets_bundler = ENV["SHAKAPACKER_ASSETS_BUNDLER"] || "webpack"
+config_extension = @use_typescript ? "ts" : "js"
+say "Copying #{assets_bundler} core config (#{config_extension.upcase})"
+config_file = "#{assets_bundler}.config.#{config_extension}"
+source_config = "#{install_dir}/config/#{assets_bundler}/#{config_file}"
+dest_config = "config/#{assets_bundler}/#{config_file}"
+empty_directory "config/#{assets_bundler}"
+copy_file source_config, dest_config, force_option
+if @use_typescript
+  say "   ✨ Using TypeScript config for enhanced type safety", :green
+end
 if Dir.exist?(Shakapacker.config.source_path)
   say "The packs app source directory already exists"
 else
   say "Creating packs app source directory"
   empty_directory "app/javascript/packs"
-  copy_file "#{__dir__}/application.js", "app/javascript/packs/application.js"
+  copy_file "#{install_dir}/application.js", "app/javascript/packs/application.js"
 end
-apply "#{__dir__}/binstubs.rb"
+apply "#{install_dir}/binstubs.rb"
 git_ignore_path = Rails.root.join(".gitignore")
 if File.exist?(git_ignore_path)
@@ -44,17 +90,8 @@ else
   say %(        Add <%= javascript_pack_tag "application" %> within the <head> tag in your custom layout.)
 end
-def package_json
-  @package_json ||= PackageJson.new
-end
 # setup the package manager with default values
-package_json.merge! do |pj|
-  babel = pj.fetch("babel", {})
-  babel["presets"] ||= []
-  babel["presets"].push("./node_modules/shakapacker/package/babel/preset.js")
+@package_json.merge! do |pj|
   package_manager = pj.fetch("packageManager") do
     "#{Shakapacker::Utils::Manager.guess_binary}@#{Shakapacker::Utils::Manager.guess_version}"
   end
@@ -63,7 +100,6 @@ package_json.merge! do |pj|
     "name" => "app",
     "private" => true,
     "version" => "0.1.0",
-    "babel" => babel,
     "browserslist" => [
       "defaults"
     ],
@@ -75,7 +111,7 @@ Shakapacker::Utils::Manager.error_unless_package_manager_is_obvious!
 # Ensure there is `system!("bin/yarn")` command in `./bin/setup` file
 if (setup_path = Rails.root.join("bin/setup")).exist?
-  native_install_command = package_json.manager.native_install_command.join(" ")
+  native_install_command = @package_json.manager.native_install_command.join(" ")
   say "Run #{native_install_command} during bin/setup"
@@ -104,40 +140,126 @@ if (setup_path = Rails.root.join("bin/setup")).exist?
   end
 end
-def add_dependencies(dependencies, type)
-  package_json.manager.add!(dependencies, type: type)
-rescue PackageJson::Error
-  say "Shakapacker installation failed 😭 See above for details.", :red
-  exit 1
-end
+Dir.chdir(Rails.root) do
+  # In CI, use the pre-packed tarball if available
+  if ENV["SHAKAPACKER_NPM_PACKAGE"]
+    package_path = ENV["SHAKAPACKER_NPM_PACKAGE"]
-def fetch_peer_dependencies
-  PackageJson.read("#{__dir__}/../../").fetch("peerDependencies")
-end
+    # Validate package path to prevent directory traversal and invalid file types
+    begin
+      # Resolve to absolute path
+      absolute_path = File.expand_path(package_path)
-Dir.chdir(Rails.root) do
-  npm_version = Shakapacker::Utils::VersionSyntaxConverter.new.rubygem_to_npm(Shakapacker::VERSION)
-  say "Installing shakapacker@#{npm_version}"
-  add_dependencies(["shakapacker@#{npm_version}"], :production)
-  package_json.merge! do |pj|
-    {
-      "dependencies" => pj["dependencies"].merge({
-        # TODO: workaround for test suite - long-run need to actually account for diff pkg manager behaviour
-        "shakapacker" => pj["dependencies"]["shakapacker"].delete_prefix("^")
-      })
-    }
+      # Reject paths containing directory traversal
+      if package_path.include?("..") || absolute_path.include?("..")
+        say "❌ Security Error: Package path contains directory traversal: #{package_path}", :red
+        exit 1
+      end
+      # Ensure filename ends with .tgz or .tar.gz
+      unless absolute_path.end_with?(".tgz", ".tar.gz")
+        say "❌ Security Error: Package must be a .tgz or .tar.gz file: #{package_path}", :red
+        exit 1
+      end
+      # Check existence only after validation
+      if File.exist?(absolute_path)
+        say "📦 Installing shakapacker from local package: #{absolute_path}", :cyan
+        begin
+          @package_json.manager.add!([absolute_path], type: :production)
+        rescue PackageJson::Error
+          say "Shakapacker installation failed 😭 See above for details.", :red
+          exit 1
+        end
+      else
+        say "⚠️  SHAKAPACKER_NPM_PACKAGE set but file not found: #{absolute_path}", :yellow
+        say "Falling back to npm registry...", :yellow
+        npm_version = Shakapacker::Utils::VersionSyntaxConverter.new.rubygem_to_npm(Shakapacker::VERSION)
+        begin
+          @package_json.manager.add!(["shakapacker@#{npm_version}"], type: :production)
+        rescue PackageJson::Error
+          say "Shakapacker installation failed 😭 See above for details.", :red
+          exit 1
+        end
+      end
+    rescue => e
+      say "❌ Error validating package path: #{e.message}", :red
+      exit 1
+    end
+  else
+    npm_version = Shakapacker::Utils::VersionSyntaxConverter.new.rubygem_to_npm(Shakapacker::VERSION)
+    say "Installing shakapacker@#{npm_version}"
+    begin
+      @package_json.manager.add!(["shakapacker@#{npm_version}"], type: :production)
+    rescue PackageJson::Error
+      say "Shakapacker installation failed 😭 See above for details.", :red
+      exit 1
+    end
+  end
+  @package_json.merge! do |pj|
+    if pj["dependencies"] && pj["dependencies"]["shakapacker"]
+      {
+        "dependencies" => pj["dependencies"].merge({
+          # TODO: workaround for test suite - long-run need to actually account for diff pkg manager behaviour
+          "shakapacker" => pj["dependencies"]["shakapacker"].delete_prefix("^")
+        })
+      }
+    else
+      pj
+    end
+  end
+  # Inline fetch_peer_dependencies and fetch_common_dependencies
+  peers = PackageJson.read(install_dir).fetch(ENV["SHAKAPACKER_ASSETS_BUNDLER"] || "webpack")
+  common_deps = ENV["SKIP_COMMON_LOADERS"] ? {} : PackageJson.read(install_dir).fetch("common")
+  peers = peers.merge(common_deps)
+  # 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
+  # Install transpiler-specific dependencies
+  # When USE_BABEL_PACKAGES is set, install both babel AND swc
+  # This ensures backward compatibility while supporting the default config
+  if @transpiler_to_install == "babel"
+    # Install babel packages
+    babel_deps = PackageJson.read(install_dir).fetch("babel")
+    peers = peers.merge(babel_deps)
+    # Also install SWC since that's what the default config uses
+    # This ensures the runtime works regardless of config
+    swc_deps = PackageJson.read(install_dir).fetch("swc")
+    peers = peers.merge(swc_deps)
+    say "ℹ️  Installing both Babel and SWC packages for compatibility:", :blue
+    say "   - Babel packages are installed as requested via USE_BABEL_PACKAGES", :blue
+    say "   - SWC packages are also installed to ensure the default config works", :blue
+    say "   - Your actual transpiler will be determined by your shakapacker.yml configuration", :blue
+  elsif @transpiler_to_install == "swc"
+    swc_deps = PackageJson.read(install_dir).fetch("swc")
+    peers = peers.merge(swc_deps)
+  elsif @transpiler_to_install == "esbuild"
+    esbuild_deps = PackageJson.read(install_dir).fetch("esbuild")
+    peers = peers.merge(esbuild_deps)
   end
-  peers = fetch_peer_dependencies
   dev_dependency_packages = ["webpack-dev-server"]
   dependencies_to_add = []
   dev_dependencies_to_add = []
   peers.each do |(package, version)|
-    major_version = version.split("||").last.match(/(\d+)/)[1]
-    entry = "#{package}@#{major_version}"
+    # Handle versions like "^1.3.0" or ">= 4 || 5"
+    if version.start_with?("^", "~") || version.match?(/^\d+\.\d+/)
+      # Already has proper version format, use as-is
+      entry = "#{package}@#{version}"
+    else
+      # Extract major version from complex version strings like ">= 4 || 5"
+      # Fallback to "latest" if no version number found
+      version_match = version.split("||").last.match(/(\d+)/)
+      major_version = version_match ? version_match[1] : "latest"
+      entry = "#{package}@#{major_version}"
+    end
     if dev_dependency_packages.include? package
       dev_dependencies_to_add << entry
@@ -147,8 +269,36 @@ Dir.chdir(Rails.root) do
   end
   say "Adding shakapacker peerDependencies"
-  add_dependencies(dependencies_to_add, :production)
+  begin
+    @package_json.manager.add!(dependencies_to_add, type: :production)
+  rescue PackageJson::Error
+    say "Shakapacker installation failed 😭 See above for details.", :red
+    exit 1
+  end
   say "Installing webpack-dev-server for live reloading as a development dependency"
-  add_dependencies(dev_dependencies_to_add, :dev)
+  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
+  # Configure babel preset in package.json if babel is being used
+  if @transpiler_to_install == "babel"
+    @package_json.merge! do |pj|
+      babel = pj.fetch("babel", {})
+      babel["presets"] ||= []
+      unless babel["presets"].include?("./node_modules/shakapacker/package/babel/preset.js")
+        babel["presets"].push("./node_modules/shakapacker/package/babel/preset.js")
+      end
+      { "babel" => babel }
+    end
+  end
+end
+# Helper methods defined at the end (Rails template convention)
+def package_json
+  @package_json
 end

data/lib/shakapacker/configuration.rb CHANGED Viewed

@@ -1,4 +1,5 @@
 require "yaml"
+require "json"
 require "active_support/core_ext/hash/keys"
 require "active_support/core_ext/hash/indifferent_access"
@@ -68,6 +69,13 @@ class Shakapacker::Configuration
     root_path.join(fetch(:public_root_path))
   end
+  def private_output_path
+    private_path = fetch(:private_output_path)
+    return nil unless private_path
+    validate_output_paths!
+    root_path.join(private_path)
+  end
   def public_output_path
     public_path.join(fetch(:public_output_path))
   end
@@ -88,6 +96,107 @@ class Shakapacker::Configuration
     fetch(:compiler_strategy)
   end
+  def assets_bundler
+    # Show deprecation warning if using old 'bundler' key
+    if data.has_key?(:bundler) && !data.has_key?(:assets_bundler)
+      $stderr.puts "⚠️  DEPRECATION WARNING: The 'bundler' configuration option is deprecated. Please use 'assets_bundler' instead to avoid confusion with Ruby's Bundler gem manager."
+    end
+    ENV["SHAKAPACKER_ASSETS_BUNDLER"] || fetch(:assets_bundler) || fetch(:bundler) || "webpack"
+  end
+  # Deprecated: Use assets_bundler instead
+  def bundler
+    assets_bundler
+  end
+  def rspack?
+    assets_bundler == "rspack"
+  end
+  def webpack?
+    assets_bundler == "webpack"
+  end
+  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
+    # Use explicit config if set, otherwise default based on bundler
+    transpiler = fetch(:javascript_transpiler) || fetch(:webpack_loader) || default_javascript_transpiler
+    # Validate transpiler configuration
+    validate_transpiler_configuration(transpiler) unless self.class.installing
+    transpiler
+  end
+  # Deprecated: Use javascript_transpiler instead
+  def webpack_loader
+    javascript_transpiler
+  end
+  private
+    def default_javascript_transpiler
+      # RSpack has built-in SWC support, use it by default
+      rspack? ? "swc" : "babel"
+    end
+    def validate_transpiler_configuration(transpiler)
+      return unless ENV["NODE_ENV"] != "test" # Skip validation in test environment
+      # Check if package.json exists
+      package_json_path = root_path.join("package.json")
+      return unless package_json_path.exist?
+      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
+        end
+      rescue JSON::ParserError
+        # Ignore if package.json is malformed
+      end
+    end
+    def warn_transpiler_mismatch(configured, message)
+      $stderr.puts <<~WARNING
+        ⚠️  Transpiler Configuration Mismatch Detected:
+           #{message}
+           Configured transpiler: #{configured}
+        #{'   '}
+           This might cause unexpected behavior or build failures.
+        #{'   '}
+           To fix this:
+           1. Run 'rails shakapacker:migrate_to_swc' to migrate to SWC (recommended for 20x faster builds)
+           2. Or install the correct packages for #{configured}
+           3. Or update your shakapacker.yml to match your installed packages
+      WARNING
+    end
+  public
   def fetch(key)
     data.fetch(key, defaults[key])
   end
@@ -104,6 +213,38 @@ class Shakapacker::Configuration
   end
   private
+    def validate_output_paths!
+      # Skip validation if already validated to avoid redundant checks
+      return if @validated_output_paths
+      @validated_output_paths = true
+      # Only validate when both paths are configured
+      return unless fetch(:private_output_path) && fetch(:public_output_path)
+      private_path_str, public_path_str = resolve_paths_for_comparison
+      if private_path_str == public_path_str
+        raise "Shakapacker configuration error: private_output_path and public_output_path must be different. " \
+              "Both paths resolve to '#{private_path_str}'. " \
+              "The private_output_path is for server-side bundles (e.g., SSR) that should not be served publicly."
+      end
+    end
+    def resolve_paths_for_comparison
+      private_full_path = root_path.join(fetch(:private_output_path))
+      public_full_path = root_path.join(fetch(:public_root_path), fetch(:public_output_path))
+      # Create directories if they don't exist (for testing)
+      private_full_path.mkpath unless private_full_path.exist?
+      public_full_path.mkpath unless public_full_path.exist?
+      # Use realpath to resolve symbolic links and relative paths
+      [private_full_path.realpath.to_s, public_full_path.realpath.to_s]
+    rescue Errno::ENOENT
+      # If paths don't exist yet, fall back to cleanpath for comparison
+      [private_full_path.cleanpath.to_s, public_full_path.cleanpath.to_s]
+    end
     def data
       @data ||= load
     end

data/lib/shakapacker/dev_server_runner.rb CHANGED Viewed

@@ -7,6 +7,10 @@ require_relative "runner"
 module Shakapacker
   class DevServerRunner < Shakapacker::Runner
+    def self.run(argv)
+      new(argv).run
+    end
     def run
       load_config
       detect_unsupported_switches!
@@ -75,12 +79,19 @@ module Shakapacker
           env["NODE_OPTIONS"] = "#{env["NODE_OPTIONS"]} --inspect-brk --trace-warnings"
         end
+        # Add config file
         cmd += ["--config", @webpack_config]
-        cmd += ["--progress", "--color"] if @pretty
-        # Default behavior of webpack-dev-server is @hot = true
-        cmd += ["--hot", "only"] if @hot == "only"
-        cmd += ["--no-hot"] if !@hot
+        # Add assets bundler-specific flags
+        if webpack?
+          cmd += ["--progress", "--color"] if @pretty
+          # Default behavior of webpack-dev-server is @hot = true
+          cmd += ["--hot", "only"] if @hot == "only"
+          cmd += ["--no-hot"] if !@hot
+        elsif rspack?
+          # Rspack supports --hot but not --no-hot or --progress/--color
+          cmd += ["--hot"] if @hot && @hot != false
+        end
         cmd += @argv
@@ -90,7 +101,16 @@ module Shakapacker
       end
       def build_cmd
-        package_json.manager.native_exec_command("webpack", ["serve"])
+        command = @config.rspack? ? "rspack" : "webpack"
+        package_json.manager.native_exec_command(command, ["serve"])
+      end
+      def webpack?
+        @config.webpack?
+      end
+      def rspack?
+        @config.rspack?
       end
   end
 end