adding docs and cleaing up nullish logic

src/task/image.ts

@@ -4,6 +4,12 @@ import { TaskInput } from '../api/db/models/Task.js';
 import type * as gql from '../@types/graphql.js';
 
 export async function DeleteImagesByFilter(task: TaskInput<gql.DeleteImagesByFilterTaskInput>) {
+  /**
+   * Deletes images that match the inputted filters in batches of 100.
+   * This is used by the frontend to delete all images currently shown.
+   * * @param {Object} input
+   * * @param {gql.FiltersInput} input.config.filters
+   */
   const context = { user: { is_superuser: true, curr_project: task.projectId } as User };
   let images = await ImageModel.queryByFilter(
     { filters: task.config.filters, limit: 100 },
@@ -26,11 +32,17 @@ export async function DeleteImagesByFilter(task: TaskInput<gql.DeleteImagesByFil
 }
 
 export async function DeleteImages(task: TaskInput<gql.DeleteImagesInput>) {
+  /**
+   * Deletes a list of images by their IDs in batches of 100.
+   * This is used by the frontend when the user is selecting more than 100 images to delete to delete at once.
+   * * @param {Object} input
+   * * @param {String[]} input.config.imageIds
+   */
   const context = { user: { is_superuser: true, curr_project: task.projectId } as User };
-  const output = task.config.imageIds?.slice();
-  while (task.config.imageIds?.length && task.config.imageIds.length > 0) {
-    const batch = task.config.imageIds?.splice(0, 100);
+  const imagesToDelete = task.config.imageIds?.slice() ?? [];
+  while (imagesToDelete.length > 0) {
+    const batch = imagesToDelete.splice(0, 100);
     await ImageModel.deleteImages({ imageIds: batch }, context);
   }
-  return { imageIds: output };
+  return { imageIds: task.config.imageIds };
 }

src/task/stats.ts

@@ -1,5 +1,5 @@
 import { ProjectModel } from '../api/db/models/Project.js';
-import { buildPipeline, isImageReviewed, idMatch } from '../api/db/models/utils.js';
+import { buildPipeline, idMatch } from '../api/db/models/utils.js';
 import Image, { type ImageSchema } from '../api/db/schemas/Image.js';
 import _ from 'lodash';
 import { type TaskInput } from '../api/db/models/Task.js';