shakapacker 9.3.0.beta.0 → 9.3.0.beta.2

data/package/configExporter/cli.ts

@@ -10,6 +10,7 @@ import { ExportOptions, ConfigMetadata, FileOutput } from "./types"
 import { YamlSerializer } from "./yamlSerializer"
 import { FileWriter } from "./fileWriter"
 import { ConfigFileLoader, generateSampleConfigFile } from "./configFile"
+import { BuildValidator } from "./buildValidator"
 
 // Read version from package.json
 const packageJson = JSON.parse(
@@ -84,6 +85,11 @@ export async function run(args: string[]): Promise<number> {
       return runListBuildsCommand(options)
     }
 
+    // Handle --validate or --validate-build command
+    if (options.validate || options.validateBuild) {
+      return await runValidateCommand(options)
+    }
+
     // Handle --all-builds command
     if (options.allBuilds) {
       return runAllBuildsCommand(options)
@@ -130,8 +136,9 @@ export async function run(args: string[]): Promise<number> {
     }
 
     return 0
-  } catch (error: any) {
-    console.error(`[Config Exporter] Error: ${error.message}`)
+  } catch (error: unknown) {
+    const errorMessage = error instanceof Error ? error.message : String(error)
+    console.error(`[Config Exporter] Error: ${errorMessage}`)
     return 1
   }
 }
@@ -240,6 +247,16 @@ QUICK START (for troubleshooting):
       default: false,
       description: "Export all builds from config file"
     })
+    .option("validate", {
+      type: "boolean",
+      default: false,
+      description:
+        "Validate all builds by running webpack/rspack (requires config file)"
+    })
+    .option("validate-build", {
+      type: "string",
+      description: "Validate specific build from config file"
+    })
     .option("webpack", {
       type: "boolean",
       default: false,
@@ -276,6 +293,16 @@ QUICK START (for troubleshooting):
           "--build and --all-builds are mutually exclusive. Use one or the other."
         )
       }
+      if (argv.validate && argv["validate-build"]) {
+        throw new Error(
+          "--validate and --validate-build are mutually exclusive. Use one or the other."
+        )
+      }
+      if (argv.validate && (argv.build || argv["all-builds"])) {
+        throw new Error(
+          "--validate cannot be used with --build or --all-builds."
+        )
+      }
       return true
     })
     .help("help")
@@ -300,6 +327,11 @@ QUICK START (for troubleshooting):
   bin/export-bundler-config --save-dir=./debug
   bin/export-bundler-config                               # Saves to shakapacker-config-exports/
 
+  # Validate builds
+  bin/export-bundler-config --validate                    # Validate all builds
+  bin/export-bundler-config --validate-build=dev          # Validate specific build
+  bin/export-bundler-config --validate --verbose          # Validate with full logs
+
   # View config in terminal (stdout)
   bin/export-bundler-config --stdout
   bin/export-bundler-config --output=config.yaml          # Save to specific file`
@@ -334,7 +366,9 @@ QUICK START (for troubleshooting):
     configFile: argv["config-file"],
     build: argv.build,
     listBuilds: argv["list-builds"],
-    allBuilds: argv["all-builds"]
+    allBuilds: argv["all-builds"],
+    validate: argv.validate,
+    validateBuild: argv["validate-build"]
   }
 }
 
@@ -392,9 +426,141 @@ function runListBuildsCommand(options: ExportOptions): number {
     const loader = new ConfigFileLoader(options.configFile)
     loader.listBuilds()
     return 0
-  } catch (error: any) {
-    console.error(`[Config Exporter] Error: ${error.message}`)
+  } catch (error: unknown) {
+    const errorMessage = error instanceof Error ? error.message : String(error)
+    console.error(`[Config Exporter] Error: ${errorMessage}`)
+    return 1
+  }
+}
+
+async function runValidateCommand(options: ExportOptions): Promise<number> {
+  const savedEnv = saveBuildEnvironmentVariables()
+
+  try {
+    // Validate that config file exists
+    const loader = new ConfigFileLoader(options.configFile)
+    if (!loader.exists()) {
+      const configPath = options.configFile || ".bundler-config.yml"
+      throw new Error(
+        `Config file ${configPath} not found. Run --init to create it.`
+      )
+    }
+
+    // Set up environment
+    const appRoot = findAppRoot()
+    process.chdir(appRoot)
+    setupNodePath(appRoot)
+
+    const config = loader.load()
+    const validator = new BuildValidator({ verbose: options.verbose || false })
+
+    // Determine which builds to validate
+    let buildsToValidate: string[]
+    if (options.validateBuild) {
+      // Validate specific build
+      if (!config.builds[options.validateBuild]) {
+        const available = Object.keys(config.builds).join(", ")
+        throw new Error(
+          `Build '${options.validateBuild}' not found in config file.\n` +
+            `Available builds: ${available}`
+        )
+      }
+      buildsToValidate = [options.validateBuild]
+    } else {
+      // Validate all builds
+      buildsToValidate = Object.keys(config.builds)
+
+      // Handle empty builds edge case
+      if (buildsToValidate.length === 0) {
+        throw new Error(
+          `No builds found in config file. Add at least one build to .bundler-config.yml or run --init to see examples.`
+        )
+      }
+    }
+
+    console.log("\n" + "=".repeat(80))
+    console.log("🔍 Validating Builds")
+    console.log("=".repeat(80))
+    console.log(`\nValidating ${buildsToValidate.length} build(s)...\n`)
+
+    if (options.verbose) {
+      console.log("⚡ VERBOSE MODE ENABLED - Full build output will be shown")
+      console.log(
+        "   This includes all webpack/rspack compilation logs, warnings, and progress messages"
+      )
+      console.log("   Use without --verbose to see only errors and summaries\n")
+      console.log("=".repeat(80) + "\n")
+    }
+
+    const results = []
+
+    // Validate each build
+    for (const buildName of buildsToValidate) {
+      if (options.verbose) {
+        console.log("\n" + "=".repeat(80))
+        console.log(`📦 VALIDATING BUILD: ${buildName}`)
+        console.log("=".repeat(80))
+      } else {
+        console.log(`\n📦 Validating build: ${buildName}`)
+      }
+
+      // Clear and restore environment to prevent leakage between builds
+      clearBuildEnvironmentVariables()
+      restoreBuildEnvironmentVariables(savedEnv)
+
+      // Get the build's environment to use for auto-detection
+      const buildConfig = config.builds[buildName]
+      const buildEnv =
+        buildConfig.environment?.NODE_ENV ||
+        (buildConfig.environment?.RAILS_ENV as
+          | "development"
+          | "production"
+          | "test"
+          | undefined) ||
+        "development"
+
+      // Auto-detect bundler using the build's environment
+      const defaultBundler = await autoDetectBundler(buildEnv, appRoot)
+
+      // Resolve build config with the correct default bundler
+      const resolvedBuild = loader.resolveBuild(
+        buildName,
+        options,
+        defaultBundler
+      )
+
+      // Validate the build
+      const result = await validator.validateBuild(resolvedBuild, appRoot)
+      results.push(result)
+
+      // Show immediate feedback
+      if (options.verbose) {
+        console.log("=".repeat(80))
+      }
+      if (result.success) {
+        console.log(`   ✅ Build passed`)
+      } else {
+        console.log(`   ❌ Build failed with ${result.errors.length} error(s)`)
+      }
+      if (options.verbose) {
+        console.log("")
+      }
+    }
+
+    // Print formatted results
+    const formattedResults = validator.formatResults(results)
+    console.log(formattedResults)
+
+    // Return exit code based on results
+    const hasFailures = results.some((r) => !r.success)
+    return hasFailures ? 1 : 0
+  } catch (error: unknown) {
+    const errorMessage = error instanceof Error ? error.message : String(error)
+    console.error(`[Config Exporter] Error: ${errorMessage}`)
     return 1
+  } finally {
+    // Restore original environment
+    restoreBuildEnvironmentVariables(savedEnv)
   }
 }
 
@@ -426,7 +592,6 @@ async function runAllBuildsCommand(options: ExportOptions): Promise<number> {
       `\n📦 Exporting ${buildNames.length} builds from config file...\n`
     )
 
-    const fileWriter = new FileWriter()
     const targetDir = options.saveDir! // Set by applyDefaults
     const createdFiles: string[] = []
 
@@ -444,7 +609,7 @@ async function runAllBuildsCommand(options: ExportOptions): Promise<number> {
 
       for (const { config: cfg, metadata } of configs) {
         const output = formatConfig(cfg, metadata, options, appRoot)
-        const filename = fileWriter.generateFilename(
+        const filename = FileWriter.generateFilename(
           metadata.bundler,
           metadata.environment,
           metadata.configType,
@@ -453,7 +618,7 @@ async function runAllBuildsCommand(options: ExportOptions): Promise<number> {
         )
 
         const fullPath = resolve(targetDir, filename)
-        fileWriter.writeSingleFile(fullPath, output)
+        FileWriter.writeSingleFile(fullPath, output)
         createdFiles.push(fullPath)
       }
     }
@@ -471,8 +636,9 @@ async function runAllBuildsCommand(options: ExportOptions): Promise<number> {
     console.log("\n" + "=".repeat(80) + "\n")
 
     return 0
-  } catch (error: any) {
-    console.error(`[Config Exporter] Error: ${error.message}`)
+  } catch (error: unknown) {
+    const errorMessage = error instanceof Error ? error.message : String(error)
+    console.error(`[Config Exporter] Error: ${errorMessage}`)
     return 1
   } finally {
     // Restore original environment
@@ -492,7 +658,6 @@ async function runDoctorMode(
     console.log("🔍 Config Exporter - Doctor Mode")
     console.log("=".repeat(80))
 
-    const fileWriter = new FileWriter()
     const targetDir = options.saveDir! // Set by applyDefaults
 
     const createdFiles: string[] = []
@@ -526,7 +691,7 @@ async function runDoctorMode(
 
             for (const { config, metadata } of configs) {
               const output = formatConfig(config, metadata, options, appRoot)
-              const filename = fileWriter.generateFilename(
+              const filename = FileWriter.generateFilename(
                 metadata.bundler,
                 metadata.environment,
                 metadata.configType,
@@ -534,7 +699,7 @@ async function runDoctorMode(
                 metadata.buildName
               )
               const fullPath = resolve(targetDir, filename)
-              fileWriter.writeSingleFile(fullPath, output)
+              FileWriter.writeSingleFile(fullPath, output)
               createdFiles.push(fullPath)
             }
           }
@@ -543,9 +708,11 @@ async function runDoctorMode(
           printDoctorSummary(createdFiles, targetDir)
           return
         }
-      } catch (error: any) {
+      } catch (error: unknown) {
         // If config file exists but is invalid, warn and fall through to default behavior
-        console.log(`\n⚠️  Config file found but invalid: ${error.message}`)
+        const errorMessage =
+          error instanceof Error ? error.message : String(error)
+        console.log(`\n⚠️  Config file found but invalid: ${errorMessage}`)
         console.log("Falling back to default doctor mode...\n")
       }
     }
@@ -592,7 +759,7 @@ async function runDoctorMode(
            * - Filename uses "client" type and "development-hmr" build name to
            *   distinguish it from regular development client bundle
            */
-          filename = fileWriter.generateFilename(
+          filename = FileWriter.generateFilename(
             metadata.bundler,
             metadata.environment,
             "client",
@@ -600,7 +767,7 @@ async function runDoctorMode(
             "development-hmr"
           )
         } else {
-          filename = fileWriter.generateFilename(
+          filename = FileWriter.generateFilename(
             metadata.bundler,
             metadata.environment,
             metadata.configType,
@@ -611,7 +778,7 @@ async function runDoctorMode(
 
         const fullPath = resolve(targetDir, filename)
         const fileOutput: FileOutput = { filename, content: output, metadata }
-        fileWriter.writeSingleFile(fullPath, output)
+        FileWriter.writeSingleFile(fullPath, output)
         createdFiles.push(fullPath)
       }
     }
@@ -665,7 +832,6 @@ async function runSaveMode(
   const env = options.env || "development"
   console.log(`[Config Exporter] Exporting ${env} configs`)
 
-  const fileWriter = new FileWriter()
   const targetDir = options.saveDir! // Set by applyDefaults
   const configs = await loadConfigsForEnv(options.env, options, appRoot)
   const createdFiles: string[] = []
@@ -683,13 +849,13 @@ async function runSaveMode(
       appRoot
     )
     const fullPath = resolve(options.output)
-    fileWriter.writeSingleFile(fullPath, output)
+    FileWriter.writeSingleFile(fullPath, output)
     createdFiles.push(fullPath)
   } else {
     // Multi-file output (one per config)
     for (const { config, metadata } of configs) {
       const output = formatConfig(config, metadata, options, appRoot)
-      const filename = fileWriter.generateFilename(
+      const filename = FileWriter.generateFilename(
         metadata.bundler,
         metadata.environment,
         metadata.configType,
@@ -697,7 +863,7 @@ async function runSaveMode(
         metadata.buildName
       )
       const fullPath = resolve(targetDir, filename)
-      fileWriter.writeSingleFile(fullPath, output)
+      FileWriter.writeSingleFile(fullPath, output)
       createdFiles.push(fullPath)
     }
   }
@@ -737,9 +903,8 @@ async function runSingleFileMode(
   const config = combined.length === 1 ? combined[0] : combined
   const output = formatConfig(config, metadata, options, appRoot)
 
-  const fileWriter = new FileWriter()
   const filePath = resolve(process.cwd(), options.output!)
-  fileWriter.writeSingleFile(filePath, output)
+  FileWriter.writeSingleFile(filePath, output)
 }
 
 async function loadConfigsForEnv(
@@ -939,9 +1104,11 @@ async function loadConfigsForEnv(
     const argv = { mode: finalEnv }
     try {
       loadedConfig = loadedConfig(envObject, argv)
-    } catch (error: any) {
+    } catch (error: unknown) {
+      const errorMessage =
+        error instanceof Error ? error.message : String(error)
       throw new Error(
-        `Failed to execute config function: ${error.message}\n` +
+        `Failed to execute config function: ${errorMessage}\n` +
           `Config file: ${configFile}\n` +
           `Environment: ${JSON.stringify(envObject)}`
       )
@@ -1175,7 +1342,7 @@ function loadShakapackerConfig(
       )
       return { bundler, configPath }
     }
-  } catch (error: any) {
+  } catch (error: unknown) {
     console.warn(
       `[Config Exporter] Error loading shakapacker config, defaulting to webpack`
     )

data/package/configExporter/fileWriter.ts

@@ -11,16 +11,16 @@ export class FileWriter {
   /**
    * Write multiple config files (one per config in array)
    */
-  writeMultipleFiles(outputs: FileOutput[], targetDir: string): void {
+  static writeMultipleFiles(outputs: FileOutput[], targetDir: string): void {
     // Ensure directory exists
-    this.ensureDirectory(targetDir)
+    FileWriter.ensureDirectory(targetDir)
 
     // Write each file
     outputs.forEach((output) => {
       const safeName = basename(output.filename)
       const filePath = resolve(targetDir, safeName)
-      this.validateOutputPath(filePath)
-      this.writeFile(filePath, output.content)
+      FileWriter.validateOutputPath(filePath)
+      FileWriter.writeFile(filePath, output.content)
       console.log(`[Config Exporter] Created: ${filePath}`)
     })
 
@@ -32,13 +32,13 @@ export class FileWriter {
   /**
    * Write a single file
    */
-  writeSingleFile(filePath: string, content: string): void {
+  static writeSingleFile(filePath: string, content: string): void {
     // Ensure parent directory exists
     const dir = dirname(filePath)
-    this.ensureDirectory(dir)
+    FileWriter.ensureDirectory(dir)
 
-    this.validateOutputPath(filePath)
-    this.writeFile(filePath, content)
+    FileWriter.validateOutputPath(filePath)
+    FileWriter.writeFile(filePath, content)
     console.log(`[Config Exporter] Created: ${filePath}`)
   }
 
@@ -54,7 +54,7 @@ export class FileWriter {
    *   webpack-dev-client.yaml (with build name)
    *   rspack-cypress-dev-server.yaml (with build name)
    */
-  generateFilename(
+  static generateFilename(
     bundler: string,
     env: string,
     configType: "client" | "server" | "all" | "client-hmr",
@@ -66,11 +66,11 @@ export class FileWriter {
     return `${bundler}-${name}-${configType}.${ext}`
   }
 
-  private writeFile(filePath: string, content: string): void {
+  private static writeFile(filePath: string, content: string): void {
     writeFileSync(filePath, content, "utf8")
   }
 
-  private ensureDirectory(dir: string): void {
+  private static ensureDirectory(dir: string): void {
     if (!existsSync(dir)) {
       mkdirSync(dir, { recursive: true })
     }
@@ -79,7 +79,7 @@ export class FileWriter {
   /**
    * Validate output path and warn if writing outside cwd
    */
-  validateOutputPath(outputPath: string): void {
+  private static validateOutputPath(outputPath: string): void {
     const absPath = resolve(outputPath)
     const cwd = process.cwd()
 

data/package/configExporter/index.ts

@@ -5,9 +5,11 @@ export type {
   FileOutput,
   BundlerConfigFile,
   BuildConfig,
-  ResolvedBuildConfig
+  ResolvedBuildConfig,
+  BuildValidationResult
 } from "./types"
 export { YamlSerializer } from "./yamlSerializer"
 export { FileWriter } from "./fileWriter"
 export { getDocForKey } from "./configDocs"
 export { ConfigFileLoader, generateSampleConfigFile } from "./configFile"
+export { BuildValidator } from "./buildValidator"

data/package/configExporter/types.ts

@@ -18,6 +18,9 @@ export interface ExportOptions {
   build?: string
   listBuilds?: boolean
   allBuilds?: boolean
+  // Validation options
+  validate?: boolean
+  validateBuild?: string
 }
 
 export interface ConfigMetadata {
@@ -34,6 +37,7 @@ export interface ConfigMetadata {
     CLIENT_BUNDLE_ONLY?: string
     SERVER_BUNDLE_ONLY?: string
     WEBPACK_SERVE?: string
+    HMR?: string
   }
 }
 
@@ -68,3 +72,17 @@ export interface ResolvedBuildConfig {
   outputs: string[]
   configFile?: string
 }
+
+export interface BuildValidationResult {
+  buildName: string
+  success: boolean
+  errors: string[]
+  warnings: string[]
+  output: string[]
+  outputs?: string[] // Build outputs (e.g., ["client", "server"])
+  configFile?: string // Config file path if specified
+  outputPath?: string // Output directory where files are written
+  startTime?: number // Unix timestamp in milliseconds
+  endTime?: number // Unix timestamp in milliseconds
+  duration?: number // Duration in milliseconds
+}

data/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "shakapacker",
-  "version": "9.3.0-beta.0",
+  "version": "9.3.0-beta.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "shakapacker",
-      "version": "9.3.0-beta.0",
+      "version": "9.3.0-beta.2",
       "license": "MIT",
       "dependencies": {
         "js-yaml": "^4.1.0",

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shakapacker",
-  "version": "9.3.0-beta.0",
+  "version": "9.3.0-beta.2",
   "description": "Use webpack to manage app-like JavaScript modules in Rails",
   "homepage": "https://github.com/shakacode/shakapacker",
   "bugs": {
@@ -49,23 +49,11 @@
     "yargs": "^17.7.2"
   },
   "devDependencies": {
+    "@eslint/eslintrc": "^3.2.0",
+    "@eslint/js": "^9.37.0",
     "@rspack/cli": "^1.5.8",
     "@rspack/core": "^1.5.8",
     "@swc/core": "^1.3.0",
-    "babel-loader": "^8.2.4",
-    "compression-webpack-plugin": "^9.0.0",
-    "css-loader": "^7.1.2",
-    "esbuild-loader": "^2.18.0",
-    "mini-css-extract-plugin": "^2.9.4",
-    "rspack-manifest-plugin": "^5.0.3",
-    "sass-loader": "^16.0.5",
-    "swc-loader": "^0.1.15",
-    "webpack": "5.93.0",
-    "webpack-assets-manifest": "^5.0.6",
-    "webpack-cli": "^6.0.0",
-    "webpack-subresource-integrity": "^5.1.0",
-    "@eslint/eslintrc": "^3.2.0",
-    "@eslint/js": "^9.37.0",
     "@types/babel__core": "^7.20.5",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^24.5.2",
@@ -76,6 +64,10 @@
     "@types/yargs": "^17.0.33",
     "@typescript-eslint/eslint-plugin": "^8.46.0",
     "@typescript-eslint/parser": "^8.46.0",
+    "babel-loader": "^8.2.4",
+    "compression-webpack-plugin": "^9.0.0",
+    "css-loader": "^7.1.2",
+    "esbuild-loader": "^2.18.0",
     "eslint": "^9.37.0",
     "eslint-config-airbnb": "^19.0.4",
     "eslint-config-prettier": "^10.1.8",
@@ -90,9 +82,18 @@
     "knip": "^5.64.2",
     "lint-staged": "^15.2.10",
     "memory-fs": "^0.5.0",
+    "mini-css-extract-plugin": "^2.9.4",
     "prettier": "^3.2.5",
+    "rspack-manifest-plugin": "^5.0.3",
+    "sass-loader": "^16.0.5",
+    "swc-loader": "^0.1.15",
     "thenify": "^3.3.1",
-    "typescript": "^5.9.2"
+    "ts-jest": "^29.4.5",
+    "typescript": "^5.9.2",
+    "webpack": "5.93.0",
+    "webpack-assets-manifest": "^5.0.6",
+    "webpack-cli": "^6.0.0",
+    "webpack-subresource-integrity": "^5.1.0"
   },
   "peerDependencies": {
     "@babel/core": "^7.17.9",