https://cantina.xyz/code/fe8e03b1-f8db-4698-8a21-44b3a862e434/findings/30
The report is overly dramatic. BatchManagerOperations is actually immutable, just not in the solidity sense. But once set in the constructor, it can never be changed.
I think we originally kept it in storage because we were optimizing for contract size, however, I saw that making it actually immutable would still leave us 1.2kb under the size limit
https://cantina.xyz/code/fe8e03b1-f8db-4698-8a21-44b3a862e434/findings/30
The report is overly dramatic. BatchManagerOperations is actually immutable, just not in the solidity sense. But once set in the constructor, it can never be changed.
I think we originally kept it in storage because we were optimizing for contract size, however, I saw that making it actually immutable would still leave us 1.2kb under the size limit