add timeout for notifications on a per check basis.

src/Checks/Base/Check.php

@@ -2,6 +2,7 @@
 
 namespace Vormkracht10\LaravelOK\Checks\Base;
 
+use Carbon\Carbon;
 use Cron\CronExpression;
 use Illuminate\Console\Scheduling\ManagesFrequencies;
 use Illuminate\Support\Facades\Date;
@@ -24,6 +25,10 @@ abstract class Check
 
     protected string $expression = '* * * * *';
 
+    protected int $repeatSeconds;
+
+    protected Carbon $reportTimeout;
+
     protected ?string $name = null;
 
     protected ?string $message = null;
@@ -34,10 +39,6 @@ abstract class Check
 
     protected int $timesToFailWithoutNotification = 1;
 
-    public function __construct()
-    {
-    }
-
     public static function config(): static
     {
         $instance = app(static::class);
@@ -47,6 +48,16 @@ public static function config(): static
         return $instance;
     }
 
+    public function getExpression(): string
+    {
+        return $this->expression;
+    }
+
+    public function getRepeatSeconds(): int
+    {
+        return $this->repeatSeconds;
+    }
+
     public function name(string $name): self
     {
         $this->name = $name;
@@ -108,4 +119,16 @@ public function markAsCrashed(): Result
     {
         return new Result(Status::CRASHED);
     }
+
+    public function getReportTimeout(): Carbon
+    {
+        return $this->reportTimeout;
+    }
+
+    public function reportTimeout(Carbon $minimumDelay): static
+    {
+        $this->reportTimeout = $minimumDelay;
+
+        return $this;
+    }
 }

src/Checks/PermissionCheck.php

@@ -12,8 +12,6 @@ class PermissionCheck extends Check
 
     public function __construct(array $configured = [])
     {
-        parent::__construct();
-
         $this->configured = $configured;
     }
 

src/Commands/RunChecksCommand.php

@@ -36,9 +36,7 @@ public function runChecks()
     {
         return app(OK::class)
             ->configuredChecks()
-            ->map(function (mixed $check) {
-                return is_string($check) ? app($check) : $check;
-            })
+            ->map(fn ($check) => is_string($check) ? app($check) : $check)
             ->map(function (Check $check): Result {
                 return $check->shouldRun()
                     ? $this->runCheck($check)

src/Listeners/SendCheckFailedNotification.php

@@ -2,10 +2,19 @@
 
 namespace Vormkracht10\LaravelOK\Listeners;
 
+use Illuminate\Support\Facades\Cache;
+use Vormkracht10\LaravelOK\Checks\Base\Check;
+use Vormkracht10\LaravelOK\Events\CheckFailed;
+use Vormkracht10\LaravelOK\Facades\OK;
+
 class SendCheckFailedNotification
 {
-    public function handle($event)
+    public function handle(CheckFailed $event)
     {
+        if (! $this->shouldSendNotification($event->check)) {
+            return;
+        }
+
         $notifiableClass = config('ok.notifications.notifiable');
 
         $notifiable = app($notifiableClass);
@@ -15,5 +24,21 @@ public function handle($event)
         $notification = (new $failedNotificationClass($event->check, $event->result));
 
         $notifiable->notify($notification);
+
+        $class = $event->check::class;
+
+        Cache::driver('file')->set(
+            "laravel-ok::runs::{$class}",
+            now()->getTimestamp(),
+        );
+    }
+
+    protected function shouldSendNotification(Check $check): bool
+    {
+        $lastRun = OK::lastRun($check::class);
+
+        $timeout = $check->getReportTimeout();
+
+        return $lastRun < $timeout;
     }
 }

src/OK.php

@@ -2,6 +2,9 @@
 
 namespace Vormkracht10\LaravelOK;
 
+use Carbon\Carbon;
+use Illuminate\Support\Facades\Cache;
+
 class OK
 {
     protected array $checks = [];
@@ -13,6 +16,13 @@ public function checks(array $checks): self
         return $this;
     }
 
+    public function lastRun(string $check): Carbon
+    {
+        $timestamp = Cache::driver('file')->get("laravel-ok::runs::{$check}");
+
+        return Carbon::createFromTimestamp($timestamp);
+    }
+
     public function configuredChecks()
     {
         return collect($this->checks);