Using basis points for percentage is not precise enough for realistic use-cases

Single-step process for critical ownership transfer is risky

Pragma non-specification can lead to non-functional / corrupted contract when deployed on Arbitrum

Zero address leads to transaction reverts

Amount != 0 checks are missing

Event Based Reentrancy due to CallBack Tokens

Multiple accesses of a mapping/array should use a local variable cache.

+= and -= are more expensive

Uncheck Arithmetic where overflow/underflow impossible

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

Constants in comparisons should appear on the left side

Floating pragma in all contracts

Mixed usage of `int`/`uint` with `int256`/`uint256`

Function ordering does not follow the Solidity style guide

Use predefined address instead of `address(this)`

Contract Can Be Deployed Without Funds.

Use assembly to check for `address(0)`

NatSpec `@param` is missing

NatSpec `@return` argument is missing

Use predefined address instead of `address(this)`

Constants should be defined rather than using magic numbers

Risk of silent overflow in reserves update