Hawk High

Summary

The newly proposed implementation contract (LevelTwo) intended to replace the existing school system fails to conform to the UUPSUpgradeable standard required for upgradeability. The contract does not inherit from UUPSUpgradeable, nor does it implement the required authorization mechanism. As a result, any attempt to upgrade to this implementation using the UUPS proxy will fail or could brick the proxy contract.

The design explicitly states that UUPS upgradeability is used as the pattern for upgrades between school sessions. Therefore, all implementation contracts must adhere to this pattern to ensure compatibility with the proxy and continued functionality.

Impact

• The current implementation of levelTwo.sol break the following protocol invariant

  "These contracts utilize the UUPSUpgradeable library from OpenZeppelin."

Recommendation

• Follow openzeppelinUpgradeableStandards