improved error handling

motatoes · motatoes · commit c08f33cdf241 · 2024-12-30T09:33:51.000Z
diff --git a/backend/controllers/github.go b/backend/controllers/github.go
@@ -517,6 +517,7 @@ func handlePullRequestEvent(gh utils.GithubClientProvider, payload *github.PullR
 		if err != nil {
 			log.Printf("could not post ai summary comment: %v", err)
 			commentReporterManager.UpdateComment(fmt.Sprintf(":x: could not post ai comment summary comment id: %v", err))
+			return fmt.Errorf("could not post ai summary comment: %v", err)
 		}
 		aiSummaryCommentId = aiSummaryComment.Id
 	}
@@ -924,6 +925,7 @@ func handleIssueCommentEvent(gh utils.GithubClientProvider, payload *github.Issu
 		if err != nil {
 			log.Printf("could not post ai summary comment: %v", err)
 			commentReporterManager.UpdateComment(fmt.Sprintf(":x: could not post ai comment summary comment id: %v", err))
+			return fmt.Errorf("could not post ai summary comment: %v", err)
 		}
 		aiSummaryCommentId = aiSummaryComment.Id
 	}

Original file line number	Diff line number	Diff line change
`@@ -517,6 +517,7 @@ func handlePullRequestEvent(gh utils.GithubClientProvider, payload *github.PullR`
`517`	`517`	`if err != nil {`
`518`	`518`	`log.Printf("could not post ai summary comment: %v", err)`
`519`	`519`	`commentReporterManager.UpdateComment(fmt.Sprintf(":x: could not post ai comment summary comment id: %v", err))`
	`520`	`+ return fmt.Errorf("could not post ai summary comment: %v", err)`
`520`	`521`	`}`
`521`	`522`	`aiSummaryCommentId = aiSummaryComment.Id`
`522`	`523`	`}`
`@@ -924,6 +925,7 @@ func handleIssueCommentEvent(gh utils.GithubClientProvider, payload *github.Issu`
`924`	`925`	`if err != nil {`
`925`	`926`	`log.Printf("could not post ai summary comment: %v", err)`
`926`	`927`	`commentReporterManager.UpdateComment(fmt.Sprintf(":x: could not post ai comment summary comment id: %v", err))`
	`928`	`+ return fmt.Errorf("could not post ai summary comment: %v", err)`
`927`	`929`	`}`
`928`	`930`	`aiSummaryCommentId = aiSummaryComment.Id`
`929`	`931`	`}`