There is a redundant status assignment in the ChakraSettlement::receive_cross_chain_msg:
ChakraSettlement.sol
When ISettlementHandler(to_handler).receive_cross_chain_msg is called we need the returned value in order to determine the status of the transaction and emit the event which will trigger the callback, but the result variable contains redundant else statement:
We can simply assign the default status to Failed and only have if  statement which will change the status to Success.
Same if check is observed in settlement contract:
settlement.cairo
Here we also want to check what is the status of the execution of handler_erc20 and emit even with the right status. Now the if is redundant because it wont change the status.
Remove the else statement and leave only the if check:
And in Cairo:
