Skip to content

fix(deps): update dependency bn.js to ^5.2.5 - #10362

Merged
cryptodev-2s merged 2 commits into
mainfrom
renovate/bn.js-5.x
Sep 22, 2026
Merged

cryptodev-2s merged 2 commits into
mainfrom
renovate/bn.js-5.x

Conversation

@metamask-ci

@metamask-ci metamask-ci Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
bn.js ^5.2.1 → ^5.2.5 age confidence
@types/bn.js (source) ^5.1.5 → ^5.2.0 age confidence

Release Notes

indutny/bn.js (bn.js)

v5.2.5

Compare Source

  • fix: correct divRound rounding for negative operands (#​320)
  • fix: normalize sign when imuln multiplies by zero (#​321)

v5.2.4

Compare Source

  • fix: clear surplus words in inotn so notn(width) stays below 2 ** width (#​319)

v5.2.3

Compare Source


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate CLI.


Note

Medium Risk
bn.js backs gas and amount math in transaction-related controllers; patch fixes can change numeric edge-case behavior even without API changes.

Overview
Bumps bn.js from ^5.2.1 to ^5.2.5 and @types/bn.js from ^5.1.5 to ^5.2.0 across six workspace packages (assets-controllers, controller-utils, gas-fee-controller, transaction-controller, transaction-pay-controller, user-operation-controller), with matching Unreleased changelog entries and yarn.lock resolution updates (runtime bn.js 5.2.5, types 5.2.0).

There are no application source changes—only dependency manifests, changelogs, and the lockfile. The newer bn.js releases are patch-level fixes (e.g. divRound with negative operands, imuln sign when multiplying by zero, bitwise helpers).

Reviewed by Cursor Bugbot for commit 463dbd0. Bugbot is set up for automated code reviews on this repo. Configure here.

@metamask-ci
metamask-ci Bot requested review from a team as code owners September 22, 2026 15:37
@metamask-ci
metamask-ci Bot deployed to dependabot September 22, 2026 15:37 Active
@metamask-ci
metamask-ci Bot deployed to default-branch September 22, 2026 15:38 Active
@socket-security

socket-security Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​types/​bn.js@​5.1.6 ⏵ 5.2.01001007480100
Updatedbn.js@​5.2.2 ⏵ 5.2.5100 +1100 +210087 -2100

View full report

@cryptodev-2s
cryptodev-2s added this pull request to the merge queue Sep 22, 2026
Merged via the queue into main with commit 11195f0 Sep 22, 2026
248 checks passed
@cryptodev-2s
cryptodev-2s deleted the renovate/bn.js-5.x branch September 22, 2026 16:01
@OGPoyraz OGPoyraz mentioned this pull request Sep 23, 2026
1 of 4 tasks
pull Bot pushed a commit to Reality2byte/core that referenced this pull request Sep 23, 2026
## Explanation

Release `1275.0.0` with a **major** version bump for
`@metamask/transaction-controller` and a **minor** version bump for
`@metamask/transaction-pay-controller`.

- **`@metamask/transaction-controller`** `70.1.0` → `71.0.0` (major)
- **`@metamask/transaction-pay-controller`** `29.0.2` → `29.1.0` (minor)

### `@metamask/transaction-controller@71.0.0`

**Breaking:** Move sponsorship and signing decisions to optional
approval-time `isSponsored` and `shouldSign` hooks
([MetaMask#10109](MetaMask#10109))
- The hooks default to non-sponsored and local signing when omitted.
Sponsored transactions skip the `shouldSign` hook and local signing,
while transactions that skip local signing retain an existing nonce and
require a publish hook.
- `isGasFeeSponsored` and `isExternalSign` remain in the public types as
deprecated compatibility properties but no longer control the
transaction lifecycle; `isGasFeeSponsored` remains available as
migration metadata.
- Add `TransactionMeta.isGasFeeSponsoredAvailable` and refresh it during
approval preparation when simulation is enabled so sponsorship hooks
receive current availability without overriding simulation preferences.

Other changes:
- Add `membershipSubscription` transaction type
([MetaMask#10340](MetaMask#10340))
- Bump `bn.js` from `^5.2.1` to `^5.2.5`
([MetaMask#10362](MetaMask#10362))

### `@metamask/transaction-pay-controller@29.1.0`

- Support subsidized max Relay deposits using atomic `EXACT_OUTPUT`
quotes with transaction calls embedded, gated by
`payStrategies.relay.atomicMaxEnabled`
([MetaMask#10224](MetaMask#10224))
- Bump `bn.js` from `^5.2.1` to `^5.2.5`
([MetaMask#10362](MetaMask#10362))
- Bump `immer` from `^9.0.6` to `^9.0.21`
([MetaMask#10331](MetaMask#10331))

### Dependency updates

16 packages had their `@metamask/transaction-controller` dependency
range updated to `^71.0.0` with `package.json` changes under this
release branch. Changelog entries for those packages will be generated
by commenting `@metamaskbot update-changelogs` on this PR.

## References

- [MetaMask#10109](MetaMask#10109) — Move
sponsorship and signing decisions to optional approval-time hooks
- [MetaMask#10340](MetaMask#10340) — Add
`membershipSubscription` transaction type
- [MetaMask#10224](MetaMask#10224) — Support
subsidized max Relay deposits using atomic `EXACT_OUTPUT` quotes

## Checklist

- [ ] I've updated the test suite for new or updated code as appropriate
- [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've communicated my changes to consumers by [updating changelogs
for packages I've
changed](https://github.com/MetaMask/core/tree/main/docs/processes/updating-changelogs.md)
- [ ] I've introduced [breaking
changes](https://github.com/MetaMask/core/tree/main/docs/processes/breaking-changes.md)
in this PR and have prepared draft pull requests for clients and
consumer packages to resolve them

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **High Risk**
> The major transaction-controller release changes core transaction
approval, sponsorship, and signing behavior across many downstream
packages that depend on it.
> 
> **Overview**
> Release **`1275.0.0`** cuts published versions for
**`@metamask/transaction-controller`** `71.0.0` (major) and
**`@metamask/transaction-pay-controller`** `29.1.0` (minor), and bumps
the monorepo root version accordingly.
> 
> The transaction-controller release documents **breaking**
approval-time behavior: sponsorship and signing now flow through
optional **`isSponsored`** / **`shouldSign`** hooks instead of
**`isGasFeeSponsored`** / **`isExternalSign`**, plus a new
**`membershipSubscription`** transaction type. This PR mostly
**propagates** that major by setting dependent packages’
`@metamask/transaction-controller` ranges to **`^71.0.0`**, updating
**`yarn.lock`**, and recording bump entries in package changelogs
(including transaction-pay’s dependency bump under `29.1.0`).
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
b93ad54. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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