refactor: a better DI-friendy logger implementation that doesn't suck

backend/internal/logger/logger.go

@@ -8,40 +8,36 @@ import (
 	"go.uber.org/zap"
 )
 
-type Logger interface {
-	Get() *zap.Logger	
-	Sync() error
-}
-
-type loggerImpl struct {
-}
-
-func NewLogger() Logger {
-	return &loggerImpl{}
-}
-
 var (
-	logger *zap.Logger
-	once   sync.Once
+	instance *zap.Logger
+	once     sync.Once
 )
 
-func (l *loggerImpl) Get() *zap.Logger {
+func NewLogger() *zap.Logger {
 	once.Do(func() {
-		var err error
 		cfg := config.GetConfig()
+		var err error
 
 		if cfg.Environment == "production" {
-			logger, err = zap.NewProduction()
+			instance, err = zap.NewProduction()
 		} else {
-			logger, err = zap.NewDevelopment()
+			instance, err = zap.NewDevelopment()
 		}
 		if err != nil {
-			panic(err)
+			panic("failed to initialize logger: " + err.Error())
 		}
 	})
-	return logger
+	return instance
 }
 
-func (l *loggerImpl) Sync() error {
-	return logger.Sync()
+type SyncLogger struct {
+	*zap.Logger
+}
+
+func NewSyncLogger(logger *zap.Logger) *SyncLogger {
+	return &SyncLogger{logger}
+}
+
+func (s *SyncLogger) Close() error {
+	return s.Sync()
 }

backend/internal/services/auth.go

@@ -3,7 +3,6 @@ package services
 import (
 	"easywish/internal/database"
 	errs "easywish/internal/errors"
-	"easywish/internal/logger"
 	"easywish/internal/models"
 	"easywish/internal/utils"
 
@@ -18,11 +17,11 @@ type AuthService interface {
 }
 
 type authServiceImpl struct {
-	log logger.Logger
+	log *zap.Logger
 	dbctx database.DbContext
 }
 
-func NewAuthService(_log logger.Logger, _dbctx database.DbContext) AuthService {
+func NewAuthService(_log *zap.Logger, _dbctx database.DbContext) AuthService {
 	return &authServiceImpl{log: _log, dbctx: _dbctx}
 }
 
@@ -33,10 +32,10 @@ func (a *authServiceImpl) RegistrationBegin(request models.RegistrationBeginRequ
 	user, err := db.TXQueries.CreateUser(db.CTX, request.Username) // TODO: validation
 
   if err != nil {
-		a.log.Get().Error("Failed to add user to database", zap.Error(err))
+		a.log.Error("Failed to add user to database", zap.Error(err))
 		return false, errs.ErrServerError
 	}
-	a.log.Get().Info("Registered a new user", zap.String("username", user.Username))
+	a.log.Info("Registered a new user", zap.String("username", user.Username))
 
 	helper.Commit()