V3.8.1

.eslintrc.yaml

@@ -11,8 +11,7 @@ extends:
     - plugin:@typescript-eslint/recommended
     - plugin:@typescript-eslint/recommended-requiring-type-checking
 
-plugins:
-    - '@typescript-eslint'
+plugins: ["@typescript-eslint"]
 
 settings:
     import/resolver:
@@ -62,6 +61,7 @@ rules:
     quotes: [warn, single, { allowTemplateLiterals: true }] # force single, but allow template literal
     no-else-return: off # else-return is a common pattern which clearly expresses the control flow
     no-unused-expressions: off # taken over by '@typescript-eslint/no-unused-expressions'
+    no-empty-function: off # taken over by '@typescript-eslint/no-empty-function'
 
     ##### AIRBNB-SPECIFIC RULE OVERRIDES #####
 
@@ -71,7 +71,14 @@ rules:
     import/extensions: off # typescript doesn't support this
     import/no-unresolved: off # TODO: fix internal modules
     import/prefer-default-export: off # prefer named exports
-    indent: off # use @typescript-eslint/indent instead for better compatibility
+    indent: [error, 4, {
+      SwitchCase: 0,
+      ignoredNodes: [ # https://stackoverflow.com/a/72897089
+          "FunctionExpression > .params[decorators.length > 0]",
+          "FunctionExpression > .params > :matches(Decorator, :not(:first-child))",
+          "ClassBody.body > PropertyDefinition[decorators.length > 0] > .key",
+      ]
+    }]
 
     lines-between-class-members: off # be more lenient on member declarations
     max-classes-per-file: off # helper classes are common
@@ -95,6 +102,7 @@ rules:
     prefer-destructuring: off # auto-fix is not smart enough to merge different instances
     linebreak-style: off # we don't enforce this on everyone's dev environment for now
     spaced-comment: off # for license declarations
+    default-case-last: off # Place default case clause to first make it more clear that this switch statement has handled all cases
 
     ##### TYPESCRIPT-SPECIFIC RULE OVERRIDES #####
 
@@ -104,22 +112,31 @@ rules:
         'ts-nocheck': true,
         'ts-check': false,
     }]
-    '@typescript-eslint/indent': [warn, 4, {
-        SwitchCase: 0
-    }]
     '@typescript-eslint/no-unused-expressions': warn
 
     # TODO: this is just too much work
     '@typescript-eslint/explicit-module-boundary-types': off
 
+    # NOTE: We don't want to rely on TS automatic type inference
+    '@typescript-eslint/no-inferrable-types': off
+
     # TODO: sadly we still rely heavily on legacyCC
     '@typescript-eslint/no-unsafe-assignment': off
     '@typescript-eslint/no-unsafe-call': off
     '@typescript-eslint/no-unsafe-member-access': off
 
     '@typescript-eslint/unbound-method': off # we exploit prototype methods sometimes to acheive better performace
     '@typescript-eslint/no-explicit-any': off # still relevant for some heavily templated usages
-    '@typescript-eslint/no-empty-function': off # may become useful in some parent classes
+
+    '@typescript-eslint/no-empty-function': [error, {
+      allow: [
+        private-constructors,
+        protected-constructors,
+        decoratedFunctions,
+        overrideMethods,
+      ]
+    }]
+
     '@typescript-eslint/no-unused-vars': off # may become useful in some parent classes
     '@typescript-eslint/no-non-null-assertion': off # sometimes we just know better than the compiler
     '@typescript-eslint/no-namespace': [warn, { # we need to declare static properties
@@ -143,3 +160,6 @@ rules:
 
     # Prefer the interface style.
     '@typescript-eslint/consistent-type-definitions': [error, interface]
+    '@typescript-eslint/explicit-function-return-type': [error, {
+      allowIIFEs: true, # IIFEs are widely used, writing their signature twice is painful
+    }]

.github/workflows/run_test_cases.yml

@@ -1,12 +1,10 @@
 name: run test cases
 
 on:
-  pull_request:
+  pull_request_target:
   issue_comment:
     types: [created, edited]
 
-permissions: read-all
-
 # github.head_ref is only defined on pull_request events
 concurrency:
   group: ${{ github.workflow }}-${{ github.actor }}-${{ github.head_ref || github.run_id }}
@@ -281,38 +279,16 @@ jobs:
           JOBID=`cat  /usr/local/lib/node_modules/@cctest/scheduler/logs/PR-TEST/nowJobId.log`
           cd ..
           python3 -u ./python/main.py --target=job_editor --jobid=$JOBID
-      - name: Write comment body and issue number to file
-        uses: actions/github-script@v5
-        env:
-          PR_AUTHOR: ${{ steps.parse_pr.outputs.pr_author }}
-          EDITOR_VERSION: ${{ steps.run_test_cases.outputs.editor }}
-          TASK_STATUS: ${{ steps.run_test_cases.outputs.status }}
-          TASK_REPORT: ${{ steps.run_test_cases.outputs.report }}
-          JOB_ID: ${{ steps.run_test_cases.outputs.jobid }}
-          GITHUB_RUN_ID: ${{ github.run_id }}
-          PR_NUMBER: ${{ steps.get_pr.outputs.pr_number }}
+      - name: Update Comment
+        uses: peter-evans/create-or-update-comment@v1
         with:
-          script: |
-            const fs = require('fs');
-            fs.writeFileSync('pr.txt', process.env.PR_NUMBER);
-            fs.writeFileSync('comment.txt', `@${process.env.PR_AUTHOR}, Please check the result of \`run test cases\`:
+          issue-number: ${{ steps.get_pr.outputs.pr_number }}
+          body: |
+            @${{ steps.parse_pr.outputs.pr_author }}, Please check the result of `run test cases`:
             - Test Platform: PR-Test
-            - Editor Version: ${process.env.EDITOR_VERSION}
-            - Task Result: ${process.env.TASK_STATUS}
-            - Task URL: http://cctest.cocos.org/#/reportview/PR-TEST/${process.env.JOB_ID}/-1
-            - GitHub Action: https://github.com/cocos/cocos-engine/actions/runs/${process.env.GITHUB_RUN_ID}
+            - Editor Version: ${{ steps.run_test_cases.outputs.editor }}
+            - Task Result: ${{ steps.run_test_cases.outputs.status }}
+            - Task URL: http://cctest.cocos.org/#/reportview/PR-TEST/${{ steps.run_test_cases.outputs.jobid }}/-1
+            - GitHub Action: https://github.com/cocos/cocos-engine/actions/runs/${{ github.run_id }}
             ## Task Details
-            ${process.env.TASK_REPORT}`);
-      - name: Upload artifact
-        uses: actions/upload-artifact@v2
-        with:
-          name: pr.txt
-          path: |
-            pr.txt
-      - name: Upload artifact
-        uses: actions/upload-artifact@v2
-        with:
-          name: comment.txt
-          path: |
-            comment.txt
-
+            ${{ steps.run_test_cases.outputs.report }}

.github/workflows/web-interface-check.yml

@@ -1,15 +1,13 @@
 name: <Web> Interface check
 
 #on: push
-on: [pull_request]
+on: [pull_request_target]
 
 # github.head_ref is only defined on pull_request events
 concurrency:
   group: ${{ github.workflow }}-${{ github.actor }}-${{ github.head_ref || github.run_id }}
   cancel-in-progress: true
 
-permissions: read-all
-
 jobs:
   interface_check:
     if: 
@@ -81,9 +79,6 @@ jobs:
           cat ./interface-diff.txt
           node ./.github/workflows/interface-check-report.js
 
-      - name: Upload interface-diff.txt artifact
-        uses: actions/upload-artifact@v2
+      - uses: marocchino/sticky-pull-request-comment@v2
         with:
-          name: interface-diff.txt
           path: ./engine/interface-diff.txt
-

.github/workflows/web-quality.yml

@@ -18,14 +18,14 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-          node-version: 12
+          node-version: '16.x'
 
       - name: Download external
         run: |
           EXT_VERSION=`node ./.github/workflows/get-native-external-version.js`
           git clone --branch $EXT_VERSION --depth 1 https://github.com/cocos/cocos-engine-external native/external
 
-      - run: npm ci
+      - run: npx npm@6 ci
 
       - name: Run ESLint
         shell: pwsh

.gitignore

@@ -46,6 +46,7 @@ native/external/
 
 scripts/typedoc-plugin/lib/
 !templates/**/*
+templates/**/.DS_Store
 
 @types/consts.d.ts
 

@types/pal/input.d.ts

@@ -52,6 +52,7 @@ declare module 'pal/input' {
      * Class designed for gamepad input
      */
     export class GamepadInputDevice {
+        private constructor(deviceId: number);
         /**
          * @engineInternal
          */

@types/pal/screen-adapter.d.ts

@@ -79,12 +79,6 @@ declare module 'pal/screen-adapter' {
          */
         public get resolution (): import('cocos/core/math').Size;
 
-        /**
-         * Update the resolution by resolutionScale.
-         * This method rebuilds the size of frame buffer and the size of canvas.
-         * This method should be called when window resized (fullscreen changing included) or the resolution scale changed.
-         */
-        private _updateResolution ();
         /**
          * Get and set the resolution scale of screen, which will affect the quality of the rendering.
          * Note: if this value is set too high, the rendering performance of GPU will be reduced, this value is 1 by default.
@@ -123,4 +117,6 @@ declare module 'pal/screen-adapter' {
     }
 
     export const screenAdapter: ScreenAdapter;
+
+    export {};
 }

@types/pal/system-info.d.ts

@@ -57,10 +57,12 @@ declare module 'pal/system-info' {
          */
         public exit(): void;
 
-        on (event: PalSystemEvent, cb: (...args: any)=>void, target?: any): void;
-        off (event: PalSystemEvent, cb?: (...args: any)=>void, target?: any): void;
+        on (event: PalSystemEvent, cb: (...args: any) => void, target?: any): void;
+        off (event: PalSystemEvent, cb?: (...args: any) => void, target?: any): void;
         // TODO: support onError
     }
 
     export const systemInfo: SystemInfo;
+
+    export {};
 }

EngineErrorMap.md

@@ -3267,6 +3267,10 @@ PassUtils: illegal uniform handle, accessing uniform at offset %d
 
 Pass: setUniform is invoked with incompatible uniform data type for binding %d, expected type is %s
 
+### 12012
+
+Can't set a material instance to a sharedMaterial slot
+
 ### 12100
 
 The font size is too big to be fitted into texture atlas. Please switch to other label cache modes or choose a smaller font size.
@@ -3339,6 +3343,18 @@ Can not update a static mesh.
 
 The primitiveIndex is out of range.
 
+### 14202
+
+meshopt asm decoder initialized
+
+### 14203
+
+meshopt wasm decoder initialized
+
+### 14204
+
+meshopt decoder error: %d
+
 ### 14300
 
 Can not keep world transform due to the zero scaling of parent node

cc.config.json

@@ -98,6 +98,9 @@
         "physics-2d-builtin": {
             "modules": ["physics-2d-builtin", "physics-2d-framework"]
         },
+        "physics-2d-box2d-wasm": {
+            "modules": ["physics-2d-box2d-wasm", "physics-2d-framework"]
+        },
         "intersection-2d": {
             "modules": ["intersection-2d"]
         },

cocos/2d/assembler/graphics/helper.ts

@@ -35,7 +35,7 @@
 
 const KAPPA90 = 0.5522847493;
 
-export function arc (ctx: Impl, cx: number, cy: number, r: number, startAngle: number, endAngle: number, counterclockwise: boolean) {
+export function arc (ctx: Impl, cx: number, cy: number, r: number, startAngle: number, endAngle: number, counterclockwise: boolean): void {
     counterclockwise = counterclockwise || false;
 
     let a = 0;
@@ -97,7 +97,7 @@
     }
 }
 
-export function ellipse (ctx: Impl, cx: number, cy: number, rx: number, ry: number) {
+export function ellipse (ctx: Impl, cx: number, cy: number, rx: number, ry: number): void {
     ctx.moveTo(cx - rx, cy);
     ctx.bezierCurveTo(cx - rx, cy + ry * KAPPA90, cx - rx * KAPPA90, cy + ry, cx, cy + ry);
     ctx.bezierCurveTo(cx + rx * KAPPA90, cy + ry, cx + rx, cy + ry * KAPPA90, cx + rx, cy);
@@ -106,7 +106,7 @@
     ctx.close();
 }
 
-export function roundRect (ctx: Impl, x: number, y: number, w: number, h: number, r: number) {
+export function roundRect (ctx: Impl, x: number, y: number, w: number, h: number, r: number): void {
     if (r < 0.1) {
         ctx.rect(x, y, w, h);
     } else {
@@ -127,12 +127,12 @@
 }
 
 export function tesselateBezier (
     ctx: Impl, x1: number, y1: number,
     x2: number, y2: number,
     x3: number, y3: number,
     x4: number, y4: number,
     level: number, type: number,
-) {
+): void {
     let x12 = 0;
     let y12 = 0;
     let x23 = 0;