[thirdweb] fix: Initialize wallet connection status as unknown

Author: joaquim-verges

.changeset/grumpy-vans-rhyme.md

@@ -0,0 +1,5 @@
+---
+"thirdweb": patch
+---
+
+Initialize activeWalletConnectionStatus as 'unknown' instead of 'disconnected'

packages/thirdweb/src/react/core/hooks/wallets/useConnect.test.tsx

@@ -48,7 +48,7 @@ describe("useAddConnectedWallet", () => {
       },
     );
     const { result } = renderHook(() => useConnect(), { wrapper });
-    expect(statusResult.current).toEqual("disconnected");
+    expect(statusResult.current).toEqual("unknown");
     await result.current.connect(async () => wallet);
     expect(statusResult.current).toEqual("connected");
 
@@ -67,7 +67,7 @@ describe("useAddConnectedWallet", () => {
       },
     );
     const { result } = renderHook(() => useConnect(), { wrapper });
-    expect(statusResult.current).toEqual("disconnected");
+    expect(statusResult.current).toEqual("unknown");
     await result.current.connect(async () => wallet);
     expect(statusResult.current).toEqual("connected");
 
@@ -85,7 +85,7 @@ describe("useAddConnectedWallet", () => {
         wrapper,
       },
     );
-    expect(statusResult.current).toEqual("disconnected");
+    expect(statusResult.current).toEqual("unknown");
     const { result } = renderHook(() => useConnect(), { wrapper });
     await result.current.connect(async () => {
       throw new Error("test");

packages/thirdweb/src/wallets/manager/index.ts

@@ -15,7 +15,11 @@ import type { SmartWalletOptions } from "../smart/types.js";
 import type { WalletId } from "../wallet-types.js";
 
 type WalletIdToConnectedWalletMap = Map<string, Wallet>;
-export type ConnectionStatus = "connected" | "disconnected" | "connecting";
+export type ConnectionStatus =
+  | "connected"
+  | "disconnected"
+  | "connecting"
+  | "unknown";
 
 const CONNECTED_WALLET_IDS = "thirdweb:connected-wallet-ids";
 const LAST_ACTIVE_EOA_ID = "thirdweb:active-wallet-id";
@@ -47,7 +51,7 @@ export function createConnectionManager(storage: AsyncStorage) {
   const activeAccountStore = createStore<Account | undefined>(undefined);
   const activeWalletChainStore = createStore<Chain | undefined>(undefined);
   const activeWalletConnectionStatusStore =
-    createStore<ConnectionStatus>("disconnected");
+    createStore<ConnectionStatus>("unknown");
 
   const definedChainsStore = createStore<Map<number, Chain>>(new Map());
 

packages/thirdweb/test/vitest.config.ts

@@ -38,7 +38,9 @@ export default defineConfig({
     globalSetup: [join(__dirname, "./globalSetup.ts")],
     testTimeout: 90_000,
     retry: 3,
-    maxConcurrency: 3,
+    maxConcurrency: 1,
+    maxWorkers: 1,
+    minWorkers: 1,
     bail: 1,
     // clear any mocks between any tests
     clearMocks: true,