Skip to content

Commit 2e8c7cd

Browse files
alan-agius4mgechev
authored andcommitted
fix(@angular-devkit/build-angular): add defaults to reduce methods in bundle calculator
Fixes #17215
1 parent 47555e2 commit 2e8c7cd

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

packages/angular_devkit/build_angular/src/angular-cli-files/utilities/bundle-calculator.ts

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -167,17 +167,16 @@ abstract class Calculator {
167167
} else {
168168
// No differential builds, get the chunk size by summing its assets.
169169
return chunk.files
170-
.filter((file) => !file.endsWith('.map'))
171-
.map((file: string) => {
170+
.filter(file => !file.endsWith('.map'))
171+
.map(file => {
172172
const asset = this.assets.find((asset) => asset.name === file);
173173
if (!asset) {
174174
throw new Error(`Could not find asset for file: ${file}`);
175175
}
176176

177-
return asset;
177+
return asset.size;
178178
})
179-
.map((asset) => asset.size)
180-
.reduce((l, r) => l + r);
179+
.reduce((l, r) => l + r, 0);
181180
}
182181
}
183182
}
@@ -197,8 +196,8 @@ class BundleCalculator extends Calculator {
197196
const buildSizes = Object.values(DifferentialBuildType).map((buildType) => {
198197
const size = this.chunks
199198
.filter(chunk => chunk.names.indexOf(budgetName) !== -1)
200-
.map((chunk) => this.calculateChunkSize(chunk, buildType))
201-
.reduce((l, r) => l + r);
199+
.map(chunk => this.calculateChunkSize(chunk, buildType))
200+
.reduce((l, r) => l + r, 0);
202201

203202
return {size, label: `${this.budget.name}-${buildType}`};
204203
});
@@ -223,8 +222,8 @@ class InitialCalculator extends Calculator {
223222
label: `initial-${buildType}`,
224223
size: this.chunks
225224
.filter(chunk => chunk.initial)
226-
.map((chunk) => this.calculateChunkSize(chunk, buildType))
227-
.reduce((l, r) => l + r),
225+
.map(chunk => this.calculateChunkSize(chunk, buildType))
226+
.reduce((l, r) => l + r, 0),
228227
};
229228
});
230229

@@ -244,7 +243,7 @@ class InitialCalculator extends Calculator {
244243
class AllScriptCalculator extends Calculator {
245244
calculate() {
246245
const size = this.assets
247-
.filter((asset) => asset.name.endsWith('.js'))
246+
.filter(asset => asset.name.endsWith('.js'))
248247
.map(asset => asset.size)
249248
.reduce((total: number, size: number) => total + size, 0);
250249

0 commit comments

Comments
 (0)