Here is a basic client without using a dependency. If this is the direction we want to go, I can clean up the code and write some proper tests mocking out the URL Session

This PR

• adds the Web3.swift package as a dependency (https://github.com/argentlabs/web3.swift)
• fetches an addresses balance on the transaction screen

In order for this to work you need to set your own values for clientUrl and ethAccountAddress in TransactionsView

Then you should see this:

Forgot that the group ID, extension bundle ID, and app group are mentioned in a Swift file too. Tests weren't working. And App Store Connect was automatically rejecting the binary because the PRODUCT_NAME was Wallet.app, changing it to Safari Wallet.app did the trick.

This fixes the error jsonRpcError(Wallet.JsonRpcError(code: -32602, message: "Invalid method parameter(s)."))

To fix it, I created an overload method that doesn't have a parameters parameter that sets a default that satisfies RPC methods not expecting parameter values.

It seems like using a task modifier on a nested view doesn't work. Not sure what exactly the reason is but moving the logic to a task modifier on the VStack solves the problem

This splits the two injected classes (SafariWallet and Ethereum) into two separate JS files. Hopefully, this is a little more clear, and sets up an easy pattern for injecting any future JS files hosted by the extension.

The added injected function provides one place to maintain the method of injection.

The two "injections" are composed of the same code that removed from content.js, except there they each include an additional info line to validate the presence of the instantiated objects on the window object.

The info line makes it easy to open a dev console and see them located on a page, e.g.:

[Info] window.ethereum is injected. (localhost, line 64)
[Info] window.safariWallet is injected. (localhost, line 24)

Quick and dirty check on MacOS if the extension is enabled by the user.

The used classes are not available on iOS, we'll need a different approach for iOS

Spec: https://docs.alchemy.com/alchemy/documentation/enhanced-apis/transfers-api#alchemy_getassettransfers

Implemented the endpoint and created relevant models. To test I added a button in the developer tab called Call alchemy_getAssetTransfers. I also still need to add a test.

https://eips.ethereum.org/EIPS/eip-2015 https://eips.ethereum.org/EIPS/eip-3085 https://eips.ethereum.org/EIPS/eip-3326

Allow dapps to switch the network of the user wallet. For example in Zapper you can click on Polygon and MetaMask will open a popup asking if the user wants to switch to the network.

Do not ask again

There should be a checkbox allowing users to prevent them for constantly opening and confirming this popup. Something along the lines of "Do not ask again when zapper.fi wants to switch to Polygon".

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

When swapping to any token in Uniswap there's an "Add token to MetaMask" button. After clicking the "Add Token" button the token will show in the Assets tab in MetaMask.

Warning if token was already added

If token was already added there should be a warning.

Activity/History

A place where the wallet keeps track of everything that the user signed which includes:

• [ ] Transactions
• [ ] Messages

Every item in the activity should have:

• [ ] Timestamp (a date and time of when the message/transaction has been signed)
• [ ] Dapp domain

If the activity item is a transaction it should have:

• [ ] From address (prefer displaying ENS)
• [ ] To address (prefer displaying ENS)
• [ ] Nonce
• [ ] Amount (the ETH amount that was sent)
• [ ] Gas Limit (Units)
• [ ] Gas Used (Units) (when the transaction has been mined)
• [ ] Base fee (GWEI)
• [ ] Priority fee (GWEI)
• [ ] Total Gas Fee (base + priority) * (if transaction is mined use gas used otherwise use gas limit)
• [ ] Total (total gas fee + amount)
• [ ] Burnt fees (only when mined) (only when enabled in settings)
• [ ] Transaction hash (only when enabled in settings)
• [ ] Mined timestamp
• [ ] Status (queued, submitted, dropped/replaced, confirmed, failed)
• [ ] Chain ID

If the activity item is a message it should have:

• [ ] The message that was signed

Filter activity

There should be an option to filter activity:

• [ ] filter by type (transaction/message)
• [ ] filter by transaction status (pending/failed/succeeded)
• [ ] filter by contract (Uniswap Router/SushiSwap router) (depends on rich activity, see below)
• [ ] filter by dapp domain (zapper.fi/app.uniswap.org)

Fetch onchain activity

• [ ] When setting up the wallet it should fetch previous transactions that are on the blockchain and display them to the user.
• [ ] Users may have made transactions using other wallets/devices. When the wallet is started again it should check for new transactions and fetch them.

Activity from other addresses

On Ethereum other people can send you assets. The activity should show when someone else has sent you assets, not just transactions/messages made by your address.

Rich activity

When I make a swap on Uniswap in MetaMask it displays Exact Input Single (which is the name of the method) and the domain of the dapp. Instead we should display the actual swap that happened like on Etherscan.

• [ ] ERC-20 approvals should display the allowance amount
• [ ] ERC-20 transfers should display the amount that was transferred
• [ ] Swaps on Uniswap and other decentralized exchanges should display the actual tokens swapped
• [ ] When NFTs are transferred the NFT should be displayed
• [ ] When there's an interaction with ENS the ENS domain should be displayed
• [ ] Bridged assets between different chains need to be marked as such

Cloud sync

• [ ] Signed messages should be synced since they're not on the blockchain. That way the user can see them on all of their devices.