[webpack5-loc-plugin] Fix missing metadata during rebuild (#5110)

Co-authored-by: David Michon <[email protected]>

Author: dmichon-msft

common/changes/@rushstack/webpack5-localization-plugin/fix-incremental-loc_2025-02-14-20-33.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@rushstack/webpack5-localization-plugin",
+      "comment": "Fix a bug where `chunk.localizedFiles` was not set in incremental rebuilds.",
+      "type": "patch"
+    }
+  ],
+  "packageName": "@rushstack/webpack5-localization-plugin"
+}

webpack/webpack5-localization-plugin/src/AssetProcessor.ts

@@ -90,7 +90,7 @@ type ItemCacheFacade = ReturnType<ReturnType<Compilation['getCache']>['getItemCa
 export async function processLocalizedAssetCachedAsync(
   options: IProcessLocalizedAssetOptions
 ): Promise<Record<string, string>> {
-  const { compilation, asset, cache } = options;
+  const { compilation, asset, cache, chunk } = options;
   const { source: originalSource } = asset;
 
   type ETag = NonNullable<ReturnType<typeof cache.getLazyHashedEtag>>;
@@ -104,7 +104,11 @@ export async function processLocalizedAssetCachedAsync(
     await cacheItem.storePromise(output);
   }
 
-  for (const { filename, source, info } of output.processedAssets) {
+  const { localizedFiles, processedAssets } = output;
+
+  (chunk as ILocalizedWebpackChunk).localizedFiles = localizedFiles;
+
+  for (const { filename, source, info } of processedAssets) {
     if (originName === filename) {
       // This helper throws if the asset doesn't already exist
       // Use the function form so that the object identity of `related` is preserved.
@@ -116,7 +120,7 @@ export async function processLocalizedAssetCachedAsync(
     }
   }
 
-  return output.localizedFiles;
+  return localizedFiles;
 }
 
 export function processLocalizedAsset(options: IProcessLocalizedAssetOptions): IProcessLocalizedAssetResult {
@@ -141,7 +145,6 @@ export function processLocalizedAsset(options: IProcessLocalizedAssetOptions): I
   const { issues } = parsedAsset;
 
   const localizedFiles: Record<string, string> = {};
-  (chunk as ILocalizedWebpackChunk).localizedFiles = localizedFiles;
 
   const processedAssets: IProcessedAsset[] = [];