refactor(x/mint)!: avoid writing same minter and use millisecond precision for minting

simapp/mint_fn.go

@@ -60,20 +60,19 @@ func ProvideExampleMintFn(bankKeeper MintBankKeeper) minttypes.MintFn {
 		minter.AnnualProvisions = minter.NextAnnualProvisions(mintParams.Params, stakingTokenSupply.Amount)
 
 		// to get a more accurate amount of tokens minted, we get, and later store, last minting time.
-
 		// if this is the first time minting, we initialize the minter.Data with the current time - 60s
 		// to mint tokens at the beginning. Note: this is a custom behavior to avoid breaking tests.
 		if minter.Data == nil {
 			minter.Data = make([]byte, 8)
-			binary.BigEndian.PutUint64(minter.Data, (uint64)(env.HeaderService.HeaderInfo(ctx).Time.Unix()-60))
+			binary.BigEndian.PutUint64(minter.Data, (uint64)(env.HeaderService.HeaderInfo(ctx).Time.UnixMilli()-60000))
 		}
 
 		lastMint := binary.BigEndian.Uint64(minter.Data)
-		binary.BigEndian.PutUint64(minter.Data, (uint64)(env.HeaderService.HeaderInfo(ctx).Time.Unix()))
+		binary.BigEndian.PutUint64(minter.Data, (uint64)(env.HeaderService.HeaderInfo(ctx).Time.UnixMilli()))
 
-		// calculate the amount of tokens to mint, based on the time since the last mint
-		secsSinceLastMint := env.HeaderService.HeaderInfo(ctx).Time.Unix() - (int64)(lastMint)
-		provisionAmt := minter.AnnualProvisions.QuoInt64(31536000).MulInt64(secsSinceLastMint) // 31536000 = seconds in a year
+		// calculate the amount of tokens to mint, based on the time since the last mint.
+		msSinceLastMint := env.HeaderService.HeaderInfo(ctx).Time.UnixMilli() - (int64)(lastMint)
+		provisionAmt := minter.AnnualProvisions.QuoInt64(31536000000).MulInt64(msSinceLastMint) // 31536000000 = milliseconds in a year
 		mintedCoin := sdk.NewCoin(mintParams.Params.MintDenom, provisionAmt.TruncateInt())
 		maxSupply := mintParams.Params.MaxSupply
 		totalSupply := stakingTokenSupply.Amount

x/mint/epoch_hooks.go

@@ -1,6 +1,7 @@
 package mint
 
 import (
+	"bytes"
 	"context"
 
 	epochstypes "cosmossdk.io/x/epochs/types"
@@ -20,11 +21,18 @@ func (am AppModule) BeforeEpochStart(ctx context.Context, epochIdentifier string
 		return err
 	}
 
+	oldBz := am.cdc.MustMarshal(&minter)
+
 	err = am.mintFn(ctx, am.keeper.Environment, &minter, epochIdentifier, epochNumber)
 	if err != nil {
 		return err
 	}
 
+	newBz := am.cdc.MustMarshal(&minter)
+	if bytes.Equal(oldBz, newBz) {
+		return nil
+	}
+
 	return am.keeper.Minter.Set(ctx, minter)
 }
 

x/mint/keeper/abci.go

@@ -1,6 +1,7 @@
 package keeper
 
 import (
+	"bytes"
 	"context"
 
 	"cosmossdk.io/x/mint/types"
@@ -18,12 +19,19 @@
 		return err
 	}
 
+	oldBz := k.cdc.MustMarshal(&minter)
+
 	// 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)
 	if err != nil {
 		return err
 	}
 
+	newBz := k.cdc.MustMarshal(&minter)
+	if bytes.Equal(oldBz, newBz) {
+		return nil
+	}
+
 	return k.Minter.Set(ctx, minter)
 }