prevent any type of ETH donation

Author: sparrowDom

src/contracts/adapters/EtherFiAssetAdapter.sol

@@ -166,11 +166,11 @@ contract EtherFiAssetAdapter is Initializable, IAssetAdapter, IERC721Receiver {
         return pendingRequestIds[index];
     }
 
-    /// @notice Accepts ETH donations and authorized Ether.fi claim proceeds.
-    /// @dev A permissionless Ether.fi claim is rejected unless it was initiated by this adapter.
-    /// Ether.fi reverts the entire claim, including the NFT burn, when this transfer fails.
+    /// @notice Accepts ETH only while this adapter is claiming Ether.fi withdrawals.
+    /// @dev ETH arriving outside an adapter-initiated claim (e.g. a permissionless Ether.fi claim) is
+    /// rejected. Ether.fi reverts the entire claim, including the NFT burn, when this transfer fails.
     receive() external payable {
-        if (msg.sender == address(etherfiWithdrawalNFT) && !claimingEtherFi) {
+        if (!claimingEtherFi) {
             revert UnauthorizedEtherFiClaim();
         }
     }

src/contracts/adapters/WeETHAssetAdapter.sol

@@ -176,11 +176,11 @@ contract WeETHAssetAdapter is Initializable, IAssetAdapter, IERC721Receiver {
         return pendingRequestIds[index];
     }
 
-    /// @notice Accepts ETH donations and authorized Ether.fi claim proceeds.
-    /// @dev A permissionless Ether.fi claim is rejected unless it was initiated by this adapter.
-    /// Ether.fi reverts the entire claim, including the NFT burn, when this transfer fails.
+    /// @notice Accepts ETH only while this adapter is claiming Ether.fi withdrawals.
+    /// @dev ETH arriving outside an adapter-initiated claim (e.g. a permissionless Ether.fi claim) is
+    /// rejected. Ether.fi reverts the entire claim, including the NFT burn, when this transfer fails.
     receive() external payable {
-        if (msg.sender == address(etherfiWithdrawalNFT) && !claimingEtherFi) {
+        if (!claimingEtherFi) {
             revert UnauthorizedEtherFiClaim();
         }
     }

test/unit/adapters/concrete/EtherFiAssetAdapter.t.sol

@@ -280,11 +280,14 @@ contract Unit_EtherFiAssetAdapter_Test is Test {
         );
     }
 
-    function test_Receive_AcceptsEth() public {
+    function test_Receive_RevertWhen_EthArrivesOutsideClaim() public {
+        // Any ETH — not just NFT-forwarded claim proceeds — is refused outside an adapter-initiated claim.
         vm.deal(address(this), 1 ether);
-        (bool ok,) = address(adapter).call{value: 1 ether}("");
-        assertTrue(ok, "adapter accepts ETH");
-        assertEq(address(adapter).balance, 1 ether, "adapter eth balance");
+        (bool ok, bytes memory ret) = address(adapter).call{value: 1 ether}("");
+
+        assertFalse(ok, "adapter rejects ETH outside a claim");
+        assertEq(bytes4(ret), EtherFiAssetAdapter.UnauthorizedEtherFiClaim.selector, "revert selector");
+        assertEq(address(adapter).balance, 0, "no ETH retained");
     }
 
     //////////////////////////////////////////////////////

test/unit/adapters/concrete/WeETHAssetAdapter.t.sol

@@ -264,11 +264,14 @@ contract Unit_WeETHAssetAdapter_Test is Test {
         );
     }
 
-    function test_Receive_AcceptsEth() public {
+    function test_Receive_RevertWhen_EthArrivesOutsideClaim() public {
+        // Any ETH — not just NFT-forwarded claim proceeds — is refused outside an adapter-initiated claim.
         vm.deal(address(this), 1 ether);
-        (bool ok,) = address(adapter).call{value: 1 ether}("");
-        assertTrue(ok, "adapter accepts ETH");
-        assertEq(address(adapter).balance, 1 ether, "adapter eth balance");
+        (bool ok, bytes memory ret) = address(adapter).call{value: 1 ether}("");
+
+        assertFalse(ok, "adapter rejects ETH outside a claim");
+        assertEq(bytes4(ret), WeETHAssetAdapter.UnauthorizedEtherFiClaim.selector, "revert selector");
+        assertEq(address(adapter).balance, 0, "no ETH retained");
     }
 
     //////////////////////////////////////////////////////