optimism
optimism copied to clipboard
batcher: remove isPectra and assume Pectra is always active
Summary
Since the Pectra fork happened long ago, we can now assume L1 is always running Pectra. This PR simplifies the batcher code by removing the isPectra checks.
Changes
- Remove
isPectraparameter fromChannelConfigProvider.ChannelConfig() - Remove
isPectraparameter fromchannelManager.TxData() - Simplify
BatchSubmitter.l1Tip()to no longer return isPectra boolean - Update
computeSingleCalldataTxCost()to always usetotalCostFloorPerToken = 10(Pectra pricing) - Update all related tests to reflect Pectra-only behavior
Files Changed
op-batcher/batcher/channel_config_provider.goop-batcher/batcher/channel_config.goop-batcher/batcher/channel_manager.goop-batcher/batcher/driver.goop-batcher/batcher/test_batch_submitter.goop-batcher/batcher/channel_config_provider_test.goop-batcher/batcher/channel_manager_test.goop-batcher/batcher/channel_manager_memory_test.go
Closes #18479
@geoknee
Checked op-e2e/system/da - the L1PragueTimeOffset in eip4844_test.go is just test config to activate Prague at genesis, not a runtime check like isPectra was.
No similar simplification needed there.
There's a comment about updating maxBlobsPerBlock to Prague's limit (9 vs 6), but that's more of a feature change than a simplification.
/ci authorize 62490c8