Change mmr ID type to uint256

l1/src/L1MessagesSender.sol

@@ -56,7 +56,7 @@ contract L1MessagesSender is Ownable {
     }
 
     /// @param aggregatorId The id of a tree previously created by the aggregators factory
-    function sendPoseidonMMRTreeToL2(uint256 aggregatorId, uint128 mmrId) external payable {
+    function sendPoseidonMMRTreeToL2(uint256 aggregatorId, uint256 mmrId) external payable {
         address existingAggregatorAddr = aggregatorsFactory.aggregatorsById(
             aggregatorId
         );
@@ -98,18 +98,19 @@ contract L1MessagesSender is Ownable {
         bytes32 poseidonMMRRoot,
         uint128 mmrSize,
         uint256 aggregatorId,
-        uint128 mmrId
+        uint256 mmrId
     ) internal {
-        uint128 aggregatorId_low = uint128(aggregatorId);
-        uint128 aggregatorId_high = uint128(aggregatorId >> 128);
+        (uint256 aggregatorIdLow, uint256 aggregatorIdHigh) = aggregatorId.split128();
+        (uint256 mmrIdLow, uint256 mmrIdHigh) = mmrId.split128();
 
-        uint256[] memory message = new uint256[](5);
+        uint256[] memory message = new uint256[](6);
 
         message[0] = uint256(poseidonMMRRoot);
         message[1] = mmrSize;
-        message[2] = aggregatorId_low;
-        message[3] = aggregatorId_high;
-        message[4] = mmrId;
+        message[2] = aggregatorIdLow;
+        message[3] = aggregatorIdHigh;
+        message[4] = mmrIdLow;
+        message[5] = mmrIdHigh;
 
         // Pass along msg.value
         starknetCore.sendMessageToL2{value: msg.value}(

l1/test/L1MessagesSender.t.sol

@@ -35,7 +35,7 @@ contract L1MessagesSenderTest is Test {
         // This aggregator id must exist in the factory
         uint256 aggregatorId = 1;
 
-        uint128 mmrId = 4;
+        uint256 mmrId = 4;
 
         // Value must be greater than 0
         sender.sendPoseidonMMRTreeToL2{value: 1}(aggregatorId, mmrId);

src/core/common.cairo

@@ -1,2 +1,2 @@
-type MmrId = u128;
+type MmrId = u256;
 type AggregatorId = u256;