Correctly hide logging when run with no logging flags

Author: Monster0506

cmd/meshexec/config.go

@@ -18,7 +18,12 @@ var configShowCmd = &cobra.Command{
 	Use:   "show",
 	Short: "Show current configuration",
 	Run: func(cmd *cobra.Command, args []string) {
-		manager := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		manager := config.NewManagerWithLevel(logLevel)
 
 		// Get config path from global flags
 		configPath, _ := cmd.Root().PersistentFlags().GetString("config")
@@ -47,7 +52,12 @@ var configInitCmd = &cobra.Command{
 	Use:   "init",
 	Short: "Initialize default configuration file",
 	Run: func(cmd *cobra.Command, args []string) {
-		manager := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		manager := config.NewManagerWithLevel(logLevel)
 
 		// Get config path from global flags
 		configPath, _ := cmd.Root().PersistentFlags().GetString("config")
@@ -78,7 +88,12 @@ var configValidateCmd = &cobra.Command{
 	Use:   "validate",
 	Short: "Validate configuration file",
 	Run: func(cmd *cobra.Command, args []string) {
-		manager := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		manager := config.NewManagerWithLevel(logLevel)
 		configPath, _ := cmd.Root().PersistentFlags().GetString("config")
 		if configPath != "" {
 			manager.SetConfigPath(configPath)

cmd/meshexec/daemon.go

@@ -24,7 +24,12 @@ var daemonCmd = &cobra.Command{
 		}
 
 		// Load configuration (if present) so we honor user/device settings early
-		manager := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		manager := config.NewManagerWithLevel(logLevel)
 		cfgPath, _ := cmd.Root().PersistentFlags().GetString("config")
 		if cfgPath != "" {
 			manager.SetConfigPath(cfgPath)

cmd/meshexec/network.go

@@ -17,7 +17,12 @@ var joinCmd = &cobra.Command{
 	Use:   "join",
 	Short: "Join the mesh network",
 	Run: func(cmd *cobra.Command, args []string) {
-		cfgMgr := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		cfgMgr := config.NewManagerWithLevel(logLevel)
 		cfgPath, _ := cmd.Root().PersistentFlags().GetString("config")
 		if cfgPath != "" {
 			cfgMgr.SetConfigPath(cfgPath)
@@ -75,7 +80,12 @@ var listCmd = &cobra.Command{
 			timeout = 5000 * time.Millisecond
 		}
 
-		cfgMgr := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		cfgMgr := config.NewManagerWithLevel(logLevel)
 		cfgPath, _ := cmd.Root().PersistentFlags().GetString("config")
 		if cfgPath != "" {
 			cfgMgr.SetConfigPath(cfgPath)
@@ -143,7 +153,12 @@ var statusCmd = &cobra.Command{
 		}
 
 		// Load configuration
-		cfgMgr := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		cfgMgr := config.NewManagerWithLevel(logLevel)
 		cfgPath, _ := cmd.Root().PersistentFlags().GetString("config")
 		if cfgPath != "" {
 			cfgMgr.SetConfigPath(cfgPath)

cmd/meshexec/run.go

@@ -30,7 +30,12 @@ var runCmd = &cobra.Command{
 	Example: "meshexec run -t \"os=linux && role=worker\" -- echo hello\nmeshexec run --target all -- uptime\nmeshexec run --dry-run -t 'arch=arm' -- cat /proc/cpuinfo",
 	Run: func(cmd *cobra.Command, args []string) {
 		// Load configuration
-		cfgMgr := config.NewManager()
+		logLevel, _ := cmd.Root().PersistentFlags().GetString("log-level")
+		verbose, _ := cmd.Root().PersistentFlags().GetBool("verbose")
+		if verbose {
+			logLevel = "debug"
+		}
+		cfgMgr := config.NewManagerWithLevel(logLevel)
 		cfgPath, _ := cmd.Root().PersistentFlags().GetString("config")
 		if cfgPath != "" {
 			cfgMgr.SetConfigPath(cfgPath)

internal/ble/manager.go

@@ -47,14 +47,10 @@ func NewManager(transport core.BLETransport, logger *logging.Logger) *Manager {
 
 // getTransportType returns a string representation of the transport type
 func getTransportType(transport core.BLETransport) string {
-	// Prefer precise identification via type switches on known implementations
+	// Prefer precise identification for the in-package simulated transport
 	switch transport.(type) {
 	case *Transport:
 		return "simulated"
-	case *SidecarTransport:
-		return "sidecar"
-	case *tgTransport:
-		return "tinygo"
 	}
 
 	// Fallback: use type name heuristics

internal/config/manager.go

@@ -403,4 +403,4 @@ func (m *Manager) CreateDefaultConfig() error {
 	m.logger.Info("Creating default configuration file", nil)
 	config := internal.DefaultConfig()
 	return m.Save(config)
-}
+}