@knowlearning/agents 0.9.140 → 0.9.141

package/agents/browser/root.js

@@ -14,30 +14,32 @@ const API_HOST = isLocal() ? DEVELOPMENT_HOST : REMOTE_HOST
 //  TODO: remove this hack when we can set partitioned sid cookie through websocket handshake
 //        deno is partly in the way on teh set side, and browser support is in the way for
 //        the client side.
-fetch(`http${ SECURE ? 's' : '' }://${API_HOST}/_sid-check`, { method: 'GET', credentials: 'include' })
-  .then(async response => {
-    const hasLocalStorageSID = !!localStorage.getItem('sid')
-    if (response.status === 201) {
-      if (!hasLocalStorageSID) {
-        const sentSid = await response.text()
-        localStorage.setItem('sid', sentSid)
-        location.reload()
-      }
+async function ensureSidEstablished() {
+  const response = await fetch(`http${ SECURE ? 's' : '' }://${API_HOST}/_sid-check`, { method: 'GET', credentials: 'include' })
+  const hasLocalStorageSID = !!localStorage.getItem('sid')
+  if (response.status === 201) {
+    if (!hasLocalStorageSID) {
+      const sentSid = await response.text()
+      localStorage.setItem('sid', sentSid)
+      location.reload()
     }
-    else if (response.status === 200) {
-      //  if we reach here, assumably the server has seen an sid cookie
-      if (hasLocalStorageSID) {
-        localStorage.removeItem('sid')
-        location.reload()
-      }
-    }
-    else {
-      console.warn('Issue Connecting To the API Server')
+  }
+  else if (response.status === 200) {
+    //  if we reach here, assumably the server has seen an sid cookie
+    if (hasLocalStorageSID) {
+      localStorage.removeItem('sid')
+      location.reload()
     }
-  })
+  }
+  else {
+    console.warn('Issue Connecting To the API Server')
+  }
+}
 
 export default options => {
+  ensureSidEstablished()
   const Connection = function () {
+
     const ws = new WebSocket(`ws${ SECURE ? 's' : '' }://${API_HOST}`)
 
     this.send = message => ws.send(JSON.stringify(message))

package/package.json

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