Have auto and interval mining at the same time

[DefiCake]

Component

Anvil

Describe the feature you would like

Use case: Have fast transactions (= fast CI pipelines and tests) with automated software that reacts on events, like bots, indexers, etc

At the moment anvil allows either one mining mode or the other - auto, or interval. This is specifically signaled by the interval mining mode, that warns that auto is disabled.

Additional context

No response

[mattsse]

this should be possible to support by adding a new variant that combines both of these:

foundry/crates/anvil/src/eth/miner.rs

Lines 102 to 108 in d7eac74

	/// A miner that listens for new transactions that are ready.
	///
	/// Either one transaction will be mined per block, or any number of transactions will be
	/// allowed
	Auto(ReadyTransactionMiner),
	/// A miner that constructs a new block every `interval` tick
	FixedBlockTime(FixedBlockTimeMiner),

[caiquejjx]

@mattsse is this high priority? I would like to work on it but it's my first time interacting with foundry so it will take some time to get this done

[mattsse]

not high priority, the task here would be to add a new variant that is

Mixed {
  auto: ReadyTransactionMiner,
  fixed: FixedBlockTimeMiner
}

and then polls both

[caiquejjx]

oh looks simple enough, thanks!

[caiquejjx]

@mattsse hey, I created a draft pr with the new variant + it's pooling but I'm wondering if I should also add RPC handler for setting the mixed mining mode and besides that I'm not sure how to test this feature, when you have some time can you take a look?
Thank you

[zerosnacks]

Hi @caiquejjx thanks for picking this up, draft is looking good!

I think it makes sense to add RPC methods for this (anvil_setMixedmine(bool), anvil_getMixedmine(bool) matching anvil_setAutomine(bool) / anvil_getAutomine(bool)).

cc @mattsse

[caiquejjx]

thanks @zerosnacks, I'll follow up with the RPC methods implementation then