BigInt in node.js

[j3bb9z]

Hi,

New versions of Node.js (10.4.0+) have BigInt support.

Just wanted to ask - are you going to make use of it?

[Cryrivers]

Would like to highlight that TypeScript also supports bigint with the feature flag enabled. So do please consider supporting bigint in TypeScript declaration as well.

[gerardmrk]

Unable to assign the return value of process.hrtime.bigint() to the protobuf type of uint64 as well. the generated type for uint64 is number|Long|null.

[betamos]

Noticed a couple of issues, this was in a node v12.13.0 environment:

1. Encoding from a BigInt (which fits) to a uint64 field results in a Long with the value 0. This is dangerous.
2. Encoding from a BigInt (which fits) to a uint32 field results in a crash: TypeError: Cannot mix BigInt and other types, use explicit conversions.
3. If Long adds support for BigInt, (1) might be fixed as a side-effect.

[mjgp2]

I opened a PR if you have comments - #1557

[mahnunchik]

Year 2023, BigInt supported by 95.14% of browsers https://caniuse.com/bigint

Can we have landed native BigInt to protobuf.js?