fix: remove unnecessary payable modifiers to reduce risk of locked ether

src/admin/TransparentUpgradeableProxy.sol

@@ -76,7 +76,7 @@ contract TransparentUpgradeableProxy is ERC1967Proxy {
      * backed by the implementation at `_logic`, and optionally initialized with `_data` as explained in
      * {ERC1967Proxy-constructor}.
      */
-    constructor(address _logic, address initialOwner, bytes memory _data) payable ERC1967Proxy(_logic, _data) {
+    constructor(address _logic, address initialOwner, bytes memory _data) ERC1967Proxy(_logic, _data) {
         ADMIN = address(new ProxyAdmin(initialOwner));
         // Set the storage value and emit an event for ERC-1967 compatibility
         ERC1967Utils.changeAdmin(_proxyAdmin());

src/flash/handlers/base/BalancerFlashloanDirectMintHandler.sol

@@ -102,7 +102,6 @@ abstract contract BalancerFlashloanDirectMintHandler is IonHandlerBase, IFlashLo
         uint256 maxResultingDebt
     )
         external
-        payable
     {
         LST_TOKEN.safeTransferFrom(msg.sender, address(this), initialDeposit);
 

src/periphery/IonZapper.sol

@@ -64,7 +64,7 @@ contract IonZapper {
         POOL.repay(ilkIndex, msg.sender, msg.sender, normalizedAmountToRepay);
     }
 
-    function zapDepositWstEth(uint256 amountStEth) external payable {
+    function zapDepositWstEth(uint256 amountStEth) external {
         STETH.transferFrom(msg.sender, address(this), amountStEth);
 
         uint256 outputWstEthAmount = WSTETH.wrap(amountStEth);