Cleanup and comments

src/LightAccount.sol

@@ -166,7 +166,7 @@ contract LightAccount is BaseLightAccount, CustomSlotInitializable {
 
     function _getStorage() internal pure returns (LightAccountStorage storage storageStruct) {
         bytes32 position = _STORAGE_POSITION;
-        assembly {
+        assembly ("memory-safe") {
             storageStruct.slot := position
         }
     }

src/LightAccountFactory.sol

@@ -45,7 +45,12 @@ contract LightAccountFactory {
         );
     }
 
+    /// @notice Compute the hash of the owner and salt in scratch space memory.
+    /// @param owner The owner of the account to be created.
+    /// @param salt A salt, which can be changed to create multiple accounts with the same owner.
+    /// @return combinedSalt The hash of the owner and salt.
     function _getCombinedSalt(address owner, uint256 salt) internal pure returns (bytes32 combinedSalt) {
+        // Compute the hash of the owner and salt in scratch space memory.
         assembly ("memory-safe") {
             mstore(0x00, owner)
             mstore(0x20, salt)

src/MultiOwnerLightAccount.sol

@@ -172,7 +172,7 @@ contract MultiOwnerLightAccount is BaseLightAccount, CustomSlotInitializable {
         returns (bool)
     {
         (address recovered, ECDSA.RecoverError error,) = derivedHash.tryRecover(trimmedSignature);
-        return (error == ECDSA.RecoverError.NoError && _getStorage().owners.contains(CastLib.toSetValue(recovered)))
+        return (error == ECDSA.RecoverError.NoError && _getStorage().owners.contains(recovered.toSetValue()))
             || _isValidERC1271SignatureNow(derivedHash, trimmedSignature);
     }
 
@@ -193,7 +193,7 @@ contract MultiOwnerLightAccount is BaseLightAccount, CustomSlotInitializable {
 
     function _getStorage() internal pure returns (LightAccountStorage storage storageStruct) {
         bytes32 position = _STORAGE_POSITION;
-        assembly {
+        assembly ("memory-safe") {
             storageStruct.slot := position
         }
     }

src/MultiOwnerLightAccountFactory.sol

@@ -75,6 +75,10 @@ contract MultiOwnerLightAccountFactory {
         );
     }
 
+    /// @notice Compute the hash of the owner and salt in scratch space memory.
+    /// @param owners The owners of the account to be created.
+    /// @param salt A salt, which can be changed to create multiple accounts with the same owner.
+    /// @return combinedSalt The hash of the owner and salt.
     function _getCombinedSalt(address[] memory owners, uint256 salt) internal pure returns (bytes32) {
         return keccak256(abi.encode(owners, salt));
     }

src/common/BaseLightAccount.sol

@@ -11,7 +11,6 @@ import {UUPSUpgradeable} from "../../ext/solady/UUPSUpgradeable.sol";
 import {ERC1271} from "./ERC1271.sol";
 
 abstract contract BaseLightAccount is BaseAccount, TokenCallbackHandler, UUPSUpgradeable, ERC1271 {
-    bytes4 internal constant _1271_MAGIC_VALUE = bytes4(keccak256("isValidSignature(bytes32,bytes)")); // 0x1626ba7e
     IEntryPoint internal immutable _ENTRY_POINT;
 
     /// @dev The length of the array does not match the expected length.
@@ -109,7 +108,7 @@ abstract contract BaseLightAccount is BaseAccount, TokenCallbackHandler, UUPSUpg
     function _call(address target, uint256 value, bytes memory data) internal {
         (bool success, bytes memory result) = target.call{value: value}(data);
         if (!success) {
-            assembly {
+            assembly ("memory-safe") {
                 revert(add(result, 32), mload(result))
             }
         }

src/common/CustomSlotInitializable.sol

@@ -182,7 +182,7 @@ abstract contract CustomSlotInitializable {
 
     function _getInitializableStorage() private view returns (CustomSlotInitializableStorage storage _storage) {
         bytes32 position = _storagePosition;
-        assembly {
+        assembly ("memory-safe") {
             _storage.slot := position
         }
     }