address lgtm issue

Author: patrickblackjr

internal/githubapi/handler.go

@@ -97,6 +97,23 @@ func handlePullRequestEvent(event *github.PullRequestEvent, client *github.Clien
 	owner := event.GetRepo().GetOwner().GetLogin()
 	repo := event.GetRepo().GetName()
 	prNumber := event.GetPullRequest().GetNumber()
+	sha := event.GetPullRequest().GetHead().GetSHA()
+
+	if minApprovals == 0 {
+		if action == "reopened" {
+			if err := pullrequest.AddComment(owner, repo, prNumber, ApprovalResetComment, client, logger); err != nil {
+				logger.Error("failed to add comment", slog.String("error", err.Error()))
+			}
+		}
+		if _, _, err := client.Issues.AddLabelsToIssue(ctx, owner, repo, prNumber, []string{"lgtm"}); err != nil {
+			logger.Error("failed to add lgtm label", slog.String("error", err.Error()))
+			return
+		}
+		if _, err := checkrun.CreateCheckRun(owner, repo, sha, "success", "Approved and ready for merge", client, logger); err != nil {
+			logger.Error("failed to create check run", slog.String("error", err.Error()))
+		}
+		return
+	}
 
 	if _, _, err := client.Issues.AddLabelsToIssue(ctx, owner, repo, prNumber, []string{"do-not-merge"}); err != nil {
 		logger.Error("failed to add do-not-merge label", slog.String("error", err.Error()))
@@ -113,26 +130,6 @@ func handlePullRequestEvent(event *github.PullRequestEvent, client *github.Clien
 		}
 	}
 
-	sha, err := pullrequest.GetPRSHA(owner, repo, prNumber, client, logger)
-	if err != nil {
-		logger.Error("failed to get PR SHA", slog.String("error", err.Error()))
-		return
-	}
-
-	if minApprovals == 0 {
-		if _, _, err := client.Issues.AddLabelsToIssue(ctx, owner, repo, prNumber, []string{"lgtm"}); err != nil {
-			logger.Error("failed to add lgtm label", slog.String("error", err.Error()))
-			return
-		}
-		if _, err := client.Issues.RemoveLabelForIssue(ctx, owner, repo, prNumber, "do-not-merge"); err != nil {
-			logger.Warn("failed to remove do-not-merge label", slog.String("error", err.Error()))
-		}
-		if _, err := checkrun.CreateCheckRun(owner, repo, sha, "success", "Approved and ready for merge", client, logger); err != nil {
-			logger.Error("failed to create check run", slog.String("error", err.Error()))
-		}
-		return
-	}
-
 	if _, err := checkrun.CreateCheckRun(owner, repo, sha, "neutral", "Approval needed", client, logger); err != nil {
 		logger.Error("failed to create check run", slog.String("error", err.Error()))
 	}

internal/githubapi/pullrequest/pullrequest.go

@@ -7,15 +7,6 @@ import (
 	"github.com/google/go-github/v71/github"
 )
 
-func GetPRSHA(owner, repo string, prNumber int, client *github.Client, logger *slog.Logger) (string, error) {
-	pr, _, err := client.PullRequests.Get(context.Background(), owner, repo, prNumber)
-	if err != nil {
-		logger.Error("failed to get pull request", slog.String("error", err.Error()))
-		return "", err
-	}
-	return pr.GetHead().GetSHA(), nil
-}
-
 func RemoveLabel(owner, repo string, prNumber int, label string, client *github.Client, logger *slog.Logger) error {
 	_, err := client.Issues.RemoveLabelForIssue(context.Background(), owner, repo, prNumber, label)
 	if err != nil {

internal/githubapi/pullrequest/pullrequest_test.go

@@ -17,38 +17,6 @@ func setupLogger() *slog.Logger {
 	return slog.New(slog.NewTextHandler(os.Stdout, &slog.HandlerOptions{Level: slog.LevelDebug}))
 }
 
-func TestGetPRSHA_Success(t *testing.T) {
-	logger := setupLogger()
-	mockClient := github.NewClient(mock.NewMockedHTTPClient(
-		mock.WithRequestMatch(
-			mock.GetReposPullsByOwnerByRepoByPullNumber,
-			github.PullRequest{
-				Head: &github.PullRequestBranch{SHA: github.Ptr("test-sha")},
-			},
-		),
-	))
-
-	sha, err := pullrequest.GetPRSHA("owner", "repo", 1, mockClient, logger)
-	assert.NoError(t, err)
-	assert.Equal(t, "test-sha", sha)
-}
-
-func TestGetPRSHA_Failure(t *testing.T) {
-	logger := setupLogger()
-	mockClient := github.NewClient(mock.NewMockedHTTPClient(
-		mock.WithRequestMatchHandler(
-			mock.GetReposPullsByOwnerByRepoByPullNumber,
-			http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-				http.Error(w, "failed to get PR", http.StatusInternalServerError)
-			}),
-		),
-	))
-
-	sha, err := pullrequest.GetPRSHA("owner", "repo", 1, mockClient, logger)
-	assert.Error(t, err)
-	assert.Empty(t, sha)
-}
-
 func TestRemoveLabel_Success(t *testing.T) {
 	logger := setupLogger()
 	mockClient := github.NewClient(mock.NewMockedHTTPClient(