@zohodesk/client_build_tool 0.0.11-exp.26.0 → 0.0.11-exp.28.0

package/lib/shared/bundler/webpack/loaderConfigs/i18nIdReplaceLoaderConfig.js

@@ -1,37 +1,32 @@
 "use strict";
 
-const {
-  getPropertiesAsJSON
-} = require('../custom_plugins/I18nSplitPlugin/utils/propertiesUtils');
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.i18nIdReplaceLoaderConfig = i18nIdReplaceLoaderConfig;
 
-const fs = require('fs');
+var _propertiesUtils = require("../custom_plugins/I18nSplitPlugin/utils/propertiesUtils.js");
+
+var _i18nOptionsValidator = require("../common/i18nOptionsValidator.js");
+
+var _fs = _interopRequireDefault(require("fs"));
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function loadJSResourcesOnce(options) {
-  let jsResourcePath; // Determine the JSResource path based on configuration
+  let jsResourcePath;
 
   if (options.i18nIndexing && options.i18nIndexing.enable) {
     jsResourcePath = options.i18nIndexing.jsResourcePath;
   } else if (options.i18nChunkSplit && options.i18nChunkSplit.chunkSplitEnable && options.i18nChunkSplit.useNumericIndexing) {
     jsResourcePath = options.i18nChunkSplit.jsResource;
-  } else {
-    throw new Error('i18nIdReplaceLoader requires either i18nIndexing to be enabled or i18nChunkSplit with useNumericIndexing');
-  }
-
-  if (!jsResourcePath) {
-    throw new Error('Missing required jsResourcePath in i18n options');
   }
 
   try {
-    const i18nData = getPropertiesAsJSON(jsResourcePath);
-
-    if (Object.keys(i18nData).length === 0) {
-      console.warn(`[i18nIdReplaceLoaderConfig] Warning: No i18n data found in JSResource file: ${jsResourcePath}`);
-      return {};
-    }
-
+    const i18nData = (0, _propertiesUtils.getPropertiesAsJSON)(jsResourcePath);
     return i18nData;
   } catch (err) {
-    throw new Error(`Error reading JSResource file ${jsResourcePath}: ${err.message}`);
+    throw new Error(`[I18nIdReplaceLoaderConfig] Error reading JSResource file ${jsResourcePath}: ${err.message}`);
   }
 }
 
@@ -39,37 +34,35 @@ function readNumericMapOnce(numericMapPath) {
   if (!numericMapPath) return null;
 
   try {
-    if (!fs.existsSync(numericMapPath)) return null;
-    const fileContent = fs.readFileSync(numericMapPath, 'utf-8');
+    const fileContent = _fs.default.readFileSync(numericMapPath, 'utf-8');
+
     const parsed = JSON.parse(fileContent);
-    if (!parsed) return null;
     return parsed.originalKeyToNumericId ? parsed.originalKeyToNumericId : parsed;
   } catch (err) {
-    return null;
+    throw new Error(`[I18nIdReplaceLoaderConfig] Failed to read numeric map from ${numericMapPath}: ${err.message}`);
   }
 }
 
-function i18nIdReplaceLoaderConfig(options, webpackContext) {
-  let numericMapPath; // Determine the numeric map path based on configuration
+function i18nIdReplaceLoaderConfig(options) {
+  (0, _i18nOptionsValidator.validateI18nOptions)(options);
+  let numericMapPath;
 
   if (options.i18nIndexing && options.i18nIndexing.enable) {
     numericMapPath = options.i18nIndexing.numericMapPath;
   } else if (options.i18nChunkSplit && options.i18nChunkSplit.chunkSplitEnable && options.i18nChunkSplit.useNumericIndexing) {
     numericMapPath = options.i18nChunkSplit.numericMapPath;
-  } else {
-    throw new Error('i18nIdReplaceLoader requires either i18nIndexing to be enabled or i18nChunkSplit with useNumericIndexing');
   }
 
-  if (!numericMapPath) {
-    throw new Error('numericMapPath is required in i18nIndexing or i18nChunkSplit config');
-  } // Load all i18n data for key detection
-
-
   const allI18nData = loadJSResourcesOnce(options);
+  const i18nKeyReplaceLoaderPath = new URL('../loaders/i18nIdReplaceLoader.js', import.meta.url).pathname;
+  let numericIdMap;
 
-  const i18nKeyReplaceLoaderPath = require.resolve('../loaders/i18nIdReplaceLoader.js');
+  try {
+    numericIdMap = readNumericMapOnce(numericMapPath);
+  } catch (err) {
+    throw new Error(`[I18nIdReplaceLoaderConfig] Failed to load numeric map: ${err.message}`);
+  }
 
-  const numericIdMap = readNumericMapOnce(numericMapPath);
   const loaderOptions = {
     allI18nData: allI18nData,
     sourceMaps: false,
@@ -83,8 +76,4 @@ function i18nIdReplaceLoaderConfig(options, webpackContext) {
     loader: i18nKeyReplaceLoaderPath,
     options: loaderOptions
   };
-}
-
-module.exports = {
-  i18nIdReplaceLoaderConfig
-};
+}

package/lib/shared/bundler/webpack/loaders/i18nIdReplaceLoader.js

@@ -20,19 +20,10 @@ module.exports = function i18nIdReplaceLoader(source, map) {
   const resourcePath = this.resourcePath;
   this.cacheable && this.cacheable();
   const options = getOptions(this) || {};
-  const callback = this.async();
-  let numericMapAbsPath = null;
+  const callback = this.async(); // Skip files in excluded paths
 
-  if (options.numericMapPath) {
-    try {
-      numericMapAbsPath = path.isAbsolute(options.numericMapPath) ? options.numericMapPath : path.resolve(this.rootContext || this.context || process.cwd(), options.numericMapPath);
-      this.addDependency(numericMapAbsPath);
-    } catch (e) {}
-  } // Skip files in excluded paths
-
-
-  if (options.excludePaths) {
-    const shouldExclude = options.excludePaths.some(excludePath => resourcePath.includes(excludePath));
+  if (options.excludePaths && Array.isArray(options.excludePaths)) {
+    const shouldExclude = options.excludePaths.some(excludePath => typeof excludePath === 'string' && resourcePath.includes(excludePath));
 
     if (shouldExclude) {
       return callback(null, source, map);
@@ -40,57 +31,35 @@ module.exports = function i18nIdReplaceLoader(source, map) {
   } // Only process files in included paths if specified
 
 
-  if (options.includePaths && options.includePaths.length > 0) {
-    const shouldInclude = options.includePaths.some(includePath => resourcePath.includes(includePath));
+  if (options.includePaths && Array.isArray(options.includePaths) && options.includePaths.length > 0) {
+    const shouldInclude = options.includePaths.some(includePath => typeof includePath === 'string' && resourcePath.includes(includePath));
 
     if (!shouldInclude) {
       return callback(null, source, map);
     }
-  } // Validate i18n data exists
+  } // Basic runtime validation - options should already be validated by config
 
 
   if (!options.allI18nData || Object.keys(options.allI18nData).length === 0) {
     return callback(new Error(`i18nIdReplaceLoader: 'allI18nData' option is missing or empty`));
-  } // Load numeric ID mapping (use injected map if provided; fallback to memoized file read)
+  } // Load numeric ID mapping - paths already validated by config
 
 
-  let numericIdMap = options.numericIdMap || null;
+  let numericIdMap = options.numericIdMap;
 
-  try {
-    if (!numericIdMap && (numericMapAbsPath || options.numericMapPath)) {
-      // Module-level static cache
-      if (!global.__CBT_I18N_NUMERIC_MAP_CACHE__) {
-        global.__CBT_I18N_NUMERIC_MAP_CACHE__ = {};
-      }
+  if (!numericIdMap && options.numericMapPath) {
+    try {
+      const mapPath = path.isAbsolute(options.numericMapPath) ? options.numericMapPath : path.resolve(this.rootContext || this.context || process.cwd(), options.numericMapPath);
+      this.addDependency(mapPath);
 
-      const cache = global.__CBT_I18N_NUMERIC_MAP_CACHE__;
-      const p = numericMapAbsPath || options.numericMapPath;
-
-      try {
-        const stat = fs.statSync(p);
-        const key = `${p}`;
-        const mtime = stat && stat.mtimeMs ? stat.mtimeMs : 0;
-        const hit = cache[key];
-
-        if (hit && hit.mtime === mtime && hit.map) {
-          numericIdMap = hit.map;
-        } else if (fs.existsSync(p)) {
-          const txt = fs.readFileSync(p, 'utf-8');
-          const parsed = JSON.parse(txt);
-          const mapObj = parsed && parsed.originalKeyToNumericId ? parsed.originalKeyToNumericId : parsed;
-
-          if (mapObj && typeof mapObj === 'object') {
-            cache[key] = {
-              mtime,
-              map: mapObj
-            };
-            numericIdMap = mapObj;
-          }
-        }
-      } catch (e) {// ignore and proceed without map
+      if (fs.existsSync(mapPath)) {
+        const mapContent = fs.readFileSync(mapPath, 'utf-8');
+        const parsed = JSON.parse(mapContent);
+        numericIdMap = parsed.originalKeyToNumericId || parsed;
       }
+    } catch (e) {
+      return callback(new Error(`i18nIdReplaceLoader: Failed to load numeric map from ${options.numericMapPath}: ${e.message}`));
     }
-  } catch (e) {// ignore
   } // If no numeric map available, return source as-is
 
 
@@ -98,8 +67,6 @@ module.exports = function i18nIdReplaceLoader(source, map) {
     return callback(null, source, map);
   }
 
-  const isDevMode = options.devMode || process.env.NODE_ENV === 'development';
-
   try {
     // Parse the JavaScript/TypeScript source code
     const ast = parser.parse(source, {
@@ -107,23 +74,20 @@ module.exports = function i18nIdReplaceLoader(source, map) {
       plugins: ['jsx', 'typescript', 'classProperties', 'optionalChaining', 'nullishCoalescingOperator'],
       sourceFilename: resourcePath
     });
-    let hasTransformations = false; // Traverse AST and replace i18n keys with numeric IDs
+    let hasTransformations = false;
+    let transformationCount = 0; // Traverse AST and replace i18n keys with numeric IDs
 
     traverse(ast, {
       StringLiteral(path) {
         const {
           node
-        } = path; // Check if this string is an i18n key
-
-        if (!options.allI18nData.hasOwnProperty(node.value)) {
-          return;
-        } // Replace with numeric ID if available
-
+        } = path; // Check if this string is an i18n key and has numeric mapping
 
-        if (numericIdMap.hasOwnProperty(node.value)) {
+        if (options.allI18nData.hasOwnProperty(node.value) && numericIdMap.hasOwnProperty(node.value)) {
           const numericId = String(numericIdMap[node.value]);
           path.replaceWith(t.stringLiteral(numericId));
           hasTransformations = true;
+          transformationCount++;
         }
       }
 
@@ -141,6 +105,6 @@ module.exports = function i18nIdReplaceLoader(source, map) {
       callback(null, source, map);
     }
   } catch (err) {
-    callback(err);
+    callback(new Error(`i18nIdReplaceLoader: Failed to process ${resourcePath}: ${err.message}`));
   }
 };

package/lib/shared/bundler/webpack/optimizationConfig.js

@@ -23,10 +23,7 @@ function optimizationConfig(options) {
   } = options;
   const {
     chunkSplitEnable
-  } = options.i18nChunkSplit; // TEMP: Test console logs for minification verification
-
-  console.log('MINIFY_TEST: optimizationConfig called in mode:', options.mode);
-  console.log('MINIFY_TEST: TerserPlugin and CSS minification active');
+  } = options.i18nChunkSplit;
   const i18nChunkFilename = (0, _nameTemplates.nameTemplates)('i18njs', options);
   const chunkFilenameHasContentHash = (0, _hashUtils.hasContentHash)(i18nChunkFilename);
   /**

package/lib/shared/bundler/webpack/pluginConfigs/configI18nNumericIndexPlugin.js

@@ -11,17 +11,26 @@ var _I18nNumericIndexHtmlInjectorPlugin = require("../custom_plugins/I18nNumeric
 
 var _readI18nValues = require("../custom_plugins/I18nSplitPlugin/readI18nValues");
 
+var _i18nOptionsValidator = require("../common/i18nOptionsValidator.js");
+
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function configI18nNumericIndexPlugin(options) {
-  if (!(options.i18nIndexing && options.i18nIndexing.enable)) {
+  if (!options.i18nIndexing) {
     return null;
   }
 
   const i18nOpts = options.i18nIndexing;
-  const isDevelopment = options.isDevelopment || false; // Get isDevelopment from options
-  // If we are only doing numeric ID replacement (no files to emit and no HTML injection, and no custom groups),
-  // we can skip creating plugin instances to save build time.
+
+  try {
+    const result = (0, _i18nOptionsValidator.validateI18nIndexingOptions)(i18nOpts);
+
+    if (result.skipValidation) {
+      return null;
+    }
+  } catch (error) {
+    throw new Error(`[I18nNumericIndexPlugin] ${error.message}`);
+  }
 
   const emitFiles = i18nOpts.emitFiles !== undefined ? i18nOpts.emitFiles : true;
   const injectHtml = i18nOpts.injectI18nUrlInIndex !== undefined ? i18nOpts.injectI18nUrlInIndex : true;
@@ -29,85 +38,55 @@ function configI18nNumericIndexPlugin(options) {
 
   if (!emitFiles && !injectHtml && !hasCustomGroups) {
     return [];
-  } // Validate required options
-
-
-  const requiredOptions = ['jsResourcePath', 'propertiesFolderPath', 'numericMapPath', 'jsonpFunc', 'htmlTemplateLabel', 'localeVarName'];
-  const missingOptions = requiredOptions.filter(opt => !i18nOpts[opt]);
+  }
 
-  if (missingOptions.length > 0) {
-    console.warn(`[I18nNumericIndexPlugin] Missing required options: ${missingOptions.join(', ')}`);
-    return null;
-  } // Read i18n values
+  let i18nData;
 
+  try {
+    i18nData = (0, _readI18nValues.readI18nValues)({
+      jsResource: i18nOpts.jsResourcePath,
+      propertiesFolder: i18nOpts.propertiesFolderPath,
+      disableDefault: false
+    });
+  } catch (error) {
+    throw new Error(`[I18nNumericIndexPlugin] Failed to read i18n data: ${error.message}`);
+  }
 
   const {
     locales,
     allI18nObject
-  } = (0, _readI18nValues.readI18nValues)({
-    jsResource: i18nOpts.jsResourcePath,
-    propertiesFolder: i18nOpts.propertiesFolderPath,
-    disableDefault: false
-  }); // Ensure templates have [locale] placeholder
-
+  } = i18nData;
   const numericTemplate = i18nOpts.numericFilenameTemplate || '[locale]/numeric.i18n.js';
   const dynamicTemplate = i18nOpts.dynamicFilenameTemplate || '[locale]/dynamic.i18n.js';
-
-  if (!numericTemplate.includes('[locale]')) {
-    console.warn('[I18nNumericIndexPlugin] numericFilenameTemplate must include [locale] placeholder');
-    return null;
-  }
-
-  if (!dynamicTemplate.includes('[locale]')) {
-    console.warn('[I18nNumericIndexPlugin] dynamicFilenameTemplate must include [locale] placeholder');
-    return null;
-  } // Resolve singleFileTemplate if it's a function
-
-
-  let resolvedSingleFileTemplate = i18nOpts.singleFileTemplate || '[locale].js';
-
-  if (typeof i18nOpts.singleFileTemplate === 'function') {
-    resolvedSingleFileTemplate = i18nOpts.singleFileTemplate(isDevelopment);
-  } // Plugin options
-
-
-  const numericIndexPluginOptions = {
-    enable: i18nOpts.enable,
+  const sharedOptions = {
     outputFolder: i18nOpts.outputFolder || 'i18n-chunk',
+    numericFilenameTemplate: numericTemplate,
+    dynamicFilenameTemplate: dynamicTemplate,
+    singleFileTemplate: i18nOpts.singleFileTemplate,
+    singleFile: i18nOpts.singleFile || false,
+    includeContentHash: i18nOpts.includeContentHash || false
+  };
+  const numericIndexPluginOptions = { ...sharedOptions,
+    enable: i18nOpts.enable,
     jsResourcePath: i18nOpts.jsResourcePath,
     propertiesFolderPath: i18nOpts.propertiesFolderPath,
     numericMapPath: i18nOpts.numericMapPath,
     locales,
     allI18nObject,
-    numericFilenameTemplate: numericTemplate,
-    dynamicFilenameTemplate: dynamicTemplate,
-    singleFileTemplate: resolvedSingleFileTemplate,
     jsonpFunc: i18nOpts.jsonpFunc,
     localeVarName: i18nOpts.localeVarName,
-    singleFile: i18nOpts.singleFile || false,
     restrictToBaseKeys: i18nOpts.restrictToBaseKeys || false,
-    includeContentHash: i18nOpts.includeContentHash || false,
     generateManifest: i18nOpts.generateManifest || false,
     manifestPath: i18nOpts.manifestPath || null,
     customGroups: i18nOpts.customGroups || null,
-    emitFiles: i18nOpts.emitFiles !== undefined ? i18nOpts.emitFiles : true,
-    isDevelopment: isDevelopment,
+    chunkToGroupMapping: i18nOpts.chunkToGroupMapping || {},
+    emitFiles,
     i18nPublicPathVar: i18nOpts.i18nPublicPathVar
-  }; // HTML injector options
-
-  const htmlInjectorOptions = {
-    outputFolder: i18nOpts.outputFolder || 'i18n-chunk',
-    numericFilenameTemplate: numericTemplate,
-    dynamicFilenameTemplate: dynamicTemplate,
-    singleFileTemplate: resolvedSingleFileTemplate,
+  };
+  const htmlInjectorOptions = { ...sharedOptions,
     htmlTemplateLabel: i18nOpts.htmlTemplateLabel,
-    singleFile: i18nOpts.singleFile || false,
     i18nAssetsPublicPathPrefix: '',
-    injectI18nUrlInIndex: i18nOpts.injectI18nUrlInIndex !== undefined ? i18nOpts.injectI18nUrlInIndex : true,
-    // Control HTML injection
-    isDevelopment: isDevelopment
+    injectI18nUrlInIndex: injectHtml
   };
-  const i18nNumericPluginInstance = new _I18nNumericIndexPlugin.default(numericIndexPluginOptions);
-  const htmlInjectorPluginInstance = new _I18nNumericIndexHtmlInjectorPlugin.I18nNumericIndexHtmlInjectorPlugin(htmlInjectorOptions);
-  return [i18nNumericPluginInstance, htmlInjectorPluginInstance];
+  return [new _I18nNumericIndexPlugin.default(numericIndexPluginOptions), new _I18nNumericIndexHtmlInjectorPlugin.I18nNumericIndexHtmlInjectorPlugin(htmlInjectorOptions)];
 }

package/lib/shared/bundler/webpack/plugins.js

@@ -55,9 +55,6 @@ var _configCustomScriptLoadingStrategyPlugin = require("./pluginConfigs/configCu
 function plugins(options) {
   const {
     webpackPlugins
-  } = options; // TEMP: Test console logs for minification verification
-
-  console.log('MINIFY_TEST: plugins function called with options:', options.mode);
-  console.log('MINIFY_TEST: This message should be visible if minification is disabled');
+  } = options;
   return [(0, _configEnvVariables.configEnvVariables)(options), (0, _configCustomAttributesPlugin.configCustomAttributesPlugin)(options), (0, _configTPHashMappingPlugin.configTPHashMappingPlugin)(options), (0, _configCopyPublicFolders.configCopyPublicFolders)(options), (0, _configIgnorePlugin.configIgnorePlugin)(options), (0, _configMiniCSSExtractPlugin.configMiniCSSExtractPlugin)(options), (0, _configSelectorWeightPlugin.configSelectorWeightPlugin)(options), (0, _configVariableConversionPlugin.configVariableConversionPlugin)(options), (0, _configI18nSplitPlugin.configI18nSplitPlugin)(options), ...((0, _configI18nNumericIndexPlugin.configI18nNumericIndexPlugin)(options) || []), (0, _configRtlCssPlugin.configRtlCssPlugin)(options), (0, _configHtmlWebpackPlugin.configHtmlWebpackPlugin)(options), (0, _configCustomScriptLoadingStrategyPlugin.configCustomScriptLoadingStrategyPlugin)(options), (0, _configCdnChangePlugin.configCdnChangePlugin)(options), (0, _configServiceWorkerPlugin.configServiceWorkerPlugin)(options), (0, _configEFCTemplatePlugin.configEFCTemplatePlugin)(options), (0, _configResourceHintsPlugin.configResourceHintsPlugin)(options), (0, _configBundleAnalyzer.configBundleAnalyzer)(options), (0, _configManifestJsonPlugin.configManifestJsonPlugin)(options), (0, _configSourceMapPlugin.configSourceMapPlugin)(options), (0, _configProgressPlugin.configProgressPlugin)(options), (0, _configBundleIntegrityReport.configBundleIntegrityReport)(options), (0, _configRuntimeResourceCleanup.configRuntimeResourceCleanup)(options), ...webpackPlugins].filter(Boolean);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zohodesk/client_build_tool",
-  "version": "0.0.11-exp.26.0",
+  "version": "0.0.11-exp.28.0",
   "description": "A CLI tool to  build  web applications and client libraries",
   "main": "lib/index.js",
   "bin": {

package/docs/DYNAMIC_TEMPLATE_EXAMPLE.md

@@ -1,129 +0,0 @@
-# Dynamic singleFileTemplate Configuration
-
-## Overview
-The `singleFileTemplate` option now supports both static strings and dynamic functions, allowing different filename patterns based on build mode.
-
-## Configuration Options
-
-### 1. Static Template (Simple)
-```javascript
-i18nIndexing: {
-  singleFileTemplate: '[locale].js'  // Always uses this format
-}
-```
-
-### 2. Dynamic Template with Function
-```javascript
-i18nIndexing: {
-  isDevelopment: isDevelopment,  // Pass from your app
-  singleFileTemplate: (isDev) => 
-    isDev ? '[locale].js' : '[locale].[contenthash].js'
-}
-```
-
-### 3. Inline Conditional (Pre-resolved)
-```javascript
-i18nIndexing: {
-  isDevelopment: isDevelopment,  // Pass from your app
-  singleFileTemplate: isDevelopment 
-    ? '[locale].js' 
-    : '[locale].[contenthash].js'
-}
-```
-
-## Template Placeholders
-
-- `[locale]` - Replaced with actual locale code (e.g., en_US, fr_FR)
-- `[contenthash]` - Replaced with content-based hash for cache busting
-
-## Output Examples
-
-### Development Mode (isDevelopment = true)
-```
-Template: '[locale].js'
-Output:
-  i18n/en_US.js
-  i18n/fr_FR.js
-  i18n/de_DE.js
-```
-
-### Production Mode (isDevelopment = false)
-```
-Template: '[locale].[contenthash].js'
-Output:
-  i18n/en_US.b9ef890a.js
-  i18n/fr_FR.3452451e.js
-  i18n/de_DE.e8b84364.js
-```
-
-## Complete Example Configuration
-
-```javascript
-exports.config = {
-  context: 'src',
-  output: 'build',
-  mode: process.env.NODE_ENV || 'production',
-  
-  // Pass isDevelopment from your app
-  isDevelopment: process.env.NODE_ENV === 'development',
-  
-  i18nIndexing: {
-    enable: true,
-    outputFolder: 'i18n',
-    jsResourcePath: './resources/ApplicationResources.properties',
-    propertiesFolderPath: './resources',
-    numericMapPath: './numericMap.json',
-    
-    // Dynamic template based on build mode
-    singleFileTemplate: function(isDevelopment) {
-      return isDevelopment ? '[locale].js' : '[locale].[contenthash].js';
-    },
-    
-    // Or using arrow function
-    // singleFileTemplate: (isDev) => isDev ? '[locale].js' : '[locale].[contenthash].js',
-    
-    // Or pre-resolved
-    // singleFileTemplate: isDevelopment ? '[locale].js' : '[locale].[contenthash].js',
-    
-    singleFile: true,
-    jsonpFunc: 'var imAppI18n=',
-    htmlTemplateLabel: '{{--user-locale}}',
-    localeVarName: 'window.userLangCode',
-    generateManifest: true,
-    manifestPath: 'i18n/manifest.json'
-    
-    // Note: includeContentHash is now optional when using [contenthash] in template
-    // includeContentHash: true  // Not needed with [contenthash] placeholder
-  }
-};
-```
-
-## Benefits
-
-1. **Development**: Clean, readable filenames without hashes for easier debugging
-2. **Production**: Content-hashed filenames for optimal caching
-3. **Flexibility**: Single configuration handles both environments
-4. **No Redundancy**: Using `[contenthash]` in template eliminates need for `includeContentHash` option
-
-## Migration from includeContentHash
-
-### Old Way
-```javascript
-singleFileTemplate: '[locale].js',
-includeContentHash: true  // Adds hash before .js
-// Output: en_US.abc123.js
-```
-
-### New Way (Recommended)
-```javascript
-singleFileTemplate: isDevelopment ? '[locale].js' : '[locale].[contenthash].js'
-// Output: en_US.abc123.js (production) or en_US.js (development)
-```
-
-## Notes
-
-- The function receives `isDevelopment` as a boolean parameter
-- You must pass `isDevelopment` in your main options object
-- The `[contenthash]` placeholder is automatically replaced with the actual hash
-- For HTML injection, the hash placeholder is removed since the exact hash isn't known at HTML generation time
-- The manifest will map clean names to hashed versions for deployment scripts