Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>

Author: nbbeeken

packages/compass-aggregations/src/components/pipeline-toolbar/pipeline-options/pipeline-collation.tsx

@@ -94,15 +94,13 @@ const PipelineCollation: React.FunctionComponent<PipelineCollationProps> = ({
   }, [showMaxTimeMSWarning, maxTimeMSLimit]);
 
   // Check if value exceeds the limit when warning is enabled
-  const exceedsLimit = Boolean(
-    useMemo(() => {
-      return (
-        showMaxTimeMSWarning &&
-        maxTimeMSValue &&
-        maxTimeMSValue >= WEB_MAX_TIME_MS_LIMIT
-      );
-    }, [showMaxTimeMSWarning, maxTimeMSValue])
-  );
+  const exceedsLimit = useMemo(() => {
+    return (
+      showMaxTimeMSWarning &&
+      maxTimeMSValue &&
+      maxTimeMSValue >= WEB_MAX_TIME_MS_LIMIT
+    );
+  }, [showMaxTimeMSWarning, maxTimeMSValue]);
 
   return (
     <div

packages/compass-query-bar/src/components/query-option.tsx

@@ -166,15 +166,13 @@ const QueryOption: React.FunctionComponent<QueryOptionProps> = ({
     return Number.isNaN(parsed) ? 0 : parsed;
   }, [value]);
 
-  const exceedsMaxTimeMSLimit = Boolean(
-    useMemo(() => {
-      return (
-        name === 'maxTimeMS' &&
-        showMaxTimeMSWarning &&
-        numericValue >= WEB_MAX_TIME_MS_LIMIT
-      );
-    }, [name, showMaxTimeMSWarning, numericValue])
-  );
+  const exceedsMaxTimeMSLimit = useMemo(() => {
+    return (
+      name === 'maxTimeMS' &&
+      showMaxTimeMSWarning &&
+      numericValue >= WEB_MAX_TIME_MS_LIMIT
+    );
+  }, [name, showMaxTimeMSWarning, numericValue]);
 
   return (
     <div

packages/compass-query-bar/src/stores/query-bar-reducer.ts

@@ -104,7 +104,7 @@ export const changeField = (
     const parsedValue = validateField(name, stringValue, {
       maxTimeMS: preferences.getPreferences().maxTimeMS ?? undefined,
       showMaxTimeMSWarning:
-        Boolean(preferences.getPreferences().showMaxTimeMSWarning) ?? false,
+        Boolean(preferences.getPreferences().showMaxTimeMSWarning),
     });
     const isValid = parsedValue !== false;
     dispatch({
@@ -166,9 +166,8 @@ export const resetQuery = (
     const fields = mapQueryToFormFields(
       {
         maxTimeMS: preferences.getPreferences().maxTimeMS,
-        showMaxTimeMSWarning: Boolean(
-          preferences.getPreferences().showMaxTimeMSWarning
-        ),
+        showMaxTimeMSWarning:
+          preferences.getPreferences().showMaxTimeMSWarning ?? false,
       },
       DEFAULT_FIELD_VALUES
     );