refactor: remove global variables

ante/cosmos.go

@@ -16,7 +16,7 @@ func newCosmosAnteHandler(ctx sdk.Context, options HandlerOptions) sdk.AnteHandl
 	feemarketParams := options.FeeMarketKeeper.GetParams(ctx)
 	var txFeeChecker ante.TxFeeChecker
 	if options.DynamicFeeChecker {
-		txFeeChecker = evmante.NewDynamicFeeChecker(&feemarketParams)
+		txFeeChecker = evmante.NewDynamicFeeChecker(options.EvmKeeper, &feemarketParams)
 	}
 
 	return sdk.ChainAnteDecorators(
@@ -30,7 +30,7 @@ func newCosmosAnteHandler(ctx sdk.Context, options HandlerOptions) sdk.AnteHandl
 		ante.NewValidateBasicDecorator(),
 		ante.NewTxTimeoutHeightDecorator(),
 		ante.NewValidateMemoDecorator(options.AccountKeeper),
-		cosmosante.NewMinGasPriceDecorator(&feemarketParams),
+		cosmosante.NewMinGasPriceDecorator(&feemarketParams, options.EvmKeeper),
 		ante.NewConsumeGasForTxSizeDecorator(options.AccountKeeper),
 		ante.NewDeductFeeDecorator(options.AccountKeeper, options.BankKeeper, options.FeegrantKeeper, txFeeChecker),
 		// SetPubKeyDecorator must be called before all signature verification decorators

ante/cosmos/min_gas_price.go

@@ -5,8 +5,8 @@ import (
 	"math/big"
 	"slices"
 
+	anteinterfaces "github.com/cosmos/evm/ante/interfaces"
 	feemarkettypes "github.com/cosmos/evm/x/feemarket/types"
-	evmtypes "github.com/cosmos/evm/x/vm/types"
 
 	errorsmod "cosmossdk.io/errors"
 	"cosmossdk.io/math"
@@ -22,12 +22,16 @@ import (
 // CONTRACT: Tx must implement FeeTx to use MinGasPriceDecorator
 type MinGasPriceDecorator struct {
 	feemarketParams *feemarkettypes.Params
+	evmKeeper       anteinterfaces.EVMKeeper
 }
 
 // NewMinGasPriceDecorator creates a new MinGasPriceDecorator instance used only for
 // Cosmos transactions.
-func NewMinGasPriceDecorator(feemarketParams *feemarkettypes.Params) MinGasPriceDecorator {
-	return MinGasPriceDecorator{feemarketParams}
+func NewMinGasPriceDecorator(feemarketParams *feemarkettypes.Params, evmKeeper anteinterfaces.EVMKeeper) MinGasPriceDecorator {
+	return MinGasPriceDecorator{
+		feemarketParams: feemarketParams,
+		evmKeeper:       evmKeeper,
+	}
 }
 
 func (mpd MinGasPriceDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate bool, next sdk.AnteHandler) (newCtx sdk.Context, err error) {
@@ -39,7 +43,7 @@ func (mpd MinGasPriceDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate
 	minGasPrice := mpd.feemarketParams.MinGasPrice
 
 	feeCoins := feeTx.GetFee()
-	evmDenom := evmtypes.GetEVMCoinDenom()
+	evmDenom := mpd.evmKeeper.EvmCoinInfo().Denom
 
 	// only allow user to pass in aatom and stake native token as transaction fees
 	// allow use stake native tokens for fees is just for unit tests to pass

ante/evm/05_signature_verification.go

@@ -32,7 +32,7 @@ func NewEthSigVerificationDecorator(ek anteinterfaces.EVMKeeper) EthSigVerificat
 // Failure in RecheckTx will prevent tx to be included into block, especially when CheckTx succeed, in which case user
 // won't see the error message.
 func (esvd EthSigVerificationDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate bool, next sdk.AnteHandler) (newCtx sdk.Context, err error) {
-	ethCfg := evmtypes.GetEthChainConfig()
+	ethCfg := esvd.evmKeeper.EthChainConfig()
 	blockNum := big.NewInt(ctx.BlockHeight())
 	signer := ethtypes.MakeSigner(ethCfg, blockNum, uint64(ctx.BlockTime().Unix())) //#nosec G115 -- int overflow is not a concern here
 

ante/evm/10_gas_wanted.go

@@ -6,7 +6,6 @@ import (
 	anteinterfaces "github.com/cosmos/evm/ante/interfaces"
 	"github.com/cosmos/evm/ante/types"
 	feemarkettypes "github.com/cosmos/evm/x/feemarket/types"
-	evmtypes "github.com/cosmos/evm/x/vm/types"
 
 	errorsmod "cosmossdk.io/errors"
 
@@ -37,7 +36,7 @@ func NewGasWantedDecorator(
 }
 
 func (gwd GasWantedDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate bool, next sdk.AnteHandler) (newCtx sdk.Context, err error) {
-	ethCfg := evmtypes.GetEthChainConfig()
+	ethCfg := gwd.evmKeeper.EthChainConfig()
 
 	blockHeight := big.NewInt(ctx.BlockHeight())
 	isLondon := ethCfg.IsLondon(blockHeight)

ante/evm/fee_checker.go

@@ -2,9 +2,8 @@
 
 import (
 	"math"
 
-	"github.com/ethereum/go-ethereum/params"
-
+	anteinterfaces "github.com/cosmos/evm/ante/interfaces"
 	cosmosevmtypes "github.com/cosmos/evm/ante/types"
 	feemarkettypes "github.com/cosmos/evm/x/feemarket/types"
 	evmtypes "github.com/cosmos/evm/x/vm/types"
@@ -15,6 +14,7 @@
 	sdk "github.com/cosmos/cosmos-sdk/types"
 	errortypes "github.com/cosmos/cosmos-sdk/types/errors"
 	authante "github.com/cosmos/cosmos-sdk/x/auth/ante"
+	gethparams "github.com/ethereum/go-ethereum/params"
 )
 
 // NewDynamicFeeChecker returns a `TxFeeChecker` that applies a dynamic fee to
@@ -25,7 +25,7 @@
 // - when `ExtensionOptionDynamicFeeTx` is omitted, `tipFeeCap` defaults to `MaxInt64`.
 // - when london hardfork is not enabled, it falls back to SDK default behavior (validator min-gas-prices).
 // - Tx priority is set to `effectiveGasPrice / DefaultPriorityReduction`.
-func NewDynamicFeeChecker(feemarketParams *feemarkettypes.Params) authante.TxFeeChecker {
+func NewDynamicFeeChecker(evmKeeper anteinterfaces.EVMKeeper, feemarketParams *feemarkettypes.Params) authante.TxFeeChecker {
 	return func(ctx sdk.Context, tx sdk.Tx) (sdk.Coins, int64, error) {
 		feeTx, ok := tx.(sdk.FeeTx)
 		if !ok {
@@ -37,8 +37,12 @@
 			// genesis transactions: fallback to min-gas-price logic
 			return checkTxFeeWithValidatorMinGasPrices(ctx, feeTx)
 		}
-		denom := evmtypes.GetEVMCoinDenom()
-		ethCfg := evmtypes.GetEthChainConfig()
+		ethCfg := evmKeeper.EthChainConfig()
+		coinInfo := evmKeeper.EvmCoinInfo()
+		denom := coinInfo.Denom
+		if denom == "" {
+			denom = evmtypes.DefaultEVMDenom
+		}
 
 		return FeeChecker(ctx, feemarketParams, denom, ethCfg, feeTx)
 	}
@@ -49,7 +53,7 @@
 	ctx sdk.Context,
 	feemarketParams *feemarkettypes.Params,
 	denom string,
-	ethConfig *params.ChainConfig,
+	ethConfig *gethparams.ChainConfig,
 	feeTx sdk.FeeTx,
 ) (sdk.Coins, int64, error) {
 	if !evmtypes.IsLondon(ethConfig, ctx.BlockHeight()) {

ante/evm/fee_checker_test.go

@@ -251,14 +251,16 @@ func TestSDKTxFeeChecker(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			cfg := evmtypes.GetEthChainConfig()
+			chainCfg := evmtypes.GetChainConfig()
+			ethCfg := chainCfg.EthereumConfig(nil)
 			if !tc.londonEnabled {
-				cfg.LondonBlock = big.NewInt(10000)
+				ethCfg.LondonBlock = big.NewInt(10000)
 			} else {
-				cfg.LondonBlock = big.NewInt(0)
+				ethCfg.LondonBlock = big.NewInt(0)
 			}
+			evmKeeper := NewExtendedEVMKeeper()
 			feemarketParams := tc.feemarketParamsFn()
-			fees, priority, err := evm.NewDynamicFeeChecker(&feemarketParams)(tc.ctx, tc.buildTx())
+			fees, priority, err := evm.NewDynamicFeeChecker(evmKeeper, &feemarketParams)(tc.ctx, tc.buildTx())
 			if tc.expSuccess {
 				require.Equal(t, tc.expFees, fees.String())
 				require.Equal(t, tc.expPriority, priority)

ante/evm/mono_decorator.go

@@ -76,7 +76,10 @@ func (md MonoDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate bool, ne
 		}
 	}
 
-	evmDenom := evmtypes.GetEVMCoinDenom()
+	evmDenom := md.evmKeeper.EvmCoinInfo().Denom
+	if evmDenom == "" {
+		evmDenom = evmtypes.DefaultEVMDenom
+	}
 
 	// 1. setup ctx
 	ctx, err = SetupContextAndResetTransientGas(ctx, tx, md.evmKeeper)
@@ -112,7 +115,7 @@ func (md MonoDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate bool, ne
 		Difficulty: big.NewInt(0),
 	}
 
-	chainConfig := evmtypes.GetEthChainConfig()
+	chainConfig := md.evmKeeper.EthChainConfig()
 
 	if err := txpool.ValidateTransaction(ethTx, &header, decUtils.Signer, &txpool.ValidationOptions{
 		Config:  chainConfig,

ante/evm/utils.go

@@ -5,7 +5,8 @@
 	"math/big"
 
 	ethtypes "github.com/ethereum/go-ethereum/core/types"
 	"github.com/ethereum/go-ethereum/params"
+	gethparams "github.com/ethereum/go-ethereum/params"
 
 	anteinterfaces "github.com/cosmos/evm/ante/interfaces"
 	feemarkettypes "github.com/cosmos/evm/x/feemarket/types"
@@ -33,6 +34,11 @@
 	TxFee              *big.Int
 }
 
+type chainConfigProvider interface {
+	EthChainConfig() *gethparams.ChainConfig
+	ChainConfig() *evmtypes.ChainConfig
+}
+
 // NewMonoDecoratorUtils returns a new DecoratorUtils instance.
 //
 // These utilities are extracted once at the beginning of the ante handle process,
@@ -47,8 +53,9 @@
 	evmParams *evmtypes.Params,
 	feemarketParams *feemarkettypes.Params,
 ) (*DecoratorUtils, error) {
-	ethCfg := evmtypes.GetEthChainConfig()
-	evmDenom := evmtypes.GetEVMCoinDenom()
+	ethCfg := ek.EthChainConfig()
+	evmDenom := ek.EvmCoinInfo().Denom
+
 	blockHeight := big.NewInt(ctx.BlockHeight())
 	rules := ethCfg.Rules(blockHeight, true, uint64(ctx.BlockTime().Unix())) //#nosec G115 -- int overflow is not a concern here
 	baseFee := evmtypes.GetBaseFee(ctx.BlockHeight(), ethCfg, feemarketParams)

ante/interfaces/evm.go

@@ -5,6 +5,7 @@ import (
 	"github.com/ethereum/go-ethereum/core"
 	"github.com/ethereum/go-ethereum/core/tracing"
 	"github.com/ethereum/go-ethereum/core/vm"
+	gethparams "github.com/ethereum/go-ethereum/params"
 	"github.com/holiman/uint256"
 
 	feemarkettypes "github.com/cosmos/evm/x/feemarket/types"
@@ -26,6 +27,9 @@ type EVMKeeper interface {
 	ResetTransientGasUsed(ctx sdk.Context)
 	GetTxIndexTransient(ctx sdk.Context) uint64
 	GetParams(ctx sdk.Context) evmtypes.Params
+	ChainConfig() *evmtypes.ChainConfig
+	EthChainConfig() *gethparams.ChainConfig
+	EvmCoinInfo() evmtypes.EvmCoinInfo
 }
 
 // FeeMarketKeeper exposes the required feemarket keeper interface required for ante handlers

evmd/app.go

@@ -7,7 +7,6 @@ import (
 	"io"
 	"os"
 
-
 	"github.com/spf13/cast"
 
 	// Force-load the tracer engines to trigger registration due to Go-Ethereum v1.10.15 changes
@@ -803,6 +802,10 @@ func (app *EVMD) RegisterPendingTxListener(listener func(common.Hash)) {
 	app.pendingTxListeners = append(app.pendingTxListeners, listener)
 }
 
+func (app *EVMD) RuntimeConfig() *evmtypes.RuntimeConfig {
+	return app.EVMKeeper.RuntimeConfig()
+}
+
 func (app *EVMD) setPostHandler() {
 	postHandler, err := posthandler.NewPostHandler(
 		posthandler.HandlerOptions{},

evmd/mempool.go

@@ -2,6 +2,7 @@ package evmd
 
 import (
 	"fmt"
+
 	"github.com/cosmos/evm/server"
 
 	"cosmossdk.io/log"
@@ -11,12 +12,11 @@ import (
 	sdkmempool "github.com/cosmos/cosmos-sdk/types/mempool"
 
 	evmmempool "github.com/cosmos/evm/mempool"
-	evmtypes "github.com/cosmos/evm/x/vm/types"
 )
 
 // configureEVMMempool sets up the EVM mempool and related handlers using viper configuration.
 func (app *EVMD) configureEVMMempool(appOpts servertypes.AppOptions, logger log.Logger) error {
-	if evmtypes.GetChainConfig() == nil {
+	if app.EVMKeeper.ChainConfig() == nil {
 		logger.Debug("evm chain config is not set, skipping mempool configuration")
 		return nil
 	}

mempool/blockchain.go

@@ -75,7 +75,13 @@ func NewBlockchain(ctx func(height int64, prove bool) (sdk.Context, error), logg
 // Config returns the Ethereum chain configuration. It should only be called after the chain is initialized.
 // This provides the necessary parameters for EVM execution and transaction validation.
 func (b *Blockchain) Config() *params.ChainConfig {
-	return evmtypes.GetEthChainConfig()
+	if cfg := b.vmKeeper.EthChainConfig(); cfg != nil {
+		return cfg
+	}
+	if chainCfg := b.vmKeeper.ChainConfig(); chainCfg != nil {
+		return chainCfg.EthereumConfig(nil)
+	}
+	return evmtypes.DefaultChainConfig(0).EthereumConfig(nil)
 }
 
 // CurrentBlock returns the current block header for the app.
@@ -109,7 +115,7 @@ func (b *Blockchain) CurrentBlock() *types.Header {
 		Difficulty: big.NewInt(0), // 0 difficulty on PoS
 	}
 
-	chainConfig := evmtypes.GetEthChainConfig()
+	chainConfig := b.Config()
 	if chainConfig.IsLondon(header.Number) {
 		baseFee := b.vmKeeper.GetBaseFee(ctx)
 		if baseFee != nil {

mempool/blockchain_test.go

@@ -56,6 +56,8 @@ func TestBlockchainRaceCondition(t *testing.T) {
 	mockVMKeeper.On("GetBaseFee", mock.Anything).Return(big.NewInt(1000000000)).Maybe()         // 1 gwei
 	mockFeeMarketKeeper.On("GetBlockGasWanted", mock.Anything).Return(uint64(10000000)).Maybe() // 10M gas
 	mockVMKeeper.On("GetParams", mock.Anything).Return(vmtypes.DefaultParams()).Maybe()
+	mockVMKeeper.On("ChainConfig").Return(ethCfg).Maybe()
+	mockVMKeeper.On("EthChainConfig").Return(ethCfg.EthereumConfig(nil)).Maybe()
 	mockVMKeeper.On("GetAccount", mock.Anything, common.Address{}).Return(&statedb.Account{}).Maybe()
 	mockVMKeeper.On("GetState", mock.Anything, common.Address{}, common.Hash{}).Return(common.Hash{}).Maybe()
 	mockVMKeeper.On("GetCode", mock.Anything, common.Hash{}).Return([]byte{}).Maybe()

mempool/interface.go

@@ -4,6 +4,7 @@ import (
 	"math/big"
 
 	"github.com/ethereum/go-ethereum/common"
+	gethparams "github.com/ethereum/go-ethereum/params"
 
 	"github.com/cosmos/evm/x/vm/statedb"
 	vmtypes "github.com/cosmos/evm/x/vm/types"
@@ -17,6 +18,8 @@ type VMKeeperI interface {
 	GetBaseFee(ctx sdk.Context) *big.Int
 	GetParams(ctx sdk.Context) (params vmtypes.Params)
 	GetEvmCoinInfo(ctx sdk.Context) (coinInfo vmtypes.EvmCoinInfo)
+	ChainConfig() *vmtypes.ChainConfig
+	EthChainConfig() *gethparams.ChainConfig
 	GetAccount(ctx sdk.Context, addr common.Address) *statedb.Account
 	GetState(ctx sdk.Context, addr common.Address, key common.Hash) common.Hash
 	GetCode(ctx sdk.Context, codeHash common.Hash) []byte