Add comments and remove unused constants.

x/ibcratelimit/errors.go

@@ -1,11 +1,11 @@
 package ibcratelimit
 
 import (
-	errorsmod "cosmossdk.io/errors"
+	cerrs "cosmossdk.io/errors"
 )
 
 var (
-	ErrRateLimitExceeded = errorsmod.Register(ModuleName, 2, "rate limit exceeded")
-	ErrBadMessage        = errorsmod.Register(ModuleName, 3, "bad message")
-	ErrContractError     = errorsmod.Register(ModuleName, 4, "contract error")
+	ErrRateLimitExceeded = cerrs.Register(ModuleName, 2, "rate limit exceeded")
+	ErrBadMessage        = cerrs.Register(ModuleName, 3, "bad message")
+	ErrContractError     = cerrs.Register(ModuleName, 4, "contract error")
 )

x/ibcratelimit/keys.go

@@ -1,9 +1,11 @@
 package ibcratelimit
 
 const (
-	ModuleName = "ratelimitedibc" // IBC at the end to avoid conflicts with the ibc prefix
-	StoreKey   = ModuleName
-	RouterKey  = ModuleName
+	// ModuleName defines the module name
+	ModuleName = "ratelimitedibc"
+
+	// StoreKey defines the primary module store key
+	StoreKey = ModuleName
 )
 
 var (

x/ibcratelimit/module/ibc_middleware.go

@@ -22,6 +22,7 @@ var (
 	_ porttypes.Middleware = &IBCMiddleware{}
 )
 
+// IBCMiddleware is the middleware used by the module.
 type IBCMiddleware struct {
 	app     porttypes.IBCModule
 	keeper  *keeper.Keeper

x/ibcratelimit/packet.go

@@ -12,30 +12,38 @@ import (
 )
 
 const (
+	// MsgSendPacket is the operation used for tracking a sent packet.
 	MsgSendPacket = "send_packet"
+	// MsgRecvPacket is the operation used for tracking a received packet.
 	MsgRecvPacket = "recv_packet"
 )
 
+// UndoSendMsg is an ibcratelimit contract message meant to undo tracked sends.
 type UndoSendMsg struct {
 	UndoSend UndoPacketMsg `json:"undo_send"`
 }
 
+// UndoPacketMsg is an operation done by the UndoSendMsg.
 type UndoPacketMsg struct {
 	Packet UnwrappedPacket `json:"packet"`
 }
 
+// SendPacketMsg is an ibcratelimit contract message meant to track sends.
 type SendPacketMsg struct {
 	SendPacket PacketMsg `json:"send_packet"`
 }
 
+// RecvPacketMsg is an ibcratelimit contract message meant to track receives.
 type RecvPacketMsg struct {
 	RecvPacket PacketMsg `json:"recv_packet"`
 }
 
+// PacketMsg contains
 type PacketMsg struct {
 	Packet UnwrappedPacket `json:"packet"`
 }
 
+// UnwrappedPacket is a FungibleTokenPacket.
 type UnwrappedPacket struct {
 	Sequence           uint64                                `json:"sequence"`
 	SourcePort         string                                `json:"source_port"`