chore(protocol): upgrade foundry version in CI (#17019)

Co-authored-by: Daniel Wang <[email protected]>
Co-authored-by: dantaik <[email protected]>

.github/workflows/protocol.yml

@@ -27,29 +27,23 @@ jobs:
           submodules: recursive
 
       - name: Install Foundry
-        uses: foundry-rs/foundry-toolchain@v1
-        with:
-          version: nightly-2cb875799419c907cc3709e586ece2559e6b340e
+        uses: foundry-rs/[email protected]
 
       - name: Install pnpm dependencies
         uses: ./.github/actions/install-pnpm-dependencies
 
-      - name: Check formatting
-        working-directory: ./packages/protocol
-        run: forge fmt --check
-
       - name: Unit tests
         working-directory: ./packages/protocol
         run: pnpm clean && pnpm test
 
-      - name: Generate contract layout table
+      - name: Format solidity && update contract layout table
         working-directory: ./packages/protocol
-        run: pnpm layout
+        run: pnpm layout && forge fmt
 
       - name: Commit contract layout table
         uses: stefanzweifel/git-auto-commit-action@v5
         with:
-          commit_message: "Add contract layout table"
+          commit_message: "forge fmt & update contract layout table"
 
       - name: Generate genesis
         working-directory: ./packages/protocol

packages/protocol/contracts/L1/TaikoData.sol

@@ -163,7 +163,7 @@ library TaikoData {
         mapping(
             uint64 blockId_mod_blockRingBufferSize
                 => mapping(uint32 transitionId => TransitionState ts)
-            ) transitions;
+        ) transitions;
         // Ring buffer for Ether deposits
         bytes32 __reserve1;
         SlotA slotA; // slot 5

packages/protocol/genesis/GenerateGenesis.g.sol

@@ -109,30 +109,6 @@ contract TestGenerateGenesis is Test, AddressResolver {
         assertEq(contractOwner, taikoL2Proxy.owner());
         assertEq(l1ChainId, taikoL2Proxy.l1ChainId());
 
-        vm.startPrank(taikoL2Proxy.GOLDEN_TOUCH_ADDRESS());
-        for (uint32 i = 0; i < 300; ++i) {
-            vm.roll(block.number + 1);
-            vm.warp(block.number + 12);
-            (uint256 basefee,) = taikoL2Proxy.getBasefee(uint64(i + 1), i + 1);
-            vm.fee(basefee);
-
-            uint256 gasLeftBefore = gasleft();
-
-            taikoL2Proxy.anchor(
-                keccak256(abi.encodePacked(block.timestamp, i)),
-                keccak256(abi.encodePacked(block.timestamp, i)),
-                i + 1,
-                i + 1
-            );
-
-            if (i == 299) {
-                console2.log(
-                    "TaikoL2.anchor gas cost after 256 L2 blocks:", gasLeftBefore - gasleft()
-                );
-            }
-        }
-        vm.stopPrank();
-
         vm.startPrank(taikoL2Proxy.owner());
 
         taikoL2Proxy.upgradeTo(address(new TaikoL2()));

packages/protocol/script/DeployERC20Airdrop.s.sol

@@ -37,7 +37,7 @@ contract DeployERC20Airdrop is DeployCapability {
                 impl: address(new ERC20Airdrop()),
                 data: abi.encodeCall(
                     ERC20Airdrop.init, (address(0), 0, 0, bytes32(0), bridgedTko, vaultAddress)
-                    )
+                )
             })
         );
 

packages/protocol/script/DeployOnL1.s.sol

@@ -152,7 +152,7 @@ contract DeployOnL1 is DeployCapability {
                 impl: address(new TaikoToken()),
                 data: abi.encodeCall(
                     TaikoToken.init, (owner, vm.envAddress("TAIKO_TOKEN_PREMINT_RECIPIENT"))
-                    ),
+                ),
                 registerTo: sharedAddressManager
             });
         }
@@ -263,7 +263,7 @@ contract DeployOnL1 is DeployCapability {
                     vm.envBytes32("L2_GENESIS_HASH"),
                     vm.envBool("PAUSE_TAIKO_L1")
                 )
-                ),
+            ),
             registerTo: rollupAddressManager
         });
 
@@ -330,7 +330,7 @@ contract DeployOnL1 is DeployCapability {
             impl: automateDcapV3AttestationImpl,
             data: abi.encodeCall(
                 AutomataDcapV3Attestation.init, (owner, address(sigVerifyLib), address(pemCertChainLib))
-                ),
+            ),
             registerTo: rollupAddressManager
         });
 

packages/protocol/test/L2/TaikoL2.t.sol

@@ -49,7 +49,7 @@ contract TestTaikoL2 is TaikoTest {
                     impl: address(new TaikoL2EIP1559Configurable()),
                     data: abi.encodeCall(
                         TaikoL2.init, (address(0), addressManager, l1ChainId, gasExcess)
-                        ),
+                    ),
                     registerTo: addressManager
                 })
             )

packages/protocol/test/L2/TaikoL2NoFeeCheck.t.sol

@@ -49,7 +49,7 @@ contract TestTaikoL2NoFeeCheck is TaikoTest {
                     impl: address(new SkipBasefeeCheckL2()),
                     data: abi.encodeCall(
                         TaikoL2.init, (address(0), addressManager, l1ChainId, gasExcess)
-                        ),
+                    ),
                     registerTo: addressManager
                 })
             )

packages/protocol/test/automata-attestation/common/AttestationBase.t.sol

@@ -61,7 +61,7 @@ contract AttestationBase is Test, DcapTestUtils, V3QuoteParseUtils {
                 data: abi.encodeCall(
                     AutomataDcapV3Attestation.init,
                     (admin, address(sigVerifyLib), address(pemCertChainLib))
-                    )
+                )
             })
         );
 

packages/protocol/test/bridge/Bridge.t.sol

@@ -93,7 +93,7 @@ contract BridgeTest is TaikoTest {
                     impl: address(new DelegateOwner()),
                     data: abi.encodeCall(
                         DelegateOwner.init, (mockDAO, address(addressManager), l1ChainId)
-                        )
+                    )
                 })
             )
         );

packages/protocol/test/team/airdrop/ERC20Airdrop.t.sol

@@ -102,7 +102,7 @@ contract TestERC20Airdrop is TaikoTest {
                 data: abi.encodeCall(
                     BridgedERC20.init,
                     (address(0), address(addressManager), randAddress(), 100, 18, "TKO", "Taiko Token")
-                    )
+                )
             })
         );
 
@@ -129,7 +129,7 @@ contract TestERC20Airdrop is TaikoTest {
                 data: abi.encodeCall(
                     ERC20Airdrop.init,
                     (address(0), claimStart, claimEnd, merkleRoot, address(token), address(vault))
-                    )
+                )
             })
         );
 

packages/protocol/test/tokenvault/BridgedERC20.t.sol

@@ -131,7 +131,7 @@ contract TestBridgedERC20 is TaikoTest {
                 data: abi.encodeCall(
                     BridgedERC20.init,
                     (owner, address(manager), srcToken, srcChainId, srcDecimals, name, name)
-                    ),
+                ),
                 registerTo: manager
             })
         );

packages/protocol/test/verifiers/RiscZeroVerifier.t.sol

@@ -48,7 +48,7 @@ contract TestRiscZeroVerifier is TaikoL1TestBase {
                 data: abi.encodeCall(
                     RiscZeroVerifier.init,
                     (address(0), address(addressManager), address(riscZeroRemoteVerifier))
-                    )
+                )
             })
         );