refactor: standardize logging messages

Author: agfunderburg10

cmd/lambda/main.go

@@ -31,7 +31,7 @@ func handler(ctx context.Context, event events.SNSEvent) error {
 
 		appInst, initErr = app.New(ctx)
 		if initErr != nil {
-			logger.Error("failed to initialize app", "error", initErr)
+			logger.Error("failed to initialize app", slog.Any("error", initErr))
 			return
 		}
 

cmd/verify/main.go

@@ -32,13 +32,13 @@ func main() {
 	path := filepath.Join(*scenarioFile)
 	raw, err := os.ReadFile(path)
 	if err != nil {
-		logger.Error("failed to read scenarios file", slog.String("error", err.Error()))
+		logger.Error("failed to read scenarios file", slog.Any("error", err))
 		os.Exit(1)
 	}
 
 	var scenarios []TestScenario
 	if err := json.Unmarshal(raw, &scenarios); err != nil {
-		logger.Error("failed to parse scenarios", slog.String("error", err.Error()))
+		logger.Error("failed to parse scenarios", slog.Any("error", err))
 		os.Exit(1)
 	}
 

cmd/verify/scenario.go

@@ -70,7 +70,7 @@ func runScenario(ctx context.Context, scenario TestScenario, verbose bool, logge
 	go func() {
 		slackReady <- true
 		if err := slackServer.ListenAndServeTLS("", ""); err != http.ErrServerClosed {
-			logger.Error("slack mock server error", slog.String("error", err.Error()))
+			logger.Error("slack mock server error", slog.Any("error", err))
 		}
 	}()
 

internal/app/app.go

@@ -77,7 +77,7 @@ func (a *App) HandleSNSEvent(ctx context.Context, event events.SNSEvent) error {
 			a.logger.Error("failed to process message",
 				"message_id", msg.MessageID,
 				"topic_arn", msg.TopicArn,
-				"error", err)
+				slog.Any("error", err))
 		}
 	}