add custom errors

contracts/SoulLinker.sol

@@ -162,8 +162,8 @@ contract SoulLinker is PaymentGateway, EIP712, Pausable {
         require(identityOwner == tokenOwner, "IDENTITY_OWNER_NOT_TOKEN_OWNER");
         if (identityOwner != _msgSender()) revert CallerNotOwner(_msgSender());
         require(expirationDate >= block.timestamp, "VALID_PERIOD_EXPIRED");
-        require(
-            _verify(
+        if (
+            !_verify(
                 _hash(
                     readerIdentityId,
                     ownerIdentityId,
@@ -175,9 +175,8 @@ contract SoulLinker is PaymentGateway, EIP712, Pausable {
                 ),
                 signature,
                 identityOwner
-            ),
-            "INVALID_SIGNATURE"
-        );
+            )
+        ) revert InvalidSignature();
 
         if (addPermissionPriceMASA > 0) {
             // if there is a price in MASA, pay it without conversion rate

contracts/SoulboundCreditScore.sol

@@ -57,7 +57,7 @@ contract SoulboundCreditScore is MasaSBTSelfSovereign {
     ) public payable virtual returns (uint256) {
         address to = soulboundIdentity.ownerOf(identityId);
         if (to != _msgSender()) revert CallerNotOwner(_msgSender());
-        require(balanceOf(to) < 1, "CREDITSCORE_ALREADY_CREATED");
+        if (balanceOf(to) > 0) revert CreditScoreAlreadyCreated(to);
 
         _verify(
             _hash(identityId, authorityAddress, signatureDate),

contracts/SoulboundIdentity.sol

@@ -46,7 +46,7 @@ contract SoulboundIdentity is MasaSBTAuthority, ISoulboundIdentity {
     /// @param to Address of the admin of the new identity
     function mint(address to) public override returns (uint256) {
         // Soulbound identity already created!
-        require(balanceOf(to) < 1, "SB_IDENTITY_ALREADY_CREATED");
+        if (balanceOf(to) > 0) revert IdentityAlreadyCreated(to);
 
         return _mintWithCounter(to);
     }
@@ -220,7 +220,7 @@ contract SoulboundIdentity is MasaSBTAuthority, ISoulboundIdentity {
     /* ========== MODIFIERS ================================================= */
 
     modifier soulNameAlreadySet() {
-        require(address(soulName) != address(0), "SOULNAME_CONTRACT_NOT_SET");
+        if (address(soulName) == address(0)) revert SoulNameContractNotSet();
         _;
     }
 

contracts/dex/PaymentGateway.sol

@@ -109,7 +109,7 @@ abstract contract PaymentGateway is Ownable {
     /// @param _erc20token New ERC20 token to add
     function addErc20Token(address _erc20token) external onlyOwner {
         if (_erc20token == address(0)) revert ZeroAddress();
-        require(!erc20token[_erc20token], "ALREADY_ADDED");
+        if (erc20token[_erc20token]) revert AlreadyAdded();
 
         erc20token[_erc20token] = true;
         erc20tokens.push(_erc20token);

contracts/libraries/Errors.sol

@@ -1,7 +1,13 @@
 // SPDX-License-Identifier: Apache-2.0
 pragma solidity ^0.8.7;
 
+error AlreadyAdded();
 error CallerNotOwner(address caller);
+error CreditScoreAlreadyCreated(address to);
+error IdentityAlreadyCreated(address to);
 error InvalidPaymentMethod(address paymentMethod);
+error InvalidSignature();
+error NotAuthorized(address signer);
 error SameValue();
+error SoulNameContractNotSet();
 error ZeroAddress();

contracts/tokens/MasaSBTSelfSovereign.sol

@@ -92,7 +92,7 @@ abstract contract MasaSBTSelfSovereign is PaymentGateway, MasaSBT, EIP712 {
         onlyRole(DEFAULT_ADMIN_ROLE)
     {
         if (_authority == address(0)) revert ZeroAddress();
-        require(!authorities[_authority], "ALREADY_ADDED");
+        if (authorities[_authority]) revert AlreadyAdded();
 
         authorities[_authority] = true;
     }
@@ -137,8 +137,8 @@ abstract contract MasaSBTSelfSovereign is PaymentGateway, MasaSBT, EIP712 {
         address signer
     ) internal view {
         address _signer = ECDSA.recover(digest, signature);
-        require(_signer == signer, "INVALID_SIGNATURE");
-        require(authorities[_signer], "NOT_AUTHORIZED");
+        if (_signer != signer) revert InvalidSignature();
+        if (!authorities[_signer]) revert NotAuthorized(_signer);
     }
 
     function _mintWithCounter(address to) internal virtual returns (uint256) {

docs/SoulLinker.md

@@ -836,6 +836,17 @@ event Unpaused(address account)
 
 ## Errors
 
+### AlreadyAdded
+
+```solidity
+error AlreadyAdded()
+```
+
+
+
+
+
+
 ### CallerNotOwner
 
 ```solidity
@@ -868,6 +879,17 @@ error InvalidPaymentMethod(address paymentMethod)
 |---|---|---|
 | paymentMethod | address | undefined |
 
+### InvalidSignature
+
+```solidity
+error InvalidSignature()
+```
+
+
+
+
+
+
 ### SameValue
 
 ```solidity

docs/SoulStore.md

@@ -649,6 +649,17 @@ event Unpaused(address account)
 
 ## Errors
 
+### AlreadyAdded
+
+```solidity
+error AlreadyAdded()
+```
+
+
+
+
+
+
 ### InvalidPaymentMethod
 
 ```solidity

docs/Soulbound2FA.md

@@ -959,6 +959,17 @@ event Soulbound2FAMinted(uint256 tokenId, uint256 identityId, address authorityA
 
 ## Errors
 
+### AlreadyAdded
+
+```solidity
+error AlreadyAdded()
+```
+
+
+
+
+
+
 ### CallerNotOwner
 
 ```solidity
@@ -991,6 +1002,33 @@ error InvalidPaymentMethod(address paymentMethod)
 |---|---|---|
 | paymentMethod | address | undefined |
 
+### InvalidSignature
+
+```solidity
+error InvalidSignature()
+```
+
+
+
+
+
+
+### NotAuthorized
+
+```solidity
+error NotAuthorized(address signer)
+```
+
+
+
+
+
+#### Parameters
+
+| Name | Type | Description |
+|---|---|---|
+| signer | address | undefined |
+
 ### SameValue
 
 ```solidity

docs/SoulboundCreditScore.md

@@ -959,6 +959,17 @@ event SoulboundCreditScoreMinted(uint256 tokenId, uint256 identityId, address au
 
 ## Errors
 
+### AlreadyAdded
+
+```solidity
+error AlreadyAdded()
+```
+
+
+
+
+
+
 ### CallerNotOwner
 
 ```solidity
@@ -975,6 +986,22 @@ error CallerNotOwner(address caller)
 |---|---|---|
 | caller | address | undefined |
 
+### CreditScoreAlreadyCreated
+
+```solidity
+error CreditScoreAlreadyCreated(address to)
+```
+
+
+
+
+
+#### Parameters
+
+| Name | Type | Description |
+|---|---|---|
+| to | address | undefined |
+
 ### InvalidPaymentMethod
 
 ```solidity
@@ -991,6 +1018,33 @@ error InvalidPaymentMethod(address paymentMethod)
 |---|---|---|
 | paymentMethod | address | undefined |
 
+### InvalidSignature
+
+```solidity
+error InvalidSignature()
+```
+
+
+
+
+
+
+### NotAuthorized
+
+```solidity
+error NotAuthorized(address signer)
+```
+
+
+
+
+
+#### Parameters
+
+| Name | Type | Description |
+|---|---|---|
+| signer | address | undefined |
+
 ### SameValue
 
 ```solidity

docs/SoulboundIdentity.md

@@ -731,6 +731,44 @@ event RoleRevoked(bytes32 indexed role, address indexed account, address indexed
 
 ## Errors
 
+### IdentityAlreadyCreated
+
+```solidity
+error IdentityAlreadyCreated(address to)
+```
+
+
+
+
+
+#### Parameters
+
+| Name | Type | Description |
+|---|---|---|
+| to | address | undefined |
+
+### SameValue
+
+```solidity
+error SameValue()
+```
+
+
+
+
+
+
+### SoulNameContractNotSet
+
+```solidity
+error SoulNameContractNotSet()
+```
+
+
+
+
+
+
 ### ZeroAddress
 
 ```solidity

docs/dex/PaymentGateway.md

@@ -337,6 +337,17 @@ event OwnershipTransferred(address indexed previousOwner, address indexed newOwn
 
 ## Errors
 
+### AlreadyAdded
+
+```solidity
+error AlreadyAdded()
+```
+
+
+
+
+
+
 ### SameValue
 
 ```solidity

docs/tokens/MasaSBTSelfSovereign.md

@@ -886,6 +886,17 @@ event RoleRevoked(bytes32 indexed role, address indexed account, address indexed
 
 ## Errors
 
+### AlreadyAdded
+
+```solidity
+error AlreadyAdded()
+```
+
+
+
+
+
+
 ### InvalidPaymentMethod
 
 ```solidity

test/SoulboundCreditScore.test.ts

@@ -219,7 +219,7 @@ describe("Soulbound Credit Score", () => {
             signatureDate,
             signature
           )
-      ).to.be.revertedWith("CREDITSCORE_ALREADY_CREATED");
+      ).to.be.revertedWith("CreditScoreAlreadyCreated");
 
       expect(await soulboundCreditScore.totalSupply()).to.equal(1);
       expect(await soulboundCreditScore.tokenByIndex(0)).to.equal(0);
@@ -279,7 +279,7 @@ describe("Soulbound Credit Score", () => {
             signatureDate,
             signatureNonAuthority
           )
-      ).to.be.revertedWith("NOT_AUTHORIZED");
+      ).to.be.revertedWith("NotAuthorized");
     });
 
     it("should fail to mint with invalid signature", async () => {
@@ -298,7 +298,7 @@ describe("Soulbound Credit Score", () => {
             signatureDate,
             signatureNonAuthority
           )
-      ).to.be.revertedWith("INVALID_SIGNATURE");
+      ).to.be.revertedWith("InvalidSignature");
     });
   });