feat: impl slog.Handler for geth v1.13. used in json-rpc

These changes update Nibiru's integration with the `go-ethereum/log` package to align with its upstream migration to Go's standard structured logging library (`slog`). The previous logging setup in Nibiru, which relied on `go-ethereum/log`'s deprecated `FuncHandler` and `Record` types, was removed.

1.  **Added `LogHandler`:** A new file `app/server/geth_log_handler.go` introduces the `LogHandler` type. This type implements the standard `slog.Handler` interface. Its primary role is to receive log records generated by Geth components (which now use `slog`) and translate them into corresponding calls on Nibiru's standard `cmtlog.Logger` (CometBFT logger). It correctly maps Geth/`slog` levels (including `Trace` and `Crit`) and formats log attributes for compatibility.
2.  **Updated Initialization:** In `app/server/json_rpc.go`, the old `ethlog.Root().SetHandler(...)` block was replaced. The code now instantiates the new `LogHandler` (providing it the context logger `ctx.Logger.With("module", "geth")`), wraps it using `gethlog.NewLogger()`, and sets the result as the default logger for `go-ethereum` components via `gethlog.SetDefault()`.

The primary reason for this refactor was the breaking change in the `go-ethereum/log` dependency, which deprecated its custom logging implementation in favor of Go's standard `slog`. These changes adapt Nibiru to the new `slog`-based API, ensuring that logs generated within embedded Geth components are correctly captured and processed by Nibiru's existing logging infrastructure (`cmtlog.Logger`). This maintains consistent logging behavior and compatibility with the updated dependency.

Author: Unique-Divine

app/server/geth_log_handler.go

@@ -0,0 +1,109 @@
+package server
+
+import (
+	"context"
+	cmtlog "github.com/cometbft/cometbft/libs/log"
+	gethlog "github.com/ethereum/go-ethereum/log"
+
+	// Use "log/slog" from the Go std lib because Geth migrated to support
+	// slog and deprecated the original go-ethereum/log implementation.
+	// For more info on the migration, see https://github.com/ethereum/go-ethereum/pull/28187
+	"golang.org/x/exp/slog"
+)
+
+// Ensure LogHandler implements slog.Handler
+var _ slog.Handler = (*LogHandler)(nil)
+
+// LogHandler implements slog.Handler, which is needed to construct the
+// conventional go-ethereum.Logger, using a CometBFT logger
+// ("github.com/cometbft/cometbft/libs/log".Logger).
+type LogHandler struct {
+	CmtLogger cmtlog.Logger
+	attrs     []slog.Attr // Attributes gathered via WithAttrs
+	group     string      // Group name gathered via WithGroup (simple implementation)
+}
+
+// Enabled decides whether a log record should be processed.
+// We let the underlying CometBFT logger handle filtering.
+func (h *LogHandler) Enabled(_ context.Context, level slog.Level) bool {
+	// You could potentially check the CmtLogger's level here if needed,
+	// but returning true is usually sufficient.
+	return true
+}
+
+// Handle processes the log record and sends it to the CometBFT logger.
+func (h *LogHandler) Handle(_ context.Context, r slog.Record) error {
+	// 1. Determine the corresponding CometBFT log function
+	var logFunc func(msg string, keyvals ...interface{})
+	switch r.Level {
+	// Check against Geth's custom levels first if they exist
+	// This handler covers all defined slog and go-ethereum log levels.
+	case gethlog.LevelTrace, slog.LevelDebug: // Handles -8, -4
+		logFunc = h.CmtLogger.Debug
+	case slog.LevelInfo, slog.LevelWarn: // Handles 0, 4
+		logFunc = h.CmtLogger.Info
+	case gethlog.LevelCrit, slog.LevelError: // Handles 12, 8
+		// Map Geth Crit level along with standard Error
+		logFunc = h.CmtLogger.Error
+	default: // Handle any other levels (e.g., below Debug)
+		logFunc = h.CmtLogger.Debug // Default to Debug or Info as appropriate
+	}
+
+	// 2. Collect attributes (key-value pairs)
+	// Preallocate assuming 2 slots per attribute plus handler attrs
+	keyvals := make([]interface{}, 0, (r.NumAttrs()+len(h.attrs))*2)
+
+	// Add attributes stored in the handler first (from WithAttrs)
+	currentGroup := h.group
+	for _, attr := range h.attrs {
+		key := attr.Key
+		if currentGroup != "" {
+			key = currentGroup + "." + key // Basic grouping
+		}
+		keyvals = append(keyvals, key, attr.Value.Any())
+	}
+
+	// Add attributes from the specific log record
+	r.Attrs(func(attr slog.Attr) bool {
+		key := attr.Key
+		if currentGroup != "" {
+			key = currentGroup + "." + key // Basic grouping
+		}
+		keyvals = append(keyvals, key, attr.Value.Any())
+		return true // Continue iterating
+	})
+
+	// 3. Call the CometBFT logger function
+	logFunc(r.Message, keyvals...)
+
+	return nil // No error to return in this basic implementation
+}
+
+// WithAttrs returns a new LogHandler with the provided attributes added.
+func (h *LogHandler) WithAttrs(attrs []slog.Attr) slog.Handler {
+	// Create a new handler, cloning existing state and adding new attrs
+	newHandler := &LogHandler{
+		CmtLogger: h.CmtLogger, // Keep the same underlying logger
+		// Important: Clone slices to avoid modifying the original handler's state
+		attrs: append(append([]slog.Attr{}, h.attrs...), attrs...),
+		group: h.group,
+	}
+	return newHandler
+}
+
+// WithGroup returns a new LogHandler associated with the specified group.
+func (h *LogHandler) WithGroup(name string) slog.Handler {
+	// Create a new handler, cloning attributes and setting/appending the group
+	newHandler := &LogHandler{
+		CmtLogger: h.CmtLogger,
+		attrs:     append([]slog.Attr{}, h.attrs...), // Clone attributes
+		// Basic implementation: Overwrites group. Could concatenate if nesting needed.
+		group: name,
+	}
+	// If nested groups are needed:
+	// if h.group != "" {
+	//  name = h.group + "." + name
+	// }
+	// newHandler.group = name
+	return newHandler
+}

app/server/json_rpc.go

@@ -14,8 +14,8 @@ import (
 	"github.com/cosmos/cosmos-sdk/client"
 	"github.com/cosmos/cosmos-sdk/server"
 	"github.com/cosmos/cosmos-sdk/server/types"
-	ethlog "github.com/ethereum/go-ethereum/log"
-	ethrpc "github.com/ethereum/go-ethereum/rpc"
+	gethlog "github.com/ethereum/go-ethereum/log"
+	gethrpc "github.com/ethereum/go-ethereum/rpc"
 
 	srvconfig "github.com/NibiruChain/nibiru/v2/app/server/config"
 )
@@ -31,20 +31,13 @@ func StartJSONRPC(
 ) (*http.Server, chan struct{}, error) {
 	tmWsClientForRPCApi := ConnectTmWS(tmRPCAddr, tmEndpoint, ctx.Logger)
 
-	logger := ctx.Logger.With("module", "geth")
-	ethlog.Root().SetHandler(ethlog.FuncHandler(func(r *ethlog.Record) error {
-		switch r.Lvl {
-		case ethlog.LvlTrace, ethlog.LvlDebug:
-			logger.Debug(r.Msg, r.Ctx...)
-		case ethlog.LvlInfo, ethlog.LvlWarn:
-			logger.Info(r.Msg, r.Ctx...)
-		case ethlog.LvlError, ethlog.LvlCrit:
-			logger.Error(r.Msg, r.Ctx...)
-		}
-		return nil
-	}))
+	// Configure the go-ethereum logger to sync with the ctx.Logger
+	gethLogger := gethlog.NewLogger(&LogHandler{
+		CmtLogger: ctx.Logger.With("module", "geth"),
+	})
+	gethlog.SetDefault(gethLogger)
 
-	rpcServer := ethrpc.NewServer()
+	rpcServer := gethrpc.NewServer()
 
 	allowUnprotectedTxs := config.JSONRPC.AllowUnprotectedTxs
 	rpcAPIArr := config.JSONRPC.API
@@ -53,7 +46,7 @@ func StartJSONRPC(
 
 	for _, api := range apis {
 		if err := rpcServer.RegisterName(api.Namespace, api.Service); err != nil {
-			ctx.Logger.Error(
+			gethLogger.Error(
 				"failed to register service in JSON RPC namespace",
 				"namespace", api.Namespace,
 				"service", api.Service,