shakapacker 9.0.0.beta.5 → 9.0.0.beta.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c4e7529c9a1e2dee69667d688581f6c684c775c18a17ac527a3328876d45c523
-  data.tar.gz: 305d71e3efd30913c856066b7195ff259094bafb2cc2f7fa7f1730cdbfc49cdf
+  metadata.gz: 65a49f86249eeb2035e56d7f21da2d8e4edbe5a26da50b0a63a9534a720a84cf
+  data.tar.gz: 153cb1df867355c881a3a7fe31b3aed526355ff30f7eaaacb497d231cb5f3bcc
 SHA512:
-  metadata.gz: cdba0281b3a0bc4e43d50ac4e84fa1a9b029ff6999afeabea56eada8ffded1c366e4b084734758566f009c1214ac36d7da874ea4887c70010551330f678bccbc
-  data.tar.gz: 97a96fa177348c19929aae4fe53865f9f738f21c060e77863dfd79c3cc22869c4a6eb78af34d2139eb842baf61f59b3a9040b1e42cec3ed2fa216d1818124ca1
+  metadata.gz: c237ee4221a14f19ae86067632ad00baaf1f05d3464c21804b8aa3fa7029429bc8d678d630f1762f940ea780f4ff81023c26d26342aa27380a461f1485f50b3d
+  data.tar.gz: b37633ed02a416c423bfb410b189b1ca7c508a162339e9def8401622c2ac198b4649ad0392795a9eb3301d6fc3d9858c2e39f1130b8c6bbd219887566126b91e

data/.eslintignore

@@ -2,3 +2,4 @@ lib/*
 node_modules/*
 vendor/*
 spec/*
+package/*

data/.github/workflows/claude-code-review.yml

@@ -50,5 +50,5 @@ jobs:
           
           # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
           # or https://docs.claude.com/en/docs/claude-code/sdk#command-line for available options
-          claude_args: '--allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'
+          claude_args: '--model claude-sonnet-4-5-20250929 --allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'
 

data/.github/workflows/test-bundlers.yml

@@ -27,8 +27,8 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: '20'
-          cache: 'npm'
-          cache-dependency-path: spec/dummy/package-lock.json
+          cache: 'yarn'
+          cache-dependency-path: spec/dummy/yarn.lock
 
       - name: Install yalc
         run: npm install -g yalc
@@ -47,7 +47,7 @@ jobs:
       - name: Install dependencies
         run: |
           yalc link shakapacker
-          npm ci
+          yarn install
 
       - name: Switch to Webpack
         run: bin/test-bundler webpack
@@ -80,8 +80,8 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: '20'
-          cache: 'npm'
-          cache-dependency-path: spec/dummy/package-lock.json
+          cache: 'yarn'
+          cache-dependency-path: spec/dummy/yarn.lock
 
       - name: Install yalc
         run: npm install -g yalc
@@ -100,7 +100,7 @@ jobs:
       - name: Install dependencies
         run: |
           yalc link shakapacker
-          npm ci
+          yarn install
 
       - name: Switch to RSpack
         run: bin/test-bundler rspack
@@ -133,8 +133,8 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: '20'
-          cache: 'npm'
-          cache-dependency-path: spec/dummy/package-lock.json
+          cache: 'yarn'
+          cache-dependency-path: spec/dummy/yarn.lock
 
       - name: Install yalc
         run: npm install -g yalc
@@ -153,7 +153,7 @@ jobs:
       - name: Install dependencies
         run: |
           yalc link shakapacker
-          npm ci
+          yarn install
 
       - name: Test switching between bundlers
         run: |

data/CONTRIBUTING.md

@@ -50,6 +50,34 @@ We welcome pull requests that fix bugs, add new features, or improve existing on
    yarn install
    ```
 
+## Understanding Optional Peer Dependencies
+
+Shakapacker uses optional peer dependencies (via `peerDependenciesMeta`) for maximum flexibility:
+
+- **All peer dependencies are optional** - Users only install what they need
+- **No installation warnings** - Package managers won't warn about missing optional dependencies
+- **Version constraints still apply** - When a package is installed, version compatibility is enforced
+
+### When modifying dependencies:
+1. Add new peer dependencies to both `peerDependencies` and `peerDependenciesMeta` (marking as optional)
+2. Keep version ranges synchronized between `devDependencies` and `peerDependencies`
+3. Test with multiple package managers: `npm`, `yarn`, and `pnpm`
+
+### Testing peer dependency changes:
+```bash
+# Test with npm (no warnings expected)
+cd /tmp && mkdir test-npm && cd test-npm
+npm init -y && npm install /path/to/shakapacker
+
+# Test with yarn (no warnings expected)
+cd /tmp && mkdir test-yarn && cd test-yarn
+yarn init -y && yarn add /path/to/shakapacker
+
+# Test with pnpm (no warnings expected)
+cd /tmp && mkdir test-pnpm && cd test-pnpm
+pnpm init && pnpm add /path/to/shakapacker
+```
+
 ## Making sure your changes pass all tests
 
 There are several specs, covering different aspects of Shakapacker gem. You may run them locally or rely on GitHub CI actions configured to test the gem functionality if different Ruby, Rails, and Node environment.
@@ -125,3 +153,39 @@ To ensure that your installer works as expected, either you can run `bundle exec
 3. Run `bundle exec rails shakapacker:install` to confirm that you got the right changes.
 
  **Note:** Ensure that you use bundle exec otherwise the installed shakapacker gem will run and not the one you are working on.
+
+## CI Workflows
+
+Shakapacker uses GitHub Actions for continuous integration. The CI workflows use **Yarn** as the package manager for consistency and reliability.
+
+### Package Manager Choice
+
+The project uses Yarn in CI workflows for the following reasons:
+- Deterministic dependency resolution with `yarn.lock`
+- Faster installation with offline mirror support
+- Better workspace support for monorepo-style testing
+- Consistent behavior across different Node.js versions
+
+### Key CI Workflow Files
+
+- `.github/workflows/test-bundlers.yml` - Tests webpack, rspack, and bundler switching
+- `.github/workflows/ruby.yml` - Ruby test suite across Ruby/Rails versions
+- `.github/workflows/node.yml` - Node.js test suite across Node versions
+- `.github/workflows/generator.yml` - Generator installation tests
+
+All workflows use:
+```yaml
+- uses: actions/setup-node@v4
+  with:
+    cache: 'yarn'
+    cache-dependency-path: spec/dummy/yarn.lock
+```
+
+And install dependencies with:
+```bash
+yarn install
+```
+
+### Testing with Other Package Managers
+
+While CI uses Yarn, the gem supports all major package managers (npm, yarn, pnpm, bun). Generator specs test against all package managers to ensure compatibility.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    shakapacker (9.0.0.beta.5)
+    shakapacker (9.0.0.beta.6)
       activesupport (>= 5.2)
       package_json
       rack-proxy (>= 0.6.1)

data/README.md

@@ -199,6 +199,83 @@ Note, in v6+, most JS packages are peer dependencies. Thus, the installer will a
 Previously, these "webpack" and "babel" packages were direct dependencies for `shakapacker`. By
 making these peer dependencies, you have control over the versions used in your webpack and babel configs.
 
+### Optional Peer Dependencies
+
+All peer dependencies in Shakapacker are marked as optional via `peerDependenciesMeta`. This design decision ensures:
+- **No warnings during package installation** when dependencies are not needed
+- **Clear visibility of supported package versions** for upgrades
+- **Flexibility to choose only the tools you need** (webpack vs rspack, babel vs swc vs esbuild)
+
+The optional peer dependencies approach means you only install what you actually use, while still maintaining
+version compatibility constraints when you do install those packages.
+
+#### Required Dependencies by Configuration
+
+Depending on your setup, you'll need different subsets of the optional peer dependencies:
+
+**For Webpack + Babel (traditional setup):**
+```json
+{
+  "dependencies": {
+    "shakapacker": "^9.0.0",
+    "@babel/core": "^7.17.9",
+    "@babel/plugin-transform-runtime": "^7.17.0",
+    "@babel/preset-env": "^7.16.11",
+    "@babel/runtime": "^7.17.9",
+    "babel-loader": "^8.2.4",
+    "compression-webpack-plugin": "^9.0.0",
+    "terser-webpack-plugin": "^5.3.1",
+    "webpack": "^5.76.0",
+    "webpack-assets-manifest": "^5.0.6",
+    "webpack-cli": "^5.0.0",
+    "webpack-dev-server": "^5.0.0"
+  }
+}
+```
+
+**For Webpack + SWC (faster alternative):**
+```json
+{
+  "dependencies": {
+    "shakapacker": "^9.0.0",
+    "@swc/core": "^1.3.0",
+    "swc-loader": "^0.2.0",
+    "compression-webpack-plugin": "^9.0.0",
+    "terser-webpack-plugin": "^5.3.1",
+    "webpack": "^5.76.0",
+    "webpack-assets-manifest": "^5.0.6",
+    "webpack-cli": "^5.0.0",
+    "webpack-dev-server": "^5.0.0"
+  }
+}
+```
+
+**For Rspack + SWC (10x faster bundling):**
+```json
+{
+  "dependencies": {
+    "shakapacker": "^9.0.0",
+    "@rspack/core": "^1.0.0",
+    "@rspack/cli": "^1.0.0",
+    "@swc/core": "^1.3.0",
+    "swc-loader": "^0.2.0",
+    "rspack-manifest-plugin": "^5.0.0"
+  }
+}
+```
+
+**For CSS/Sass processing (add to any config above):**
+```json
+{
+  "dependencies": {
+    "css-loader": "^6.8.1",
+    "mini-css-extract-plugin": "^2.0.0",
+    "sass": "^1.50.0",
+    "sass-loader": "^13.0.0"
+  }
+}
+```
+
 ## Concepts
 
 At its core, Shakapacker's essential function is to:

data/docs/optional-peer-dependencies.md

@@ -0,0 +1,198 @@
+# Optional Peer Dependencies in Shakapacker
+
+## Overview
+
+As of Shakapacker v9, all peer dependencies are marked as optional via `peerDependenciesMeta`. This design provides maximum flexibility while maintaining clear version constraints.
+
+## Key Benefits
+
+1. **No Installation Warnings** - Package managers (npm, yarn, pnpm) won't warn about missing peer dependencies
+2. **Install Only What You Need** - Users only install packages for their chosen configuration
+3. **Clear Version Constraints** - When packages are installed, version compatibility is still enforced
+4. **Smaller Node Modules** - Reduced disk usage by not installing unnecessary packages
+
+## Implementation Details
+
+### Package.json Structure
+
+```json
+{
+  "dependencies": {
+    "js-yaml": "^4.1.0",
+    "path-complete-extname": "^1.0.0",
+    "webpack-merge": "^5.8.0"  // Direct dependency - always available
+  },
+  "peerDependencies": {
+    "webpack": "^5.76.0",
+    "@rspack/core": "^1.0.0",
+    // ... all build tools
+  },
+  "peerDependenciesMeta": {
+    "webpack": { "optional": true },
+    "@rspack/core": { "optional": true },
+    // ... all marked as optional
+  }
+}
+```
+
+### TypeScript Type-Only Imports
+
+To prevent runtime errors when optional packages aren't installed, all webpack imports use type-only syntax:
+
+```typescript
+// @ts-ignore: webpack is an optional peer dependency (using type-only import)
+import type { Configuration } from "webpack"
+```
+
+Type-only imports are erased during compilation and don't trigger module resolution at runtime.
+
+## Configuration Examples
+
+### Webpack + Babel (Traditional)
+```json
+{
+  "dependencies": {
+    "shakapacker": "^9.0.0",
+    "webpack": "^5.76.0",
+    "webpack-cli": "^5.0.0",
+    "babel-loader": "^8.2.4",
+    "@babel/core": "^7.17.9",
+    "@babel/preset-env": "^7.16.11"
+  }
+}
+```
+
+### Webpack + SWC (20x Faster)
+```json
+{
+  "dependencies": {
+    "shakapacker": "^9.0.0",
+    "webpack": "^5.76.0",
+    "webpack-cli": "^5.0.0",
+    "@swc/core": "^1.3.0",
+    "swc-loader": "^0.2.0"
+  }
+}
+```
+
+### Rspack + SWC (10x Faster Bundling)
+```json
+{
+  "dependencies": {
+    "shakapacker": "^9.0.0",
+    "@rspack/core": "^1.0.0",
+    "@rspack/cli": "^1.0.0",
+    "rspack-manifest-plugin": "^5.0.0"
+  }
+}
+```
+
+## Migration Guide
+
+### From v8 to v9
+
+If upgrading from Shakapacker v8:
+
+1. **No action required** - Your existing dependencies will continue to work
+2. **No more warnings** - Peer dependency warnings will disappear after upgrading
+3. **Option to optimize** - You can now remove unused dependencies (e.g., remove Babel if using SWC)
+
+### New Installations
+
+The installer (`rails shakapacker:install`) only adds packages needed for your configuration:
+- Detects your preferred bundler (webpack/rspack)
+- Installs appropriate JavaScript transpiler (babel/swc/esbuild)
+- Adds only required dependencies
+
+## Version Constraints
+
+Version ranges are carefully chosen for compatibility:
+
+- **Broader ranges for peer deps** - Allows flexibility (e.g., `^5.76.0` for webpack)
+- **Specific versions in devDeps** - Ensures testing against known versions
+- **Forward compatibility** - Ranges include future minor versions (e.g., `^5.0.0 || ^6.0.0`)
+
+## Testing
+
+### Installation Tests
+
+Test that no warnings appear during installation:
+
+```bash
+# Test script available at test/peer-dependencies.sh
+./test/peer-dependencies.sh
+```
+
+### Runtime Tests
+
+Verify Shakapacker loads without optional dependencies:
+
+```javascript
+// This works even without webpack installed (when using rspack)
+const shakapacker = require('shakapacker');
+```
+
+### CI Integration
+
+The test suite includes:
+- `spec/shakapacker/optional_dependencies_spec.rb` - Package.json structure validation
+- `spec/shakapacker/doctor_optional_peer_spec.rb` - Doctor command validation
+- `test/peer-dependencies.sh` - Installation warning tests
+
+## Troubleshooting
+
+### Still seeing peer dependency warnings?
+
+1. Ensure you're using Shakapacker v9.0.0 or later
+2. Clear your package manager cache:
+   - npm: `npm cache clean --force`
+   - yarn: `yarn cache clean`
+   - pnpm: `pnpm store prune`
+3. Reinstall dependencies
+
+### Module not found errors?
+
+1. Check you've installed required dependencies for your configuration
+2. Refer to the configuration examples above
+3. Run `rails shakapacker:doctor` for diagnostics
+
+### TypeScript errors?
+
+The `@ts-ignore` comments are intentional and necessary for optional dependencies.
+They prevent TypeScript errors when optional packages aren't installed.
+
+## Contributing
+
+When adding new dependencies:
+
+1. Add to `peerDependencies` with appropriate version range
+2. Mark as optional in `peerDependenciesMeta`
+3. Use type-only imports in TypeScript: `import type { ... }`
+4. Test with all package managers (npm, yarn, pnpm)
+5. Update this documentation if needed
+
+## Design Rationale
+
+This approach balances several concerns:
+
+1. **User Experience** - No confusing warnings during installation
+2. **Flexibility** - Support multiple configurations without forcing unnecessary installs
+3. **Compatibility** - Maintain version constraints for safety
+4. **Performance** - Reduce installation time and disk usage
+5. **Type Safety** - TypeScript support without runtime dependencies
+
+## Future Improvements
+
+Potential enhancements for future versions:
+
+1. **Conditional exports** - Use package.json exports field for better tree-shaking
+2. **Dynamic imports** - Load bundler-specific code only when needed
+3. **Doctor updates** - Enhance doctor command to better understand optional dependencies
+4. **Automated testing** - Add CI jobs testing each configuration combination
+
+## References
+
+- [npm: peerDependenciesMeta](https://docs.npmjs.com/cli/v8/configuring-npm/package-json#peerdependenciesmeta)
+- [TypeScript: Type-Only Imports](https://www.typescriptlang.org/docs/handbook/modules.html#type-only-imports-and-exports)
+- [Shakapacker Issue #565](https://github.com/shakacode/shakapacker/issues/565)
+- [Pull Request #615](https://github.com/shakacode/shakapacker/pull/615)

data/docs/v9_upgrade.md

@@ -130,6 +130,35 @@ npm install esbuild esbuild-loader
 assets_bundler: 'rspack'  # or 'webpack' (default)
 ```
 
+### 5. All Peer Dependencies Now Optional
+
+**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": {
+    "shakapacker": "^9.0.0",
+    "@swc/core": "^1.3.0",
+    "swc-loader": "^0.2.0",
+    "webpack": "^5.76.0",
+    "webpack-cli": "^5.0.0",
+    "webpack-dev-server": "^5.0.0"
+  }
+}
+```
+You won't get warnings about missing Babel, Rspack, or esbuild packages.
+
 ## Migration Steps
 
 ### Step 1: Update Dependencies
@@ -224,6 +253,41 @@ Update your global type definitions as shown in Step 2.
 
 If you see warnings about CSS module exports, ensure you've updated all imports to use named exports or have properly configured the override.
 
+### Unexpected Peer Dependency Warnings After Upgrade
+
+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)
+
+**Note:** This is typically only needed once after the v8 → v9 upgrade. Subsequent installs will use the correct dependency resolution.
+
 ## Need Help?
 
 - See [CSS Modules Export Mode documentation](./css-modules-export-mode.md) for detailed configuration options
@@ -238,4 +302,4 @@ If you have a large codebase and need to migrate gradually:
 2. Migrate files incrementally
 3. Remove the override once migration is complete
 
-This allows you to upgrade to v9 immediately while taking time to update your CSS module imports.
+This allows you to upgrade to v9 immediately while taking time to update your CSS module imports.

data/lib/shakapacker/doctor.rb

@@ -211,8 +211,7 @@ module Shakapacker
       def check_webpack_peer_deps(deps)
         essential_webpack = {
           "webpack" => "^5.76.0",
-          "webpack-cli" => "^4.9.2 || ^5.0.0",
-          "webpack-merge" => "^5.8.0 || ^6.0.0"
+          "webpack-cli" => "^4.9.2 || ^5.0.0"
         }
 
         essential_webpack.each do |package, version|

data/lib/shakapacker/version.rb

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

data/package/environments/base.js

@@ -1,116 +1,103 @@
+"use strict";
 /* eslint global-require: 0 */
 /* eslint import/no-dynamic-require: 0 */
-const { basename, dirname, join, relative, resolve } = require("path")
-const { existsSync, readdirSync } = require("fs")
-const extname = require("path-complete-extname")
-const config = require("../config")
-const { isProduction } = require("../env")
-
-const pluginsPath = resolve(
-  __dirname,
-  "..",
-  "plugins",
-  `${config.assets_bundler}.js`
-)
-const { getPlugins } = require(pluginsPath)
-const rulesPath = resolve(
-  __dirname,
-  "..",
-  "rules",
-  `${config.assets_bundler}.js`
-)
-const rules = require(rulesPath)
-
+const { basename, dirname, join, relative, resolve } = require("path");
+const { existsSync, readdirSync } = require("fs");
+const extname = require("path-complete-extname");
+const config = require("../config");
+const { isProduction } = require("../env");
+const pluginsPath = resolve(__dirname, "..", "plugins", `${config.assets_bundler}.js`);
+const { getPlugins } = require(pluginsPath);
+const rulesPath = resolve(__dirname, "..", "rules", `${config.assets_bundler}.js`);
+const rules = require(rulesPath);
 // Don't use contentHash except for production for performance
 // https://webpack.js.org/guides/build-performance/#avoid-production-specific-tooling
-const hash = isProduction || config.useContentHash ? "-[contenthash]" : ""
+const hash = isProduction || config.useContentHash ? "-[contenthash]" : "";
 const getFilesInDirectory = (dir, includeNested) => {
-  if (!existsSync(dir)) {
-    return []
-  }
-  return readdirSync(dir, { withFileTypes: true }).flatMap((dirent) => {
-    const filePath = join(dir, dirent.name)
-    if (dirent.isDirectory() && includeNested) {
-      return getFilesInDirectory(filePath, includeNested)
+    if (!existsSync(dir)) {
+        return [];
     }
-    if (dirent.isFile()) {
-      return filePath
-    }
-    return []
-  })
-}
+    return readdirSync(dir, { withFileTypes: true }).flatMap((dirent) => {
+        const filePath = join(dir, dirent.name);
+        if (dirent.isDirectory() && includeNested) {
+            return getFilesInDirectory(filePath, includeNested);
+        }
+        if (dirent.isFile()) {
+            return filePath;
+        }
+        return [];
+    });
+};
 const getEntryObject = () => {
-  const entries = {}
-  const rootPath = join(config.source_path, config.source_entry_path)
-  if (config.source_entry_path === "/" && config.nested_entries) {
-    throw new Error(
-      `Invalid Shakapacker configuration detected!\n\n` +
-        `You have set source_entry_path to '/' with nested_entries enabled.\n` +
-        `This would create webpack entry points for EVERY file in your source directory,\n` +
-        `which would severely impact build performance.\n\n` +
-        `To fix this issue, either:\n` +
-        `1. Set 'nested_entries: false' in your shakapacker.yml\n` +
-        `2. Change 'source_entry_path' to a specific subdirectory (e.g., 'packs')\n` +
-        `3. Or use both options for better organization of your entry points`
-    )
-  }
-  getFilesInDirectory(rootPath, config.nested_entries).forEach((path) => {
-    const namespace = relative(join(rootPath), dirname(path))
-    const name = join(namespace, basename(path, extname(path)))
-    const assetPath = resolve(path)
-    // Allows for multiple filetypes per entry (https://webpack.js.org/guides/entry-advanced/)
-    // Transforms the config object value to an array with all values under the same name
-    const previousPaths = entries[name]
-    if (previousPaths) {
-      const pathArray = Array.isArray(previousPaths)
-        ? previousPaths
-        : [previousPaths]
-      pathArray.push(assetPath)
-      entries[name] = pathArray
-    } else {
-      entries[name] = assetPath
+    const entries = {};
+    const rootPath = join(config.source_path, config.source_entry_path);
+    if (config.source_entry_path === "/" && config.nested_entries) {
+        throw new Error(`Invalid Shakapacker configuration detected!\n\n` +
+            `You have set source_entry_path to '/' with nested_entries enabled.\n` +
+            `This would create webpack entry points for EVERY file in your source directory,\n` +
+            `which would severely impact build performance.\n\n` +
+            `To fix this issue, either:\n` +
+            `1. Set 'nested_entries: false' in your shakapacker.yml\n` +
+            `2. Change 'source_entry_path' to a specific subdirectory (e.g., 'packs')\n` +
+            `3. Or use both options for better organization of your entry points`);
     }
-  })
-  return entries
-}
+    getFilesInDirectory(rootPath, config.nested_entries).forEach((path) => {
+        const namespace = relative(join(rootPath), dirname(path));
+        const name = join(namespace, basename(path, extname(path)));
+        const assetPath = resolve(path);
+        // Allows for multiple filetypes per entry (https://webpack.js.org/guides/entry-advanced/)
+        // Transforms the config object value to an array with all values under the same name
+        const previousPaths = entries[name];
+        if (previousPaths) {
+            const pathArray = Array.isArray(previousPaths)
+                ? previousPaths
+                : [previousPaths];
+            pathArray.push(assetPath);
+            entries[name] = pathArray;
+        }
+        else {
+            entries[name] = assetPath;
+        }
+    });
+    return entries;
+};
 const getModulePaths = () => {
-  const result = [resolve(config.source_path)]
-  if (config.additional_paths) {
-    config.additional_paths.forEach((path) => result.push(resolve(path)))
-  }
-  result.push("node_modules")
-  return result
-}
+    const result = [resolve(config.source_path)];
+    if (config.additional_paths) {
+        config.additional_paths.forEach((path) => result.push(resolve(path)));
+    }
+    result.push("node_modules");
+    return result;
+};
 const baseConfig = {
-  mode: "production",
-  output: {
-    filename: `js/[name]${hash}.js`,
-    chunkFilename: `js/[name]${hash}.chunk.js`,
-    // https://webpack.js.org/configuration/output/#outputhotupdatechunkfilename
-    hotUpdateChunkFilename: "js/[id].[fullhash].hot-update.js",
-    path: config.outputPath,
-    publicPath: config.publicPath,
-    // This is required for SRI to work.
-    crossOriginLoading:
-      config.integrity && config.integrity.enabled
-        ? config.integrity.cross_origin
-        : false
-  },
-  entry: getEntryObject(),
-  resolve: {
-    extensions: [".js", ".jsx", ".mjs", ".ts", ".tsx", ".coffee"],
-    modules: getModulePaths()
-  },
-  plugins: getPlugins(),
-  resolveLoader: {
-    modules: ["node_modules"]
-  },
-  optimization: {
-    splitChunks: { chunks: "all" },
-    runtimeChunk: "single"
-  },
-  module: {
-    rules
-  }
-}
-module.exports = baseConfig
+    mode: "production",
+    output: {
+        filename: `js/[name]${hash}.js`,
+        chunkFilename: `js/[name]${hash}.chunk.js`,
+        // https://webpack.js.org/configuration/output/#outputhotupdatechunkfilename
+        hotUpdateChunkFilename: "js/[id].[fullhash].hot-update.js",
+        path: config.outputPath,
+        publicPath: config.publicPath,
+        // This is required for SRI to work.
+        crossOriginLoading: config.integrity && config.integrity.enabled
+            ? config.integrity.cross_origin
+            : false
+    },
+    entry: getEntryObject(),
+    resolve: {
+        extensions: [".js", ".jsx", ".mjs", ".ts", ".tsx", ".coffee"],
+        modules: getModulePaths()
+    },
+    plugins: getPlugins(),
+    resolveLoader: {
+        modules: ["node_modules"]
+    },
+    optimization: {
+        splitChunks: { chunks: "all" },
+        runtimeChunk: "single"
+    },
+    module: {
+        rules
+    }
+};
+module.exports = baseConfig;

data/package/environments/base.ts

@@ -5,7 +5,8 @@ const { basename, dirname, join, relative, resolve } = require("path")
 const { existsSync, readdirSync } = require("fs")
 import { Dirent } from "fs"
 const extname = require("path-complete-extname")
-import { Configuration, Entry } from "webpack"
+// @ts-ignore: webpack is an optional peer dependency (using type-only import)
+import type { Configuration, Entry } from "webpack"
 const config = require("../config")
 const { isProduction } = require("../env")
 

data/package/index.ts

@@ -4,7 +4,8 @@
 const webpackMerge = require("webpack-merge")
 import { resolve } from "path"
 import { existsSync } from "fs"
-import { Configuration } from "webpack"
+// @ts-ignore: webpack is an optional peer dependency (using type-only import)
+import type { Configuration } from "webpack"
 const config = require("./config")
 const baseConfig = require("./environments/base")
 const devServer = require("./dev_server")

data/package/loaders.d.ts

@@ -1,3 +1,4 @@
+// @ts-ignore: webpack is an optional peer dependency (using type-only import)
 import type { LoaderDefinitionFunction } from 'webpack'
 
 export interface ShakapackerLoaderOptions {

data/package/webpack-types.d.ts

@@ -1,3 +1,4 @@
+// @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.json

@@ -1,6 +1,6 @@
 {
   "name": "shakapacker",
-  "version": "9.0.0-beta.5",
+  "version": "9.0.0-beta.6",
   "description": "Use webpack to manage app-like JavaScript modules in Rails",
   "homepage": "https://github.com/shakacode/shakapacker",
   "bugs": {
@@ -38,7 +38,8 @@
   },
   "dependencies": {
     "js-yaml": "^4.1.0",
-    "path-complete-extname": "^1.0.0"
+    "path-complete-extname": "^1.0.0",
+    "webpack-merge": "^5.8.0"
   },
   "devDependencies": {
     "@rspack/cli": "^1.4.11",
@@ -73,9 +74,112 @@
     "typescript": "^5.9.2",
     "webpack": "5.93.0",
     "webpack-assets-manifest": "^5.0.6",
-    "webpack-merge": "^5.8.0",
     "webpack-subresource-integrity": "^5.1.0"
   },
+  "peerDependencies": {
+    "@babel/core": "^7.17.9",
+    "@babel/plugin-transform-runtime": "^7.17.0",
+    "@babel/preset-env": "^7.16.11",
+    "@babel/runtime": "^7.17.9",
+    "@rspack/core": "^1.0.0",
+    "@rspack/cli": "^1.0.0",
+    "@rspack/plugin-react-refresh": "^1.0.0",
+    "@types/babel__core": "^7.0.0",
+    "@types/webpack": "^5.0.0",
+    "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",
+    "esbuild-loader": "^2.0.0 || ^3.0.0 || ^4.0.0",
+    "mini-css-extract-plugin": "^2.0.0",
+    "rspack-manifest-plugin": "^5.0.0",
+    "sass": "^1.50.0",
+    "sass-loader": "^13.0.0 || ^14.0.0 || ^15.0.0 || ^16.0.0",
+    "swc-loader": "^0.1.15 || ^0.2.0",
+    "terser-webpack-plugin": "^5.3.1",
+    "webpack": "^5.76.0",
+    "webpack-assets-manifest": "^5.0.6 || ^6.0.0",
+    "webpack-cli": "^4.9.2 || ^5.0.0 || ^6.0.0",
+    "webpack-dev-server": "^4.15.2 || ^5.2.2",
+    "webpack-subresource-integrity": "^5.1.0"
+  },
+  "peerDependenciesMeta": {
+    "@babel/core": {
+      "optional": true
+    },
+    "@babel/plugin-transform-runtime": {
+      "optional": true
+    },
+    "@babel/preset-env": {
+      "optional": true
+    },
+    "@babel/runtime": {
+      "optional": true
+    },
+    "@rspack/core": {
+      "optional": true
+    },
+    "@rspack/cli": {
+      "optional": true
+    },
+    "@rspack/plugin-react-refresh": {
+      "optional": true
+    },
+    "@types/babel__core": {
+      "optional": true
+    },
+    "@types/webpack": {
+      "optional": true
+    },
+    "babel-loader": {
+      "optional": true
+    },
+    "compression-webpack-plugin": {
+      "optional": true
+    },
+    "css-loader": {
+      "optional": true
+    },
+    "esbuild": {
+      "optional": true
+    },
+    "esbuild-loader": {
+      "optional": true
+    },
+    "mini-css-extract-plugin": {
+      "optional": true
+    },
+    "rspack-manifest-plugin": {
+      "optional": true
+    },
+    "sass": {
+      "optional": true
+    },
+    "sass-loader": {
+      "optional": true
+    },
+    "swc-loader": {
+      "optional": true
+    },
+    "terser-webpack-plugin": {
+      "optional": true
+    },
+    "webpack": {
+      "optional": true
+    },
+    "webpack-assets-manifest": {
+      "optional": true
+    },
+    "webpack-cli": {
+      "optional": true
+    },
+    "webpack-dev-server": {
+      "optional": true
+    },
+    "webpack-subresource-integrity": {
+      "optional": true
+    }
+  },
   "packageManager": "yarn@1.22.22",
   "engines": {
     "node": ">= 14",

data/test/peer-dependencies.sh

@@ -0,0 +1,85 @@
+#!/bin/bash
+
+# Test script for verifying optional peer dependencies work correctly
+# This ensures no warnings are shown during installation with different package managers
+
+set -e
+
+echo "Testing optional peer dependencies installation..."
+
+# Colors for output
+RED='\033[0;31m'
+GREEN='\033[0;32m'
+NC='\033[0m' # No Color
+
+# Get the current directory (shakapacker root)
+SHAKAPACKER_PATH=$(pwd)
+
+# Create a temporary directory for tests
+TEST_DIR=$(mktemp -d)
+echo "Testing in: $TEST_DIR"
+
+# Function to check for peer dependency warnings
+check_warnings() {
+    local output=$1
+    local pkg_manager=$2
+
+    # Check for common peer dependency warning patterns
+    if echo "$output" | grep -i "peer" | grep -i "warn" > /dev/null 2>&1; then
+        echo -e "${RED}✗ $pkg_manager shows peer dependency warnings${NC}"
+        return 1
+    else
+        echo -e "${GREEN}✓ $pkg_manager installation clean (no warnings)${NC}"
+        return 0
+    fi
+}
+
+# Test with npm
+echo ""
+echo "Testing with npm..."
+mkdir -p "$TEST_DIR/npm-test"
+cd "$TEST_DIR/npm-test"
+npm init -y > /dev/null 2>&1
+NPM_OUTPUT=$(npm install "$SHAKAPACKER_PATH" 2>&1)
+check_warnings "$NPM_OUTPUT" "npm"
+NPM_RESULT=$?
+
+# Test with yarn
+echo ""
+echo "Testing with yarn..."
+mkdir -p "$TEST_DIR/yarn-test"
+cd "$TEST_DIR/yarn-test"
+yarn init -y > /dev/null 2>&1
+YARN_OUTPUT=$(yarn add "$SHAKAPACKER_PATH" 2>&1)
+check_warnings "$YARN_OUTPUT" "yarn"
+YARN_RESULT=$?
+
+# Test with pnpm (if available)
+if command -v pnpm &> /dev/null; then
+    echo ""
+    echo "Testing with pnpm..."
+    mkdir -p "$TEST_DIR/pnpm-test"
+    cd "$TEST_DIR/pnpm-test"
+    pnpm init > /dev/null 2>&1
+    PNPM_OUTPUT=$(pnpm add "$SHAKAPACKER_PATH" 2>&1)
+    check_warnings "$PNPM_OUTPUT" "pnpm"
+    PNPM_RESULT=$?
+else
+    echo ""
+    echo "Skipping pnpm test (not installed)"
+    PNPM_RESULT=0
+fi
+
+# Cleanup
+rm -rf "$TEST_DIR"
+
+# Summary
+echo ""
+echo "===== Test Summary ====="
+if [ $NPM_RESULT -eq 0 ] && [ $YARN_RESULT -eq 0 ] && [ $PNPM_RESULT -eq 0 ]; then
+    echo -e "${GREEN}All tests passed! No peer dependency warnings detected.${NC}"
+    exit 0
+else
+    echo -e "${RED}Some tests failed. Peer dependency warnings were detected.${NC}"
+    exit 1
+fi

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shakapacker
 version: !ruby/object:Gem::Version
-  version: 9.0.0.beta.5
+  version: 9.0.0.beta.6
 platform: ruby
 authors:
 - David Heinemeier Hansson
@@ -172,6 +172,7 @@ files:
 - docs/customizing_babel_config.md
 - docs/deployment.md
 - docs/developing_shakapacker.md
+- docs/optional-peer-dependencies.md
 - docs/peer-dependencies.md
 - docs/react.md
 - docs/rspack.md
@@ -317,6 +318,7 @@ files:
 - test/package/rules/webpack.test.js
 - test/package/staging.test.js
 - test/package/test.test.js
+- test/peer-dependencies.sh
 - test/resolver.js
 - test/typescript/build.test.js
 - tools/README.md
@@ -327,7 +329,7 @@ homepage: https://github.com/shakacode/shakapacker
 licenses:
 - MIT
 metadata:
-  source_code_uri: https://github.com/shakacode/shakapacker/tree/v9.0.0.beta.5
+  source_code_uri: https://github.com/shakacode/shakapacker/tree/v9.0.0.beta.6
 rdoc_options: []
 require_paths:
 - lib
@@ -367,5 +369,6 @@ test_files:
 - test/package/rules/webpack.test.js
 - test/package/staging.test.js
 - test/package/test.test.js
+- test/peer-dependencies.sh
 - test/resolver.js
 - test/typescript/build.test.js