@eth-optimism/actions-sdk 0.0.2

package/src/wallet/core/wallets/smart/default/constants/index.ts

@@ -0,0 +1,482 @@
+export const smartWalletFactoryAddress =
+  '0xBA5ED110eFDBa3D005bfC882d75358ACBbB85842'
+
+export const smartWalletAbi = [
+  { type: 'constructor', inputs: [], stateMutability: 'nonpayable' },
+  { type: 'fallback', stateMutability: 'payable' },
+  { type: 'receive', stateMutability: 'payable' },
+  {
+    type: 'function',
+    name: 'REPLAYABLE_NONCE_KEY',
+    inputs: [],
+    outputs: [{ name: '', type: 'uint256', internalType: 'uint256' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'addOwnerAddress',
+    inputs: [{ name: 'owner', type: 'address', internalType: 'address' }],
+    outputs: [],
+    stateMutability: 'nonpayable',
+  },
+  {
+    type: 'function',
+    name: 'addOwnerPublicKey',
+    inputs: [
+      { name: 'x', type: 'bytes32', internalType: 'bytes32' },
+      { name: 'y', type: 'bytes32', internalType: 'bytes32' },
+    ],
+    outputs: [],
+    stateMutability: 'nonpayable',
+  },
+  {
+    type: 'function',
+    name: 'canSkipChainIdValidation',
+    inputs: [
+      { name: 'functionSelector', type: 'bytes4', internalType: 'bytes4' },
+    ],
+    outputs: [{ name: '', type: 'bool', internalType: 'bool' }],
+    stateMutability: 'pure',
+  },
+  {
+    type: 'function',
+    name: 'domainSeparator',
+    inputs: [],
+    outputs: [{ name: '', type: 'bytes32', internalType: 'bytes32' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'eip712Domain',
+    inputs: [],
+    outputs: [
+      { name: 'fields', type: 'bytes1', internalType: 'bytes1' },
+      { name: 'name', type: 'string', internalType: 'string' },
+      { name: 'version', type: 'string', internalType: 'string' },
+      { name: 'chainId', type: 'uint256', internalType: 'uint256' },
+      { name: 'verifyingContract', type: 'address', internalType: 'address' },
+      { name: 'salt', type: 'bytes32', internalType: 'bytes32' },
+      { name: 'extensions', type: 'uint256[]', internalType: 'uint256[]' },
+    ],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'entryPoint',
+    inputs: [],
+    outputs: [{ name: '', type: 'address', internalType: 'address' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'execute',
+    inputs: [
+      { name: 'target', type: 'address', internalType: 'address' },
+      { name: 'value', type: 'uint256', internalType: 'uint256' },
+      { name: 'data', type: 'bytes', internalType: 'bytes' },
+    ],
+    outputs: [],
+    stateMutability: 'payable',
+  },
+  {
+    type: 'function',
+    name: 'executeBatch',
+    inputs: [
+      {
+        name: 'calls',
+        type: 'tuple[]',
+        internalType: 'struct CoinbaseSmartWallet.Call[]',
+        components: [
+          { name: 'target', type: 'address', internalType: 'address' },
+          { name: 'value', type: 'uint256', internalType: 'uint256' },
+          { name: 'data', type: 'bytes', internalType: 'bytes' },
+        ],
+      },
+    ],
+    outputs: [],
+    stateMutability: 'payable',
+  },
+  {
+    type: 'function',
+    name: 'executeWithoutChainIdValidation',
+    inputs: [{ name: 'calls', type: 'bytes[]', internalType: 'bytes[]' }],
+    outputs: [],
+    stateMutability: 'payable',
+  },
+  {
+    type: 'function',
+    name: 'getUserOpHashWithoutChainId',
+    inputs: [
+      {
+        name: 'userOp',
+        type: 'tuple',
+        internalType: 'struct UserOperation',
+        components: [
+          { name: 'sender', type: 'address', internalType: 'address' },
+          { name: 'nonce', type: 'uint256', internalType: 'uint256' },
+          { name: 'initCode', type: 'bytes', internalType: 'bytes' },
+          { name: 'callData', type: 'bytes', internalType: 'bytes' },
+          { name: 'callGasLimit', type: 'uint256', internalType: 'uint256' },
+          {
+            name: 'verificationGasLimit',
+            type: 'uint256',
+            internalType: 'uint256',
+          },
+          {
+            name: 'preVerificationGas',
+            type: 'uint256',
+            internalType: 'uint256',
+          },
+          { name: 'maxFeePerGas', type: 'uint256', internalType: 'uint256' },
+          {
+            name: 'maxPriorityFeePerGas',
+            type: 'uint256',
+            internalType: 'uint256',
+          },
+          { name: 'paymasterAndData', type: 'bytes', internalType: 'bytes' },
+          { name: 'signature', type: 'bytes', internalType: 'bytes' },
+        ],
+      },
+    ],
+    outputs: [{ name: '', type: 'bytes32', internalType: 'bytes32' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'implementation',
+    inputs: [],
+    outputs: [{ name: '$', type: 'address', internalType: 'address' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'initialize',
+    inputs: [{ name: 'owners', type: 'bytes[]', internalType: 'bytes[]' }],
+    outputs: [],
+    stateMutability: 'payable',
+  },
+  {
+    type: 'function',
+    name: 'isOwnerAddress',
+    inputs: [{ name: 'account', type: 'address', internalType: 'address' }],
+    outputs: [{ name: '', type: 'bool', internalType: 'bool' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'isOwnerBytes',
+    inputs: [{ name: 'account', type: 'bytes', internalType: 'bytes' }],
+    outputs: [{ name: '', type: 'bool', internalType: 'bool' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'isOwnerPublicKey',
+    inputs: [
+      { name: 'x', type: 'bytes32', internalType: 'bytes32' },
+      { name: 'y', type: 'bytes32', internalType: 'bytes32' },
+    ],
+    outputs: [{ name: '', type: 'bool', internalType: 'bool' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'isValidSignature',
+    inputs: [
+      { name: 'hash', type: 'bytes32', internalType: 'bytes32' },
+      { name: 'signature', type: 'bytes', internalType: 'bytes' },
+    ],
+    outputs: [{ name: 'result', type: 'bytes4', internalType: 'bytes4' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'nextOwnerIndex',
+    inputs: [],
+    outputs: [{ name: '', type: 'uint256', internalType: 'uint256' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'ownerAtIndex',
+    inputs: [{ name: 'index', type: 'uint256', internalType: 'uint256' }],
+    outputs: [{ name: '', type: 'bytes', internalType: 'bytes' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'ownerCount',
+    inputs: [],
+    outputs: [{ name: '', type: 'uint256', internalType: 'uint256' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'proxiableUUID',
+    inputs: [],
+    outputs: [{ name: '', type: 'bytes32', internalType: 'bytes32' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'removeLastOwner',
+    inputs: [
+      { name: 'index', type: 'uint256', internalType: 'uint256' },
+      { name: 'owner', type: 'bytes', internalType: 'bytes' },
+    ],
+    outputs: [],
+    stateMutability: 'nonpayable',
+  },
+  {
+    type: 'function',
+    name: 'removeOwnerAtIndex',
+    inputs: [
+      { name: 'index', type: 'uint256', internalType: 'uint256' },
+      { name: 'owner', type: 'bytes', internalType: 'bytes' },
+    ],
+    outputs: [],
+    stateMutability: 'nonpayable',
+  },
+  {
+    type: 'function',
+    name: 'removedOwnersCount',
+    inputs: [],
+    outputs: [{ name: '', type: 'uint256', internalType: 'uint256' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'replaySafeHash',
+    inputs: [{ name: 'hash', type: 'bytes32', internalType: 'bytes32' }],
+    outputs: [{ name: '', type: 'bytes32', internalType: 'bytes32' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'upgradeToAndCall',
+    inputs: [
+      { name: 'newImplementation', type: 'address', internalType: 'address' },
+      { name: 'data', type: 'bytes', internalType: 'bytes' },
+    ],
+    outputs: [],
+    stateMutability: 'payable',
+  },
+  {
+    type: 'function',
+    name: 'validateUserOp',
+    inputs: [
+      {
+        name: 'userOp',
+        type: 'tuple',
+        internalType: 'struct UserOperation',
+        components: [
+          { name: 'sender', type: 'address', internalType: 'address' },
+          { name: 'nonce', type: 'uint256', internalType: 'uint256' },
+          { name: 'initCode', type: 'bytes', internalType: 'bytes' },
+          { name: 'callData', type: 'bytes', internalType: 'bytes' },
+          { name: 'callGasLimit', type: 'uint256', internalType: 'uint256' },
+          {
+            name: 'verificationGasLimit',
+            type: 'uint256',
+            internalType: 'uint256',
+          },
+          {
+            name: 'preVerificationGas',
+            type: 'uint256',
+            internalType: 'uint256',
+          },
+          { name: 'maxFeePerGas', type: 'uint256', internalType: 'uint256' },
+          {
+            name: 'maxPriorityFeePerGas',
+            type: 'uint256',
+            internalType: 'uint256',
+          },
+          { name: 'paymasterAndData', type: 'bytes', internalType: 'bytes' },
+          { name: 'signature', type: 'bytes', internalType: 'bytes' },
+        ],
+      },
+      { name: 'userOpHash', type: 'bytes32', internalType: 'bytes32' },
+      { name: 'missingAccountFunds', type: 'uint256', internalType: 'uint256' },
+    ],
+    outputs: [
+      { name: 'validationData', type: 'uint256', internalType: 'uint256' },
+    ],
+    stateMutability: 'nonpayable',
+  },
+  {
+    type: 'event',
+    name: 'AddOwner',
+    inputs: [
+      {
+        name: 'index',
+        type: 'uint256',
+        indexed: true,
+        internalType: 'uint256',
+      },
+      { name: 'owner', type: 'bytes', indexed: false, internalType: 'bytes' },
+    ],
+    anonymous: false,
+  },
+  {
+    type: 'event',
+    name: 'RemoveOwner',
+    inputs: [
+      {
+        name: 'index',
+        type: 'uint256',
+        indexed: true,
+        internalType: 'uint256',
+      },
+      { name: 'owner', type: 'bytes', indexed: false, internalType: 'bytes' },
+    ],
+    anonymous: false,
+  },
+  {
+    type: 'event',
+    name: 'Upgraded',
+    inputs: [
+      {
+        name: 'implementation',
+        type: 'address',
+        indexed: true,
+        internalType: 'address',
+      },
+    ],
+    anonymous: false,
+  },
+  {
+    type: 'error',
+    name: 'AlreadyOwner',
+    inputs: [{ name: 'owner', type: 'bytes', internalType: 'bytes' }],
+  },
+  { type: 'error', name: 'Initialized', inputs: [] },
+  {
+    type: 'error',
+    name: 'InvalidEthereumAddressOwner',
+    inputs: [{ name: 'owner', type: 'bytes', internalType: 'bytes' }],
+  },
+  {
+    type: 'error',
+    name: 'InvalidImplementation',
+    inputs: [
+      { name: 'implementation', type: 'address', internalType: 'address' },
+    ],
+  },
+  {
+    type: 'error',
+    name: 'InvalidNonceKey',
+    inputs: [{ name: 'key', type: 'uint256', internalType: 'uint256' }],
+  },
+  {
+    type: 'error',
+    name: 'InvalidOwnerBytesLength',
+    inputs: [{ name: 'owner', type: 'bytes', internalType: 'bytes' }],
+  },
+  { type: 'error', name: 'LastOwner', inputs: [] },
+  {
+    type: 'error',
+    name: 'NoOwnerAtIndex',
+    inputs: [{ name: 'index', type: 'uint256', internalType: 'uint256' }],
+  },
+  {
+    type: 'error',
+    name: 'NotLastOwner',
+    inputs: [
+      { name: 'ownersRemaining', type: 'uint256', internalType: 'uint256' },
+    ],
+  },
+  {
+    type: 'error',
+    name: 'SelectorNotAllowed',
+    inputs: [{ name: 'selector', type: 'bytes4', internalType: 'bytes4' }],
+  },
+  { type: 'error', name: 'Unauthorized', inputs: [] },
+  { type: 'error', name: 'UnauthorizedCallContext', inputs: [] },
+  { type: 'error', name: 'UpgradeFailed', inputs: [] },
+  {
+    type: 'error',
+    name: 'WrongOwnerAtIndex',
+    inputs: [
+      { name: 'index', type: 'uint256', internalType: 'uint256' },
+      { name: 'expectedOwner', type: 'bytes', internalType: 'bytes' },
+      { name: 'actualOwner', type: 'bytes', internalType: 'bytes' },
+    ],
+  },
+] as const
+
+export const smartWalletFactoryAbi = [
+  {
+    type: 'constructor',
+    inputs: [
+      { name: 'implementation_', type: 'address', internalType: 'address' },
+    ],
+    stateMutability: 'payable',
+  },
+  {
+    type: 'function',
+    name: 'createAccount',
+    inputs: [
+      { name: 'owners', type: 'bytes[]', internalType: 'bytes[]' },
+      { name: 'nonce', type: 'uint256', internalType: 'uint256' },
+    ],
+    outputs: [
+      {
+        name: 'account',
+        type: 'address',
+        internalType: 'contract CoinbaseSmartWallet',
+      },
+    ],
+    stateMutability: 'payable',
+  },
+  {
+    type: 'function',
+    name: 'getAddress',
+    inputs: [
+      { name: 'owners', type: 'bytes[]', internalType: 'bytes[]' },
+      { name: 'nonce', type: 'uint256', internalType: 'uint256' },
+    ],
+    outputs: [{ name: '', type: 'address', internalType: 'address' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'implementation',
+    inputs: [],
+    outputs: [{ name: '', type: 'address', internalType: 'address' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'function',
+    name: 'initCodeHash',
+    inputs: [],
+    outputs: [{ name: '', type: 'bytes32', internalType: 'bytes32' }],
+    stateMutability: 'view',
+  },
+  {
+    type: 'event',
+    name: 'AccountCreated',
+    inputs: [
+      {
+        name: 'account',
+        type: 'address',
+        indexed: true,
+        internalType: 'address',
+      },
+      {
+        name: 'owners',
+        type: 'bytes[]',
+        indexed: false,
+        internalType: 'bytes[]',
+      },
+      {
+        name: 'nonce',
+        type: 'uint256',
+        indexed: false,
+        internalType: 'uint256',
+      },
+    ],
+    anonymous: false,
+  },
+  { type: 'error', name: 'ImplementationUndeployed', inputs: [] },
+  { type: 'error', name: 'OwnerRequired', inputs: [] },
+] as const

package/src/wallet/core/wallets/smart/default/utils/__tests__/findSignerInArray.spec.ts

@@ -0,0 +1,68 @@
+import type { Address, Hex, LocalAccount } from 'viem'
+import { pad } from 'viem'
+import type { WebAuthnAccount } from 'viem/account-abstraction'
+import { describe, expect, it } from 'vitest'
+
+import { getRandomAddress } from '@/test/utils.js'
+import { findSignerInArray } from '@/wallet/core/wallets/smart/default/utils/findSignerInArray.js'
+
+describe('findSignerInArray', () => {
+  const mockAddress1 = getRandomAddress()
+  const mockAddress2 = getRandomAddress()
+  const mockPublicKey1 = pad('0xaabb', { size: 32 }) as Hex
+
+  it('should find LocalAccount address in owners array with EOA addresses', () => {
+    const owners = [mockAddress1, mockAddress2]
+    const signer = {
+      address: mockAddress1,
+      type: 'local',
+    } as unknown as LocalAccount
+
+    const index = findSignerInArray(owners, signer)
+
+    expect(index).toBe(0)
+  })
+
+  it('should find LocalAccount in owners array by matching address', () => {
+    const owners = [mockAddress1, mockAddress2]
+    const signer = {
+      address: mockAddress2,
+      type: 'local',
+    } as unknown as LocalAccount
+
+    const index = findSignerInArray(owners, signer)
+
+    expect(index).toBe(1)
+  })
+
+  it('should return -1 when LocalAccount address is not found in owners', () => {
+    const owners = [mockAddress1, mockAddress2]
+    const signer = {
+      address: '0x3333333333333333333333333333333333333333' as Address,
+      type: 'local',
+    } as unknown as LocalAccount
+
+    const index = findSignerInArray(owners, signer)
+
+    expect(index).toBe(-1)
+  })
+
+  it('should ignore WebAuthn accounts in owners array and only match addresses', () => {
+    const webAuthnAccount: WebAuthnAccount = {
+      type: 'webAuthn',
+      publicKey: mockPublicKey1,
+      address: mockAddress2,
+    } as unknown as WebAuthnAccount
+
+    const owners = [mockAddress1, webAuthnAccount]
+    const signer = {
+      address: mockAddress2,
+      type: 'local',
+    } as unknown as LocalAccount
+
+    // Should return -1 because we only match EOA addresses, not WebAuthn accounts
+    const index = findSignerInArray(owners, signer)
+
+    expect(index).toBe(-1)
+  })
+})

package/src/wallet/core/wallets/smart/default/utils/__tests__/findSignerIndexOnChain.spec.ts

@@ -0,0 +1,147 @@
+import { createPublicClient, http, pad } from 'viem'
+import type { WebAuthnAccount } from 'viem/account-abstraction'
+import { readContract } from 'viem/actions'
+import type { Mock } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
+
+import { findSignerIndexOnChain } from '@/wallet/core/wallets/smart/default/utils/findSignerIndexOnChain.js'
+
+const client = createPublicClient({
+  transport: http('http://localhost:8545'),
+})
+
+vi.mock('viem/actions', () => ({
+  readContract: vi.fn(),
+  getCode: vi.fn(),
+}))
+
+describe('findSignerIndexOnChain', () => {
+  it('returns correct index when owner found', async () => {
+    const mockReadContract = vi
+      .fn()
+      .mockResolvedValueOnce(BigInt(3)) // ownerCount
+      .mockResolvedValueOnce(
+        '0x00000000000000000000000046440ECd6746f7612809eFED388347d476369f6D',
+      ) // owner at index 2
+      .mockResolvedValueOnce(
+        '0x000000000000000000000000d9Ec1a8603125732c1ee35147619BbFA769A062b',
+      ) // owner at index 1
+      .mockResolvedValueOnce(
+        '0x0000000000000000000000007838d2724FC686813CAf81d4429beff1110c739a',
+      ) // owner at index 0
+
+    ;(readContract as Mock).mockImplementation(mockReadContract)
+
+    const result = await findSignerIndexOnChain({
+      address: '0xe6c7D51b0d5ECC217BE74019447aeac4580Afb54',
+      client,
+      signerPublicKey: '0x7838d2724FC686813CAf81d4429beff1110c739a',
+    })
+
+    expect(result).toBe(0)
+    expect(mockReadContract).toHaveBeenCalledTimes(4)
+  })
+
+  it('only calls readContract as needed', async () => {
+    const mockReadContract = vi
+      .fn()
+      .mockResolvedValueOnce(BigInt(3)) // nextOwnerIndex
+      .mockResolvedValueOnce(
+        '0x00000000000000000000000046440ECd6746f7612809eFED388347d476369f6D',
+      ) // owner at index 2
+      .mockResolvedValueOnce(
+        '0x000000000000000000000000d9Ec1a8603125732c1ee35147619BbFA769A062b',
+      ) // owner at index 1
+      .mockResolvedValueOnce(
+        '0x0000000000000000000000007838d2724FC686813CAf81d4429beff1110c739a',
+      ) // owner at index 0
+
+    ;(readContract as Mock).mockImplementation(mockReadContract)
+
+    const result = await findSignerIndexOnChain({
+      address: '0xe6c7D51b0d5ECC217BE74019447aeac4580Afb54',
+      client,
+      signerPublicKey: '0x46440ECd6746f7612809eFED388347d476369f6D',
+    })
+
+    expect(result).toBe(2)
+    expect(mockReadContract).toHaveBeenCalledTimes(2)
+  })
+
+  it('handles 64 byte public keys', async () => {
+    const mockReadContract = vi
+      .fn()
+      .mockResolvedValueOnce(BigInt(2)) // nextOwnerIndex
+      .mockResolvedValueOnce(
+        '0xe7575170745fe55d7a26190c6d5504743496c49498b129d2b3660da3697e81d4daebb2496f89aa4a05f1705e1d5d316153211c198f80d3100b51489bf4963f47',
+      ) // owner at index 1
+      .mockResolvedValueOnce(
+        '0x0000000000000000000000007838d2724FC686813CAf81d4429beff1110c739a',
+      ) // owner at index 0
+
+    ;(readContract as Mock).mockImplementation(mockReadContract)
+
+    const webAuthnSigner = {
+      type: 'webAuthn',
+      publicKey:
+        '0xe7575170745fe55d7a26190c6d5504743496c49498b129d2b3660da3697e81d4daebb2496f89aa4a05f1705e1d5d316153211c198f80d3100b51489bf4963f47',
+    } as unknown as WebAuthnAccount
+
+    const result = await findSignerIndexOnChain({
+      address: '0xe6c7D51b0d5ECC217BE74019447aeac4580Afb54',
+      client,
+      signerPublicKey: webAuthnSigner.publicKey,
+    })
+
+    expect(result).toBe(1)
+    expect(mockReadContract).toHaveBeenCalledTimes(2)
+  })
+
+  it('is case insensitive when matching signers', async () => {
+    const mockReadContract = vi
+      .fn()
+      .mockResolvedValueOnce(BigInt(2)) // nextOwnerIndex
+      .mockResolvedValueOnce(pad('0xAAA')) // owner at index 1
+      .mockResolvedValueOnce(pad('0xBBB')) // owner at index 0
+
+    ;(readContract as Mock).mockImplementation(mockReadContract)
+
+    const result = await findSignerIndexOnChain({
+      address: '0xabc',
+      client,
+      signerPublicKey: pad('0xaaa'),
+    })
+
+    expect(result).toBe(1)
+  })
+
+  it('returns -1 when signer not found', async () => {
+    const mockReadContract = vi
+      .fn()
+      .mockResolvedValueOnce(BigInt(2)) // nextOwnerIndex
+      .mockResolvedValueOnce(pad('0xaaa')) // owner at index 1
+      .mockResolvedValueOnce(pad('0xbbb')) // owner at index 0
+
+    ;(readContract as Mock).mockImplementation(mockReadContract)
+
+    const result = await findSignerIndexOnChain({
+      address: '0xabc',
+      client,
+      signerPublicKey: '0xccc',
+    })
+    expect(result).toBe(-1)
+  })
+
+  it('handles empty signer list', async () => {
+    const mockReadContract = vi.fn().mockResolvedValueOnce(BigInt(0)) // nextOwnerIndex
+
+    ;(readContract as Mock).mockImplementation(mockReadContract)
+
+    const result = await findSignerIndexOnChain({
+      address: '0xabc',
+      client,
+      signerPublicKey: '0xccc',
+    })
+    expect(result).toBe(-1)
+  })
+})