update gas snapshot

Author: O1ahmad

.gas-snapshot

@@ -26,7 +26,7 @@ DeploymentAddressesTest:test_sweepDepositAccountERC20Action_deployedAddress() (g
 DeploymentAddressesTest:test_transferAction_deployedAddress() (gas: 810463)
 DeploymentAddressesTest:test_transferERC20Action_deployedAddress() (gas: 811664)
 DeploymentAddressesTest:test_treasury_deployedAddress() (gas: 392232)
-DepositAddressCalculation:testFuzz_depositAccountAddressCalculation(address,address) (runs: 256, μ: 16368, ~: 16368)
+DepositAddressCalculation:testFuzz_depositAccountAddressCalculation(address,address) (runs: 257, μ: 16368, ~: 16368)
 EstimateDeactivateInstructionGasConstant:testFuzz_deactivateInstruction_gasConstant(uint256,(bytes32,(address,uint256,uint256,uint256))) (runs: 256, μ: 3632, ~: 3632)
 EstimateRefuelERC20GasConstant:testFuzz_refuelERC20_gasConstant(uint256,uint256,(address,address,uint256,uint256,(address,uint256,uint256,uint256))) (runs: 256, μ: 3859, ~: 3858)
 EstimateRefuelGasConstant:testFuzz_refuel_gasConstant(uint256,uint256,(address,uint256,uint256,uint256,(address,uint256,uint256,uint256))) (runs: 256, μ: 3777, ~: 3777)
@@ -35,7 +35,7 @@ EstimateSweepDepositAccountGasConstant:testFuzz_sweepDepositAccount_gasConstant(
 EstimateSweepSkipCCTPDepositAccountGasConstant:testFuzz_sweepSkipCCTPDepositAccount_gasConstant(uint256,uint256,(address,uint32,bytes32,uint256,(address,uint256,uint256,uint256))) (runs: 256, μ: 3794, ~: 3794)
 EstimateTransferERC20GasConstant:testFuzz_transferERC20_gasConstant(uint256,uint256,(address,address,uint256,(uint256,uint256,uint256,uint256),(address,uint256,uint256,uint256))) (runs: 256, μ: 4152, ~: 4151)
 EstimateTransferGasConstant:testFuzz_transfer_gasConstant(uint256,uint256,(address,uint256,uint256,(uint256,uint256,uint256,uint256),(address,uint256,uint256,uint256))) (runs: 256, μ: 4061, ~: 4058)
-EstimateUniswapV3ExactInputConstant:testFuzz_uniswapV3ExactInput_gasConstant(uint256,uint256,(address,address,address,uint24,uint256,uint256,uint32,uint32,(uint256,uint256,uint256,uint256),(address,uint256,uint256,uint256))) (runs: 256, μ: 4799, ~: 4690)
+EstimateUniswapV3ExactInputConstant:testFuzz_uniswapV3ExactInput_gasConstant(uint256,uint256,(address,address,address,uint24,uint256,uint256,uint32,uint32,(uint256,uint256,uint256,uint256),(address,uint256,uint256,uint256))) (runs: 256, μ: 4779, ~: 4690)
 FeeTokenRegistryTest:test_addFeeToken_alreadyRegistered() (gas: 1925)
 FeeTokenRegistryTest:test_addFeeToken_happyPath() (gas: 37614)
 FeeTokenRegistryTest:test_addFeeToken_roundIdZero() (gas: 12031)
@@ -48,10 +48,10 @@ FeeTokenRegistryTest:test_removeFeeToken_notRegistered() (gas: 10352)
 FeeTokenRegistryTest:test_weiToToken_correctnessDAI() (gas: 4243)
 FeeTokenRegistryTest:test_weiToToken_correctnessUSDT() (gas: 4243)
 FeeTokenRegistryTest:test_weiToToken_latestPriceNegative() (gas: 4245)
-FeeTokenRegistryTest:test_weiToToken_overflow(int256,uint256,uint64) (runs: 256, μ: 4239, ~: 4238)
+FeeTokenRegistryTest:test_weiToToken_overflow(int256,uint256,uint64) (runs: 257, μ: 4239, ~: 4238)
 FeeTokenRegistryTest:test_weiToToken_stalePriceFeed() (gas: 4357)
 FeeTokenRegistryTest:test_weiToToken_tokenNotRegistered() (gas: 9163)
-FeeTokenRegistryTest:test_weiToToken_underflow(int256,uint256,uint64) (runs: 256, μ: 4210, ~: 4205)
+FeeTokenRegistryTest:test_weiToToken_underflow(int256,uint256,uint64) (runs: 257, μ: 4210, ~: 4205)
 GatewayTest:test_isDelegated_false() (gas: 13528)
 GatewayTest:test_isDelegated_true() (gas: 13222)
 GatewayTest:test_safeDeactivateInstruction_happyPath() (gas: 43884)
@@ -68,19 +68,19 @@ GenerateOffchainTestData:test_generate_instructionId() (gas: 3043)
 GenerateOffchainTestData:test_generate_instructionSignature() (gas: 3020)
 GenerateOffchainTestData:test_generate_signatureHash() (gas: 3063)
 GenerateOffchainTestData:test_generate_userAddress() (gas: 3064)
-InstructionStorageDifferential:testDiff_deactivateStorage(bytes32) (runs: 256, μ: 3117, ~: 3117)
-InstructionStorageDifferential:testDiff_deactivateStorage_beforeExecution(bytes32) (runs: 256, μ: 3095, ~: 3095)
-InstructionStorageDifferential:testDiff_incrementAndDeactivate(bytes32) (runs: 256, μ: 3072, ~: 3072)
-InstructionStorageDifferential:testDiff_incrementAndDeactivate_beforeIncrementExecutionCounter(bytes32) (runs: 256, μ: 3074, ~: 3074)
-InstructionStorageDifferential:testDiff_incrementExecutionCounter(bytes32) (runs: 256, μ: 3095, ~: 3095)
+InstructionStorageDifferential:testDiff_deactivateStorage(bytes32) (runs: 257, μ: 3117, ~: 3117)
+InstructionStorageDifferential:testDiff_deactivateStorage_beforeExecution(bytes32) (runs: 257, μ: 3095, ~: 3095)
+InstructionStorageDifferential:testDiff_incrementAndDeactivate(bytes32) (runs: 257, μ: 3072, ~: 3072)
+InstructionStorageDifferential:testDiff_incrementAndDeactivate_beforeIncrementExecutionCounter(bytes32) (runs: 257, μ: 3074, ~: 3074)
+InstructionStorageDifferential:testDiff_incrementExecutionCounter(bytes32) (runs: 257, μ: 3095, ~: 3095)
 InstructionStorageTest:test_accessControl() (gas: 427093)
 InstructionStorageTest:test_deactivate_happyPath() (gas: 2273)
 InstructionStorageTest:test_incrementAndDeactivate_happyPath() (gas: 2523)
 InstructionStorageTest:test_incrementExecutionCounter_happyPath() (gas: 2555)
 IntervalTest:test_checkInterval_tooEarly() (gas: 12976)
 IntervalTest:test_checkInterval_tooLate() (gas: 13215)
 IntervalTest:test_checkStart_startByZero(uint64) (runs: 256, μ: 93767, ~: 93767)
-IntervalTest:test_checkStart_timeoutZero(uint64) (runs: 256, μ: 93747, ~: 93747)
+IntervalTest:test_checkStart_timeoutZero(uint64) (runs: 257, μ: 93747, ~: 93747)
 IntervalTest:test_checkStart_tooEarly() (gas: 36295)
 IntervalTest:test_checkStart_tooLate() (gas: 36369)
 InvertedPriceFeedTest:test_latestRoundData() (gas: 22884)
@@ -106,7 +106,7 @@ OtimDelegateTest:test_supportsInterface() (gas: 17863)
 OtimFeeTest:test_chargeFee_baseFeePerGasTooHigh() (gas: 71050)
 OtimFeeTest:test_chargeFee_erc20() (gas: 135603)
 OtimFeeTest:test_chargeFee_ether() (gas: 96915)
-OtimFeeTest:test_chargeFee_ether_maxBaseFeePerGasZero(uint256) (runs: 256, μ: 96934, ~: 96935)
+OtimFeeTest:test_chargeFee_ether_maxBaseFeePerGasZero(uint256) (runs: 257, μ: 96934, ~: 96935)
 OtimFeeTest:test_chargeFee_insufficientFeeBalance_erc20() (gas: 86654)
 OtimFeeTest:test_chargeFee_insufficientFeeBalance_ether() (gas: 66981)
 OtimFeeTest:test_chargeFee_noFee() (gas: 89267)
@@ -184,8 +184,8 @@ TreasuryTest:test_withdraw_invalidTarget() (gas: 10078)
 TreasuryTest:test_withdraw_transferFailed() (gas: 17244)
 UniswapV3ExactInputTest:test_uniswapV3ExactInput_amountInZero() (gas: 38829)
 UniswapV3ExactInputTest:test_uniswapV3ExactInput_ethToToken() (gas: 3064)
-UniswapV3ExactInputTest:test_uniswapV3ExactInput_insufficientEthBalance() (gas: 82148)
-UniswapV3ExactInputTest:test_uniswapV3ExactInput_insufficientTokenBalance() (gas: 99577)
+UniswapV3ExactInputTest:test_uniswapV3ExactInput_insufficientEthBalance() (gas: 59275)
+UniswapV3ExactInputTest:test_uniswapV3ExactInput_insufficientTokenBalance() (gas: 76704)
 UniswapV3ExactInputTest:test_uniswapV3ExactInput_nonExistentPool() (gas: 44870)
 UniswapV3ExactInputTest:test_uniswapV3ExactInput_priceDeviationTooHigh() (gas: 3019)
 UniswapV3ExactInputTest:test_uniswapV3ExactInput_receivedTooLittle() (gas: 3041)

.github/scripts/directory-comparison.rs

@@ -58,11 +58,13 @@ fn directories_match(dir1: &str, dir2: &str, ignore: Option<&str>, show_diff: bo
 
     let files1: HashMap<_, _> = tree1.iter()
         .filter(|i| !should_ignore_path(&i.path.relative, ignore))
+        .filter(|i| std::path::Path::new(dir1).join(&i.path.relative).is_file())
         .map(|i| (i.path.relative.to_string(), i.hash.to_hex_string()))
         .collect();
 
     let files2: HashMap<_, _> = tree2.iter()
         .filter(|i| !should_ignore_path(&i.path.relative, ignore))
+        .filter(|i| std::path::Path::new(dir2).join(&i.path.relative).is_file())
         .map(|i| (i.path.relative.to_string(), i.hash.to_hex_string()))
         .collect();
 

.github/workflows/artifacts.yml

@@ -1,6 +1,9 @@
 name: Sync Contract Artifacts
 
 on:
+  push:
+    branches:
+      - ahmad/readd_offchain_artifact_match_sync
   workflow_call:
     inputs:
       protocol_ref:
@@ -44,7 +47,7 @@ jobs:
       - name: Checkout protocol
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
         with:
-          ref: ${{ inputs.protocol_ref || 'main' }}
+          ref: ${{ inputs.protocol_ref || 'ahmad/readd_offchain_artifact_match_sync' }}
 
       - name: Checkout offchain
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
@@ -64,13 +67,8 @@ jobs:
         id: build-protocol
         run: |
           echo "protocol_sha=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT
-          forge soldeer update --config-location foundry
-          forge build ./src --sizes
-
-      - name: Setup Rust
-        uses: actions-rust-lang/setup-rust-toolchain@fb51252c7ba57d633bc668f941da052e410add48 # v1.13.0
-        with:
-          working-directory: ./offchain
+          forge soldeer update
+          forge build --sizes
 
       - name: Install rust-script
         run: |
@@ -102,7 +100,7 @@ jobs:
           commit-message: "sync: otim-protocol artifacts (${{ steps.build-protocol.outputs.protocol_sha }})"
           title: "Sync: otim-protocol build artifacts"
           body: |
-            Syncing build artifacts from otim-protocol.
+            Syncing build artifacts from otim-protocol ([artifact comparison](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}))
 
             - Protocol: `${{ steps.build-protocol.outputs.protocol_sha }}`
             - Offchain: `${{ inputs.offchain_ref || 'main' }}`