Emilia Hane

Results 232 comments of Emilia Hane

> Does Base need a start import? no, only op mainnet, because op mainnet used to run in the OVM, which the EVM can't execute. then op mainnet switched to...

not sure if we can remove the trait `UpdateTxForwarder` since the type of the field on `OpEthApi` is a generic, however we could remove the trait as super trait from...

> one q, How to pass the ext_args and then I can call below func as your mentioned you need to add the `EngineApi` type to be built as AT...

on second look, addressing issue https://github.com/paradigmxyz/reth/issues/9562 shouldn't have to touch `EngineApi`. we can make a new type `OpEthApiBuilderCtx`, which wraps the `EthApiBuilderCtx` and the sequencer url string. then the tx...

actually @nkysg, there should be no need for an `OpEthApiBuilderCtx` type for now. just hard code the sequencer url in the `OpEthApi` builder function for this pr.

check it out @nkysg, at this point, it should be known that the eth api type returned by `RpcRegistryInner::eth_api` is `OpEthApi`, https://github.com/paradigmxyz/reth/blob/8a802dab600cd9a2dc0470eeeb22d1a0ccc30d8a/bin/reth/src/optimism.rs#L38-L47 so we should be able to add method...

need any help here @nkysg ?

is the end goal to replace `reth_primitives::PooledTransactionsElement` with `alloy_consensus::TypedTransaction` ? if so we'd be more efficient if we tackle this by making types that use `reth_primitives::PooledTransactionElement` generic over a type...

`T: alloy_consensus::Transaction + `, and impl the codec trait for alloy types where we define the codec trait