Wrong parameter passed in `TradingAccount::deductAccountMargin` function that results in excess margin withdrawal

Incompatibility with Multisig Wallets in `TempleGold::send` Function

Not upadting `_totalAuctionTokenAllocation` when removing last auction config at cooldown leads to wrong accounting of `_totalAuctionTokenAllocation` and permanent lock of auction tokens

TempleGold tokens cannot be recovered when a `DaiGoldAuction` ends with 0 bids

Malicious user can prevent `rewardData.perodfinish` from ending by calling `TempleGoldStaking::distributeRewards()` before the end of the reward duration when no starter is set.

`Edition.mintBatch()` uses `msg.value` in a for loop, allowing users to mint tokens for free

Emergency Closed Vault Can Be Paused Then Resume

Lenders can escape the blacklisting of their accounts because they can move their MarketTokens to different accounts and gain the WithdrawOnly Role on any account they want

Protocol markets are incompatible with rebasing tokens

Incorrect check for cRation_MAX

Instant arbitrage opportunity through rETH and stETH price discrepancy

Malicious/Compromised organiser can reclaw all funds, stealing work from supporters

Centralization Risk for trusted organizers

Hardcoded Router Address May Cause Token Lockup in Non-Standard Networks

Fixed fee level is used when swap tokens on Uniswap

Zero address leads to transaction reverts

Missing Events Emitting

Amount != 0 checks are missing

Cannot use `_burn` Function in Beedle.sol Contract

[L-07] interface Staking.FeeDistribution is not used

NatSpec documentation for function is missing

Floating pragma in all contracts

[L-06] Some imported libraries are not used

Typo in README "potisitions"

All of the USD pair price feeds doesn't have 8 decimals

`liquidate` does not allow the liquidator to liquidate a user if the liquidator HF < 1

Improving the burnDsc() to allow users to mitigate their liquidation's impact

Pragma isn't specified correctly which can lead to nonfunction/damaged contract when deployed on Arbitrum

Precision loss when calculating the health factor

`burn()` and `staleCheckLatestRoundData()` and `getTimeout()` can be `external`

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

No amountCollateral > balance check

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