[Refactor] Always expect id field when creating paginator

Author: lindyhopchris

Diff for: CHANGELOG.md

@@ -8,6 +8,8 @@ All notable changes to this project will be documented in this file. This projec
 ### Changed
 
 - Upgrade to Laravel 10 and set minimum PHP version to 8.1.
+- **BREAKING**: The schema's id field must now always be provided to the `CursorPagination::make()` method and/or
+  constructor.
 
 ## [2.1.0] - 2023-01-24
 

Diff for: src/CursorPagination.php

@@ -31,11 +31,6 @@ class CursorPagination implements Paginator
 {
     use HasPageMeta;
 
-    /**
-     * @var ID|null
-     */
-    private ?ID $id;
-
     /**
      * @var string
      */
@@ -79,22 +74,21 @@ class CursorPagination implements Paginator
     /**
      * Fluent constructor.
      *
-     * @param ID|null $id
+     * @param ID $id
      * @return CursorPagination
      */
-    public static function make(ID $id = null): self
+    public static function make(ID $id): self
     {
         return new static($id);
     }
 
     /**
      * CursorPagination constructor.
      *
-     * @param ID|null $id
+     * @param ID $id
      */
-    public function __construct(ID $id = null)
+    public function __construct(private readonly ID $id)
     {
-        $this->id = $id;
         $this->before = 'before';
         $this->after = 'after';
         $this->limit = 'limit';

Diff for: tests/lib/Acceptance/Test.php

@@ -30,6 +30,7 @@
 use LaravelJsonApi\CursorPagination\CursorPage;
 use LaravelJsonApi\CursorPagination\CursorPagination;
 use LaravelJsonApi\CursorPagination\Tests\EncodedId;
+use LaravelJsonApi\Eloquent\Fields\ID;
 
 class Test extends TestCase
 {
@@ -61,7 +62,9 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $this->paginator = CursorPagination::make();
+        $this->paginator = CursorPagination::make(
+            ID::make()->uuid()
+        );
 
         $this->videos = $this
             ->getMockBuilder(VideoSchema::class)