[TS] Using type only imports to use types "normally"? about nakama-common HOT 5 OPEN

What I did was add:

/// <reference types="nakama-runtime" />

To the top of my index.ts file for the server, which brought in the imports without a tsconfig change.

I debated adding export = nkruntime at the end of the .d.ts in this file and then submitting those changes here which would allow for import type nkruntime from "nakama-runtime". But if all of the globals are always in the runtime, then it felt like allowing for the import is a bit unrealistic and left it at that.

from nakama-common.

@MarcusRiemer Happy to take on this advice. Can you explain what you mean by dangerous about adding to the typeRoots? I'm not a TypeScript export so any help is appreciated.

from nakama-common.

"Dangerous" in the sense of "Marcus was once bitten by a globally declared conflicting var Observable: any that some unrelated package provided and therefor forgot to import RxJS after copy and pasting stuff around which led to surprising errors at runtime".

If I understand correctly the nkruntime functions are exposed to Typescript but are implemented in Go? So they are guaranteed to be available, even though there is no visible implementation? In that case the type only imports should perfectly fit this usecase, as they are purely compile time constructs and therefore never surface in the compiled JavaScript.

from nakama-common.

fwiw: I ran through most of the Typescript setup process (including Rollup, excluding building via docker) with the import type { nkruntime } from "nakama-runtime"; and the hacked index.d.ts and everything seems to work fine. I was a little puzzled that everything also works with the normal import statement which is probably due to the fact that all callbacks receive the relevant instances as arguments anyway. So I guess its possible to just use a normal import, but I will see what happens if I try to do some more interesting things.

from nakama-common.

That's also how I solved it (with the old style reference)

from nakama-common.