lens-modules 3.1.2 → 3.1.3

package/contracts/actions/account/base/BaseAccountAction.sol

@@ -12,6 +12,7 @@ abstract contract BaseAccountAction is BaseAction, IAccountAction {
 
     function configure(address originalMsgSender, address account, KeyValue[] calldata params)
         external
+        payable
         override
         onlyActionHub
         returns (bytes memory)
@@ -21,6 +22,7 @@ abstract contract BaseAccountAction is BaseAction, IAccountAction {
 
     function execute(address originalMsgSender, address account, KeyValue[] calldata params)
         external
+        payable
         override
         onlyActionHub
         returns (bytes memory)
@@ -30,6 +32,7 @@ abstract contract BaseAccountAction is BaseAction, IAccountAction {
 
     function setDisabled(address originalMsgSender, address account, bool isDisabled, KeyValue[] calldata params)
         external
+        payable
         override
         onlyActionHub
         returns (bytes memory)

package/contracts/actions/post/base/BasePostAction.sol

@@ -12,6 +12,7 @@ abstract contract BasePostAction is BaseAction, IPostAction {
 
     function configure(address originalMsgSender, address feed, uint256 postId, KeyValue[] calldata params)
         external
+        payable
         override
         onlyActionHub
         returns (bytes memory)
@@ -21,6 +22,7 @@ abstract contract BasePostAction is BaseAction, IPostAction {
 
     function execute(address originalMsgSender, address feed, uint256 postId, KeyValue[] calldata params)
         external
+        payable
         override
         onlyActionHub
         returns (bytes memory)
@@ -34,7 +36,7 @@ abstract contract BasePostAction is BaseAction, IPostAction {
         uint256 postId,
         bool isDisabled,
         KeyValue[] calldata params
-    ) external override onlyActionHub returns (bytes memory) {
+    ) external payable override onlyActionHub returns (bytes memory) {
         return _setDisabled(originalMsgSender, feed, postId, isDisabled, params);
     }
 

package/contracts/actions/post/collect/LensCollectedPost.sol

@@ -3,11 +3,15 @@
 pragma solidity ^0.8.26;
 
 import "lens-modules/contracts/core/base/LensERC721.sol";
-import {IERC7572} from "lens-modules/contracts/actions/post/collect/IERC7572.sol";
 import {IFeed} from "lens-modules/contracts/core/interfaces/IFeed.sol";
 import {ITokenURIProvider} from "lens-modules/contracts/core/interfaces/ITokenURIProvider.sol";
 import {Errors} from "lens-modules/contracts/core/types/Errors.sol";
 
+struct ContentURISnapshot {
+    string contentURI;
+    uint256 tokenId;
+}
+
 /**
  * @notice A contract that represents a Lens Collected Post.
  *
@@ -19,54 +23,93 @@ import {Errors} from "lens-modules/contracts/core/types/Errors.sol";
  * If the Collect is immutable - it will snapshot the content of the post and always return the snapshotted tokenURI
  * even if the post was updated or deleted. The contractURI, however, always stays the same, as it was at the moment of
  * Collect creation.
+ *
+ * We assume tokenIds are sequential and start from 1.
  */
-contract LensCollectedPost is LensERC721, IERC7572 {
+contract LensCollectedPost is LensERC721 {
     event Lens_LensCollectedPost_Transfer(address indexed from, address indexed to, uint256 indexed tokenId);
 
-    string internal _contentURISnapshot;
-    string internal _contractURI;
-    address internal immutable _feed;
-    uint256 internal immutable _postId;
-    address internal immutable _collectAction;
+    ContentURISnapshot[] internal _contentURISnapshots;
+    bool internal _isImmutable;
+    address internal immutable FEED;
+    uint256 internal immutable POST_ID;
+    address internal immutable COLLECT_ACTION;
 
-    constructor(address feed, uint256 postId, bool isImmutable) {
+    constructor(address feed, uint256 postId, bool isImmutableCollect) {
         LensERC721._initialize("Lens Collected Post", "LCP", ITokenURIProvider(address(0)));
-        string memory contentURI = IFeed(feed).getPost(postId).contentURI;
-        require(bytes(contentURI).length > 0, Errors.InvalidParameter());
-        _feed = feed;
-        _postId = postId;
-        _contractURI = contentURI;
-        _collectAction = msg.sender;
-        if (isImmutable) {
-            _contentURISnapshot = contentURI;
+        COLLECT_ACTION = msg.sender;
+        FEED = feed;
+        POST_ID = postId;
+        // Getting the URI outside the if to use it as a length validation too.
+        string memory contentURI = _getNonEmptyContentURIFromPost();
+        if (isImmutableCollect) {
+            _turnImmutable(contentURI);
         }
-        emit ContractURIUpdated();
     }
 
     function mint(address to, uint256 tokenId) external {
-        require(msg.sender == _collectAction, Errors.InvalidMsgSender());
+        require(msg.sender == COLLECT_ACTION, Errors.InvalidMsgSender());
+        _takeContentURISnapshotIfNeeded(tokenId);
         _mint(to, tokenId);
     }
 
-    // Getters
-
-    function contractURI() external view returns (string memory) {
-        return _contractURI;
+    function turnImmutable() external {
+        require(msg.sender == COLLECT_ACTION, Errors.InvalidMsgSender());
+        if (!_isImmutable) {
+            _turnImmutable(_getNonEmptyContentURIFromPost());
+        }
     }
 
-    function tokenURI(uint256 /*tokenId*/ ) public view override returns (string memory) {
-        if (bytes(_contentURISnapshot).length > 0) {
-            return _contentURISnapshot;
+    // Getters
+
+    function tokenURI(uint256 tokenId) public view override returns (string memory) {
+        if (_isImmutable) {
+            // Searching for snapshots (starting from latest)
+            for (uint256 i = _contentURISnapshots.length - 1; i > 0; i--) {
+                if (_contentURISnapshots[i].tokenId <= tokenId) {
+                    return _contentURISnapshots[i].contentURI;
+                }
+            }
+            // contentURISnapshot[0] is always taken when the collection is marked as immutable,
+            // so we can guarantee it's present and non-empty.
+            return _contentURISnapshots[0].contentURI;
         } else {
-            string memory contentURI = IFeed(_feed).getPost(_postId).contentURI;
-            // If content was deleted we fail. You can override this to return the empty URI if preferred.
-            require(bytes(contentURI).length > 0, Errors.DoesNotExist());
-            return contentURI;
+            return _getNonEmptyContentURIFromPost();
         }
     }
 
+    function getPostId() external view returns (uint256) {
+        return POST_ID;
+    }
+
+    function isImmutable() external view returns (bool) {
+        return _isImmutable;
+    }
+
     // Internal
 
+    function _getNonEmptyContentURIFromPost() internal view returns (string memory) {
+        string memory contentURI = IFeed(FEED).getPost(POST_ID).contentURI;
+        require(bytes(contentURI).length > 0, Errors.InvalidParameter());
+        return contentURI;
+    }
+
+    function _turnImmutable(string memory contentURI) internal {
+        _isImmutable = true;
+        _contentURISnapshots.push(ContentURISnapshot(contentURI, 0));
+    }
+
+    function _takeContentURISnapshotIfNeeded(uint256 tokenId) internal {
+        if (_isImmutable) {
+            // Getting the content URI will revert if the post was deleted or does not exist.
+            string memory contentURI = _getNonEmptyContentURIFromPost();
+            string memory latestContentURISnapshot = _contentURISnapshots[_contentURISnapshots.length - 1].contentURI;
+            if (keccak256(bytes(contentURI)) != keccak256(bytes(latestContentURISnapshot))) {
+                _contentURISnapshots.push(ContentURISnapshot(contentURI, tokenId));
+            }
+        }
+    }
+
     function _afterTokenTransfer(address from, address to, uint256 tokenId) internal virtual override {
         emit Lens_LensCollectedPost_Transfer(from, to, tokenId);
     }

package/contracts/actions/post/collect/SimpleCollectAction.sol

@@ -3,7 +3,7 @@
 pragma solidity ^0.8.26;
 
 import {ISimpleCollectAction, CollectActionData} from "lens-modules/contracts/actions/post/collect/ISimpleCollectAction.sol";
-import {IFeed} from "lens-modules/contracts/core/interfaces/IFeed.sol";
+import {IFeed, Post} from "lens-modules/contracts/core/interfaces/IFeed.sol";
 import {IGraph} from "lens-modules/contracts/core/interfaces/IGraph.sol";
 import {LensCollectedPost} from "lens-modules/contracts/actions/post/collect/LensCollectedPost.sol";
 import {OwnableMetadataBasedPostAction} from "lens-modules/contracts/actions/post/base/OwnableMetadataBasedPostAction.sol";
@@ -69,11 +69,15 @@ contract SimpleCollectAction is
      * @param recipients Recipient(s) of collect fees.
      * @param referralFeeBps The fee percentage that is distributed to referrals.
      * @param isImmutable If true, it means that:
-     *          - The Post URI is snapshotted at configuration time and cannot be changed later.
+     *          - The Post URI is snapshotted at collect time for each collected NFT.
      *          - Collected posts' NFTs remain permanently available.
-     *          - What you see is what you get; editing the Post URI or deleting the post will disable collection.
+     *          - What you see is what you get (*).
+     *          - Deleting the post will disable further collection.
      *         Note: This immutability is only guaranteed if the URI is hosted on immutable storage. Mutability inherent
      *         to the chosen storage technology exceeds the on-chain verification capabilities.
+     *         (*) WYSIWYG is not preserved for Legacy LensCollectedPost collections. Those will have a snapshot of the
+     *           post at configuration time and, if the post was edited, the NFT would still have same snapshot.
+     *           This decision was made to prevent legacy collections from breaking on editing.
      */
     struct CollectActionConfigureParams {
         uint160 amount; ///////////// (Optional) Default: 0
@@ -140,13 +144,33 @@ contract SimpleCollectAction is
                 storedData.referralFeeBps = configData.referralFeeBps;
                 storedData.followerOnlyGraph = configData.followerOnlyGraph;
                 storedData.endTimestamp = configData.endTimestamp;
-                // Immutability cannot be flipped to true.
-                require(configData.isImmutable == false, Errors.InvalidParameter());
+                if (storedData.currentCollects == 0) {
+                    // Re-deploy collection as a fix for some broken mutable (i.e. isImmutable = false) collections.
+                    storedData.collectionAddress = address(new LensCollectedPost(feed, postId, configData.isImmutable));
+                } else if (configData.isImmutable == true) {
+                    // Tries to turn existing collection immutable, which is not supported for older collections.
+                    _tryTurnImmutable(feed, postId, storedData.collectionAddress);
+                    storedData.isImmutable = true;
+                }
             }
         }
         return abi.encode(storedData);
     }
 
+    function _tryTurnImmutable(address feed, uint256 postId, address collectionAddress) internal {
+        Post memory post = IFeed(feed).getPostUnchecked(postId);
+        if (post.isDeleted || bytes(post.contentURI).length == 0) {
+            // Call will fail anyways when the collection tries to take the content URI snapshot #0.
+            revert Errors.UnexpectedValue();
+        } else {
+            (bool callSucceeded,) = collectionAddress.call(abi.encodeCall(LensCollectedPost.turnImmutable, ()));
+            if (!callSucceeded) {
+                // Collection is from an older version that does not support turning immutable.
+                revert Errors.UnsupportedOperation();
+            }
+        }
+    }
+
     function _execute(address originalMsgSender, address feed, uint256 postId, KeyValue[] calldata params)
         internal
         override
@@ -203,8 +227,7 @@ contract SimpleCollectAction is
             require(configData.recipients.length == 0, Errors.InvalidParameter());
             require(configData.referralFeeBps == 0, Errors.InvalidParameter());
         } else {
-            // We expect token to support ERC-20 interface (call balanceOf and expect it to not revert)
-            IERC20(configData.token).balanceOf(address(this));
+            _validateToken(configData.token);
             require(configData.recipients.length > 0, Errors.InvalidParameter());
             require(configData.referralFeeBps <= BPS_MAX, Errors.InvalidParameter());
         }
@@ -310,16 +333,6 @@ contract SimpleCollectAction is
             );
         }
 
-        if (data.isImmutable) {
-            // If post is edited to a different content, we fail so people do not collect an unexpected thing.
-            string memory contentURI = IFeed(feed).getPost(postId).contentURI;
-            require(
-                keccak256(bytes(contentURI))
-                    == keccak256(bytes(LensCollectedPost(data.collectionAddress).tokenURI(data.currentCollects))),
-                Errors.InvalidParameter()
-            );
-        }
-
         if (data.isDisabled) {
             revert Errors.Disabled();
         }

package/contracts/core/base/BaseSource.sol

@@ -42,7 +42,6 @@ abstract contract BaseSource is ISource {
         emit Lens_Source_NonceUsed(nonce);
     }
 
-    // Signature Standard: EIP-191 - Version Byte: 0x00
     function _validateSource(SourceStamp calldata sourceStamp) internal virtual {
         require(!$baseSourceStorage().wasSourceStampNonceUsed[sourceStamp.nonce], Errors.NonceUsed());
         require(sourceStamp.deadline >= block.timestamp, Errors.Expired());

package/contracts/core/base/LensERC721.sol

@@ -16,6 +16,7 @@ abstract contract LensERC721 is IERC721Metadata, ERC165 {
     using AddressUpgradeable for address;
 
     event Lens_ERC721_TokenURIProviderSet(address indexed tokenURIProvider);
+    event Lens_ERC721_Initialized(string name, string symbol);
 
     struct ERC721Storage {
         // Token name
@@ -47,6 +48,7 @@ abstract contract LensERC721 is IERC721Metadata, ERC165 {
         $erc721Storage().name = nftName;
         $erc721Storage().symbol = nftSymbol;
         $erc721Storage().tokenURIProvider = tokenURIProvider;
+        emit Lens_ERC721_Initialized(nftName, nftSymbol);
         emit Lens_ERC721_TokenURIProviderSet(address(tokenURIProvider));
     }
 

package/contracts/core/base/RuleBasedPrimitive.sol

@@ -13,8 +13,9 @@ import {
 } from "lens-modules/contracts/core/types/Types.sol";
 import {CallLib} from "lens-modules/contracts/core/libraries/CallLib.sol";
 import {Errors} from "lens-modules/contracts/core/types/Errors.sol";
+import {PayableUsingNativePaymentHelper} from "lens-modules/contracts/extensions/fees/LensNativePaymentHelper.sol";
 
-abstract contract RuleBasedPrimitive {
+abstract contract RuleBasedPrimitive is PayableUsingNativePaymentHelper {
     using RulesLib for RulesStorage;
     using CallLib for address;
 

package/contracts/core/base/SourceStampBased.sol

@@ -4,6 +4,7 @@ pragma solidity ^0.8.26;
 import {KeyValue, SourceStamp} from "lens-modules/contracts/core/types/Types.sol";
 import {ExtraStorageBased} from "lens-modules/contracts/core/base/ExtraStorageBased.sol";
 import {ISource} from "lens-modules/contracts/core/interfaces/ISource.sol";
+import {Errors} from "lens-modules/contracts/core/types/Errors.sol";
 
 abstract contract SourceStampBased is ExtraStorageBased {
     /// @custom:keccak lens.param.sourceStamp
@@ -30,7 +31,7 @@ abstract contract SourceStampBased is ExtraStorageBased {
         for (uint256 i = 0; i < customParams.length; i++) {
             if (customParams[i].key == PARAM__SOURCE_STAMP) {
                 SourceStamp memory sourceStamp = abi.decode(customParams[i].value, (SourceStamp));
-                require(sourceStamp.originalMsgSender == msg.sender);
+                require(sourceStamp.originalMsgSender == msg.sender, Errors.InvalidSourceStampOriginalMsgSender());
                 ISource(sourceStamp.source).validateSource(sourceStamp);
                 return sourceStamp.source;
             }

package/contracts/core/interfaces/IFeed.sol

@@ -121,7 +121,7 @@ interface IFeed is IMetadataBased {
 
     function initialize(string memory metadataURI, IAccessControl accessControl) external;
 
-    function changeFeedRules(RuleChange[] calldata ruleChanges) external;
+    function changeFeedRules(RuleChange[] calldata ruleChanges) external payable;
 
     function createPost(
         CreatePostParams calldata postParams,
@@ -129,7 +129,7 @@ interface IFeed is IMetadataBased {
         RuleProcessingParams[] calldata feedRulesParams,
         RuleProcessingParams[] calldata rootPostRulesParams,
         RuleProcessingParams[] calldata quotedPostRulesParams
-    ) external returns (uint256);
+    ) external payable returns (uint256);
 
     function editPost(
         uint256 postId,
@@ -138,19 +138,19 @@ interface IFeed is IMetadataBased {
         RuleProcessingParams[] calldata feedRulesParams,
         RuleProcessingParams[] calldata rootPostRulesParams,
         RuleProcessingParams[] calldata quotedPostRulesParams
-    ) external;
+    ) external payable;
 
     function deletePost(
         uint256 postId,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata feedRulesParams
-    ) external;
+    ) external payable;
 
     function changePostRules(
         uint256 postId,
         RuleChange[] calldata ruleChanges,
         RuleProcessingParams[] calldata feedRulesParams
-    ) external;
+    ) external payable;
 
     function setExtraData(KeyValue[] calldata extraDataToSet) external;
 

package/contracts/core/interfaces/IGraph.sol

@@ -68,13 +68,13 @@ interface IGraph is IMetadataBased {
 
     function initialize(string memory metadataURI, IAccessControl accessControl) external;
 
-    function changeGraphRules(RuleChange[] calldata ruleChanges) external;
+    function changeGraphRules(RuleChange[] calldata ruleChanges) external payable;
 
     function changeFollowRules(
         address account,
         RuleChange[] calldata ruleChanges,
         RuleProcessingParams[] calldata graphRulesProcessingParams
-    ) external;
+    ) external payable;
 
     function follow(
         address followerAccount,
@@ -83,14 +83,14 @@ interface IGraph is IMetadataBased {
         RuleProcessingParams[] calldata graphRulesProcessingParams,
         RuleProcessingParams[] calldata followRulesProcessingParams,
         KeyValue[] calldata extraData
-    ) external returns (uint256);
+    ) external payable returns (uint256);
 
     function unfollow(
         address followerAccount,
         address accountToUnfollow,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata graphRulesProcessingParams
-    ) external returns (uint256);
+    ) external payable returns (uint256);
 
     function setExtraData(KeyValue[] calldata extraDataToSet) external;
 

package/contracts/core/interfaces/IGroup.sol

@@ -70,27 +70,27 @@ interface IGroup is IMetadataBased {
         address account,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata ruleProcessingParams
-    ) external;
+    ) external payable;
 
     function removeMember(
         address account,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata ruleProcessingParams
-    ) external;
+    ) external payable;
 
     function joinGroup(
         address account,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata ruleProcessingParams
-    ) external;
+    ) external payable;
 
     function leaveGroup(
         address account,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata ruleProcessingParams
-    ) external;
+    ) external payable;
 
-    function changeGroupRules(RuleChange[] calldata ruleChanges) external;
+    function changeGroupRules(RuleChange[] calldata ruleChanges) external payable;
 
     function setExtraData(KeyValue[] calldata extraDataToSet) external;
 

package/contracts/core/interfaces/INamespace.sol

@@ -85,14 +85,14 @@ interface INamespace is IMetadataBased {
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata ruleProcessingParams,
         KeyValue[] calldata extraData
-    ) external;
+    ) external payable;
 
     function removeUsername(
         string calldata username,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata unassigningRuleProcessingParams,
         RuleProcessingParams[] calldata removalRuleProcessingParams
-    ) external;
+    ) external payable;
 
     function assignUsername(
         address account,
@@ -101,13 +101,13 @@ interface INamespace is IMetadataBased {
         RuleProcessingParams[] calldata unassignAccountRuleProcessingParams,
         RuleProcessingParams[] calldata unassignUsernameRuleProcessingParams,
         RuleProcessingParams[] calldata assignRuleProcessingParams
-    ) external;
+    ) external payable;
 
     function unassignUsername(
         string calldata username,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata ruleProcessingParams
-    ) external;
+    ) external payable;
 
     function setUsernameExtraData(string memory username, KeyValue[] calldata extraDataToSet) external;
 

package/contracts/core/primitives/feed/Feed.sol

@@ -123,7 +123,7 @@ contract Feed is
         RuleProcessingParams[] memory feedRulesParams,
         RuleProcessingParams[] memory rootPostRulesParams,
         RuleProcessingParams[] memory quotedPostRulesParams
-    ) external virtual override returns (uint256) {
+    ) external payable virtual override usingNativePaymentHelper returns (uint256) {
         require(msg.sender == postParams.author, Errors.InvalidMsgSender());
         (uint256 postId, uint256 localSequentialId, uint256 rootPostId) = Core._createPost(postParams);
         _validateExpectedPostIdIfPresent(customParams, postId);
@@ -172,7 +172,7 @@ contract Feed is
         RuleProcessingParams[] memory feedRulesParams,
         RuleProcessingParams[] memory rootPostRulesParams,
         RuleProcessingParams[] memory quotedPostRulesParams
-    ) external virtual override {
+    ) external payable virtual override usingNativePaymentHelper {
         require(Core._postExists(postId), Errors.DoesNotExist());
         address author = Core.$storage().posts[postId].author;
         // You can have this if you want to allow moderator editing:
@@ -204,7 +204,7 @@ contract Feed is
         uint256 postId,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata feedRulesParams
-    ) external virtual override {
+    ) external payable virtual override usingNativePaymentHelper {
         require(Core._postExists(postId), Errors.DoesNotExist());
         address author = Core.$storage().posts[postId].author;
         require(msg.sender == author || _hasAccess(msg.sender, PID__REMOVE_POST), Errors.InvalidMsgSender());

package/contracts/core/primitives/feed/RuleBasedFeed.sol

@@ -40,7 +40,13 @@ abstract contract RuleBasedFeed is IFeed, RuleBasedPrimitive {
 
     ////////////////////////////  CONFIGURATION FUNCTIONS  ////////////////////////////
 
-    function changeFeedRules(RuleChange[] calldata ruleChanges) external virtual override {
+    function changeFeedRules(RuleChange[] calldata ruleChanges)
+        external
+        payable
+        virtual
+        override
+        usingNativePaymentHelper
+    {
         _changePrimitiveRules($feedRulesStorage(), ruleChanges);
     }
 
@@ -48,7 +54,7 @@ abstract contract RuleBasedFeed is IFeed, RuleBasedPrimitive {
         uint256 postId,
         RuleChange[] calldata ruleChanges,
         RuleProcessingParams[] calldata feedRulesParams
-    ) external virtual override {
+    ) external payable virtual override usingNativePaymentHelper {
         _changeEntityRules($postRulesStorage(postId), postId, ruleChanges, feedRulesParams);
     }
 

package/contracts/core/primitives/graph/Graph.sol

@@ -101,7 +101,7 @@ contract Graph is
         RuleProcessingParams[] calldata graphRulesProcessingParams,
         RuleProcessingParams[] calldata followRulesProcessingParams,
         KeyValue[] calldata extraData
-    ) external virtual override returns (uint256) {
+    ) external payable virtual override usingNativePaymentHelper returns (uint256) {
         require(msg.sender == followerAccount, Errors.InvalidMsgSender());
         // If some implementation wants to allow followId specification, it can be implemented using customParams.
         uint256 assignedFollowId = Core._follow(followerAccount, accountToFollow, 0, block.timestamp);
@@ -126,7 +126,7 @@ contract Graph is
         address accountToUnfollow,
         KeyValue[] calldata customParams,
         RuleProcessingParams[] calldata graphRulesProcessingParams
-    ) external virtual override returns (uint256) {
+    ) external payable virtual override usingNativePaymentHelper returns (uint256) {
         require(msg.sender == followerAccount, Errors.InvalidMsgSender());
         uint256 followId = Core._unfollow(followerAccount, accountToUnfollow);
         address source = _processSourceStamp(customParams);

package/contracts/core/primitives/graph/RuleBasedGraph.sol

@@ -39,7 +39,13 @@ abstract contract RuleBasedGraph is IGraph, RuleBasedPrimitive {
 
     ////////////////////////////  CONFIGURATION FUNCTIONS  ////////////////////////////
 
-    function changeGraphRules(RuleChange[] calldata ruleChanges) external virtual override {
+    function changeGraphRules(RuleChange[] calldata ruleChanges)
+        external
+        payable
+        virtual
+        override
+        usingNativePaymentHelper
+    {
         _changePrimitiveRules($graphRulesStorage(), ruleChanges);
     }
 
@@ -47,7 +53,7 @@ abstract contract RuleBasedGraph is IGraph, RuleBasedPrimitive {
         address account,
         RuleChange[] calldata ruleChanges,
         RuleProcessingParams[] calldata ruleChangesProcessingParams
-    ) external virtual override {
+    ) external payable virtual override usingNativePaymentHelper {
         _changeEntityRules(
             $followRulesStorage(account), uint256(uint160(account)), ruleChanges, ruleChangesProcessingParams
         );

package/contracts/core/primitives/group/RuleBasedGroup.sol

@@ -33,7 +33,13 @@ abstract contract RuleBasedGroup is IGroup, RuleBasedPrimitive {
 
     ////////////////////////////  CONFIGURATION FUNCTIONS  ////////////////////////////
 
-    function changeGroupRules(RuleChange[] calldata ruleChanges) external virtual override {
+    function changeGroupRules(RuleChange[] calldata ruleChanges)
+        external
+        payable
+        virtual
+        override
+        usingNativePaymentHelper
+    {
         _changePrimitiveRules($groupRulesStorage(), ruleChanges);
     }