@muze-nl/simplystore 0.5.6 → 0.5.8

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@muze-nl/simplystore",
-	"version": "0.5.6",
+	"version": "0.5.8",
 	"main": "src/server.mjs",
 	"type": "module",
 	"scripts": {

package/src/command-worker-module.mjs

@@ -1,5 +1,5 @@
 import JSONTag from '@muze-nl/jsontag'
-import {source, isChanged} from './symbols.mjs'
+import {source, isChanged, getIndex} from './symbols.mjs'
 import fastParse from './fastParse.mjs'
 import {stringToSAB,resultSetStringify} from './fastStringify.mjs'
 import writeFileAtomic from 'write-file-atomic'
@@ -25,8 +25,16 @@ export const metaIdProxy = {
         })
     },
     set: (id,ref) => {
-        //FIXME: is this correct?
-        meta.index.id.set(id, resultSet.length-1)
+        if (!meta.index.id.has(id)) {
+            if (ref[getIndex]) {
+                meta.index.id.set(id, ref[getIndex])
+            } else {
+                throw new Error('cannot set index.id for non-proxy')
+            }
+        } else {
+            let line = meta.index.id.get(id)
+            resultSet[line] = ref
+        }
     },
     get: (id) => {
         let index = meta.index.id.get(id)
@@ -44,15 +52,16 @@ export const metaIdProxy = {
 }
 
 export const FastJSONTag = {
-    getType: (obj) => JSONTag.getType(obj[source]),
-    getAttribute: (obj, attr) => JSONTag.getAttribute(obj[source],attr),
+    getType: (obj) => JSONTag.getType(obj?.[source]),
+    getAttribute: (obj, attr) => JSONTag.getAttribute(obj?.[source],attr),
     setAttribute: (obj, attr, value) => {
+        if (!obj) return
         obj[isChanged] = true
         return JSONTag.setAttribute(obj[source], attr, value)
     },
-    getAttributes: (obj) => JSONTag.getAttributes(obj[source]),
-    getAttributeString: (obj) => JSONTag.getAttributesString(obj[source]),
-    getTypeString: (obj) => JSONTag.getTypeString(obj[source])
+    getAttributes: (obj) => JSONTag.getAttributes(obj?.[source]),
+    getAttributeString: (obj) => JSONTag.getAttributesString(obj?.[source]),
+    getTypeString: (obj) => JSONTag.getTypeString(obj?.[source])
 }
 
 export async function initialize(task) {

package/src/fastParse.mjs

@@ -726,7 +726,7 @@ export default function parse(input, meta, immutable=true)
                 if (JSONTag.getType(value)==='object' && !value[isProxy]) {
                     value = getNewValueProxy(value)
                 }
-                target[prop] = val
+                target[prop] = value
                 return true                    
             }
         }
@@ -761,11 +761,15 @@ export default function parse(input, meta, immutable=true)
             get(target, prop) {
                 if (target[prop] instanceof Function) {
                     if (['copyWithin','fill','pop','push','reverse','shift','sort','splice','unshift'].indexOf(prop)!==-1) {
+                        if (immutable) {
+                            throw new Error('dataspace is immutable')
+                        }
                         targetIsChanged = true
                     }
                     return (...args) => {
                         args = args.map(arg => {
                             if (JSONTag.getType(arg)==='object' && !arg[isProxy]) {
+                                console.log('proxying arg')
                                 arg = getNewValueProxy(arg)
                             }
                             return arg
@@ -782,6 +786,9 @@ export default function parse(input, meta, immutable=true)
                 }
             },
             set(target, prop, value) {
+                if (immutable) {
+                    throw new Error('dataspace is immutable')
+                }
                 if (JSONTag.getType(value)==='object' && !value[isProxy]) {
                     value = getNewValueProxy(value)
                 } 
@@ -790,12 +797,24 @@ export default function parse(input, meta, immutable=true)
                 return true
             },
             deleteProperty(target, prop) {
+                if (immutable) {
+                    throw new Error('dataspace is immutable')
+                }
                 //FIXME: if target[prop] was the last reference to an object
                 //that object should be deleted so that its line will become empty
                 //when stringifying resultArray again
                 delete target[prop]
                 targetIsChanged = true
                 return true
+            },
+            ownKeys: (target) => {
+                return Reflect.ownKeys(target)
+            },
+            getOwnPropertyDescriptor: (target, prop) => {
+                return {
+                    enumerable: true,
+                    configurable: true
+                }
             }
         }
         let handler = {
@@ -831,7 +850,7 @@ export default function parse(input, meta, immutable=true)
                         if (!immutable && Array.isArray(target[prop])) {
                             return new Proxy(target[prop], arrayHandler)
                         }
-                        return target[prop] // FIXME: make arrays immutable as well
+                        return target[prop]
                     break
                 }
             },
@@ -1021,5 +1040,4 @@ export default function parse(input, meta, immutable=true)
     }
     
     return resultArray
-}
-
+}

package/src/server.mjs

@@ -289,12 +289,12 @@ async function main(options) {
     })
 
     function checkCommand(req, res) {
-        let error
+        let error, command, commandOK
         let commandStr = req.body.toString() // raw body through express.raw()
         try {
-            let command = JSONTag.parse(commandStr)
+            command = JSONTag.parse(commandStr)
 
-            let commandOK = {
+            commandOK = {
                 command: command?.id,
                 code: 202,
                 status: 'accepted'