Merge pull request #76 from TheDragonCode/1.x

Added the ability to exclude user mentions from processing

Author: andrey-helldar

README.md

@@ -85,11 +85,12 @@ By default, only those Issues and Pull Requests that have been closed or merged
 But you can define the parameters yourself:
 
 ```Bash
--i, --except-issues  Exclude issues from processing
--p, --except-pulls   Exclude Pull Requests from processing
--o, --with-open       Process including open Issues and Pull Requests
--n, --no-interaction  Do not ask any interactive question
--q, --quiet           Do not output any message
+-i, --except-issues     Exclude issues from processing
+-p, --except-pulls      Exclude Pull Requests from processing
+-m, --except-mentions   Exclude notifications with your mention from processing
+-o, --with-open         Process including open Issues and Pull Requests
+-n, --no-interaction    Do not ask any interactive question
+-q, --quiet             Do not output any message
 ```
 
 For example:

app/Commands/ReadCommand.php

@@ -17,6 +17,7 @@ class ReadCommand extends Command
         . ' {repository?* : Full or partial repository names}'
         . ' {--i|except-issues : Exclude issues from processing}'
         . ' {--p|except-pulls : Exclude Pull Requests from processing}'
+        . ' {--m|except-mentions : Exclude notifications with your mention from processing}'
         . ' {--o|with-open : Process including open Issues and Pull Requests}'
         . ' {--token= : Specifies the token to use}';
 
@@ -57,6 +58,7 @@ protected function read(array $repositories): void
             ->repositories($repositories)
             ->exceptIssues($this->exceptIssues())
             ->exceptPulls($this->exceptPulls())
+            ->exceptMentions($this->exceptMentions())
             ->withOpen($this->withOpen())
             ->when(
                 $this->shouldBeAll($repositories),
@@ -70,6 +72,7 @@ protected function shouldBeAll(array $repositories): bool
         return empty($repositories)
             && ! $this->exceptIssues()
             && ! $this->exceptPulls()
+            && ! $this->exceptMentions()
             && $this->withOpen();
     }
 
@@ -103,6 +106,11 @@ protected function exceptPulls(): bool
         return $this->option('except-pulls');
     }
 
+    protected function exceptMentions(): bool
+    {
+        return $this->option('except-mentions');
+    }
+
     protected function withOpen(): bool
     {
         return $this->option('with-open');

app/Data/NotificationData.php

@@ -12,6 +12,8 @@ class NotificationData extends Data
 
     public string $type;
 
+    public string $reason;
+
     public string $fullName;
 
     public string $organization;
@@ -25,7 +27,8 @@ public function __construct(array $data, int $index)
         $this->id      = (int) $this->get($data, 'id');
         $this->issueId = $this->issueId($data);
 
-        $this->type = $this->get($data, 'subject.type');
+        $this->type   = $this->get($data, 'subject.type');
+        $this->reason = $this->get($data, 'reason');
 
         $this->fullName     = $this->get($data, 'repository.full_name');
         $this->organization = $this->get($data, 'repository.owner.login');

app/Services/GitHub.php

@@ -21,6 +21,8 @@ class GitHub
 
     protected bool $exceptPulls = false;
 
+    protected bool $exceptMentions = false;
+
     protected bool $withOpen = false;
 
     protected int $index = 0;
@@ -40,16 +42,23 @@ public function repositories(array $repositories): self
         return $this;
     }
 
-    public function exceptIssues(bool $exceptIssues): self
+    public function exceptIssues(bool $except): self
     {
-        $this->exceptIssues = $exceptIssues;
+        $this->exceptIssues = $except;
 
         return $this;
     }
 
-    public function exceptPulls(bool $exceptPulls): self
+    public function exceptPulls(bool $except): self
     {
-        $this->exceptPulls = $exceptPulls;
+        $this->exceptPulls = $except;
+
+        return $this;
+    }
+
+    public function exceptMentions(bool $except): self
+    {
+        $this->exceptMentions = $except;
 
         return $this;
     }
@@ -159,6 +168,10 @@ protected function shouldSkip(NotificationData $notification, ItemData $item): b
             return true;
         }
 
+        if ($this->exceptMentions && $notification->reason === 'mention') {
+            return true;
+        }
+
         if ($this->withOpen && $item->isOpen) {
             return false;
         }