fix: review_requested uses normalized reviewers (fixes PR 184 slack notification)

2026-09-03 11:09:03 +00:00
parent 6337feeae1
commit ce845a7baf
1 changed files with 10 additions and 5 deletions
@@ -114,15 +114,20 @@ func (p *Processor) HandlePullRequest(e *webhook.PullRequestEvent) {

 	case "review_requested":
 		event.Type = TypePRReviewRequested
-		// Notify newly requested reviewers (array)
-		if len(e.RequestedReviewers) > 0 {
-			for _, reviewer := range e.RequestedReviewers {
-				p.logger.Debug().Str("requested_reviewer", reviewer.Login).Msg("review_requested event")
+		if len(event.Reviewers) > 0 {
+			for _, u := range event.Reviewers {
+				p.logger.Debug().Str("requested_reviewer", u.GiteaUsername).Msg("review_requested event")
 				usersToNotify = append(usersToNotify, struct {
 					user   User
 					reason NotificationReason
-				}{giteaUserToUser(reviewer), ReasonReviewer})
+				}{u, ReasonReviewer})
 			}
+		} else if e.RequestedReviewer != nil {
+			p.logger.Debug().Str("requested_reviewer", e.RequestedReviewer.Login).Msg("review_requested event")
+			usersToNotify = append(usersToNotify, struct {
+				user   User
+				reason NotificationReason
+			}{giteaUserToUser(*e.RequestedReviewer), ReasonReviewer})
 		} else {
 			p.logger.Debug().Msg("review_requested event but no RequestedReviewers field")
 		}