feat: Add more pigeon status update fields (#1051)

* feat: Add more pigeon status update fields

* chore: remove deprecated creator field usage

* fix: val address parsing

* fix: get creator from metadata

proto/palomachain/paloma/paloma/tx.proto

@@ -17,10 +17,15 @@ message MsgAddStatusUpdate {
     LEVEL_INFO = 1;
     LEVEL_ERROR = 2;
   }
+  message KeyValuePair {
+      string key = 1;
+      string value = 2;
+  }
   string creator = 1 [deprecated = true];
   string status = 2;
   Level level = 3;
   palomachain.paloma.valset.MsgMetadata metadata = 4 [(gogoproto.nullable) = false];
+  repeated KeyValuePair args = 5 [(gogoproto.nullable) = false];
 }
 
 message EmptyResponse {}

x/paloma/keeper/msg_server.go

@@ -2,9 +2,11 @@ package keeper
 
 import (
 	"context"
+	"fmt"
 	"os"
 
 	sdk "github.com/cosmos/cosmos-sdk/types"
+	"github.com/palomachain/paloma/util/liblog"
 	"github.com/palomachain/paloma/x/paloma/types"
 )
 
@@ -31,24 +33,36 @@ func (k msgServer) AddStatusUpdate(goCtx context.Context, msg *types.MsgAddStatu
 		return &types.EmptyResponse{}, nil
 	}
 
-	creator, _ := sdk.AccAddressFromBech32(msg.Creator)
+	creator, err := sdk.AccAddressFromBech32(msg.GetMetadata().GetCreator())
+	if err != nil {
+		err = fmt.Errorf("failed to parse creator address: %w", err)
+		liblog.FromSDKLogger(k.Logger(ctx)).WithError(err).WithFields("component", "pigeon-status-update").Error("Failed to parse creator from message")
+		return nil, err
+	}
 	valAddr := sdk.ValAddress(creator.Bytes())
 	status := msg.GetStatus()
 
 	var logFn func(string, ...interface{})
 	switch msg.Level {
 	case types.MsgAddStatusUpdate_LEVEL_DEBUG:
-		logFn = k.Logger(ctx).Debug
+		logFn = liblog.FromSDKLogger(k.Logger(ctx)).Debug
 	case types.MsgAddStatusUpdate_LEVEL_INFO:
-		logFn = k.Logger(ctx).Info
+		logFn = liblog.FromSDKLogger(k.Logger(ctx)).Info
 	case types.MsgAddStatusUpdate_LEVEL_ERROR:
-		logFn = k.Logger(ctx).Error
+		logFn = liblog.FromSDKLogger(k.Logger(ctx)).Error
+	}
+
+	args := make([]interface{}, 0, len(msg.GetArgs())*2+6) // len: (k+v) + 6 for static appends
+	for _, v := range msg.GetArgs() {
+		args = append(args, fmt.Sprintf("msg.args.%s", v.GetKey()), v.GetValue())
 	}
 
-	logFn(status,
+	args = append(args,
 		"component", "pigeon-status-update",
 		"status", status,
 		"sender", valAddr)
 
+	logFn(status, args...)
+
 	return &types.EmptyResponse{}, nil
 }