Revert treeshake option, configure it in cc.config.json

.api/public.d.ts

@@ -1,4 +1,4 @@
 declare module "@cocos/ccbuild" {
     /**
      * @group Merged Types
      */
@@ -27,22 +27,6 @@
          */
         function enumerateAllDependents(meta: buildEngine.Result, featureUnits: string[]): string[];
         export type ModuleFormat = "esm" | "cjs" | "system" | "iife";
-        export type HasModuleSideEffects = (id: string, external: boolean) => boolean;
-        export type ModuleSideEffectsOption = boolean | "no-external" | string[] | HasModuleSideEffects;
-        export type TreeshakingPreset = "smallest" | "safest" | "recommended";
-        export interface NormalizedTreeshakingOptions {
-            annotations: boolean;
-            correctVarValueBeforeDeclaration: boolean;
-            manualPureFunctions: readonly string[];
-            moduleSideEffects: HasModuleSideEffects;
-            propertyReadSideEffects: boolean | "always";
-            tryCatchDeoptimization: boolean;
-            unknownGlobalSideEffects: boolean;
-        }
-        export interface TreeshakingOptions extends Partial<Omit<NormalizedTreeshakingOptions, "moduleSideEffects">> {
-            moduleSideEffects?: ModuleSideEffectsOption;
-            preset?: TreeshakingPreset;
-        }
         export interface Options {
             /**
              * 引擎仓库目录。
@@ -178,7 +162,6 @@
              * @note It's only avaiable when options.moduleFormat is 'system'.
              */
             enableNamedRegisterForSystemJSModuleFormat?: boolean;
-            treeshake?: TreeshakingOptions;
         }
         export interface Result {
             /**
@@ -231,6 +214,10 @@
          * Gets all optimzie decorators
          */
         getOptimizeDecorators(): ConfigInterface.IOptimizeDecorators;
+        /**
+         * Gets TreeShake config
+         */
+        getTreeShakeConfig(): ConfigInterface.ITreeShakeConfig | undefined;
         /**
          * Gets all features defined.
          */
@@ -375,6 +362,10 @@
              * The decorators to be optimize when build engine.
              */
             optimizeDecorators: IOptimizeDecorators;
+            /**
+             * The TreeShake config
+             */
+            treeShake?: ITreeShakeConfig;
         }
         export interface IndexConfig {
             modules?: Record<string, {
@@ -481,6 +472,9 @@
              */
             editorDecorators: string[];
         }
+        export interface ITreeShakeConfig {
+            noSideEffectFiles: string[];
+        }
     }
     export namespace Transformer {
         export namespace babel {

modules/build-engine/src/engine-js/index.ts

@@ -9,7 +9,7 @@
 import type { buildEngine } from '../index';
 import { externalWasmLoader } from './rollup-plugins/external-wasm-loader';
 import { StatsQuery } from '@ccbuild/stats-query';
-import { filePathToModuleRequest } from '@ccbuild/utils';
+import { filePathToModuleRequest, formatPath } from '@ccbuild/utils';
 import { rpNamedChunk } from './rollup-plugins/systemjs-named-register-plugin';
 import { rpInlineEnum } from './rollup-plugins/inline-enum';
 
@@ -175,7 +175,7 @@
         presetEnvOptions.targets = options.targets;
     }
 
     const babelPlugins: any[] = [];
     if (!options.targets) {
         babelPlugins.push([babelPluginTransformForOf, {
             loose: true,
@@ -231,7 +231,7 @@
         if (!process.env.ENGINE_PATH) {
             throw new Error('ENGINE_PATH environment variable not set');
         }
         babelOptions.presets?.push([(): any => ({ plugins: [[decoratorRecorder]] })]);
     }
 
     const rollupPlugins: rollup.Plugin[] = [];
@@ -260,7 +260,7 @@
 
         {
             name: '@cocos/ccbuild|module-overrides',
             resolveId(source, importer): string | null {
                 if (moduleOverrides[source]) {
                     return source;
                 } else {
@@ -402,8 +402,21 @@
         onwarn: rollupWarningHandler,
     };
 
-    if (options.treeshake) {
-        rollupOptions.treeshake = options.treeshake;
+    const treeshakeConfig = statsQuery.getTreeShakeConfig();
+    const noSideEffectFiles = treeshakeConfig?.noSideEffectFiles;
+    if (noSideEffectFiles && noSideEffectFiles.length > 0) {
+        rollupOptions.treeshake = {
+            moduleSideEffects: (id: string): boolean => {
+                const relativePath = formatPath(ps.relative(engineRoot, id));
+                if (noSideEffectFiles.includes(relativePath)) {
+                    console.info(`>>> Found no side-effect path: ${relativePath}`);
+                    return false;
+                }
+                return true;
+            }
+        };
+    } else {
+        console.info(`>>> No treeshake config found!`);
     }
 
     const perf = true;

modules/build-engine/src/index.ts

@@ -4,7 +4,7 @@
 import fs from 'fs-extra';
 import { buildTsEngine } from './engine-ts';
 
 function verifyCache (options: buildEngine.Options): boolean {
     // TODO
     return false;
 }
@@ -69,28 +69,7 @@
  */
 export namespace buildEngine {
     export type ModuleFormat = 'esm' | 'cjs' | 'system' | 'iife';
-
-    export type HasModuleSideEffects = (id: string, external: boolean) => boolean;
-    export type ModuleSideEffectsOption = boolean | 'no-external' | string[] | HasModuleSideEffects;
-
-    export type TreeshakingPreset = 'smallest' | 'safest' | 'recommended';
-
-    export interface NormalizedTreeshakingOptions {
-        annotations: boolean;
-        correctVarValueBeforeDeclaration: boolean;
-        manualPureFunctions: readonly string[];
-        moduleSideEffects: HasModuleSideEffects;
-        propertyReadSideEffects: boolean | 'always';
-        tryCatchDeoptimization: boolean;
-        unknownGlobalSideEffects: boolean;
-    }
-
-    // See https://rollupjs.org/configuration-options/#treeshake for more details.
-    export interface TreeshakingOptions extends Partial<Omit<NormalizedTreeshakingOptions, 'moduleSideEffects'>> {
-        moduleSideEffects?: ModuleSideEffectsOption;
-        preset?: TreeshakingPreset;
-    }
-
+
     export interface Options {
         /**
          * 引擎仓库目录。
@@ -258,8 +237,6 @@
          * @note It's only avaiable when options.moduleFormat is 'system'.
          */
         enableNamedRegisterForSystemJSModuleFormat?: boolean;
-
-        treeshake?: TreeshakingOptions;
     }
 
     export interface Result {

modules/stats-query/src/config-interface.ts

@@ -31,6 +31,11 @@ export interface Config {
      * The decorators to be optimize when build engine.
      */
     optimizeDecorators: IOptimizeDecorators;
+
+    /**
+     * The TreeShake config
+     */
+    treeShake?: ITreeShakeConfig;
 }
 
 export interface IndexConfig {
@@ -150,4 +155,8 @@ export interface IOptimizeDecorators {
      * The decorators which should be removed directly when they only work in Cocos Creator editor.
      */
     editorDecorators: string[],
+}
+
+export interface ITreeShakeConfig {
+    noSideEffectFiles: string[];
 }

modules/stats-query/src/index.ts

@@ -52,6 +52,13 @@ export class StatsQuery {
         return this._config.optimizeDecorators;
     }
 
+    /**
+     * Gets TreeShake config
+     */
+    public getTreeShakeConfig (): ConfigInterface.ITreeShakeConfig | undefined {
+        return this._config.treeShake;
+    }
+
     /**
      * Gets all features defined.
      */

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocos/ccbuild",
-  "version": "2.2.13",
+  "version": "2.2.14",
   "description": "The next generation of build tool for Cocos engine.",
   "main": "./lib/index.js",
   "types": "./lib/index.d.ts",

scripts/test-build-cocos.js

@@ -1,6 +1,6 @@
 const ps = require('path');
 const { ensureDir, emptyDir } = require('fs-extra');
-const { buildEngine } = require('../lib/src');
+const { buildEngine, StatsQuery } = require('../lib/src');
 
 const argv = process.argv;
 
@@ -47,16 +47,6 @@ const argv = process.argv;
         "wasmCompressionMode": 'brotli',
         "visualize": true,
         "inlineEnum": true,
-        treeshake: {
-            moduleSideEffects: (id, isExternal) => {
-                const fileName = ps.basename(id);
-                if (noSideEffectFiles.indexOf(fileName) >= 0) {
-                    console.info(`--> Found fileName: ${fileName}`);
-                    return false;
-                }
-                return true;
-            }
-        },
     };
 
     await ensureDir(outDir);