🔨 contracts: hardcode exa bridge role proposal salt

Author: cruzdanilo

contracts/.gas-snapshot

@@ -202,19 +202,19 @@ ExaPreviewerTest:test_collect_reverts_whenProposalsLeaveNoLiquidity() (gas: 1437
 ExaPreviewerTest:test_markets_returnsMarkets() (gas: 150447)
 ExaPreviewerTest:test_pendingProposals_returnsPendingProposals() (gas: 1983943)
 ExaPreviewerTest:test_utilizations_returns() (gas: 134922)
-HypEXATest:test_deployRouter_enrollsBase_whenNoRemotesOnOptimism() (gas: 30706283)
-HypEXATest:test_deployRouter_enrollsOptimism_whenNoRemotesOnBase() (gas: 30811917)
-HypEXATest:test_deployRouter_reverts_whenNoRemotesOnUnsupportedChain() (gas: 20627783)
+HypEXATest:test_deployRouter_enrollsBase_whenNoRemotesOnOptimism() (gas: 30706715)
+HypEXATest:test_deployRouter_enrollsOptimism_whenNoRemotesOnBase() (gas: 30812349)
+HypEXATest:test_deployRouter_reverts_whenNoRemotesOnUnsupportedChain() (gas: 20628215)
 HypEXATest:test_deployRouter_setsHookAndIsm() (gas: 270936)
 HypEXATest:test_handle_reverts_withoutBridgeRole() (gas: 578581)
 HypEXATest:test_hookAndIsm_trackMailboxDefaults_withoutRedeploy() (gas: 154299)
 HypEXATest:test_pause_pausesHookAndIsmViaPauser() (gas: 144877)
 HypEXATest:test_process_reverts_whenIsmPaused() (gas: 712859)
-HypEXATest:test_proposeBridgeRole_reverts_whenAlreadyGranted() (gas: 273918)
-HypEXATest:test_proposeBridgeRole_reverts_whenRouterNotDeployed() (gas: 20387421)
-HypEXATest:test_proposeBridgeRole_schedulesGrantOnTimelock() (gas: 2303697)
-HypEXATest:test_rotateRouterPausable_restoresProcessAfterPauserIsmPaused() (gas: 22139948)
-HypEXATest:test_rotateRouterPausable_restoresTransferRemoteAfterPauserHookPaused() (gas: 2126569)
+HypEXATest:test_proposeBridgeRole_reverts_whenAlreadyGranted() (gas: 273332)
+HypEXATest:test_proposeBridgeRole_reverts_whenRouterNotDeployed() (gas: 20387237)
+HypEXATest:test_proposeBridgeRole_schedulesGrantOnTimelock() (gas: 2303104)
+HypEXATest:test_rotateRouterPausable_restoresProcessAfterPauserIsmPaused() (gas: 22140415)
+HypEXATest:test_rotateRouterPausable_restoresTransferRemoteAfterPauserHookPaused() (gas: 2126591)
 HypEXATest:test_roundTrip_opToBaseToOp() (gas: 1312360)
 HypEXATest:test_roundTrip_opToPolygonToBaseToOp() (gas: 1960009)
 HypEXATest:test_transferRemote_reverts_whenHookPaused() (gas: 1106969)
@@ -255,4 +255,4 @@ RefunderTest:test_refund_reverts_whenTimelocked() (gas: 77924)
 RefunderTest:test_refund_reverts_whenWrongSignature() (gas: 109942)
 RefunderTest:test_refund_toleratesTimeDrift() (gas: 263419)
 RefunderTest:test_withdraw_reverts_whenNotAdmin() (gas: 35779)
-RefunderTest:test_withdraw_withdraws() (gas: 67492)
+RefunderTest:test_withdraw_withdraws() (gas: 67492)

contracts/script/HypEXA.s.sol

@@ -74,7 +74,8 @@ contract HypEXA is BaseScript {
     vm.stopBroadcast();
   }
 
-  function proposeBridgeRole(bytes32 salt) external {
+  function proposeBridgeRole() external {
+    bytes32 salt = keccak256("propose-exa-bridge-role");
     address router = CREATE3_FACTORY.getDeployed(acct("admin"), keccak256(abi.encode("HypEXA")));
     if (router.code.length == 0) revert RouterNotDeployed();
     address exa = protocol("EXA", true, getChain("optimism").chainId);

contracts/test/HypEXA.t.sol

@@ -234,16 +234,16 @@ contract HypEXATest is ForkTest {
     HypEXA hypEXA = new HypEXA();
 
     vm.expectRevert(RouterNotDeployed.selector);
-    hypEXA.proposeBridgeRole(keccak256("HypEXA.BRIDGE_ROLE"));
+    hypEXA.proposeBridgeRole();
   }
 
   function test_proposeBridgeRole_schedulesGrantOnTimelock() external {
     vm.selectFork(opFork);
     vm.prank(protocol("TimelockController"));
     exa.revokeRole(keccak256("BRIDGE_ROLE"), address(opRouter));
 
-    bytes32 salt = keccak256("HypEXA.BRIDGE_ROLE");
-    opHypEXA.proposeBridgeRole(salt);
+    bytes32 salt = keccak256("propose-exa-bridge-role");
+    opHypEXA.proposeBridgeRole();
 
     TimelockController timelock = TimelockController(payable(protocol("TimelockController")));
     bytes32 id = timelock.hashOperation(
@@ -259,7 +259,7 @@ contract HypEXATest is ForkTest {
   function test_proposeBridgeRole_reverts_whenAlreadyGranted() external {
     vm.selectFork(opFork);
     vm.expectRevert(AlreadyGranted.selector);
-    opHypEXA.proposeBridgeRole(keccak256("HypEXA.BRIDGE_ROLE"));
+    opHypEXA.proposeBridgeRole();
   }
 
   function test_deployRouter_setsHookAndIsm() external {