hypha-rpc 0.21.29 → 0.21.31

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-01T02:41:30.895Z
+                at 2026-03-04T22:36:31.483Z
             </div>
         <script src="prettify.js"></script>
         <script>

package/dist/hypha-rpc-websocket.js

@@ -4022,6 +4022,13 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
             const serviceRegistrationTimeout = (this._method_timeout || 30) * 1000;
 
             for (let service of services) {
+              // Skip local-only services (e.g. _rintf_ callback proxies) —
+              // they must never be registered with the server; doing so
+              // creates zombie entries in Redis.
+              if (service.config && service.config._local_only) {
+                servicesCount--;
+                continue;
+              }
               try {
                 const serviceInfo = this._extract_service_info(service);
                 await withTimeout(
@@ -4173,10 +4180,11 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
       // This ensures no remote function call hangs forever when the connection drops
       if (typeof connection.on_disconnected === "function") {
         connection.on_disconnected((reason) => {
-          // If reconnection is enabled, don't reject pending calls immediately.
-          // The timeout mechanism will handle them if reconnection fails,
-          // allowing calls to succeed after a successful reconnection.
-          if (connection._enable_reconnect) {
+          // If reconnection is enabled AND the connection is not permanently closed,
+          // don't reject pending calls immediately — they may succeed after reconnect.
+          // When _closed is true (max retries exhausted or server refused reconnect),
+          // we must reject immediately so callers are not left hanging forever.
+          if (connection._enable_reconnect && !connection._closed) {
             this._logger.info(
               `Connection lost (${reason}), reconnection enabled - pending calls will be handled by timeout`,
             );
@@ -6043,6 +6051,8 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
   get_client_info() {
     const services = [];
     for (let service of Object.values(this._services)) {
+      // Exclude local-only services (e.g. _rintf_ callback proxies)
+      if (service.config && service.config._local_only) continue;
       services.push(this._extract_service_info(service));
     }
 
@@ -6776,11 +6786,12 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
           }
         }
         const serviceApi = { id: serviceId };
+        serviceApi.config = {
+          visibility: "protected",
+          _local_only: true,
+        };
         if (allowedCaller) {
-          serviceApi.config = {
-            visibility: "protected",
-            _rintf_allowed_caller: allowedCaller,
-          };
+          serviceApi.config._rintf_allowed_caller = allowedCaller;
         }
 
         // Add _dispose method for active lifecycle management.