QA Report

[code423n4]

1.HomeFi.sol setTrustedForwarder()
change the forwarder has a great impact, recommended to add events, good for monitoring important event

    /// @inheritdoc IHomeFi
    function setTrustedForwarder(address _newForwarder)
        external
        override
        onlyAdmin
        noChange(trustedForwarder, _newForwarder)
    {
        trustedForwarder = _newForwarder;

        ++++ emit ForwarderReplaced(_newForwarder);
    }

2. project.currency() is not IDebtToken, it should be IERC20
   The wrong type of IDebtToken is used, which leads to confusion in understanding the logic of contract, DebtToken is not used for payment and cannot be transfer.
   Community.sol also has the same problem, suggest to change it to IERC20:

contract Project is
    IProject,
    ReentrancyGuardUpgradeable,
    ERC2771ContextUpgradeable
{
   --- using SafeERC20Upgradeable for IDebtToken;
   +++ using SafeERC20 for IERC20; 

...
...
    ---- IDebtToken public override currency;
    ++++ IERC20 public override currency;
....
}

function initialize(
    address _currency,
    address _sender,
    address _homeFiAddress
) external override initializer {
    // Initialize variables
    homeFi = IHomeFi(_homeFiAddress);
    disputes = homeFi.disputesContract();
    lenderFee = homeFi.lenderFee();
    builder = _sender;
    --- currency = IDebtToken(_currency);
    +++ currency = IERC20(_currency);
}