feat: NetworkController changes to support Network Syncing

[Prithpal-Sooriya]

Explanation

A set of changes to support network syncing. This is a big scary change, so will have a 1:1 with assets team and pair review this.

I'll also record a loom vid for a quick code walkthrough.

References

N/A

Changelog

@metamask/network-controller

• ADDED: logic to use added network networkClientId if provided instead of uuidV4()
• ADDED: new method dangerouslySetNetworkConfiguration to cautiously overwrite a given network configuration. Used for network syncing, where we need to override local state with the synced remote state.

Checklist

• [x] I've updated the test suite for new or updated code as appropriate
• [x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• [x] I've highlighted breaking changes using the "BREAKING" category above as appropriate
• [x] I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

[mcmire]

Hello! I have some concerns here and would definitely love to understand the use case for the new method you're adding. As you've mentioned elsewhere, there is a lot going on already in the network controller and I just want to make sure that we absolutely need a different way of updating a network configuration.

[Prithpal-Sooriya]

@mcmire yep I agree, this is a very big change. As mentioned in the PR description, I'm planning on having a 1:1 with the assets team and also include a code walkthrough.

Happy to include you and discuss in DMs the context for this change.

[mcmire]

@Prithpal-Sooriya That would be great, thanks!

[Prithpal-Sooriya]

@Gudahtt yep sure, will add an ADR.

I'm not planning on merging this work until we get team-wide acceptance.