hypha-rpc 0.21.32 → 0.21.34

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-06T00:02:24.136Z
+                at 2026-02-10T16:00:13.893Z
             </div>
         <script src="prettify.js"></script>
         <script>

package/dist/hypha-rpc-websocket.js

@@ -4002,10 +4002,12 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
 
           try {
             // Get fresh manager service (one RPC roundtrip, ~50-100ms)
+            const _t0 = Date.now();
             const manager = await this.get_manager_service({
               timeout: 20,
               case_conversion: "camel",
             });
+            console.error(`[DEBUG] get_manager_service took ${Date.now() - _t0}ms`);
 
             // Fire manager_refreshed IMMEDIATELY — before service re-registration.
             // This allows connectToServer's wm proxy to be updated as soon as
@@ -4030,12 +4032,14 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
                 continue;
               }
               try {
+                const _t1 = Date.now();
                 const serviceInfo = this._extract_service_info(service);
                 await withTimeout(
                   manager.registerService(serviceInfo),
                   serviceRegistrationTimeout,
                   `Timeout registering service ${service.id || "unknown"}`,
                 );
+                console.error(`[DEBUG] registerService(${service.id}) took ${Date.now() - _t1}ms`);
                 registeredCount++;
               } catch (serviceError) {
                 failedServices.push(service.id || "unknown");
@@ -4180,11 +4184,10 @@ 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 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) {
+          // 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) {
             this._logger.info(
               `Connection lost (${reason}), reconnection enabled - pending calls will be handled by timeout`,
             );
@@ -4538,15 +4541,7 @@ class RPC extends _utils_index_js__WEBPACK_IMPORTED_MODULE_0__.MessageEmitter {
     delete this._rintfCallerIndex[clientId];
     let cleaned = 0;
     for (const sid of serviceIds) {
-      const svc = this._services[sid];
-      if (svc) {
-        if (typeof svc._dispose === "function") {
-          try {
-            svc._dispose();
-          } catch (e) {
-            /* ignore errors from dispose handlers */
-          }
-        }
+      if (this._services[sid]) {
         delete this._services[sid];
         cleaned++;
       }
@@ -11064,7 +11059,9 @@ async function connectToServer(config) {
     config.ping_interval,
     config.logger,
   );
+  const _tOpen = Date.now();
   const connection_info = await connection.open();
+  console.error(`[DEBUG] connection.open() took ${Date.now() - _tOpen}ms`);
   (0,_utils_index_js__WEBPACK_IMPORTED_MODULE_2__.assert)(
     connection_info,
     "Failed to connect to the server, no connection info obtained. This issue is most likely due to an outdated Hypha server version. Please use `imjoy-rpc` for compatibility, or upgrade the Hypha server to the latest version.",
@@ -11098,6 +11095,7 @@ async function connectToServer(config) {
   }
 
   const workspace = connection_info.workspace;
+  const _tRpc = Date.now();
   const rpc = new _rpc_js__WEBPACK_IMPORTED_MODULE_0__.RPC(connection, {
     client_id: clientId,
     workspace,
@@ -11114,12 +11112,17 @@ async function connectToServer(config) {
     encryption_private_key: config.encryption_private_key || null,
     encryption_public_key: config.encryption_public_key || null,
   });
+  console.error(`[DEBUG] RPC constructor took ${Date.now() - _tRpc}ms`);
+  const _tWait = Date.now();
   await rpc.waitFor("services_registered", config.method_timeout || 120);
+  console.error(`[DEBUG] waitFor('services_registered') took ${Date.now() - _tWait}ms`);
+  const _tWm = Date.now();
   const wm = await rpc.get_manager_service({
     timeout: config.method_timeout,
     case_conversion: "camel",
     kwargs_expansion: config.kwargs_expansion || false,
   });
+  console.error(`[DEBUG] get_manager_service (connectToServer) took ${Date.now() - _tWm}ms`);
   wm.rpc = rpc;
 
   // Auto-refresh workspace manager proxy after reconnection.