avoid debug formating as possible

Author: olekukonko

cmd/csv2table/csv2table.go

@@ -51,7 +51,7 @@ func main() {
 	// Handle non-flag filename argument
 	if flag.NArg() > 0 {
 		*fileName = flag.Arg(0)
-		logger.Info("Using filename from argument: %s", *fileName)
+		logger.Infof("Using filename from argument: %s", *fileName)
 	}
 
 	// Determine input source
@@ -62,7 +62,7 @@ func main() {
 		logger.Info("Reading CSV from STDIN (pipe mode).")
 		inputReader = os.Stdin
 	} else if *fileName != "" {
-		logger.Info("Reading CSV from file: %s", *fileName)
+		logger.Infof("Reading CSV from file: %s", *fileName)
 		file, errFile := os.Open(*fileName)
 		if errFile != nil {
 			logger.Fatal("failed to open file '%s': %w", *fileName, errFile)
@@ -158,7 +158,7 @@ func process(r io.Reader) error {
 	case "none":
 		selectedSymbols = tw.NewSymbols(tw.StyleNone)
 	default:
-		logger.Warn(fmt.Sprintf("Unknown symbol style '%s', using default (Light).", *symbolStyle))
+		logger.Warnf("Unknown symbol style '%s', using default (Light).", *symbolStyle)
 		selectedSymbols = tw.NewSymbols(tw.StyleLight)
 	}
 
@@ -226,7 +226,7 @@ func process(r io.Reader) error {
 		fallthrough
 	default:
 		if *rendererType != "" && strings.ToLower(*rendererType) != "blueprint" {
-			logger.Warn(fmt.Sprintf("Unknown renderer type '%s', using Blueprint.", *rendererType))
+			logger.Warnf("Unknown renderer type '%s', using Blueprint.", *rendererType)
 		}
 		selectedRenderer = renderer.NewBlueprint(baseRendition)
 	}
@@ -243,7 +243,7 @@ func process(r io.Reader) error {
 		// If termSize fails or is 0, calculatedMaxWidth remains 0 (content-based width)
 	}
 	if calculatedMaxWidth > 0 {
-		logger.Info(fmt.Sprintf("Calculated table max width: %d", calculatedMaxWidth))
+		logger.Infof("Calculated table max width: %d", calculatedMaxWidth)
 	}
 
 	tableOpts := []tablewriter.Option{
@@ -315,7 +315,7 @@ func process(r io.Reader) error {
 		if maxCols == 0 && len(headerData) > 0 { // Only header was present
 			maxCols = len(headerData)
 		}
-		logger.Info(fmt.Sprintf("Inferred max columns: %d", maxCols))
+		logger.Infof("Inferred max columns: %d", maxCols)
 
 		// Normalize header
 		if *header && len(headerData) > 0 {

config.go

@@ -671,7 +671,7 @@ func WithAutoHide(state tw.State) Option {
 	return func(target *Table) {
 		target.config.Behavior.AutoHide = state
 		if target.logger != nil {
-			target.logger.Debug("Option: WithAutoHide applied to Table: %v", state)
+			target.logger.Debugf("Option: WithAutoHide applied to Table: %v", state)
 		}
 	}
 }
@@ -684,7 +684,7 @@ func WithBorders(borders tw.Border) Option {
 			cfg := target.renderer.Config()
 			cfg.Borders = borders
 			if target.logger != nil {
-				target.logger.Debug("Option: WithBorders applied to Table: %+v", borders)
+				target.logger.Debugf("Option: WithBorders applied to Table: %+v", borders)
 			}
 		}
 	}
@@ -699,7 +699,7 @@ func WithColumnMax(width int) Option {
 		}
 		target.config.Stream.Widths.Global = width
 		if target.logger != nil {
-			target.logger.Debug("Option: WithColumnMax applied to Table: %v", width)
+			target.logger.Debugf("Option: WithColumnMax applied to Table: %v", width)
 		}
 	}
 }
@@ -713,7 +713,7 @@ func WithTableMax(width int) Option {
 		}
 		target.config.MaxWidth = width
 		if target.logger != nil {
-			target.logger.Debug("Option: WithTableMax applied to Table: %v", width)
+			target.logger.Debugf("Option: WithTableMax applied to Table: %v", width)
 		}
 	}
 }
@@ -729,7 +729,7 @@ func WithColumnWidths(widths map[int]int) Option {
 		}
 		target.config.Stream.Widths.PerColumn = widths
 		if target.logger != nil {
-			target.logger.Debug("Option: WithColumnWidths applied to Table: %v", widths)
+			target.logger.Debugf("Option: WithColumnWidths applied to Table: %v", widths)
 		}
 	}
 }
@@ -776,7 +776,7 @@ func WithFooterMergeMode(mergeMode int) Option {
 		}
 		target.config.Footer.Formatting.MergeMode = mergeMode
 		if target.logger != nil {
-			target.logger.Debug("Option: WithFooterMergeMode applied to Table: %v", mergeMode)
+			target.logger.Debugf("Option: WithFooterMergeMode applied to Table: %v", mergeMode)
 		}
 	}
 }
@@ -797,7 +797,7 @@ func WithHeaderAlignment(align tw.Align) Option {
 		}
 		target.config.Header.Formatting.Alignment = align
 		if target.logger != nil {
-			target.logger.Debug("Option: WithHeaderAlignment applied to Table: %v", align)
+			target.logger.Debugf("Option: WithHeaderAlignment applied to Table: %v", align)
 		}
 	}
 }
@@ -833,7 +833,7 @@ func WithRenderer(f tw.Renderer) Option {
 	return func(target *Table) {
 		target.renderer = f
 		if target.logger != nil {
-			target.logger.Debug("Option: WithRenderer applied to Table: %T", f)
+			target.logger.Debugf("Option: WithRenderer applied to Table: %T", f)
 			f.Logger(target.logger)
 		}
 	}
@@ -847,7 +847,7 @@ func WithRendererSettings(settings tw.Settings) Option {
 			cfg := target.renderer.Config()
 			cfg.Settings = settings
 			if target.logger != nil {
-				target.logger.Debug("Option: WithRendererSettings applied to Table: %+v", settings)
+				target.logger.Debugf("Option: WithRendererSettings applied to Table: %+v", settings)
 			}
 		}
 	}
@@ -873,7 +873,7 @@ func WithRowMaxWidth(maxWidth int) Option {
 		}
 		target.config.Row.ColMaxWidths.Global = maxWidth
 		if target.logger != nil {
-			target.logger.Debug("Option: WithRowMaxWidth applied to Table: %v", maxWidth)
+			target.logger.Debugf("Option: WithRowMaxWidth applied to Table: %v", maxWidth)
 		}
 	}
 }
@@ -928,7 +928,7 @@ func WithTrimSpace(state tw.State) Option {
 	return func(target *Table) {
 		target.config.Behavior.TrimSpace = state
 		if target.logger != nil {
-			target.logger.Debug("Option: WithAutoHide applied to Table: %v", state)
+			target.logger.Debugf("Option: WithAutoHide applied to Table: %v", state)
 		}
 	}
 }
@@ -954,9 +954,9 @@ func WithRendition(rendition tw.Rendition) Option {
 
 		if ru, ok := target.renderer.(tw.Renditioning); ok {
 			ru.Rendition(rendition)
-			target.logger.Debug("Option: WithRendition: Applied to renderer via Renditioning.SetRendition(): %+v", rendition)
+			target.logger.Debugf("Option: WithRendition: Applied to renderer via Renditioning.SetRendition(): %+v", rendition)
 		} else {
-			target.logger.Warn("Option: WithRendition: Current renderer type %T does not implement tw.Renditioning. Rendition may not be applied as expected.", target.renderer)
+			target.logger.Warnf("Option: WithRendition: Current renderer type %T does not implement tw.Renditioning. Rendition may not be applied as expected.", target.renderer)
 		}
 	}
 }

csv.go

@@ -44,7 +44,7 @@ func NewCSVReader(writer io.Writer, csvReader *csv.Reader, hasHeader bool, opts
 				return t, nil
 			}
 			// Log other read errors
-			t.logger.Error("NewCSVReader: Error reading CSV header: %v", err)
+			t.logger.Errorf("NewCSVReader: Error reading CSV header: %v", err)
 			return nil, err // Return nil *Table on critical read error
 		}
 
@@ -59,7 +59,7 @@ func NewCSVReader(writer io.Writer, csvReader *csv.Reader, hasHeader bool, opts
 
 		if !isEmptyHeader {
 			t.Header(headers) // Use the Table method to set the header data
-			t.logger.Debug("NewCSVReader: Header set from CSV: %v", headers)
+			t.logger.Debugf("NewCSVReader: Header set from CSV: %v", headers)
 		} else {
 			t.logger.Debug("NewCSVReader: Read an empty header line, skipping setting table header.")
 		}
@@ -74,20 +74,20 @@ func NewCSVReader(writer io.Writer, csvReader *csv.Reader, hasHeader bool, opts
 		}
 		if err != nil {
 			// Log other read errors during data processing
-			t.logger.Error("NewCSVReader: Error reading CSV record: %v", err)
+			t.logger.Errorf("NewCSVReader: Error reading CSV record: %v", err)
 			return nil, err // Return nil *Table on critical read error
 		}
 
 		// Append the record to the table's internal buffer (for batch rendering).
 		// The Table.Append method handles conversion and storage.
 		if appendErr := t.Append(record); appendErr != nil {
-			t.logger.Error("NewCSVReader: Error appending record #%d: %v", rowCount+1, appendErr)
+			t.logger.Errorf("NewCSVReader: Error appending record #%d: %v", rowCount+1, appendErr)
 			// Decide if append error is fatal. For now, let's treat it as fatal.
 			return nil, appendErr
 		}
 		rowCount++
 	}
-	t.logger.Debug("NewCSVReader: Finished reading CSV. Appended %d data rows.", rowCount)
+	t.logger.Debugf("NewCSVReader: Finished reading CSV. Appended %d data rows.", rowCount)
 
 	// Return the configured and populated table instance, ready for Render() call.
 	return t, nil

go.mod

@@ -11,7 +11,7 @@ require (
 require (
 	github.com/mattn/go-colorable v0.1.13 // indirect
 	github.com/mattn/go-isatty v0.0.19 // indirect
-	github.com/olekukonko/ll v0.0.6-0.20250513034024-62c5c2714b38 // indirect
+	github.com/olekukonko/ll v0.0.7 // indirect
 	github.com/olekukonko/ts v0.0.0-20171002115256-78ecb04241c0 // indirect
 	github.com/rivo/uniseg v0.2.0 // indirect
 	golang.org/x/sys v0.12.0 // indirect

go.sum

@@ -33,6 +33,8 @@ github.com/olekukonko/ll v0.0.6-0.20250513001635-5dbed4661872 h1:44lc4ASGSoHddTt
 github.com/olekukonko/ll v0.0.6-0.20250513001635-5dbed4661872/go.mod h1:En+sEW0JNETl26+K8eZ6/W4UQ7CYSrrgg/EdIYT2H8g=
 github.com/olekukonko/ll v0.0.6-0.20250513034024-62c5c2714b38 h1:Lnuzuijce7bnRJnlns11a2mUTW9lMF5kUW+ZriZh/hI=
 github.com/olekukonko/ll v0.0.6-0.20250513034024-62c5c2714b38/go.mod h1:En+sEW0JNETl26+K8eZ6/W4UQ7CYSrrgg/EdIYT2H8g=
+github.com/olekukonko/ll v0.0.7 h1:K66xcUlG2qWRhPoLw/cidmbv4pDDJtZuvJGsR5QTzXo=
+github.com/olekukonko/ll v0.0.7/go.mod h1:En+sEW0JNETl26+K8eZ6/W4UQ7CYSrrgg/EdIYT2H8g=
 github.com/olekukonko/ts v0.0.0-20171002115256-78ecb04241c0 h1:LiZB1h0GIcudcDci2bxbqI6DXV8bF8POAnArqvRrIyw=
 github.com/olekukonko/ts v0.0.0-20171002115256-78ecb04241c0/go.mod h1:F/7q8/HZz+TXjlsoZQQKVYvXTZaFH4QRa3y+j1p7MS0=
 github.com/rivo/uniseg v0.2.0 h1:S1pD9weZBuJdFmowNwbpi7BJ8TNftyUImj/0WQi72jY=