fix(protocol): fix and improve CrossChainSynced event (#15202)

Co-authored-by: Daniel Wang <dong77@gmail.com>
taikoxyz · Nov 14, 2023 · c124fcb · c124fcb
1 parent 716644f
commit c124fcb
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 9 deletions.
diff --git a/packages/protocol/contracts/L1/libs/LibVerifying.sol b/packages/protocol/contracts/L1/libs/LibVerifying.sol
@@ -30,7 +30,10 @@ library LibVerifying {
     );
 
     event CrossChainSynced(
-        uint64 indexed srcHeight, bytes32 blockHash, bytes32 signalRoot
+        uint64 indexed syncedInBlock,
+        uint64 indexed blockId,
+        bytes32 blockHash,
+        bytes32 signalRoot
     );
 
     // Warning: Any errors defined here must also be defined in TaikoErrors.sol.
@@ -222,8 +225,8 @@ library LibVerifying {
                     blockId: blockId,
                     assignedProver: blk.assignedProver,
                     prover: ts.prover,
-                    blockHash: ts.blockHash,
-                    signalRoot: ts.signalRoot,
+                    blockHash: blockHash,
+                    signalRoot: signalRoot,
                     tier: ts.tier,
                     contestations: ts.contestations
                 });
@@ -244,7 +247,10 @@ library LibVerifying {
                     .sendSignal(signalRoot);
 
                 emit CrossChainSynced(
-                    lastVerifiedBlockId, blockHash, signalRoot
+                    uint64(block.number),
+                    lastVerifiedBlockId,
+                    blockHash,
+                    signalRoot
                 );
             }
         }

diff --git a/packages/protocol/contracts/L2/TaikoL2.sol b/packages/protocol/contracts/L2/TaikoL2.sol
@@ -134,7 +134,7 @@ contract TaikoL2 is Ownable2StepUpgradeable, TaikoL2Signer, ICrossChainSync {
         // allow for multi-hop bridging.
         ISignalService(signalService).sendSignal(l1SignalRoot);
         emit CrossChainSynced(
-            l1Height, uint64(block.number), l1BlockHash, l1SignalRoot
+            uint64(block.number), l1Height, l1BlockHash, l1SignalRoot
         );
 
         // Update state variables

diff --git a/packages/protocol/contracts/common/ICrossChainSync.sol b/packages/protocol/contracts/common/ICrossChainSync.sol
@@ -21,15 +21,15 @@ interface ICrossChainSync {
     }
 
     /// @dev Emitted when a block has been synced across chains.
-    /// @param remoteBlockId The ID of the remote block whose block hash and
-    /// signal root are synced.
     /// @param syncedInBlock The ID of this chain's block where the sync
     /// happened.
+    /// @param blockId The ID of the remote block whose block hash and
+    /// signal root are synced.
     /// @param blockHash The hash of the synced block.
     /// @param signalRoot The root hash representing cross-chain signals.
     event CrossChainSynced(
-        uint64 indexed remoteBlockId,
-        uint64 syncedInBlock,
+        uint64 indexed syncedInBlock,
+        uint64 indexed blockId,
         bytes32 blockHash,
         bytes32 signalRoot
     );