Fix transform id while checking moduleOverrides. Update version to 2.2.16

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

@@ -2,6 +2,7 @@
 import ps from 'path';
 import { babel as Transformer } from '@ccbuild/transformer';
 import { rollup as Bundler } from '@ccbuild/bundler';
+import { ps as pathUtils } from '@ccbuild/utils';
 import realFs from 'fs';
 import tsConfigPaths from './rollup-plugins/ts-paths';
 import moduleQueryPlugin from './rollup-plugins/module-query-plugin';
@@ -49,9 +50,6 @@
     });
 })();
 
-function makePathEqualityKey(path: string): string {
-    return process.platform === 'win32' ? path.toLocaleLowerCase() : path;
-}
 
 export async function buildJsEngine(options: Required<buildEngine.Options>): Promise<buildEngine.Result> {
     const doUglify = !!options.compress;
@@ -116,7 +114,7 @@
         platform: options.platform,
         buildTimeConstants,
     })).reduce((result, [k, v]) => {
-        result[makePathEqualityKey(k)] = v;
+        result[pathUtils.makePathEqualityKey(k)] = v;
         return result;
     }, {} as Record<string, string>);
 
@@ -175,7 +173,7 @@
         presetEnvOptions.targets = options.targets;
     }
 
     const babelPlugins: any[] = [];
     if (!options.targets) {
         babelPlugins.push([babelPluginTransformForOf, {
             loose: true,
@@ -231,7 +229,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[] = [];
@@ -261,7 +259,7 @@
 
         {
             name: '@cocos/ccbuild|module-overrides',
             resolveId(source, importer): string | null {
                 if (moduleOverrides[source]) {
                     return source;
                 } else {
@@ -269,7 +267,7 @@
                 }
             },
             load(this, id: string): string | null {
-                const key = makePathEqualityKey(id);
+                const key = pathUtils.makePathEqualityKey(id);
                 if (!(key in moduleOverrides)) {
                     return null;
                 }
@@ -406,6 +404,13 @@
     const treeshakeConfig = statsQuery.getTreeShakeConfig();
     const noSideEffectFiles = treeshakeConfig?.noSideEffectFiles;
     if (noSideEffectFiles && noSideEffectFiles.length > 0) {
+        for (const noSideEffectFile of noSideEffectFiles) {
+            const absolutePath = ps.join(engineRoot, noSideEffectFile);
+            if (!fs.pathExistsSync(absolutePath)) {
+                console.error(`>>> ERROR: noSideEffectFile: ( ${noSideEffectFile} ) doesn't exist!`);
+            }
+        }
+
         rollupOptions.treeshake = {
             moduleSideEffects: (id: string): boolean => {
                 const relativePath = formatPath(ps.relative(engineRoot, id));

modules/build-engine/src/engine-js/rollup-plugins/inline-enum/index.ts

@@ -6,6 +6,7 @@
 import { type Options, resolveOptions } from './core/options';
 import { IDefines, scanEnums } from './core/enum';
 import { rollup as Bundler } from '@ccbuild/bundler';
+import { ps as pathUtils } from '@ccbuild/utils';
 import rollup = Bundler.core;
 
 
@@ -23,7 +24,7 @@
 /**
  * The main unplugin instance.
  */
 export async function rpInlineEnum(rawOptions: Options, meta?: any): Promise<rollup.Plugin[]> {
     const options = resolveOptions(rawOptions);
     const filter = createFilter(options.include, options.exclude);
 
@@ -55,18 +56,20 @@
     );
 
     const name = 'cc-inline-enum';
+
     return [
         {
             name,
             //   enforce: options.enforce,
 
             async resolveId (this, source, importer): Promise<string | { id: string; external: true; } | null> {
                 return filter(source) ? source : null;
             },
 
             transform(this, code: string, moduleId: string): rollup.TransformResult {
                 // Don't transform a module that is overrode
-                if (options.moduleOverrides && (moduleId in options.moduleOverrides)) {
+                const cacheKey = pathUtils.makePathEqualityKey(moduleId);
+                if (options.moduleOverrides && (cacheKey in options.moduleOverrides)) {
                     return;
                 }
 
@@ -90,7 +93,7 @@
                             `${prefix} const ${id} = {${members
                                 .flatMap(({ name, value }) => {
                                     const forwardMapping = `${JSON.stringify(name)}: ${JSON.stringify(value)}`;
                                     const reverseMapping = `${JSON.stringify(value.toString())}: ${JSON.stringify(name)}`;
 
                                     // see https://www.typescriptlang.org/docs/handbook/enums.html#reverse-mappings
                                     return typeof value === 'string'

modules/utils/src/path.ts

@@ -59,3 +59,7 @@ export async function readdirR (item: string, reduceOutput: string[]): Promise<v
         reduceOutput.push(item);
     }
 }
+
+export function makePathEqualityKey(path: string): string {
+    return process.platform === 'win32' ? path.toLocaleLowerCase() : path;
+}

package.json

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

test/build-engine/__snapshots__/engine-js.test.ts.snap

@@ -1940,6 +1940,28 @@ exports[`engine-js inline enum 2`] = `
             console.log(4026531840);
             console.log(268435455);
 
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            console.log("Testing enum override .......");
+            var TestOverrideEnum;
+            (function (TestOverrideEnum) {
+              TestOverrideEnum["HAHA"] = "hello_haha";
+              TestOverrideEnum["WOWO"] = "hello_wowo";
+              TestOverrideEnum["HEIHEI"] = "hello_heihei";
+            })(TestOverrideEnum || (TestOverrideEnum = {}));
+
+            console.log(2);
+            console.log(0);
+            console.log(1);
+
             var MyEnum2 = exports("MyEnum2", {
               "AAABBB": 0,
               "Haha": 0,

test/test-engine-source/cc.config.json

@@ -53,10 +53,19 @@
                 "internal:native": "./audio/internal-native.ts"
             }
         },
+        {
+            "test": "true",
+            "isVirtualModule": false,
+            "overrides": {
+                "./cocos/enums/TestOverride.ts": "./cocos/enums/TestOverride.jsb.ts",
+                "./cocos/enums/test-override-entry.ts": "./cocos/enums/test-override-entry.jsb.ts"
+            }
+        },
         {
             "test": "context.mode === 'BUILD'",
             "isVirtualModule": false,
-            "overrides": {}
+            "overrides": {
+            }
         },
         {
             "test": "context.buildTimeConstants.NATIVE",
@@ -428,8 +437,7 @@
 
     "treeShake": {
         "noSideEffectFiles": [
-            "cocos/serialization/instantiate-jit.ts",
-            "cocos/game/splash-screen.ts"
+            "cocos/serialization/instantiate-jit.ts"
         ]
     }
 }

test/test-engine-source/cocos/enums/TestOverride.jsb.ts

@@ -0,0 +1,17 @@
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+console.log(`Testing enum override .......`);
+export enum TestOverrideEnum {
+    HAHA = 'hello_haha',
+    WOWO = 'hello_wowo',
+    HEIHEI = 'hello_heihei'
+}
+

test/test-engine-source/cocos/enums/TestOverride.ts

@@ -0,0 +1,14 @@
+import { MyEnum4 } from './my-enum-4';
+
+export enum TestOverrideEnum {
+    HAHA,
+    WOWO,
+    HEIHEI
+}
+
+console.log(TestOverrideEnum.WOWO);
+console.log(TestOverrideEnum.HAHA);
+console.log(TestOverrideEnum.HEIHEI);
+
+
+console.log(MyEnum4.AAABBB, MyEnum4.Haha, MyEnum4.Hi);

test/test-engine-source/cocos/enums/index.ts

@@ -4,6 +4,7 @@ import * as gfx from './define';
 
 import { PixelFormat } from './define';
 
+import './test-override-entry';
 
 export enum MyEnum2 {
     AAABBB,

test/test-engine-source/cocos/enums/test-override-entry.jsb.ts

@@ -0,0 +1,4 @@
+import * as Override from './TestOverride';
+console.log(Override.TestOverrideEnum.HEIHEI);
+console.log(Override.TestOverrideEnum.HAHA);
+console.log(Override.TestOverrideEnum.WOWO);

test/test-engine-source/cocos/enums/test-override-entry.ts

@@ -0,0 +1,8 @@
+
+
+import * as Override from './TestOverride';
+
+console.log(Override.TestOverrideEnum.WOWO);
+console.log(Override.TestOverrideEnum.HAHA);
+console.log(Override.TestOverrideEnum.HEIHEI);
+