deps: upgrade nest + packages (#990)

Author: annarhughes

package.json

@@ -32,26 +32,26 @@
     "@nestjs/axios": "^4.0.1",
     "@nestjs/common": "^11.1.9",
     "@nestjs/config": "^4.0.1",
-    "@nestjs/core": "^10.4.15",
-    "@nestjs/platform-express": "^10.4.12",
-    "@nestjs/swagger": "^7.4.0",
+    "@nestjs/core": "^11.1.9",
+    "@nestjs/platform-express": "^11.1.9",
+    "@nestjs/swagger": "^11.2.3",
     "@nestjs/terminus": "^11.0.0",
     "@nestjs/throttler": "^6.5.0",
     "@nestjs/typeorm": "^11.0.0",
     "@storyblok/js": "^4.4.3",
     "axios": "^1.13.2",
     "class-transformer": "^0.5.1",
     "class-validator": "^0.14.3",
-    "crisp-api": "^9.10.0",
+    "crisp-api": "^10.1.0",
     "date-fns": "^4.1.0",
-    "dotenv": "^17.2.3",
     "dompurify": "^3.3.1",
+    "dotenv": "^17.2.3",
     "firebase": "^12.7.0",
     "firebase-admin": "^13.6.0",
     "jsdom": "^27.3.0",
     "lodash": "^4.17.21",
     "nestjs-cls": "^6.1.0",
-    "newrelic": "^12.21.0",
+    "newrelic": "^13.8.1",
     "pg": "^8.16.3",
     "pg-connection-string": "^2.7.0",
     "reflect-metadata": "^0.2.1",
@@ -67,20 +67,20 @@
     "@golevelup/ts-jest": "^0.7.0",
     "@nestjs/cli": "^11.0.14",
     "@nestjs/schematics": "^11.0.9",
-    "@nestjs/testing": "^10.4.15",
+    "@nestjs/testing": "^11.1.9",
     "@types/date-fns": "^2.6.0",
     "@types/dompurify": "^3.2.0",
     "@types/express": "^5.0.6",
-    "@types/jest": "^29.5.13",
+    "@types/jest": "^30.0.0",
     "@types/jsdom": "^27.0.0",
     "@types/lodash": "^4.17.21",
-    "@types/node": "^22.15.29",
+    "@types/node": "^25.0.3",
     "@types/supertest": "^6.0.2",
     "@types/validator": "^13.15.10",
     "eslint": "^9.39.2",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-prettier": "^5.5.4",
-    "jest": "^29.7.0",
+    "jest": "^30.2.0",
     "prettier": "^3.7.4",
     "supertest": "^7.1.4",
     "ts-jest": "^29.4.6",

src/app.module.ts

@@ -46,7 +46,10 @@ import { WebhooksModule } from './webhooks/webhooks.module';
       middleware: {
         mount: true,
         generateId: true,
-        idGenerator: (req: Request) => req.headers['X-Request-Id'] ?? uuidv4(),
+        idGenerator: (req: Request) => {
+          const requestId = req.headers['x-request-id'];
+          return (Array.isArray(requestId) ? requestId[0] : requestId) ?? uuidv4();
+        },
       },
     }),
     LoggerModule,

src/crisp/crisp.interface.ts

@@ -60,8 +60,10 @@ export interface NewCrispProfileBaseResponse {
   data: CrispProfileBaseResponse;
 }
 
+// Response type from crisp-api getPeopleData - matches PeopleData type
 export interface CrispProfileDataResponse {
-  error: boolean;
-  reason: string;
-  data: { data: CrispProfileCustomFields };
+  data?: CrispProfileCustomFields;
 }
+
+// Type for updatePeopleData params - crisp-api expects flat Record, not nested object
+export type CrispPeopleDataUpdateParams = Record<string, string | number | boolean>;

src/crisp/crisp.service.spec.ts

@@ -230,7 +230,7 @@ describe('CrispService', () => {
       expect(mockWebsite.updatePeopleData).toHaveBeenCalledWith(
         crispWebsiteId,
         '[email protected]',
-        { data: customFields },
+        customFields,
       );
       expect(response).toEqual(expectedResponse);
     });

src/crisp/crisp.service.ts

@@ -4,6 +4,7 @@ import { EventLoggerService } from 'src/event-logger/event-logger.service';
 import { crispPluginId, crispPluginKey, crispWebsiteId } from 'src/utils/constants';
 import { isCypressTestEmail } from 'src/utils/utils';
 import {
+  CrispPeopleDataUpdateParams,
   CrispProfileBase,
   CrispProfileBaseResponse,
   CrispProfileCustomFields,
@@ -21,6 +22,19 @@ export class CrispService {
     CrispClient.authenticateTier('plugin', crispPluginId, crispPluginKey);
   }
 
+  // Convert CrispProfileCustomFields to the format crisp-api expects (primitive values only)
+  private toCrispDataParams(
+    peopleData: CrispProfileCustomFields,
+  ): CrispPeopleDataUpdateParams {
+    const data: CrispPeopleDataUpdateParams = {};
+    for (const [key, value] of Object.entries(peopleData)) {
+      if (value !== undefined && value !== null) {
+        data[key] = value;
+      }
+    }
+    return data;
+  }
+
   private isProfileNotFoundError(error: unknown): boolean {
     // Based on Crisp API docs, error format is: { reason: 'error', message: 'not_found', code: 404 }
     const errorObj = error as Record<string, unknown>;
@@ -143,19 +157,21 @@ export class CrispService {
       return null;
     }
 
+    const params = this.toCrispDataParams(peopleData);
+
     try {
-      const crispPeopleData = CrispClient.website.updatePeopleData(crispWebsiteId, email, {
-        data: peopleData,
-      });
+      const crispPeopleData = CrispClient.website.updatePeopleData(
+        crispWebsiteId,
+        email,
+        params,
+      );
       return crispPeopleData;
     } catch (error) {
       // Only handle profile not found errors (404, not_found, or profile-related errors)
       if (this.isProfileNotFoundError(error)) {
         try {
           await this.createCrispProfile({ email });
-          return await CrispClient.website.updatePeopleData(crispWebsiteId, email, {
-            data: peopleData,
-          });
+          return await CrispClient.website.updatePeopleData(crispWebsiteId, email, params);
         } catch {
           throw new Error(`Update crisp profile API call failed: ${error}`);
         }

src/user/dtos/admin-update-user.dto.ts

@@ -28,15 +28,15 @@ export class AdminUpdateUserDto {
 
   @IsOptional()
   @IsDate()
-  @ApiProperty({ type: 'date' })
+  @ApiProperty({ type: Date })
   lastActiveAt: Date;
 
   @SecureInput('email', { required: false, maxLength: 255 })
-  @ApiProperty({ type: 'email' })
+  @ApiProperty({ type: String, format: 'email' })
   email: string;
 
   @IsOptional()
   @IsBoolean({})
-  @ApiProperty({ type: 'boolean' })
+  @ApiProperty({ type: Boolean })
   isSuperAdmin: boolean;
 }

src/user/dtos/update-user.dto.ts

@@ -28,10 +28,10 @@ export class UpdateUserDto {
 
   @IsOptional()
   @IsDate()
-  @ApiProperty({ type: 'date' })
+  @ApiProperty({ type: Date })
   lastActiveAt: Date;
 
   @SecureInput('email', { required: false, maxLength: 255 })
-  @ApiProperty({ type: 'email' })
+  @ApiProperty({ type: String, format: 'email' })
   email: string;
 }