Skip to content

feat(proto-loader): support longs: BigInt for int64 fields - #3072

Closed
brzosthub wants to merge 1 commit into
grpc:masterfrom
brzosthub:feat/proto-loader-bigint-longs
Closed

feat(proto-loader): support longs: BigInt for int64 fields#3072
brzosthub wants to merge 1 commit into
grpc:masterfrom
brzosthub:feat/proto-loader-bigint-longs

Conversation

@brzosthub

@brzosthub brzosthub commented Aug 11, 2026

Copy link
Copy Markdown

Summary

  • Allow longs: BigInt in @grpc/proto-loader (runtime options, CLI, and proto-loader-gen-types)
  • Bump protobufjs floor to ^7.6.0 (BigInt conversion support)
  • Add a decode test for int64/uint64/sint64/fixed64/sfixed64 as native bigint

Closes #3047

Depends on protobufjs BigInt conversion from protobufjs/protobuf.js#2257 (backported in 7.6.0).

Test plan

  • npm run compile in packages/proto-loader
  • mocha build/test/**/*.js — includes new BigInt deserialize test
  • npm run validate-golden

@linux-foundation-easycla

linux-foundation-easycla Bot commented Aug 11, 2026

Copy link
Copy Markdown

CLA Not Signed

@brzosthub
brzosthub force-pushed the feat/proto-loader-bigint-longs branch 2 times, most recently from 281b336 to 3842bd3 Compare August 11, 2026 18:57
Expose protobufjs BigInt conversion (available since 7.6.0) through
@grpc/proto-loader options, CLI, and generated TypeScript types.
Fixes #3047.
@brzosthub
brzosthub force-pushed the feat/proto-loader-bigint-longs branch from 3842bd3 to b2b7572 Compare August 11, 2026 18:57
@brzosthub

Copy link
Copy Markdown
Author

Closing to recreate without leaked author email history.

@brzosthub brzosthub closed this Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow configuration to choose BigInt over Long.js for int64 types

1 participant