Bump nim-eth (#3877)

bhartnett · web-flow · commit 67d615412ec7 · 2025-12-15T06:35:46.000+01:00
* Bump nim-eth.

* BAL updates for slot change from Bytes32 to UInt256.
diff --git a/execution_chain/block_access_list/block_access_list_builder.nim b/execution_chain/block_access_list/block_access_list_builder.nim
@@ -113,11 +113,8 @@ proc addCodeChange*(
 func balIndexCmp(x, y: StorageChange | BalanceChange | NonceChange | CodeChange): int =
   cmp(x.blockAccessIndex, y.blockAccessIndex)
 
-func slotCmp(x, y: StorageKey | StorageValue): int =
-  cmp(x.data.toHex(), y.data.toHex())
-
 func slotChangesCmp(x, y: SlotChanges): int =
-  cmp(x.slot.data.toHex(), y.slot.data.toHex())
+  cmp(x.slot, y.slot)
 
 func addressCmp(x, y: AccountChanges): int =
   cmp(x.address.data.toHex(), y.address.data.toHex())
@@ -132,18 +129,18 @@ func buildBlockAccessList*(builder: BlockAccessListBuilderRef): BlockAccessList
       var slotChanges: seq[StorageChange]
 
       for balIndex, value in changes:
-        slotChanges.add((BlockAccessIndex(balIndex), StorageValue(value.toBytesBE())))
+        slotChanges.add((BlockAccessIndex(balIndex), StorageValue(value)))
       slotChanges.sort(balIndexCmp)
 
-      storageChanges.add((StorageKey(slot.toBytesBE()), slotChanges))
+      storageChanges.add((StorageKey(slot), slotChanges))
     storageChanges.sort(slotChangesCmp)
 
     # Collect and sort storageReads
     var storageReads: seq[StorageKey]
     for slot in accData.storageReads:
       if slot notin accData.storageChanges:
-        storageReads.add(StorageKey(slot.toBytesBE()))
-    storageReads.sort(slotCmp)
+        storageReads.add(StorageKey(slot))
+    storageReads.sort()
 
     # Collect and sort balanceChanges
     var balanceChanges: seq[BalanceChange]
diff --git a/execution_chain/block_access_list/block_access_list_validation.nim b/execution_chain/block_access_list/block_access_list_validation.nim
@@ -18,9 +18,6 @@ import
 
 export block_access_lists, hashes, results
 
-func slotCmp(x, y: StorageKey | StorageValue): int =
-  cmp(x.data.toHex(), y.data.toHex())
-
 func validate*(bal: BlockAccessList, expectedHash: Hash32): Result[void, string] =
   ## Validate that a block access list is structurally correct and matches the expected hash.
 
@@ -44,7 +41,7 @@ func validate*(bal: BlockAccessList, expectedHash: Hash32): Result[void, string]
   for accountChanges in bal:
     # Validate storage changes slots are sorted lexicographically
     let storageChangesSlots = accountChanges.storageChanges.mapIt(it.slot)
-    if storageChangesSlots != storageChangesSlots.sorted(slotCmp):
+    if storageChangesSlots != storageChangesSlots.sorted():
       return err("Storage changes slots should be sorted lexicographically")
 
     # Check storage changes are sorted by blockAccessIndex
@@ -55,7 +52,7 @@ func validate*(bal: BlockAccessList, expectedHash: Hash32): Result[void, string]
 
     # Validate storage reads are sorted lexicographically
     let storageReadsSlots = accountChanges.storageReads
-    if storageReadsSlots != storageReadsSlots.sorted(slotCmp):
+    if storageReadsSlots != storageReadsSlots.sorted():
       return err("Storage reads should be sorted lexicographically")
 
     # Check balance changes are sorted by blockAccessIndex
diff --git a/tests/test_block_access_list_builder.nim b/tests/test_block_access_list_builder.nim
@@ -14,9 +14,6 @@ import
   unittest2,
   ../execution_chain/block_access_list/block_access_list_builder
 
-template toBytes32(slot: UInt256): Bytes32 =
-  Bytes32(slot.toBytesBE())
-
 suite "Block access list builder":
   const
     address1 = address"0x10007bc31cedb7bfb8a345f31e668033056b2728"
@@ -63,12 +60,12 @@ suite "Block access list builder":
       bal.len() == 2
       bal[0].address == address1
       bal[0].storageChanges.len() == 3
-      bal[0].storageChanges[0] == (slot1.toBytes32(), @[(1.BlockAccessIndex, 1.u256.toBytes32())])
-      bal[0].storageChanges[1] == (slot2.toBytes32(), @[(2.BlockAccessIndex, 2.u256.toBytes32())])
-      bal[0].storageChanges[2] == (slot3.toBytes32(), @[(0.BlockAccessIndex, 3.u256.toBytes32()), (3.BlockAccessIndex, 5.u256.toBytes32())])
+      bal[0].storageChanges[0] == (slot1, @[(1.BlockAccessIndex, 1.u256)])
+      bal[0].storageChanges[1] == (slot2, @[(2.BlockAccessIndex, 2.u256)])
+      bal[0].storageChanges[2] == (slot3, @[(0.BlockAccessIndex, 3.u256), (3.BlockAccessIndex, 5.u256)])
       bal[1].address == address2
       bal[1].storageChanges.len() == 1
-      bal[1].storageChanges[0] == (slot1.toBytes32(), @[(1.BlockAccessIndex, 1.u256.toBytes32())])
+      bal[1].storageChanges[0] == (slot1, @[(1.BlockAccessIndex, 1.u256)])
 
   test "Add storage read":
     builder.addStorageRead(address2, slot3)
@@ -81,11 +78,11 @@ suite "Block access list builder":
     check:
       bal.len() == 3
       bal[0].address == address1
-      bal[0].storageReads == @[slot1.toBytes32()]
+      bal[0].storageReads == @[slot1]
       bal[1].address == address2
-      bal[1].storageReads == @[slot2.toBytes32(), slot3.toBytes32()]
+      bal[1].storageReads == @[slot2, slot3]
       bal[2].address == address3
-      bal[2].storageReads == @[slot3.toBytes32()]
+      bal[2].storageReads == @[slot3]
 
   test "Add balance change":
     builder.addBalanceChange(address2, 1, 0.u256)
@@ -177,23 +174,23 @@ suite "Block access list builder":
 
       bal[0].address == address1
       bal[0].storageChanges.len() == 3
-      bal[0].storageChanges[0] == (slot1.toBytes32(), @[(1.BlockAccessIndex, 1.u256.toBytes32())])
-      bal[0].storageChanges[1] == (slot2.toBytes32(), @[(2.BlockAccessIndex, 2.u256.toBytes32())])
-      bal[0].storageChanges[2] == (slot3.toBytes32(), @[(0.BlockAccessIndex, 3.u256.toBytes32()), (3.BlockAccessIndex, 5.u256.toBytes32())])
+      bal[0].storageChanges[0] == (slot1, @[(1.BlockAccessIndex, 1.u256)])
+      bal[0].storageChanges[1] == (slot2, @[(2.BlockAccessIndex, 2.u256)])
+      bal[0].storageChanges[2] == (slot3, @[(0.BlockAccessIndex, 3.u256), (3.BlockAccessIndex, 5.u256)])
       bal[0].storageReads.len() == 0 # read removed by storage change with the same slot
       bal[0].balanceChanges == @[(2.BlockAccessIndex, 10.u256)]
       bal[0].nonceChanges == @[(3.BlockAccessIndex, 3.AccountNonce)]
       bal[0].codeChanges == @[(3.BlockAccessIndex, @[0x4.byte])]
 
       bal[1].address == address2
       bal[1].storageChanges.len() == 1
-      bal[1].storageChanges[0] == (slot1.toBytes32(), @[(1.BlockAccessIndex, 1.u256.toBytes32())])
-      bal[1].storageReads == @[slot2.toBytes32(), slot3.toBytes32()]
+      bal[1].storageChanges[0] == (slot1, @[(1.BlockAccessIndex, 1.u256)])
+      bal[1].storageReads == @[slot2, slot3]
       bal[1].balanceChanges == @[(0.BlockAccessIndex, 1.u256), (1.BlockAccessIndex, 0.u256)]
       bal[1].nonceChanges == @[(1.BlockAccessIndex, 1.AccountNonce), (2.BlockAccessIndex, 2.AccountNonce)]
       bal[1].codeChanges == @[(0.BlockAccessIndex, @[0x1.byte]), (1.BlockAccessIndex, @[0x2.byte])]
 
       bal[2].address == address3
-      bal[2].storageReads == @[slot3.toBytes32()]
+      bal[2].storageReads == @[slot3]
       bal[2].balanceChanges == @[(3.BlockAccessIndex, 3.u256)]
       bal[2].nonceChanges == @[(1.BlockAccessIndex, 10.AccountNonce)]
diff --git a/tests/test_block_access_list_validation.nim b/tests/test_block_access_list_validation.nim
@@ -15,9 +15,6 @@ import
   eth/common/block_access_lists_rlp,
   ../execution_chain/block_access_list/[block_access_list_builder, block_access_list_validation]
 
-template toBytes32(slot: UInt256): Bytes32 =
-  Bytes32(slot.toBytesBE())
-
 suite "Block access list validation":
   const
     address1 = address"0x10007bc31cedb7bfb8a345f31e668033056b2728"
@@ -81,7 +78,7 @@ suite "Block access list validation":
     builder.addStorageWrite(address1, slot3, 3, 3.u256)
 
     var bal = builder.buildBlockAccessList()
-    bal[0].storageReads = @[slot1.toBytes32()]
+    bal[0].storageReads = @[slot1]
     check bal.validate(bal.computeBlockAccessListHash()).isErr()
 
   test "Account changes out of order should fail validation":
diff --git a/vendor/nim-eth b/vendor/nim-eth
@@ -1 +1 @@
-Subproject commit d86bb194402ac53a1dbd5e8a0e374c4da81afc6d
+Subproject commit fa62e8b5b01e07d9e8fe1a91cd0f76afb173ec35
