How would you go about using viem as a peer dependency in an npm package?

[salieflewis]

Currently I have this weird setup for Public Clients that allows the user to configure an environment variable to handle the transport.

import { WalletClient, createPublicClient, createWalletClient, http } from 'viem'
import { mainnet, goerli } from 'viem/chains'
import type { PublicClient } from 'viem'

export const mainnetClient: PublicClient = createPublicClient({
  batch: {
    multicall: true,
  },
  chain: mainnet,
  transport: http(process.env.NEXT_PUBLIC_MAINNET_ALCHEMY_ENDPOINT),
})

export const goerliClient: PublicClient = createPublicClient({
  batch: {
    multicall: true,
  },
  chain: goerli,
  transport: http(process.env.NEXT_PUBLIC_GOERLI_ALCHEMY_ENDPOINT),
})

export const viemClient: PublicClient | undefined =
  process.env.NEXT_PUBLIC_CHAIN_ID == '5' ? goerliClient : mainnetClient

However, for Wallet Clients, there is a top level await in the example code when the account is expected to be provided by a browser extension, so I created this useWalletClient hook to fetch that instance, but obviously it doesn't not run if the user is not connected, so it throws an error.

import { useEffect, useState } from 'react'
import { WalletClient, createWalletClient, http } from 'viem'
import { mainnet, goerli } from 'viem/chains'

export function useWalletClient(): WalletClient | undefined {
  const [account, setAccount] = useState()

  useEffect(() => {
    // prettier-ignore
    (async () => {
      try {
        // @ts-ignore
        const fetchedAccount = await window.ethereum.request({
          method: 'eth_requestAccounts',
        })
        setAccount(fetchedAccount)
      } catch (error) {
        console.log(error)
      }
    })()
  }, [])

  const mainnetWalletClient = createWalletClient({
    account,
    chain: mainnet,
    transport: http(),
  })

  const goerliWalletClient = createWalletClient({
    account,
    chain: goerli,
    transport: http(),
  })

  const viemWalletClient: WalletClient | undefined =
    process.env.NEXT_PUBLIC_CHAIN_ID == '5' ? goerliWalletClient : mainnetWalletClient

  return viemWalletClient
}

Thoughts and guidance around these patterns?