Tell serialize fn if it's being used on arguments

borisdiakur · Apr 22, 2024 · 8a0e0f2 · 8a0e0f2
1 parent 4eb6826
commit 8a0e0f2
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -18,11 +18,11 @@ export interface MemoizerOptions {
   noBody: boolean
   throwError: boolean
   retryOnInvalidCache: boolean
-  serialize: (val: unknown) => string
+  serialize: (val: unknown, isArgs: boolean) => string
   deserialize: (val: string) => unknown
 }
 
-function serialize(val: unknown) {
+function serialize(val: unknown, _isArgs: boolean) {
   const circRefColl: unknown[] = []
   return JSON.stringify(val, function (name, value) {
     if (typeof value === 'function') {
@@ -60,7 +60,7 @@ export function getCacheFilePath(
     fnStr = options.astBody ? JSON.stringify(fnStr) : fnStr
   }
 
-  const argsStr = options.serialize(args)
+  const argsStr = options.serialize(args, true)
   const hash = crypto
     .createHash('md5')
     .update(fnStr + argsStr + salt)
@@ -127,7 +127,7 @@ async function writeResult(
   let resultString
   if ((r && typeof r === 'object') || typeof r === 'string') {
     resultObj = { data: r }
-    resultString = optExt.serialize(resultObj)
+    resultString = optExt.serialize(resultObj, false)
   } else {
     resultString = '{"data":' + r + '}'
   }