shakapacker 9.2.0 → 9.3.0.beta.0

data/package/utils/errorCodes.ts

@@ -9,68 +9,68 @@
  */
 export enum ErrorCode {
   // Configuration errors (1xxx)
-  CONFIG_NOT_FOUND = 'SHAKAPACKER_1001',
-  CONFIG_INVALID_YAML = 'SHAKAPACKER_1002',
-  CONFIG_MISSING_REQUIRED = 'SHAKAPACKER_1003',
-  CONFIG_VALIDATION_FAILED = 'SHAKAPACKER_1004',
-  CONFIG_MERGE_FAILED = 'SHAKAPACKER_1005',
-  CONFIG_TYPE_MISMATCH = 'SHAKAPACKER_1006',
+  CONFIG_NOT_FOUND = "SHAKAPACKER_1001",
+  CONFIG_INVALID_YAML = "SHAKAPACKER_1002",
+  CONFIG_MISSING_REQUIRED = "SHAKAPACKER_1003",
+  CONFIG_VALIDATION_FAILED = "SHAKAPACKER_1004",
+  CONFIG_MERGE_FAILED = "SHAKAPACKER_1005",
+  CONFIG_TYPE_MISMATCH = "SHAKAPACKER_1006",
 
   // File system errors (2xxx)
-  FILE_NOT_FOUND = 'SHAKAPACKER_2001',
-  FILE_READ_ERROR = 'SHAKAPACKER_2002',
-  FILE_WRITE_ERROR = 'SHAKAPACKER_2003',
-  FILE_PERMISSION_DENIED = 'SHAKAPACKER_2004',
-  PATH_TRAVERSAL_DETECTED = 'SHAKAPACKER_2005',
-  INVALID_PATH = 'SHAKAPACKER_2006',
+  FILE_NOT_FOUND = "SHAKAPACKER_2001",
+  FILE_READ_ERROR = "SHAKAPACKER_2002",
+  FILE_WRITE_ERROR = "SHAKAPACKER_2003",
+  FILE_PERMISSION_DENIED = "SHAKAPACKER_2004",
+  PATH_TRAVERSAL_DETECTED = "SHAKAPACKER_2005",
+  INVALID_PATH = "SHAKAPACKER_2006",
 
   // Module errors (3xxx)
-  MODULE_NOT_FOUND = 'SHAKAPACKER_3001',
-  MODULE_LOAD_FAILED = 'SHAKAPACKER_3002',
-  MODULE_INVALID_EXPORT = 'SHAKAPACKER_3003',
-  LOADER_NOT_FOUND = 'SHAKAPACKER_3004',
-  PLUGIN_NOT_FOUND = 'SHAKAPACKER_3005',
-  PLUGIN_INVALID = 'SHAKAPACKER_3006',
+  MODULE_NOT_FOUND = "SHAKAPACKER_3001",
+  MODULE_LOAD_FAILED = "SHAKAPACKER_3002",
+  MODULE_INVALID_EXPORT = "SHAKAPACKER_3003",
+  LOADER_NOT_FOUND = "SHAKAPACKER_3004",
+  PLUGIN_NOT_FOUND = "SHAKAPACKER_3005",
+  PLUGIN_INVALID = "SHAKAPACKER_3006",
 
   // Environment errors (4xxx)
-  ENV_INVALID_NODE_ENV = 'SHAKAPACKER_4001',
-  ENV_MISSING_REQUIRED = 'SHAKAPACKER_4002',
-  ENV_INVALID_VALUE = 'SHAKAPACKER_4003',
-  ENV_SANITIZATION_REQUIRED = 'SHAKAPACKER_4004',
+  ENV_INVALID_NODE_ENV = "SHAKAPACKER_4001",
+  ENV_MISSING_REQUIRED = "SHAKAPACKER_4002",
+  ENV_INVALID_VALUE = "SHAKAPACKER_4003",
+  ENV_SANITIZATION_REQUIRED = "SHAKAPACKER_4004",
 
   // Bundler errors (5xxx)
-  BUNDLER_UNSUPPORTED = 'SHAKAPACKER_5001',
-  BUNDLER_CONFIG_INVALID = 'SHAKAPACKER_5002',
-  WEBPACK_CONFIG_INVALID = 'SHAKAPACKER_5003',
-  RSPACK_CONFIG_INVALID = 'SHAKAPACKER_5004',
-  TRANSPILER_NOT_FOUND = 'SHAKAPACKER_5005',
-  TRANSPILER_CONFIG_INVALID = 'SHAKAPACKER_5006',
+  BUNDLER_UNSUPPORTED = "SHAKAPACKER_5001",
+  BUNDLER_CONFIG_INVALID = "SHAKAPACKER_5002",
+  WEBPACK_CONFIG_INVALID = "SHAKAPACKER_5003",
+  RSPACK_CONFIG_INVALID = "SHAKAPACKER_5004",
+  TRANSPILER_NOT_FOUND = "SHAKAPACKER_5005",
+  TRANSPILER_CONFIG_INVALID = "SHAKAPACKER_5006",
 
   // Dev server errors (6xxx)
-  DEVSERVER_CONFIG_INVALID = 'SHAKAPACKER_6001',
-  DEVSERVER_PORT_INVALID = 'SHAKAPACKER_6002',
-  DEVSERVER_PORT_IN_USE = 'SHAKAPACKER_6003',
-  DEVSERVER_START_FAILED = 'SHAKAPACKER_6004',
+  DEVSERVER_CONFIG_INVALID = "SHAKAPACKER_6001",
+  DEVSERVER_PORT_INVALID = "SHAKAPACKER_6002",
+  DEVSERVER_PORT_IN_USE = "SHAKAPACKER_6003",
+  DEVSERVER_START_FAILED = "SHAKAPACKER_6004",
 
   // Security errors (7xxx)
-  SECURITY_PATH_TRAVERSAL = 'SHAKAPACKER_7001',
-  SECURITY_INVALID_INPUT = 'SHAKAPACKER_7002',
-  SECURITY_CONTROL_CHARS = 'SHAKAPACKER_7003',
-  SECURITY_INJECTION_ATTEMPT = 'SHAKAPACKER_7004',
+  SECURITY_PATH_TRAVERSAL = "SHAKAPACKER_7001",
+  SECURITY_INVALID_INPUT = "SHAKAPACKER_7002",
+  SECURITY_CONTROL_CHARS = "SHAKAPACKER_7003",
+  SECURITY_INJECTION_ATTEMPT = "SHAKAPACKER_7004",
 
   // Validation errors (8xxx)
-  VALIDATION_FAILED = 'SHAKAPACKER_8001',
-  VALIDATION_TYPE_ERROR = 'SHAKAPACKER_8002',
-  VALIDATION_RANGE_ERROR = 'SHAKAPACKER_8003',
-  VALIDATION_FORMAT_ERROR = 'SHAKAPACKER_8004',
-  VALIDATION_CONSTRAINT_ERROR = 'SHAKAPACKER_8005',
+  VALIDATION_FAILED = "SHAKAPACKER_8001",
+  VALIDATION_TYPE_ERROR = "SHAKAPACKER_8002",
+  VALIDATION_RANGE_ERROR = "SHAKAPACKER_8003",
+  VALIDATION_FORMAT_ERROR = "SHAKAPACKER_8004",
+  VALIDATION_CONSTRAINT_ERROR = "SHAKAPACKER_8005",
 
   // Generic errors (9xxx)
-  UNKNOWN_ERROR = 'SHAKAPACKER_9000',
-  INTERNAL_ERROR = 'SHAKAPACKER_9001',
-  DEPRECATED_FEATURE = 'SHAKAPACKER_9002',
-  NOT_IMPLEMENTED = 'SHAKAPACKER_9003',
-  OPERATION_FAILED = 'SHAKAPACKER_9004'
+  UNKNOWN_ERROR = "SHAKAPACKER_9000",
+  INTERNAL_ERROR = "SHAKAPACKER_9001",
+  DEPRECATED_FEATURE = "SHAKAPACKER_9002",
+  NOT_IMPLEMENTED = "SHAKAPACKER_9003",
+  OPERATION_FAILED = "SHAKAPACKER_9004"
 }
 
 /**
@@ -78,68 +78,86 @@ export enum ErrorCode {
  */
 export const ErrorMessages: Record<ErrorCode, string> = {
   // Configuration errors
-  [ErrorCode.CONFIG_NOT_FOUND]: 'Configuration file not found: {path}',
-  [ErrorCode.CONFIG_INVALID_YAML]: 'Invalid YAML in configuration file: {path}',
-  [ErrorCode.CONFIG_MISSING_REQUIRED]: 'Missing required configuration field: {field}',
-  [ErrorCode.CONFIG_VALIDATION_FAILED]: 'Configuration validation failed: {reason}',
-  [ErrorCode.CONFIG_MERGE_FAILED]: 'Failed to merge configurations: {reason}',
-  [ErrorCode.CONFIG_TYPE_MISMATCH]: 'Configuration type mismatch for {field}: expected {expected}, got {actual}',
+  [ErrorCode.CONFIG_NOT_FOUND]: "Configuration file not found: {path}",
+  [ErrorCode.CONFIG_INVALID_YAML]: "Invalid YAML in configuration file: {path}",
+  [ErrorCode.CONFIG_MISSING_REQUIRED]:
+    "Missing required configuration field: {field}",
+  [ErrorCode.CONFIG_VALIDATION_FAILED]:
+    "Configuration validation failed: {reason}",
+  [ErrorCode.CONFIG_MERGE_FAILED]: "Failed to merge configurations: {reason}",
+  [ErrorCode.CONFIG_TYPE_MISMATCH]:
+    "Configuration type mismatch for {field}: expected {expected}, got {actual}",
 
   // File system errors
-  [ErrorCode.FILE_NOT_FOUND]: 'File not found: {path}',
-  [ErrorCode.FILE_READ_ERROR]: 'Error reading file: {path}',
-  [ErrorCode.FILE_WRITE_ERROR]: 'Error writing file: {path}',
-  [ErrorCode.FILE_PERMISSION_DENIED]: 'Permission denied accessing: {path}',
-  [ErrorCode.PATH_TRAVERSAL_DETECTED]: 'Path traversal attempt detected: {path}',
-  [ErrorCode.INVALID_PATH]: 'Invalid path: {path}',
+  [ErrorCode.FILE_NOT_FOUND]: "File not found: {path}",
+  [ErrorCode.FILE_READ_ERROR]: "Error reading file: {path}",
+  [ErrorCode.FILE_WRITE_ERROR]: "Error writing file: {path}",
+  [ErrorCode.FILE_PERMISSION_DENIED]: "Permission denied accessing: {path}",
+  [ErrorCode.PATH_TRAVERSAL_DETECTED]:
+    "Path traversal attempt detected: {path}",
+  [ErrorCode.INVALID_PATH]: "Invalid path: {path}",
 
   // Module errors
-  [ErrorCode.MODULE_NOT_FOUND]: 'Module not found: {module}',
-  [ErrorCode.MODULE_LOAD_FAILED]: 'Failed to load module: {module}',
-  [ErrorCode.MODULE_INVALID_EXPORT]: 'Invalid export from module: {module}',
-  [ErrorCode.LOADER_NOT_FOUND]: 'Loader not found: {loader}',
-  [ErrorCode.PLUGIN_NOT_FOUND]: 'Plugin not found: {plugin}',
-  [ErrorCode.PLUGIN_INVALID]: 'Invalid plugin: {plugin}',
+  [ErrorCode.MODULE_NOT_FOUND]: "Module not found: {module}",
+  [ErrorCode.MODULE_LOAD_FAILED]: "Failed to load module: {module}",
+  [ErrorCode.MODULE_INVALID_EXPORT]: "Invalid export from module: {module}",
+  [ErrorCode.LOADER_NOT_FOUND]: "Loader not found: {loader}",
+  [ErrorCode.PLUGIN_NOT_FOUND]: "Plugin not found: {plugin}",
+  [ErrorCode.PLUGIN_INVALID]: "Invalid plugin: {plugin}",
 
   // Environment errors
-  [ErrorCode.ENV_INVALID_NODE_ENV]: 'Invalid NODE_ENV value: {value}. Valid values are: {valid}',
-  [ErrorCode.ENV_MISSING_REQUIRED]: 'Missing required environment variable: {variable}',
-  [ErrorCode.ENV_INVALID_VALUE]: 'Invalid value for environment variable {variable}: {value}',
-  [ErrorCode.ENV_SANITIZATION_REQUIRED]: 'Environment variable {variable} contained unsafe characters and was sanitized',
+  [ErrorCode.ENV_INVALID_NODE_ENV]:
+    "Invalid NODE_ENV value: {value}. Valid values are: {valid}",
+  [ErrorCode.ENV_MISSING_REQUIRED]:
+    "Missing required environment variable: {variable}",
+  [ErrorCode.ENV_INVALID_VALUE]:
+    "Invalid value for environment variable {variable}: {value}",
+  [ErrorCode.ENV_SANITIZATION_REQUIRED]:
+    "Environment variable {variable} contained unsafe characters and was sanitized",
 
   // Bundler errors
-  [ErrorCode.BUNDLER_UNSUPPORTED]: 'Unsupported bundler: {bundler}',
-  [ErrorCode.BUNDLER_CONFIG_INVALID]: 'Invalid bundler configuration: {reason}',
-  [ErrorCode.WEBPACK_CONFIG_INVALID]: 'Invalid webpack configuration: {reason}',
-  [ErrorCode.RSPACK_CONFIG_INVALID]: 'Invalid rspack configuration: {reason}',
-  [ErrorCode.TRANSPILER_NOT_FOUND]: 'Transpiler not found: {transpiler}',
-  [ErrorCode.TRANSPILER_CONFIG_INVALID]: 'Invalid transpiler configuration: {reason}',
+  [ErrorCode.BUNDLER_UNSUPPORTED]: "Unsupported bundler: {bundler}",
+  [ErrorCode.BUNDLER_CONFIG_INVALID]: "Invalid bundler configuration: {reason}",
+  [ErrorCode.WEBPACK_CONFIG_INVALID]: "Invalid webpack configuration: {reason}",
+  [ErrorCode.RSPACK_CONFIG_INVALID]: "Invalid rspack configuration: {reason}",
+  [ErrorCode.TRANSPILER_NOT_FOUND]: "Transpiler not found: {transpiler}",
+  [ErrorCode.TRANSPILER_CONFIG_INVALID]:
+    "Invalid transpiler configuration: {reason}",
 
   // Dev server errors
-  [ErrorCode.DEVSERVER_CONFIG_INVALID]: 'Invalid dev server configuration: {reason}',
-  [ErrorCode.DEVSERVER_PORT_INVALID]: 'Invalid port: {port}',
-  [ErrorCode.DEVSERVER_PORT_IN_USE]: 'Port {port} is already in use',
-  [ErrorCode.DEVSERVER_START_FAILED]: 'Failed to start dev server: {reason}',
+  [ErrorCode.DEVSERVER_CONFIG_INVALID]:
+    "Invalid dev server configuration: {reason}",
+  [ErrorCode.DEVSERVER_PORT_INVALID]: "Invalid port: {port}",
+  [ErrorCode.DEVSERVER_PORT_IN_USE]: "Port {port} is already in use",
+  [ErrorCode.DEVSERVER_START_FAILED]: "Failed to start dev server: {reason}",
 
   // Security errors
-  [ErrorCode.SECURITY_PATH_TRAVERSAL]: 'Security: Path traversal attempt blocked: {path}',
-  [ErrorCode.SECURITY_INVALID_INPUT]: 'Security: Invalid input detected: {input}',
-  [ErrorCode.SECURITY_CONTROL_CHARS]: 'Security: Control characters detected and removed from: {field}',
-  [ErrorCode.SECURITY_INJECTION_ATTEMPT]: 'Security: Potential injection attempt blocked: {details}',
+  [ErrorCode.SECURITY_PATH_TRAVERSAL]:
+    "Security: Path traversal attempt blocked: {path}",
+  [ErrorCode.SECURITY_INVALID_INPUT]:
+    "Security: Invalid input detected: {input}",
+  [ErrorCode.SECURITY_CONTROL_CHARS]:
+    "Security: Control characters detected and removed from: {field}",
+  [ErrorCode.SECURITY_INJECTION_ATTEMPT]:
+    "Security: Potential injection attempt blocked: {details}",
 
   // Validation errors
-  [ErrorCode.VALIDATION_FAILED]: 'Validation failed: {reason}',
-  [ErrorCode.VALIDATION_TYPE_ERROR]: 'Type validation error: {field} should be {type}',
-  [ErrorCode.VALIDATION_RANGE_ERROR]: 'Value out of range: {field} must be between {min} and {max}',
-  [ErrorCode.VALIDATION_FORMAT_ERROR]: 'Format error: {field} does not match expected format',
-  [ErrorCode.VALIDATION_CONSTRAINT_ERROR]: 'Constraint violation: {constraint}',
+  [ErrorCode.VALIDATION_FAILED]: "Validation failed: {reason}",
+  [ErrorCode.VALIDATION_TYPE_ERROR]:
+    "Type validation error: {field} should be {type}",
+  [ErrorCode.VALIDATION_RANGE_ERROR]:
+    "Value out of range: {field} must be between {min} and {max}",
+  [ErrorCode.VALIDATION_FORMAT_ERROR]:
+    "Format error: {field} does not match expected format",
+  [ErrorCode.VALIDATION_CONSTRAINT_ERROR]: "Constraint violation: {constraint}",
 
   // Generic errors
-  [ErrorCode.UNKNOWN_ERROR]: 'An unknown error occurred',
-  [ErrorCode.INTERNAL_ERROR]: 'Internal error: {details}',
-  [ErrorCode.DEPRECATED_FEATURE]: 'Deprecated feature: {feature}. Use {alternative} instead',
-  [ErrorCode.NOT_IMPLEMENTED]: 'Feature not yet implemented: {feature}',
-  [ErrorCode.OPERATION_FAILED]: 'Operation failed: {operation}'
+  [ErrorCode.UNKNOWN_ERROR]: "An unknown error occurred",
+  [ErrorCode.INTERNAL_ERROR]: "Internal error: {details}",
+  [ErrorCode.DEPRECATED_FEATURE]:
+    "Deprecated feature: {feature}. Use {alternative} instead",
+  [ErrorCode.NOT_IMPLEMENTED]: "Feature not yet implemented: {feature}",
+  [ErrorCode.OPERATION_FAILED]: "Operation failed: {operation}"
 }
 
 /**
@@ -149,12 +167,17 @@ export class ShakapackerError extends Error {
   public readonly code: ErrorCode
   public readonly details?: Record<string, any>
 
-  constructor(code: ErrorCode, details?: Record<string, any>, customMessage?: string) {
-    const template = ErrorMessages[code] || 'An error occurred'
-    const message = customMessage || ShakapackerError.formatMessage(template, details)
+  constructor(
+    code: ErrorCode,
+    details?: Record<string, any>,
+    customMessage?: string
+  ) {
+    const template = ErrorMessages[code] || "An error occurred"
+    const message =
+      customMessage || ShakapackerError.formatMessage(template, details)
 
     super(message)
-    this.name = 'ShakapackerError'
+    this.name = "ShakapackerError"
     this.code = code
     this.details = details
 
@@ -167,13 +190,16 @@ export class ShakapackerError extends Error {
   /**
    * Format error message with template values
    */
-  private static formatMessage(template: string, details?: Record<string, any>): string {
+  private static formatMessage(
+    template: string,
+    details?: Record<string, any>
+  ): string {
     if (!details) return template
 
     return template.replace(/{(\w+)}/g, (match, key) => {
       const value = details[key]
       if (value === undefined) return match
-      if (typeof value === 'object') {
+      if (typeof value === "object") {
         return JSON.stringify(value)
       }
       return String(value)
@@ -197,7 +223,10 @@ export class ShakapackerError extends Error {
 /**
  * Helper function to create a Shakapacker error
  */
-export function createError(code: ErrorCode, details?: Record<string, any>): ShakapackerError {
+export function createError(
+  code: ErrorCode,
+  details?: Record<string, any>
+): ShakapackerError {
   return new ShakapackerError(code, details)
 }
 
@@ -216,4 +245,4 @@ export function getErrorCode(error: unknown): ErrorCode | null {
     return error.code
   }
   return null
-}
+}

data/package/utils/errorHelpers.ts

@@ -2,7 +2,7 @@
  * Error handling utilities for consistent error management
  */
 
-import { ErrorCode, ShakapackerError } from './errorCodes'
+import { ErrorCode, ShakapackerError } from "./errorCodes"
 
 /**
  * Checks if an error is a file not found error (ENOENT)
@@ -10,9 +10,9 @@ import { ErrorCode, ShakapackerError } from './errorCodes'
 export function isFileNotFoundError(error: unknown): boolean {
   return (
     error !== null &&
-    typeof error === 'object' &&
-    'code' in error &&
-    (error as NodeJS.ErrnoException).code === 'ENOENT'
+    typeof error === "object" &&
+    "code" in error &&
+    (error as NodeJS.ErrnoException).code === "ENOENT"
   )
 }
 
@@ -22,9 +22,9 @@ export function isFileNotFoundError(error: unknown): boolean {
 export function isModuleNotFoundError(error: unknown): boolean {
   return (
     error !== null &&
-    typeof error === 'object' &&
-    'code' in error &&
-    (error as NodeJS.ErrnoException).code === 'MODULE_NOT_FOUND'
+    typeof error === "object" &&
+    "code" in error &&
+    (error as NodeJS.ErrnoException).code === "MODULE_NOT_FOUND"
   )
 }
 
@@ -32,15 +32,16 @@ export function isModuleNotFoundError(error: unknown): boolean {
  * Creates a consistent error message for file operations
  */
 export function createFileOperationError(
-  operation: 'read' | 'write' | 'delete',
+  operation: "read" | "write" | "delete",
   filePath: string,
   details?: string
 ): ShakapackerError {
-  const errorCode = operation === 'read'
-    ? ErrorCode.FILE_READ_ERROR
-    : operation === 'write'
-    ? ErrorCode.FILE_WRITE_ERROR
-    : ErrorCode.FILE_NOT_FOUND
+  const errorCode =
+    operation === "read"
+      ? ErrorCode.FILE_READ_ERROR
+      : operation === "write"
+        ? ErrorCode.FILE_WRITE_ERROR
+        : ErrorCode.FILE_NOT_FOUND
 
   return new ShakapackerError(errorCode, {
     path: filePath,
@@ -53,17 +54,18 @@ export function createFileOperationError(
  * Creates a consistent error message for file operations (backward compatibility)
  */
 export function createFileOperationErrorLegacy(
-  operation: 'read' | 'write' | 'delete',
+  operation: "read" | "write" | "delete",
   filePath: string,
   details?: string
 ): Error {
   const baseMessage = `Failed to ${operation} file at path '${filePath}'`
-  const errorDetails = details ? ` - ${details}` : ''
-  const suggestion = operation === 'read'
-    ? ' (check if file exists and permissions are correct)'
-    : operation === 'write'
-    ? ' (check write permissions and disk space)'
-    : ' (check permissions)'
+  const errorDetails = details ? ` - ${details}` : ""
+  const suggestion =
+    operation === "read"
+      ? " (check if file exists and permissions are correct)"
+      : operation === "write"
+        ? " (check write permissions and disk space)"
+        : " (check permissions)"
   return new Error(`${baseMessage}${errorDetails}${suggestion}`)
 }
 
@@ -73,13 +75,15 @@ export function createFileOperationErrorLegacy(
 export function getErrorMessage(error: unknown): string {
   if (error instanceof Error) {
     // Include stack trace for better debugging in development
-    const isDev = process.env.NODE_ENV === 'development'
-    return isDev && error.stack ? `${error.message}\n${error.stack}` : error.message
+    const isDev = process.env.NODE_ENV === "development"
+    return isDev && error.stack
+      ? `${error.message}\n${error.stack}`
+      : error.message
   }
-  if (typeof error === 'string') {
+  if (typeof error === "string") {
     return error
   }
-  if (error && typeof error === 'object' && 'message' in error) {
+  if (error && typeof error === "object" && "message" in error) {
     return String((error as { message: unknown }).message)
   }
   // Provide more context for truly unknown errors
@@ -92,8 +96,8 @@ export function getErrorMessage(error: unknown): string {
 export function isNodeError(error: unknown): error is NodeJS.ErrnoException {
   return (
     error instanceof Error &&
-    'code' in error &&
-    typeof (error as NodeJS.ErrnoException).code === 'string'
+    "code" in error &&
+    typeof (error as NodeJS.ErrnoException).code === "string"
   )
 }
 
@@ -115,7 +119,10 @@ export function createConfigValidationErrorWithCode(
 /**
  * Creates a module not found error
  */
-export function createModuleNotFoundError(moduleName: string, details?: string): ShakapackerError {
+export function createModuleNotFoundError(
+  moduleName: string,
+  details?: string
+): ShakapackerError {
   return new ShakapackerError(ErrorCode.MODULE_NOT_FOUND, {
     module: moduleName,
     details
@@ -139,5 +146,3 @@ export function createPortValidationError(port: unknown): ShakapackerError {
     port: String(port)
   })
 }
-
-

data/package/utils/getStyleRule.ts

@@ -10,7 +10,10 @@ interface StyleRule {
   type?: string
 }
 
-const getStyleRule = (test: RegExp, preprocessors: any[] = []): StyleRule | null => {
+const getStyleRule = (
+  test: RegExp,
+  preprocessors: any[] = []
+): StyleRule | null => {
   if (moduleExists("css-loader")) {
     const tryPostcss = () =>
       canProcess("postcss-loader", (loaderPath: string) => ({
@@ -38,7 +41,7 @@ const getStyleRule = (test: RegExp, preprocessors: any[] = []): StyleRule | null
             // Note: css-loader requires 'camelCaseOnly' or 'dashesOnly' when namedExport is true
             // Using 'camelCase' with namedExport: true causes a build error
             namedExport: true,
-            exportLocalsConvention: 'camelCaseOnly'
+            exportLocalsConvention: "camelCaseOnly"
           }
         }
       },

data/package/utils/helpers.ts

@@ -1,8 +1,10 @@
 const { isModuleNotFoundError, getErrorMessage } = require("./errorHelpers")
 
-const isBoolean = (str: string): boolean => /^true/.test(str) || /^false/.test(str)
+const isBoolean = (str: string): boolean =>
+  /^true/.test(str) || /^false/.test(str)
 
-const ensureTrailingSlash = (path: string): string => (path.endsWith("/") ? path : `${path}/`)
+const ensureTrailingSlash = (path: string): string =>
+  path.endsWith("/") ? path : `${path}/`
 
 const resolvedPath = (packageName: string): string | null => {
   try {
@@ -15,9 +17,13 @@ const resolvedPath = (packageName: string): string | null => {
   }
 }
 
-const moduleExists = (packageName: string): boolean => !!resolvedPath(packageName)
+const moduleExists = (packageName: string): boolean =>
+  !!resolvedPath(packageName)
 
-const canProcess = <T = unknown>(rule: string, fn: (modulePath: string) => T): T | null => {
+const canProcess = <T = unknown>(
+  rule: string,
+  fn: (modulePath: string) => T
+): T | null => {
   const modulePath = resolvedPath(rule)
 
   if (modulePath) {
@@ -27,7 +33,11 @@ const canProcess = <T = unknown>(rule: string, fn: (modulePath: string) => T): T
   return null
 }
 
-const loaderMatches = <T = unknown>(configLoader: string, loaderToCheck: string, fn: () => T): T | null => {
+const loaderMatches = <T = unknown>(
+  configLoader: string,
+  loaderToCheck: string,
+  fn: () => T
+): T | null => {
   if (configLoader !== loaderToCheck) {
     return null
   }
@@ -37,10 +47,10 @@ const loaderMatches = <T = unknown>(configLoader: string, loaderToCheck: string,
   if (!moduleExists(loaderName)) {
     throw new Error(
       `Your Shakapacker config specified using ${configLoader}, but ${loaderName} package is not installed.\n` +
-      `\nTo fix this issue, run one of the following commands:\n` +
-      `  npm install --save-dev ${loaderName}\n` +
-      `  yarn add --dev ${loaderName}\n` +
-      `\nOr change your 'javascript_transpiler' setting in shakapacker.yml to use a different loader.`
+        `\nTo fix this issue, run one of the following commands:\n` +
+        `  npm install --save-dev ${loaderName}\n` +
+        `  yarn add --dev ${loaderName}\n` +
+        `\nOr change your 'javascript_transpiler' setting in shakapacker.yml to use a different loader.`
     )
   }
 
@@ -68,9 +78,9 @@ const packageFullVersion = (packageName: string): string => {
   }
 }
 
-const packageMajorVersion = (packageName: string): string => {
+const packageMajorVersion = (packageName: string): number => {
   const match = packageFullVersion(packageName).match(/^\d+/)
-  return match ? match[0] : "0"
+  return match ? parseInt(match[0], 10) : 0
 }
 
 export {