shakapacker 9.0.0.beta.9 → 9.0.0.beta.11

data/docs/v9_upgrade.md

@@ -9,12 +9,38 @@ This guide outlines new features, breaking changes, and migration steps for upgr
 Shakapacker v9 includes TypeScript definitions for better IDE support and type safety.
 
 - **No breaking changes** - JavaScript configs continue to work
-- **Optional** - Use TypeScript only if you want it  
+- **Optional** - Use TypeScript only if you want it
 - **Type safety** - Catch configuration errors at compile-time
 - **IDE support** - Full autocomplete for all options
 
 See the [TypeScript Documentation](./typescript.md) for usage examples.
 
+### NODE_ENV Default Behavior Fixed
+
+**What changed:** NODE_ENV now intelligently defaults based on RAILS_ENV instead of always defaulting to "production".
+
+**New behavior:**
+
+- When `RAILS_ENV=production` → `NODE_ENV` defaults to `"production"`
+- When `RAILS_ENV=development` or unset → `NODE_ENV` defaults to `"development"`
+- When `RAILS_ENV` is any other value (test, staging, etc.) → `NODE_ENV` defaults to `"development"`
+
+**Benefits:**
+
+- **Dev server "just works"** - No need to explicitly set NODE_ENV when running the development server
+- **Correct configuration loaded** - Development server now properly loads the development configuration from shakapacker.yml
+- **Fixes port issues** - Dev server uses the configured port (e.g., 3035) instead of defaulting to 8080
+- **Fixes 404 errors** - Assets load correctly without requiring manual NODE_ENV configuration
+
+**No action required** - This change improves the default behavior and requires no migration.
+
+**If you previously worked around this bug**, you can now remove these workarounds:
+
+- Remove `NODE_ENV=development` from your `.env`, `.env.development`, or `.env.local` files
+- Remove `NODE_ENV=development` from your `docker-compose.yml` or Dockerfile
+- Remove custom scripts that set NODE_ENV before running the dev server
+- Remove `NODE_ENV=development` from your `bin/dev` or Procfile.dev
+
 ## Breaking Changes
 
 ### 1. CSS Modules Configuration Changed to Named Exports
@@ -25,25 +51,27 @@ See the [TypeScript Documentation](./typescript.md) for usage examples.
 
 **Quick Reference: Configuration Options**
 
-| Configuration | namedExport | exportLocalsConvention | CSS: `.my-button` | Export Available | Works With |
-|---------------|-------------|------------------------|-------------------|------------------|------------|
-| **v9 Default** | `true` | `'camelCaseOnly'` | `.my-button` | `myButton` only | ✅ Named exports |
-| **Alternative** | `true` | `'dashesOnly'` | `.my-button` | `'my-button'` only | ✅ Named exports |
-| **v8 Style** | `false` | `'camelCase'` | `.my-button` | Both `myButton` AND `'my-button'` | ✅ Default export |
-| **❌ Invalid** | `true` | `'camelCase'` | - | - | ❌ Build Error |
+| Configuration   | namedExport | exportLocalsConvention | CSS: `.my-button` | Export Available                  | Works With        |
+| --------------- | ----------- | ---------------------- | ----------------- | --------------------------------- | ----------------- |
+| **v9 Default**  | `true`      | `'camelCaseOnly'`      | `.my-button`      | `myButton` only                   | ✅ Named exports  |
+| **Alternative** | `true`      | `'dashesOnly'`         | `.my-button`      | `'my-button'` only                | ✅ Named exports  |
+| **v8 Style**    | `false`     | `'camelCase'`          | `.my-button`      | Both `myButton` AND `'my-button'` | ✅ Default export |
+| **❌ Invalid**  | `true`      | `'camelCase'`          | -                 | -                                 | ❌ Build Error    |
 
 **JavaScript Projects:**
+
 ```js
 // Before (v8)
-import styles from './Component.module.css';
-<button className={styles.button} />
+import styles from "./Component.module.css"
+;<button className={styles.button} />
 
 // After (v9)
-import { button } from './Component.module.css';
-<button className={button} />
+import { button } from "./Component.module.css"
+;<button className={button} />
 ```
 
 **TypeScript Projects:**
+
 ```typescript
 // Before (v8)
 import styles from './Component.module.css';
@@ -57,6 +85,7 @@ import * as styles from './Component.module.css';
 **Migration Options:**
 
 1. **Update your code** (Recommended):
+
    - JavaScript: Change to named imports (`import { className }`)
    - TypeScript: Change to namespace imports (`import * as styles`)
    - Kebab-case class names are automatically converted to camelCase
@@ -66,6 +95,7 @@ import * as styles from './Component.module.css';
    - This gives you time to migrate gradually
 
 **Benefits of the change:**
+
 - Eliminates webpack/TypeScript warnings
 - Better tree-shaking of unused CSS classes
 - More explicit about which classes are used
@@ -76,15 +106,17 @@ import * as styles from './Component.module.css';
 **What changed:** The configuration option has been renamed to better reflect its purpose.
 
 **Before (v8):**
+
 ```yml
 # config/shakapacker.yml
-webpack_loader: 'babel'
+webpack_loader: "babel"
 ```
 
 **After (v9):**
+
 ```yml
 # config/shakapacker.yml
-javascript_transpiler: 'babel'
+javascript_transpiler: "babel"
 ```
 
 **Note:** The old `webpack_loader` option is deprecated but still supported with a warning.
@@ -96,38 +128,48 @@ javascript_transpiler: 'babel'
 **Why:** SWC is 20x faster than Babel while maintaining compatibility with most JavaScript and TypeScript code.
 
 **Impact on existing projects:**
+
 - Your project will continue using Babel if you already have babel packages in package.json
 - To switch to SWC for better performance, see migration options below
 
 **Impact on new projects:**
+
 - New installations will use SWC by default
 - Babel dependencies won't be installed unless explicitly configured
 
 ### Migration Options
 
 #### Option 1 (Recommended): Switch to SWC
+
 ```yml
 # config/shakapacker.yml
-javascript_transpiler: 'swc'
+javascript_transpiler: "swc"
 ```
+
 Then install SWC:
+
 ```bash
 npm install @swc/core swc-loader
 ```
 
 #### Option 2: Keep using Babel
+
 ```yml
 # config/shakapacker.yml
-javascript_transpiler: 'babel'
+javascript_transpiler: "babel"
 ```
+
 No other changes needed - your existing babel packages will continue to work.
 
 #### Option 3: Use esbuild
+
 ```yml
 # config/shakapacker.yml
-javascript_transpiler: 'esbuild'
+javascript_transpiler: "esbuild"
 ```
+
 Then install esbuild:
+
 ```bash
 npm install esbuild esbuild-loader
 ```
@@ -138,7 +180,7 @@ npm install esbuild esbuild-loader
 
 ```yml
 # config/shakapacker.yml
-assets_bundler: 'rspack'  # or 'webpack' (default)
+assets_bundler: "rspack" # or 'webpack' (default)
 ```
 
 ### 5. All Peer Dependencies Now Optional
@@ -146,16 +188,19 @@ assets_bundler: 'rspack'  # or 'webpack' (default)
 **What changed:** All peer dependencies are now marked as optional via `peerDependenciesMeta`.
 
 **Benefits:**
+
 - **No installation warnings** - You won't see peer dependency warnings for packages you don't use
 - **Install only what you need** - Using webpack? Don't install rspack. Using SWC? Don't install Babel.
 - **Clear version constraints** - When you do install a package, version compatibility is still enforced
 
 **What this means for you:**
+
 - **Existing projects:** No changes needed. Your existing dependencies will continue to work.
 - **New projects:** The installer only adds the packages you actually need based on your configuration.
 - **Package manager behavior:** npm, yarn, and pnpm will no longer warn about missing peer dependencies.
 
 **Example:** If you're using SWC with webpack, you only need:
+
 ```json
 {
   "dependencies": {
@@ -168,6 +213,7 @@ assets_bundler: 'rspack'  # or 'webpack' (default)
   }
 }
 ```
+
 You won't get warnings about missing Babel, Rspack, or esbuild packages.
 
 ## Migration Steps
@@ -186,22 +232,22 @@ yarn upgrade shakapacker@^9.0.0
 
 ```js
 // Find imports like this:
-import styles from './styles.module.css';
+import styles from "./styles.module.css"
 
 // Replace with named imports:
-import { className1, className2 } from './styles.module.css';
+import { className1, className2 } from "./styles.module.css"
 ```
 
 #### Update TypeScript definitions:
 
 ```typescript
 // Update your CSS module type definitions
-declare module '*.module.css' {
+declare module "*.module.css" {
   // With namedExport: true, css-loader generates individual named exports
   // TypeScript can't know the exact names at compile time, so we declare
   // a module with any number of string exports
-  const classes: { readonly [key: string]: string };
-  export = classes;
+  const classes: { readonly [key: string]: string }
+  export = classes
   // Note: This allows 'import * as styles' but not 'import styles from'
   // because css-loader with namedExport: true doesn't generate a default export
 }
@@ -213,13 +259,15 @@ v9 automatically converts kebab-case to camelCase with `exportLocalsConvention:
 
 ```css
 /* styles.module.css */
-.my-button { }
-.primary-color { }
+.my-button {
+}
+.primary-color {
+}
 ```
 
 ```js
 // v9 default - camelCase conversion
-import { myButton, primaryColor } from './styles.module.css';
+import { myButton, primaryColor } from "./styles.module.css"
 ```
 
 **Alternative: Keep kebab-case names with 'dashesOnly'**
@@ -256,7 +304,7 @@ If you have `webpack_loader` in your configuration:
 # webpack_loader: 'babel'
 
 # NEW:
-javascript_transpiler: 'babel'
+javascript_transpiler: "babel"
 ```
 
 ### Step 5: Run Tests
@@ -291,6 +339,7 @@ If you see warnings about CSS module exports, ensure you've updated all imports
 ### Build Error: exportLocalsConvention Incompatible with namedExport
 
 If you see this error:
+
 ```
 "exportLocalsConvention" with "camelCase" value is incompatible with "namedExport: true" option
 ```
@@ -312,30 +361,35 @@ If you want to use `'camelCase'` (which exports both original and camelCase vers
 If you experience unexpected peer dependency warnings after upgrading to v9, you may need to clear your package manager's cache and reinstall dependencies. This ensures the new optional peer dependency configuration takes effect properly.
 
 **For npm:**
+
 ```bash
 rm -rf node_modules package-lock.json
 npm install
 ```
 
 **For Yarn:**
+
 ```bash
 rm -rf node_modules yarn.lock
 yarn install
 ```
 
 **For pnpm:**
+
 ```bash
 rm -rf node_modules pnpm-lock.yaml
 pnpm install
 ```
 
 **For Bun:**
+
 ```bash
 rm -rf node_modules bun.lockb
 bun install
 ```
 
 **When is this necessary?**
+
 - If you see peer dependency warnings for packages you don't use (e.g., warnings about Babel when using SWC)
 - If your package manager cached the old dependency resolution from v8
 - After switching transpilers or bundlers (e.g., from Babel to SWC, or webpack to rspack)

data/lib/shakapacker/doctor.rb

@@ -442,18 +442,17 @@ module Shakapacker
 
       def check_swc_config_conflicts
         swcrc_path = root_path.join(".swcrc")
-        return unless swcrc_path.exist?
+        swc_config_path = root_path.join("config/swc.config.js")
 
-        begin
-          swcrc = JSON.parse(File.read(swcrc_path))
-          # Check for conflicting jsc.target and env settings
-          if swcrc.dig("jsc", "target") && swcrc["env"]
-            @issues << "SWC configuration conflict: .swcrc contains both 'jsc.target' and 'env' settings, which are mutually exclusive. Remove 'jsc.target' from .swcrc"
-          elsif swcrc.dig("jsc", "target")
-            @warnings << "SWC configuration: .swcrc contains 'jsc.target' which may conflict with the loader's 'env' setting. Consider removing 'jsc.target' from .swcrc to avoid build errors"
-          end
-        rescue JSON::ParserError
-          @warnings << "SWC configuration: .swcrc exists but contains invalid JSON"
+        if swcrc_path.exist?
+          @warnings << "SWC configuration: .swcrc file detected. This file completely overrides Shakapacker's default SWC settings and may cause build failures. " \
+                      "Please migrate to config/swc.config.js which properly merges with Shakapacker defaults. " \
+                      "To migrate: Move your custom settings from .swcrc to config/swc.config.js (see docs for format). " \
+                      "See: https://github.com/shakacode/shakapacker/blob/main/docs/using_swc_loader.md"
+        end
+
+        if swc_config_path.exist?
+          @info << "SWC configuration: Using config/swc.config.js (recommended). This config is merged with Shakapacker's defaults."
         end
       end
 

data/lib/shakapacker/swc_migrator.rb

@@ -8,8 +8,9 @@ module Shakapacker
   class SwcMigrator
     attr_reader :root_path, :logger
 
+    # Babel packages safe to remove when migrating to SWC
+    # Note: @babel/core and @babel/eslint-parser are excluded as they may be needed for ESLint
     BABEL_PACKAGES = [
-      "@babel/core",
       "@babel/plugin-proposal-class-properties",
       "@babel/plugin-proposal-object-rest-spread",
       "@babel/plugin-syntax-dynamic-import",
@@ -25,6 +26,12 @@ module Shakapacker
       "babel-plugin-transform-react-remove-prop-types"
     ].freeze
 
+    # Babel packages that may be needed for ESLint - only remove if user explicitly confirms
+    ESLINT_BABEL_PACKAGES = [
+      "@babel/core",
+      "@babel/eslint-parser"
+    ].freeze
+
     SWC_PACKAGES = {
       "@swc/core" => "^1.7.39",
       "swc-loader" => "^0.2.6"
@@ -39,23 +46,21 @@ module Shakapacker
       .eslintrc.json
     ].freeze
 
-    DEFAULT_SWCRC_CONFIG = {
-      "jsc" => {
-        "parser" => {
-          "syntax" => "ecmascript",
-          "jsx" => true,
-          "dynamicImport" => true
-        },
-        "transform" => {
-          "react" => {
-            "runtime" => "automatic"
+    DEFAULT_SWC_CONFIG = <<~JS.freeze
+      // config/swc.config.js
+      // This file is merged with Shakapacker's default SWC configuration
+      // See: https://swc.rs/docs/configuration/compilation
+
+      module.exports = {
+        jsc: {
+          transform: {
+            react: {
+              runtime: "automatic"
+            }
           }
         }
-      },
-      "module" => {
-        "type" => "es6"
       }
-    }.freeze
+    JS
 
     def initialize(root_path, logger: nil)
       @root_path = Pathname.new(root_path)
@@ -68,7 +73,7 @@ module Shakapacker
       results = {
         config_updated: update_shakapacker_config,
         packages_installed: install_swc_packages,
-        swcrc_created: create_swcrc,
+        swc_config_created: create_swc_config,
         babel_packages_found: find_babel_packages
       }
 
@@ -76,9 +81,9 @@ module Shakapacker
       logger.info "   Note: SWC is approximately 20x faster than Babel for transpilation."
       logger.info "   Please test your application thoroughly after migration."
       logger.info "\n📝 Configuration Info:"
-      logger.info "   - .swcrc provides base configuration for all environments"
-      logger.info "   - The SWC loader adds automatic environment targeting (via 'env' setting)"
-      logger.info "   - You can customize .swcrc, but avoid setting 'jsc.target' as it conflicts with 'env'"
+      logger.info "   - config/swc.config.js is merged with Shakapacker's default SWC configuration"
+      logger.info "   - You can customize config/swc.config.js to add additional options"
+      logger.info "   - Avoid using .swcrc as it overrides Shakapacker defaults completely"
 
       # Show cleanup recommendations if babel packages found
       if results[:babel_packages_found].any?
@@ -108,20 +113,21 @@ module Shakapacker
       package_json_path = root_path.join("package.json")
       unless package_json_path.exist?
         logger.error "❌ No package.json found"
-        return { removed_packages: [], config_files_deleted: [] }
+        return { removed_packages: [], config_files_deleted: [], preserved_packages: [] }
       end
 
       # Check if ESLint uses Babel parser
+      preserved_for_eslint = []
       if eslint_uses_babel?
-        logger.info "\n⚠️  WARNING: ESLint configuration detected that may use Babel"
-        logger.info "   If you use @babel/eslint-parser or babel-eslint, you may need to:"
-        logger.info "   1. Keep @babel/core and related Babel packages for ESLint"
-        logger.info "   2. Or switch to @typescript-eslint/parser for TypeScript files"
-        logger.info "   3. Or use espree (ESLint's default parser) for JavaScript files"
-        logger.info "\n   Proceeding with Babel package removal. Check your ESLint config after."
+        logger.info "\n⚠️  ESLint configuration detected that uses Babel parser"
+        logger.info "   Preserving @babel/core and @babel/eslint-parser for ESLint compatibility"
+        logger.info "   To switch ESLint parser:"
+        logger.info "   1. For TypeScript: use @typescript-eslint/parser"
+        logger.info "   2. For JavaScript: use espree (ESLint's default parser)"
+        preserved_for_eslint = ESLINT_BABEL_PACKAGES
       end
 
-      removed_packages = remove_babel_from_package_json(package_json_path)
+      removed_packages = remove_babel_from_package_json(package_json_path, preserve: preserved_for_eslint)
       deleted_files = delete_babel_config_files
 
       if removed_packages.any?
@@ -131,7 +137,7 @@ module Shakapacker
         logger.info "ℹ️  No Babel packages found to remove"
       end
 
-      { removed_packages: removed_packages, config_files_deleted: deleted_files }
+      { removed_packages: removed_packages, config_files_deleted: deleted_files, preserved_packages: preserved_for_eslint }
     end
 
     def find_babel_packages
@@ -144,7 +150,9 @@ module Shakapacker
         dev_dependencies = package_json["devDependencies"] || {}
         all_deps = dependencies.merge(dev_dependencies)
 
-        found_packages = BABEL_PACKAGES.select { |pkg| all_deps.key?(pkg) }
+        # Find all babel packages (including ESLint-related ones for display)
+        all_babel_packages = BABEL_PACKAGES + ESLINT_BABEL_PACKAGES
+        found_packages = all_babel_packages.select { |pkg| all_deps.key?(pkg) }
         found_packages
       rescue JSON::ParserError => e
         logger.error "Failed to parse package.json: #{e.message}"
@@ -174,6 +182,11 @@ module Shakapacker
           begin
             package_json = JSON.parse(File.read(package_json_path))
             if package_json["eslintConfig"]
+              # Check parser field explicitly
+              parser = package_json["eslintConfig"]["parser"]
+              return true if parser && parser.match?(/@babel\/eslint-parser|babel-eslint/)
+
+              # Also check entire config for babel parser references (catches nested configs)
               return true if package_json["eslintConfig"].to_json.match?(/@babel\/eslint-parser|babel-eslint/)
             end
 
@@ -254,29 +267,35 @@ module Shakapacker
         {}
       end
 
-      def create_swcrc
-        swcrc_path = root_path.join(".swcrc")
-        if swcrc_path.exist?
-          logger.info "ℹ️  .swcrc already exists"
+      def create_swc_config
+        config_dir = root_path.join("config")
+        swc_config_path = config_dir.join("swc.config.js")
+
+        if swc_config_path.exist?
+          logger.info "ℹ️  config/swc.config.js already exists"
           return false
         end
 
-        logger.info "📄 Creating .swcrc configuration..."
-        File.write(swcrc_path, JSON.pretty_generate(DEFAULT_SWCRC_CONFIG) + "\n")
-        logger.info "✅ .swcrc created"
+        FileUtils.mkdir_p(config_dir) unless config_dir.exist?
+
+        logger.info "📄 Creating config/swc.config.js..."
+        File.write(swc_config_path, DEFAULT_SWC_CONFIG)
+        logger.info "✅ config/swc.config.js created"
         true
       rescue StandardError => e
-        logger.error "Failed to create .swcrc: #{e.message}"
+        logger.error "Failed to create config/swc.config.js: #{e.message}"
         false
       end
 
-      def remove_babel_from_package_json(package_json_path)
+      def remove_babel_from_package_json(package_json_path, preserve: [])
         package_json = JSON.parse(File.read(package_json_path))
         dependencies = package_json["dependencies"] || {}
         dev_dependencies = package_json["devDependencies"] || {}
         removed_packages = []
 
         BABEL_PACKAGES.each do |package|
+          next if preserve.include?(package)
+
           if dependencies.delete(package)
             removed_packages << package
             logger.info "  - Removed #{package} from dependencies"
@@ -287,6 +306,13 @@ module Shakapacker
           end
         end
 
+        # Log preserved packages
+        preserve.each do |package|
+          if dependencies[package] || dev_dependencies[package]
+            logger.info "  - Preserved #{package} (needed for ESLint)"
+          end
+        end
+
         if removed_packages.any?
           package_json["dependencies"] = dependencies
           package_json["devDependencies"] = dev_dependencies

data/lib/shakapacker/version.rb

@@ -1,4 +1,4 @@
 module Shakapacker
   # Change the version in package.json too, please!
-  VERSION = "9.0.0.beta.9".freeze
+  VERSION = "9.0.0.beta.11".freeze
 end

data/lib/shakapacker.rb

@@ -6,7 +6,7 @@ require "active_support/tagged_logging"
 module Shakapacker
   extend self
 
-  DEFAULT_ENV = "production".freeze
+  DEFAULT_ENV = "development".freeze
 
   def instance=(instance)
     @instance = instance

data/package/env.ts

@@ -6,21 +6,32 @@ const { isFileNotFoundError } = require("./utils/errorHelpers")
 const { sanitizeEnvValue } = require("./utils/pathValidation")
 
 const NODE_ENVIRONMENTS = ["development", "production", "test"] as const
-const DEFAULT = "production"
 
 // Sanitize environment variables to prevent injection
 const initialRailsEnv = sanitizeEnvValue(process.env.RAILS_ENV)
 const rawNodeEnv = sanitizeEnvValue(process.env.NODE_ENV)
 
+// Default NODE_ENV based on RAILS_ENV to match bin/shakapacker behavior (see lib/shakapacker/runner.rb:27)
+// - RAILS_ENV=production → DEFAULT="production" (safe for production builds)
+// - RAILS_ENV=development, test, staging, or unset → DEFAULT="development" (good DX for dev server)
+// This ensures the dev server works out of the box without requiring NODE_ENV to be set explicitly
+const DEFAULT = initialRailsEnv === "production" ? "production" : "development"
+
 // Validate NODE_ENV strictly
 const nodeEnv =
-  rawNodeEnv && NODE_ENVIRONMENTS.includes(rawNodeEnv as typeof NODE_ENVIRONMENTS[number]) ? rawNodeEnv : DEFAULT
+  rawNodeEnv &&
+  NODE_ENVIRONMENTS.includes(rawNodeEnv as (typeof NODE_ENVIRONMENTS)[number])
+    ? rawNodeEnv
+    : DEFAULT
 
 // Log warning if NODE_ENV was invalid
-if (rawNodeEnv && !NODE_ENVIRONMENTS.includes(rawNodeEnv as typeof NODE_ENVIRONMENTS[number])) {
+if (
+  rawNodeEnv &&
+  !NODE_ENVIRONMENTS.includes(rawNodeEnv as (typeof NODE_ENVIRONMENTS)[number])
+) {
   console.warn(
     `[SHAKAPACKER WARNING] Invalid NODE_ENV value: ${rawNodeEnv}. ` +
-    `Valid values are: ${NODE_ENVIRONMENTS.join(', ')}. Using default: ${DEFAULT}`
+      `Valid values are: ${NODE_ENVIRONMENTS.join(", ")}. Using default: ${DEFAULT}`
   )
 }
 
@@ -42,13 +53,13 @@ try {
     } catch (defaultError) {
       throw new Error(
         `Failed to load Shakapacker configuration.\n` +
-        `Neither user config (${configPath}) nor default config (${defaultConfigPath}) could be loaded.\n\n` +
-        `To fix this issue:\n` +
-        `1. Create a config/shakapacker.yml file in your project\n` +
-        `2. Or set the SHAKAPACKER_CONFIG environment variable to point to your config file\n` +
-        `3. Or reinstall Shakapacker to restore the default configuration:\n` +
-        `   npm install shakapacker --force\n` +
-        `   yarn add shakapacker --force`
+          `Neither user config (${configPath}) nor default config (${defaultConfigPath}) could be loaded.\n\n` +
+          `To fix this issue:\n` +
+          `1. Create a config/shakapacker.yml file in your project\n` +
+          `2. Or set the SHAKAPACKER_CONFIG environment variable to point to your config file\n` +
+          `3. Or reinstall Shakapacker to restore the default configuration:\n` +
+          `   npm install shakapacker --force\n` +
+          `   yarn add shakapacker --force`
       )
     }
   } else {
@@ -61,13 +72,14 @@ const regex = new RegExp(`^(${availableEnvironments})$`, "g")
 
 const runningWebpackDevServer = process.env.WEBPACK_SERVE === "true"
 
-const validatedRailsEnv = initialRailsEnv && initialRailsEnv.match(regex) ? initialRailsEnv : DEFAULT
+const validatedRailsEnv =
+  initialRailsEnv && initialRailsEnv.match(regex) ? initialRailsEnv : DEFAULT
 
 if (initialRailsEnv && validatedRailsEnv !== initialRailsEnv) {
   /* eslint no-console:0 */
   console.warn(
     `[SHAKAPACKER WARNING] Environment '${initialRailsEnv}' not found in the configuration.\n` +
-    `Using '${DEFAULT}' configuration as a fallback.`
+      `Using '${DEFAULT}' configuration as a fallback.`
   )
 }
 

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shakapacker",
-  "version": "9.0.0-beta.9",
+  "version": "9.0.0-beta.11",
   "description": "Use webpack to manage app-like JavaScript modules in Rails",
   "homepage": "https://github.com/shakacode/shakapacker",
   "bugs": {

data/test/package/config.test.js

@@ -71,6 +71,7 @@ describe("Config", () => {
   })
 
   test("should allow enabling integrity", () => {
+    process.env.RAILS_ENV = "production"
     process.env.SHAKAPACKER_CONFIG = "config/shakapacker_integrity.yml"
     const config = require("../../package/config")
 
@@ -78,6 +79,7 @@ describe("Config", () => {
   })
 
   test("should allow configuring hash functions", () => {
+    process.env.RAILS_ENV = "production"
     process.env.SHAKAPACKER_CONFIG = "config/shakapacker_integrity.yml"
     const config = require("../../package/config")
 
@@ -89,6 +91,7 @@ describe("Config", () => {
   })
 
   test("should allow configuring crossorigin", () => {
+    process.env.RAILS_ENV = "production"
     process.env.SHAKAPACKER_CONFIG = "config/shakapacker_integrity.yml"
     const config = require("../../package/config")