hypha-rpc 0.21.38 → 0.21.40

package/coverage/html/index.html

@@ -86,7 +86,7 @@
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-03-19T04:33:41.281Z
+                at 2026-04-18T12:07:11.181Z
             </div>
         <script src="prettify.js"></script>
         <script>

package/dist/hypha-rpc-websocket.js

@@ -4333,10 +4333,21 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
 
   _create_message(key, heartbeat, overwrite, context) {
     if (heartbeat) {
-      if (!this._object_store[key]) {
+      const session = this._object_store[key];
+      if (!session) {
         throw new Error(`session does not exist anymore: ${key}`);
       }
-      this._object_store[key]["timer"].reset();
+      // Sessions created without a promise/timeout (e.g. _get_session_store
+      // calls where timer && reject && _method_timeout was false) have no
+      // timer. A chunked message still arrives with heartbeat=true because
+      // the sender sets it based on whether session_id is truthy (rpc.js
+      // ~L2358), not on whether the receiver's session owns a timer. Without
+      // this guard, every chunk of such a session throws "Cannot read
+      // properties of undefined (reading 'reset')", which cascades and
+      // silently breaks the RPC message pipeline for the client.
+      if (session.timer && typeof session.timer.reset === "function") {
+        session.timer.reset();
+      }
     }
 
     if (!this._object_store["message_cache"]) {
@@ -4385,10 +4396,21 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
 
   _append_message(key, data, heartbeat, context) {
     if (heartbeat) {
-      if (!this._object_store[key]) {
+      const session = this._object_store[key];
+      if (!session) {
         throw new Error(`session does not exist anymore: ${key}`);
       }
-      this._object_store[key]["timer"].reset();
+      // Sessions created without a promise/timeout (e.g. _get_session_store
+      // calls where timer && reject && _method_timeout was false) have no
+      // timer. A chunked message still arrives with heartbeat=true because
+      // the sender sets it based on whether session_id is truthy (rpc.js
+      // ~L2358), not on whether the receiver's session owns a timer. Without
+      // this guard, every chunk of such a session throws "Cannot read
+      // properties of undefined (reading 'reset')", which cascades and
+      // silently breaks the RPC message pipeline for the client.
+      if (session.timer && typeof session.timer.reset === "function") {
+        session.timer.reset();
+      }
     }
     const cache = this._object_store["message_cache"];
     if (!cache[key]) {
@@ -4400,10 +4422,21 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
 
   _set_message(key, index, data, heartbeat, context) {
     if (heartbeat) {
-      if (!this._object_store[key]) {
+      const session = this._object_store[key];
+      if (!session) {
         throw new Error(`session does not exist anymore: ${key}`);
       }
-      this._object_store[key]["timer"].reset();
+      // Sessions created without a promise/timeout (e.g. _get_session_store
+      // calls where timer && reject && _method_timeout was false) have no
+      // timer. A chunked message still arrives with heartbeat=true because
+      // the sender sets it based on whether session_id is truthy (rpc.js
+      // ~L2358), not on whether the receiver's session owns a timer. Without
+      // this guard, every chunk of such a session throws "Cannot read
+      // properties of undefined (reading 'reset')", which cascades and
+      // silently breaks the RPC message pipeline for the client.
+      if (session.timer && typeof session.timer.reset === "function") {
+        session.timer.reset();
+      }
     }
     const cache = this._object_store["message_cache"];
     if (!cache[key]) {
@@ -4423,10 +4456,21 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
 
   _process_message(key, heartbeat, context) {
     if (heartbeat) {
-      if (!this._object_store[key]) {
+      const session = this._object_store[key];
+      if (!session) {
         throw new Error(`session does not exist anymore: ${key}`);
       }
-      this._object_store[key]["timer"].reset();
+      // Sessions created without a promise/timeout (e.g. _get_session_store
+      // calls where timer && reject && _method_timeout was false) have no
+      // timer. A chunked message still arrives with heartbeat=true because
+      // the sender sets it based on whether session_id is truthy (rpc.js
+      // ~L2358), not on whether the receiver's session owns a timer. Without
+      // this guard, every chunk of such a session throws "Cannot read
+      // properties of undefined (reading 'reset')", which cascades and
+      // silently breaks the RPC message pipeline for the client.
+      if (session.timer && typeof session.timer.reset === "function") {
+        session.timer.reset();
+      }
     }
     const cache = this._object_store["message_cache"];
     (0,_utils_index_js__WEBPACK_IMPORTED_MODULE_0__.assert)(!!context, "Context is required");