From 710dcf3a02e557780fd1037a71ccc72bf7b3725f Mon Sep 17 00:00:00 2001
From: Joel Wetzell <git@jwetzell.com>
Date: Mon, 29 Dec 2025 16:07:56 -0600
Subject: [PATCH] change logging flags

---
 .vscode/launch.json    |  2 +-
 cmd/showbridge/main.go | 54 ++++++++++++++++++++++++++++++++++--------
 2 files changed, 45 insertions(+), 11 deletions(-)

diff --git a/.vscode/launch.json b/.vscode/launch.json
index f429342..01b2c20 100644
--- a/.vscode/launch.json
+++ b/.vscode/launch.json
@@ -11,7 +11,7 @@
             "request": "launch",
             "mode": "auto",
             "program": "cmd/showbridge",
-            "args": ["--debug"],
+            "args": ["--log-level", "debug"],
             "cwd": "./"
         }
     ]
diff --git a/cmd/showbridge/main.go b/cmd/showbridge/main.go
index 5aa1c9f..e691015 100644
--- a/cmd/showbridge/main.go
+++ b/cmd/showbridge/main.go
@@ -3,9 +3,11 @@ package main
 import (
 	"context"
 	"errors"
+	"fmt"
 	"log/slog"
 	"os"
 	"os/signal"
+	"slices"
 	"sync"
 
 	"github.com/jwetzell/showbridge-go"
@@ -29,15 +31,29 @@ func main() {
 				Value: "./config.yaml",
 				Usage: "path to config file",
 			},
-			&cli.BoolFlag{
-				Name:  "debug",
-				Value: false,
+			&cli.StringFlag{
+				Name:  "log-level",
+				Value: "debug",
 				Usage: "set log level to DEBUG",
+				Validator: func(level string) error {
+					levels := []string{"debug", "info", "warn", "error"}
+					if !slices.Contains(levels, level) {
+						return fmt.Errorf("unknown log level: %s", level)
+					}
+					return nil
+				},
 			},
-			&cli.BoolFlag{
-				Name:  "json",
-				Value: false,
-				Usage: "log using JSON",
+			&cli.StringFlag{
+				Name:  "log-format",
+				Value: "text",
+				Usage: "log format to use",
+				Validator: func(format string) error {
+					formats := []string{"text", "json"}
+					if !slices.Contains(formats, format) {
+						return fmt.Errorf("unknown log format: %s", format)
+					}
+					return nil
+				},
 			},
 		},
 		Action: run,
@@ -83,8 +99,19 @@ func run(ctx context.Context, c *cli.Command) error {
 
 	logLevel := slog.LevelInfo
 
-	if c.Bool("debug") {
+	logLevelFromFlag := c.String("log-level")
+
+	switch logLevelFromFlag {
+	case "debug":
 		logLevel = slog.LevelDebug
+	case "info":
+		logLevel = slog.LevelInfo
+	case "warn":
+		logLevel = slog.LevelWarn
+	case "error":
+		logLevel = slog.LevelError
+	default:
+		logLevel = slog.LevelInfo
 	}
 
 	logHandlerOptions := &slog.HandlerOptions{
@@ -93,10 +120,17 @@ func run(ctx context.Context, c *cli.Command) error {
 
 	logOutput := os.Stderr
 
-	var logHandler slog.Handler = slog.NewTextHandler(logOutput, logHandlerOptions)
+	var logHandler slog.Handler
 
-	if c.Bool("json") {
+	logFormat := c.String("log-format")
+
+	switch logFormat {
+	case "json":
 		logHandler = slog.NewJSONHandler(logOutput, logHandlerOptions)
+	case "text":
+		logHandler = slog.NewTextHandler(logOutput, logHandlerOptions)
+	default:
+		logHandler = slog.NewTextHandler(logOutput, logHandlerOptions)
 	}
 
 	slog.SetDefault(slog.New(logHandler))