Fix PostgreSQL not-null violation when adding redirects (#619)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: duncanmcclean

src/Redirects/Eloquent/ErrorRepository.php

@@ -49,7 +49,10 @@ private function toModel(Error $error): ErrorModel
     {
         $model = ErrorModel::find($error->id()) ?? new ErrorModel;
 
-        $model->id = $error->id();
+        if (! is_null($error->id())) {
+            $model->id = $error->id();
+        }
+
         $model->site = $error->site();
         $model->url = $error->url();
         $model->hits = $error->hits();

src/Redirects/Eloquent/RedirectRepository.php

@@ -52,7 +52,10 @@ private function toModel(Redirect $redirect): RedirectModel
     {
         $model = RedirectModel::find($redirect->id()) ?? new RedirectModel;
 
-        $model->id = $redirect->id();
+        if (! is_null($redirect->id())) {
+            $model->id = $redirect->id();
+        }
+
         $model->site = $redirect->site();
         $model->source = $redirect->source();
         $model->destination = $redirect->destination();

tests/Redirects/Eloquent/ErrorRepositoryTest.php

@@ -88,6 +88,20 @@ public function can_delete_error()
         $this->assertDatabaseMissing('seo_pro_errors', ['id' => $error->id()]);
     }
 
+    #[Test]
+    public function it_does_not_set_a_null_id_when_saving_a_new_error()
+    {
+        $error = Facades\Error::make()
+            ->url('/missing-page')
+            ->hits(1)
+            ->lastHitAt('2026-04-21 12:00:00');
+
+        $method = new \ReflectionMethod($this->repo, 'toModel');
+        $model = $method->invoke($this->repo, $error);
+
+        $this->assertArrayNotHasKey('id', $model->getAttributes());
+    }
+
     #[Test]
     public function it_sets_default_site_when_saving_without_one()
     {

tests/Redirects/Eloquent/RedirectRepositoryTest.php

@@ -96,6 +96,21 @@ public function can_delete_redirect()
         $this->assertDatabaseMissing('seo_pro_redirects', ['id' => $redirect->id()]);
     }
 
+    #[Test]
+    public function it_does_not_set_a_null_id_when_saving_a_new_redirect()
+    {
+        $redirect = Facades\Redirect::make()
+            ->source('/old-url')
+            ->destination('/new-url')
+            ->responseCode(302)
+            ->enabled(true);
+
+        $method = new \ReflectionMethod($this->repo, 'toModel');
+        $model = $method->invoke($this->repo, $redirect);
+
+        $this->assertArrayNotHasKey('id', $model->getAttributes());
+    }
+
     #[Test]
     public function it_sets_default_site_when_saving_without_one()
     {