Feat/bridge status controller

[infiniteflower]

Explanation

This PR adds a new controller: BridgeStatusController.

This controller handles the bridge transaction status fetching and polling from the Bridge API.

References

This is a port of the BridgeStatusController from Extension: https://github.com/MetaMask/metamask-extension/tree/main/app/scripts/controllers/bridge-status

Some minor changes were needed to fill in the missing functions and variables from Extension.

This package will be consumed initially by the Metamask Mobile application first. Eventually, we wish to migrate the Extension to use this core/bridge-status-controller package.

Very closely related to the BridgeController: #5276

Changelog

@metamask/bridge-status-controller

• ADDED: New BridgeStatusController!

@metamask/bridge-controller

• CHANGED: BridgeController FeeType enum now exported as an enum, not just a type

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
• I've highlighted breaking changes using the "BREAKING" category above as appropriate
• I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

[mcmire]

Had several comments on dependencies, exports, etc.

Would you mind re-running yarn update-readme-content? It looks like it might be missing updates to the dependency graph.

[mcmire]

Should we also export this type? Otherwise if another controller wanted to just be able to call, say BridgeStatusController:startPollingForBridgeTxStatus, then it wouldn't be possible to set up the messenger.

Alternatively, what are your thoughts on explicitly defining the action types BridgeStatusControllerStartPollingForBridgeTxStatusAction and BridgeStatusControllerWipeBridgeStatusAction? This would be consistent with how action/event types work in other controllers.

[infiniteflower]

Done in 33dab2e, added BridgeStatusControllerResetStateAction as well

[yalan2ny]

mcmir Thank you, please let me know if there is anything you need to add or add, thank you

[infiniteflower]

yarn update-readme-content

Done in 0b09fa9

[mcmire]

For future reviewers: there seems to be a bug in the Yarn constraints which forces any controller packages added as dependencies to also be added as peer dependencies. We should fix this.

[mcmire]

LGTM.

[yalan2ny]

I checked the merged file, the added features and changes cover and solve the problems in Metamask, especially the solution to the bridge problem, which was necessary, is available in the packages and will fix the bugs, thank you for the help and cooperation of all of you friends, who with your presence contribute to the development and progress of the community.