refactor!: remove InternalAccount.metadata.snap.{name,enabled}

[ccharly]

Those metadata should be pulled out from the SnapController itself to avoid duplicated state and potential inconsistencies.

This would also remove some non-needed state-watch on the AccountsController (since we are listening to SnapController:stateChange to update those accordingly, thus triggering some other AccountsController:stateChange which could be avoided).

[ccharly]

@metamaskbot publish-preview

[github-actions]

Preview builds have been published. See these instructions (from the core monorepo) for more information about preview builds.

Expand for full list of packages and versions.

{
  "@metamask-previews/keyring-api": "17.5.0-33a98a2",
  "@metamask-previews/eth-hd-keyring": "12.1.0-33a98a2",
  "@metamask-previews/eth-ledger-bridge-keyring": "11.0.3-33a98a2",
  "@metamask-previews/eth-simple-keyring": "10.0.0-33a98a2",
  "@metamask-previews/eth-trezor-keyring": "8.0.0-33a98a2",
  "@metamask-previews/keyring-internal-api": "6.0.1-33a98a2",
  "@metamask-previews/keyring-internal-snap-client": "4.0.2-33a98a2",
  "@metamask-previews/eth-snap-keyring": "12.1.1-33a98a2",
  "@metamask-previews/keyring-snap-client": "4.1.0-33a98a2",
  "@metamask-previews/keyring-snap-sdk": "3.2.0-33a98a2",
  "@metamask-previews/keyring-utils": "3.0.0-33a98a2"
}