Improve types and make wallet providers more flexible

[gskril]

What changed? Why?

Replaced all cases of 0x${string} with Hex from Viem for cleanliness.

Also fixed the response type of waitForTransactionReceipt() in a few wallet providers.

Qualified Impact

Exposes publicClient and walletClient from VieimWalletProvider to allow more custom provider flexibility, since not all viem methods are explicitly included in the class.

Closes #272

[cb-heimdall]

🟡 Heimdall Review Status

Requirement	Status	More Info
Reviews	🟡 0/1	Denominator calculation Show calculation 1 if user is bot 0 1 if user is external 0 From .codeflow.yml 1 Additional review requirements Show calculation Max 0 0 From CODEOWNERS 0 Global minimum 0 Max 1 1 1 if commit is unverified 0 Sum 1

[cb-heimdall]

Review Error for BradleeA2024 @ 2025-02-09 18:09:16 UTC
User must have write permissions to review

[debuggingfuture]

wondered the same if agentkit original motivation is try to stick with own type 0x${string} to avoid the dependency
if viem types are used, will suggest distinguishing between Hex and Address to further clarify the type.

[gskril]

It already depends on viem pretty heavily so I figured it's safe. And if that is the case, I'd still be in favor of AgentKit's own reusable Hex type rather than 0x${string} everywhere.

I never really understood why Viem has a Hex and Address when they're exactly the same thing 😅