From e1decffe738e7b8194f3c345a27c26ef9b608eb6 Mon Sep 17 00:00:00 2001
From: Urgau <urgau@numericable.fr>
Date: Sun, 29 Jun 2025 14:46:08 +0200
Subject: [PATCH 1/4] Remove `new_pr` labels when converting to draft

---
 src/handlers/autolabel.rs | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/src/handlers/autolabel.rs b/src/handlers/autolabel.rs
index 0a16dbe7c..a84da6cf0 100644
--- a/src/handlers/autolabel.rs
+++ b/src/handlers/autolabel.rs
@@ -39,7 +39,10 @@ pub(super) async fn parse_input(
     // synchronize may be straddling a rebase, which will break diff generation.
     if matches!(
         event.action,
-        IssuesAction::Opened | IssuesAction::Synchronize | IssuesAction::ReadyForReview
+        IssuesAction::Opened
+            | IssuesAction::Synchronize
+            | IssuesAction::ReadyForReview
+            | IssuesAction::ConvertedToDraft
     ) {
         let mut db = ctx.db.get().await;
         let mut state: IssueData<'_, AutolabelState> =
@@ -55,7 +58,9 @@ pub(super) async fn parse_input(
                 log::error!("failed to fetch diff: {:?}", e);
             })
             .unwrap_or_default();
+
         let mut autolabels = Vec::new();
+        let mut to_remove = Vec::new();
 
         'outer: for (label, cfg) in config.labels.iter() {
             let exclude_patterns: Vec<glob::Pattern> = cfg
@@ -108,6 +113,14 @@ pub(super) async fn parse_input(
                     });
                     state.data.new_pr_labels_applied = true;
                 }
+
+                // If a PR is converted to draft remove all the "new PR" labels
+                if cfg.new_pr && event.action == IssuesAction::ConvertedToDraft {
+                    to_remove.push(Label {
+                        name: label.to_owned(),
+                    });
+                    state.data.new_pr_labels_applied = false;
+                }
             } else {
                 if cfg.new_issue && event.action == IssuesAction::Opened {
                     autolabels.push(Label {
@@ -119,10 +132,10 @@ pub(super) async fn parse_input(
 
         state.save().await.map_err(|e| e.to_string())?;
 
-        if !autolabels.is_empty() {
+        if !autolabels.is_empty() || !to_remove.is_empty() {
             return Ok(Some(AutolabelInput {
                 add: autolabels,
-                remove: vec![],
+                remove: to_remove,
             }));
         }
     }

From 0c505345678fda3b0f703bc2110ef352ef252e56 Mon Sep 17 00:00:00 2001
From: Urgau <urgau@numericable.fr>
Date: Mon, 30 Jun 2025 23:07:16 +0200
Subject: [PATCH 2/4] Remove autolabel `new_pr_labels_applied` state

---
 src/handlers/autolabel.rs | 28 +++-------------------------
 1 file changed, 3 insertions(+), 25 deletions(-)

diff --git a/src/handlers/autolabel.rs b/src/handlers/autolabel.rs
index a84da6cf0..1b03ee42c 100644
--- a/src/handlers/autolabel.rs
+++ b/src/handlers/autolabel.rs
@@ -1,4 +1,3 @@
-use crate::db::issue_data::IssueData;
 use crate::{
     config::AutolabelConfig,
     github::{IssuesAction, IssuesEvent, Label},
@@ -7,16 +6,6 @@ use crate::{
 use anyhow::Context as _;
 use tracing as log;
 
-/// Key for the state in the database
-const AUTOLABEL_KEY: &str = "autolabel";
-
-/// State stored in the database
-#[derive(Debug, Default, PartialEq, Clone, serde::Deserialize, serde::Serialize)]
-struct AutolabelState {
-    /// If true, then `autolabel.new_pr` labels have already been applied to this PR.
-    new_pr_labels_applied: bool,
-}
-
 pub(super) struct AutolabelInput {
     add: Vec<Label>,
     remove: Vec<Label>,
@@ -44,12 +33,6 @@ pub(super) async fn parse_input(
             | IssuesAction::ReadyForReview
             | IssuesAction::ConvertedToDraft
     ) {
-        let mut db = ctx.db.get().await;
-        let mut state: IssueData<'_, AutolabelState> =
-            IssueData::load(&mut db, &event.issue, AUTOLABEL_KEY)
-                .await
-                .map_err(|e| e.to_string())?;
-
         let files = event
             .issue
             .diff(&ctx.github)
@@ -102,16 +85,14 @@ pub(super) async fn parse_input(
 
                 // Treat the following situations as a "new PR":
                 // 1) New PRs opened as non-draft
-                // 2) PRs opened as draft that are marked as "ready for review" for the first time.
+                // 2) PRs opened as draft that are marked as "ready for review".
                 let is_new_non_draft_pr =
                     event.action == IssuesAction::Opened && !event.issue.draft;
-                let is_first_time_ready_for_review = event.action == IssuesAction::ReadyForReview
-                    && !state.data.new_pr_labels_applied;
-                if cfg.new_pr && (is_new_non_draft_pr || is_first_time_ready_for_review) {
+                let is_ready_for_review = event.action == IssuesAction::ReadyForReview;
+                if cfg.new_pr && (is_new_non_draft_pr || is_ready_for_review) {
                     autolabels.push(Label {
                         name: label.to_owned(),
                     });
-                    state.data.new_pr_labels_applied = true;
                 }
 
                 // If a PR is converted to draft remove all the "new PR" labels
@@ -119,7 +100,6 @@ pub(super) async fn parse_input(
                     to_remove.push(Label {
                         name: label.to_owned(),
                     });
-                    state.data.new_pr_labels_applied = false;
                 }
             } else {
                 if cfg.new_issue && event.action == IssuesAction::Opened {
@@ -130,8 +110,6 @@ pub(super) async fn parse_input(
             }
         }
 
-        state.save().await.map_err(|e| e.to_string())?;
-
         if !autolabels.is_empty() || !to_remove.is_empty() {
             return Ok(Some(AutolabelInput {
                 add: autolabels,

From ccbeb66d95d66b3a4db9c2c6965248c16f1d3ce3 Mon Sep 17 00:00:00 2001
From: Urgau <urgau@numericable.fr>
Date: Tue, 1 Jul 2025 18:42:42 +0200
Subject: [PATCH 3/4] Handle closing and reopening PRs for the `[autolabel]`
 `new_pr` labels

---
 src/handlers/autolabel.rs | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/handlers/autolabel.rs b/src/handlers/autolabel.rs
index 1b03ee42c..5d565520d 100644
--- a/src/handlers/autolabel.rs
+++ b/src/handlers/autolabel.rs
@@ -29,6 +29,8 @@ pub(super) async fn parse_input(
     if matches!(
         event.action,
         IssuesAction::Opened
+            | IssuesAction::Closed
+            | IssuesAction::Reopened
             | IssuesAction::Synchronize
             | IssuesAction::ReadyForReview
             | IssuesAction::ConvertedToDraft
@@ -84,19 +86,25 @@ pub(super) async fn parse_input(
                 }
 
                 // Treat the following situations as a "new PR":
-                // 1) New PRs opened as non-draft
-                // 2) PRs opened as draft that are marked as "ready for review".
-                let is_new_non_draft_pr =
-                    event.action == IssuesAction::Opened && !event.issue.draft;
+                // 1) PRs that were (re)opened and are not draft
+                // 2) PRs that have been converted from a draft to being "ready for review"
+                let is_opened_non_draft =
+                    matches!(event.action, IssuesAction::Opened | IssuesAction::Reopened)
+                        && !event.issue.draft;
                 let is_ready_for_review = event.action == IssuesAction::ReadyForReview;
-                if cfg.new_pr && (is_new_non_draft_pr || is_ready_for_review) {
+                if cfg.new_pr && (is_opened_non_draft || is_ready_for_review) {
                     autolabels.push(Label {
                         name: label.to_owned(),
                     });
                 }
 
-                // If a PR is converted to draft remove all the "new PR" labels
-                if cfg.new_pr && event.action == IssuesAction::ConvertedToDraft {
+                // If a PR is converted to draft or closed, remove all the "new PR" labels
+                if cfg.new_pr
+                    && matches!(
+                        event.action,
+                        IssuesAction::ConvertedToDraft | IssuesAction::Closed
+                    )
+                {
                     to_remove.push(Label {
                         name: label.to_owned(),
                     });

From 4af2080338c4b3f5a384ec6f6df81e3a1f12a1c2 Mon Sep 17 00:00:00 2001
From: Urgau <urgau@numericable.fr>
Date: Tue, 1 Jul 2025 20:19:07 +0200
Subject: [PATCH 4/4] Only look at the diff in `[autolabel]` when it can have
 changed

---
 src/handlers/autolabel.rs | 41 ++++++++++++++++++++++++---------------
 1 file changed, 25 insertions(+), 16 deletions(-)

diff --git a/src/handlers/autolabel.rs b/src/handlers/autolabel.rs
index 5d565520d..83adf78cf 100644
--- a/src/handlers/autolabel.rs
+++ b/src/handlers/autolabel.rs
@@ -26,23 +26,32 @@ pub(super) async fn parse_input(
     // FIXME: This will re-apply labels after a push that the user had tried to
     // remove. Not much can be done about that currently; the before/after on
     // synchronize may be straddling a rebase, which will break diff generation.
-    if matches!(
+    let can_trigger_files = matches!(
         event.action,
-        IssuesAction::Opened
-            | IssuesAction::Closed
-            | IssuesAction::Reopened
-            | IssuesAction::Synchronize
-            | IssuesAction::ReadyForReview
-            | IssuesAction::ConvertedToDraft
-    ) {
-        let files = event
-            .issue
-            .diff(&ctx.github)
-            .await
-            .map_err(|e| {
-                log::error!("failed to fetch diff: {:?}", e);
-            })
-            .unwrap_or_default();
+        IssuesAction::Opened | IssuesAction::Synchronize
+    );
+
+    if can_trigger_files
+        || matches!(
+            event.action,
+            IssuesAction::Closed
+                | IssuesAction::Reopened
+                | IssuesAction::ReadyForReview
+                | IssuesAction::ConvertedToDraft
+        )
+    {
+        let files = if can_trigger_files {
+            event
+                .issue
+                .diff(&ctx.github)
+                .await
+                .map_err(|e| {
+                    log::error!("failed to fetch diff: {:?}", e);
+                })
+                .unwrap_or_default()
+        } else {
+            Default::default()
+        };
 
         let mut autolabels = Vec::new();
         let mut to_remove = Vec::new();