@platformatic/itc - npm Package Compare versions

Comparing version 3.13.1 to 3.14.0

lib/index.js

@@ -100,11 +100,13 @@ import { Unpromise } from '@watchable/unpromise'
 
-  let sanitized
-
   if (Buffer.isBuffer(data) || data instanceof Uint8Array) {
     // This will convert as Uint8Array
     return data
-  } else if (Array.isArray(data)) {
-    sanitized = []
+  }
 
-    for (const value of data) {
+  if (Array.isArray(data)) {
+    let sanitized = null
+    let needsSanitization = false
+
+    for (let i = 0; i < data.length; i++) {
+      const value = data[i]
       const valueType = typeof value
@@ -114,22 +116,66 @@
       if (valueType === 'function' || valueType === 'symbol') {
+        if (!needsSanitization) {
+          sanitized = data.slice(0, i)
+          needsSanitization = true
+        }
         continue
       }
 
-      sanitized.push(value && typeof value === 'object' ? sanitize(value, transferList) : value)
+      let sanitizedValue = value
+      if (value && typeof value === 'object') {
+        sanitizedValue = sanitize(value, transferList)
+        if (sanitizedValue !== value && !needsSanitization) {
+          sanitized = data.slice(0, i)
+          needsSanitization = true
+        }
+      }
+
+      if (needsSanitization) {
+        sanitized.push(sanitizedValue)
+      }
     }
-  } else {
-    sanitized = {}
 
-    for (const [key, value] of Object.entries(data)) {
-      const valueType = typeof value
+    return needsSanitization ? sanitized : data
+  }
 
-      if (valueType === 'function' || valueType === 'symbol') {
-        continue
+  // Handle plain objects
+  let sanitized = null
+  let needsSanitization = false
+
+  for (const [key, value] of Object.entries(data)) {
+    const valueType = typeof value
+
+    if (valueType === 'function' || valueType === 'symbol') {
+      if (!needsSanitization) {
+        sanitized = {}
+        // Copy all previous properties
+        for (const [k] of Object.entries(data)) {
+          if (k === key) break
+          sanitized[k] = data[k]
+        }
+        needsSanitization = true
       }
+      continue
+    }
 
-      sanitized[key] = value && typeof value === 'object' ? sanitize(value, transferList) : value
+    let sanitizedValue = value
+    if (value && typeof value === 'object') {
+      sanitizedValue = sanitize(value, transferList)
+      if (sanitizedValue !== value && !needsSanitization) {
+        sanitized = {}
+        // Copy all previous properties
+        for (const [k] of Object.entries(data)) {
+          if (k === key) break
+          sanitized[k] = data[k]
+        }
+        needsSanitization = true
+      }
     }
+
+    if (needsSanitization) {
+      sanitized[key] = sanitizedValue
+    }
   }
 
-  return sanitized
+  return needsSanitization ? sanitized : data
 }
@@ -136,0 +182,0 @@

package.json

 {
   "name": "@platformatic/itc",
-  "version": "3.13.1",
+  "version": "3.14.0",
   "description": "",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

New alerts

Long strings

Supply chain risk

Contains long string literals, which may be a sign of obfuscated or packed code.

Found 1 instance in 1 package

Fixed alerts

Long strings

Supply chain risk

Contains long string literals, which may be a sign of obfuscated or packed code.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

25912

4.97%

Lines of code

397

11.83%

No dependency changes