refactor!: Change Operator event names in LSP7 and LSP8

[YamenMerhi]

What does this PR introduce?

⚠️ BREAKING CHANGES

This PR has an impact on the Event signatures of some LSP7 and LSP8 events, please check the latest event signatures in contract.ts and the respective solidity interface.

♻️ Refactor

LSP7

• Change AuthorizedOperator to OperatorAuthorizationChanged

// OLD
    event AuthorizedOperator(
        address indexed operator,
        address indexed tokenOwner,
        uint256 indexed amount,
        bytes operatorNotificationData
    );

// NEW
    event OperatorAuthorizationChanged(
        address indexed operator,
        address indexed tokenOwner,
        uint256 indexed amount,
        bytes operatorNotificationData
    );

• Change RevokedOperator to OperatorRevoked
• Index the notified boolean

// OLD
    event RevokedOperator(
        address indexed operator,
        address indexed tokenOwner,
        bool notified,
        bytes operatorNotificationData
    );

// NEW
    event OperatorRevoked(
        address indexed operator,
        address indexed tokenOwner,
        bool indexed notified,
        bytes operatorNotificationData
    );

---

LSP8

• Change AuthorizedOperator to OperatorAuthorizationChanged

// OLD
    event AuthorizedOperator(
        address indexed operator,
        address indexed tokenOwner,
        bytes32 indexed tokenId,
        bytes operatorNotificationData
    );

// NEW
    event OperatorAuthorizationChanged(
        address indexed operator,
        address indexed tokenOwner,
        bytes32 indexed tokenId
        bytes operatorNotificationData
    );

• Change RevokedOperator to OperatorRevoked

// OLD
    event RevokedOperator(
        address indexed operator,
        address indexed tokenOwner,
        bytes32 indexed tokenId,
        bool notified,
        bytes operatorNotificationData
    );

// NEW
    event OperatorRevoked(
        address indexed operator,
        address indexed tokenOwner,
        bytes32 indexed tokenId,
        bool notified,
        bytes operatorNotificationData
    );

PR Checklist

• Wrote Tests
• Wrote & Generated Documentation (readme/natspec/dodoc)
• Ran npm run lint && npm run lint:solidity (solhint)
• Ran npm run format (prettier)
• Ran npm run build
• Ran npm run test

[github-actions]

👋 Hello
⛽ I am the Gas Bot Reporter. I keep track of the gas costs of common interactions using Universal Profiles 🆙 !
📊 Here is a summary of the gas cost with the code introduced by this PR.

⛽📊 Gas Benchmark Report

Deployment Costs

Deployed contracts	⛽ Deployment cost
UniversalProfile	3185468 (0 )
KeyManager	3659015 (0 )
LSP1DelegateUP	1637510 (24 📈❌)
LSP7Mintable	2316901 (-5,624 📉✅)
LSP8Mintable	2424126 (192 📈❌)

Runtime Costs

UniversalProfile owned by an 🔑 EOA

🔀 execute scenarios

execute scenarios - UP owned by 🔑 EOA	⛽ Gas Usage
Transfer 1 LYX to an EOA without data	37560 (0 )
Transfer 1 LYX to a UP without data	46253 (0 )
Transfer 1 LYX to an EOA with 256 bytes of data	42233 (24 📈❌)
Transfer 1 LYX to a UP with 256 bytes of data	57174 (-12 📉✅)
Transfer 0.1 LYX to 3x EOA without data	70862 (0 )
Transfer 0.1 LYX to 3x UP without data	104453 (0 )
Transfer 0.1 LYX to 3x EOA with 256 bytes of data	84826 (12 📈❌)
Transfer 0.1 LYX to 3x UPs with 256 bytes of data	137209 (36 📈❌)

🗄️ setData scenarios

setData scenarios - UP owned by 🔑 EOA	⛽ Gas Usage
Set a 20 bytes long value	49951 (0 )
Set a 60 bytes long value	95273 (0 )
Set a 160 bytes long value	164433 (0 )
Set a 300 bytes long value	279668 (-12 📉✅)
Set a 600 bytes long value	484128 (12 📈❌)
Change the value of a data key already set	32839 (0 )
Remove the value of a data key already set	27313 (0 )
Set 2 data keys of 20 bytes long value	78488 (-12 📉✅)
Set 2 data keys of 100 bytes long value	260664 (12 📈❌)
Set 3 data keys of 20 bytes long value	105218 (0 )
Change the value of three data keys already set of 20 bytes long value	45518 (0 )
Remove the value of three data keys already set	41397 (0 )

🗄️ Tokens scenarios

Tokens scenarios - UP owned by 🔑 EOA	⛽ Gas Usage
Minting a LSP7Token to a UP (No Delegate) from an EOA	93092 (0 )
Minting a LSP7Token to an EOA from an EOA	59368 (0 )
Transferring an LSP7Token from a UP to another UP (No Delegate)	102378 (0 )
Minting a LSP8Token to a UP (No Delegate) from an EOA	159983 (0 )
Minting a LSP8Token to an EOA from an EOA	126260 (0 )
Transferring an LSP8Token from a UP to another UP (No Delegate)	151071 (0 )

UniversalProfile owned by a 🔒📄 LSP6KeyManager

🔀 execute scenarios

execute scenarios	👑 main controller	🛃 restricted controller
LYX transfer --> to an EOA	64356 (0 )	75306 (0 )
LYX transfer --> to a UP	78489 (0 )	93386 (0 )
LSP7 token transfer --> to an EOA	116849 (0 )	131599 (0 )
LSP7 token transfer --> to a UP	250551 (0 )	265301 (0 )
LSP8 NFT transfer --> to an EOA	180957 (0 )	195684 (0 )
LSP8 NFT transfer --> to a UP	297883 (0 )	312610 (0 )

🗄️ setData scenarios

setData scenarios	👑 main controller	🛃 restricted controller
Update Profile details (LSP3Profile Metadata)	67294 (0 )	77316 (0 )
Add a new controller with permission to SET_DATA + 3x allowed data keys: AddressPermissions[] + AddressPermissions[index] + AddressPermissions:Permissions:<controller> + AddressPermissions:AllowedERC725YDataKeys:<controller)	209636 (0 )	219793 (0 )
Update permissions of previous controller. Allow it now to SUPER_SETDATA	52322 (0 )	55328 (0 )
Remove a controller: 1. decrease AddressPermissions[] Array length 2. remove the controller address at AddressPermissions[index] 3. set "0x" for the controller permissions under AddressPermissions:Permissions:	78861 (0 )	90160 (0 )
Write 5x new LSP12 Issued Assets	66989 (0 )	101616 (0 )
Update 3x data keys (first 3)	125501 (0 )	159605 (0 )
Update 3x data keys (middle 3)	105589 (0 )	143759 (0 )
Update 3x data keys (last 3)	125501 (0 )	169092 (0 )
Set 2 x new data keys + add 3x new controllers	810481 (0 )	872281 (0 )

[github-actions]

Changes to gas cost

Generated at commit: 268833f5f3b303865cd155f65e2ef6392d2eeee2, compared to commit: 6594f678c362bf26882023aa1a108bfd8188924d

🧾 Summary (10% most significant diffs)

Contract	Method	Avg (+/-)	%
LSP6ExecuteUnrestrictedController	transferTokensToRandomEOA	+397 ❌	+0.54%
LSP6ExecuteRestrictedController	transferTokensToRandomEOA	+397 ❌	+0.53%

---

Full diff report 👇

Contract	Deployment Cost (+/-)	Method	Min (+/-)	%	Avg (+/-)	%	Median (+/-)	%	Max (+/-)	%	# Calls (+/-)
LSP6ExecuteUnrestrictedController	3,047,114 (0)	transferNFTToRandomEOA transferNFTToRandomUP transferTokensToRandomEOA transferTokensToRandomUP	142,150 (+396) 248,150 (+774) 74,150 (+397) 205,094 (+968)	+0.28% +0.31% +0.54% +0.47%	142,150 (+396) 248,150 (+774) 74,150 (+397) 205,094 (+968)	+0.28% +0.31% +0.54% +0.47%	142,150 (+396) 248,150 (+774) 74,150 (+397) 205,094 (+968)	+0.28% +0.31% +0.54% +0.47%	142,150 (+396) 248,150 (+774) 74,150 (+397) 205,094 (+968)	+0.28% +0.31% +0.54% +0.47%	1 (0) 1 (0) 1 (0) 1 (0)
LSP6ExecuteRestrictedController	3,047,114 (0)	transferNFTToRandomEOA transferNFTToRandomUP transferTokensToRandomEOA transferTokensToRandomUP	143,406 (+396) 249,406 (+774) 75,406 (+397) 206,664 (+968)	+0.28% +0.31% +0.53% +0.47%	143,406 (+396) 249,406 (+774) 75,406 (+397) 206,664 (+968)	+0.28% +0.31% +0.53% +0.47%	143,406 (+396) 249,406 (+774) 75,406 (+397) 206,664 (+968)	+0.28% +0.31% +0.53% +0.47%	143,406 (+396) 249,406 (+774) 75,406 (+397) 206,664 (+968)	+0.28% +0.31% +0.53% +0.47%	1 (0) 1 (0) 1 (0) 1 (0)

[CJ42]

LGTM ✅