Feat/draft prs lock unlock fixes (#2290)

* draft prs lock and unlock fixing

* check explicitly for opened event

* ensure that the digger unlock success message is posted

Author: motatoes

backend/controllers/github_comment.go

@@ -193,18 +193,6 @@ func handleIssueCommentEvent(gh utils.GithubClientProvider, payload *github.Issu
 		slog.Debug("Added eyes reaction to comment", "commentId", commentIdStr)
 	}
 
-	if !config.AllowDraftPRs && isDraft {
-		slog.Info("Draft PRs are disabled, skipping",
-			"issueNumber", issueNumber,
-			"isDraft", isDraft,
-		)
-		if os.Getenv("DIGGER_REPORT_BEFORE_LOADING_CONFIG") == "1" {
-			// This one is for aggregate reporting
-			commentReporterManager.UpdateComment(":construction_worker: Ignoring event as it is a draft and draft PRs are configured to be ignored")
-		}
-		return nil
-	}
-
 	commentReporter, err := commentReporterManager.UpdateComment(":construction_worker: Digger starting.... config loaded successfully")
 	if err != nil {
 		slog.Error("Error initializing comment reporter",
@@ -339,6 +327,35 @@ func handleIssueCommentEvent(gh utils.GithubClientProvider, payload *github.Issu
 		return fmt.Errorf("error processing event")
 	}
 
+	if !config.AllowDraftPRs && isDraft {
+		slog.Info("Draft PRs are disabled, skipping",
+			"issueNumber", issueNumber,
+			"isDraft", isDraft,
+		)
+
+		if os.Getenv("DIGGER_REPORT_BEFORE_LOADING_CONFIG") == "1" {
+			// This one is for aggregate reporting
+			commentReporterManager.UpdateComment(":construction_worker: Ignoring event as it is a draft and draft PRs are configured to be ignored")
+		}
+
+		// special case to unlock all locks aquired by this PR
+		if *diggerCommand == scheduler.DiggerCommandUnlock {
+			err := models.DB.DeleteAllLocksAcquiredByPR(issueNumber, repoFullName, orgId)
+			if err != nil {
+				slog.Error("Failed to delete locks",
+					"prNumber", issueNumber,
+					"command", *diggerCommand,
+					"error", err,
+				)
+				commentReporterManager.UpdateComment(fmt.Sprintf(":x: Failed to delete locks: %v", err))
+				return fmt.Errorf("failed to delete locks: %v", err)
+			}
+			commentReporterManager.UpdateComment(fmt.Sprintf(":white_check_mark: Command %v completed successfully", *diggerCommand))
+		}
+
+		return nil
+	}
+
 	// perform unlocking in backend
 	if config.PrLocks {
 		slog.Info("Processing PR locks for impacted projects",

backend/controllers/github_pull_request.go

@@ -256,6 +256,19 @@ func handlePullRequestEvent(gh utils.GithubClientProvider, payload *github.PullR
 		return nil
 	}
 
+	// special case for when a draft pull request is opened and ignore PRs is set to true we DO NOT want to lock the projects
+	if !config.AllowDraftPRs && isDraft && action == "opened" {
+		slog.Info("Draft PRs are disabled, skipping PR",
+			"prNumber", prNumber,
+			"isDraft", isDraft,
+		)
+		if os.Getenv("DIGGER_REPORT_BEFORE_LOADING_CONFIG") == "1" {
+			// This one is for aggregate reporting
+			commentReporterManager.UpdateComment(":construction_worker: Ignoring event as it is a draft and draft PRs are configured to be ignored")
+		}
+		return nil
+	}
+
 	// perform locking/unlocking in backend
 	if config.PrLocks {
 		slog.Info("Processing PR locks for impacted projects",