Skip to content

feat: bump @grpc/grpc-js minimum from ^1.9.3 to ^1.12.5#742

Draft
nelson-parente wants to merge 1 commit intodapr:mainfrom
nelson-parente:feat/grpc-js-update
Draft

feat: bump @grpc/grpc-js minimum from ^1.9.3 to ^1.12.5#742
nelson-parente wants to merge 1 commit intodapr:mainfrom
nelson-parente:feat/grpc-js-update

Conversation

@nelson-parente
Copy link
Copy Markdown

Summary

  • Bump @grpc/grpc-js minimum from ^1.9.3 to ^1.12.5
  • Regenerate package-lock.json (resolves to 1.14.3)

Motivation

The previous ^1.9.3 floor allowed installing grpc-js versions with known issues on Node.js 22 LTS. While semver already resolves to 1.14.3, the low floor is misleading — it implies 1.9.x is a supported baseline. Bumping to ^1.12.5 documents the actual minimum tested version and ensures:

  • Node.js 22 LTS compatibility
  • Improved channel connectivity state management
  • Better error messages for connection failures
  • Performance improvements in call credential handling

Test plan

  • Run npm install to verify dependency resolution
  • Run npm run test:unit:all to verify no regressions
  • Run E2E tests to verify gRPC communication with Dapr sidecar

Raise the @grpc/grpc-js floor version to 1.12.5 which includes:
- Node.js 22 LTS compatibility fixes
- Improved channel connectivity state management
- Better error messages for connection failures
- Performance improvements in call credential handling

The previous ^1.9.3 range technically resolves to 1.14.3 via semver,
but the low floor allowed installing versions with known issues on
Node 22. Bumping to ^1.12.5 ensures a minimum version that is tested
and stable on Node 22 LTS (used in CI).

Signed-off-by: Nelson Parente <nelson_parente@live.com.pt>
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d1bf38a) to head (159f8e8).
⚠️ Report is 34 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #742   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            6         6           
  Branches         1         1           
=========================================
  Hits             6         6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant