feat: Add Flash Loan Amount Validation Logic

[Neros0]

📄 Summary

This pull request introduces enhanced validation mechanisms for flash loan operations within the TestArbitrage.sol contract. It defines safety constraints for both minimum and maximum flash loan amounts to prevent dust attacks and unintentional large-volume tests.

✨ Features Added

• maxFlashAmount variable

  • Sets an upper bound for allowable flash loan amounts.
  • Ensures test safety and limits risk exposure during simulations.

• validFlashAmount modifier

  • Validates that the requested flash loan amount:
    • Is greater than zero.
    • Meets or exceeds minFlashAmount.
    • Does not exceed maxFlashAmount.
  • Helps maintain consistent operational bounds for on-chain testing.

• NatSpec comments

  • Added descriptive NatSpec documentation for both new additions to improve code clarity and maintainability.

🧠 Rationale

By introducing these constraints, the contract gains additional safeguards against:

• Dust attacks (where negligible loan sizes clutter execution).
• Stress conditions from excessively large loan requests during testing.

🧪 Files Modified

• src/onchain/TestArbitrage.sol
  • 14 additions
  • 0 deletions

✅ Commit History

• feat: Add validFlashAmount Modifier
• feat: Add validFlashAmount Modifier Natspec Comments
• feat: Add maxFlashAmount Variable

📅 Date

October 7, 2025

---

Reviewer Note:
Please verify integration with existing loan request flows and ensure the modifier is properly applied in relevant execution paths (e.g., _executeOperation).