fix logger cli

beacond/cmd/main.go

@@ -65,7 +65,6 @@ func run() error {
 				clicomponents.DefaultClientComponents(),
 				// TODO: remove these, and eventually pull cfg and chainspec
 				// from built node
-				nodecomponents.ProvideConfig,
 				nodecomponents.ProvideChainSpec,
 			),
 		),

mod/cli/pkg/components/logger.go

@@ -30,7 +30,6 @@ import (
 
 type LoggerInput struct {
 	depinject.In
-
 	Out io.Writer
 }
 
@@ -39,6 +38,8 @@ type LoggerInput struct {
 func ProvideLogger(
 	in LoggerInput,
 ) *phuslu.Logger {
+	// the logger config should be passed in here, but it is not yet populated
+	// so we pass in nil for now to get the default logger.
 	logger := phuslu.NewLogger(in.Out, nil)
 	logger.AddKeyColor("error", log.Red)
 	logger.AddKeyColor("err", log.Red)

mod/node-core/pkg/builder/builder.go

@@ -26,6 +26,7 @@ import (
 	"cosmossdk.io/depinject"
 	sdklog "cosmossdk.io/log"
 	servertypes "github.com/berachain/beacon-kit/mod/cli/pkg/commands/server/types"
+	"github.com/berachain/beacon-kit/mod/config"
 	cometbft "github.com/berachain/beacon-kit/mod/consensus/pkg/cometbft/service"
 	"github.com/berachain/beacon-kit/mod/log"
 	"github.com/berachain/beacon-kit/mod/node-core/pkg/types"
@@ -72,7 +73,7 @@ func New[
 // build a new instance of the node.
 // It is necessary to adhere to the types.AppCreator[T] interface.
 func (nb *NodeBuilder[NodeT, LoggerT, LoggerConfigT]) Build(
-	logger sdklog.Logger,
+	sdklogger sdklog.Logger,
 	db dbm.DB,
 	_ io.Writer,
 	cmtCfg *cmtcfg.Config,
@@ -85,6 +86,8 @@ func (nb *NodeBuilder[NodeT, LoggerT, LoggerConfigT]) Build(
 		}
 		beaconNode NodeT
 		cmtService *cometbft.Service
+		config     *config.Config
+		logger     = sdklogger.Impl().(LoggerT)
 	)
 
 	// build all node components using depinject
@@ -95,21 +98,20 @@ func (nb *NodeBuilder[NodeT, LoggerT, LoggerConfigT]) Build(
 			),
 			depinject.Supply(
 				appOpts,
-				logger.Impl().(LoggerT),
+				logger,
 				db,
 				cmtCfg,
 			),
-			// TODO: cosmos depinject bad project, fixed with dig.
-			// depinject.Invoke(
-			// 	SetLoggerConfig[LoggerT, LoggerConfigT],
-			// ),
 		),
-		&beaconNode,
 		&apiBackend,
+		&beaconNode,
 		&cmtService,
+		&config,
 	); err != nil {
 		panic(err)
 	}
+	// attach the logger config
+	logger.WithConfig(any(config.GetLogger()).(LoggerConfigT))
 
 	if apiBackend == nil {
 		panic("node or api backend is nil")