Lack of Minimum Amount Check in `SmartVaultV3::mint`, `SmartVaultV3::burn`, and `SmartVaultV3::swap` Can Result in Loss of Fees

Zero address leads to transaction reverts

Lender fails to giveLoan because of inconsistent length between `loadIds` and `poolIds`

Uncheck Arithmetic where overflow/underflow impossible

Using Private Rather Than Public For Constants,Saves Gas

Use of magic numbers

Named parameter mappings

For the borrow(), repay() & startAuction() functions in Lender.sol the public visibility modifiers should be changed to external, to help optimize gas usage

Use assembly to check for `address(0)`

NatSpec documentation for function is missing

Large multiples of ten should use scientific notation

Constants in comparisons should appear on the left side

Cache array length outside of loop

Function ordering does not follow the Solidity style guide

Long functions should be refactored into multiple, smaller, functions

Interfaces should be indicated with an `I` prefix in the contract name

Caching the pool variable for gas savings

Use `==` instead for `<=` for `uints` when comparing for `zero` values

`++i`/`i++` should be `unchecked{++i}`/`unchecked{i++}` when it is not possible for them to overflow, as is the case when used in `for`- and `while`-loops

Constants should be be used for hardcoded values

The nonReentrant modifier should occur before all other modifiers

NatSpec `@return` argument is missing

[I-4] Constants in comparisons should appear on the left side

Use `assembly` to check for `address(0)`

Constructor of `Escrow` should make sure that `buyer`, `seller`, `arbiter` are different from each other.

Use assembly to check for `address(0)`

The `nonReentrant` `modifier` should occur before all other modifiers

NatSpec `@param` is missing

NatSpec `@return` argument is missing

Constants in comparisons should appear on the left side