chore: 🤖 bump NestJS and other deps

Author: prashantasdeveloper

package.json

@@ -39,58 +39,58 @@
     "docker:build:no-cache": "docker build --no-cache -t polymeshassociation/polymesh-rest-api:local ."
   },
   "dependencies": {
-    "@golevelup/ts-jest": "^0.3.3",
-    "@nestjs/axios": "^3.0.2",
-    "@nestjs/common": "^10.3.7",
-    "@nestjs/config": "^3.2.2",
-    "@nestjs/core": "^10.3.7",
-    "@nestjs/passport": "^10.0.3",
-    "@nestjs/platform-express": "^10.3.7",
-    "@nestjs/schedule": "^4.0.2",
-    "@nestjs/swagger": "^7.3.1",
-    "@nestjs/typeorm": "^10.0.2",
+    "@golevelup/ts-jest": "0.3.3",
+    "@nestjs/axios": "^4.0.1",
+    "@nestjs/common": "^11.1.6",
+    "@nestjs/config": "^4.0.2",
+    "@nestjs/core": "^11.1.6",
+    "@nestjs/passport": "^11.0.5",
+    "@nestjs/platform-express": "^11.1.6",
+    "@nestjs/schedule": "^6.0.1",
+    "@nestjs/swagger": "^11.2.0",
+    "@nestjs/typeorm": "^11.0.0",
     "@polymeshassociation/fireblocks-signing-manager": "^2.5.0",
-    "@polymeshassociation/hashicorp-vault-signing-manager": "^3.4.0",
-    "@polymeshassociation/local-signing-manager": "^3.3.0",
+    "@polymeshassociation/hashicorp-vault-signing-manager": "^3.5.0",
+    "@polymeshassociation/local-signing-manager": "^3.5.2",
     "@polymeshassociation/polymesh-sdk": "v29.1.0-beta.9",
     "@polymeshassociation/polymesh-types": "^6.2.0",
-    "@polymeshassociation/signing-manager-types": "^3.2.0",
+    "@polymeshassociation/signing-manager-types": "^3.4.2",
     "class-transformer": "0.5.1",
     "class-validator": "^0.14.0",
-    "joi": "17.4.0",
-    "json-stable-stringify": "^1.0.2",
+    "joi": "18.0.1",
+    "json-stable-stringify": "^1.3.0",
     "lodash": "4.17.21",
     "passport": "^0.7.0",
     "passport-custom": "^1.1.1",
     "passport-headerapikey": "^1.2.2",
-    "pg": "^8.11.5",
+    "pg": "^8.16.3",
     "reflect-metadata": "0.2.2",
     "rhea-promise": "^3.0.1",
     "rimraf": "3.0.2",
-    "rxjs": "^7.5.7",
-    "swagger-ui-express": "5.0.0",
-    "typeorm": "^0.3.17"
+    "rxjs": "^7.8.2",
+    "swagger-ui-express": "5.0.1",
+    "typeorm": "^0.3.27"
   },
   "devDependencies": {
     "@babel/plugin-transform-modules-commonjs": "7.22.11",
-    "@commitlint/cli": "^17.7.1",
-    "@commitlint/config-conventional": "^17.7.0",
-    "@nestjs/cli": "^10.1.17",
-    "@nestjs/schematics": "^10.0.2",
-    "@nestjs/testing": "^10.2.4",
+    "@commitlint/cli": "^20.1.0",
+    "@commitlint/config-conventional": "^20.0.0",
+    "@nestjs/cli": "^11.0.10",
+    "@nestjs/schematics": "^11.0.8",
+    "@nestjs/testing": "^11.1.6",
     "@semantic-release/changelog": "^6.0.3",
-    "@semantic-release/exec": "^6.0.3",
-    "@semantic-release/npm": "^11.0.0",
-    "@types/axios": "^0.14.0",
-    "@types/cron": "^1.7.3",
-    "@types/express": "4.17.17",
+    "@semantic-release/exec": "^7.1.0",
+    "@semantic-release/npm": "^12.0.2",
+    "@types/axios": "^0.14.4",
+    "@types/cron": "^2.4.3",
+    "@types/express": "5.0.3",
     "@types/jest": "29.5.4",
-    "@types/jest-when": "^3.5.2",
-    "@types/json-stable-stringify": "^1.0.34",
+    "@types/jest-when": "^3.5.5",
+    "@types/json-stable-stringify": "^1.2.0",
     "@types/lodash": "^4.14.197",
     "@types/node": "^18.15.11",
-    "@types/passport": "^1.0.11",
-    "@types/supertest": "2.0.12",
+    "@types/passport": "^1.0.17",
+    "@types/supertest": "6.0.3",
     "@typescript-eslint/eslint-plugin": "6.9.1",
     "@typescript-eslint/parser": "6.9.1",
     "@zerollup/ts-transform-paths": "1.7.11",
@@ -106,16 +106,17 @@
     "eslint-plugin-simple-import-sort": "10.0.0",
     "husky": "8.0.3",
     "jest": "29.7.0",
-    "jest-when": "^3.6.0",
+    "jest-when": "^3.7.0",
     "lint-staged": "14.0.1",
+    "path-to-regexp": "^8.3.0",
     "prettier": "2.8.8",
     "prettier-eslint": "15.0.1",
     "prettier-eslint-cli": "7.1.0",
-    "semantic-release": "^22.0.0",
-    "supertest": "6.1.3",
-    "ts-jest": "29.1.1",
-    "ts-loader": "9.4.4",
-    "ts-node": "10.9.1",
+    "semantic-release": "^24.2.9",
+    "supertest": "7.1.4",
+    "ts-jest": "29.4.4",
+    "ts-loader": "9.5.4",
+    "ts-node": "10.9.2",
     "tsconfig-paths": "^4.2.0",
     "typescript": "4.8.2"
   },

src/accounts/accounts.service.spec.ts

@@ -86,7 +86,7 @@ describe('AccountsService', () => {
 
         const address = 'address';
 
-        await expect(() => service.findOne(address)).rejects.toThrowError();
+        await expect(() => service.findOne(address)).rejects.toThrow();
 
         expect(handleSdkErrorSpy).toHaveBeenCalledWith(mockError);
       });
@@ -217,7 +217,7 @@ describe('AccountsService', () => {
 
         const handleSdkErrorSpy = jest.spyOn(transactionsUtilModule, 'handleSdkError');
 
-        await expect(() => service.getPermissions('address')).rejects.toThrowError();
+        await expect(() => service.getPermissions('address')).rejects.toThrow();
 
         expect(handleSdkErrorSpy).toHaveBeenCalledWith(mockError);
       });

src/assets/assets.service.spec.ts

@@ -117,7 +117,7 @@ describe('AssetsService', () => {
 
         const address = 'address';
 
-        await expect(() => service.findOne(address)).rejects.toThrowError();
+        await expect(() => service.findOne(address)).rejects.toThrow();
 
         expect(handleSdkErrorSpy).toHaveBeenCalledWith(mockError);
       });
@@ -144,7 +144,7 @@ describe('AssetsService', () => {
 
         const address = 'address';
 
-        await expect(() => service.findFungible(address)).rejects.toThrowError();
+        await expect(() => service.findFungible(address)).rejects.toThrow();
 
         expect(handleSdkErrorSpy).toHaveBeenCalledWith(mockError);
       });

src/assets/models/transfer-restrictions-values.model.ts

@@ -129,7 +129,6 @@ export class TransferRestrictionClaimDetailsModel {
   @ApiPropertyOneOf({
     description:
       'Breakdown of holders or balances for the statistic. Jurisdiction claims include an entry per country code, while other claims display holders with/without the claim.',
-    required: false,
     union: [
       TransferRestrictionClaimValueBreakdownModel,
       {

src/auth/auth.service.spec.ts

@@ -76,7 +76,7 @@ describe('AuthService', () => {
 
       await service.deleteApiKey({ apiKey: testApiKey });
 
-      expect(mockApiKeyRepo.deleteApiKey).toBeCalledWith(testApiKey);
+      expect(mockApiKeyRepo.deleteApiKey).toHaveBeenCalledWith(testApiKey);
     });
   });
 });

src/auth/strategies/api-key.strategy.spec.ts

@@ -1,9 +1,7 @@
 import { Test, TestingModule } from '@nestjs/testing';
 import { when } from 'jest-when';
-import passport from 'passport';
 
 import { ApiKeyStrategy } from '~/auth/strategies/api-key.strategy';
-import { AuthStrategy } from '~/auth/strategies/strategies.consts';
 import { MockAuthService, mockAuthServiceProvider } from '~/test-utils/service-mocks';
 
 describe('ApiKeyStrategy', () => {
@@ -26,42 +24,15 @@ describe('ApiKeyStrategy', () => {
   });
 
   it('should verify with the user when given a valid api key', async () => {
-    const mockRequest = {
-      headers: {
-        'x-api-key': mockApiKey,
-      },
-    };
+    when(authService.validateApiKey).calledWith(mockApiKey).mockResolvedValue(mockedUser);
 
-    when(authService.validateApiKey).calledWith(mockApiKey).mockReturnValue(mockedUser);
-
-    let authorizedUser;
-    passport.authenticate(
-      AuthStrategy.ApiKey,
-      (request: unknown, user: Express.User | false | null) => {
-        authorizedUser = user;
-      }
-    )(mockRequest, {}, {});
-
-    expect(authorizedUser).toEqual(mockedUser);
+    const result = await strategy.validate(mockApiKey);
+    expect(result).toEqual(mockedUser);
   });
 
   it('should return an Unauthorized response if the key is not found', async () => {
-    const mockRequest = {
-      headers: {
-        'x-api-key': 'not-a-secret',
-      },
-    };
-
-    when(authService.validateApiKey).calledWith(mockApiKey).mockReturnValue(mockedUser);
-
-    let authorizedUser;
-    passport.authenticate(
-      AuthStrategy.ApiKey,
-      (request: unknown, user: Express.User | false | null) => {
-        authorizedUser = user;
-      }
-    )(mockRequest, {}, {});
+    when(authService.validateApiKey).calledWith('not-a-secret').mockResolvedValue(null);
 
-    expect(authorizedUser).toBeFalsy();
+    await expect(strategy.validate('not-a-secret')).rejects.toThrow('API key not found');
   });
 });

src/auth/strategies/api-key.strategy.ts

@@ -5,24 +5,26 @@ import { HeaderAPIKeyStrategy } from 'passport-headerapikey';
 import { AuthService } from '~/auth/auth.service';
 import { AuthStrategy } from '~/auth/strategies/strategies.consts';
 import { AppUnauthorizedError } from '~/common/errors';
+import { UserModel } from '~/users/model/user.model';
 
 export const apiKeyHeader = 'x-api-key';
 
-// eslint-disable-next-line @typescript-eslint/ban-types
-type Callback = (err: Error | null, user?: Object, info?: Object) => void;
-
 /**
  * authenticate with an API key
  */
 @Injectable()
 export class ApiKeyStrategy extends PassportStrategy(HeaderAPIKeyStrategy, AuthStrategy.ApiKey) {
-  constructor(private authService: AuthService) {
-    super({ header: apiKeyHeader }, false, (apiKey: string, done: Callback) => {
-      const user = this.authService.validateApiKey(apiKey);
-      if (!user) {
-        return done(new AppUnauthorizedError('API key not found'), undefined);
-      }
-      return done(null, user);
-    });
+  constructor(private readonly authService: AuthService) {
+    super({ header: apiKeyHeader, prefix: '' }, false);
+  }
+
+  public async validate(apiKey: string): Promise<UserModel> {
+    const user = await this.authService.validateApiKey(apiKey);
+
+    if (!user) {
+      throw new AppUnauthorizedError('API key not found');
+    }
+
+    return user;
   }
 }

src/auth/strategies/open.strategy.spec.ts

@@ -1,8 +1,6 @@
 import { Test, TestingModule } from '@nestjs/testing';
-import passport from 'passport';
 
 import { OpenStrategy } from '~/auth/strategies/open.strategy';
-import { AuthStrategy } from '~/auth/strategies/strategies.consts';
 import { defaultUser } from '~/users/user.consts';
 
 describe('OpenStrategy', () => {
@@ -21,14 +19,8 @@ describe('OpenStrategy', () => {
   });
 
   it('should verify with the open user', async () => {
-    let authorizedUser;
-    passport.authenticate(
-      AuthStrategy.Open,
-      (request: unknown, user: Express.User | false | null) => {
-        authorizedUser = user;
-      }
-    )({}, {}, {});
-
-    expect(authorizedUser).toEqual(defaultUser);
+    // Test the validate method directly
+    const result = strategy.validate();
+    expect(result).toEqual(defaultUser);
   });
 });

src/auth/strategies/open.strategy.ts

@@ -1,6 +1,6 @@
 import { Injectable } from '@nestjs/common';
 import { PassportStrategy } from '@nestjs/passport';
-import { Strategy, VerifyCallback } from 'passport-custom';
+import { Strategy } from 'passport-custom';
 
 import { AuthStrategy } from '~/auth/strategies/strategies.consts';
 import { defaultUser } from '~/users/user.consts';
@@ -12,9 +12,7 @@ import { defaultUser } from '~/users/user.consts';
  */
 @Injectable()
 export class OpenStrategy extends PassportStrategy(Strategy, AuthStrategy.Open) {
-  constructor() {
-    const verifyEveryone: VerifyCallback = (req, done) => done(null, defaultUser);
-
-    super(verifyEveryone);
+  public validate(): Express.User {
+    return defaultUser;
   }
 }

src/authorizations/authorizations.service.spec.ts

@@ -178,7 +178,7 @@ describe('AuthorizationsService', () => {
         await expect(() =>
           // eslint-disable-next-line @typescript-eslint/no-explicit-any
           service.findOne(mockAccount as any, new BigNumber(1))
-        ).rejects.toThrowError();
+        ).rejects.toThrow();
 
         expect(handleSdkErrorSpy).toHaveBeenCalledWith(mockError);
       });
@@ -226,7 +226,7 @@ describe('AuthorizationsService', () => {
         .calledWith(mockIdentity as any, id)
         .mockRejectedValue(mockError);
 
-      await expect(() => service.getAuthRequest(address, id)).rejects.toThrowError(mockError);
+      await expect(() => service.getAuthRequest(address, id)).rejects.toThrow(mockError);
 
       when(findOneSpy)
         // eslint-disable-next-line @typescript-eslint/no-explicit-any