@mblaney/holster 1.0.18 → 1.0.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mblaney/holster",
-  "version": "1.0.18",
+  "version": "1.0.20",
   "description": "Holster is a service that allows synchronising data between devices using Node, Deno, Bun or the browser.",
   "type": "module",
   "main": "index.js",

package/src/wire.js

@@ -15,7 +15,7 @@ if (typeof globalThis.WebSocket === "undefined") {
 // Rate limiting with throttling
 const createRateLimiter = isTestEnv => {
   const clients = new Map()
-  const maxRequests = 300 // 5 requests per second
+  const maxRequests = 1000 // requests per minute
   const windowMs = 60000 // 1 minute window
   const disconnectThreshold = 10 // Disconnect after 10 violations
   let cleanupInterval = null
@@ -264,20 +264,27 @@ const Wire = opt => {
   const put = async (msg, send) => {
     // Store updates returned from Ham.mix and defer updates if required.
     const update = await Ham.mix(msg.put, graph, opt.secure, listen)
-    if (Object.keys(update.now).length) {
-      if (!(await check(update.now, send))) return
-
-      store.put(update.now, err => {
-        send(
-          JSON.stringify({
-            "#": dup.track(utils.text.random(9)),
-            "@": msg["#"],
-            err: err,
-          }),
-        )
-      })
+    if (Object.keys(update.now).length === 0) {
+      // No updates to store, check deferred.
+      if (Object.keys(update.defer).length !== 0) {
+        setTimeout(() => put({put: update.defer}, send), update.wait)
+      }
+      return
     }
-    if (Object.keys(update.defer).length) {
+
+    if (!(await check(update.now, send))) return
+
+    store.put(update.now, err => {
+      send(
+        JSON.stringify({
+          "#": dup.track(utils.text.random(9)),
+          "@": msg["#"],
+          err: err,
+        }),
+      )
+    })
+
+    if (Object.keys(update.defer).length !== 0) {
       setTimeout(() => put({put: update.defer}, send), update.wait)
     }
   }
@@ -353,29 +360,29 @@ const Wire = opt => {
         // used whereas wire spec needs to handle clock skew for updates
         // across the network.
         const update = await Ham.mix(data, graph, opt.secure, listen)
+        if (Object.keys(update.now).length === 0) {
+          // No updates, still respond to callback.
+          if (cb) cb(null)
+          return
+        }
 
-        if (Object.keys(update.now).length !== 0) {
-          if (!(await check(update.now, send, cb))) return
-
-          // Seed pendingReferences with any new references from API calls
-          for (const [soul, node] of Object.entries(update.now)) {
-            if (node && typeof node === "object") {
-              for (const [key, value] of Object.entries(node)) {
-                const soulId = utils.rel.is(value)
-                if (soulId) {
-                  // Add referenced soul to pending list
-                  pendingReferences.add(soulId)
-                }
+        if (!(await check(update.now, send, cb))) return
+
+        // Seed pendingReferences with any new references from API calls
+        for (const [soul, node] of Object.entries(update.now)) {
+          if (node && typeof node === "object") {
+            for (const [key, value] of Object.entries(node)) {
+              const soulId = utils.rel.is(value)
+              if (soulId) {
+                // Add referenced soul to pending list
+                pendingReferences.add(soulId)
               }
             }
           }
-
-          store.put(update.now, cb)
-        } else {
-          // No local update needed, but still callback
-          if (cb) cb(null)
         }
 
+        store.put(update.now, cb)
+
         // Always put data on the wire spec even if no local update needed
         const sendResult = send(
           JSON.stringify({
@@ -640,8 +647,8 @@ const Wire = opt => {
 
     // Schedule processing of next message if queue not empty
     if (messageQueue.length > 0) {
-      // Add delay between messages to respect rate limits (300 req/min = 200ms)
-      queueProcessor = setTimeout(processQueue, 250)
+      // Add delay between messages to respect rate limit
+      queueProcessor = setTimeout(processQueue, 100)
     } else {
       queueProcessor = null
     }
@@ -685,7 +692,7 @@ const Wire = opt => {
     messageQueue.push(data)
     // Start queue processor if not already running
     if (!queueProcessor) {
-      queueProcessor = setTimeout(processQueue, 250)
+      queueProcessor = setTimeout(processQueue, 100)
     }
   }
   if (!(opt.peers instanceof Array)) {