@0xsequence/wallet-wdk 3.0.0-beta.9 → 3.0.0

package/src/sequence/wallets.ts

@@ -1,17 +1,17 @@
 import { Wallet as CoreWallet, Envelope, Signers, State } from '@0xsequence/wallet-core'
-import { Config, Constants, GenericTree, Payload, SessionConfig } from '@0xsequence/wallet-primitives'
+import { Config, Constants, Payload } from '@0xsequence/wallet-primitives'
 import { Address, Hex, Provider, RpcTransport } from 'ox'
 import { AuthCommitment } from '../dbs/auth-commitments.js'
 import { AuthCodeHandler } from './handlers/authcode.js'
 import { MnemonicHandler } from './handlers/mnemonic.js'
 import { OtpHandler } from './handlers/otp.js'
-import { ManagerOptionsDefaults, Shared } from './manager.js'
+import { Shared } from './manager.js'
 import { Device } from './types/device.js'
 import { Action, Module } from './types/index.js'
 import { Kinds, SignerWithKind, WitnessExtraSignerKind } from './types/signer.js'
 import { Wallet, WalletSelectionUiHandler } from './types/wallet.js'
 import { PasskeysHandler } from './handlers/passkeys.js'
-import { GuardRole } from './guards.js'
+import type { PasskeySigner } from './passkeys-provider.js'
 
 export type StartSignUpWithRedirectArgs = {
   kind: 'google-pkce' | 'apple' | `custom-${string}`
@@ -417,17 +417,17 @@ function buildCappedTree(members: { address: Address.Address; imageHash?: Hex.He
   } as Config.NestedLeaf
 }
 
-function buildCappedTreeFromTopology(weight: bigint, topology: Config.Topology): Config.Topology {
-  // We may optimize this for some topology types
-  // but it is not worth it, because the topology
-  // that we will use for prod won't be optimizable
-  return {
-    type: 'nested',
-    weight: weight,
-    threshold: weight,
-    tree: topology,
-  }
-}
+// function buildCappedTreeFromTopology(weight: bigint, topology: Config.Topology): Config.Topology {
+//   // We may optimize this for some topology types
+//   // but it is not worth it, because the topology
+//   // that we will use for prod won't be optimizable
+//   return {
+//     type: 'nested',
+//     weight: weight,
+//     threshold: weight,
+//     tree: topology,
+//   }
+// }
 
 function toConfig(
   checkpoint: bigint,
@@ -625,8 +625,8 @@ export class Wallets implements WalletsInterface {
     loginEmail?: string
   }> {
     switch (args.kind) {
-      case 'passkey':
-        const passkeySigner = await Signers.Passkey.Passkey.create(this.shared.sequence.extensions, {
+      case 'passkey': {
+        const passkeySigner = await this.shared.passkeyProvider.create(this.shared.sequence.extensions, {
           stateProvider: this.shared.sequence.stateProvider,
           credentialName: args.name,
         })
@@ -638,8 +638,9 @@ export class Wallets implements WalletsInterface {
             signerKind: Kinds.LoginPasskey,
           },
         }
+      }
 
-      case 'mnemonic':
+      case 'mnemonic': {
         const mnemonicSigner = MnemonicHandler.toSigner(args.mnemonic)
         if (!mnemonicSigner) {
           throw new Error('invalid-mnemonic')
@@ -653,6 +654,7 @@ export class Wallets implements WalletsInterface {
             signerKind: Kinds.LoginMnemonic,
           },
         }
+      }
 
       case 'email-otp': {
         const handler = this.shared.handlers.get(Kinds.LoginEmailOtp) as OtpHandler
@@ -834,7 +836,7 @@ export class Wallets implements WalletsInterface {
     const sessionsGuardTopology = args.noGuard ? undefined : this.shared.modules.guards.topology('sessions')
 
     // Add modules
-    let modules: Module[] = []
+    const modules: Module[] = []
 
     if (!args.noSessionManager) {
       const identitySigners = [device.address]
@@ -891,7 +893,7 @@ export class Wallets implements WalletsInterface {
     }
 
     // Store passkey credential ID mapping if this is a passkey signup
-    if (args.kind === 'passkey' && loginSigner.signer instanceof Signers.Passkey.Passkey) {
+    if (args.kind === 'passkey' && this.isPasskeySigner(loginSigner.signer)) {
       try {
         await this.shared.databases.passkeyCredentials.saveCredential(
           loginSigner.signer.credentialId,
@@ -1043,8 +1045,6 @@ export class Wallets implements WalletsInterface {
         })
 
         return requestId
-      } catch (error) {
-        throw error
       } finally {
         this.pendingMnemonicOrPasskeyLogin = undefined
       }
@@ -1075,7 +1075,7 @@ export class Wallets implements WalletsInterface {
     }
 
     if (isLoginToPasskeyArgs(args)) {
-      let passkeySigner: Signers.Passkey.Passkey
+      let passkeySigner: PasskeySigner
 
       if (args.credentialId) {
         // Application-controlled login: use the provided credentialId
@@ -1087,7 +1087,7 @@ export class Wallets implements WalletsInterface {
         }
 
         // Create passkey signer from stored credential
-        passkeySigner = new Signers.Passkey.Passkey({
+        passkeySigner = this.shared.passkeyProvider.fromCredential({
           credentialId: credential.credentialId,
           publicKey: credential.publicKey,
           extensions: this.shared.sequence.extensions,
@@ -1098,7 +1098,7 @@ export class Wallets implements WalletsInterface {
         // Default discovery behavior: use WebAuthn discovery
         this.shared.modules.logger.log('No credentialId provided, using discovery method')
 
-        const foundPasskeySigner = await Signers.Passkey.Passkey.find(
+        const foundPasskeySigner = await this.shared.passkeyProvider.find(
           this.shared.sequence.stateProvider,
           this.shared.sequence.extensions,
         )
@@ -1153,6 +1153,20 @@ export class Wallets implements WalletsInterface {
     throw new Error('invalid-login-args')
   }
 
+  private isPasskeySigner(signer: unknown): signer is PasskeySigner {
+    const guard = this.shared.passkeyProvider.isSigner
+    if (guard) {
+      return guard(signer)
+    }
+    return (
+      typeof signer === 'object' &&
+      signer !== null &&
+      'credentialId' in signer &&
+      'publicKey' in signer &&
+      'imageHash' in signer
+    )
+  }
+
   async completeLogin(requestId: string) {
     const request = await this.shared.modules.signatures.get(requestId)
 
@@ -1220,7 +1234,7 @@ export class Wallets implements WalletsInterface {
     return requestId
   }
 
-  async completeLogout(requestId: string, options?: { skipValidateSave?: boolean }) {
+  async completeLogout(requestId: string, _options?: { skipValidateSave?: boolean }) {
     const request = await this.shared.modules.signatures.get(requestId)
     const walletEntry = await this.shared.databases.manager.get(request.wallet)
     if (!walletEntry) {

package/test/authcode-pkce.test.ts

@@ -1,5 +1,4 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { Address, Hex, Bytes } from 'ox'
 import * as Identity from '@0xsequence/identity-instrument'
 import { AuthCodePkceHandler } from '../src/sequence/handlers/authcode-pkce.js'
 import { Signatures } from '../src/sequence/signatures.js'
@@ -68,7 +67,7 @@ describe('AuthCodePkceHandler', () => {
     handler.setRedirectUri('https://example.com/auth/callback')
 
     // Mock inherited methods
-    vi.spyOn(handler as any, 'nitroCommitVerifier').mockImplementation(async (challenge) => {
+    vi.spyOn(handler as any, 'nitroCommitVerifier').mockImplementation(async () => {
       return {
         verifier: 'mock-verifier-code',
         loginHint: 'user@example.com',
@@ -76,7 +75,7 @@ describe('AuthCodePkceHandler', () => {
       }
     })
 
-    vi.spyOn(handler as any, 'nitroCompleteAuth').mockImplementation(async (challenge) => {
+    vi.spyOn(handler as any, 'nitroCompleteAuth').mockImplementation(async () => {
       return {
         signer: mockIdentitySigner,
         email: 'user@example.com',

package/test/authcode.test.ts

@@ -1,5 +1,5 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { Address, Hex, Bytes } from 'ox'
+import { Address, Hex } from 'ox'
 import { Network, Payload } from '@0xsequence/wallet-primitives'
 import { IdentityInstrument, IdentityType, KeyType, AuthCodeChallenge } from '@0xsequence/identity-instrument'
 import { AuthCodeHandler } from '../src/sequence/handlers/authcode.js'
@@ -45,8 +45,8 @@ class MockURLSearchParams {
   }
 }
 
-// @ts-ignore - Override global URLSearchParams for testing
-global.URLSearchParams = MockURLSearchParams as any
+// Override global URLSearchParams for testing
+globalThis.URLSearchParams = MockURLSearchParams as any
 
 // Mock dependencies with proper vi.fn() types
 const mockCommitVerifier = vi.fn()
@@ -285,8 +285,7 @@ describe('AuthCodeHandler', () => {
     })
 
     it('Should generate random state when not provided', async () => {
-      const result = await authCodeHandler.commitAuth('/target', false)
-
+      await authCodeHandler.commitAuth('/target', false)
       const commitmentCall = mockAuthCommitmentsSet.mock.calls[0]![0]!
       expect(commitmentCall.id).toBeDefined()
       expect(typeof commitmentCall.id).toBe('string')
@@ -316,8 +315,7 @@ describe('AuthCodeHandler', () => {
     })
 
     it('Should create commitment without signer', async () => {
-      const result = await authCodeHandler.commitAuth('/target', true)
-
+      await authCodeHandler.commitAuth('/target', true)
       const commitmentCall = mockAuthCommitmentsSet.mock.calls[0]![0]!
       expect(commitmentCall.signer).toBeUndefined()
       expect(commitmentCall.isSignUp).toBe(true)
@@ -329,7 +327,7 @@ describe('AuthCodeHandler', () => {
   describe('completeAuth()', () => {
     it('Should complete auth flow with code and return signer', async () => {
       const authCode = 'test-auth-code-123'
-      const mockSigner = {} as IdentitySigner
+      // const mockSigner = {} as IdentitySigner
       const mockEmail = 'test@example.com'
 
       mockCommitVerifier.mockResolvedValueOnce(undefined)

package/test/constants.ts

@@ -7,6 +7,7 @@ import { Relayer } from '@0xsequence/relayer'
 import * as Db from '../src/dbs/index.js'
 import { Network } from '@0xsequence/wallet-primitives'
 
+// eslint-disable-next-line turbo/no-undeclared-env-vars
 const envFile = process.env.CI ? '.env.test' : '.env.test.local'
 dotenvConfig({ path: envFile })
 
@@ -14,6 +15,7 @@ export const EMITTER_ADDRESS: Address.Address = '0xb7bE532959236170064cf099e1a33
 export const EMITTER_ABI = Abi.from(['function explicitEmit()', 'function implicitEmit()'])
 
 // Environment variables
+// eslint-disable-next-line turbo/no-undeclared-env-vars
 export const LOCAL_RPC_URL = process.env.LOCAL_RPC_URL || 'http://localhost:8545'
 
 let testIdCounter = 0
@@ -81,8 +83,8 @@ export function newRemoteManager(
     ? `_${remoteManagerOptions.tag}_testrun_${testIdCounter}`
     : `_testrun_${testIdCounter}`
 
-  let relayers: Relayer.Relayer[] = []
-  let bundlers: Bundler.Bundler[] = []
+  const relayers: Relayer.Relayer[] = []
+  const bundlers: Bundler.Bundler[] = []
 
   if (remoteManagerOptions.network.relayerPk) {
     const provider = Provider.from(RpcTransport.fromHttp(remoteManagerOptions.network.rpcUrl))

package/test/guard.test.ts

@@ -5,7 +5,7 @@ import { Address, Bytes, Hex, TypedData } from 'ox'
 import { Config, Constants, Network, Payload } from '@0xsequence/wallet-primitives'
 import { Kinds } from '../src/sequence/types/signer.js'
 import { newManager } from './constants.js'
-import { GuardRole, Guards } from '../src/sequence/guards.js'
+import { Guards } from '../src/sequence/guards.js'
 
 // Mock fetch globally for guard API calls
 const mockFetch = vi.fn()
@@ -16,8 +16,8 @@ describe('GuardHandler', () => {
   let guards: Guards
   let testWallet: Address.Address
   let testPayload: Payload.Payload
-  let testMessageDigest: Bytes.Bytes
-  let testMessage: Hex.Hex
+  let _testMessageDigest: Bytes.Bytes
+  let _testMessage: Hex.Hex
 
   beforeEach(async () => {
     vi.clearAllMocks()
@@ -28,8 +28,8 @@ describe('GuardHandler', () => {
 
     testWallet = '0x1234567890123456789012345678901234567890' as Address.Address
     testPayload = Payload.fromMessage(Hex.fromString('Test message'))
-    testMessage = TypedData.encode(Payload.toTyped(testWallet, Network.ChainId.ARBITRUM, testPayload))
-    testMessageDigest = Payload.hash(testWallet, Network.ChainId.ARBITRUM, testPayload)
+    _testMessage = TypedData.encode(Payload.toTyped(testWallet, Network.ChainId.ARBITRUM, testPayload))
+    _testMessageDigest = Payload.hash(testWallet, Network.ChainId.ARBITRUM, testPayload)
   })
 
   afterEach(async () => {

package/test/identity-signer.test.ts

@@ -1,5 +1,5 @@
 import { afterEach, beforeEach, describe, expect, it, Mock, vi } from 'vitest'
-import { Address, Bytes, Hex } from 'ox'
+import { Address, Hex } from 'ox'
 import { Network, Payload } from '@0xsequence/wallet-primitives'
 import { IdentityInstrument, KeyType } from '@0xsequence/identity-instrument'
 import { State } from '@0xsequence/wallet-core'

package/test/otp.test.ts

@@ -1,7 +1,7 @@
 import { afterEach, beforeEach, describe, expect, it, Mock, vi } from 'vitest'
 import { Address, Hex } from 'ox'
 import { Network, Payload } from '@0xsequence/wallet-primitives'
-import { IdentityInstrument, IdentityType, KeyType, OtpChallenge } from '@0xsequence/identity-instrument'
+import { IdentityInstrument, IdentityType, KeyType } from '@0xsequence/identity-instrument'
 import { OtpHandler, PromptOtpHandler } from '../src/sequence/handlers/otp.js'
 import { Signatures } from '../src/sequence/signatures.js'
 import * as Db from '../src/dbs/index.js'

package/test/passkeys.test.ts

@@ -1,7 +1,7 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 import { Address, Hex } from 'ox'
 import { Network, Payload } from '@0xsequence/wallet-primitives'
-import { Signers, State } from '@0xsequence/wallet-core'
+import { State } from '@0xsequence/wallet-core'
 import { Extensions } from '@0xsequence/wallet-primitives'
 import { PasskeysHandler } from '../src/sequence/handlers/passkeys.js'
 import { Signatures } from '../src/sequence/signatures.js'

package/test/recovery.test.ts

@@ -174,7 +174,7 @@ describe('Recovery', () => {
     const recoverySigner = request4.signers.find((s) => s.handler?.kind === 'recovery-extension')
     expect(recoverySigner).toBeDefined()
     expect(recoverySigner!.status).toBe('ready')
-    1
+
     // Handle the recovery signer
     const result4 = await (recoverySigner as SignerReady).handle()
     expect(result4).toBeDefined()
@@ -303,9 +303,9 @@ describe('Recovery', () => {
     })
 
     // Set up mnemonic handler and sign the payload
-    let handledMnemonic2 = 0
+    let _handledMnemonic2 = 0
     const unregisterHandler = manager.registerMnemonicUI(async (respond) => {
-      handledMnemonic2++
+      _handledMnemonic2++
       await respond(mnemonic2)
     })
 

package/test/sessions.test.ts

@@ -358,7 +358,7 @@ for (const extension of ALL_EXTENSIONS) {
         throw new Error('Failed to create pk store')
       }
       // Create the initial permissions
-      let explicitSession: ExplicitSession = {
+      const explicitSession: ExplicitSession = {
         type: 'explicit',
         sessionAddress: e.address,
         chainId,