remove old system artifacts

Author: breardon2011

backend/controllers/github.go

@@ -441,7 +441,7 @@ func handlePushEvent(ctx context.Context, gh utils.GithubClientProvider, payload
 if repoCacheEnabled == "1" && strings.HasSuffix(ref, defaultBranch) {
 go func(ctx context.Context) {
 if err := sendProcessCacheRequest(repoFullName, defaultBranch, installationId); err != nil {
-logging.Error("Failed to process cache request", map[string]any{
+slog.Error("Failed to process cache request", map[string]any{
 "error": err,
 "repo_full_name": repoFullName,
 })

backend/controllers/projects.go

@@ -793,7 +793,7 @@ func (d DiggerController) SetJobStatusForProject(c *gin.Context) {
 }()
 defer logging.InheritRequestLogger(ctx)()
 
-logging.Debug("Starting post-success job processing", "job_id", jobId)
+slog.Debug("Starting post-success job processing", "job_id", jobId)
 
 ghClientProvider := d.GithubClientProvider
 installationLink, err := models.DB.GetGithubInstallationLinkForOrg(orgId)

backend/logging/context_logger.go

@@ -224,51 +224,10 @@ func InheritRequestLogger(ctx context.Context) (cleanup func()) {
 return func() {}
 }
 
-// Smart logging functions that can handle multiple signatures
-// These are the ONLY logging functions we need - they handle all cases
-func Info(msg string, args ...any) {
-ctx, attrs := parseLogArgs(args...)
-if ctx != nil {
-From(ctx).Info(msg, attrs...)
-} else if logger := getGoroutineLogger(); logger != nil {
-logger.Info(msg, attrs...)
-} else {
-slog.Info(msg, attrs...)
-}
-}
 
-func Error(msg string, args ...any) {
-ctx, attrs := parseLogArgs(args...)
-if ctx != nil {
-From(ctx).Error(msg, attrs...)
-} else if logger := getGoroutineLogger(); logger != nil {
-logger.Error(msg, attrs...)
-} else {
-slog.Error(msg, attrs...)
-}
-}
 
-func Debug(msg string, args ...any) {
-ctx, attrs := parseLogArgs(args...)
-if ctx != nil {
-From(ctx).Debug(msg, attrs...)
-} else if logger := getGoroutineLogger(); logger != nil {
-logger.Debug(msg, attrs...)
-} else {
-slog.Debug(msg, attrs...)
-}
-}
 
-func Warn(msg string, args ...any) {
-ctx, attrs := parseLogArgs(args...)
-if ctx != nil {
-From(ctx).Warn(msg, attrs...)
-} else if logger := getGoroutineLogger(); logger != nil {
-logger.Warn(msg, attrs...)
-} else {
-slog.Warn(msg, attrs...)
-}
-}
+
 
 // parseLogArgs intelligently parses variadic arguments to extract context and attributes
 func parseLogArgs(args ...any) (context.Context, []any) {
@@ -303,7 +262,3 @@ func parseLogArgs(args ...any) (context.Context, []any) {
 return ctx, attrs
 }
 
-// Backward compatibility functions (optional - can be removed if not needed)
-func InfoMsg(msg string, attrs ...any) { slog.Default().Info(msg, attrs...) }
-func ErrorMsg(msg string, attrs ...any) { slog.Default().Error(msg, attrs...) }
-func DebugMsg(msg string, attrs ...any) { slog.Default().Debug(msg, attrs...) }

backend/models/setup.go

@@ -36,7 +36,7 @@ func ConnectDatabase() {
 Logger: glogger,
 })
 if err != nil {
-logging.Error("Failed to connect to database", "error", err)
+slog.Error("Failed to connect to database", "error", err)
 panic("Failed to connect to database!")
 }
 
@@ -45,10 +45,10 @@ func ConnectDatabase() {
 // data and fixtures added
 orgNumberOne, err := DB.GetOrganisation(DEFAULT_ORG_NAME)
 if orgNumberOne == nil {
-logging.Info("No default organization found, creating default organisation", "name", DEFAULT_ORG_NAME)
+slog.Info("No default organization found, creating default organisation", "name", DEFAULT_ORG_NAME)
 _, err := DB.CreateOrganisation("digger", "", DEFAULT_ORG_NAME)
 if err != nil {
-logging.Error("Failed to create default organization", "error", err)
+slog.Error("Failed to create default organization", "error", err)
 }
 }
 }