@dynamic-labs/bitcoin 3.0.0-alpha.31 → 3.0.0-alpha.33

package/CHANGELOG.md

@@ -1,4 +1,19 @@
 
+## [3.0.0-alpha.33](https://github.com/dynamic-labs/DynamicAuth/compare/v3.0.0-alpha.32...v3.0.0-alpha.33) (2024-08-02)
+
+
+### Features
+
+* add async handler for mfa enrollment and completion ([#6481](https://github.com/dynamic-labs/DynamicAuth/issues/6481)) ([63ce10d](https://github.com/dynamic-labs/DynamicAuth/commit/63ce10dd9f9aa58f43411455d6bbeb2f440ddcac))
+
+## [3.0.0-alpha.32](https://github.com/dynamic-labs/DynamicAuth/compare/v3.0.0-alpha.31...v3.0.0-alpha.32) (2024-07-30)
+
+
+### Bug Fixes
+
+* react native flicker when redirecting back from farcaster ([#6338](https://github.com/dynamic-labs/DynamicAuth/issues/6338)) ([6ab8464](https://github.com/dynamic-labs/DynamicAuth/commit/6ab846476f7961564445223dd12a552ed4c0f0c8))
+* stop unnecessary phantom btc popup when disconnecting - Revert ([#6439](https://github.com/dynamic-labs/DynamicAuth/issues/6439)) ([79ba97d](https://github.com/dynamic-labs/DynamicAuth/commit/79ba97d82c4eb89f0118a925e4dc899853e04550)), closes [#6188](https://github.com/dynamic-labs/DynamicAuth/issues/6188)
+
 ## [3.0.0-alpha.31](https://github.com/dynamic-labs/DynamicAuth/compare/v3.0.0-alpha.30...v3.0.0-alpha.31) (2024-07-27)
 
 ## [3.0.0-alpha.30](https://github.com/dynamic-labs/DynamicAuth/compare/v3.0.0-alpha.29...v3.0.0-alpha.30) (2024-07-27)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dynamic-labs/bitcoin",
-  "version": "3.0.0-alpha.31",
+  "version": "3.0.0-alpha.33",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/dynamic-labs/dynamic-auth.git",
@@ -27,14 +27,14 @@
   },
   "dependencies": {
     "@btckit/types": "0.0.19",
-    "@dynamic-labs/sdk-api-core": "0.0.497",
+    "@dynamic-labs/sdk-api-core": "0.0.506",
     "@wallet-standard/app": "1.0.1",
     "@wallet-standard/base": "1.0.1",
     "bitcoinjs-lib": "6.1.5",
     "sats-connect": "2.0.0",
-    "@dynamic-labs/utils": "3.0.0-alpha.31",
-    "@dynamic-labs/wallet-book": "3.0.0-alpha.31",
-    "@dynamic-labs/wallet-connector-core": "3.0.0-alpha.31",
+    "@dynamic-labs/utils": "3.0.0-alpha.33",
+    "@dynamic-labs/wallet-book": "3.0.0-alpha.33",
+    "@dynamic-labs/wallet-connector-core": "3.0.0-alpha.33",
     "stream": "0.0.2"
   },
   "peerDependencies": {}

package/src/connectors/BitcoinWalletConnector.cjs

@@ -110,7 +110,31 @@ class BitcoinWalletConnector extends walletConnectorCore.WalletConnectorBase {
     }
     getConnectedAccounts() {
         return _tslib.__awaiter(this, void 0, void 0, function* () {
-            return this.getConnectedAccountsFromCache();
+            // some wallets like xverse don't support fetching connected accounts
+            // without prompting for a connection
+            // to avoid this behavior, we cache the connected accounts
+            if (!this.canFetchConnectedAccounts) {
+                return this.getConnectedAccountsFromCache();
+            }
+            // if we decide that is ok to prompt for a connection when fetching connected accounts
+            // we shouldn't prompt every time we call this method (which is a lot of times)
+            // so we just store in a promise and return the same promise every time
+            if (!this.getAddressPromise) {
+                this.getAddressPromise = this.getAddress();
+            }
+            let connectedAccount;
+            try {
+                connectedAccount = yield this.getAddressPromise;
+            }
+            catch (error) {
+                walletConnectorCore.logger.error(`${this.key} getConnectedAccounts - error fetching connected account`);
+                //don't throw error just return empty array after clearing the promise
+            }
+            this.getAddressPromise = undefined;
+            if (!connectedAccount) {
+                return [];
+            }
+            return [connectedAccount];
         });
     }
     getAdditionalAddresses(mainAddress) {
@@ -193,14 +217,7 @@ class BitcoinWalletConnector extends walletConnectorCore.WalletConnectorBase {
             return;
         }
         const { handleAccountChange, handleChainChange, handleDisconnect } = walletConnectorCore.eventListenerHandlers(this);
-        const handleBitcoinDisconnect = (error) => _tslib.__awaiter(this, void 0, void 0, function* () {
-            this.cache.clearConnectedAcccounts();
-            handleDisconnect(error);
-        });
         const handleBitcoinAccountChange = (accounts) => _tslib.__awaiter(this, void 0, void 0, function* () {
-            if (accounts.length === 0) {
-                this.cache.clearConnectedAcccounts();
-            }
             let connectedAccounts = accounts;
             let ordinalsAccount, paymentAccount;
             // if accounts is an array of objects, we need to parse them to return only addresses
@@ -233,7 +250,7 @@ class BitcoinWalletConnector extends walletConnectorCore.WalletConnectorBase {
         });
         provider.on('accountsChanged', handleBitcoinAccountChange);
         provider.on('networkChanged', handleChainChange);
-        provider.on('disconnect', handleBitcoinDisconnect);
+        provider.on('disconnect', handleDisconnect);
         const tearDownEventListeners = () => {
             const provider = this.getProvider();
             if (!(provider === null || provider === void 0 ? void 0 : provider.removeListener)) {
@@ -241,7 +258,7 @@ class BitcoinWalletConnector extends walletConnectorCore.WalletConnectorBase {
             }
             provider.removeListener('accountsChanged', handleBitcoinAccountChange);
             provider.removeListener('networkChanged', handleChainChange);
-            provider.removeListener('disconnect', handleBitcoinDisconnect);
+            provider.removeListener('disconnect', handleDisconnect);
         };
         this.teardownEventListeners = tearDownEventListeners;
     }

package/src/connectors/BitcoinWalletConnector.js

@@ -106,7 +106,31 @@ class BitcoinWalletConnector extends WalletConnectorBase {
     }
     getConnectedAccounts() {
         return __awaiter(this, void 0, void 0, function* () {
-            return this.getConnectedAccountsFromCache();
+            // some wallets like xverse don't support fetching connected accounts
+            // without prompting for a connection
+            // to avoid this behavior, we cache the connected accounts
+            if (!this.canFetchConnectedAccounts) {
+                return this.getConnectedAccountsFromCache();
+            }
+            // if we decide that is ok to prompt for a connection when fetching connected accounts
+            // we shouldn't prompt every time we call this method (which is a lot of times)
+            // so we just store in a promise and return the same promise every time
+            if (!this.getAddressPromise) {
+                this.getAddressPromise = this.getAddress();
+            }
+            let connectedAccount;
+            try {
+                connectedAccount = yield this.getAddressPromise;
+            }
+            catch (error) {
+                logger.error(`${this.key} getConnectedAccounts - error fetching connected account`);
+                //don't throw error just return empty array after clearing the promise
+            }
+            this.getAddressPromise = undefined;
+            if (!connectedAccount) {
+                return [];
+            }
+            return [connectedAccount];
         });
     }
     getAdditionalAddresses(mainAddress) {
@@ -189,14 +213,7 @@ class BitcoinWalletConnector extends WalletConnectorBase {
             return;
         }
         const { handleAccountChange, handleChainChange, handleDisconnect } = eventListenerHandlers(this);
-        const handleBitcoinDisconnect = (error) => __awaiter(this, void 0, void 0, function* () {
-            this.cache.clearConnectedAcccounts();
-            handleDisconnect(error);
-        });
         const handleBitcoinAccountChange = (accounts) => __awaiter(this, void 0, void 0, function* () {
-            if (accounts.length === 0) {
-                this.cache.clearConnectedAcccounts();
-            }
             let connectedAccounts = accounts;
             let ordinalsAccount, paymentAccount;
             // if accounts is an array of objects, we need to parse them to return only addresses
@@ -229,7 +246,7 @@ class BitcoinWalletConnector extends WalletConnectorBase {
         });
         provider.on('accountsChanged', handleBitcoinAccountChange);
         provider.on('networkChanged', handleChainChange);
-        provider.on('disconnect', handleBitcoinDisconnect);
+        provider.on('disconnect', handleDisconnect);
         const tearDownEventListeners = () => {
             const provider = this.getProvider();
             if (!(provider === null || provider === void 0 ? void 0 : provider.removeListener)) {
@@ -237,7 +254,7 @@ class BitcoinWalletConnector extends WalletConnectorBase {
             }
             provider.removeListener('accountsChanged', handleBitcoinAccountChange);
             provider.removeListener('networkChanged', handleChainChange);
-            provider.removeListener('disconnect', handleBitcoinDisconnect);
+            provider.removeListener('disconnect', handleDisconnect);
         };
         this.teardownEventListeners = tearDownEventListeners;
     }