Fixing bug where .map files would be included in the reported bundle size

Author: sindrisig

dist/index.js

@@ -96252,12 +96252,19 @@ var __webpack_exports__ = {};
             }
             const data = JSON.parse(external_fs_.readFileSync(filePath, 'utf8'));
             const { assets, chunks } = data.data.chunkGraph;
+            const excludedExtensions = [
+                '.js.map',
+                '.css.map',
+                '.ts.map',
+                '.LICENSE.txt'
+            ];
             let totalSize = 0;
             let jsSize = 0;
             let cssSize = 0;
             let htmlSize = 0;
             let otherSize = 0;
-            const assetAnalysis = assets.map((asset)=>{
+            const assetAnalysis = assets.reduce((acc, asset)=>{
+                if (excludedExtensions.some((ext)=>asset.path.endsWith(ext))) return acc;
                 totalSize += asset.size;
                 let type = 'other';
                 if (asset.path.endsWith('.js')) {
@@ -96270,12 +96277,13 @@ var __webpack_exports__ = {};
                     type = 'html';
                     htmlSize += asset.size;
                 } else otherSize += asset.size;
-                return {
+                acc.push({
                     path: asset.path,
                     size: asset.size,
                     type
-                };
-            });
+                });
+                return acc;
+            }, []);
             const chunkAnalysis = chunks.map((chunk)=>({
                     name: chunk.name,
                     size: chunk.size,

src/__tests__/fixtures/rsdoctor-data.json

@@ -13,6 +13,24 @@
           "path": "dist/static/css/index.css",
           "size": 10485760,
           "chunks": ["index"]
+        },
+        {
+          "id": 3,
+          "path": "dist/static/js/index.js.map",
+          "size": 104857600,
+          "chunks": ["index"]
+        },
+        {
+          "id": 4,
+          "path": "dist/static/css/index.css.map",
+          "size": 5242880,
+          "chunks": ["index"]
+        },
+        {
+          "id": 5,
+          "path": "dist/static/js/vendor.LICENSE.txt",
+          "size": 10240,
+          "chunks": ["index"]
         }
       ],
       "chunks": [

src/__tests__/report.test.ts

@@ -25,6 +25,18 @@ describe('Report Module', () => {
       const result = parseRsdoctorData('non-existent.json');
       expect(result).toBeNull();
     });
+
+    describe('should exclude source maps and license files from size calculations', () => {
+      it('should exclude .js.map, .css.map and .LICENSE.txt assets (fixture has 5, 3 should be excluded)', () => {
+        const result = parseRsdoctorData(mockRsdoctorDataPath);
+        expect(result?.assets).toHaveLength(2);
+      });
+
+      it('should not include source map or license file sizes in otherSize', () => {
+        const result = parseRsdoctorData(mockRsdoctorDataPath);
+        expect(result?.otherSize).toBe(0);
+      });
+    });
   });
 
   describe('generateProjectMarkdown', () => {

src/report.ts

@@ -96,16 +96,19 @@ export function parseRsdoctorData(filePath: string): BundleAnalysis | null {
 
     const data: RsdoctorData = JSON.parse(fs.readFileSync(filePath, 'utf8'));
     const { assets, chunks } = data.data.chunkGraph;
-    
+    const excludedExtensions = ['.js.map', '.css.map', '.ts.map', '.LICENSE.txt'];
+
     let totalSize = 0;
     let jsSize = 0;
     let cssSize = 0;
     let htmlSize = 0;
     let otherSize = 0;
-    
-    const assetAnalysis = assets.map(asset => {
+
+    const assetAnalysis = assets.reduce((acc: Array<{ path: string; size: number; type: 'js' | 'css' | 'html' | 'other' }>, asset) => {
+      if (excludedExtensions.some(ext => asset.path.endsWith(ext))) return acc;
+
       totalSize += asset.size;
-      
+
       let type: 'js' | 'css' | 'html' | 'other' = 'other';
       if (asset.path.endsWith('.js')) {
         type = 'js';
@@ -119,13 +122,10 @@ export function parseRsdoctorData(filePath: string): BundleAnalysis | null {
       } else {
         otherSize += asset.size;
       }
-      
-      return {
-        path: asset.path,
-        size: asset.size,
-        type
-      };
-    });
+
+      acc.push({ path: asset.path, size: asset.size, type });
+      return acc;
+    }, []);
 
     const chunkAnalysis = chunks.map(chunk => ({
       name: chunk.name,