Bug: ERC-7677 support in coinbase smart wallet is outdated

[Sednaoui]

Describe the bug

Support for paymaster capabilities is out of date as the erc-7677 standard evolved

Steps

The coinbase smart wallet follows the old implementation of the standard to send the paymaster URL, which is outdated:

capabilities: {
  paymasterService: {
    url: paymasterServiceUrl,
}

Expected behavior

The new updated implementation requires a chainId and the optional field, like the following:

capabilities: {
  paymasterService: {
    ["0x01"]: {
      url: paymasterServiceUrl,
     optional: true,
    }
  }
}

Version

4.3.2

Additional info

https://eips.ethereum.org/EIPS/eip-7677

Desktop

No response

Smartphone

No response

[kickinchicken]

If I’m correct, this bug stole a lot of money for me

[vanshb03]

@Sednaoui I believe this EIP is in the process of being peer-reviewed. I did, however, make the snippet as per the current writeup, so once that's confirmed, I will make the PR.

params: [
          {
            version: '1.0', 
            from: subAccountAddress,
            calls: [
              {
                to: '0x000000000000000000000000000000000000dead',
                data: '0x',
                value: '0x0',
                chainId: numberToHex(baseSepolia.id), 
              },
            ],
            capabilities: {
              paymasterService: {
                [`0x${baseSepolia.id.toString(16)}`]: {
                  url: 'https://api.developer.coinbase.com/rpc/v1/base-sepolia/S-fOd2n2Oi4fl4e1Crm83XeDXZ7tkg8O',
                  context: {},
                },
              },
            },
          },
        ],

[Sednaoui]

@vanshb03, there's a new optional filed in EIP-5792 that apps can use with capabilities. This is particularly useful for apps that include capabilities not supported by the Coinbase Smart Wallet. With this field, the wallet can still process transactions, even if certain capabilities are missing, making sure that the overall flow remains intact. So it would be like the following:

params: [
          {
            version: '1.0', 
            from: subAccountAddress,
            calls: [
              {
                to: '0x000000000000000000000000000000000000dead',
                data: '0x',
                value: '0x0',
                chainId: numberToHex(baseSepolia.id), 
              },
            ],
            capabilities: {
              paymasterService: {
                [`0x${baseSepolia.id.toString(16)}`]: {
                  optional: true,
                  url: 'https://api.developer.coinbase.com/rpc/v1/base-sepolia/S-fOd2n2Oi4fl4e1Crm83XeDXZ7tkg8O',
                  context: {},
                },
              },
            },
          },
        ],

[cb-jake]

Thanks @Sednaoui and @vanshb03. I'll be taking a look into this later this week. The team is working on updating to the final spec of 5792. I don't have a timeline yet but it's in progress