feat: support bitcoin (#15263)

## **Description**

Bump Bitcoin Snap version.

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/32471?quickstart=1)

## **Related issues**

Fixes: https://consensyssoftware.atlassian.net/browse/NNT-338

## **Manual testing steps**

1. Install MM Flask
2. Create Bitcoin account

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [x] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask
Extension Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/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.

Author: darioAnongba

app/core/SnapKeyring/MultichainWalletSnapClient.test.ts

@@ -11,6 +11,7 @@ import Engine from '../Engine';
 import { Sender } from '@metamask/keyring-snap-client';
 import { SnapKeyring } from '@metamask/eth-snap-keyring';
 import { BtcScope, SolScope } from '@metamask/keyring-api';
+import { BITCOIN_WALLET_SNAP_ID } from './BitcoinWalletSnap';
 
 jest.mock('../Engine', () => ({
   controllerMessenger: {
@@ -292,6 +293,33 @@ describe('Wallet Client Implementations', () => {
       const bitcoinClient = new BitcoinWalletSnapClient(mockSnapKeyringOptions);
       expect(bitcoinClient.getScope()).toEqual(BtcScope.Mainnet);
     });
+
+    it('adds synchronize parameter to createAccount', async () => {
+      const mockOptions = {
+        scope: BtcScope.Mainnet,
+        accountNameSuggestion: 'Bitcoin Account 1',
+        entropySource: 'test-entropy',
+      };
+
+      const mockKeyring = {
+        createAccount: jest.fn(),
+      };
+
+      (Engine.controllerMessenger.call as jest.Mock).mockImplementationOnce(
+        async (_, __, callback) => {
+          await callback({ keyring: mockKeyring });
+        },
+      );
+
+      const bitcoinClient = new BitcoinWalletSnapClient(mockSnapKeyringOptions);
+      await bitcoinClient.createAccount(mockOptions);
+
+      expect(mockKeyring.createAccount).toHaveBeenCalledWith(
+        BITCOIN_WALLET_SNAP_ID,
+        { ...mockOptions, synchronize: true },
+        mockSnapKeyringOptions,
+      );
+    });
   });
 
   describe('SolanaWalletSnapClient', () => {

app/core/SnapKeyring/MultichainWalletSnapClient.ts

@@ -38,6 +38,7 @@ export const WALLET_SNAP_MAP = {
 
 export interface MultichainWalletSnapOptions {
   scope: CaipChainId;
+  synchronize?: boolean;
   ///: BEGIN:ONLY_INCLUDE_IF(multi-srp)
   entropySource?: string;
   accountNameSuggestion?: string;
@@ -253,6 +254,16 @@ export class BitcoinWalletSnapClient extends MultichainWalletSnapClient {
   protected getSnapSender(): Sender {
     return new BitcoinWalletSnapSender();
   }
+
+  async createAccount(
+    options: MultichainWalletSnapOptions,
+    snapKeyringOptions?: SnapKeyringOptions,
+  ) {
+    return super.createAccount(
+      { ...options, synchronize: true },
+      snapKeyringOptions,
+    );
+  }
 }
 
 export class SolanaWalletSnapClient extends MultichainWalletSnapClient {

package.json

@@ -165,7 +165,7 @@
     "@metamask/approval-controller": "^7.1.3",
     "@metamask/assets-controllers": "^61.0.0",
     "@metamask/base-controller": "^8.0.0",
-    "@metamask/bitcoin-wallet-snap": "^0.9.0",
+    "@metamask/bitcoin-wallet-snap": "^0.12.1",
     "@metamask/bridge-controller": "^21.0.0",
     "@metamask/bridge-status-controller": "^18.0.0",
     "@metamask/chain-agnostic-permission": "^0.3.0",

yarn.lock

@@ -4765,10 +4765,10 @@
     "@metamask/utils" "^11.2.0"
     immer "^9.0.6"
 
-"@metamask/bitcoin-wallet-snap@^0.9.0":
-  version "0.9.0"
-  resolved "https://registry.yarnpkg.com/@metamask/bitcoin-wallet-snap/-/bitcoin-wallet-snap-0.9.0.tgz#47014c75c1e1ed84fef57177901f458fd2b6510a"
-  integrity sha512-983twhfOfSCQljT+eV8JLcV0bqGVMObXnEy8TOFAkwwRW2I1ZsVRAfylbGiSwxlhxLntcpLqVu0Sj29xNyA06Q==
+"@metamask/bitcoin-wallet-snap@^0.12.1":
+  version "0.12.1"
+  resolved "https://registry.yarnpkg.com/@metamask/bitcoin-wallet-snap/-/bitcoin-wallet-snap-0.12.1.tgz#2b60ac613992e97302a9898b3d4030395b2b2123"
+  integrity sha512-8gsN7a5OyRMg6c0jw0tDrqePHjFv9uHi36eL2PdTMUYU9J3dL/t9cOWe/3RPSwu7tw/tMNXyM4mzwZvq7Kxs2Q==
 
 "@metamask/bridge-controller@^21.0.0":
   version "21.0.0"