chore: Refactor SearchHelper internals

server/models/helpers/SearchHelper.test.ts

@@ -194,6 +194,9 @@ describe("#searchForUser", () => {
 
   test("should search only drafts created by user", async () => {
     const user = await buildUser();
+    await buildDraftDocument({
+      title: "test",
+    });
     await buildDraftDocument({
       teamId: user.teamId,
       userId: user.id,

server/models/helpers/SearchHelper.ts

@@ -177,24 +177,23 @@ export default class SearchHelper {
   ): Promise<Document[]> {
     const { limit = 15, offset = 0 } = options;
 
-    let where: WhereOptions<Document> = {
+    const where: WhereOptions<Document> = {
+      teamId: user.teamId,
       title: {
         [Op.iLike]: `%${query}%`,
       },
+      [Op.and]: [],
     };
 
     // Ensure we're filtering by the users accessible collections. If
     // collectionId is passed as an option it is assumed that the authorization
     // has already been done in the router
     if (options.collectionId) {
-      where = {
+      where[Op.and].push({
-        ...where,
         collectionId: options.collectionId,
-      };
+      });
     } else {
-      // @ts-expect-error doesn't like OR null
+      where[Op.and].push({
-      where = {
-        ...where,
         [Op.or]: [
           {
             collectionId: {
@@ -208,32 +207,29 @@ export default class SearchHelper {
             createdById: user.id,
           },
         ],
-      };
+      });
     }
 
     if (options.dateFilter) {
-      where = {
+      where[Op.and].push({
-        ...where,
         updatedAt: {
           [Op.gt]: sequelize.literal(
             `now() - interval '1 ${options.dateFilter}'`
           ),
         },
-      };
+      });
     }
 
     if (!options.includeArchived) {
-      where = {
+      where[Op.and].push({
-        ...where,
         archivedAt: {
           [Op.is]: null,
         },
-      };
+      });
     }
 
     if (options.includeDrafts) {
-      where = {
+      where[Op.and].push({
-        ...where,
         [Op.or]: [
           {
             publishedAt: {
@@ -244,23 +240,21 @@ export default class SearchHelper {
             createdById: user.id,
           },
         ],
-      };
+      });
     } else {
-      where = {
+      where[Op.and].push({
-        ...where,
         publishedAt: {
           [Op.ne]: null,
         },
-      };
+      });
     }
 
     if (options.collaboratorIds) {
-      where = {
+      where[Op.and].push({
-        ...where,
         collaboratorIds: {
           [Op.contains]: options.collaboratorIds,
         },
-      };
+      });
     }
 
     return await Document.scope([