feat: implement aggregate3Value function in IMulticall3 and update TrailsRouterShim

- Added the aggregate3Value function to the IMulticall3 interface to support calls with value transfers.
- Updated the TrailsRouterShim contract to validate and forward calls specifically for aggregate3Value.
- Modified related tests to ensure proper functionality and event emissions for the new aggregate3Value implementation.

Author: shunkakinoki

.gas-snapshot

@@ -48,23 +48,23 @@ TrailsRouterShimDeploymentTest:test_DeployRouterShim_SameAddress() (gas: 6683217
 TrailsRouterShimDeploymentTest:test_DeployRouterShim_Success() (gas: 4320770)
 TrailsRouterShimDeploymentTest:test_DeployedContract_HasCorrectConfiguration() (gas: 4320816)
 TrailsRouterShimTest:testConstructorValidation() (gas: 69386)
-TrailsRouterShimTest:testForwardToRouterReturnValue() (gas: 747637)
+TrailsRouterShimTest:testForwardToRouterReturnValue() (gas: 750442)
 TrailsRouterShimTest:testRouterAddressImmutable() (gas: 1428747)
 TrailsRouterShimTest:test_constructor_revert_zeroRouter() (gas: 69034)
-TrailsRouterShimTest:test_delegatecall_forwards_and_sets_sentinel_sstore_inactive() (gas: 1553903)
-TrailsRouterShimTest:test_delegatecall_forwards_and_sets_sentinel_tstore_active() (gas: 37963)
-TrailsRouterShimTest:test_delegatecall_router_revert_bubbles_as_RouterCallFailed() (gas: 82093)
-TrailsRouterShimTest:test_delegatecall_sets_sentinel_with_sstore_when_no_tstore() (gas: 1536669)
-TrailsRouterShimTest:test_delegatecall_sets_sentinel_with_tstore_when_supported() (gas: 20808)
+TrailsRouterShimTest:test_delegatecall_forwards_and_sets_sentinel_sstore_inactive() (gas: 1554508)
+TrailsRouterShimTest:test_delegatecall_forwards_and_sets_sentinel_tstore_active() (gas: 38562)
+TrailsRouterShimTest:test_delegatecall_router_revert_bubbles_as_RouterCallFailed() (gas: 82200)
+TrailsRouterShimTest:test_delegatecall_sets_sentinel_with_sstore_when_no_tstore() (gas: 1536737)
+TrailsRouterShimTest:test_delegatecall_sets_sentinel_with_tstore_when_supported() (gas: 20876)
 TrailsRouterShimTest:test_direct_handleSequenceDelegateCall_reverts_not_delegatecall() (gas: 9840)
-TrailsRouterShimTest:test_forwardToRouter_return_data_handling() (gas: 763078)
-TrailsRouterShimTest:test_forwardToRouter_revert_with_custom_error() (gas: 721440)
-TrailsRouterShimTest:test_handleSequenceDelegateCall_empty_calldata() (gas: 13849)
-TrailsRouterShimTest:test_handleSequenceDelegateCall_large_calldata() (gas: 1681891)
-TrailsRouterShimTest:test_handleSequenceDelegateCall_max_call_value() (gas: 31292)
-TrailsRouterShimTest:test_handleSequenceDelegateCall_with_eth_value() (gas: 31593)
-TrailsRouterShimTest:test_handleSequenceDelegateCall_zero_call_value() (gas: 26363)
-TrailsRouterShimTest:test_sentinel_setting_with_different_op_hashes() (gas: 36047)
+TrailsRouterShimTest:test_forwardToRouter_return_data_handling() (gas: 766351)
+TrailsRouterShimTest:test_forwardToRouter_revert_with_custom_error() (gas: 724461)
+TrailsRouterShimTest:test_handleSequenceDelegateCall_empty_calldata() (gas: 13917)
+TrailsRouterShimTest:test_handleSequenceDelegateCall_large_calldata() (gas: 1682519)
+TrailsRouterShimTest:test_handleSequenceDelegateCall_max_call_value() (gas: 31893)
+TrailsRouterShimTest:test_handleSequenceDelegateCall_with_eth_value() (gas: 32194)
+TrailsRouterShimTest:test_handleSequenceDelegateCall_zero_call_value() (gas: 26964)
+TrailsRouterShimTest:test_sentinel_setting_with_different_op_hashes() (gas: 36731)
 TrailsRouterTest:testDelegateCallWithETH() (gas: 325931)
 TrailsRouterTest:testExecute_WithFailingMulticall() (gas: 362156)
 TrailsRouterTest:testHandleSequenceDelegateCall_InjectAndCall() (gas: 72380)

src/TrailsRouterShim.sol

@@ -55,7 +55,7 @@ contract TrailsRouterShim is ITrailsRouterShim, DelegatecallGuard, Tstorish {
         // Decode the inner call data and call value forwarded to the router
         (bytes memory inner, uint256 callValue) = abi.decode(data, (bytes, uint256));
 
-        // Validate that only aggregate3() is called
+        // Validate that only aggregate3Value() is called
         _validateRouterCall(inner);
 
         bytes memory routerReturn = _forwardToRouter(inner, callValue);
@@ -83,8 +83,8 @@ contract TrailsRouterShim is ITrailsRouterShim, DelegatecallGuard, Tstorish {
             selector := mload(add(callData, 32))
         }
 
-        // Only allow `aggregate3` calls (0x82ad56cb)
-        if (selector != 0x82ad56cb) {
+        // Only allow `aggregate3Value` calls (0x174dea71)
+        if (selector != 0x174dea71) {
             revert InvalidFunctionSelector(selector);
         }
     }

src/interfaces/IMulticall3.sol

@@ -15,6 +15,13 @@ interface IMulticall3 {
         bytes callData;
     }
 
+    struct Call3Value {
+        address target;
+        bool allowFailure;
+        uint256 value;
+        bytes callData;
+    }
+
     struct Result {
         bool success;
         bytes returnData;
@@ -25,4 +32,6 @@ interface IMulticall3 {
     // -------------------------------------------------------------------------
 
     function aggregate3(Call3[] calldata calls) external payable returns (Result[] memory returnData);
+
+    function aggregate3Value(Call3Value[] calldata calls) external payable returns (Result[] memory returnData);
 }