addAssets bug fix

jinxiaolu · web-flow · commit a6e84834b830 · 2024-03-26T10:24:36.000+08:00
There is an error message "Maximum call stack size exceeded" when the number of assets files is too large.
diff --git a/src/tools/esbuild/bundler-execution-result.js b/src/tools/esbuild/bundler-execution-result.js
@@ -31,7 +31,7 @@ class ExecutionResult {
         this.outputFiles.push((0, utils_1.createOutputFileFromText)(path, content, type));
     }
     addAssets(assets) {
-        this.assetFiles.push(...assets);
+        this.assetFiles = this.assetFiles.concat(assets);
     }
     addLog(value) {
         this.logs.push(value);

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ class ExecutionResult {`
`31`	`31`	`this.outputFiles.push((0, utils_1.createOutputFileFromText)(path, content, type));`
`32`	`32`	`}`
`33`	`33`	`addAssets(assets) {`
`34`		`- this.assetFiles.push(...assets);`
	`34`	`+ this.assetFiles = this.assetFiles.concat(assets);`
`35`	`35`	`}`
`36`	`36`	`addLog(value) {`
`37`	`37`	`this.logs.push(value);`