feat: bump typed-express-router to 2.0.0

Ticket: DX-1672

Author: starfy84

package-lock.json

@@ -26,7 +26,7 @@
     "@api-ts/openapi-generator": "^5.7.0",
     "@api-ts/response": "^2.1.0",
     "@api-ts/superagent-wrapper": "^1.3.3",
-    "@api-ts/typed-express-router": "^1.1.13",
+    "@api-ts/typed-express-router": "2.0.0",
     "@bitgo-beta/sdk-api": "^1.10.1-beta.1067",
     "@bitgo-beta/abstract-cosmos": "^1.0.1-beta.1049",
     "@bitgo-beta/sdk-lib-mpc": "8.2.0-beta.1060",

package.json

@@ -35,6 +35,7 @@ import { ecdsaMPCv2Finalize } from '../../api/advancedWalletManager/handlers/ecd
 import { ecdsaMPCv2Recovery } from '../../api/advancedWalletManager/handlers/ecdsaMPCv2Recovery';
 import { signEddsaRecoveryTransaction } from '../../api/advancedWalletManager/handlers/signEddsaRecoveryTransaction';
 import { isEddsaCoin } from '../../shared/coinUtils';
+import { customDecodeErrorFormatter } from '../../shared/errorFormatters';
 
 // Request type for /key/independent endpoint
 const IndependentKeyRequest = {
@@ -489,7 +490,9 @@ export type GenericAwmApiSpecRouteRequest = AwmApiSpecRouteRequest<any, any>;
 export function createKeyGenRouter(
   config: AdvancedWalletManagerConfig,
 ): WrappedRouter<typeof AdvancedWalletManagerApiSpec> {
-  const router = createRouter(AdvancedWalletManagerApiSpec);
+  const router = createRouter(AdvancedWalletManagerApiSpec, {
+    decodeErrorFormatter: customDecodeErrorFormatter,
+  });
   // Add middleware
   router.use(express.json());
   router.use(prepareBitGo(config));

src/advancedWalletManager/routers/advancedWalletManagerApiSpec.ts

@@ -4,6 +4,7 @@ import { Response } from '@api-ts/response';
 import pjson from '../../../package.json';
 import { responseHandler } from '../../shared/middleware';
 import { PingResponseType, VersionResponseType } from '../../types/health';
+import { customDecodeErrorFormatter } from '../../shared/errorFormatters';
 
 // API Response types
 const PingResponse: HttpResponse = {
@@ -38,7 +39,9 @@ export const HealthCheckApiSpec = apiSpec({
 
 // Create router with handlers
 export function createHealthCheckRouter(): WrappedRouter<typeof HealthCheckApiSpec> {
-  const router = createRouter(HealthCheckApiSpec);
+  const router = createRouter(HealthCheckApiSpec, {
+    decodeErrorFormatter: customDecodeErrorFormatter,
+  });
   // Ping endpoint handler
   router.post('v1.health.ping', [
     responseHandler(() =>

src/advancedWalletManager/routers/healthCheck.ts

@@ -7,6 +7,7 @@ import logger from '../../../logger';
 import { responseHandler } from '../../../shared/middleware';
 import { AdvancedWalletManagerClient } from '../clients/advancedWalletManagerClient';
 import { PingResponseType, VersionResponseType } from '../../../types/health';
+import { customDecodeErrorFormatter } from '../../../shared/errorFormatters';
 
 // Response type for /ping/advancedWalletManager endpoint
 const PingAwmResponse: HttpResponse = {
@@ -54,7 +55,9 @@ export const AdvancedWalletManagerHealthSpec = apiSpec({
 export function createAdvancedWalletManagerHealthRouter(
   cfg: MasterExpressConfig,
 ): WrappedRouter<typeof AdvancedWalletManagerHealthSpec> {
-  const router = createRouter(AdvancedWalletManagerHealthSpec);
+  const router = createRouter(AdvancedWalletManagerHealthSpec, {
+    decodeErrorFormatter: customDecodeErrorFormatter,
+  });
 
   // Create an instance of awmClient
   const awmClient = new AdvancedWalletManagerClient(cfg);

src/api/master/routers/awmExpressHealth.ts

@@ -4,6 +4,7 @@ import { Response } from '@api-ts/response';
 import pjson from '../../../../package.json';
 import { responseHandler } from '../../../shared/middleware';
 import { PingResponseType, VersionResponseType } from '../../../types/health';
+import { customDecodeErrorFormatter } from '../../../shared/errorFormatters';
 
 // API Response types
 const PingResponse: HttpResponse = {
@@ -40,7 +41,9 @@ export const HealthCheckApiSpec = apiSpec({
 export function createHealthCheckRouter(
   serverType: string,
 ): WrappedRouter<typeof HealthCheckApiSpec> {
-  const router = createRouter(HealthCheckApiSpec);
+  const router = createRouter(HealthCheckApiSpec, {
+    decodeErrorFormatter: customDecodeErrorFormatter,
+  });
 
   // Ping endpoint handler
   router.post('v1.health.ping', [

src/api/master/routers/healthCheck.ts

@@ -28,6 +28,7 @@ import { handleConsolidateUnspents } from '../handlers/handleConsolidateUnspents
 import { handleSignAndSendTxRequest } from '../handlers/handleSignAndSendTxRequest';
 import { handleRecoveryConsolidationsOnPrem } from '../handlers/recoveryConsolidationsWallet';
 import { ErrorResponses } from '../../../shared/errors';
+import { customDecodeErrorFormatter } from '../../../shared/errorFormatters';
 
 export type ScriptType2Of3 = utxolib.bitgo.outputScripts.ScriptType2Of3;
 
@@ -801,7 +802,9 @@ export type GenericMasterApiSpecRouteRequest = MasterApiSpecRouteRequest<any, an
 export function createMasterApiRouter(
   cfg: MasterExpressConfig,
 ): WrappedRouter<typeof MasterBitGoExpressApiSpec> {
-  const router = createRouter(MasterBitGoExpressApiSpec);
+  const router = createRouter(MasterBitGoExpressApiSpec, {
+    decodeErrorFormatter: customDecodeErrorFormatter,
+  });
 
   // Add middleware to all routes
   router.use(parseBody);

src/api/master/routers/masterBitGoExpressApiSpec.ts

@@ -0,0 +1,8 @@
+import * as PathReporter from 'io-ts/lib/PathReporter';
+import { DecodeErrorFormatterFn } from '@api-ts/typed-express-router';
+
+export const customDecodeErrorFormatter: DecodeErrorFormatterFn = (errs, _req) => {
+  const validationErrors = PathReporter.failure(errs);
+  const validationErrorMessage = validationErrors.join('\n');
+  return { error: validationErrorMessage };
+};