perf(github): skip comment edit when body is already up to date

theakshaypant · theakshaypant · commit 26121895cfbf · 2026-02-05T18:07:43.000+05:30
Avoid unnecessary GitHub API calls by checking if comment body
already matches before editing. Use github.Ptr() for consistency.

Signed-off-by: Akshay Pant &lt;akshay.akshaypant@gmail.com&gt;
diff --git a/pkg/provider/github/github.go b/pkg/provider/github/github.go
@@ -776,9 +776,13 @@ func (v *Provider) CreateComment(ctx context.Context, event *info.Event, commit,
 		re := regexp.MustCompile(regexp.QuoteMeta(updateMarker))
 		for _, comment := range comments {
 			if re.MatchString(comment.GetBody()) {
+				// No edit required if the comment is exactly the same.
+				if comment.GetBody() == commit {
+					return nil
+				}
 				if _, _, err := wrapAPI(v, "edit_comment", func() (*github.IssueComment, *github.Response, error) {
 					return v.Client().Issues.EditComment(ctx, event.Organization, event.Repository, comment.GetID(), &github.IssueComment{
-						Body: &commit,
+						Body: github.Ptr(commit),
 					})
 				}); err != nil {
 					return err
@@ -790,7 +794,7 @@ func (v *Provider) CreateComment(ctx context.Context, event *info.Event, commit,
 
 	_, _, err := wrapAPI(v, "create_comment", func() (*github.IssueComment, *github.Response, error) {
 		return v.Client().Issues.CreateComment(ctx, event.Organization, event.Repository, event.PullRequestNumber, &github.IssueComment{
-			Body: &commit,
+			Body: github.Ptr(commit),
 		})
 	})
 	return err
