ops(mempool) sorted

[octavei]

We assume that a person has many ops in the transaction pool, and the op monotonically increases the max_priority_fee_per_gas value. At this time, since the max_priority_fee_per_gas is ranked first, it eventually leads to the op with a large Nonce being submitted first, and the verification simulation fails.

code in project(crates/uopool/src/memory/mempool.rs):

    fn get_sorted(&self) -> anyhow::Result<Self::UserOperations> {
        let mut uos: Vec<UserOperation> = self.user_operations.values().cloned().collect();
        uos.sort_by(|a, b| {
            if a.max_priority_fee_per_gas != b.max_priority_fee_per_gas {
                b.max_priority_fee_per_gas.cmp(&a.max_priority_fee_per_gas)
            } else {
                a.nonce.cmp(&b.nonce)
            }
        });
        Ok(uos)
    }

[octavei]

Nice project. Has anyone already adapted it to a production environment?

[zsluedem]

Sadly no. We are looking for people to use it now.

[octavei]

There are still some small details that have not been dealt with. But already very good. We'd like to improve it together.

[zsluedem]

Are you going to use our bundler? You are more than welcome to use the bundler. Do you want to have a chat together?

[octavei]

This is an excellent project, we are currently planning to use it, but what we do may be different from yours. We apply zero-knowledge proof to sign the op under the chain, and Nonce value and Gas (deposit) are stored off-chain.

[Vid201]

Cool! You can also fork the code or use only certain crates if your use case differs from standard bundler a lot. :)

[octavei]

Like the sorting problem of the transaction pool above, it is actually the same problem as the Nonce value problem I mentioned earlier. At present, each person can only have one op at the same time. If it is changed to be compatible with many ops, the sorting function of the transaction pool also needs to be further changed.

[zsluedem]

Yes. I think you totally make sense. I took your issue as a feature request now.We would address that later.

[Vid201]

Just adding that there were some changes to 4337 about how the nonces are handled: https://eips.ethereum.org/EIPS/eip-4337#semi-abstracted-nonce-support

Bundler doesn't yet handle that. We are happy to accept contributions @octavei