fix: solhint

Author: YukiTsuchida

src/core/Authenticator.sol

@@ -4,7 +4,6 @@ pragma solidity ^0.8.20;
 import {IAuthenticator} from "./IAuthenticator.sol";
 import {TxAuthManagerBase} from "./TxAuthManager.sol";
 import {TxManagerBase} from "./TxManagerBase.sol";
-import {IAuthExtensionVerifier} from "./IAuthExtensionVerifier.sol";
 
 import {
     AuthType,
@@ -14,8 +13,7 @@ import {
     MsgExtSignTxResponse,
     QueryTxAuthStateRequest,
     QueryTxAuthStateResponse,
-    Account,
-    TxAuthState
+    Account
 } from "../proto/cross/core/auth/Auth.sol";
 import {GoogleProtobufAny} from "@hyperledger-labs/yui-ibc-solidity/contracts/proto/GoogleProtobufAny.sol";
 

src/core/TxAuthManager.sol

@@ -8,16 +8,16 @@ import {IAuthExtensionVerifier} from "./IAuthExtensionVerifier.sol";
 
 abstract contract TxAuthManager is TxAuthManagerBase, CrossStore {
     constructor(string[] memory typeUrls, IAuthExtensionVerifier[] memory verifiers) {
-        require(typeUrls.length == verifiers.length, "TxAuthManager: array mismatch");
+        if (typeUrls.length != verifiers.length) revert ArrayLengthMismatch();
 
         CrossStore.AuthStorage storage s = _getAuthStorage();
 
-        for (uint256 i = 0; i < typeUrls.length; i++) {
+        for (uint256 i = 0; i < typeUrls.length; ++i) {
             string memory typeUrl = typeUrls[i];
             IAuthExtensionVerifier verifier = verifiers[i];
 
-            require(bytes(typeUrl).length > 0, "TxAuthManager: empty typeUrl");
-            require(address(verifier) != address(0), "TxAuthManager: zero address verifier");
+            if (bytes(typeUrl).length == 0) revert EmptyTypeUrl();
+            if (address(verifier) == address(0)) revert ZeroAddressVerifier();
 
             s.authVerifiers[typeUrl] = verifier;
             emit VerifierRegistered(typeUrl, address(verifier));
@@ -91,7 +91,7 @@ abstract contract TxAuthManager is TxAuthManagerBase, CrossStore {
 
         CrossStore.AuthStorage storage s = _getAuthStorage();
 
-        for (uint256 i = 0; i < signers.length; i++) {
+        for (uint256 i = 0; i < signers.length; ++i) {
             Account.Data calldata signer = signers[i];
 
             if (signer.auth_type.mode != AuthType.AuthMode.AUTH_MODE_EXTENSION) {

src/core/TxAuthManagerBase.sol

@@ -11,8 +11,11 @@ abstract contract TxAuthManagerBase {
     error AuthModeMismatch();
     error VerifierNotFound(string typeUrl);
     error SignatureVerificationFailed(bytes32 txID);
+    error ArrayLengthMismatch();
+    error EmptyTypeUrl();
+    error ZeroAddressVerifier();
 
-    event VerifierRegistered(string typeUrl, address verifier);
+    event VerifierRegistered(string typeUrl, address indexed verifier);
 
     function initAuthState(bytes32 txID, Account.Data[] memory signers) internal virtual;
     function isCompletedAuth(bytes32 txID) internal view virtual returns (bool);

test/Authenticator.t.sol

@@ -9,7 +9,6 @@ import {TxAuthManagerBase} from "../src/core/TxAuthManagerBase.sol";
 
 import {MsgInitiateTx} from "../src/proto/cross/core/initiator/Initiator.sol";
 import {IAuthenticator} from "../src/core/IAuthenticator.sol";
-import {IAuthExtensionVerifier} from "../src/core/IAuthExtensionVerifier.sol";
 import {
     Account as AuthAccount,
     AuthType,