Use literals for creating BigInts when possible (#5093)

watson · web-flow · commit f76beab3ad7f · 2025-01-13T09:09:25.000Z
diff --git a/packages/dd-trace/src/datastreams/fnv.js b/packages/dd-trace/src/datastreams/fnv.js
@@ -15,7 +15,7 @@ function fnv64 (data) {
     data = Buffer.from(data, 'utf-8')
   }
   const byteArray = new Uint8Array(data)
-  return fnv(byteArray, FNV1_64_INIT, FNV_64_PRIME, BigInt(2) ** BigInt(64))
+  return fnv(byteArray, FNV1_64_INIT, FNV_64_PRIME, 2n ** 64n)
 }
 
 module.exports = {
diff --git a/packages/dd-trace/src/debugger/devtools_client/index.js b/packages/dd-trace/src/debugger/devtools_client/index.js
@@ -25,8 +25,8 @@ const expression = `
 const threadId = parentThreadId === 0 ? `pid:${process.pid}` : `pid:${process.pid};tid:${parentThreadId}`
 const threadName = parentThreadId === 0 ? 'MainThread' : `WorkerThread:${parentThreadId}`
 
-const oneSecondNs = BigInt(1_000_000_000)
-let globalSnapshotSamplingRateWindowStart = BigInt(0)
+const oneSecondNs = 1_000_000_000n
+let globalSnapshotSamplingRateWindowStart = 0n
 let snapshotsSampledWithinTheLastSecond = 0
 
 // WARNING: The code above the line `await session.post('Debugger.resume')` is highly optimized. Please edit with care!
diff --git a/packages/dd-trace/test/llmobs/span_processor.spec.js b/packages/dd-trace/test/llmobs/span_processor.spec.js
@@ -119,7 +119,7 @@ describe('span processor', () => {
     it('removes problematic fields from the metadata', () => {
       // problematic fields are circular references or bigints
       const metadata = {
-        bigint: BigInt(1),
+        bigint: 1n,
         deep: {
           foo: 'bar'
         },

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ function fnv64 (data) {`
`15`	`15`	`data = Buffer.from(data, 'utf-8')`
`16`	`16`	`}`
`17`	`17`	`const byteArray = new Uint8Array(data)`
`18`		`- return fnv(byteArray, FNV1_64_INIT, FNV_64_PRIME, BigInt(2) ** BigInt(64))`
	`18`	`+ return fnv(byteArray, FNV1_64_INIT, FNV_64_PRIME, 2n ** 64n)`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`module.exports = {`