refactor(x/mint): remove staking as a required module

server/v2/cometbft/mempool/noop.go

@@ -8,8 +8,10 @@ import (
  sdk "github.com/cosmos/cosmos-sdk/types"
 )
 
-var _ Mempool[sdk.Tx] = (*NoOpMempool[sdk.Tx])(nil) // verify interface at compile time
-var _ Mempool[transaction.Tx] = (*NoOpMempool[transaction.Tx])(nil)
+var (
+ _ Mempool[sdk.Tx] = (*NoOpMempool[sdk.Tx])(nil) // verify interface at compile time
+ _ Mempool[transaction.Tx] = (*NoOpMempool[transaction.Tx])(nil)
+)
 
 // NoOpMempool defines a no-op mempool. Transactions are completely discarded and
 // ignored when BaseApp interacts with the mempool.

simapp/app.go

@@ -372,7 +372,10 @@ func NewSimApp(
  cometService,
  )
 
- app.MintKeeper = mintkeeper.NewKeeper(appCodec, runtime.NewEnvironment(runtime.NewKVStoreService(keys[minttypes.StoreKey]), logger.With(log.ModuleKey, "x/mint")), app.StakingKeeper, app.AuthKeeper, app.BankKeeper, authtypes.FeeCollectorName, govModuleAddr)
+ app.MintKeeper = mintkeeper.NewKeeper(appCodec, runtime.NewEnvironment(runtime.NewKVStoreService(keys[minttypes.StoreKey]), logger.With(log.ModuleKey, "x/mint")), app.AuthKeeper, app.BankKeeper, authtypes.FeeCollectorName, govModuleAddr)
+ if err := app.MintKeeper.SetMintFn(mintkeeper.DefaultMintFn(minttypes.DefaultInflationCalculationFn, app.StakingKeeper, app.MintKeeper)); err != nil {
+ panic(err)
+ }
 
  app.PoolKeeper = poolkeeper.NewKeeper(appCodec, runtime.NewEnvironment(runtime.NewKVStoreService(keys[pooltypes.StoreKey]), logger.With(log.ModuleKey, "x/protocolpool")), app.AuthKeeper, app.BankKeeper, app.StakingKeeper, govModuleAddr)
 
@@ -467,7 +470,7 @@ func NewSimApp(
  bank.NewAppModule(appCodec, app.BankKeeper, app.AuthKeeper),
  feegrantmodule.NewAppModule(appCodec, app.FeeGrantKeeper, app.interfaceRegistry),
  gov.NewAppModule(appCodec, &app.GovKeeper, app.AuthKeeper, app.BankKeeper, app.PoolKeeper),
- mint.NewAppModule(appCodec, app.MintKeeper, app.AuthKeeper, nil),
+ mint.NewAppModule(appCodec, app.MintKeeper, app.AuthKeeper),
  slashing.NewAppModule(appCodec, app.SlashingKeeper, app.AuthKeeper, app.BankKeeper, app.StakingKeeper, app.interfaceRegistry, cometService),
  distr.NewAppModule(appCodec, app.DistrKeeper, app.StakingKeeper),
  staking.NewAppModule(appCodec, app.StakingKeeper),

tests/integration/example/example_test.go

@@ -74,8 +74,8 @@ func Example() {
 
  // here bankkeeper and staking keeper is nil because we are not testing them
  // subspace is nil because we don't test params (which is legacy anyway)
- mintKeeper := mintkeeper.NewKeeper(encodingCfg.Codec, runtime.NewEnvironment(runtime.NewKVStoreService(keys[minttypes.StoreKey]), logger), nil, accountKeeper, nil, authtypes.FeeCollectorName, authority)
- mintModule := mint.NewAppModule(encodingCfg.Codec, mintKeeper, accountKeeper, nil)
+ mintKeeper := mintkeeper.NewKeeper(encodingCfg.Codec, runtime.NewEnvironment(runtime.NewKVStoreService(keys[minttypes.StoreKey]), logger), accountKeeper, nil, authtypes.FeeCollectorName, authority)
+ mintModule := mint.NewAppModule(encodingCfg.Codec, mintKeeper, accountKeeper)
 
  // create the application and register all the modules from the previous step
  integrationApp := integration.NewIntegrationApp(

x/accounts/README.md

@@ -56,8 +56,8 @@ func (a Account) AuthRetroCompatibility(ctx context.Context, _ *authtypes.QueryL
 
 ## Usage Notes
 
-- Implement this handler only for account types you want to expose via x/auth gRPC methods.
-- The `info` field in the response can be nil if your account doesn't fit the `BaseAccount` structure.
+* Implement this handler only for account types you want to expose via x/auth gRPC methods.
+* The `info` field in the response can be nil if your account doesn't fit the `BaseAccount` structure.
 
 # Genesis
 
@@ -102,4 +102,4 @@ For example, given the following `genesis.json` file:
 }
 ```
 
-The accounts module will run the lockup account initialization message.
+The accounts module will run the lockup account initialization message.

x/mint/CHANGELOG.md

@@ -36,5 +36,9 @@ Ref: https://keepachangelog.com/en/1.0.0/
 
 * [#20363](https://github.com/cosmos/cosmos-sdk/pull/20363) Deprecated InflationCalculationFn in favor of MintFn, `keeper.DefaultMintFn` wrapper must be used in order to continue using it in `NewAppModule`. This is not breaking for depinject users, as both `MintFn` and `InflationCalculationFn` are accepted.
 * [#19367](https://github.com/cosmos/cosmos-sdk/pull/19398) `appmodule.Environment` is received on the Keeper to get access to different application services.
+* [#21858](https://github.com/cosmos/cosmos-sdk/pull/21858) `DefaultMintFn` now takes `StakingKeeper` and `MintKeeper` as arguments to avoid staking keeper being required by mint. 
+ * `SetMintFn` is used to replace the default minting function.
+ * `InflationCalculationFn` is not passed through depinject any longer, a MintFn is required instead.
+
 
 ### Bug Fixes

x/mint/depinject.go

@@ -27,16 +27,15 @@ func init() {
 type ModuleInputs struct {
  depinject.In
 
- ModuleKey depinject.OwnModuleKey
- Config *modulev1.Module
- Environment appmodule.Environment
- Cdc codec.Codec
- MintFn types.MintFn `optional:"true"`
- InflationCalculationFn types.InflationCalculationFn `optional:"true"` // deprecated
+ ModuleKey depinject.OwnModuleKey
+ Config *modulev1.Module
+ Environment appmodule.Environment
+ Cdc codec.Codec
+ MintFn types.MintFn `optional:"true"`
 
  AccountKeeper types.AccountKeeper
  BankKeeper types.BankKeeper
- StakingKeeper types.StakingKeeper
+ StakingKeeper types.StakingKeeper `optional:"true"`
 }
 
 type ModuleOutputs struct {
@@ -67,28 +66,21 @@ func ProvideModule(in ModuleInputs) ModuleOutputs {
  k := keeper.NewKeeper(
  in.Cdc,
  in.Environment,
- in.StakingKeeper,
  in.AccountKeeper,
  in.BankKeeper,
  feeCollectorName,
  as,
  )
-
- if in.MintFn != nil && in.InflationCalculationFn != nil {
- panic("MintFn and InflationCalculationFn cannot both be set")
+ if in.MintFn == nil && in.StakingKeeper != nil {
+ panic("custom minting function or staking keeper must be supplied or available")
+ } else if in.MintFn == nil {
+ in.MintFn = keeper.DefaultMintFn(types.DefaultInflationCalculationFn, in.StakingKeeper, k)
  }
-
- // if no mintFn is provided, use the default minting function
- if in.MintFn == nil {
- // if no inflationCalculationFn is provided, use the default inflation calculation function
- if in.InflationCalculationFn == nil {
- in.InflationCalculationFn = types.DefaultInflationCalculationFn
- }
-
- in.MintFn = k.DefaultMintFn(in.InflationCalculationFn)
+ if err := k.SetMintFn(in.MintFn); err != nil {
+ panic(err)
  }
 
- m := NewAppModule(in.Cdc, k, in.AccountKeeper, in.MintFn)
+ m := NewAppModule(in.Cdc, k, in.AccountKeeper)
 
  return ModuleOutputs{MintKeeper: k, Module: m, EpochHooks: epochstypes.EpochHooksWrapper{EpochHooks: m}}
 }

x/mint/epoch_hooks.go

@@ -17,7 +17,7 @@ func (am AppModule) BeforeEpochStart(ctx context.Context, epochIdentifier string
 
  oldMinter := minter
 
- err = am.mintFn(ctx, am.keeper.Environment, &minter, epochIdentifier, epochNumber)
+ err = am.keeper.MintFn(ctx, &minter, epochIdentifier, epochNumber)
  if err != nil {
  return err
  }

x/mint/keeper/abci.go

@@ -9,7 +9,7 @@ import (
 )
 
 // BeginBlocker mints new tokens for the previous block.
-func (k Keeper) BeginBlocker(ctx context.Context, mintFn types.MintFn) error {
+func (k Keeper) BeginBlocker(ctx context.Context) error {
  defer telemetry.ModuleMeasureSince(types.ModuleName, telemetry.Now(), telemetry.MetricKeyBeginBlocker)
 
  // fetch stored minter & params
@@ -22,7 +22,7 @@ func (k Keeper) BeginBlocker(ctx context.Context, mintFn types.MintFn) error {
 
  // we pass -1 as epoch number to indicate that this is not an epoch minting,
  // but a regular block minting. Same with epoch id "block".
- err = mintFn(ctx, k.Environment, &minter, "block", -1)
+ err = k.MintFn(ctx, &minter, "block", -1)
  if err != nil {
  return err
  }

x/mint/keeper/genesis.go

@@ -18,6 +18,10 @@ func (keeper Keeper) InitGenesis(ctx context.Context, ak types.AccountKeeper, da
 
  ak.GetModuleAccount(ctx, types.ModuleName)
 
+ if keeper.mintFn == nil {
+ panic("mintFn cannot be nil")
+ }
+
  return nil
 }
 

x/mint/keeper/genesis_test.go

@@ -1,12 +1,14 @@
 package keeper_test
 
 import (
+ "context"
  "testing"
 
  "github.com/golang/mock/gomock"
  "github.com/stretchr/testify/suite"
 
  "cosmossdk.io/collections"
+ "cosmossdk.io/core/appmodule"
  "cosmossdk.io/log"
  "cosmossdk.io/math"
  storetypes "cosmossdk.io/store/types"
@@ -51,14 +53,17 @@ func (s *GenesisTestSuite) SetupTest() {
  s.sdkCtx = testCtx.Ctx
  s.key = key
 
- stakingKeeper := minttestutil.NewMockStakingKeeper(ctrl)
  accountKeeper := minttestutil.NewMockAccountKeeper(ctrl)
  bankKeeper := minttestutil.NewMockBankKeeper(ctrl)
  s.accountKeeper = accountKeeper
  accountKeeper.EXPECT().GetModuleAddress(minterAcc.Name).Return(minterAcc.GetAddress())
  accountKeeper.EXPECT().GetModuleAccount(s.sdkCtx, minterAcc.Name).Return(minterAcc)
 
- s.keeper = keeper.NewKeeper(s.cdc, runtime.NewEnvironment(runtime.NewKVStoreService(key), log.NewNopLogger()), stakingKeeper, accountKeeper, bankKeeper, "", "")
+ s.keeper = keeper.NewKeeper(s.cdc, runtime.NewEnvironment(runtime.NewKVStoreService(key), log.NewNopLogger()), accountKeeper, bankKeeper, "", "")
+ err := s.keeper.SetMintFn(func(ctx context.Context, env appmodule.Environment, minter *types.Minter, epochId string, epochNumber int64) error {
+ return nil
+ })
+ s.NoError(err)
 }
 
 func (s *GenesisTestSuite) TestImportExportGenesis() {

x/mint/keeper/grpc_query_test.go

@@ -42,14 +42,12 @@ func (suite *MintTestSuite) SetupTest() {
  ctrl := gomock.NewController(suite.T())
  accountKeeper := minttestutil.NewMockAccountKeeper(ctrl)
  bankKeeper := minttestutil.NewMockBankKeeper(ctrl)
- stakingKeeper := minttestutil.NewMockStakingKeeper(ctrl)
 
  accountKeeper.EXPECT().GetModuleAddress("mint").Return(sdk.AccAddress{})
 
  suite.mintKeeper = keeper.NewKeeper(
  encCfg.Codec,
  env,
- stakingKeeper,
  accountKeeper,
  bankKeeper,
  authtypes.FeeCollectorName,

x/mint/keeper/keeper.go

@@ -2,6 +2,7 @@ package keeper
 
 import (
  "context"
+ "errors"
  "fmt"
 
  "cosmossdk.io/collections"
@@ -20,7 +21,6 @@ type Keeper struct {
  appmodule.Environment
 
  cdc codec.BinaryCodec
- stakingKeeper types.StakingKeeper
  bankKeeper types.BankKeeper
  feeCollectorName string
  // the address capable of executing a MsgUpdateParams message. Typically, this
@@ -30,13 +30,17 @@ type Keeper struct {
  Schema collections.Schema
  Params collections.Item[types.Params]
  Minter collections.Item[types.Minter]
+
+ // mintFn is used to mint new coins during BeginBlock. This function is in charge of
+ // minting new coins based on arbitrary logic, previously done through InflationCalculationFn.
+ // If mintFn is nil, the default minting logic is used.
+ mintFn types.MintFn
 }
 
 // NewKeeper creates a new mint Keeper instance
 func NewKeeper(
  cdc codec.BinaryCodec,
  env appmodule.Environment,
- sk types.StakingKeeper,
  ak types.AccountKeeper,
  bk types.BankKeeper,
  feeCollectorName string,
@@ -51,7 +55,6 @@ func NewKeeper(
  k := Keeper{
  Environment: env,
  cdc: cdc,
- stakingKeeper: sk,
  bankKeeper: bk,
  feeCollectorName: feeCollectorName,
  authority: authority,
@@ -67,21 +70,20 @@ func NewKeeper(
  return k
 }
 
-// GetAuthority returns the x/mint module's authority.
-func (k Keeper) GetAuthority() string {
- return k.authority
-}
+// SetMintFn is used to mint new coins during BeginBlock. The mintFn function is in charge of
+// minting new coins based on arbitrary logic, previously done through InflationCalculationFn.
+func (k *Keeper) SetMintFn(mintFn types.MintFn) error {
+ if mintFn == nil {
+ return errors.New("mintFn cannot be nil")
+ }
 
-// StakingTokenSupply implements an alias call to the underlying staking keeper's
-// StakingTokenSupply to be used in BeginBlocker.
-func (k Keeper) StakingTokenSupply(ctx context.Context) (math.Int, error) {
- return k.stakingKeeper.StakingTokenSupply(ctx)
+ k.mintFn = mintFn
+ return nil
 }
 
-// BondedRatio implements an alias call to the underlying staking keeper's
-// BondedRatio to be used in BeginBlocker.
-func (k Keeper) BondedRatio(ctx context.Context) (math.LegacyDec, error) {
- return k.stakingKeeper.BondedRatio(ctx)
+// GetAuthority returns the x/mint module's authority.
+func (k Keeper) GetAuthority() string {
+ return k.authority
 }
 
 // MintCoins implements an alias call to the underlying supply keeper's
@@ -101,20 +103,26 @@ func (k Keeper) AddCollectedFees(ctx context.Context, fees sdk.Coins) error {
  return k.bankKeeper.SendCoinsFromModuleToModule(ctx, types.ModuleName, k.feeCollectorName, fees)
 }
 
-func (k Keeper) DefaultMintFn(ic types.InflationCalculationFn) types.MintFn {
+func (k Keeper) MintFn(ctx context.Context, minter *types.Minter, epochId string, epochNumber int64) error {
+ return k.mintFn(ctx, k.Environment, minter, epochId, epochNumber)
+}
+
+// DefaultMintFn returns a default mint function. It requires the Staking module and the mint keeper.
+// The default Mintfn has a requirement on staking as it uses bond to calculate inflation.
+func DefaultMintFn(ic types.InflationCalculationFn, staking types.StakingKeeper, k Keeper) types.MintFn {
  return func(ctx context.Context, env appmodule.Environment, minter *types.Minter, epochId string, epochNumber int64) error {
  // the default mint function is called every block, so we only check if epochId is "block" which is
  // a special value to indicate that this is not an epoch minting, but a regular block minting.
  if epochId != "block" {
  return nil
  }
 
- stakingTokenSupply, err := k.StakingTokenSupply(ctx)
+ stakingTokenSupply, err := staking.StakingTokenSupply(ctx)
  if err != nil {
  return err
  }
 
- bondedRatio, err := k.BondedRatio(ctx)
+ bondedRatio, err := staking.BondedRatio(ctx)
  if err != nil {
  return err
  }