Added processing of notifications other than Issues and PR

Author: andrey-helldar

app/Data/ItemData.php

@@ -6,15 +6,18 @@
 
 class ItemData extends Data
 {
-    public int $id;
-
     public bool $isOpen;
 
     public bool $isMerged;
 
-    public function __construct(array $data)
+    public function __construct(?array $data)
     {
-        $this->id = (int) $this->get($data, 'number');
+        if (is_null($data)) {
+            $this->isOpen   = false;
+            $this->isMerged = true;
+
+            return;
+        }
 
         $this->isOpen   = $this->get($data, 'state') === 'open';
         $this->isMerged = (bool) $this->get($data, 'merged');

app/Services/GitHub.php

@@ -78,19 +78,15 @@ public function mark(): void
         foreach ($items as $data) {
             $notification = new NotificationData($data);
 
-            $data = $notification->type === 'Issue'
-                ? $this->issue($notification)
-                : $this->pullRequest($notification);
-
-            $item = new ItemData($data);
+            $item = new ItemData($this->requestByType($notification));
 
             $this->shouldSkip($notification, $item)
                 ? $this->output->twoColumnDetail($notification->fullName, 'SKIP')
                 : $this->output->task($notification->fullName, fn () => $this->markAsRead($notification));
         }
     }
 
-    protected function markAsRead(NotificationData $data): bool
+    protected function markAsRead(NotificationData $data): void
     {
         $this->github->notification()->markThreadRead($data->id);
     }
@@ -105,6 +101,15 @@ protected function notifications(): Notification
         return $this->github->notifications();
     }
 
+    protected function requestByType(NotificationData $notification): ?array
+    {
+        return match ($notification->type) {
+            'Issue'       => $this->issue($notification),
+            'PullRequest' => $this->pullRequest($notification),
+            default       => null
+        };
+    }
+
     protected function issue(NotificationData $notification): array
     {
         return $this->github->issues()->show(
@@ -141,6 +146,6 @@ protected function shouldSkip(NotificationData $notification, ItemData $item): b
             return true;
         }
 
-        return (bool) ($item->isOpen || ! $item->isMerged);
+        return $item->isOpen || ! $item->isMerged;
     }
 }