feat: eip-6963 support

examples/sandbox/index.html

@@ -117,10 +117,7 @@ <h4>Select</h4>
       <button id="native">Pair Native</button>
       <button id="metaMask">Pair MetaMask</button>
       <button id="phantom">Pair Phantom</button>
-      <button id="xdefi">Pair XDEFI</button>
       <button id="keplr">Pair Keplr</button>
-      <button id="tallyHo">Pair Tally Ho</button>
-      <button id="coinbase">Pair Coinbase</button>
       <button id="walletConnect">Pair WalletConnect</button>
       <button id="walletConnectV2">Pair WalletConnect V2</button>
 

examples/sandbox/index.ts

@@ -1,6 +1,4 @@
 import * as sigUtil from "@metamask/eth-sig-util";
-import * as coinbase from "@shapeshiftoss/hdwallet-coinbase";
-import { CoinbaseProviderConfig } from "@shapeshiftoss/hdwallet-coinbase";
 import * as core from "@shapeshiftoss/hdwallet-core";
 import { BTCSignTxInput, BTCSignTxOutput, EosTx, Thorchain } from "@shapeshiftoss/hdwallet-core";
 import * as keepkey from "@shapeshiftoss/hdwallet-keepkey";
@@ -9,16 +7,14 @@ import * as keepkeyWebUSB from "@shapeshiftoss/hdwallet-keepkey-webusb";
 import * as keplr from "@shapeshiftoss/hdwallet-keplr";
 import * as ledgerWebHID from "@shapeshiftoss/hdwallet-ledger-webhid";
 import * as ledgerWebUSB from "@shapeshiftoss/hdwallet-ledger-webusb";
-import * as metaMask from "@shapeshiftoss/hdwallet-metamask";
+import * as metaMask from "@shapeshiftoss/hdwallet-metamask-multichain";
 import * as native from "@shapeshiftoss/hdwallet-native";
 import * as phantom from "@shapeshiftoss/hdwallet-phantom";
 import * as portis from "@shapeshiftoss/hdwallet-portis";
-import * as tallyHo from "@shapeshiftoss/hdwallet-tallyho";
 import * as trezorConnect from "@shapeshiftoss/hdwallet-trezor-connect";
 import { WalletConnectProviderConfig } from "@shapeshiftoss/hdwallet-walletconnect";
 import * as walletConnect from "@shapeshiftoss/hdwallet-walletconnect";
 import * as walletConnectv2 from "@shapeshiftoss/hdwallet-walletconnectv2";
-import * as xdefi from "@shapeshiftoss/hdwallet-xdefi";
 import { EthereumProviderOptions } from "@walletconnect/ethereum-provider/dist/types/EthereumProvider";
 import { TypedData } from "eip-712";
 import $, { noop } from "jquery";
@@ -97,14 +93,6 @@ const walletConnectV2Options: EthereumProviderOptions = {
   showQrModal: true,
 };
 
-const coinbaseOptions: CoinbaseProviderConfig = {
-  appName: "ShapeShift Sandbox",
-  appLogoUrl: "https://shapeshift.com/favicon.ico",
-  defaultJsonRpcUrl: "https://avatars.githubusercontent.com/u/52928763?s=50&v=4",
-  defaultChainId: 1,
-  darkMode: false,
-};
-
 const testPublicWalletXpubs = [
   "xpub661MyMwAqRbcFLgDU7wpcEVubSF7NkswwmXBUkDiGUW6uopeUMys4AqKXNgpfZKRTLnpKQgffd6a2c3J8JxLkF1AQN17Pm9QYHEqEfo1Rsx", // all seed root key
   "xpub68Zyu13qjcQxDzLNfTYnUXtJuX2qJgnxP6osrcAvJGdo6bs9M2Adt2BunbwiYrZS5qpA1QKoMf3uqS2NHpbyZp4KMJxDrL58NTyvHXBeAv6", // all seed m/44'
@@ -120,17 +108,14 @@ const testPublicWalletXpubs = [
   "xpub6DDUPHpUo4pcy43iJeZjbSVWGav1SMMmuWdMHiGtkK8rhKmfbomtkwW6GKs1GGAKehT6QRocrmda3WWxXawpjmwaUHfFRXuKrXSapdckEYF", // all seed m/84'/0'/0'
 ].join(" ");
 
-const coinbaseAdapter = coinbase.CoinbaseAdapter.useKeyring(keyring, coinbaseOptions);
 const keepkeyAdapter = keepkeyWebUSB.WebUSBKeepKeyAdapter.useKeyring(keyring);
 const kkbridgeAdapter = keepkeyTcp.TCPKeepKeyAdapter.useKeyring(keyring);
 const kkemuAdapter = keepkeyTcp.TCPKeepKeyAdapter.useKeyring(keyring);
 const portisAdapter = portis.PortisAdapter.useKeyring(keyring, { portisAppId });
-const metaMaskAdapter = metaMask.MetaMaskAdapter.useKeyring(keyring);
+const metaMaskAdapter = metaMask.MetaMaskAdapter.useKeyring(keyring, "io.metamask");
 const phantomAdapter = phantom.PhantomAdapter.useKeyring(keyring);
-const tallyHoAdapter = tallyHo.TallyHoAdapter.useKeyring(keyring);
 const walletConnectAdapter = walletConnect.WalletConnectAdapter.useKeyring(keyring, walletConnectOptions);
 const walletConnectV2Adapter = walletConnectv2.WalletConnectV2Adapter.useKeyring(keyring, walletConnectV2Options);
-const xdefiAdapter = xdefi.XDEFIAdapter.useKeyring(keyring);
 const keplrAdapter = keplr.KeplrAdapter.useKeyring(keyring);
 const nativeAdapter = native.NativeAdapter.useKeyring(keyring);
 const trezorAdapter = trezorConnect.TrezorAdapter.useKeyring(keyring, {
@@ -160,11 +145,8 @@ const $portis = $("#portis");
 const $native = $("#native");
 const $metaMask = $("#metaMask");
 const $phantom = $("#phantom");
-const $coinbase = $("#coinbase");
-const $tallyHo = $("#tallyHo");
 const $walletConnect = $("#walletConnect");
 const $walletConnectV2 = $("#walletConnectV2");
-const $xdefi = $("#xdefi");
 const $keplr = $("#keplr");
 const $keyring = $("#keyring");
 
@@ -257,19 +239,6 @@ $phantom.on("click", async (e) => {
   }
 });
 
-$coinbase.on("click", async (e) => {
-  e.preventDefault();
-  wallet = await coinbaseAdapter.pairDevice();
-  window["wallet"] = wallet;
-  let deviceID = "nothing";
-  try {
-    deviceID = await wallet.getDeviceID();
-    $("#keyring select").val(deviceID);
-  } catch (error) {
-    console.error(error);
-  }
-});
-
 $keplr.on("click", async (e) => {
   e.preventDefault();
   wallet = await keplrAdapter.pairDevice();
@@ -283,19 +252,6 @@ $keplr.on("click", async (e) => {
   }
 });
 
-$tallyHo.on("click", async (e) => {
-  e.preventDefault();
-  wallet = await tallyHoAdapter.pairDevice();
-  window["wallet"] = wallet;
-  let deviceID = "nothing";
-  try {
-    deviceID = await wallet.getDeviceID();
-    $("#keyring select").val(deviceID);
-  } catch (error) {
-    console.error(error);
-  }
-});
-
 $walletConnect.on("click", async (e) => {
   e.preventDefault();
   try {
@@ -322,19 +278,6 @@ $walletConnectV2.on("click", async (e) => {
   }
 });
 
-$xdefi.on("click", async (e) => {
-  e.preventDefault();
-  wallet = await xdefiAdapter.pairDevice();
-  window["wallet"] = wallet;
-  let deviceID = "nothing";
-  try {
-    deviceID = await wallet.getDeviceID();
-    $("#keyring select").val(deviceID);
-  } catch (error) {
-    console.error(error);
-  }
-});
-
 async function deviceConnected(deviceId) {
   wallet = keyring.get(deviceId);
   if (!$keyring.find(`option[value="${deviceId}"]`).length) {
@@ -425,24 +368,12 @@ async function deviceConnected(deviceId) {
     console.error("Could not initialize PhantomAdapter", e);
   }
 
-  try {
-    await tallyHoAdapter.initialize();
-  } catch (e) {
-    console.error("Could not initialize TallyHoAdapter", e);
-  }
-
   try {
     await keplrAdapter.initialize();
   } catch (e) {
     console.error("Could not initialize KeplrAdapter", e);
   }
 
-  try {
-    await coinbaseAdapter.initialize();
-  } catch (e) {
-    console.error("Could not initialize CoinbaseAdapter", e);
-  }
-
   try {
     await walletConnectAdapter.initialize();
   } catch (e) {

examples/sandbox/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shapeshiftoss/hdwallet-sandbox",
-  "version": "1.55.9",
+  "version": "1.55.10",
   "license": "MIT",
   "private": true,
   "browserslist": "> 0.5%, last 2 versions, not dead",
@@ -12,25 +12,22 @@
   "dependencies": {
     "@esm2cjs/p-queue": "^7.3.0",
     "@metamask/eth-sig-util": "^7.0.0",
-    "@shapeshiftoss/hdwallet-coinbase": "1.55.9",
-    "@shapeshiftoss/hdwallet-core": "1.55.9",
-    "@shapeshiftoss/hdwallet-keepkey": "1.55.9",
-    "@shapeshiftoss/hdwallet-keepkey-tcp": "1.55.9",
-    "@shapeshiftoss/hdwallet-keepkey-webusb": "1.55.9",
-    "@shapeshiftoss/hdwallet-keplr": "1.55.9",
-    "@shapeshiftoss/hdwallet-ledger": "1.55.9",
-    "@shapeshiftoss/hdwallet-ledger-webhid": "1.55.9",
-    "@shapeshiftoss/hdwallet-ledger-webusb": "1.55.9",
-    "@shapeshiftoss/hdwallet-metamask": "1.55.9",
-    "@shapeshiftoss/hdwallet-native": "1.55.9",
-    "@shapeshiftoss/hdwallet-phantom": "1.55.9",
-    "@shapeshiftoss/hdwallet-portis": "1.55.9",
-    "@shapeshiftoss/hdwallet-tallyho": "1.55.9",
-    "@shapeshiftoss/hdwallet-trezor": "1.55.9",
-    "@shapeshiftoss/hdwallet-trezor-connect": "1.55.9",
-    "@shapeshiftoss/hdwallet-walletconnect": "1.55.9",
-    "@shapeshiftoss/hdwallet-walletconnectv2": "1.55.9",
-    "@shapeshiftoss/hdwallet-xdefi": "1.55.9",
+    "@shapeshiftoss/hdwallet-core": "1.55.10",
+    "@shapeshiftoss/hdwallet-keepkey": "1.55.10",
+    "@shapeshiftoss/hdwallet-keepkey-tcp": "1.55.10",
+    "@shapeshiftoss/hdwallet-keepkey-webusb": "1.55.10",
+    "@shapeshiftoss/hdwallet-keplr": "1.55.10",
+    "@shapeshiftoss/hdwallet-ledger": "1.55.10",
+    "@shapeshiftoss/hdwallet-ledger-webhid": "1.55.10",
+    "@shapeshiftoss/hdwallet-ledger-webusb": "1.55.10",
+    "@shapeshiftoss/hdwallet-metamask-multichain": "1.55.10-mipd.1",
+    "@shapeshiftoss/hdwallet-native": "1.55.10",
+    "@shapeshiftoss/hdwallet-phantom": "1.55.10",
+    "@shapeshiftoss/hdwallet-portis": "1.55.10",
+    "@shapeshiftoss/hdwallet-trezor": "1.55.10",
+    "@shapeshiftoss/hdwallet-trezor-connect": "1.55.10",
+    "@shapeshiftoss/hdwallet-walletconnect": "1.55.10",
+    "@shapeshiftoss/hdwallet-walletconnectv2": "1.55.10",
     "bip32": "^2.0.4",
     "eip-712": "^1.0.0",
     "jquery": "^3.7.1",

integration/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shapeshiftoss/integration",
-  "version": "1.55.9",
+  "version": "1.55.10",
   "main": "index.js",
   "license": "MIT",
   "private": true,
@@ -10,15 +10,15 @@
     "dev": "lerna run test --scope integration --parallel --include-filtered-dependencies"
   },
   "dependencies": {
-    "@shapeshiftoss/hdwallet-core": "1.55.9",
-    "@shapeshiftoss/hdwallet-keepkey": "1.55.9",
-    "@shapeshiftoss/hdwallet-keepkey-nodewebusb": "1.55.9",
-    "@shapeshiftoss/hdwallet-keepkey-tcp": "1.55.9",
-    "@shapeshiftoss/hdwallet-ledger": "1.55.9",
-    "@shapeshiftoss/hdwallet-native": "1.55.9",
-    "@shapeshiftoss/hdwallet-portis": "1.55.9",
-    "@shapeshiftoss/hdwallet-trezor": "1.55.9",
-    "@shapeshiftoss/hdwallet-xdefi": "1.55.9",
+    "@shapeshiftoss/hdwallet-core": "1.55.10",
+    "@shapeshiftoss/hdwallet-keepkey": "1.55.10",
+    "@shapeshiftoss/hdwallet-keepkey-nodewebusb": "1.55.10",
+    "@shapeshiftoss/hdwallet-keepkey-tcp": "1.55.10",
+    "@shapeshiftoss/hdwallet-ledger": "1.55.10",
+    "@shapeshiftoss/hdwallet-metamask-multichain": "1.55.10",
+    "@shapeshiftoss/hdwallet-native": "1.55.10",
+    "@shapeshiftoss/hdwallet-portis": "1.55.10",
+    "@shapeshiftoss/hdwallet-trezor": "1.55.10",
     "fast-json-stable-stringify": "^2.1.0",
     "msw": "^0.27.1",
     "whatwg-fetch": "^3.6.2"

integration/src/bitcoin/bitcoin.ts

@@ -1,5 +1,6 @@
 import * as core from "@shapeshiftoss/hdwallet-core";
 import * as ledger from "@shapeshiftoss/hdwallet-ledger";
+import * as metamask from "@shapeshiftoss/hdwallet-metamask-multichain";
 import * as native from "@shapeshiftoss/hdwallet-native";
 import * as portis from "@shapeshiftoss/hdwallet-portis";
 import * as trezor from "@shapeshiftoss/hdwallet-trezor";
@@ -61,6 +62,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcSupportsCoin()",
       async () => {
         if (!wallet || portis.isPortis(wallet)) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
         expect(wallet.btcSupportsCoin("Bitcoin")).toBeTruthy();
         expect(await info.btcSupportsCoin("Bitcoin")).toBeTruthy();
         expect(wallet.btcSupportsCoin("Testnet")).toBeTruthy();
@@ -72,6 +75,9 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
     test("getPublicKeys", async () => {
       if (!wallet || ledger.isLedger(wallet) || trezor.isTrezor(wallet) || portis.isPortis(wallet)) return;
 
+      // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+      if (metamask.isMetaMask(wallet)) return;
+
       /* FIXME: Expected failure (trezor does not use scriptType in deriving public keys
           and ledger's dependency bitcoinjs-lib/src/crypto.js throws a mysterious TypeError
           in between mock transport calls.
@@ -194,10 +200,14 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
             const scriptType = args[3] as core.BTCInputScriptType;
             const expected = args[4] as string;
 
+            // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+            if (metamask.isMetaMask(wallet)) return;
+
             if (!(await wallet.btcSupportsCoin(coin))) return;
             expect(await info.btcSupportsCoin(coin)).toBeTruthy();
             if (!(await wallet.btcSupportsScriptType(coin, scriptType))) return;
             expect(await info.btcSupportsScriptType(coin, scriptType)).toBeTruthy();
+
             const res = await wallet.btcGetAddress({
               addressNList: core.bip32ToAddressNList(path),
               coin: coin,
@@ -215,6 +225,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcSignTx() - p2pkh",
       async () => {
         if (!wallet || portis.isPortis(wallet)) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
         if (ledger.isLedger(wallet)) return; // FIXME: Expected failure
         const tx: core.BitcoinTx = {
           version: 1,
@@ -290,6 +302,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcSignTx() - thorchain swap",
       async () => {
         if (!wallet || portis.isPortis(wallet)) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
         if (ledger.isLedger(wallet)) return; // FIXME: Expected failure
         if (trezor.isTrezor(wallet)) return; //TODO: Add trezor support for op return data passed at top level
         const tx: core.BitcoinTx = {
@@ -375,6 +389,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcSignMessage()",
       async () => {
         if (!wallet) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
 
         // not implemented for native
         if (native.isNative(wallet)) {
@@ -408,6 +424,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcVerifyMessage() - good",
       async () => {
         if (!wallet) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
 
         // not implemented for native
         if (native.isNative(wallet)) {
@@ -431,6 +449,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcVerifyMessage() - bad",
       async () => {
         if (!wallet) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
 
         // not implemented for native
         if (native.isNative(wallet)) {
@@ -454,6 +474,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcSupportsSecureTransfer()",
       async () => {
         if (!wallet) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
         expect(typeof (await wallet.btcSupportsSecureTransfer()) === typeof true).toBeTruthy();
         if (await wallet.btcSupportsSecureTransfer()) {
           // eslint-disable-next-line jest/no-conditional-expect
@@ -468,6 +490,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       "btcSupportsNativeShapeShift()",
       async () => {
         if (!wallet) return;
+        // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+        if (metamask.isMetaMask(wallet)) return;
         expect(typeof wallet.btcSupportsNativeShapeShift()).toBe("boolean");
         if (wallet.btcSupportsNativeShapeShift()) {
           // eslint-disable-next-line jest/no-conditional-expect
@@ -499,6 +523,8 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
             const accountIdx = args[1] as number;
             const scriptType = args[2] as core.BTCInputScriptType;
             if (!wallet) return;
+            // Non-EVM things are a pain to test with snaps on test env, this wasn't tested before and still isn't
+            if (metamask.isMetaMask(wallet)) return;
             if (!(await wallet.btcSupportsCoin(coin))) return;
             expect(await info.btcSupportsCoin(coin)).toBeTruthy();
             if (!(await wallet.btcSupportsScriptType(coin, scriptType))) return;
@@ -521,29 +547,5 @@ export function bitcoinTests(get: () => { wallet: core.HDWallet; info: core.HDWa
       },
       TIMEOUT
     );
-
-    test(
-      "btcIsSameAccount()",
-      async () => {
-        if (!wallet) return;
-        [0, 1, 9].forEach((idx) => {
-          const paths = wallet.btcGetAccountPaths({
-            coin: "Bitcoin",
-            accountIdx: idx,
-          });
-          expect(typeof wallet.btcIsSameAccount(paths) === typeof true).toBeTruthy();
-          paths.forEach((path) => {
-            if (wallet.getVendor() === "Portis") {
-              // eslint-disable-next-line jest/no-conditional-expect
-              expect(wallet.btcNextAccountPath(path)).toBeUndefined();
-            } else {
-              // eslint-disable-next-line jest/no-conditional-expect
-              expect(wallet.btcNextAccountPath(path)).not.toBeUndefined();
-            }
-          });
-        });
-      },
-      TIMEOUT
-    );
   });
 }