chore: update Accounts CO (#27089)

<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

## **Description**

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

Update Accounts CO

## **Changelog**

<!--
If this PR is not End-User-Facing and should not show up in the
CHANGELOG, you can choose to either:
1. Write `CHANGELOG entry: null`
2. Label with `no-changelog`

If this PR is End-User-Facing, please write a short User-Facing
description in the past tense like:
`CHANGELOG entry: Added a new tab for users to see their NFTs`
`CHANGELOG entry: Fixed a bug that was causing some NFTs to flicker`

(This helps the Release Engineer do their job more quickly and
accurately)
-->

CHANGELOG entry: null

## **Related issues**

Fixes: https://consensyssoftware.atlassian.net/browse/MUL-1260

## **Manual testing steps**

Not applicable

## **Screenshots/Recordings**

Not applicable

## **Pre-merge author checklist**

- [ ] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've completed the PR template to the best of my ability
- [ ] I've included tests if applicable
- [ ] I've documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I've applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk because changes are limited to CODEOWNERS updates and
import-path adjustments after reorganizing controller init modules, with
no functional logic changes.
> 
> **Overview**
> Updates `.github/CODEOWNERS` to assign the Accounts team ownership of
additional Engine controller directories (e.g.,
`multichain-account-service`, `snap-keyring`, `keyring-controller`,
`storage-service`) and expands multichain account path patterns.
> 
> Adjusts `Engine.ts` and related controller init tests to import
`snap-keyring`, `keyring-controller`, and `storage-service` init modules
from their new subdirectories.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
e3e1512. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: gantunesr

.github/CODEOWNERS

@@ -103,6 +103,10 @@ app/util/walletconnect.js                                   @MetaMask/wallet-int
 # Accounts Team
 app/core/Encryptor/                                         @MetaMask/accounts-engineers
 app/core/Engine/controllers/accounts-controller             @MetaMask/accounts-engineers
+app/core/Engine/controllers/multichain-account-service      @MetaMask/accounts-engineers
+app/core/Engine/controllers/snap-keyring                    @MetaMask/accounts-engineers
+app/core/Engine/controllers/keyring-controller              @MetaMask/accounts-engineers
+app/core/Engine/controllers/storage-service                 @MetaMask/accounts-engineers
 app/core/Engine/messengers/accounts-controller-messenger    @MetaMask/accounts-engineers
 app/core/SnapKeyring                                        @MetaMask/accounts-engineers
 **/Identity/**                                              @MetaMask/accounts-engineers

app/core/Engine/Engine.ts

@@ -90,7 +90,7 @@ import { multichainAssetsControllerInit } from './controllers/multichain-assets-
 import { multichainAssetsRatesControllerInit } from './controllers/multichain-assets-rates-controller/multichain-assets-rates-controller-init';
 import { multichainTransactionsControllerInit } from './controllers/multichain-transactions-controller/multichain-transactions-controller-init';
 import { multichainAccountServiceInit } from './controllers/multichain-account-service/multichain-account-service-init';
-import { snapKeyringBuilderInit } from './controllers/snap-keyring-builder-init';
+import { snapKeyringBuilderInit } from './controllers/snap-keyring/snap-keyring-builder-init';
 import { SnapKeyring } from '@metamask/eth-snap-keyring';
 ///: END:ONLY_INCLUDE_IF
 ///: BEGIN:ONLY_INCLUDE_IF(preinstalled-snaps,external-snaps)
@@ -144,7 +144,7 @@ import { subjectMetadataControllerInit } from './controllers/subject-metadata-co
 ///: END:ONLY_INCLUDE_IF
 import { PreferencesController } from '@metamask/preferences-controller';
 import { preferencesControllerInit } from './controllers/preferences-controller-init';
-import { keyringControllerInit } from './controllers/keyring-controller-init';
+import { keyringControllerInit } from './controllers/keyring-controller/keyring-controller-init';
 import { networkControllerInit } from './controllers/network-controller-init';
 import { TransactionPayControllerInit } from './controllers/transaction-pay-controller';
 import { tokenSearchDiscoveryDataControllerInit } from './controllers/token-search-discovery-data-controller-init';
@@ -165,7 +165,7 @@ import { rewardsDataServiceInit } from './controllers/rewards-data-service-init'
 import { swapsControllerInit } from './controllers/swaps-controller-init';
 import { remoteFeatureFlagControllerInit } from './controllers/remote-feature-flag-controller-init';
 import { errorReportingServiceInit } from './controllers/error-reporting-service-init';
-import { storageServiceInit } from './controllers/storage-service-init';
+import { storageServiceInit } from './controllers/storage-service/storage-service-init';
 import { loggingControllerInit } from './controllers/logging-controller-init';
 import { phishingControllerInit } from './controllers/phishing-controller-init';
 import { addressBookControllerInit } from './controllers/address-book-controller-init';

…trollers/keyring-controller-init.test.ts …ntroller/keyring-controller-init.test.tsapp/core/Engine/controllers/keyring-controller-init.test.ts renamed to app/core/Engine/controllers/keyring-controller/keyring-controller-init.test.ts app/core/Engine/controllers/keyring-controller-init.test.ts renamed to app/core/Engine/controllers/keyring-controller/keyring-controller-init.test.ts

@@ -1,13 +1,13 @@
-import { buildControllerInitRequestMock } from '../utils/test-utils';
-import { ExtendedMessenger } from '../../ExtendedMessenger';
-import { getKeyringControllerMessenger } from '../messengers/keyring-controller-messenger';
-import { ControllerInitRequest } from '../types';
+import { buildControllerInitRequestMock } from '../../utils/test-utils';
+import { ExtendedMessenger } from '../../../ExtendedMessenger';
+import { getKeyringControllerMessenger } from '../../messengers/keyring-controller-messenger';
+import { ControllerInitRequest } from '../../types';
 import { keyringControllerInit } from './keyring-controller-init';
 import {
   KeyringController,
   KeyringControllerMessenger,
 } from '@metamask/keyring-controller';
-import { Encryptor } from '../../Encryptor';
+import { Encryptor } from '../../../Encryptor';
 import { MOCK_ANY_NAMESPACE, MockAnyNamespace } from '@metamask/messenger';
 
 jest.mock('@metamask/keyring-controller');

…e/controllers/keyring-controller-init.ts …ng-controller/keyring-controller-init.tsapp/core/Engine/controllers/keyring-controller-init.ts renamed to app/core/Engine/controllers/keyring-controller/keyring-controller-init.ts app/core/Engine/controllers/keyring-controller-init.ts renamed to app/core/Engine/controllers/keyring-controller/keyring-controller-init.ts

@@ -1,4 +1,4 @@
-import { ControllerInitFunction } from '../types';
+import { ControllerInitFunction } from '../../types';
 import {
   KeyringController,
   KeyringControllerMessenger,
@@ -11,7 +11,11 @@ import {
 } from '@metamask/eth-ledger-bridge-keyring';
 import { HdKeyring } from '@metamask/eth-hd-keyring';
 import { hmacSha512 } from '@metamask/native-utils';
-import { Encryptor, LEGACY_DERIVATION_OPTIONS, pbkdf2 } from '../../Encryptor';
+import {
+  Encryptor,
+  LEGACY_DERIVATION_OPTIONS,
+  pbkdf2,
+} from '../../../Encryptor';
 
 const encryptor = new Encryptor({
   keyDerivationOptions: LEGACY_DERIVATION_OPTIONS,

…ollers/snap-keyring-builder-init.test.ts …eyring/snap-keyring-builder-init.test.tsapp/core/Engine/controllers/snap-keyring-builder-init.test.ts renamed to app/core/Engine/controllers/snap-keyring/snap-keyring-builder-init.test.ts app/core/Engine/controllers/snap-keyring-builder-init.test.ts renamed to app/core/Engine/controllers/snap-keyring/snap-keyring-builder-init.test.ts

@@ -1,14 +1,14 @@
-import { buildControllerInitRequestMock } from '../utils/test-utils';
-import { ExtendedMessenger } from '../../ExtendedMessenger';
+import { buildControllerInitRequestMock } from '../../utils/test-utils';
+import { ExtendedMessenger } from '../../../ExtendedMessenger';
 import {
   getSnapKeyringBuilderInitMessenger,
   getSnapKeyringBuilderMessenger,
   SnapKeyringBuilderInitMessenger,
-} from '../messengers/snap-keyring-builder-messenger';
-import { ControllerInitRequest } from '../types';
+} from '../../messengers/snap-keyring-builder-messenger';
+import { ControllerInitRequest } from '../../types';
 import { snapKeyringBuilderInit } from './snap-keyring-builder-init';
 import { MOCK_ANY_NAMESPACE, MockAnyNamespace } from '@metamask/messenger';
-import { SnapKeyringBuilderMessenger } from '../../SnapKeyring/types';
+import { SnapKeyringBuilderMessenger } from '../../../SnapKeyring/types';
 
 function getInitRequestMock(): jest.Mocked<
   ControllerInitRequest<

…controllers/snap-keyring-builder-init.ts …nap-keyring/snap-keyring-builder-init.tsapp/core/Engine/controllers/snap-keyring-builder-init.ts renamed to app/core/Engine/controllers/snap-keyring/snap-keyring-builder-init.ts app/core/Engine/controllers/snap-keyring-builder-init.ts renamed to app/core/Engine/controllers/snap-keyring/snap-keyring-builder-init.ts

@@ -1,10 +1,10 @@
-import { ControllerInitFunction } from '../types';
-import { SnapKeyringBuilderInitMessenger } from '../messengers/snap-keyring-builder-messenger';
+import { ControllerInitFunction } from '../../types';
+import { SnapKeyringBuilderInitMessenger } from '../../messengers/snap-keyring-builder-messenger';
 import {
   snapKeyringBuilder,
   SnapKeyringBuilder,
-} from '../../SnapKeyring/SnapKeyring';
-import { SnapKeyringBuilderMessenger } from '../../SnapKeyring/types';
+} from '../../../SnapKeyring/SnapKeyring';
+import { SnapKeyringBuilderMessenger } from '../../../SnapKeyring/types';
 
 /**
  * Initialize the Snap keyring builder.

…controllers/storage-service-init.test.ts …age-service/storage-service-init.test.tsapp/core/Engine/controllers/storage-service-init.test.ts renamed to app/core/Engine/controllers/storage-service/storage-service-init.test.ts app/core/Engine/controllers/storage-service-init.test.ts renamed to app/core/Engine/controllers/storage-service/storage-service-init.test.ts

@@ -1,7 +1,7 @@
-import { buildControllerInitRequestMock } from '../utils/test-utils';
-import { ExtendedMessenger } from '../../ExtendedMessenger';
-import { getStorageServiceMessenger } from '../messengers/storage-service-messenger';
-import { ControllerInitRequest } from '../types';
+import { buildControllerInitRequestMock } from '../../utils/test-utils';
+import { ExtendedMessenger } from '../../../ExtendedMessenger';
+import { getStorageServiceMessenger } from '../../messengers/storage-service-messenger';
+import { ControllerInitRequest } from '../../types';
 import { storageServiceInit } from './storage-service-init';
 import {
   StorageService,
@@ -10,13 +10,13 @@ import {
 } from '@metamask/storage-service';
 import { MOCK_ANY_NAMESPACE, MockAnyNamespace } from '@metamask/messenger';
 import FilesystemStorage from 'redux-persist-filesystem-storage';
-import Device from '../../../util/device';
-import Logger from '../../../util/Logger';
+import Device from '../../../../util/device';
+import Logger from '../../../../util/Logger';
 
 jest.mock('@metamask/storage-service');
 jest.mock('redux-persist-filesystem-storage');
-jest.mock('../../../util/device');
-jest.mock('../../../util/Logger');
+jest.mock('../../../../util/device');
+jest.mock('../../../../util/Logger');
 
 const mockFilesystemStorage = jest.mocked(FilesystemStorage);
 const mockDevice = jest.mocked(Device);

…gine/controllers/storage-service-init.ts …/storage-service/storage-service-init.tsapp/core/Engine/controllers/storage-service-init.ts renamed to app/core/Engine/controllers/storage-service/storage-service-init.ts app/core/Engine/controllers/storage-service-init.ts renamed to app/core/Engine/controllers/storage-service/storage-service-init.ts

@@ -1,20 +1,20 @@
 import type { Json } from '@metamask/utils';
 import FilesystemStorage from 'redux-persist-filesystem-storage';
 
-import { ControllerInitFunction } from '../types';
+import { ControllerInitFunction } from '../../types';
 import {
   StorageService,
   StorageServiceMessenger,
   StorageAdapter,
   StorageGetResult,
   STORAGE_KEY_PREFIX,
 } from '@metamask/storage-service';
-import Device from '../../../util/device';
-import Logger from '../../../util/Logger';
+import Device from '../../../../util/device';
+import Logger from '../../../../util/Logger';
 import {
   encodeStorageKey,
   decodeStorageKey,
-} from '../utils/storage-service-utils';
+} from '../../utils/storage-service-utils';
 
 /**
  * Mobile-specific storage adapter using FilesystemStorage.