MathUtils does not support treeshake.

[linbingquan]

Description

As the title says.

Reproduction steps

1. pnpm create vite@latest app --template vanilla
2. replace code to main.js
3. pnpm build

// main.js
import { MathUtils } from "three";

const v = MathUtils.clamp(0.5, 0, 1);
console.log(v);

Code

// main.js
import { MathUtils } from "three";

const v = MathUtils.clamp(0.5, 0, 1);
console.log(v);

Live example

None

Screenshots

No response

Version

dev

Device

Desktop

Browser

No response

OS

Windows

[linbingquan]

@Mugen87 MathUtils.* method does not support treeshake. Whether we want to export its methods.

--- a/src/Three.js
+++ b/src/Three.js
@@ -111,7 +111,7 @@ 
 export { Triangle } from './math/Triangle.js';
-export { MathUtils } from './math/MathUtils.js';
+export * from './math/MathUtils.js';
 export { Spherical } from './math/Spherical.js';

[mrdoob]

Is this issue about MathUtils or about ColorManagement?

[linbingquan]

Is this issue about MathUtils or about ColorManagement?

About MathUtils. There are two subproblems in this issue, but I didn't actually mention MathUtils before ColorManagement.

I'm not sure if we will do this, I can create a new issue if necessary.

[Mugen87]

I don't think we need to support tree-shaking on that level. MathUtils isn't a large component...