@muze-nl/od-jsontag 0.2.3 → 0.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@muze-nl/od-jsontag",
-	"version": "0.2.3",
+	"version": "0.2.5",
 	"description": "On Demand JSONTag: parse/serialize large datastructures on demand, useful for sharing data between threads",
 	"type": "module",
 	"author": "Auke van Slooten <auke@muze.nl>",
@@ -19,7 +19,7 @@
 	},
 	"repository": {
 		"type": "git",
-		"url": "https://github.com/muze-nl/od-jsontag.git"
+		"url": "git+https://github.com/muze-nl/od-jsontag.git"
 	},
 	"engines": {
 		"node": ">=20.0.0"

package/src/parse.mjs

@@ -14,15 +14,6 @@ function stringToSAB(strData) {
     return uint8sab
 }
 
-function JSONTagProxy(target, handler) {
-    let p = new Proxy(target, handler)
-    let a = JSONTag.getAttributes(target)
-    JSONTag.setAttributes(target, a)
-    let t = JSONTag.getType(target)
-    JSONTag.setType(target, t)
-    return p
-}
-
 export default function parse(input, meta, immutable=true)
 {
     if (!meta) {
@@ -703,7 +694,7 @@ export default function parse(input, meta, immutable=true)
                         return undefined
                     }
                     if (Array.isArray(target[prop])) {
-                        return JSONTagProxy(target[prop], handlers.newArrayHandler)
+                        return new Proxy(target[prop], handlers.newArrayHandler)
                     }
                     return target[prop]
                 }
@@ -754,7 +745,7 @@ export default function parse(input, meta, immutable=true)
                             return undefined
                         }
                         if (Array.isArray(target[prop])) {
-                            return JSONTagProxy(target[prop], handlers.newArrayHandler)
+                            return new Proxy(target[prop], handlers.newArrayHandler)
                         }
                         return target[prop]
                     break
@@ -798,7 +789,7 @@ export default function parse(input, meta, immutable=true)
                     }
                     if (Array.isArray(target[prop])) {
                         target[prop][parent] = target[parent]
-                        return JSONTagProxy(target[prop], handlers.arrayHandler)
+                        return new Proxy(target[prop], handlers.arrayHandler)
                     }
                     return target[prop]
                 }
@@ -834,7 +825,7 @@ export default function parse(input, meta, immutable=true)
         },
         handler: {
             get(target, prop, receiver) {
-                firstParse(target)
+                firstParse(target, receiver)
                 switch(prop) {
                     case resultSet:
                         return meta.resultArray
@@ -875,17 +866,17 @@ export default function parse(input, meta, immutable=true)
                         }
                         if (Array.isArray(target[prop])) {
                             target[prop][parent] = target
-                            return JSONTagProxy(target[prop], handlers.arrayHandler)
+                            return new Proxy(target[prop], handlers.arrayHandler)
                         }
                         return target[prop]
                     break
                 }
             },
-            set(target, prop, value) {
+            set(target, prop, value, receiver) {
                 if (immutable && prop!==resultSet && prop!==source) {
                     throw new Error('dataspace is immutable')
                 }
-                firstParse(target)
+                firstParse(target, receiver)
                 switch(prop) {
                     case isChanged:
                         break
@@ -955,10 +946,20 @@ export default function parse(input, meta, immutable=true)
         }
     }
 
-    const firstParse = function(target) {
+    const firstParse = function(target, receiver) {
         if (!target[isParsed]) {
             parseValue(target[position], target)
             target[isParsed] = true
+            if (receiver) {
+                let tag = JSONTag.getType(target)
+                if (tag) {
+                    JSONTag.setType(receiver, tag)
+                }
+                let attributes = JSONTag.getAttributes(target)
+                if (attributes) {
+                    JSONTag.setAttributes(receiver, attributes)
+                }
+            }
         }
     }
 
@@ -973,7 +974,7 @@ export default function parse(input, meta, immutable=true)
         meta.resultArray.push('')
         value[getIndex] = index
         makeChildProxies(value)
-        let result = JSONTagProxy(value, handlers.newValueHandler)
+        let result = new Proxy(value, handlers.newValueHandler)
         meta.resultArray[index] = result
         return result
     }
@@ -994,7 +995,7 @@ export default function parse(input, meta, immutable=true)
         next()
         // newValueHandler makes sure that value[getBuffer] runs stringify
         // arrayHandler makes sure that changes in the array set targetIsChanged to true
-        return JSONTagProxy(cache, handlers.handler)
+        return new Proxy(cache, handlers.handler)
     }
 
     value = function(ob={})

package/src/serialize.mjs

@@ -157,10 +157,7 @@ export default function serialize(value, options={}) {
 	let currentSource = 0
 	let currentResult = 0
 	let skipCount = 0
-	let result = resultArray
-	if (options.changes) {
-		result = []
-	}
+	let result = []
 	while(currentSource<resultArray.length) {
 		if (resultArray[currentSource][isChanged] || !resultArray[currentSource][isProxy]) {
 			if (skipCount) {
@@ -169,7 +166,7 @@ export default function serialize(value, options={}) {
 				currentResult++
 			}
 			result[currentResult] = encoder.encode(innerStringify(currentSource))
-			if (options.meta?.index?.id) {
+			if (options.meta) {
 				const id=odJSONTag.getAttribute(resultArray[currentSource],'id')
 				if (id) {
 					options.meta.index.id.set(id, currentSource)
@@ -177,8 +174,8 @@ export default function serialize(value, options={}) {
 			}
 			currentResult++
 		} else if (!options.changes) {
-			resultArray[currentResult] = resultArray[currentSource][getBuffer](currentSource)
-			if (options.meta?.index?.id) {
+			result[currentResult] = resultArray[currentSource][getBuffer](currentSource)
+			if (options.meta) {
 				const id=odJSONTag.getAttribute(resultArray[currentSource],'id')
 				if (id) {
 					options.meta.index.id.set(id, currentSource)