chore(backend): Remove @ts-ignore directives (#5323)

Author: jacekradko

.changeset/quick-countries-learn.md

@@ -0,0 +1,5 @@
+---
+'@clerk/backend': patch
+---
+
+Refactoring code to remove @ts-ignore directives

packages/backend/src/api/request.ts

@@ -193,8 +193,7 @@ type LegacyRequestFunction = <T>(requestOptions: ClerkBackendApiRequestOptions)
 // TODO(dimkl): Will be probably be dropped in next major version
 function withLegacyRequestReturn(cb: any): LegacyRequestFunction {
   return async (...args) => {
-    // @ts-ignore
-    const { data, errors, totalCount, status, statusText, clerkTraceId } = await cb<T>(...args);
+    const { data, errors, totalCount, status, statusText, clerkTraceId } = await cb(...args);
     if (errors) {
       // instead of passing `data: errors`, we have set the `error.errors` because
       // the errors returned from callback is already parsed and passing them as `data`

packages/backend/src/util/decorateObjectWithResources.ts

@@ -55,10 +55,12 @@ export function stripPrivateDataFromObject<T extends WithResources<object>>(auth
 function prunePrivateMetadata(resource?: { private_metadata: any } | { privateMetadata: any } | null) {
   // Delete sensitive private metadata from resource before rendering in SSR
   if (resource) {
-    // @ts-ignore
-    delete resource['privateMetadata'];
-    // @ts-ignore
-    delete resource['private_metadata'];
+    if ('privateMetadata' in resource) {
+      delete resource['privateMetadata'];
+    }
+    if ('private_metadata' in resource) {
+      delete resource['private_metadata'];
+    }
   }
 
   return resource;

packages/backend/tsup.config.ts

@@ -2,7 +2,6 @@ import type { Options } from 'tsup';
 import { defineConfig } from 'tsup';
 
 import { runAfterLast } from '../../scripts/utils';
-// @ts-ignore
 import { name, version } from './package.json';
 
 export default defineConfig(overrideOptions => {