fix(types): make batchStrategy optional for pre-v1.15 server compat

Co-authored feedback from CodeRabbit review: batchStrategy should be optional
since pre-v1.15 Meilisearch servers do not return this field.

Author: mixelburg

src/types/task-and-batch.ts

@@ -249,7 +249,7 @@ export type Batch = {
   startedAt: string;
   finishedAt: string | null;
   /** Explains why the batch was finalized and stopped accepting more tasks. */
-  batchStrategy: string;
+  batchStrategy?: string;
 };
 
 /**

tests/batch.test.ts

@@ -45,7 +45,7 @@ describe.each([{ permission: "Master" }, { permission: "Admin" }])(
       expect(batch.startedAt).toBeDefined();
       expect(batch.finishedAt).toBeDefined();
       expect(batch.progress).toBeDefined();
-      expect(typeof batch.batchStrategy).toBe("string");
+      expect(["string", "undefined"]).toContain(typeof batch.batchStrategy);
     });
   },
 );

tests/utils/assertions/tasks-and-batches.ts

@@ -53,7 +53,11 @@ export const tasksAndBatchesAssertions = {
   },
 
   isBatch(batch: Batch) {
-    assert.lengthOf(Object.keys(batch), 8);
+    const keyCount = Object.keys(batch).length;
+    assert(
+      keyCount === 7 || keyCount === 8,
+      `expected batch to have 7 or 8 keys, got ${keyCount}`,
+    );
 
     const {
       uid,
@@ -154,7 +158,10 @@ export const tasksAndBatchesAssertions = {
       "expected finishedAt to be null or string",
     );
 
-    assert.typeOf(batchStrategy, "string");
+    assert(
+      batchStrategy === undefined || typeof batchStrategy === "string",
+      "expected batchStrategy to be undefined or string",
+    );
   },
 
   isTasksOrBatchesResults(value: TasksResults | BatchesResults) {