refactor: add missing maxWithdraw/maxRedeem 2-param overloads

- Add maxWithdraw(address, uint256) overload to match Vyper interface
- Add maxRedeem(address, uint256) overload to match Vyper interface
- Add set_deposit_limit_module(address) single-param overload
- Fix maxWithdraw(address) default maxLoss from MAX_BPS to 0 in LockedVault

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: skimaharvey

src/core/MultistrategyLockedVault.sol

@@ -609,10 +609,33 @@ contract MultistrategyLockedVault is MultistrategyVault, IMultistrategyLockedVau
         return Math.min(parentMax, custodyAssets);
     }
 
+    /**
+     * @notice Returns maximum assets that owner can withdraw with custom loss tolerance
+     * @dev Uses default queue for withdrawal strategies
+     * @param owner_ Address that owns the shares
+     * @param maxLoss_ Maximum acceptable loss in basis points (0-10000)
+     * @return max Maximum withdrawable assets (constrained by custody)
+     */
+
+    function maxWithdraw(
+        address owner_,
+        uint256 maxLoss_
+    ) public view override(MultistrategyVault, IMultistrategyVault) returns (uint256) {
+        return maxWithdraw(owner_, maxLoss_, new address[](0));
+    }
+
+    /**
+     * @notice Returns maximum assets that owner can withdraw with default parameters
+     * @dev Overload to match Vyper's default parameters behavior (maxLoss = 0, default queue)
+     *      Enforces custody constraints - returns 0 if cooldown period not passed
+     * @param owner_ Address that owns the shares
+     * @return max Maximum withdrawable assets (constrained by custody)
+     */
+
     function maxWithdraw(
         address owner_
     ) external view override(MultistrategyVault, IMultistrategyVault) returns (uint256) {
-        return maxWithdraw(owner_, MAX_BPS, new address[](0));
+        return maxWithdraw(owner_, 0, new address[](0));
     }
 
     /**
@@ -649,6 +672,21 @@ contract MultistrategyLockedVault is MultistrategyVault, IMultistrategyLockedVau
         return Math.min(parentMax, lockedShares);
     }
 
+    /**
+     * @notice Returns maximum shares that owner can redeem with custom loss tolerance
+     * @dev Uses default queue for withdrawal strategies
+     * @param owner_ Address that owns the shares
+     * @param maxLoss_ Maximum acceptable loss in basis points (0-10000)
+     * @return max Maximum redeemable shares (constrained by custody)
+     */
+
+    function maxRedeem(
+        address owner_,
+        uint256 maxLoss_
+    ) public view override(MultistrategyVault, IMultistrategyVault) returns (uint256) {
+        return maxRedeem(owner_, maxLoss_, new address[](0));
+    }
+
     /**
      * @notice Returns maximum shares that owner can redeem with default parameters
      * @dev Overload to match Vyper's default parameters behavior (maxLoss = MAX_BPS, default queue)

src/core/MultistrategyVault.sol

@@ -470,6 +470,18 @@ contract MultistrategyVault is IMultistrategyVault {
         emit UpdateDepositLimit(depositLimit_);
     }
 
+    /**
+     * @notice Sets a module contract to dynamically control deposit limits with default override behavior
+     * @dev Overload to match Vyper's default parameter behavior (shouldOverride = false by default)
+     *      See set_deposit_limit_module(address, bool) for full documentation
+     * @param depositLimitModule_ Address of IDepositLimitModule contract (or address(0) to disable)
+     * @custom:security Only callable by DEPOSIT_LIMIT_MANAGER role
+     * @custom:security Reverts if vault is shutdown
+     */
+    function set_deposit_limit_module(address depositLimitModule_) public {
+        set_deposit_limit_module(depositLimitModule_, false);
+    }
+
     /**
      * @notice Sets a module contract to dynamically control deposit limits
      * @dev Module overrides static depositLimit. Requires depositLimit = type(uint256).max
@@ -479,7 +491,7 @@ contract MultistrategyVault is IMultistrategyVault {
      * @custom:security Only callable by DEPOSIT_LIMIT_MANAGER role
      * @custom:security Reverts if vault is shutdown
      */
-    function set_deposit_limit_module(address depositLimitModule_, bool shouldOverride_) external override {
+    function set_deposit_limit_module(address depositLimitModule_, bool shouldOverride_) public override {
         require(_shutdown == false, VaultShutdown());
         _enforceRole(msg.sender, Roles.DEPOSIT_LIMIT_MANAGER);
 
@@ -1755,6 +1767,34 @@ contract MultistrategyVault is IMultistrategyVault {
             );
     }
 
+    /**
+     * @notice Returns maximum assets that owner can withdraw with custom loss tolerance
+     * @dev Uses default queue for withdrawal strategies
+     * @param owner_ Address that owns the shares
+     * @param maxLoss_ Maximum acceptable loss in basis points (0-10000)
+     * @return max Maximum withdrawable assets
+     */
+    function maxWithdraw(address owner_, uint256 maxLoss_) external view virtual returns (uint256) {
+        address[] memory emptyArray = new address[](0);
+        return _max_withdraw(owner_, maxLoss_, emptyArray);
+    }
+
+    /**
+     * @notice Returns maximum shares that owner can redeem with custom loss tolerance
+     * @dev Uses default queue for withdrawal strategies
+     * @param owner_ Address that owns the shares
+     * @param maxLoss_ Maximum acceptable loss in basis points (0-10000)
+     * @return max Maximum redeemable shares
+     */
+    function maxRedeem(address owner_, uint256 maxLoss_) external view virtual returns (uint256) {
+        address[] memory emptyArray = new address[](0);
+        return
+            Math.min(
+                _convertToShares(_max_withdraw(owner_, maxLoss_, emptyArray), Rounding.ROUND_DOWN),
+                _balanceOf[owner_]
+            );
+    }
+
     /**
      * @notice Returns maximum assets that owner can withdraw with default parameters
      * @dev Overload to match Vyper's default parameters behavior (maxLoss = 0, default queue)

src/core/interfaces/IMultistrategyVault.sol

@@ -396,10 +396,12 @@ interface IMultistrategyVault {
 
     // MaxWithdraw functions with different parameter combinations to match Vyper
     function maxWithdraw(address owner) external view returns (uint256);
+    function maxWithdraw(address owner, uint256 maxLoss) external view returns (uint256);
     function maxWithdraw(address owner, uint256 maxLoss, address[] memory strategies) external view returns (uint256);
 
     // MaxRedeem functions with different parameter combinations to match Vyper
     function maxRedeem(address owner) external view returns (uint256);
+    function maxRedeem(address owner, uint256 maxLoss) external view returns (uint256);
     function maxRedeem(address owner, uint256 maxLoss, address[] memory strategies) external view returns (uint256);
 
     function FACTORY() external view returns (address);