feat(role): add revoke proposed roles

.changeset/selfish-llamas-raise.md

@@ -0,0 +1,5 @@
+---
+'@axelar-network/axelar-gmp-sdk-solidity': patch
+---
+
+feat(role): add revoke proposed roles

contracts/utils/RolesBase.sol

@@ -141,6 +141,16 @@ contract RolesBase is IRolesBase {
         }
     }
 
+    /**
+     * @notice Internal function to revoke any proposed roles between two accounts.
+     * @dev Sets the proposed roles from `fromAccount` to `toAccount` to zero, effectively canceling any pending role transfer.
+     * @param fromAccount The address proposing the role transfer.
+     * @param toAccount The address set to receive the proposed roles.
+     */
+    function _revokeProposedRoles(address fromAccount, address toAccount) internal {
+        _setProposedRoles(fromAccount, toAccount, 0);
+    }
+
     /**
      * @notice Internal function to add a role to an account.
      * @dev emits a RolesAdded event.
@@ -342,7 +352,6 @@ contract RolesBase is IRolesBase {
             revert InvalidProposedRoles(fromAccount, toAccount, accountRoles);
         }
 
-        _setProposedRoles(fromAccount, toAccount, 0);
         _transferAccountRoles(fromAccount, toAccount, accountRoles);
     }
 
@@ -390,6 +399,7 @@ contract RolesBase is IRolesBase {
     ) internal {
         if (!_hasAllTheRoles(_getRoles(fromAccount), accountRoles)) revert MissingAllRoles(fromAccount, accountRoles);
 
+        _revokeProposedRoles(fromAccount, toAccount);
         _removeAccountRoles(fromAccount, accountRoles);
         _addAccountRoles(toAccount, accountRoles);
     }