Removed sponsor == operator requirement

Author: mgretzke

src/allocators/SimpleAllocator.sol

@@ -67,18 +67,10 @@ contract SimpleAllocator is ISimpleAllocator {
     }
 
     /// @inheritdoc IAllocator
-    function attest(address operator_, address from_, address, uint256 id_, uint256 amount_)
-        external
-        view
-        returns (bytes4)
-    {
+    function attest(address, address from_, address, uint256 id_, uint256 amount_) external view returns (bytes4) {
         if (msg.sender != COMPACT_CONTRACT) {
             revert InvalidCaller(msg.sender, COMPACT_CONTRACT);
         }
-        // For a transfer, the sponsor is the arbiter
-        if (operator_ != from_) {
-            revert InvalidCaller(operator_, from_);
-        }
         uint256 balance = ERC6909(COMPACT_CONTRACT).balanceOf(from_, id_);
         // Check unlocked balance
         bytes32 tokenHash = _getTokenHash(id_, from_);

test/SimpleAllocator.t.sol

@@ -447,12 +447,6 @@ contract SimpleAllocator_Attest is Deposited {
         simpleAllocator.attest(address(user), address(user), address(usdc), usdcId, defaultAmount);
     }
 
-    function test_revert_InvalidCaller_FromNotOperator() public {
-        vm.prank(attacker);
-        vm.expectRevert(abi.encodeWithSelector(ISimpleAllocator.InvalidCaller.selector, attacker, user));
-        compactContract.transfer(user, attacker, defaultAmount, address(usdc), address(simpleAllocator));
-    }
-
     function test_revert_InsufficientBalance_NoActiveLock(uint128 falseAmount_) public {
         vm.assume(falseAmount_ > defaultAmount);