Merge remote-tracking branch 'origin/main' into beta-releases

mongodb-js · Sep 16, 2024 · 88054a2 · 88054a2
2 parents 934bdc0 + f15e88f
commit 88054a2
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 15 deletions.
diff --git a/THIRD-PARTY-NOTICES.md b/THIRD-PARTY-NOTICES.md
@@ -1,5 +1,5 @@
 The following third-party software is used by and included in **Mongodb Compass**.
-This document was automatically generated on Sun Sep 15 2024.
+This document was automatically generated on Mon Sep 16 2024.
 
 ## List of dependencies
 

diff --git a/docs/tracking-plan.md b/docs/tracking-plan.md
@@ -1,7 +1,7 @@
 
 # Compass Tracking Plan
 
-Generated on Sun, Sep 15, 2024 at 03:14 AM
+Generated on Mon, Sep 16, 2024 at 08:22 AM
 
 ## Table of Contents
 

diff --git a/packages/compass-connections/src/connection-info-provider.tsx b/packages/compass-connections/src/connection-info-provider.tsx
@@ -1,4 +1,4 @@
-import React, { createContext, useContext } from 'react';
+import React, { createContext, useContext, useState } from 'react';
 import type { ConnectionInfo } from '@mongodb-js/connection-info';
 import {
   createServiceLocator,
@@ -88,19 +88,25 @@ export const useConnectionInfoAccess = (): ConnectionInfoAccess => {
   // This is stable in all environments
   // eslint-disable-next-line react-hooks/rules-of-hooks
   const connectionInfoRef = useConnectionInfoRefForId(connectionId);
-  return {
-    getCurrentConnectionInfo() {
-      if (!connectionInfoRef.current) {
-        if (process.env.NODE_ENV !== 'test') {
-          throw new Error(
-            'Could not find the current ConnectionInfo. Did you forget to setup the ConnectionInfoContext?'
-          );
+  // eslint-disable-next-line react-hooks/rules-of-hooks
+  const [access] = useState(() => {
+    // Return the function from useState to make sure the value doesn't change
+    // when component re-renders
+    return {
+      getCurrentConnectionInfo() {
+        if (!connectionInfoRef.current) {
+          if (process.env.NODE_ENV !== 'test') {
+            throw new Error(
+              'Could not find the current ConnectionInfo. Did you forget to setup the ConnectionInfoContext?'
+            );
+          }
+          return TEST_CONNECTION_INFO;
         }
-        return TEST_CONNECTION_INFO;
-      }
-      return connectionInfoRef.current;
-    },
-  };
+        return connectionInfoRef.current;
+      },
+    };
+  });
+  return access;
 };
 export const connectionInfoAccessLocator = createServiceLocator(
   useConnectionInfoAccess,