Optimize code size for draw-batch.ts and batcher-2d.ts, remove needless setter/getter.

[dumganhar]

Re: #18222

Changelog

---

Continuous Integration

This pull request:

• needs automatic test cases check.

  Manual trigger with @cocos-robot run test cases afterward.

• does not change any runtime related code or build configuration

  If any reviewer thinks the CI checks are needed, please uncheck this option, then close and reopen the issue.

---

Compatibility Check

This pull request:

• changes public API, and have ensured backward compatibility with deprecated features.
• affects platform compatibility, e.g. system version, browser version, platform sdk version, platform toolchain, language version, hardware compatibility etc.
• affects file structure of the build package or build configuration which requires user project upgrade.
• introduces breaking changes, please list all changes, affected features and the scope of violation.

[github-actions]

👍 Package size ⤵ -1220 bytes, old: 5226328, new: 5225108

Interface Check Report

! WARNING this pull request has changed these public interfaces:

@@ -61331,15 +61331,10 @@
             allocateChunk(vertexCount: number, indexCount: number): _cocos_2d_renderer_static_vb_accessor__StaticVBChunk | null;
             recycleChunk(chunk: _cocos_2d_renderer_static_vb_accessor__StaticVBChunk): void;
             static generateID(): number;
         }
+        /** @mangle */
         export class _cocos_2d_renderer_draw_batch__DrawBatch2D {
-            get inputAssembler(): gfx.InputAssembler | null;
-            set inputAssembler(ia: gfx.InputAssembler | null);
-            get descriptorSet(): gfx.DescriptorSet | null;
-            set descriptorSet(ds: gfx.DescriptorSet | null);
-            get visFlags(): number;
-            set visFlags(vis: number);
             get passes(): renderer.Pass[];
             get shaders(): gfx.Shader[];
             model: renderer.scene.Model | null;
             texture: gfx.Texture | null;
@@ -61347,8 +61342,11 @@
             useLocalData: Node | null;
             isStatic: boolean;
             textureHash: number;
             samplerHash: number;
+            visFlags: number;
+            inputAssembler: gfx.InputAssembler | null;
+            descriptorSet: gfx.DescriptorSet | null;
             destroy(ui: _cocos_2d_renderer_i_batcher__IBatcher): void;
             clear(): void;
             fillPasses(mat: Material | null, dss: gfx.DepthStencilState | null, dssHash: number, patches: Readonly<renderer.IMacroPatch[] | null>): void;
         }

[dumganhar]

@cocos-robot run test cases

[github-actions]

@dumganhar, Please check the result of run test cases:

• Test Platform: PR-Test
• Editor Version: v3.8.6
• Task Result: PASS
• Task URL: https://cctest.cocos.org/#/reportview/PR-TEST/01fab358-d43e-4c1f-974b-e4fa51cb6a4d/-1
• GitHub Action: https://github.com/cocos/cocos-engine/actions/runs/12910377052

Task Details

[github-actions]

@dumganhar, Please check the result of run test cases:

• Test Platform: PR-Test
• Editor Version: v3.8.6
• Task Result: PASS
• Task URL: https://cctest.cocos.org/#/reportview/PR-TEST/40379136-e272-4fc1-98cb-4bb0459e86c8/-1
• GitHub Action: https://github.com/cocos/cocos-engine/actions/runs/12910377052

Task Details

[minggo]

Why have to change it?

[dumganhar]

It's because we're using babel to compile ts to js.
babel has its own getter/setter accessor definition which could not be understood by terser minifier.

See this issue: terser/terser#322

Since LocalDescriptorSet is an internal module, change the get descriptSet to function getDescriptorSet is fine for us.

[minggo]

Ok, it is better to add a description.

[dumganhar]

Added comments.