Update assets-controllers to version ^9.0.0

[tommasini]

What is this about?

The assets-controllers can be updated independently to version ^9.0.0.

We will need to rebuild the patch manually, we can follow up the changes made on this core branch, that represents the assets-controllers on the earliest version.

The files that were changed were:

• AccountTrackerController: https://github.com/MetaMask/core/pull/1146/files#diff-917e371461a96a1d0cd3b06add0191b8a6b3d505e961a82d86ca4483c67a6a30
• AssetsUtils - https://github.com/MetaMask/core/pull/1173/files
• NftController - https://github.com/MetaMask/core/pull/1173/files , https://github.com/MetaMask/core/pull/1406/files, https://github.com/MetaMask/core/pull/1455/files
• NftDetectionController - https://github.com/MetaMask/core/pull/1417/files#diff-30fbe131e95d33ea72848ab3efd6f0a34de27eebf85cdf6f749b814fe2ad4f69

Breaking Changes

• Add logic to support validating and adding ERC721 and ERC1155 tokens to NFTController state via wallet_watchAsset API. (https://github.com/MetaMask/core/pull/1173, https://github.com/MetaMask/core/pull/1406)
  • The NFTController now has a new watchNFT method that can be called to send a message to the ApprovalController and prompt the user to add an NFT to their wallet state.
  • The NFTController now requires an instance of a ControllerMessenger to be passed to its constructor. This is messenger is used to pass the watchNFT message to the ApprovalController.

Scenario

No response

Design

No response

Technical Details

• Upgrade assets-controllers to v^9.0.0
• Re-build the patch of assets-controllers
• Add messenger prop to the NftController initialization on Engine file

Threat Modeling Framework

No response

Acceptance Criteria

• [ ] All old features remain working as expected
• [ ] Assets-controller upgrade to version ^9.0.0
• [ ] Links to bitrise E2E test pipeline are added to the ticket description
• [ ] Ensure that all scenarios have passed.

Stakeholder review needed before the work gets merged

• [X] Engineering (needed in most cases)
• [ ] Design
• [ ] Product
• [X] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
• [ ] Security
• [ ] Legal
• [ ] Marketing
• [ ] Management (please specify)
• [ ] Other (please specify)

References

• Changelog core v55 - https://github.com/MetaMask/core/releases/tag/v55.0.0

[tommasini]

Hey team! Please add your planning poker estimate with Zenhub @Andepande @Cal-L @chrisleewilcox @cortisiko @frankvonhoven @jpcloureiro @MarioAslau @NicolasMassart @SamuelSalas