chore: interface update (#66)

Author: github-actions[bot]

src/interfaces/IAccountKeychain.sol

@@ -40,7 +40,7 @@ interface IAccountKeychain {
     struct KeyInfo {
         SignatureType signatureType; // Signature type of the key
         address keyId; // The key identifier (address)
-        uint64 expiry; // Unix timestamp when key expires (0 = never)
+        uint64 expiry; // Unix timestamp when key expires (use type(uint64).max for never)
         bool enforceLimits; // Whether spending limits are enforced for this key
         bool isRevoked; // Whether this key has been revoked
     }
@@ -72,6 +72,7 @@ interface IAccountKeychain {
     error SpendingLimitExceeded();
     error InvalidSignatureType();
     error ZeroPublicKey();
+    error ExpiryInPast();
     error UnauthorizedCaller();
 
     /*//////////////////////////////////////////////////////////////
@@ -84,7 +85,7 @@ interface IAccountKeychain {
      *      The protocol enforces this restriction by checking transactionKey[msg.sender]
      * @param keyId The key identifier (address) to authorize
      * @param signatureType Signature type of the key (0: Secp256k1, 1: P256, 2: WebAuthn)
-     * @param expiry Unix timestamp when key expires (0 = never expires)
+     * @param expiry Unix timestamp when key expires (use type(uint64).max for never expires)
      * @param enforceLimits Whether to enforce spending limits for this key
      * @param limits Initial spending limits for tokens (only used if enforceLimits is true)
      */

src/interfaces/IValidatorConfig.sol

@@ -80,8 +80,15 @@ interface IValidatorConfig {
     /// @notice Change validator active status (owner only)
     /// @param validator The validator address
     /// @param active Whether the validator should be active
+    /// @dev Deprecated: Use changeValidatorStatusByIndex to prevent front-running attacks
     function changeValidatorStatus(address validator, bool active) external;
 
+    /// @notice Change validator active status by index (owner only) - T1+
+    /// @param index The validator index in the validators array
+    /// @param active Whether the validator should be active
+    /// @dev Added in T1 to prevent front-running attacks where a validator changes its address
+    function changeValidatorStatusByIndex(uint64 index, bool active) external;
+
     /// @notice Get the owner of the precompile
     /// @return The owner address
     function owner() external view returns (address);