@aptos-labs/wallet-adapter-core 2.3.0 → 2.3.2

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @aptos-labs/wallet-adapter-core
 
+## 2.3.2
+
+### Patch Changes
+
+- 22ecf6a: Throw `wallet already connected` error when trying to connect to an already connected wallet
+
+## 2.3.1
+
+### Patch Changes
+
+- 06f334f: @aptos-labs/wallet-adapter-core:
+  Fixes ssr issue with checking for mobile wallets
+
+  @aptos-labs/wallet-adapter-mui-design:
+  Breaking:
+  When on a mobile phone on the native browser, we removed all wallets that are not able to be deep linked to.
+  The previous functionally would take them to the extension, which would not help users on mobile phones.
+
 ## 2.3.0
 
 ### Minor Changes

package/dist/index.d.ts

@@ -160,7 +160,7 @@ declare class WalletCore extends EventEmitter<WalletCoreEvents> {
     @emit emits "connect" event
     @throws WalletConnectionError
     */
-    connect(walletName: WalletName): Promise<void>;
+    connect(walletName: WalletName): Promise<void | string>;
     /**
     Disconnect the exisitng wallet. On success, we clear the
     current account, current network and LocalStorage data.

package/dist/index.js

@@ -210,7 +210,7 @@ function isInAppBrowser() {
   return isIphone || isAndroid;
 }
 function isRedirectable() {
-  if (!navigator)
+  if (typeof navigator === "undefined" || !navigator)
     return false;
   return isMobile() && !isInAppBrowser();
 }
@@ -344,7 +344,9 @@ var WalletCore = class extends import_eventemitter3.default {
         return;
       if (this._connected) {
         if (((_b = this.wallet) == null ? void 0 : _b.name) === walletName)
-          return;
+          throw new WalletConnectionError(
+            `${walletName} wallet is already connected`
+          ).message;
         await this.disconnect();
       }
       if (isRedirectable()) {

package/dist/index.mjs

@@ -169,7 +169,7 @@ function isInAppBrowser() {
   return isIphone || isAndroid;
 }
 function isRedirectable() {
-  if (!navigator)
+  if (typeof navigator === "undefined" || !navigator)
     return false;
   return isMobile() && !isInAppBrowser();
 }
@@ -303,7 +303,9 @@ var WalletCore = class extends EventEmitter {
         return;
       if (this._connected) {
         if (((_b = this.wallet) == null ? void 0 : _b.name) === walletName)
-          return;
+          throw new WalletConnectionError(
+            `${walletName} wallet is already connected`
+          ).message;
         await this.disconnect();
       }
       if (isRedirectable()) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aptos-labs/wallet-adapter-core",
-  "version": "2.3.0",
+  "version": "2.3.2",
   "description": "Aptos Wallet Adapter Core",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",

package/src/WalletCore.ts

@@ -181,7 +181,7 @@ export class WalletCore extends EventEmitter<WalletCoreEvents> {
   @emit emits "connect" event
   @throws WalletConnectionError
   */
-  async connect(walletName: WalletName): Promise<void> {
+  async connect(walletName: WalletName): Promise<void | string> {
     try {
       const selectedWallet = this._wallets?.find(
         (wallet: Wallet) => wallet.name === walletName
@@ -191,7 +191,10 @@ export class WalletCore extends EventEmitter<WalletCoreEvents> {
 
       if (this._connected) {
         // if the selected wallet is already connected, we don't need to connect again
-        if (this.wallet?.name === walletName) return;
+        if (this.wallet?.name === walletName)
+          throw new WalletConnectionError(
+            `${walletName} wallet is already connected`
+          ).message;
 
         await this.disconnect();
       }

package/src/utils/helpers.ts

@@ -18,7 +18,7 @@ export function isInAppBrowser(): boolean {
 
 export function isRedirectable(): boolean {
   // SSR: return false
-  if (!navigator) return false;
+  if (typeof navigator === 'undefined' || !navigator) return false;
 
   // if we are on mobile and NOT in a in-app browser we will redirect to a wallet app