Greptile Summary

This PR makes two fixes: it replaces the generic tuple[] ABI type specifier with the fully-qualified (address,uint24,int24,address,bytes)[] in the encode() calls for both token_to_token_swap_exact_input and token_to_token_swap_exact_output, correctly matching the PathKey dataclass layout. It also adds a does_not_raise context manager to the test file, though it is not yet used.

Confidence Score: 4/5

Safe to merge; the ABI fix is correct and the only concern is an unused helper in tests.

The core change — replacing tuple[] with the explicit (address,uint24,int24,address,bytes)[] ABI type — is correct and matches the PathKey dataclass definition. The only finding is a P2: does_not_raise is defined but never used in the test file.

tests/test_uniswap4.py — does_not_raise helper is unused dead code.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Caller
    participant Uniswap4
    participant eth_abi as eth_abi.encode
    participant UniversalRouter

    Caller->>Uniswap4: token_to_token_swap_exact_input(input_token, qty, ...)
    Uniswap4->>eth_abi: encode(["(address,(address,uint24,int24,address,bytes)[],uint128,uint128)"], [params])
    Note over eth_abi: Previously used tuple[] — now uses explicit PathKey ABI type
    eth_abi-->>Uniswap4: exact_input_params bytes
    Uniswap4->>UniversalRouter: execute(V4_SWAP commands, inputs, deadline)
    UniversalRouter-->>Caller: tx hash

_{Reviews (1): Last reviewed commit: "misc fixes" | Re-trigger Greptile}