Function order is incorrect, event definition can not go after custom error definition (line 87)

imported name SafeERC20 is not used

Unexpected import of console file

Explicitly mark visibility of state

Explicitly mark visibility of state

Function body contains 111 lines but allowed no more than 90 lines

global import of path @openzeppelin/contracts/utils/introspection/IERC165.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)

global import of path account-abstraction/core/UserOperationLib.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)

Immutable variables name are set to be in capitalized SNAKE_CASE

GC: Use Custom Errors instead of require statements

GC: Use Custom Errors instead of revert statements

GC: Use Custom Errors instead of require statements

Avoid to use inline assembly. It is acceptable only in rare cases

global import of path @openzeppelin/contracts/token/ERC20/IERC20.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)

'_paymasterId' should not start with _