add coderabbitai suggested changes

src/resolvers/User/tagsAssignedWith.ts

@@ -119,7 +119,7 @@ type ParsedCursor = string;
  * @param cursorValue - The cursor value to be parsed.
  * @param cursorName - The name of the cursor argument.
  * @param cursorPath - The path of the cursor argument in the GraphQL query.
- * @param tagId - The ID of the user to which the tags are assigned.
+ * @param userId - The ID of the user for which assigned tags are being queried.
  * @returns An object containing the parsed cursor value or an array of errors if the cursor value is invalid.
  *
  * @see TagUser - The TagUser model used to interact with the tag users collection in the database.

src/resolvers/UserTag/usersToAssignTo.ts

@@ -112,7 +112,7 @@ export const usersToAssignTo: UserTagResolvers["usersToAssignTo"] = async (
       {
         $match: {
           ...objectListFilter,
-          joinedOrganizations: parent.organizationId,
+          joinedOrganizations: { $in: [parent.organizationId] },
         },
       },
       ...commonPipeline,

src/utilities/graphQLConnection/getUserTagGraphQLConnectionFilter.ts

@@ -5,9 +5,9 @@ import type {
 } from "../userTagsPaginationUtils";
 
 /**
- * This is typescript type of the object returned from function `getGraphQLConnectionFilter`.
+ * This is typescript type of the object returned from function `getUserTagGraphQLConnectionFilter`.
  */
-type GraphQLConnectionFilter =
+type UserTagGraphQLConnectionFilter =
   | {
       _id: {
         $lt: string;
@@ -37,11 +37,14 @@ export function getUserTagGraphQLConnectionFilter({
   ParseUserTagWhereResult & {
     cursor: string | null;
     direction: GraphQLConnectionTraversalDirection;
-  }): GraphQLConnectionFilter {
-  const filter = {} as GraphQLConnectionFilter;
+  }): UserTagGraphQLConnectionFilter {
+  const filter = {} as UserTagGraphQLConnectionFilter;
 
   filter.name = {
-    $regex: new RegExp(`^${nameStartsWith}`, "i"),
+    $regex: new RegExp(
+      `^${nameStartsWith.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}`,
+      "i",
+    ),
   };
 
   if (cursor !== null) {

src/utilities/graphQLConnection/getUserTagGraphQLConnectionSort.ts

@@ -2,9 +2,9 @@ import type { GraphQLConnectionTraversalDirection } from ".";
 import type { ParseSortedByResult } from "../userTagsPaginationUtils";
 
 /**
- *This is typescript type of the object returned from `getGraphQLConnectionSort` function.
+ *This is typescript type of the object returned from `getUserTagGraphQLConnectionSort` function.
  */
-type GraphQLConnectionSort =
+type UserTagGraphQLConnectionSort =
   | {
       _id: 1;
     }
@@ -20,7 +20,7 @@ export function getUserTagGraphQLConnectionSort({
   sortById,
 }: ParseSortedByResult & {
   direction: GraphQLConnectionTraversalDirection;
-}): GraphQLConnectionSort {
+}): UserTagGraphQLConnectionSort {
   if (sortById === "ASCENDING") {
     if (direction === "BACKWARD") {
       return {

src/utilities/graphQLConnection/getUserTagMemberGraphQLConnectionFilter.ts

@@ -6,9 +6,9 @@ import type {
 } from "../userTagsPaginationUtils";
 
 /**
- * This is typescript type of the object returned from function `getGraphQLConnectionFilter`.
+ * This is typescript type of the object returned from function `getUserTagMemberGraphQLConnectionFilter`.
  */
-type GraphQLConnectionFilter =
+type UserTagMemberGraphQLConnectionFilter =
   | {
       _id: {
         $lt: Types.ObjectId;
@@ -45,14 +45,20 @@ export function getUserTagMemberGraphQLConnectionFilter({
   ParseUserTagMemberWhereResult & {
     cursor: string | null;
     direction: GraphQLConnectionTraversalDirection;
-  }): GraphQLConnectionFilter {
-  const filter = {} as GraphQLConnectionFilter;
+  }): UserTagMemberGraphQLConnectionFilter {
+  const filter = {} as UserTagMemberGraphQLConnectionFilter;
 
   filter.firstName = {
-    $regex: new RegExp(`^${firstNameStartsWith}`, "i"),
+    $regex: new RegExp(
+      `^${firstNameStartsWith.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}`,
+      "i",
+    ),
   };
   filter.lastName = {
-    $regex: new RegExp(`^${lastNameStartsWith}`, "i"),
+    $regex: new RegExp(
+      `^${lastNameStartsWith.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}`,
+      "i",
+    ),
   };
 
   if (cursor !== null) {

src/utilities/userTagsPaginationUtils/parseUserTagMemberWhere.ts

@@ -31,7 +31,7 @@ export function parseUserTagMemberWhere(
   } else {
     if (!where.firstName && !where.lastName) {
       errors.push({
-        message: `Atleast one of firstName or lastName should be provided`,
+        message: `At least one of firstName or lastName should be provided`,
         path: ["where"],
       });
 
@@ -45,7 +45,7 @@ export function parseUserTagMemberWhere(
     ) {
       errors.push({
         message: "Invalid firstName provided. It must be a string.",
-        path: ["whereFirstName"],
+        path: ["where", "firstName", "starts_with"],
       });
 
       return {
@@ -58,7 +58,7 @@ export function parseUserTagMemberWhere(
     ) {
       errors.push({
         message: "Invalid lastName provided. It must be a string.",
-        path: ["whereLastName"],
+        path: ["where", "lastName", "starts_with"],
       });
 
       return {

src/utilities/userTagsPaginationUtils/parseUserTagSortedBy.ts

@@ -27,25 +27,24 @@ export function parseUserTagSortedBy(
       isSuccessful: true,
       parsedSortedBy: { sortById: "DESCENDING" },
     };
-  } else {
-    if (sortedBy.id !== "DESCENDING" && sortedBy.id !== "ASCENDING") {
-      errors.push({
-        message:
-          "Invalid sortedById provided. It must be a of type SortedByOrder.",
-        path: ["sortedById"],
-      });
-
-      return {
-        isSuccessful: false,
-        errors,
-      };
-    }
+  }
 
+  if (sortedBy.id !== "DESCENDING" && sortedBy.id !== "ASCENDING") {
+    errors.push({
+      message:
+        "Invalid sortedById provided. It must be a of type SortedByOrder.",
+      path: ["sortedBy", "id"],
+    });
     return {
-      isSuccessful: true,
-      parsedSortedBy: {
-        sortById: sortedBy.id,
-      },
+      isSuccessful: false,
+      errors,
     };
   }
+
+  return {
+    isSuccessful: true,
+    parsedSortedBy: {
+      sortById: sortedBy.id,
+    },
+  };
 }

src/utilities/userTagsPaginationUtils/parseUserTagWhere.ts

@@ -30,7 +30,7 @@ export function parseUserTagWhere(
     if (typeof where.name.starts_with !== "string") {
       errors.push({
         message: "Invalid name provided. It must be a string.",
-        path: ["whereName"],
+        path: ["where", "name", "starts_with"],
       });
 
       return {

tests/resolvers/UserTag/ancestorTags.spec.ts

@@ -32,7 +32,15 @@ afterAll(async () => {
 });
 
 describe("resolvers -> Tag -> ancestorTags", () => {
-  it(`returns the correct ancestorTags array`, async () => {
+  it(`returns an empty ancestorTags array for the root tag`, async () => {
+    const parent = testRootTag as InterfaceOrganizationTagUser;
+
+    const payload = await ancestorTagsResolver?.(parent, {}, {});
+
+    expect(payload).toEqual([]);
+  });
+
+  it(`returns the correct ancestorTags array for a nested tag`, async () => {
     const parent = testSubTagLevel2 as InterfaceOrganizationTagUser;
 
     const payload = await ancestorTagsResolver?.(parent, {}, {});