shakapacker 9.3.0 → 9.3.2

data/package/rspack/index.ts

@@ -1,4 +1,3 @@
-/* eslint global-require: 0 */
 /* eslint import/no-dynamic-require: 0 */
 
 // Mixed require/import syntax:
@@ -7,9 +6,10 @@
 import { resolve } from "path"
 import { existsSync } from "fs"
 import type { RspackConfigWithDevServer } from "../environments/types"
+import type { Config } from "../types"
 
 const webpackMerge = require("webpack-merge")
-const config = require("../config")
+const config = require("../config") as Config
 const baseConfig = require("../environments/base")
 const devServer = require("../dev_server")
 const env = require("../env")
@@ -36,7 +36,7 @@ const generateRspackConfig = (
 
   const { nodeEnv } = env
   const path = resolve(__dirname, "../environments", `${nodeEnv}.js`)
-  // eslint-disable-next-line @typescript-eslint/no-require-imports
+
   const environmentConfig = existsSync(path) ? require(path) : baseConfig
 
   // Create base rspack config

data/package/rules/file.ts

@@ -1,4 +1,4 @@
-import { dirname, sep, normalize } from "path"
+import { dirname, normalize } from "path"
 
 const {
   additional_paths: additionalPaths,

data/package/rules/raw.ts

@@ -1,4 +1,6 @@
-const config = require("../config")
+import type { Config } from "../types"
+
+const config = require("../config") as Config
 
 const rspackRawConfig = () => ({
   resourceQuery: /raw/,

data/package/rules/rspack.ts

@@ -1,5 +1,3 @@
-/* eslint global-require: 0 */
-
 const { moduleExists } = require("../utils/helpers")
 const { debug, info, warn } = require("../utils/debug")
 

data/package/rules/sass.ts

@@ -1,5 +1,3 @@
-/* eslint global-require: 0 */
-
 const { getStyleRule } = require("../utils/getStyleRule")
 const { canProcess, packageMajorVersion } = require("../utils/helpers")
 const { additional_paths: extraPaths } = require("../config")

data/package/rules/webpack.ts

@@ -1,4 +1,3 @@
-/* eslint global-require: 0 */
 /* eslint import/no-dynamic-require: 0 */
 
 export = [

data/package/swc/index.ts

@@ -1,4 +1,3 @@
-/* eslint global-require: 0 */
 /* eslint import/no-dynamic-require: 0 */
 
 import { resolve } from "path"
@@ -18,7 +17,6 @@ const isTypescriptFile = (filename: string): boolean =>
 const getCustomConfig = (): Partial<RuleSetRule> => {
   const path = resolve("config", "swc.config.js")
   if (existsSync(path)) {
-    // eslint-disable-next-line @typescript-eslint/no-require-imports
     return require(path)
   }
   return {}

data/package/types.ts

@@ -15,6 +15,7 @@ export interface Config {
   source_entry_path: string
   nested_entries: boolean
   css_extract_ignore_order_warnings: boolean
+  css_modules_export_mode?: "named" | "default"
   public_root_path: string
   public_output_path: string
   private_output_path?: string
@@ -53,8 +54,6 @@ export interface Env {
 type Header =
   | Array<{ key: string; value: string }>
   | Record<string, string | string[]>
-type ServerType = "http" | "https" | "spdy"
-type WebSocketType = "sockjs" | "ws"
 
 /**
  * This has the same keys and behavior as https://webpack.js.org/configuration/dev-server/ except:
@@ -63,7 +62,7 @@ type WebSocketType = "sockjs" | "ws"
  * @see {import('webpack-dev-server').Configuration}
  */
 export interface DevServerConfig {
-  allowed_hosts?: "all" | "auto" | string | string[]
+  allowed_hosts?: string | string[]
   bonjour?: boolean | Record<string, unknown> // bonjour.BonjourOptions
   client?: Record<string, unknown> // Client
   compress?: boolean
@@ -71,7 +70,7 @@ export interface DevServerConfig {
   headers?: Header | (() => Header)
   history_api_fallback?: boolean | Record<string, unknown> // HistoryApiFallbackOptions
   hmr?: "only" | boolean
-  host?: "local-ip" | "local-ipv4" | "local-ipv6" | string
+  host?: string
   http2?: boolean
   https?: boolean | https.ServerOptions
   ipc?: boolean | string
@@ -85,23 +84,21 @@ export interface DevServerConfig {
     | string[]
     | Record<string, unknown>
     | Record<string, unknown>[]
-  port?: "auto" | string | number
+  port?: string | number
   proxy?: unknown // ProxyConfigMap | ProxyConfigArray
   setup_exit_signals?: boolean
-  static?: boolean | string | unknown // Static | Array<string | Static>
-  watch_files?: string | string[] | unknown // WatchFiles | Array<WatchFiles | string>
+  static?: unknown // Static | Array<string | Static>
+  watch_files?: unknown // WatchFiles | Array<WatchFiles | string>
   web_socket_server?:
     | string
     | boolean
-    | WebSocketType
     | {
-        type?: string | boolean | WebSocketType
+        type?: string | boolean
         options?: Record<string, unknown>
       }
   server?:
     | string
     | boolean
-    | ServerType
-    | { type?: string | boolean | ServerType; options?: https.ServerOptions }
+    | { type?: string | boolean; options?: https.ServerOptions }
   [otherWebpackDevServerConfigKey: string]: unknown
 }

data/package/utils/debug.ts

@@ -18,24 +18,20 @@ const isDebugMode = (): boolean => {
 
 const debug = (message: string, ...args: unknown[]): void => {
   if (isDebugMode()) {
-    // eslint-disable-next-line no-console
     console.log(`[Shakapacker] ${message}`, ...args)
   }
 }
 
 const warn = (message: string, ...args: unknown[]): void => {
-  // eslint-disable-next-line no-console
   console.warn(`[Shakapacker] WARNING: ${message}`, ...args)
 }
 
 const error = (message: string, ...args: unknown[]): void => {
-  // eslint-disable-next-line no-console
   console.error(`[Shakapacker] ERROR: ${message}`, ...args)
 }
 
 const info = (message: string, ...args: unknown[]): void => {
   if (isDebugMode()) {
-    // eslint-disable-next-line no-console
     console.info(`[Shakapacker] INFO: ${message}`, ...args)
   }
 }

data/package/utils/getStyleRule.ts

@@ -1,18 +1,19 @@
-/* eslint global-require: 0 */
+import type { Config } from "../types"
+
 const { canProcess, moduleExists } = require("./helpers")
 const { requireOrError } = require("./requireOrError")
-const config = require("../config")
+const config = require("../config") as Config
 const inliningCss = require("./inliningCss")
 
 interface StyleRule {
   test: RegExp
-  use: any[]
+  use: unknown[]
   type?: string
 }
 
 const getStyleRule = (
   test: RegExp,
-  preprocessors: any[] = []
+  preprocessors: unknown[] = []
 ): StyleRule | null => {
   if (moduleExists("css-loader")) {
     const tryPostcss = () =>
@@ -28,6 +29,11 @@ const getStyleRule = (
         ? requireOrError("@rspack/core").CssExtractRspackPlugin.loader
         : requireOrError("mini-css-extract-plugin").loader
 
+    // Determine CSS Modules export mode based on configuration
+    // 'named' (default): Use named exports with camelCaseOnly (v9 behavior)
+    // 'default': Use default exports with camelCase (v8 behavior)
+    const useNamedExports = config.css_modules_export_mode !== "default"
+
     const use = [
       inliningCss ? "style-loader" : extractionPlugin,
       {
@@ -37,11 +43,13 @@ const getStyleRule = (
           importLoaders: 2,
           modules: {
             auto: true,
-            // v9 defaults: Use named exports with camelCase conversion
-            // Note: css-loader requires 'camelCaseOnly' or 'dashesOnly' when namedExport is true
-            // Using 'camelCase' with namedExport: true causes a build error
-            namedExport: true,
-            exportLocalsConvention: "camelCaseOnly"
+            // Use named exports for v9 (default), or default exports for v8 compatibility
+            namedExport: useNamedExports,
+            // 'camelCaseOnly' with namedExport: true (v9 default)
+            // 'camelCase' with namedExport: false (v8 behavior - exports both original and camelCase)
+            exportLocalsConvention: useNamedExports
+              ? "camelCaseOnly"
+              : "camelCase"
           }
         }
       },

data/package/utils/helpers.ts

@@ -38,6 +38,11 @@ const loaderMatches = <T = unknown>(
   loaderToCheck: string,
   fn: () => T
 ): T | null => {
+  // If transpiler is set to 'none', skip all transpiler rules (for custom webpack configs)
+  if (configLoader === "none") {
+    return null
+  }
+
   if (configLoader !== loaderToCheck) {
     return null
   }
@@ -60,13 +65,13 @@ const loaderMatches = <T = unknown>(
 const packageFullVersion = (packageName: string): string => {
   try {
     const packageJsonPath = require.resolve(`${packageName}/package.json`)
-    // eslint-disable-next-line import/no-dynamic-require, global-require
+    // eslint-disable-next-line import/no-dynamic-require
     const packageJson = require(packageJsonPath) as { version: string }
     return packageJson.version
-  } catch (error: any) {
+  } catch (error: unknown) {
     // Re-throw the error with proper code to maintain compatibility with babel preset
     // The preset expects MODULE_NOT_FOUND errors to handle missing core-js gracefully
-    if (error.code === "MODULE_NOT_FOUND") {
+    if ((error as NodeJS.ErrnoException).code === "MODULE_NOT_FOUND") {
       throw error
     }
     // For other errors, warn and re-throw

data/package/utils/pathValidation.ts

@@ -138,17 +138,16 @@ export function validatePaths(paths: string[], basePath: string): string[] {
       console.warn(
         `[SHAKAPACKER WARNING] Skipping potentially unsafe path: ${userPath}`
       )
-      continue
-    }
-
-    try {
-      const safePath = safeResolvePath(basePath, userPath)
-      validatedPaths.push(safePath)
-    } catch (error) {
-      console.warn(
-        `[SHAKAPACKER WARNING] Invalid path configuration: ${userPath}\n` +
-          `Error: ${error instanceof Error ? error.message : String(error)}`
-      )
+    } else {
+      try {
+        const safePath = safeResolvePath(basePath, userPath)
+        validatedPaths.push(safePath)
+      } catch (error) {
+        console.warn(
+          `[SHAKAPACKER WARNING] Invalid path configuration: ${userPath}\n` +
+            `Error: ${error instanceof Error ? error.message : String(error)}`
+        )
+      }
     }
   }
 

data/package/utils/requireOrError.ts

@@ -1,12 +1,13 @@
-/* eslint global-require: 0 */
 /* eslint import/no-dynamic-require: 0 */
-const config = require("../config")
+import type { Config } from "../types"
+
+const config = require("../config") as Config
 
 interface ErrorWithCause extends Error {
   cause?: unknown
 }
 
-const requireOrError = (moduleName: string): any => {
+const requireOrError = (moduleName: string): unknown => {
   try {
     return require(moduleName)
   } catch (originalError: unknown) {

data/package/webpack-types.d.ts

@@ -1,4 +1,4 @@
-// @ts-expect-error: webpack is an optional peer dependency (using type-only import)
+// @ts-ignore: webpack is an optional peer dependency (using type-only import)
 import type { Configuration, RuleSetRule, RuleSetUseItem } from "webpack"
 
 export interface ShakapackerWebpackConfig extends Configuration {

data/package/webpackDevServerConfig.ts

@@ -25,11 +25,11 @@ interface WebpackDevServerConfig {
     [key: string]: unknown
   }
   client?: Record<string, unknown>
-  allowedHosts?: "all" | "auto" | string | string[]
+  allowedHosts?: string | string[]
   bonjour?: boolean | Record<string, unknown>
   compress?: boolean
   headers?: Record<string, unknown> | (() => Record<string, unknown>)
-  host?: "local-ip" | "local-ipv4" | "local-ipv6" | string
+  host?: string
   http2?: boolean
   https?: boolean | Record<string, unknown>
   ipc?: boolean | string
@@ -42,12 +42,12 @@ interface WebpackDevServerConfig {
     | string[]
     | Record<string, unknown>
     | Record<string, unknown>[]
-  port?: "auto" | string | number
+  port?: string | number
   proxy?: unknown
   server?: string | boolean | Record<string, unknown>
   setupExitSignals?: boolean
   setupMiddlewares?: (middlewares: unknown[], devServer: unknown) => unknown[]
-  watchFiles?: string | string[] | unknown
+  watchFiles?: unknown
   webSocketServer?: string | boolean | Record<string, unknown>
   [key: string]: unknown
 }

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shakapacker",
-  "version": "9.3.0",
+  "version": "9.3.2",
   "description": "Use webpack to manage app-like JavaScript modules in Rails",
   "homepage": "https://github.com/shakacode/shakapacker",
   "bugs": {
@@ -82,7 +82,7 @@
     "lint-staged": "^15.2.10",
     "memory-fs": "^0.5.0",
     "mini-css-extract-plugin": "^2.9.4",
-    "prettier": "^3.2.5",
+    "prettier": "^3.6.2",
     "rspack-manifest-plugin": "^5.0.3",
     "sass-loader": "^16.0.5",
     "swc-loader": "^0.1.15",
@@ -108,7 +108,7 @@
     "babel-loader": "^8.2.4 || ^9.0.0 || ^10.0.0",
     "compression-webpack-plugin": "^9.0.0 || ^10.0.0 || ^11.0.0",
     "css-loader": "^6.8.1 || ^7.0.0",
-    "esbuild": "^0.14.0 || ^0.15.0 || ^0.16.0 || ^0.17.0 || ^0.18.0 || ^0.19.0 || ^0.20.0 || ^0.21.0 || ^0.22.0 || ^0.23.0 || ^0.24.0 || ^0.25.0",
+    "esbuild": "^0.14.0 || ^0.15.0 || ^0.16.0 || ^0.17.0 || ^0.18.0 || ^0.19.0 || ^0.20.0 || ^0.21.0 || ^0.22.0 || ^0.23.0 || ^0.24.0 || ^0.25.0 || ^0.26.0 || ^0.27.0",
     "esbuild-loader": "^2.0.0 || ^3.0.0 || ^4.0.0",
     "mini-css-extract-plugin": "^2.0.0",
     "rspack-manifest-plugin": "^5.0.0",

data/test/package/transpiler-defaults.test.js

@@ -124,4 +124,46 @@ describe("JavaScript Transpiler Defaults", () => {
       expect(config.javascript_transpiler).toBe("swc")
     })
   })
+
+  describe("none transpiler option", () => {
+    it("accepts 'none' as a valid transpiler value", () => {
+      process.env.SHAKAPACKER_JAVASCRIPT_TRANSPILER = "none"
+
+      jest.resetModules()
+      const config = require("../../package/config")
+
+      expect(config.javascript_transpiler).toBe("none")
+    })
+
+    it("'none' option doesn't throw errors for missing transpiler packages", () => {
+      process.env.SHAKAPACKER_JAVASCRIPT_TRANSPILER = "none"
+
+      jest.resetModules()
+
+      // Should not throw even though there's no 'none-loader' package
+      expect(() => {
+        require("../../package/config")
+      }).not.toThrow()
+    })
+
+    it("'none' option results in no transpiler rules being applied", () => {
+      process.env.SHAKAPACKER_JAVASCRIPT_TRANSPILER = "none"
+      delete process.env.SHAKAPACKER_ASSETS_BUNDLER
+
+      jest.resetModules()
+
+      // Load the webpack rules - should not include babel, swc, or esbuild rules
+      const rules = require("../../package/rules/webpack")
+
+      // Verify no transpiler rules are present by checking rules array
+      // When transpiler is 'none', loaderMatches returns null for all transpilers
+      // which means babel.ts, swc.ts, and esbuild.ts all export null
+      // These get filtered out, so rules array should not contain them
+      const rulesJson = JSON.stringify(rules)
+
+      expect(rulesJson).not.toContain("babel-loader")
+      expect(rulesJson).not.toContain("swc-loader")
+      expect(rulesJson).not.toContain("esbuild-loader")
+    })
+  })
 })

data/yarn.lock

@@ -5440,7 +5440,7 @@ prelude-ls@^1.2.1:
   resolved "https://registry.npmjs.org/prelude-ls/-/prelude-ls-1.2.1.tgz#debc6489d7a6e6b0e7611888cec880337d316396"
   integrity sha512-vkcDPrRZo1QZLbn5RLGPpg/WmIQ65qoWWhcGKf/b5eplkkarX0m9z8ppCat4mlOqUsWpyNuYgO3VRyrYHSzX5g==
 
-prettier@^3.2.5:
+prettier@^3.6.2:
   version "3.6.2"
   resolved "https://registry.npmjs.org/prettier/-/prettier-3.6.2.tgz#ccda02a1003ebbb2bfda6f83a074978f608b9393"
   integrity sha512-I7AIg5boAr5R0FFtJ6rCfD+LFsWHp81dolrFD8S79U9tb8Az2nGrJncnMSnys+bpQJfRUzqs9hnA81OAA3hCuQ==

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shakapacker
 version: !ruby/object:Gem::Version
-  version: 9.3.0
+  version: 9.3.2
 platform: ruby
 authors:
 - David Heinemeier Hansson
@@ -131,11 +131,13 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".claude/commands/update-changelog.md"
 - ".github/FUNDING.yml"
 - ".github/ISSUE_TEMPLATE/bug_report.md"
 - ".github/ISSUE_TEMPLATE/feature_request.md"
 - ".github/PULL_REQUEST_TEMPLATE.md"
 - ".github/STATUS.md"
+- ".github/actionlint-matcher.json"
 - ".github/workflows/claude-code-review.yml"
 - ".github/workflows/claude.yml"
 - ".github/workflows/dummy.yml"
@@ -171,6 +173,7 @@ files:
 - conductor.json
 - config/README.md
 - config/shakapacker.yml
+- docs/api-reference.md
 - docs/cdn_setup.md
 - docs/common-upgrades.md
 - docs/configuration.md
@@ -386,7 +389,7 @@ homepage: https://github.com/shakacode/shakapacker
 licenses:
 - MIT
 metadata:
-  source_code_uri: https://github.com/shakacode/shakapacker/tree/v9.3.0
+  source_code_uri: https://github.com/shakacode/shakapacker/tree/v9.3.2
 rdoc_options: []
 require_paths:
 - lib