Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ERC: Batch Calls Encoding in SCA #282

Merged
merged 30 commits into from
Apr 16, 2024
Merged
Changes from 17 commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
bdacf2b
Create ERC:Optimized Calls Encoding
JXRow Feb 27, 2024
dc49fd4
add discussions-to
JXRow Feb 27, 2024
53c6a4b
rename to erc-xxx
JXRow Feb 27, 2024
274548a
rename to draft-OCE
JXRow Feb 27, 2024
74bcc30
rename to erc-9999
JXRow Feb 27, 2024
29228df
Update erc-9999.md
JXRow Feb 27, 2024
82c21f5
add co-author
JXRow Feb 27, 2024
9088f2a
fixed bytes32 to 32bytes
JXRow Feb 27, 2024
2cf553d
Merge branch 'master' into lightning
JXRow Feb 27, 2024
b5100cc
Update ERCS/erc-9999.md
JXRow Feb 28, 2024
83dbca1
Update ERCS/erc-9999.md
JXRow Feb 28, 2024
55338f2
rename to erc-7638
JXRow Feb 28, 2024
65455fa
Merge branch 'master' into lightning
JXRow Feb 28, 2024
0a62a3f
Merge branch 'master' into lightning
JXRow Mar 1, 2024
aca1483
Merge branch 'master' into lightning
JXRow Mar 4, 2024
4853797
Merge branch 'master' into lightning
JXRow Mar 6, 2024
6c473a8
Merge branch 'master' into lightning
JXRow Mar 19, 2024
3cb1aba
Update ERCS/erc-7638.md
JXRow Mar 26, 2024
29ec39a
Update ERCS/erc-7638.md
JXRow Mar 26, 2024
e7951e0
Update ERCS/erc-7638.md
JXRow Mar 26, 2024
2238e8a
Update ERCS/erc-7638.md
JXRow Mar 26, 2024
bb2c641
Update ERCS/erc-7638.md
JXRow Mar 26, 2024
2ed3f5a
Update ERCS/erc-7638.md
JXRow Mar 26, 2024
f74cfdf
Update ERCS/erc-7638.md
JXRow Mar 26, 2024
37cca5d
Merge branch 'master' into lightning
JXRow Mar 26, 2024
73d8ad6
update done
JXRow Mar 26, 2024
9db1c44
Update erc-7638.md
JXRow Mar 26, 2024
474fded
Update erc-7638.md
JXRow Mar 26, 2024
17f8ea8
Merge branch 'master' into lightning
JXRow Mar 29, 2024
93cae33
Apply suggestions from code review
SamWilsn Apr 16, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
190 changes: 190 additions & 0 deletions ERCS/erc-7638.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,190 @@
---
eip: 7638
title: Optimized Calls Encoding
JXRow marked this conversation as resolved.
Show resolved Hide resolved
description: Encode mutiple calls for smart contract wallet with less gas
author: George (@JXRow), Zisu (@lazy1523)
discussions-to: https://ethereum-magicians.org/t/erc-7638-optimized-calls-encoding/18966
status: Draft
type: Standards Track
category: ERC
created: 2024-02-26
---

## Abstract
`Optimized Calls Encoding`(OCE) outlines a solution for Smart Contract Account(SCA) wallets to consolidate multiple calls into a single call, encoding multiple parameters into bytes, compressing on-chain data, and saving gas. It can be used to implement atomic operations as well as non-atomic operations.
JXRow marked this conversation as resolved.
Show resolved Hide resolved

## Motivation
Typically, interactions between users and contracts involve a series of coherent operations, such as `approve-transferFrom`. While EOA wallets require users to confirm each operation sequentially, SCA wallets can confirm all operations with a single confirmation, completing all operations within a single call, thus achieving atomicity. If approve succeeds but transferFrom fails, it poses a security risk. The secure approach is to ensure that if one operation fails, all associated operations also fail, thereby ensuring atomicity. Therefore, we propose this encoding method to encode multiple parameters into bytes, compress on-chain data, and save gas. It can be used to implement both atomic and non-atomic operations.
JXRow marked this conversation as resolved.
Show resolved Hide resolved

In addition to the atomic operation of `approve-transferFrom` mentioned above, gas payment delegation can also be achieved. It involves users and bundlers signing a set of calls, where the content of the calls includes:
SamWilsn marked this conversation as resolved.
Show resolved Hide resolved

1. The user wishes to initiate multiple calls through his SCA.
2. The user transfers 10 USDT to the bundler as fee, included within the calls.
3. The bundler submits the calls, pay ETH gas and get the 10 USDT.

The user encodes the content of the calls, attaches their signature to ensure its integrity, and sends it to the bundler. If the bundler considers the gas payment insufficient, they may choose not to submit it. However, if they approve the content of the calls, the signed transaction can be submitted. After execution, the user obtains the desired operations, and the bundler receives the fee.

[EIP-4337](./eip-4337.md) also implements gas payment delegation. OCE and [EIP-4337](./eip-4337.md) are not mutually exclusive and can be implemented concurrently within an SCA.

Based on empirical testing, OCE is simpler and more gas-efficient compared to alternative methods.

## Specification
The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and "OPTIONAL" in this document are to be interpreted as described in RFC 2119 and RFC 8174.

`Optimized Calls` comprises multiple `Optimized Call` bytes, each defined by the encoding of `to\value\data` as follows:

### <center>| to(20bytes) | value(32bytes) | data length(32bytes) | data(bytes) |</center>
JXRow marked this conversation as resolved.
Show resolved Hide resolved

Let:
- `to`: The address of the called contract, corresponding to the Solidity address type, 20 bytes.
- `value`: The amount of ETH(in wei) sent to the contract, in wei, corresponding to the Solidity uint type, 32 bytes.
- `data length`: The length of the data(in bytes), corresponding to the Solidity uint type, 32 bytes.
- `data`: The encoded functionData sent to the contract, corresponding to the Solidity bytes type, with a length defined by `data length`.

Multiple `Optimized Call` units are concatenated to form an `Optimized Calls` sequence.

JXRow marked this conversation as resolved.
Show resolved Hide resolved

## Rationale
Each call encapsulates 3 parameters: `to\vallue\data`. The conventional approach involves packaging these 3 parameters into a struct and then placing multiple structs into an array. However, using a struct adds overhead as it also packages the types of `to\vallue\data`, increasing the size of the encoding. Since `to\vallue\data` have fixed types, this additional encoding can be omitted. In Solidity, reading data from `bytes calldata` using slice is a gas-efficient method. Considering these factors, `Optimized Calls Encoding` can compress on-chain data and save gas.

## Backwards Compatibility
This ERC does not change the consensus layer, so there are no backwards compatibility issues for Ethereum as a whole.

This ERC does not change other ERC standars, so there are no backwards compatibility issues for Ethereum applications.
JXRow marked this conversation as resolved.
Show resolved Hide resolved


## Reference Implementation
This proposal only specifies the encoding of `Optimized Calls`, while the specific implementation and naming are left to the discretion of the project. Below is an example of an SCA contract utilizing `Optimized Calls` (referred to as atomCallbytes), where the user atomically signs multiple operations, enabling the bundler to pay gas on behalf of the user:
JXRow marked this conversation as resolved.
Show resolved Hide resolved

SmartWallet.sol
JXRow marked this conversation as resolved.
Show resolved Hide resolved

```solidity
pragma solidity ^0.8.0;

import "@openzeppelin/contracts/utils/cryptography/ECDSA.sol";

contract SmartWallet {
using ECDSA for bytes32;

uint32 public valid = 1; //to make AtomSign invalid

address private immutable original;
address public owner;
address public bundler;

mapping(bytes32 => bool) public usedMsgHashes;

modifier onlyBundler() {
require(
bundler == msg.sender,
"onlyBundler: caller is not the bundler"
);
_;
}

modifier onlyOwnerAndOriginal() {
require(
owner == msg.sender || original == msg.sender,
"onlyOwnerAndOriginal: caller is not the owner"
);
_;
}

constructor(address _bundler) {
original = address(this);
owner = msg.sender;
bundler = _bundler;
}

function atomSignCall(
bytes calldata atomCallbytes,
uint32 deadline,
bytes calldata signature
) external onlyBundler {
require(deadline >= block.timestamp, "atomSignCall: Expired");
bytes32 msgHash = keccak256(
bytes.concat(
msg.data[:msg.data.length - signature.length - 32],
bytes32(block.chainid),
bytes20(address(this)),
bytes4(valid)
)
);
require(!usedMsgHashes[msgHash], "atomSignCall: Used msgHash");
require(
owner == msgHash.toEthSignedMessageHash().recover(signature),
"atomSignCall: Invalid Signature"
);

//do calls
uint i;
while(i < atomCallbytes.length) {
address to = address(uint160(bytes20(atomCallbytes[i:i+20])));
uint value = uint(bytes32(atomCallbytes[i+20:i+52]));
uint len = uint(bytes32(atomCallbytes[i+52:i+84]));

(bool success, bytes memory result) = to.call{value: value}(atomCallbytes[i+84:i+84+len]);
if (!success) {
assembly {
revert(add(result, 32), mload(result))
}
}

i += 84 + len;
}

usedMsgHashes[msgHash] = true;
}

/**
* if you signed something then regretted, make it invalid
*/
function makeAtomSignInvalid() public onlyOwnerAndOriginal {
valid = uint32(uint(blockhash(block.number)));
}
}
```

Bundler.sol
JXRow marked this conversation as resolved.
Show resolved Hide resolved

```solidity
pragma solidity ^0.8.0;

contract Bundler {

address public owner;

modifier onlyOwner() {
require(
owner == msg.sender,
"onlyOwner: caller is not the owner"
);
_;
}

constructor() {
owner = msg.sender;
}

function executeOperation(
address wallet,
bytes calldata data
) public onlyOwner {
(bool success, bytes memory result) = _callTo.call{value: 0}(data);

if (!success) {
assembly {
revert(add(result, 32), mload(result))
}
}
}
}
```

## Security Considerations
This proposal introduces a data encoding scheme aimed at data compression. It solely concerns data compression and does not lead to data loss or concealment of private data.


## Copyright
Copyright and related rights waived via [CC0](../LICENSE.md).

Loading