[TODOs] grace_period_end_offset_blocks validation & cleanup

x/shared/types/params.go

@@ -89,9 +89,6 @@ func (params *Params) ParamSetPairs() paramtypes.ParamSetPairs {
 
 // Validate validates the set of params
 func (params *Params) ValidateBasic() error {
-	// TODO_BLOCKER(@bryanchriswhite): Add `SessionGracePeriodBlocks` as a shared param,
-	// introduce validation, and ensure it is strictly less than num blocks per session.
-
 	if err := ValidateNumBlocksPerSession(params.NumBlocksPerSession); err != nil {
 		return err
 	}
@@ -116,6 +113,10 @@ func (params *Params) ValidateBasic() error {
 		return err
 	}
 
+	if err := validateGracePeriodOffsetBlocksIsLessThanNumBlocksPerSession(params); err != nil {
+		return err
+	}
+
 	if err := validateClaimWindowOpenOffsetIsAtLeastGracePeriodEndOffset(params); err != nil {
 		return err
 	}
@@ -178,6 +179,10 @@ func validateIsUint64(value any) error {
 	return nil
 }
 
+// validateClaimWindowOpenOffsetIsAtLeastGracePeriodEndOffset validates that the ClaimWindowOpenOffsetBlocks
+// is at least as big as GracePeriodEndOffsetBlocks. The claim window cannot open until the grace period ends
+// to ensure that the seed for the earliest supplier claim commit height is only observed after the last relay
+// for a given session could be serviced.
 func validateClaimWindowOpenOffsetIsAtLeastGracePeriodEndOffset(params *Params) error {
 	if params.ClaimWindowOpenOffsetBlocks < params.GracePeriodEndOffsetBlocks {
 		return ErrSharedParamInvalid.Wrapf(
@@ -186,6 +191,18 @@ func validateClaimWindowOpenOffsetIsAtLeastGracePeriodEndOffset(params *Params)
 			params.GracePeriodEndOffsetBlocks,
 		)
 	}
+	return nil
+}
 
+// validateGracePeriodOffsetBlocksIsLessThanNumBlocksPerSession validates that the
+// GracePeriodEndOffsetBlocks is less than NumBlocksPerSession; i.e., one session.
+func validateGracePeriodOffsetBlocksIsLessThanNumBlocksPerSession(params *Params) error {
+	if params.GracePeriodEndOffsetBlocks >= params.NumBlocksPerSession {
+		return ErrSharedParamInvalid.Wrapf(
+			"GracePeriodEndOffsetBlocks (%v) must be less than NumBlocksPerSession (%v)",
+			params.GracePeriodEndOffsetBlocks,
+			params.NumBlocksPerSession,
+		)
+	}
 	return nil
 }

x/tokenomics/keeper/msg_update_params.go

@@ -14,10 +14,6 @@ func (k msgServer) UpdateParams(ctx context.Context, msg *types.MsgUpdateParams)
 		return nil, err
 	}
 
-	// TODO_BLOCKER(@Olshansk): How do we validate this is the same address that signed the request?
-	// Do we have to use `msg.GetSigners()` explicitly during the check/validation or
-	// does the `cosmos.msg.v1.signer` tag in the protobuf definition enforce
-	// this somewhere in the Cosmos SDK?
 	if msg.Authority != k.GetAuthority() {
 		return nil, types.ErrTokenomicsInvalidSigner.Wrapf(
 			"invalid authority; expected %s, got %s",