audit patches + improvement (#1600)

Signed-off-by: shankar <shankar@layerzerolabs.org>

Author: shankars99

packages/ovault-evm/contracts/OVaultComposer.sol

@@ -14,6 +14,7 @@ import { OFTComposeMsgCodec } from "@layerzerolabs/oft-evm/contracts/libs/OFTCom
 
 import { IOFTWithDecimalConversionRate as IOFT } from "./interfaces/IOFTWithDecimalConversionRate.sol";
 import { IOVaultComposer, FailedMessage, FailedState } from "./interfaces/IOVaultComposer.sol";
+import { IOFTWithDecimalConversionRate as IOFT } from "./interfaces/IOFTWithDecimalConversionRate.sol";
 
 contract OVaultComposer is IOVaultComposer, ReentrancyGuard {
     using OFTComposeMsgCodec for bytes;
@@ -34,6 +35,11 @@ contract OVaultComposer is IOVaultComposer, ReentrancyGuard {
 
     address public immutable REFUND_OVERPAY_ADDRESS;
 
+    address public immutable REFUND_OVERPAY_ADDRESS;
+
+    uint256 public immutable ASSET_DECIMAL_CONVERSION_RATE;
+    uint256 public immutable SHARE_DECIMAL_CONVERSION_RATE;
+
     mapping(bytes32 guid => FailedMessage) public failedMessages;
 
     constructor(address _ovault, address _assetOFT, address _shareOFT, address _refundOverpayAddress) {

packages/ovault-evm/contracts/interfaces/IOVaultComposer.sol

@@ -49,7 +49,6 @@ interface IOVaultComposer is IOAppComposer {
     error CanNotRefund(bytes32 guid);
     error CanNotRetry(bytes32 guid);
     error CanNotSwap(bytes32 guid);
-
     error NotEnoughTargetTokens(uint256 amountLD, uint256 minAmountLD);
     error NoMsgValueWhenSkippingRetry();
     error NoMsgValueOnSameChainOVaultAction();

packages/ovault-evm/test/composer/OVaultComposer_Base.t.sol

@@ -48,6 +48,8 @@ contract OVaultComposerBaseTest is TestHelperOz5 {
     address public userB = makeAddr("userB");
     address public refundOverpayAddress = makeAddr("refundOverpayAddress");
 
+    address public refundOverpayAddress = makeAddr("refundOverpayAddress");
+
     address public arbEndpoint;
     address public arbExecutor = makeAddr("arbExecutor");
     bytes public OPTIONS_LZRECEIVE_100k = OptionsBuilder.newOptions().addExecutorLzReceiveOption(100_000, 0);