feat: align Logger interface and rename constructors

examples/hello-mysql/cmd/api/startup.go

@@ -8,7 +8,7 @@ import (
 
 func logStartup(logger modkitlogging.Logger, addr string) {
 	if logger == nil {
-		logger = modkitlogging.Nop()
+		logger = modkitlogging.NewNopLogger()
 	}
 	logger = logger.With(slog.String("scope", "api"))
 	logger.Info("server starting",

examples/hello-mysql/cmd/api/startup_test.go

@@ -12,17 +12,32 @@ type captureLogger struct {
 	attrs    []slog.Attr
 }
 
-func (c *captureLogger) Debug(string, ...slog.Attr) {}
-func (c *captureLogger) Info(msg string, attrs ...slog.Attr) {
+func (c *captureLogger) Debug(string, ...any) {}
+func (c *captureLogger) Info(msg string, args ...any) {
 	c.messages = append(c.messages, msg)
-	c.attrs = append(c.attrs, attrs...)
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 }
-func (c *captureLogger) Error(string, ...slog.Attr) {}
-func (c *captureLogger) With(attrs ...slog.Attr) modkitlogging.Logger {
-	c.attrs = append(c.attrs, attrs...)
+func (c *captureLogger) Warn(string, ...any)  {}
+func (c *captureLogger) Error(string, ...any) {}
+func (c *captureLogger) With(args ...any) modkitlogging.Logger {
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 	return c
 }
 
+func attrsFromArgs(args []any) []slog.Attr {
+	if len(args) == 0 {
+		return nil
+	}
+	attrs := make([]slog.Attr, 0, len(args))
+	for _, arg := range args {
+		attr, ok := arg.(slog.Attr)
+		if ok {
+			attrs = append(attrs, attr)
+		}
+	}
+	return attrs
+}
+
 func TestLogStartup_EmitsMessage(t *testing.T) {
 	logger := &captureLogger{}
 	logStartup(logger, ":8080")

examples/hello-mysql/cmd/migrate/debug.go

@@ -8,7 +8,7 @@ import (
 
 func logMigrateDebug(logger modkitlogging.Logger, msg string) {
 	if logger == nil {
-		logger = modkitlogging.Nop()
+		logger = modkitlogging.NewNopLogger()
 	}
 	logger.Debug(msg, slog.String("scope", "migrate"))
 }

examples/hello-mysql/cmd/migrate/debug_test.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"log/slog"
 	"testing"
 
 	modkitlogging "github.com/aryeko/modkit/modkit/logging"
@@ -11,12 +10,13 @@ type debugCaptureLogger struct {
 	debugMessages []string
 }
 
-func (c *debugCaptureLogger) Debug(msg string, _ ...slog.Attr) {
+func (c *debugCaptureLogger) Debug(msg string, _ ...any) {
 	c.debugMessages = append(c.debugMessages, msg)
 }
-func (c *debugCaptureLogger) Info(string, ...slog.Attr)              {}
-func (c *debugCaptureLogger) Error(string, ...slog.Attr)             {}
-func (c *debugCaptureLogger) With(...slog.Attr) modkitlogging.Logger { return c }
+func (c *debugCaptureLogger) Info(string, ...any)              {}
+func (c *debugCaptureLogger) Warn(string, ...any)              {}
+func (c *debugCaptureLogger) Error(string, ...any)             {}
+func (c *debugCaptureLogger) With(...any) modkitlogging.Logger { return c }
 
 func TestLogMigrateDebug(t *testing.T) {
 	logger := &debugCaptureLogger{}

examples/hello-mysql/cmd/migrate/logging.go

@@ -8,7 +8,7 @@ import (
 
 func logMigrateComplete(logger modkitlogging.Logger) {
 	if logger == nil {
-		logger = modkitlogging.Nop()
+		logger = modkitlogging.NewNopLogger()
 	}
 	logger = logger.With(slog.String("scope", "migrate"))
 	logger.Info("migrations complete")

examples/hello-mysql/cmd/migrate/logging_test.go

@@ -12,17 +12,32 @@ type captureLogger struct {
 	attrs    []slog.Attr
 }
 
-func (c *captureLogger) Debug(string, ...slog.Attr) {}
-func (c *captureLogger) Info(msg string, attrs ...slog.Attr) {
+func (c *captureLogger) Debug(string, ...any) {}
+func (c *captureLogger) Info(msg string, args ...any) {
 	c.messages = append(c.messages, msg)
-	c.attrs = append(c.attrs, attrs...)
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 }
-func (c *captureLogger) Error(string, ...slog.Attr) {}
-func (c *captureLogger) With(attrs ...slog.Attr) modkitlogging.Logger {
-	c.attrs = append(c.attrs, attrs...)
+func (c *captureLogger) Warn(string, ...any)  {}
+func (c *captureLogger) Error(string, ...any) {}
+func (c *captureLogger) With(args ...any) modkitlogging.Logger {
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 	return c
 }
 
+func attrsFromArgs(args []any) []slog.Attr {
+	if len(args) == 0 {
+		return nil
+	}
+	attrs := make([]slog.Attr, 0, len(args))
+	for _, arg := range args {
+		attr, ok := arg.(slog.Attr)
+		if ok {
+			attrs = append(attrs, attr)
+		}
+	}
+	return attrs
+}
+
 func TestLogMigrateComplete(t *testing.T) {
 	logger := &captureLogger{}
 	logMigrateComplete(logger)

examples/hello-mysql/cmd/seed/debug.go

@@ -8,7 +8,7 @@ import (
 
 func logSeedDebug(logger modkitlogging.Logger, msg string) {
 	if logger == nil {
-		logger = modkitlogging.Nop()
+		logger = modkitlogging.NewNopLogger()
 	}
 	logger.Debug(msg, slog.String("scope", "seed"))
 }

examples/hello-mysql/cmd/seed/debug_test.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"log/slog"
 	"testing"
 
 	modkitlogging "github.com/aryeko/modkit/modkit/logging"
@@ -11,12 +10,13 @@ type debugCaptureLogger struct {
 	debugMessages []string
 }
 
-func (c *debugCaptureLogger) Debug(msg string, _ ...slog.Attr) {
+func (c *debugCaptureLogger) Debug(msg string, _ ...any) {
 	c.debugMessages = append(c.debugMessages, msg)
 }
-func (c *debugCaptureLogger) Info(string, ...slog.Attr)              {}
-func (c *debugCaptureLogger) Error(string, ...slog.Attr)             {}
-func (c *debugCaptureLogger) With(...slog.Attr) modkitlogging.Logger { return c }
+func (c *debugCaptureLogger) Info(string, ...any)              {}
+func (c *debugCaptureLogger) Warn(string, ...any)              {}
+func (c *debugCaptureLogger) Error(string, ...any)             {}
+func (c *debugCaptureLogger) With(...any) modkitlogging.Logger { return c }
 
 func TestLogSeedDebug(t *testing.T) {
 	logger := &debugCaptureLogger{}

examples/hello-mysql/cmd/seed/logging.go

@@ -8,7 +8,7 @@ import (
 
 func logSeedComplete(logger modkitlogging.Logger) {
 	if logger == nil {
-		logger = modkitlogging.Nop()
+		logger = modkitlogging.NewNopLogger()
 	}
 	logger = logger.With(slog.String("scope", "seed"))
 	logger.Info("seed complete")

examples/hello-mysql/cmd/seed/logging_test.go

@@ -12,17 +12,32 @@ type captureLogger struct {
 	attrs    []slog.Attr
 }
 
-func (c *captureLogger) Debug(string, ...slog.Attr) {}
-func (c *captureLogger) Info(msg string, attrs ...slog.Attr) {
+func (c *captureLogger) Debug(string, ...any) {}
+func (c *captureLogger) Info(msg string, args ...any) {
 	c.messages = append(c.messages, msg)
-	c.attrs = append(c.attrs, attrs...)
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 }
-func (c *captureLogger) Error(string, ...slog.Attr) {}
-func (c *captureLogger) With(attrs ...slog.Attr) modkitlogging.Logger {
-	c.attrs = append(c.attrs, attrs...)
+func (c *captureLogger) Warn(string, ...any)  {}
+func (c *captureLogger) Error(string, ...any) {}
+func (c *captureLogger) With(args ...any) modkitlogging.Logger {
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 	return c
 }
 
+func attrsFromArgs(args []any) []slog.Attr {
+	if len(args) == 0 {
+		return nil
+	}
+	attrs := make([]slog.Attr, 0, len(args))
+	for _, arg := range args {
+		attr, ok := arg.(slog.Attr)
+		if ok {
+			attrs = append(attrs, attr)
+		}
+	}
+	return attrs
+}
+
 func TestLogSeedComplete(t *testing.T) {
 	logger := &captureLogger{}
 	logSeedComplete(logger)

examples/hello-mysql/internal/modules/users/service.go

@@ -22,7 +22,7 @@ type service struct {
 
 func NewService(repo Repository, logger modkitlogging.Logger) Service {
 	if logger == nil {
-		logger = modkitlogging.Nop()
+		logger = modkitlogging.NewNopLogger()
 	}
 	logger = logger.With(slog.String("scope", "users"))
 	return &service{repo: repo, logger: logger}

examples/hello-mysql/internal/modules/users/service_logging_test.go

@@ -13,17 +13,32 @@ type captureLogger struct {
 	attrs         []slog.Attr
 }
 
-func (c *captureLogger) Debug(msg string, attrs ...slog.Attr) {
+func (c *captureLogger) Debug(msg string, args ...any) {
 	c.debugMessages = append(c.debugMessages, msg)
-	c.attrs = append(c.attrs, attrs...)
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 }
-func (c *captureLogger) Info(string, ...slog.Attr)  {}
-func (c *captureLogger) Error(string, ...slog.Attr) {}
-func (c *captureLogger) With(attrs ...slog.Attr) modkitlogging.Logger {
-	c.attrs = append(c.attrs, attrs...)
+func (c *captureLogger) Info(string, ...any)  {}
+func (c *captureLogger) Warn(string, ...any)  {}
+func (c *captureLogger) Error(string, ...any) {}
+func (c *captureLogger) With(args ...any) modkitlogging.Logger {
+	c.attrs = append(c.attrs, attrsFromArgs(args)...)
 	return c
 }
 
+func attrsFromArgs(args []any) []slog.Attr {
+	if len(args) == 0 {
+		return nil
+	}
+	attrs := make([]slog.Attr, 0, len(args))
+	for _, arg := range args {
+		attr, ok := arg.(slog.Attr)
+		if ok {
+			attrs = append(attrs, attr)
+		}
+	}
+	return attrs
+}
+
 func TestService_EmitsDebugLogs(t *testing.T) {
 	repo := &stubRepo{}
 	logger := &captureLogger{}

examples/hello-mysql/internal/platform/logging/log.go

@@ -22,7 +22,7 @@ type config struct {
 
 func New() modkitlogging.Logger {
 	logger := newLogger(os.Stdout)
-	return modkitlogging.NewSlog(logger)
+	return modkitlogging.NewSlogLogger(logger)
 }
 
 func newLogger(w io.Writer) *slog.Logger {