Add ModuleHashByHeightQuery command in server/v2/store

[julienrbrt]

Noticed while doing #23018 that the newly added ModuleHashByHeightQuery command in 0.52 doesn't exist in a server/v2 world.

cosmos-sdk/server/module_hash.go

Lines 22 to 53 in 4f5c3d8

	func ModuleHashByHeightQuery[T servertypes.Application](appCreator servertypes.AppCreator[T]) *cobra.Command {
	cmd := &cobra.Command{
	Use: "module-hash-by-height <height>",
	Short: "Get module hashes at a given height",
	Long: "Get module hashes at a given height. This command is useful for debugging and verifying the state of the application at a given height. Daemon should not be running when calling this command.",
	Example: fmt.Sprintf("%s module-hash-by-height 16841115", version.AppName),
	Args: cobra.ExactArgs(1),
	RunE: func(cmd *cobra.Command, args []string) error {
	heightToRetrieveString := args[0]
	serverCtx := GetServerContextFromCmd(cmd)
	height, err := strconv.ParseInt(heightToRetrieveString, 10, 64)
	if err != nil {
	return fmt.Errorf("invalid height: %w", err)
	}
	commitInfoForHeight, err := getModuleHashesAtHeight(serverCtx, appCreator, height)
	if err != nil {
	return err
	}
	clientCtx := client.GetClientContextFromCmd(cmd)
	return clientCtx.PrintProto(commitInfoForHeight)
	},
	}
	flags.AddQueryFlagsToCmd(cmd)
	return cmd
	}

/ #20779

We should add this command in server/v2/store component: https://github.com/cosmos/cosmos-sdk/tree/main/server/v2/store

[DioBr4nd0]

Hey @julienrbrt I want to work on this issue , can you please assign this one to me.

[julienrbrt]

Hey! That's great thanks!

[DioBr4nd0]

Hey @julienrbrt , I was working on this issue from the pull of the main branch but couldn't find the file server/module_hash.go in the main branch, then I tried to add it to server but there are some functions that are also not present in the code , Like the GetServerContextFromCmd also it's type is invalid, I also went through changelogs and found it got replaced. So do I need to pull any other branch for this code to be present . Please clarify these points.
Thanks

[julienrbrt]

Hey that code just got deleted from main, you can find it on main in commits from two days ago. Or release/v0.52.x branch.

Additionally it needs to use the server/v2 APIs yes, so I'd invite you to check that package, it should have everything you need.

[DioBr4nd0]

sure there are some functions, types like GetServerContextFromCmd and those functions and data members are already deleted, am I doing something wrong? Apologies for asking again and again @julienrbrt