Avoid bare imports other than 'three'

[donmccurdy]

Related:

• NodeMaterial: Move new NodeMaterial system to ./jsm/nodes/ #23560

With r138 the following syntax was introduced:

import { NormalNode } from 'three-nodes/Nodes.js';

I do not think we should be using bare imports (except three) unless we are publishing it to a separate NPM package. Bundlers cannot resolve three-nodes without custom resolution configuration, and we're seeing confusion about this in Discord. By contrast, anything under the three import is fine if we configure it in package.json exports:

import { NormalNode } from 'three/nodes/Nodes.js';

/cc @sunag

[marcofugaro]

Ah yes, about this, I'm awaiting @mrdoob's response in #23406

[Mugen87]

It seems this bug is fixed with r144, right?

[donmccurdy]

Agreed, thanks!