add confirmation banner

Author: DarshanaVenkatesh

packages/compass-aggregations/src/modules/search-indexes.ts

@@ -80,14 +80,18 @@ const reducer: Reducer<State, Action> = (state = INITIAL_STATE, action) => {
   return state;
 };
 
-export const fetchIndexes = (): PipelineBuilderThunkAction<Promise<void>> => {
+export const fetchIndexes = (
+  viewNamespace?: string
+): PipelineBuilderThunkAction<Promise<void>> => {
   return async (dispatch, getState) => {
     const {
-      namespace,
+      namespace: collectionNamespace,
       dataService: { dataService },
       searchIndexes: { status },
     } = getState();
 
+    const namespace = viewNamespace || collectionNamespace; // fetch indexes for view if passed in
+
     if (
       !dataService ||
       status === SearchIndexesStatuses.LOADING ||

packages/compass-aggregations/src/modules/update-view.ts

@@ -10,6 +10,7 @@ import { isAction } from '../utils/is-action';
 import type { AnyAction } from 'redux';
 import { showConfirmation } from '@mongodb-js/compass-components';
 import { fetchIndexes } from './search-indexes';
+import { isPipelineSearchQueryable } from 'mongodb-compass/src/app/utils/view-search-queryable';
 
 export type UpdateViewState = null | string;
 
@@ -75,37 +76,6 @@ export const dismissViewError = (): DismissViewUpdateErrorAction => ({
   type: DISMISS_VIEW_UPDATE_ERROR,
 });
 
-const isPipelineSearchQueryable = (
-  pipeline: Array<Record<string, any>>
-): boolean => {
-  for (const stage of pipeline) {
-    const stageKey = Object.keys(stage)[0];
-
-    // Check if the stage is $addFields, $set, or $match
-    if (
-      !(
-        stageKey === '$addFields' ||
-        stageKey === '$set' ||
-        stageKey === '$match'
-      )
-    ) {
-      return false; // Not searchable
-    }
-
-    // If the stage is $match, check if uses $expr
-    if (stageKey === '$match') {
-      const matchStage = stage['$match'];
-      const allKeys = Object.keys(matchStage);
-
-      if (!(allKeys.length === 1 && allKeys.includes('$expr'))) {
-        return false; // Not searchable
-      }
-    }
-  }
-
-  return true;
-};
-
 /**
  * Updates a view.
  *
@@ -141,7 +111,7 @@ export const updateView = (): PipelineBuilderThunkAction<Promise<void>> => {
       pipelineBuilder
     );
 
-    await dispatch(fetchIndexes());
+    await dispatch(fetchIndexes(viewNamespace));
     if (state.searchIndexes.indexes.length > 0) {
       const pipelineIsSearchQueryable = isPipelineSearchQueryable(viewPipeline);
       const confirmed = await showConfirmation({

packages/compass/src/app/utils/view-search-queryable.ts

@@ -0,0 +1,30 @@
+export const isPipelineSearchQueryable = (
+  pipeline: Array<Record<string, any>>
+): boolean => {
+  for (const stage of pipeline) {
+    const stageKey = Object.keys(stage)[0];
+
+    // Check if the stage is $addFields, $set, or $match
+    if (
+      !(
+        stageKey === '$addFields' ||
+        stageKey === '$set' ||
+        stageKey === '$match'
+      )
+    ) {
+      return false;
+    }
+
+    // If the stage is $match, check if uses $expr
+    if (stageKey === '$match') {
+      const matchStage = stage['$match'];
+      const matchKeys = Object.keys(matchStage);
+
+      if (!(matchKeys.length === 1 && matchKeys.includes('$expr'))) {
+        return false;
+      }
+    }
+  }
+
+  return true;
+};