TS1362 'GLTFLoader' cannot be used as a value because it was exported using 'export type'.

[rosskevin]

• three-stdlib version: 2.35.11

Problem description:

Unable to instantiate in a module/moduleResolution nodenext due to changes in patch-ts

Relevant code:

import { GLTFLoader } from 'three-stdlib'

const gltfLoader = new GLTFLoader()

I'm looking at this @CodyJasonBennett - thank you for the effort so far. I'll experiment a bit on my side.

[rosskevin]

@CodyJasonBennett I'm not sure we need type added to everything, or if we want to blanket sed everything, we need to duplicate (the line with type * and *)

This works in index.d.ts

export * from './loaders/GLTFLoader.d.ts';

and this works as well

export type * from './loaders/GLTFLoader.d.ts';
export * from './loaders/GLTFLoader.d.ts';

export type as a blanket option helps with only type-based files, but we still need the * without type to export classes.

[CodyJasonBennett]

Then let's remove it. I was under the impression it was needed when exporting from annotations.

[AbdBarho]

The following error I have now could be related to this issue:

Failed to compile.
./node_modules/three-stdlib/csm/CSMFrustum.d.ts:13:13
Type error: Identifier expected. 'default' is a reserved word that cannot be used here.
  11 | }
  12 |
> 13 | export type default class CSMFrustum {
     |             ^
  14 |   constructor(data?: CSMFrustumParameters)
  15 |   vertices: CSMFrustumVerticies
  16 |   setFromProjectionMatrix(projectionMatrix: Matrix4, maxFar: number): CSMFrustumVerticies

[CodyJasonBennett]

I reverted the typescript changes which leaves us solely with #342 again. I think I need to express this issue upstream or extend the TypeScript compiler directly.