@chromahq/react 1.0.19 → 1.0.21

package/dist/index.js

@@ -1,6 +1,7 @@
 import { jsx } from 'react/jsx-runtime';
 import { createContext, useState, useRef, useEffect, useCallback, useMemo, useContext } from 'react';
 
+const BRIDGE_ENABLE_LOGS = true;
 const CONFIG = {
   RETRY_AFTER: 1e3,
   MAX_RETRIES: 10,
@@ -66,11 +67,15 @@ function createBridgeInstance(deps) {
         if (!pendingRequestsRef.current.has(id)) return;
         pendingRequestsRef.current.delete(id);
         consecutiveTimeoutsRef.current++;
-        console.warn(`[Bridge] Request timed out: ${key} (${timeoutDuration}ms)`);
+        {
+          console.warn(`[Bridge] Request timed out: ${key} (${timeoutDuration}ms)`);
+        }
         if (consecutiveTimeoutsRef.current >= CONFIG.CONSECUTIVE_FAILURE_THRESHOLD) {
-          console.warn(
-            `[Bridge] ${consecutiveTimeoutsRef.current} consecutive timeouts, reconnecting...`
-          );
+          {
+            console.warn(
+              `[Bridge] ${consecutiveTimeoutsRef.current} consecutive timeouts, reconnecting...`
+            );
+          }
           rejectAllPending("Bridge reconnecting due to timeouts");
           consecutiveTimeoutsRef.current = 0;
           onReconnectNeeded();
@@ -111,13 +116,17 @@ function createBridgeInstance(deps) {
   };
   const broadcast = (key, payload) => {
     if (!portRef.current) {
-      console.warn("[Bridge] Cannot broadcast - disconnected");
+      {
+        console.warn("[Bridge] Cannot broadcast - disconnected");
+      }
       return;
     }
     try {
       portRef.current.postMessage({ type: "broadcast", key, payload });
     } catch (e) {
-      console.warn("[Bridge] Broadcast failed:", e);
+      {
+        console.warn("[Bridge] Broadcast failed:", e);
+      }
     }
   };
   const on = (key, handler) => {
@@ -175,9 +184,13 @@ function startHealthMonitor(deps) {
       return;
     }
     consecutivePingFailuresRef.current++;
-    console.warn(`[Bridge] Ping failed (${consecutivePingFailuresRef.current}x)`);
+    {
+      console.warn(`[Bridge] Ping failed (${consecutivePingFailuresRef.current}x)`);
+    }
     if (consecutivePingFailuresRef.current >= CONFIG.CONSECUTIVE_FAILURE_THRESHOLD) {
-      console.warn("[Bridge] Service worker unresponsive, reconnecting...");
+      {
+        console.warn("[Bridge] Service worker unresponsive, reconnecting...");
+      }
       consecutivePingFailuresRef.current = 0;
       onReconnectNeeded();
     }
@@ -277,7 +290,9 @@ const BridgeProvider = ({
         try {
           handler(message.payload);
         } catch (err) {
-          console.warn("[Bridge] Event handler error:", err);
+          {
+            console.warn("[Bridge] Event handler error:", err);
+          }
         }
       });
     }
@@ -292,17 +307,25 @@ const BridgeProvider = ({
           retryAfter,
           CONFIG.MAX_RETRY_DELAY
         );
-        console.log(`[Bridge] Reconnecting in ${delay}ms (${retryCountRef.current}/${maxRetries})`);
+        {
+          console.log(
+            `[Bridge] Reconnecting in ${delay}ms (${retryCountRef.current}/${maxRetries})`
+          );
+        }
         updateStatus("reconnecting");
         reconnectTimeoutRef.current = setTimeout(() => {
           if (isMountedRef.current) connectFn();
         }, delay);
       } else {
-        console.warn(`[Bridge] Max retries reached. Cooldown: ${maxRetryCooldown}ms`);
+        {
+          console.warn(`[Bridge] Max retries reached. Cooldown: ${maxRetryCooldown}ms`);
+        }
         clearTimeoutSafe(maxRetryCooldownRef);
         maxRetryCooldownRef.current = setTimeout(() => {
           if (!isMountedRef.current) return;
-          console.log("[Bridge] Cooldown complete, reconnecting...");
+          {
+            console.log("[Bridge] Cooldown complete, reconnecting...");
+          }
           retryCountRef.current = 0;
           connectFn();
         }, maxRetryCooldown);
@@ -319,9 +342,11 @@ const BridgeProvider = ({
         CONFIG.SW_RESTART_RETRY_DELAY,
         CONFIG.SW_RESTART_MAX_DELAY
       );
-      console.log(
-        `[Bridge] Service worker not ready, retrying in ${delay}ms (attempt ${swRestartRetryCountRef.current})`
-      );
+      {
+        console.log(
+          `[Bridge] Service worker not ready, retrying in ${delay}ms (attempt ${swRestartRetryCountRef.current})`
+        );
+      }
       updateStatus("reconnecting");
       reconnectTimeoutRef.current = setTimeout(() => {
         if (isMountedRef.current) connectFn();
@@ -331,10 +356,6 @@ const BridgeProvider = ({
   );
   const connect = useCallback(() => {
     if (isConnectingRef.current) return;
-    if (retryCountRef.current >= maxRetries) {
-      console.warn("[Bridge] Waiting for cooldown...");
-      return;
-    }
     isConnectingRef.current = true;
     cleanup();
     if (!chrome?.runtime?.connect) {
@@ -354,7 +375,9 @@ const BridgeProvider = ({
         if (err) {
           clearIntervalSafe(errorCheckIntervalRef);
           if (err.includes("Receiving end does not exist")) {
-            console.warn("[Bridge] Service worker not ready (may be restarting)...");
+            if (BRIDGE_ENABLE_LOGS) {
+              console.warn("[Bridge] Service worker not ready (may be restarting)...");
+            }
             cleanup();
             isConnectingRef.current = false;
             scheduleSwRestartReconnect(connect);
@@ -367,17 +390,21 @@ const BridgeProvider = ({
       }, CONFIG.ERROR_CHECK_INTERVAL);
       port.onMessage.addListener(handleMessage);
       port.onDisconnect.addListener(() => {
-        console.warn("[Bridge] Disconnected");
+        if (BRIDGE_ENABLE_LOGS) {
+          console.warn("[Bridge] Disconnected");
+        }
         isConnectingRef.current = false;
         const disconnectError = consumeRuntimeError();
-        if (disconnectError) {
-          handleError(new Error(disconnectError));
-        } else {
-          updateStatus("disconnected");
+        if (disconnectError && BRIDGE_ENABLE_LOGS) {
+          console.warn("[Bridge] Disconnect error:", disconnectError);
         }
+        updateStatus("disconnected");
         cleanup();
         if (isMountedRef.current) {
-          scheduleReconnect(connect);
+          if (BRIDGE_ENABLE_LOGS) {
+            console.log("[Bridge] Will retry until service worker is available...");
+          }
+          scheduleSwRestartReconnect(connect);
         }
       });
       const triggerReconnect = () => {
@@ -416,7 +443,9 @@ const BridgeProvider = ({
         try {
           handler({ timestamp: Date.now() });
         } catch (err) {
-          console.warn("[Bridge] bridge:connected handler error:", err);
+          if (BRIDGE_ENABLE_LOGS) {
+            console.warn("[Bridge] bridge:connected handler error:", err);
+          }
         }
       });
       startHealthMonitor({
@@ -457,15 +486,21 @@ const BridgeProvider = ({
       if (isConnectingRef.current) return;
       const currentStatus = statusRef.current;
       const currentBridge = bridgeRef.current;
-      if (currentStatus === "disconnected" || currentStatus === "error") {
-        console.log("[Bridge] Tab visible, reconnecting...");
+      if (currentStatus === "disconnected" || currentStatus === "error" || currentStatus === "reconnecting") {
+        {
+          console.log("[Bridge] Tab visible, reconnecting...");
+        }
         retryCountRef.current = 0;
+        swRestartRetryCountRef.current = 0;
+        isConnectingRef.current = false;
         connect();
       } else if (currentStatus === "connected" && currentBridge) {
         currentBridge.ping().then((alive) => {
           if (!isMountedRef.current) return;
           if (!alive) {
-            console.warn("[Bridge] Tab visible but unresponsive, reconnecting...");
+            {
+              console.warn("[Bridge] Tab visible but unresponsive, reconnecting...");
+            }
             retryCountRef.current = 0;
             isConnectingRef.current = false;
             cleanup();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chromahq/react",
-  "version": "1.0.19",
+  "version": "1.0.21",
   "description": "React bindings for the Chroma Chrome extension framework",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",