Transform BigInt syntax into BigInt calls (12n --> BigInt(12))

[cyan-2048]

currently when using BigInt and having the target environment to be es6 you get this error

however no errors occur when you do BigInt(0)

this is actually what some libraries do to test BigInt support, although sometimes for some reason some test BigInt by using the syntax (weird? maybe they weren't informed that using newer syntax on older environments results to everything else in the code not working)

slight suggestion: add a warning if this transform happens as this will still create errors when using on environments where BigInt does not exist

another suggestion: when the BigInt can be represented as a normal JS number (1, 2, anything less than the Number.MAX_SAFE_INTEGER) transform it to BigInt(n) else wrap it in a string BigInt("n")

[evanw]

this is actually what some libraries do to test BigInt support, although sometimes for some reason some test BigInt by using the syntax (weird? maybe they weren't informed that using newer syntax on older environments results to everything else in the code not working)

Which libraries specifically? This sounds like a bug with the library.

[cyan-2048]

this is actually what some libraries do to test BigInt support, although sometimes for some reason some test BigInt by using the syntax (weird? maybe they weren't informed that using newer syntax on older environments results to everything else in the code not working)

Which libraries specifically? This sounds like a bug with the library.

yes the library is also at fault, however this transform would definitely make life easier.

The library I was using was using an outdated version of nodejs/readable-stream
https://github.com/javascriptke/bun/blob/5108e3e0d9f7806c3884e5a678a81dedd61791bc/src/js/node/stream.ts#L605
(this is not the library but I could not find it so I'm citing a different source)

This syntax can also be found in the buffer npm package
https://github.com/feross/buffer/blob/02c782a9127b0fea99c74465c046d4af7e3c839d/index.js#L1528
some libraries use an older version of this package

If this syntax is not transformed to something older runtimes support, the entire code will not run.