fix(contracts): make releaseValueToRecipient internal (#4989)

### Description

- partially fixes
chainlight-io/2024-08-hyperlane#1

### Drive-by changes

<!--
Are there any minor or drive-by changes also included?
-->

### Related issues

<!--
- Fixes #[issue number here]
-->

### Backward compatibility

<!--
Are these changes backward compatible? Are there any infrastructure
implications, e.g. changes that would prohibit deploying older commits
using this infra tooling?

Yes/No
-->

### Testing

<!--
What kind of testing have these changes undergone?

None/Manual/Unit Tests
-->

.changeset/nine-eyes-smile.md

@@ -0,0 +1,5 @@
+---
+'@hyperlane-xyz/core': minor
+---
+
+Made releaseValueToRecipient internal

solidity/contracts/isms/hook/AbstractMessageIdAuthorizedIsm.sol

@@ -78,28 +78,13 @@ abstract contract AbstractMessageIdAuthorizedIsm is
     ) external virtual returns (bool) {
         bool verified = isVerified(message);
         if (verified) {
-            releaseValueToRecipient(message);
+            _releaseValueToRecipient(message);
         }
         return verified;
     }
 
     // ============ Public Functions ============
 
-    /**
-     * @notice Release the value to the recipient if the message is verified.
-     * @param message Message to release value for.
-     */
-    function releaseValueToRecipient(bytes calldata message) public {
-        bytes32 messageId = message.id();
-        uint256 _msgValue = verifiedMessages[messageId].clearBit(
-            VERIFIED_MASK_INDEX
-        );
-        if (_msgValue > 0) {
-            verifiedMessages[messageId] -= _msgValue;
-            payable(message.recipientAddress()).sendValue(_msgValue);
-        }
-    }
-
     /**
      * @notice Check if a message is verified through preVerifyMessage first.
      * @param message Message to check.
@@ -138,6 +123,21 @@ abstract contract AbstractMessageIdAuthorizedIsm is
 
     // ============ Internal Functions ============
 
+    /**
+     * @notice Release the value to the recipient if the message is verified.
+     * @param message Message to release value for.
+     */
+    function _releaseValueToRecipient(bytes calldata message) internal {
+        bytes32 messageId = message.id();
+        uint256 _msgValue = verifiedMessages[messageId].clearBit(
+            VERIFIED_MASK_INDEX
+        );
+        if (_msgValue > 0) {
+            verifiedMessages[messageId] -= _msgValue;
+            payable(message.recipientAddress()).sendValue(_msgValue);
+        }
+    }
+
     /**
      * @notice Check if sender is authorized to message `preVerifyMessage`.
      */

solidity/contracts/isms/hook/ArbL2ToL1Ism.sol

@@ -69,7 +69,7 @@ contract ArbL2ToL1Ism is
             _verifyWithOutboxCall(metadata, message);
             require(isVerified(message), "ArbL2ToL1Ism: message not verified");
         }
-        releaseValueToRecipient(message);
+        _releaseValueToRecipient(message);
         return true;
     }
 

solidity/contracts/isms/hook/OPL2ToL1Ism.sol

@@ -70,7 +70,7 @@ contract OPL2ToL1Ism is
             _verifyWithPortalCall(metadata, message);
             require(isVerified(message), "OPL2ToL1Ism: message not verified");
         }
-        releaseValueToRecipient(message);
+        _releaseValueToRecipient(message);
         return true;
     }