fix(client): parse Presto JSON response with custom reviver and BigInts

fix #24

Author: alonsovb

.nvmrc

@@ -1 +1 @@
-18.17.1
+21.0.0

presto-client/src/client.ts

@@ -9,6 +9,10 @@ import {
   Table,
 } from './types'
 
+function digitsToBigInt(_: string, value: unknown, { source }: { source: string }) {
+  return /^\d+$/.test(source) ? BigInt(source) : value
+}
+
 export class PrestoClient {
   private baseUrl: string
   private catalog?: string
@@ -270,7 +274,7 @@ export class PrestoClient {
         throw new Error(`Query failed: ${JSON.stringify(await response.text())}`)
       }
 
-      const prestoResponse = (await response.json()) as PrestoResponse
+      const prestoResponse = (await this.prestoConversionToJSON({ response })) as PrestoResponse
       if (!prestoResponse) {
         throw new Error(`Query failed with an empty response from the server.`)
       }
@@ -334,6 +338,13 @@ export class PrestoClient {
       method,
     })
   }
+
+  private async prestoConversionToJSON({ response }: { response: Response }): Promise<unknown> {
+    const text = await response.text()
+    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+    // @ts-ignore JSON.parse with a 3 argument reviver is a stage 3 proposal with some support, allow it here.
+    return JSON.parse(text, digitsToBigInt)
+  }
 }
 
 export default PrestoClient