From 1fdf0a55cf219868c120f0cf6818a722e962e6a5 Mon Sep 17 00:00:00 2001
From: berni44 <github@bernis-buecher.de>
Date: Mon, 22 Feb 2021 17:36:07 +0100
Subject: [PATCH] Fix #191 - remove common attributes when not in PR title

---
 source/dlangbot/github.d | 27 +++++++++++++++++++++++----
 1 file changed, 23 insertions(+), 4 deletions(-)

diff --git a/source/dlangbot/github.d b/source/dlangbot/github.d
index cb3a9fe..c699f73 100644
--- a/source/dlangbot/github.d
+++ b/source/dlangbot/github.d
@@ -322,9 +322,9 @@ void searchForAutoMergePrs(string repoSlug)
 }
 
 /**
-Allows contributors to use [<label>] messages in the title.
-If they are part of a pre-defined, allowed list, the bot will add the
-respective label.
+Allows contributors to use [<label>] and [-<label>] messages in the title.
+If they are part of a pre-defined, allowed list, the bot will add or
+remove the respective label.
 */
 void checkTitleForLabels(in ref PullRequest pr)
 {
@@ -334,10 +334,16 @@ void checkTitleForLabels(in ref PullRequest pr)
 
     static labelRe = regex(`\[(.*)\]`);
     string[] userLabels;
+    string[] removeLabels;
     foreach (m; pr.title.matchAll(labelRe))
     {
         foreach (el; m[1].splitter(","))
-            userLabels ~= el;
+        {
+            if (el.length > 0 && el[0] == '-')
+                removeLabels ~= el[1 .. $];
+            else
+                userLabels ~= el;
+        }
     }
 
     const string[string] userLabelsMap = [
@@ -356,4 +362,17 @@ void checkTitleForLabels(in ref PullRequest pr)
 
     if (mappedLabels.length)
         pr.addLabels(mappedLabels);
+
+    auto mappedRemoveLabels = removeLabels
+                            .sort()
+                            .uniq
+                            .map!strip
+                            .map!toLower
+                            .filter!(l => l in userLabelsMap)
+                            .map!(l => userLabelsMap[l])
+                            .array;
+
+    if (mappedRemoveLabels.length)
+        checkAndRemoveLabels(pr.labels, pr, mappedRemoveLabels);
+
 }