Release/892.0.0#8340
Conversation
|
|
||
| ### Changed | ||
|
|
||
| - **BREAKING:** Bump `@metamask/bridge-controller` from `^69.2.3` to `^70.0.0` ([#8340](https://github.com/MetaMask/core/pull/8340)) |
There was a problem hiding this comment.
Sorry one more thing. What is breaking about this dependency bump, i.e., what change needs to be made in either the extension or mobile app to accommodate this change (assuming that change has not already been made)? Or, does this dependency bump cause any incompatible changes to types exported by this package? It would be helpful to explain this, if only for posterity.
(Same for transaction-pay-controller)
There was a problem hiding this comment.
The main thing is that bridge-controller has a new state field and if we bump the version on the clients then the E2E tests will break due to this new non-optional field.
There was a problem hiding this comment.
That seems like that describes the breaking change for bridge-controller. You can't upgrade this package in a client without needing to add a migration or update the tests. But if you just upgraded bridge-status-controller in a client without also upgrading bridge-controller, would the e2e tests break or would CI fail in some other way? Or same for transaction-pay-controller?
Explanation
This releases
BridgeController,BridgeStatusControllerandTransactionPayControllerReferences
Checklist
Note
Medium Risk
Primarily a release/version bump, but it publishes
@metamask/bridge-controller@70.0.0, which includes a breaking state shape addition (quoteStreamComplete) that downstream consumers must handle.Overview
Release bump across bridge-related packages. This increments the monorepo version to
892.0.0and releases@metamask/bridge-controllerto70.0.0(documenting a breaking newquoteStreamCompletefield onBridgeControllerState).Downstream packages are revved to consume the new bridge-controller release:
@metamask/bridge-status-controllerto70.0.4and@metamask/transaction-pay-controllerto19.0.1, updating their dependency ranges andyarn.lockaccordingly.Written by Cursor Bugbot for commit 4e698f0. This will update automatically on new commits. Configure here.