@knowlearning/agents 0.4.5 → 0.4.7

package/agents/generic.js

@@ -14,10 +14,6 @@ function sanitizeJSONPatchPathSegment(s) {
   else return s
 }
 
-function copy(value) {
-  return JSON.parse(JSON.stringify(value))
-}
-
 export default function Agent({ host, token, WebSocket, protocol='ws', uuid, fetch, applyPatch, login, logout, reboot }) {
   let ws
   let user
@@ -40,11 +36,10 @@ export default function Agent({ host, token, WebSocket, protocol='ws', uuid, fet
   let lastSentSI = -1
   let lastHeartbeat
 
-  //  TODO: probably an use "mutate" function from below
-  const subscriptions = new MutableProxy({}, patch => queueMessage({scope: 'subscriptions', patch}))
-  const uploads = new MutableProxy({}, patch => queueMessage({scope: 'uploads', patch}))
-  const downloads = new MutableProxy({}, patch => queueMessage({scope: 'downloads', patch}))
-  const patches = new MutableProxy({}, patch => queueMessage({scope: 'patches', patch}))
+  const subscriptions = mutate('subscriptions', false)
+  const uploads = mutate('uploads', false)
+  const downloads = mutate('downloads', false)
+  const patches = mutate('patches', false)
 
   log('INITIALIZING AGENT CONNECTION')
   initWS()
@@ -219,10 +214,14 @@ export default function Agent({ host, token, WebSocket, protocol='ws', uuid, fet
         keyToSubscriptionId[k] = id
         watchers[k] = []
         states[k] = new Promise(async resolve => {
-          subscriptions[id] = { scope, ii: null, session }
 
-          if (d !== domain) subscriptions[id].domain = d
-          if (u !== user) subscriptions[id].user = u
+          subscriptions[id] = {
+            session,
+            domain: d,
+            user: u,
+            scope,
+            ii: null
+          }
 
           const { ii, state } = await lastMessageResponse()
           subscriptions[id].ii = ii
@@ -344,10 +343,9 @@ export default function Agent({ host, token, WebSocket, protocol='ws', uuid, fet
     return interact('claims', [{ op: 'add', path: [domain], value: null }])
   }
 
-  async function mutate(scope, initialize=true) {
-    //  TODO: probably can remove redundant copy
-    const initial = initialize ? copy(await state(scope) || {}) : {}
-    return new MutableProxy(initial, patch => interact(scope, patch))
+  function mutate(scope, initialize=true) {
+    const mp = s => new MutableProxy(s || {}, patch => interact(scope, patch))
+    return initialize ? state.then(mp) : mp({})
   }
 
   function reset(scope) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@knowlearning/agents",
-    "version": "0.4.5",
+    "version": "0.4.7",
     "description": "API for embedding applications in KnowLearning systems.",
     "main": "node.js",
     "browser": "browser.js",