@ensuro/core 2.8.0 → 2.8.2

package/contracts/SignedBucketRiskModule.sol

@@ -2,15 +2,16 @@
 pragma solidity ^0.8.0;
 
 import {ECDSA} from "@openzeppelin/contracts/utils/cryptography/ECDSA.sol";
+
 import {IPolicyPool} from "./interfaces/IPolicyPool.sol";
 import {IPremiumsAccount} from "./interfaces/IPremiumsAccount.sol";
-import {RiskModule} from "./RiskModule.sol";
 import {Policy} from "./Policy.sol";
+import {RiskModule} from "./RiskModule.sol";
 
 /**
- * @title SignedQuote Risk Module
+ * @title SignedBucket Risk Module
  * @dev Risk Module that for policy creation verifies the different components of the price have been signed by a
-        trusted account (PRICER_ROLE). One of the components of the price it's a bucket id that groups policies within
+        trusted account (PRICER_ROLE). One of the components of the price is a bucket id that groups policies within
         a risk module, with different parameters (such as collaterallization levels or fees).
         For the resolution (resolvePolicy), it has to be called by an authorized user
   * @custom:security-contact security@ensuro.co
@@ -18,12 +19,10 @@ import {Policy} from "./Policy.sol";
  */
 contract SignedBucketRiskModule is RiskModule {
   bytes32 public constant POLICY_CREATOR_ROLE = keccak256("POLICY_CREATOR_ROLE");
+  bytes32 public constant REPLACER_ROLE = keccak256("REPLACER_ROLE");
   bytes32 public constant PRICER_ROLE = keccak256("PRICER_ROLE");
   bytes32 public constant RESOLVER_ROLE = keccak256("RESOLVER_ROLE");
 
-  /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
-  bool internal immutable _creationIsOpen;
-
   mapping(uint256 => PackedParams) internal _buckets;
 
   /**
@@ -47,14 +46,12 @@ contract SignedBucketRiskModule is RiskModule {
    */
   event NewSignedPolicy(uint256 indexed policyId, bytes32 policyData);
 
+  error QuoteExpired();
+  error BucketCannotBeZero();
+  error BucketNotFound();
+
   /// @custom:oz-upgrades-unsafe-allow constructor
-  constructor(
-    IPolicyPool policyPool_,
-    IPremiumsAccount premiumsAccount_,
-    bool creationIsOpen_
-  ) RiskModule(policyPool_, premiumsAccount_) {
-    _creationIsOpen = creationIsOpen_;
-  }
+  constructor(IPolicyPool policyPool_, IPremiumsAccount premiumsAccount_) RiskModule(policyPool_, premiumsAccount_) {}
 
   /**
    * @dev Initializes the RiskModule
@@ -89,9 +86,7 @@ contract SignedBucketRiskModule is RiskModule {
     bytes32 quoteSignatureVS,
     uint40 quoteValidUntil
   ) internal view {
-    if (!_creationIsOpen)
-      _policyPool.access().checkComponentRole(address(this), POLICY_CREATOR_ROLE, _msgSender(), false);
-    require(quoteValidUntil >= block.timestamp, "Quote expired");
+    if (quoteValidUntil < block.timestamp) revert QuoteExpired();
 
     /**
      * Checks the quote has been signed by an authorized user
@@ -134,55 +129,6 @@ contract SignedBucketRiskModule is RiskModule {
     return createdPolicy;
   }
 
-  /**
-   * @dev Creates a new Policy using a signed quote. The caller is the payer of the policy. Returns all the struct, not just the id.
-   *
-   * Requirements:
-   * - The caller approved the spending of the premium to the PolicyPool
-   * - The quote has been signed by an address with the component role PRICER_ROLE
-   *
-   *  Emits:
-   * - {PolicyPool.NewPolicy}
-   *  - {NewSignedPolicy}
-   *
-   * @param payout The exposure (maximum payout) of the policy
-   * @param premium The premium that will be paid by the payer
-   * @param lossProb The probability of having to pay the maximum payout (wad)
-   * @param expiration The expiration of the policy (timestamp)
-   * @param onBehalfOf The policy holder
-   * @param policyData A hash of the private details of the policy. The last 96 bits will be used as internalId
-   * @param bucketId Identifies the group to which the policy belongs (that defines the RM parameters applicable to it)
-   * @param quoteSignatureR The signature of the quote. R component (EIP-2098 signature)
-   * @param quoteSignatureVS The signature of the quote. VS component (EIP-2098 signature)
-   * @param quoteValidUntil The expiration of the quote
-   * @return createdPolicy Returns the created policy
-   */
-  function newPolicyFull(
-    uint256 payout,
-    uint256 premium,
-    uint256 lossProb,
-    uint40 expiration,
-    address onBehalfOf,
-    bytes32 policyData,
-    uint256 bucketId,
-    bytes32 quoteSignatureR,
-    bytes32 quoteSignatureVS,
-    uint40 quoteValidUntil
-  ) external whenNotPaused returns (Policy.PolicyData memory createdPolicy) {
-    _checkSignature(
-      payout,
-      premium,
-      lossProb,
-      expiration,
-      policyData,
-      bucketId,
-      quoteSignatureR,
-      quoteSignatureVS,
-      quoteValidUntil
-    );
-    return _newPolicySigned(payout, premium, lossProb, expiration, policyData, bucketId, _msgSender(), onBehalfOf);
-  }
-
   /**
    * @dev Creates a new Policy using a signed quote. The caller is the payer of the policy.
    *
@@ -217,7 +163,7 @@ contract SignedBucketRiskModule is RiskModule {
     bytes32 quoteSignatureR,
     bytes32 quoteSignatureVS,
     uint40 quoteValidUntil
-  ) external whenNotPaused returns (uint256) {
+  ) external whenNotPaused onlyComponentRole(POLICY_CREATOR_ROLE) returns (uint256) {
     _checkSignature(
       payout,
       premium,
@@ -233,21 +179,22 @@ contract SignedBucketRiskModule is RiskModule {
   }
 
   /**
-   * @dev Creates a new Policy using a signed quote. The payer is the policy holder
+   * @dev Replace a policy with a new one, reusing the premium and the capital locked
    *
    * Requirements:
-   * - currency().allowance(onBehalfOf, _msgSender()) > 0
+   * - The caller approved the spending of the premium to the PolicyPool
    * - The quote has been signed by an address with the component role PRICER_ROLE
+   * - The caller has been granted component role REPLACER_ROLE or creation is open
    *
    * Emits:
+   * - {PolicyPool.PolicyReplaced}
    * - {PolicyPool.NewPolicy}
-   * - {NewSignedPolicy}
    *
-   * @param payout The exposure (maximum payout) of the policy
-   * @param premium The premium that will be paid by the payer
+   * @param oldPolicy The policy to be replaced
+   * @param payout The exposure (maximum payout) of the new policy
+   * @param premium The premium that will be paid by the caller
    * @param lossProb The probability of having to pay the maximum payout (wad)
    * @param expiration The expiration of the policy (timestamp)
-   * @param onBehalfOf The policy holder
    * @param policyData A hash of the private details of the policy. The last 96 bits will be used as internalId
    * @param bucketId Identifies the group to which the policy belongs (that defines the RM parameters applicable to it)
    * @param quoteSignatureR The signature of the quote. R component (EIP-2098 signature)
@@ -255,32 +202,18 @@ contract SignedBucketRiskModule is RiskModule {
    * @param quoteValidUntil The expiration of the quote
    * @return Returns the id of the created policy
    */
-  function newPolicyPaidByHolder(
+  function replacePolicy(
+    Policy.PolicyData calldata oldPolicy,
     uint256 payout,
     uint256 premium,
     uint256 lossProb,
     uint40 expiration,
-    address onBehalfOf,
     bytes32 policyData,
     uint256 bucketId,
     bytes32 quoteSignatureR,
     bytes32 quoteSignatureVS,
     uint40 quoteValidUntil
-  ) external whenNotPaused returns (uint256) {
-    require(
-      onBehalfOf == _msgSender() || currency().allowance(onBehalfOf, _msgSender()) > 0,
-      "Sender is not authorized to create policies onBehalfOf"
-    );
-    /**
-     * The standard is the payer should be the _msgSender() but usually, in this type of module,
-     * the sender is an operative account managed by software, where the onBehalfOf is a more
-     * secure account (hardware wallet) that does the cash movements.
-     * This non standard behaviour allows for a more secure setup, where the sender never manages
-     * cash.
-     * We leverage the currency's allowance mechanism to allow the sender access to the payer's
-     * funds.
-     * Note that this allowance won't be spent, so anything above 0 is accepted.
-     */
+  ) external whenNotPaused onlyComponentRole(REPLACER_ROLE) returns (uint256) {
     _checkSignature(
       payout,
       premium,
@@ -292,7 +225,10 @@ contract SignedBucketRiskModule is RiskModule {
       quoteSignatureVS,
       quoteValidUntil
     );
-    return _newPolicySigned(payout, premium, lossProb, expiration, policyData, bucketId, onBehalfOf, onBehalfOf).id;
+    uint96 internalId = uint96(uint256(policyData) % 2 ** 96);
+    return
+      _replacePolicy(oldPolicy, payout, premium, lossProb, expiration, _msgSender(), internalId, bucketParams(bucketId))
+        .id;
   }
 
   function resolvePolicy(
@@ -316,7 +252,7 @@ contract SignedBucketRiskModule is RiskModule {
     uint256 bucketId,
     Params calldata params_
   ) external onlyGlobalOrComponentRole2(LEVEL1_ROLE, LEVEL2_ROLE) {
-    require(bucketId != 0, "SignedBucketRiskModule: bucketId can't be zero, set default RM parameters");
+    if (bucketId == 0) revert BucketCannotBeZero();
     _buckets[bucketId] = PackedParams({
       moc: _wadTo4(params_.moc),
       jrCollRatio: _wadTo4(params_.jrCollRatio),
@@ -343,7 +279,8 @@ contract SignedBucketRiskModule is RiskModule {
    * @param bucketId Group identifier for the policies that will have these parameters
    */
   function deleteBucket(uint256 bucketId) external onlyGlobalOrComponentRole2(LEVEL1_ROLE, LEVEL2_ROLE) {
-    require(bucketId != 0, "SignedBucketRiskModule: bucketId can't be zero, set default RM parameters");
+    if (bucketId == 0) revert BucketCannotBeZero();
+    if (_buckets[bucketId].moc == 0) revert BucketNotFound();
     delete _buckets[bucketId];
     emit BucketDeleted(bucketId);
   }
@@ -351,12 +288,12 @@ contract SignedBucketRiskModule is RiskModule {
   /**
    * @dev returns the risk bucket parameters for the given bucketId
    *
-   * @param bucketId Id of the bucket of 0 if you want the default params
+   * @param bucketId Id of the bucket or 0 if you want the default params
    */
   function bucketParams(uint256 bucketId) public view returns (Params memory params_) {
     if (bucketId != 0) {
       PackedParams storage bucketParams_ = _buckets[bucketId];
-      require(bucketParams_.moc != 0, "SignedBucketRiskModule: bucket not found!");
+      if (bucketParams_.moc == 0) revert BucketNotFound();
       params_ = _unpackParams(bucketParams_);
     } else {
       params_ = params();
@@ -377,7 +314,7 @@ contract SignedBucketRiskModule is RiskModule {
     uint40 expiration,
     uint256 bucketId
   ) public view virtual returns (uint256) {
-    return _getMinimumPremium(payout, lossProb, expiration, bucketParams(bucketId));
+    return _getMinimumPremium(payout, lossProb, expiration, uint40(block.timestamp), bucketParams(bucketId));
   }
 
   /**

package/contracts/SignedQuoteRiskModule.sol

@@ -30,6 +30,8 @@ contract SignedQuoteRiskModule is RiskModule {
    */
   event NewSignedPolicy(uint256 indexed policyId, bytes32 policyData);
 
+  error QuoteExpired();
+
   /// @custom:oz-upgrades-unsafe-allow constructor
   constructor(
     IPolicyPool policyPool_,
@@ -75,7 +77,7 @@ contract SignedQuoteRiskModule is RiskModule {
   ) internal returns (Policy.PolicyData memory createdPolicy) {
     if (!_creationIsOpen)
       _policyPool.access().checkComponentRole(address(this), POLICY_CREATOR_ROLE, _msgSender(), false);
-    require(quoteValidUntil >= block.timestamp, "Quote expired");
+    if (quoteValidUntil < block.timestamp) revert QuoteExpired();
 
     /**
      * Checks the quote has been signed by an authorized user

package/contracts/interfaces/IPolicyHolder.sol

@@ -16,7 +16,7 @@ interface IPolicyHolder is IERC721Receiver {
    * No mather what's the return value or even if this function reverts, this function will not revert the policy
    * expiration.
    *
-   * The selector can be obtained in Solidity with `IPolicyPool.onPolicyExpired.selector`.
+   * The selector can be obtained in Solidity with `IPolicyHolder.onPolicyExpired.selector`.
    */
   function onPolicyExpired(address operator, address from, uint256 policyId) external returns (bytes4);
 
@@ -27,7 +27,7 @@ interface IPolicyHolder is IERC721Receiver {
    * If interface is not implemented by the recipient, it will be ignored and the payout will be successful.
    * If any other value is returned or it reverts, the policy resolution / payout will be reverted.
    *
-   * The selector can be obtained in Solidity with `IPolicyPool.onPayoutReceived.selector`.
+   * The selector can be obtained in Solidity with `IPolicyHolder.onPayoutReceived.selector`.
    */
   function onPayoutReceived(address operator, address from, uint256 policyId, uint256 amount) external returns (bytes4);
 }

package/contracts/interfaces/IPolicyHolderV2.sol

@@ -0,0 +1,29 @@
+// SPDX-License-Identifier: Apache-2.0
+pragma solidity ^0.8.0;
+
+import {IPolicyHolder} from "./IPolicyHolder.sol";
+
+/**
+ * @title Policy holder interface - V2 of the interface that adds onPolicyReplaced endpoint
+ * @dev Interface for any contract that wants to be a holder of Ensuro Policies and receive notification of payouts and
+ *      replacements.
+ *
+ *      Implementors of this interface MUST return true on supportsInterface for both IPolicyHolder and IPolicyHolderV2.
+ */
+interface IPolicyHolderV2 is IPolicyHolder {
+  /**
+   * @dev Whenever a policy is replaced, this function is called
+   *
+   * It must return its Solidity selector to confirm the payout.
+   * If interface is not implemented by the recipient, it will be ignored and the replacement will be successful.
+   * If any other value is returned or it reverts, the policy replacement will be reverted.
+   *
+   * The selector can be obtained in Solidity with `IPolicyHolderV2.onPolicyReplaced.selector`.
+   */
+  function onPolicyReplaced(
+    address operator,
+    address from,
+    uint256 oldPolicyId,
+    uint256 newPolicyId
+  ) external returns (bytes4);
+}

package/contracts/mocks/PolicyHolderMock.sol

@@ -2,22 +2,29 @@
 pragma solidity ^0.8.0;
 
 import {IPolicyHolder} from "../interfaces/IPolicyHolder.sol";
+import {IPolicyHolderV2} from "../interfaces/IPolicyHolderV2.sol";
 import {IERC721Receiver} from "@openzeppelin/contracts/token/ERC721/IERC721Receiver.sol";
 import {IERC165} from "@openzeppelin/contracts/utils/introspection/IERC165.sol";
+import {StorageSlot} from "@openzeppelin/contracts/utils/StorageSlot.sol";
 
-contract PolicyHolderMock is IPolicyHolder {
+contract PolicyHolderMock is IPolicyHolderV2 {
   uint256 public policyId;
   uint256 public payout;
   bool public fail;
+  bool public failReplace;
   bool public emptyRevert;
   bool public notImplemented;
   bool public badlyImplemented;
+  bool public badlyImplementedReplace;
   bool public noERC165;
+  bool public noV2;
+  uint256 public spendGasCount;
 
   enum NotificationKind {
     PolicyReceived,
     PayoutReceived,
-    PolicyExpired
+    PolicyExpired,
+    PolicyReplaced
   }
 
   event NotificationReceived(NotificationKind kind, uint256 policyId, address operator, address from);
@@ -29,12 +36,21 @@ contract PolicyHolderMock is IPolicyHolder {
     emptyRevert = false;
     noERC165 = false;
     payout = type(uint256).max;
+    spendGasCount = 0;
   }
 
   function setFail(bool fail_) external {
     fail = fail_;
   }
 
+  function setFailReplace(bool failReplace_) external {
+    failReplace = failReplace_;
+  }
+
+  function setSpendGasCount(uint256 spendGasCount_) external {
+    spendGasCount = spendGasCount_;
+  }
+
   function setNotImplemented(bool notImplemented_) external {
     notImplemented = notImplemented_;
   }
@@ -43,6 +59,14 @@ contract PolicyHolderMock is IPolicyHolder {
     badlyImplemented = badlyImplemented_;
   }
 
+  function setBadlyImplementedReplace(bool badlyImplementedReplace_) external {
+    badlyImplementedReplace = badlyImplementedReplace_;
+  }
+
+  function setNoV2(bool noV2_) external {
+    noV2 = noV2_;
+  }
+
   function setNoERC165(bool noERC165_) external {
     noERC165 = noERC165_;
   }
@@ -51,6 +75,13 @@ contract PolicyHolderMock is IPolicyHolder {
     emptyRevert = emptyRevert_;
   }
 
+  function spendGas() internal {
+    // Spends gas doing storage writes
+    for (uint256 i = 0; i < spendGasCount; i++) {
+      StorageSlot.getUint256Slot(bytes32(100 + i)).value = i + 1;
+    }
+  }
+
   /**
    * @dev See {IERC165-supportsInterface}.
    */
@@ -60,7 +91,10 @@ contract PolicyHolderMock is IPolicyHolder {
         revert(0, 0)
       }
     if (notImplemented) return false;
-    return interfaceId == type(IPolicyHolder).interfaceId || interfaceId == type(IERC165).interfaceId;
+    return
+      interfaceId == type(IPolicyHolder).interfaceId ||
+      (!noV2 && interfaceId == type(IPolicyHolderV2).interfaceId) ||
+      interfaceId == type(IERC165).interfaceId;
   }
 
   function onERC721Received(
@@ -82,6 +116,8 @@ contract PolicyHolderMock is IPolicyHolder {
 
     if (badlyImplemented) return bytes4(0x0badf00d);
 
+    spendGas();
+
     return IERC721Receiver.onERC721Received.selector;
   }
 
@@ -98,17 +134,13 @@ contract PolicyHolderMock is IPolicyHolder {
 
     if (badlyImplemented) return bytes4(0x0badf00d);
 
+    spendGas();
+
     return IPolicyHolder.onPolicyExpired.selector;
   }
 
   /**
-   * @dev Whenever an Policy is resolved with payout > 0, this function is called
-   *
-   * It must return its Solidity selector to confirm the payout.
-   * If interface is not implemented by the recipient, it will be ignored and the payout will be successful.
-   * If any other value is returned or it reverts, the policy resolution / payout will be reverted.
-   *
-   * The selector can be obtained in Solidity with `IPolicyPool.onPayoutReceived.selector`.
+   * @dev See {IPolicyHolderV2-onPayoutReceived}.
    */
   function onPayoutReceived(
     address operator,
@@ -128,6 +160,35 @@ contract PolicyHolderMock is IPolicyHolder {
 
     if (badlyImplemented) return bytes4(0x0badf00d);
 
+    spendGas();
+
     return IPolicyHolder.onPayoutReceived.selector;
   }
+
+  /**
+   * @dev See {IPolicyHolderV2-onPolicyReplaced}.
+   */
+  function onPolicyReplaced(
+    address operator,
+    address from,
+    uint256 oldPolicyId,
+    uint256 newPolicyId
+  ) external override returns (bytes4) {
+    if (noV2) revert("Shouldn't call this method if V2 not enabled");
+    if (failReplace)
+      if (emptyRevert)
+        assembly {
+          revert(0, 0)
+        }
+      else revert("onPolicyReplaced: They told me I have to fail");
+    policyId = oldPolicyId;
+    payout = newPolicyId;
+    emit NotificationReceived(NotificationKind.PolicyReplaced, oldPolicyId, operator, from);
+
+    if (badlyImplementedReplace) return bytes4(0x0badf00d);
+
+    spendGas();
+
+    return IPolicyHolderV2.onPolicyReplaced.selector;
+  }
 }

package/js/deploy.js

@@ -1,4 +1,3 @@
-const upgrades_core = require("@openzeppelin/upgrades-core");
 const helpers = require("@nomicfoundation/hardhat-network-helpers");
 const fs = require("fs");
 const ethers = require("ethers");
@@ -75,16 +74,12 @@ async function logContractCreated(hre, contractName, address) {
 }
 
 async function verifyContract(hre, contract, isProxy, constructorArguments) {
-  if (isProxy === undefined) isProxy = false;
-  if (constructorArguments === undefined) constructorArguments = [];
-  const contractAddr = await ethers.resolveAddress(contract);
-  const address = isProxy
-    ? await upgrades_core.getImplementationAddress(hre.network.provider, contractAddr)
-    : contractAddr;
+  constructorArguments = constructorArguments || [];
+  const address = await ethers.resolveAddress(contract);
   try {
     await hre.run("verify:verify", {
-      address: address,
-      constructorArguments: constructorArguments,
+      address,
+      constructorArguments,
     });
   } catch (error) {
     console.log("Error verifying contract implementation: ", error);
@@ -95,18 +90,6 @@ async function verifyContract(hre, contract, isProxy, constructorArguments) {
         .join(" ")}`
     );
   }
-  try {
-    if (isProxy) {
-      await hre.run("verify:verify", {
-        address: contractAddr,
-        constructorArguments: constructorArguments,
-      });
-    }
-  } catch (error) {
-    console.log("Error verifying contract proxy: ", error);
-    const endpoints = await hre.run("verify:get-etherscan-endpoint");
-    console.log(`You should do it manually at ${endpoints.urls.browserURL}/proxyContractChecker?a=${contractAddr}`);
-  }
 }
 
 async function deployContract({ saveAddr, verify, contractClass, constructorArgs }, hre) {
@@ -114,8 +97,7 @@ async function deployContract({ saveAddr, verify, contractClass, constructorArgs
   const contract = await ContractFactory.deploy(...constructorArgs, txOverrides());
   const contractAddr = await ethers.resolveAddress(contract);
   if (verify) {
-    // From https://ethereum.stackexchange.com/a/119622/79726
-    await contract.deployTransaction.wait(6);
+    await contract.deploymentTransaction().wait(6);
   } else {
     await contract.waitForDeployment();
   }
@@ -138,8 +120,7 @@ async function deployProxyContract(
   });
   const contractAddr = await ethers.resolveAddress(contract);
   if (verify) {
-    // From https://ethereum.stackexchange.com/a/119622/79726
-    await contract.deployTransaction.wait(6);
+    await contract.deploymentTransaction().wait(6);
   } else {
     await contract.waitForDeployment();
   }

package/js/utils.js

@@ -144,22 +144,27 @@ async function grantRole(hre, contract, role, user, txOverrides = {}, log = () =
  * @param {Interface} interface The interface of the contract that contains the requested event
  * @param {TransactionReceipt} receipt Transaction receipt containing the events in the logs
  * @param {String} eventName The name of the event we are interested in
+ * @param {Boolean} firstOnly If false, returns all the events matching the name
+ * @param {String} contractAddress If not null, if returns only those events generated by the specified contract
  * @returns {LogDescription}
  */
-function getTransactionEvent(interface, receipt, eventName) {
+function getTransactionEvent(interface, receipt, eventName, firstOnly = true, contractAddress = null) {
+  const ret = [];
   // for each log in the transaction receipt
   for (const log of receipt.logs) {
     let parsedLog;
+    if (contractAddress !== null && contractAddress !== log.address) continue;
     try {
       parsedLog = interface.parseLog(log);
     } catch (error) {
       continue;
     }
     if (parsedLog?.name == eventName) {
-      return parsedLog;
+      if (firstOnly) return parsedLog;
+      ret.push(parsedLog);
     }
   }
-  return null; // not found
+  return firstOnly ? null : ret;
 }
 
 /**
@@ -246,7 +251,7 @@ async function defaultPolicyParams(
   _A = amountFunction(6)
 ) {
   const now = await helpers.time.latest();
-  const rmAddress = rm ? await ethers.resolveAddress(rm) : rm;
+  const rmAddress = rm ? getAddress(rm) : null;
   return {
     rmAddress,
     payout: payout || _A(1000),

package/package.json

@@ -1,14 +1,12 @@
 {
   "name": "@ensuro/core",
   "description": "Ensuro - Decentralized insurance protocol",
-  "version": "2.8.0",
+  "version": "2.8.2",
   "files": [
     "**/*.sol",
-    "/build/contracts/*.json",
-    "/build/interfaces/*.json",
+    "/build",
     "/js/*.js",
-    "/scripts/*.sh",
-    "/scripts/*.js"
+    "/scripts"
   ],
   "repository": {
     "type": "git",