Fix bot review

Author: norkunas

src/Contracts/Task.php

@@ -147,7 +147,7 @@ public function isFinished(): bool
      * @param array{
      *     taskUid?: int,
      *     uid?: int,
-     *     indexUid: non-empty-string,
+     *     indexUid?: non-empty-string,
      *     status: non-empty-string,
      *     type: non-empty-string,
      *     enqueuedAt: non-empty-string,
@@ -165,7 +165,7 @@ public static function fromArray(array $data): Task
     {
         return new self(
             $data['taskUid'] ?? $data['uid'],
-            $data['indexUid'],
+            $data['indexUid'] ?? null,
             TaskStatus::from($data['status']),
             $type = TaskType::from($data['type']),
             new \DateTimeImmutable($data['enqueuedAt']),
@@ -186,6 +186,8 @@ public static function fromArray(array $data): Task
                 TaskType::DumpCreation => null !== $data['details'] ? DumpCreationDetails::fromArray($data['details']) : null,
                 TaskType::TaskCancelation => null !== $data['details'] ? TaskCancelationDetails::fromArray($data['details']) : null,
                 TaskType::TaskDeletion => null !== $data['details'] ? TaskDeletionDetails::fromArray($data['details']) : null,
+                // It’s intentional that SnapshotCreation tasks don’t have a details object
+                // (no SnapshotCreationDetails exists and tests don’t exercise any details)
                 TaskType::SnapshotCreation => null,
             },
             null !== $data['error'] ? TaskError::fromArray($data['error']) : null,

src/Contracts/TaskDetails/DocumentEditionDetails.php

@@ -34,7 +34,7 @@ public static function fromArray(array $data): self
         return new self(
             $data['context'],
             $data['deletedDocuments'],
-            $data['deletedDocuments'],
+            $data['editedDocuments'],
             $data['function'],
             $data['originalFilter'],
         );