querysub 0.168.0 → 0.169.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "querysub",
-  "version": "0.168.0",
+  "version": "0.169.0",
   "main": "index.js",
   "license": "MIT",
   "note1": "note on node-forge fork, see https://github.com/digitalbazaar/forge/issues/744 for details",

package/src/2-proxy/PathValueProxyWatcher.ts

@@ -1141,10 +1141,17 @@ export class PathValueProxyWatcher {
                     },
                 }) as Result;
 
-                // NOTE: Always deep clone. It's too confusing for a partial object to be returned. Any issues caused by
+                // NOTE: Deep clone non-local paths. It's too confusing for a partial object to be returned. Any issues caused by
                 //  this (aka, returning the entire database), would also happen if we manually added deep clones before
                 //  we returned values (and both should be throttled at a framework level so we don't break the database).
-                rawResult = deepCloneCborx(rawResult);
+                //  - For local paths there is a risk that there are functions
+                try {
+                    rawResult = deepCloneCborx(rawResult);
+                } catch {
+                    // Unfortunately, cborx throws on functions.
+                    // TODO: Use a recursive clone technique that doesn't throw on functions
+                    rawResult = atomicObjectRead(rawResult);
+                }
 
                 result = { result: rawResult };
                 watcher.consecutiveErrors = 0;