refactor(pera): extract auto-connect logic into private method

Extract auto-connect logic from constructor into a private async method for better
error handling and testability. Update tests to use prototype mocking pattern
consistent with other wallet implementations.

- Move auto-connect logic into private `autoConnect` method
- Use async/await with try/catch for better error handling
- Update tests to mock method on prototype before wallet creation
- Align test patterns with other wallet implementations

Author: drichar

packages/use-wallet/src/__tests__/wallets/pera.test.ts

@@ -541,6 +541,53 @@ describe('PeraWallet', () => {
     })
   })
 
+  describe('autoConnect', () => {
+    let mockUserAgent: string
+
+    beforeEach(() => {
+      mockUserAgent = ''
+      vi.clearAllMocks()
+
+      vi.stubGlobal('window', {
+        navigator: {
+          get userAgent() {
+            return mockUserAgent
+          }
+        }
+      })
+    })
+
+    afterEach(() => {
+      vi.unstubAllGlobals()
+    })
+
+    it('should attempt auto-connect in Pera browser', () => {
+      mockUserAgent = 'pera/1.0.0'
+
+      // Mock the private method before creating the wallet
+      vi.spyOn(PeraWallet.prototype as any, 'autoConnect')
+        .mockReset()
+        .mockImplementation(() => Promise.resolve())
+
+      createWalletWithStore(store)
+
+      expect(PeraWallet.prototype['autoConnect']).toHaveBeenCalled()
+    })
+
+    it('should not attempt auto-connect in other browsers', () => {
+      mockUserAgent = 'chrome/1.0.0'
+
+      // Mock the private method before creating the wallet
+      vi.spyOn(PeraWallet.prototype as any, 'autoConnect')
+        .mockReset()
+        .mockImplementation(() => Promise.resolve())
+
+      createWalletWithStore(store)
+
+      expect(PeraWallet.prototype['autoConnect']).not.toHaveBeenCalled()
+    })
+  })
+
   describe('signing transactions', () => {
     // Connected accounts
     const connectedAcct1 = '7ZUECA7HFLZTXENRV24SHLU4AVPUTMTTDUFUBNBD64C73F3UHRTHAIOF6Q'

packages/use-wallet/src/__tests__/wallets/pera2.test.ts

@@ -334,6 +334,53 @@ describe('PeraWallet', () => {
     })
   })
 
+  describe('autoConnect', () => {
+    let mockUserAgent: string
+
+    beforeEach(() => {
+      mockUserAgent = ''
+      vi.clearAllMocks()
+
+      vi.stubGlobal('window', {
+        navigator: {
+          get userAgent() {
+            return mockUserAgent
+          }
+        }
+      })
+    })
+
+    afterEach(() => {
+      vi.unstubAllGlobals()
+    })
+
+    it('should attempt auto-connect in Pera browser', () => {
+      mockUserAgent = 'pera/1.0.0'
+
+      // Mock the private method before creating the wallet
+      vi.spyOn(PeraWallet.prototype as any, 'autoConnect')
+        .mockReset()
+        .mockImplementation(() => Promise.resolve())
+
+      createWalletWithStore(store)
+
+      expect(PeraWallet.prototype['autoConnect']).toHaveBeenCalled()
+    })
+
+    it('should not attempt auto-connect in other browsers', () => {
+      mockUserAgent = 'chrome/1.0.0'
+
+      // Mock the private method before creating the wallet
+      vi.spyOn(PeraWallet.prototype as any, 'autoConnect')
+        .mockReset()
+        .mockImplementation(() => Promise.resolve())
+
+      createWalletWithStore(store)
+
+      expect(PeraWallet.prototype['autoConnect']).not.toHaveBeenCalled()
+    })
+  })
+
   describe('signing transactions', () => {
     // Connected accounts
     const connectedAcct1 = '7ZUECA7HFLZTXENRV24SHLU4AVPUTMTTDUFUBNBD64C73F3UHRTHAIOF6Q'

packages/use-wallet/src/wallets/pera.ts

@@ -47,16 +47,8 @@ export class PeraWallet extends BaseWallet {
 
     if (typeof window !== 'undefined' && window.navigator) {
       const isPeraDiscover = window.navigator.userAgent.includes('pera')
-
       if (isPeraDiscover) {
-        this.logger.info('Pera Discover browser detected, auto connecting...')
-        void this.connect()
-          .then(() => {
-            this.logger.info('Auto-connect successful')
-          })
-          .catch((error) => {
-            this.logger.warn('Auto-connect failed:', error.message)
-          })
+        void this.autoConnect()
       }
     }
   }
@@ -66,6 +58,16 @@ export class PeraWallet extends BaseWallet {
     icon: ICON
   }
 
+  private async autoConnect(): Promise<void> {
+    this.logger.info('Pera Discover browser detected, auto connecting...')
+    try {
+      await this.connect()
+      this.logger.info('Auto-connect successful')
+    } catch (error: any) {
+      this.logger.warn('Auto-connect failed:', error.message)
+    }
+  }
+
   private async initializeClient(): Promise<PeraWalletConnect> {
     this.logger.info('Initializing client...')
     const module = await import('@perawallet/connect')

packages/use-wallet/src/wallets/pera2.ts

@@ -56,16 +56,8 @@ export class PeraWallet extends BaseWallet {
 
     if (typeof window !== 'undefined' && window.navigator) {
       const isPeraDiscover = window.navigator.userAgent.includes('pera')
-
       if (isPeraDiscover) {
-        this.logger.info('Pera Discover browser detected, auto connecting...')
-        void this.connect()
-          .then(() => {
-            this.logger.info('Auto-connect successful')
-          })
-          .catch((error) => {
-            this.logger.warn('Auto-connect failed:', error.message)
-          })
+        void this.autoConnect()
       }
     }
   }
@@ -75,6 +67,16 @@ export class PeraWallet extends BaseWallet {
     icon: ICON
   }
 
+  private async autoConnect(): Promise<void> {
+    this.logger.info('Pera Discover browser detected, auto connecting...')
+    try {
+      await this.connect()
+      this.logger.info('Auto-connect successful')
+    } catch (error: any) {
+      this.logger.warn('Auto-connect failed:', error.message)
+    }
+  }
+
   private async initializeClient(): Promise<PeraWalletConnect> {
     this.logger.info('Initializing client...')
     const module = await import('@perawallet/connect-beta')