chore: update optimizers runs to 3

Author: skimaharvey

foundry.toml

@@ -6,7 +6,7 @@ exclude = ["script/**/*"]
 evm_version = "prague"
 optimizer = true
 # Runs changed from 10M to 25k to fit code in EVM
-optimizer_runs = 5
+optimizer_runs = 3
 via_ir = false
 solc = "0.8.30"
 verbosity = 3

src/core/interfaces/IMultistrategyVault.sol

@@ -10,10 +10,7 @@ pragma solidity ^0.8.25;
  * @dev Defines all external functions, events, errors, and types for the vault
  *
  * @custom:design-decisions
- * 1. Function Naming Convention: All external functions use snake_case to maintain 1:1 compatibility
- *    with the original Vyper implementation. This ensures seamless integration for existing consumers.
- *
- * 2. Withdraw/Redeem Functions: The interface includes multiple overloads for withdraw/redeem operations:
+ * Withdraw/Redeem Functions: The interface includes multiple overloads for withdraw/redeem operations:
  *    - 3-parameter versions: Basic ERC4626 compliant functions with default parameters
  *    - 5-parameter versions: Advanced functions with maxLoss and custom strategies parameters
  *    Note: Additional overloads with loss parameters exist in Vyper but were omitted due to Solidity