Merge branch 'main' into connection-error-entrypoint

Author: lerouxb

THIRD-PARTY-NOTICES.md

@@ -138,12 +138,12 @@ module.exports = {
 
     return {
       ImportDeclaration(node) {
-        if (isImportSourceEquals(/^@leafygreen-ui/, node)) {
+        if (isImportSourceEquals(/^@(leafygreen-ui|lg-chat)/, node)) {
           reportLeafygreenUsage(context, node, node.source);
         }
       },
       CallExpression(node) {
-        if (isRequireSourceEquals(/^@leafygreen-ui/, node)) {
+        if (isRequireSourceEquals(/^@(leafygreen-ui|lg-chat)/, node)) {
           reportLeafygreenUsage(context, node, node.arguments[0]);
         }
       },

configs/eslint-plugin-compass/rules/no-leafygreen-outside-compass-components.js

@@ -6,7 +6,7 @@
 > the tracking plan for the specific Compass version you can use the following
 > URL: `https://github.com/mongodb-js/compass/blob/<compass version>/docs/tracking-plan.md`
 
-Generated on Fri, Aug 22, 2025
+Generated on Mon, Aug 25, 2025
 
 ## Table of Contents
 

docs/tracking-plan.md

@@ -110,6 +110,27 @@
       "eslint-plugin-jsx-a11y": "^6.10.2",
       "eslint-plugin-react": "^7.37.5",
       "eslint-plugin-react-hooks": "^5.2.0"
-    }
+    },
+    "@leafygreen-ui/emotion": "^4.0.9",
+    "@leafygreen-ui/lib": "^15.2.0",
+    "@leafygreen-ui/palette": "^4.1.3",
+    "@leafygreen-ui/tokens": "^2.11.3",
+    "@leafygreen-ui/leafygreen-provider": "^4.0.2",
+    "@leafygreen-ui/marketing-modal": "^5.0.2",
+    "@leafygreen-ui/typography": "^20.0.2",
+    "@leafygreen-ui/icon": "^13.1.2",
+    "@leafygreen-ui/popover": "^13.0.11",
+    "@leafygreen-ui/badge": "^9.0.2",
+    "@leafygreen-ui/banner": "^9.0.2",
+    "@leafygreen-ui/button": "^22.0.2",
+    "@leafygreen-ui/hooks": "^8.3.4",
+    "@leafygreen-ui/icon-button": "^16.0.2",
+    "@leafygreen-ui/input-option": "^3.0.4",
+    "@leafygreen-ui/polymorphic": "^2.0.5",
+    "@leafygreen-ui/search-input": "^5.0.2",
+    "@leafygreen-ui/code": "^16.0.2",
+    "@leafygreen-ui/text-area": "^10.0.2",
+    "@leafygreen-ui/card": "^12.0.2",
+    "@leafygreen-ui/logo": "^10.0.2"
   }
 }

package-lock.json

@@ -72,7 +72,7 @@
     "@mongodb-js/compass-utils": "^0.9.10",
     "@mongodb-js/compass-workspaces": "^0.51.0",
     "@mongodb-js/explain-plan-helper": "^1.4.17",
-    "@mongodb-js/mongodb-constants": "^0.12.2",
+    "@mongodb-js/mongodb-constants": "^0.14.0",
     "@mongodb-js/my-queries-storage": "^0.37.0",
     "@mongodb-js/shell-bson-parser": "^1.2.0",
     "bson": "^6.10.4",

package.json

@@ -123,4 +123,26 @@ export const createSearchIndex = (): PipelineBuilderThunkAction<void> => {
   };
 };
 
+/**
+ * Checks whether a namespace has existing search indexes
+ *
+ * @param namespace - collection/view namespace
+ * @param dataService - dataService instance
+ * @returns whether namespace has existing search indexes
+ */
+export const namespaceHasSearchIndexes = async (
+  namespace: string,
+  dataService: { getSearchIndexes?: (ns: string) => Promise<SearchIndex[]> }
+): Promise<boolean> => {
+  try {
+    if (!dataService.getSearchIndexes) {
+      throw new Error('Cannot get search indexes in this environment');
+    }
+    const indexes = await dataService.getSearchIndexes(namespace);
+    return indexes.length > 0;
+  } catch {
+    return false;
+  }
+};
+
 export default reducer;

packages/compass-aggregations/package.json

@@ -10,6 +10,10 @@ import {
 } from '@mongodb-js/compass-connections/provider';
 import { createDefaultConnectionInfo } from '@mongodb-js/testing-library-compass';
 
+// Importing this to stub showConfirmation
+import * as updateViewSlice from './update-view';
+import * as searchIndexesSlice from './search-indexes';
+
 const TEST_CONNECTION_INFO = { ...createDefaultConnectionInfo(), title: '' };
 
 describe('update-view module', function () {
@@ -48,10 +52,20 @@ describe('update-view module', function () {
     let stateMock: any;
     let getStateMock: () => any;
     let updateCollectionFake = sinon.fake();
+    let showConfirmationStub: sinon.SinonStub;
+    let namespaceHasSearchIndexesStub: sinon.SinonStub;
 
-    beforeEach(async function () {
+    beforeEach(function () {
       dispatchFake = sinon.fake();
       updateCollectionFake = sinon.fake.resolves(undefined);
+      showConfirmationStub = sinon
+        .stub(updateViewSlice, 'showConfirmation')
+        .resolves(true);
+
+      namespaceHasSearchIndexesStub = sinon
+        .stub(searchIndexesSlice, 'namespaceHasSearchIndexes')
+        .resolves(true);
+
       stateMock = {
         pipelineBuilder: { pipelineMode: 'builder-ui' },
         focusMode: { isEnabled: false },
@@ -62,20 +76,57 @@ describe('update-view module', function () {
             updateCollection: updateCollectionFake,
           },
         },
+        serverVersion: '8.1.0',
       };
       getStateMock = () => stateMock;
+    });
 
+    afterEach(function () {
+      showConfirmationStub.restore();
+      namespaceHasSearchIndexesStub.restore();
+    });
+
+    it('first it calls to dismiss any existing error', async function () {
       const runUpdateView = updateView();
       await runUpdateView(dispatchFake, getStateMock, thunkArg as any);
-    });
 
-    it('first it calls to dismiss any existing error', function () {
       expect(dispatchFake.firstCall.args[0]).to.deep.equal({
         type: 'aggregations/update-view/DISMISS_VIEW_UPDATE_ERROR',
       });
     });
 
-    it('calls the data service to update the view for the provided ns', function () {
+    it('does not shows confirmation banner if search indexes are not present', async function () {
+      namespaceHasSearchIndexesStub.resolves(false);
+      const runUpdateView = updateView();
+      await runUpdateView(dispatchFake, getStateMock, thunkArg as any);
+
+      expect(showConfirmationStub.calledOnce).to.be.false;
+    });
+
+    it('shows confirmation banner when search indexes are present', async function () {
+      const runUpdateView = updateView();
+      await runUpdateView(dispatchFake, getStateMock, thunkArg as any);
+
+      expect(showConfirmationStub.calledOnce).to.be.true;
+      expect(showConfirmationStub.firstCall.args[0]).to.deep.include({
+        title: `Are you sure you want to update the view?`,
+        buttonText: 'Update',
+      });
+    });
+
+    it('does not update view if not confirmed', async function () {
+      showConfirmationStub.resolves(false);
+
+      const runUpdateView = updateView();
+      await runUpdateView(dispatchFake, getStateMock, thunkArg as any);
+
+      expect(updateCollectionFake.calledOnce).to.be.false;
+    });
+
+    it('calls the data service to update the view for the provided ns', async function () {
+      const runUpdateView = updateView();
+      await runUpdateView(dispatchFake, getStateMock, thunkArg as any);
+
       expect(updateCollectionFake.firstCall.args[0]).to.equal('aa.bb');
       expect(updateCollectionFake.firstCall.args[1]).to.deep.equal({
         viewOn: 'bb',

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

@@ -8,6 +8,9 @@ import {
 import type { PipelineBuilderThunkAction } from '.';
 import { isAction } from '../utils/is-action';
 import type { AnyAction } from 'redux';
+import { showConfirmation as showConfirmationModal } from '@mongodb-js/compass-components';
+import { namespaceHasSearchIndexes } from './search-indexes';
+import { VIEW_PIPELINE_UTILS } from '@mongodb-js/mongodb-constants';
 
 export type UpdateViewState = null | string;
 
@@ -73,6 +76,8 @@ export const dismissViewError = (): DismissViewUpdateErrorAction => ({
   type: DISMISS_VIEW_UPDATE_ERROR,
 });
 
+//Exporting this for test only to stub it and set its value
+export const showConfirmation = showConfirmationModal;
 /**
  * Updates a view.
  *
@@ -96,6 +101,7 @@ export const updateView = (): PipelineBuilderThunkAction<Promise<void>> => {
     const state = getState();
     const ds = state.dataService.dataService;
     const viewNamespace = state.editViewName;
+    const serverVersion = state.serverVersion;
 
     if (!viewNamespace) {
       return;
@@ -107,6 +113,30 @@ export const updateView = (): PipelineBuilderThunkAction<Promise<void>> => {
       getState(),
       pipelineBuilder
     );
+
+    if (
+      VIEW_PIPELINE_UTILS.isVersionSearchCompatibleForViewsDataExplorer(
+        serverVersion
+      ) &&
+      ds &&
+      (await namespaceHasSearchIndexes(viewNamespace, ds))
+    ) {
+      const pipelineIsSearchQueryable =
+        VIEW_PIPELINE_UTILS.isPipelineSearchQueryable(viewPipeline);
+      const confirmed = await showConfirmation({
+        title: `Are you sure you want to update the view?`,
+        description: pipelineIsSearchQueryable
+          ? 'There are search indexes created on this view. Updating the view will result in an index rebuild, which will consume additional resources on your cluster.'
+          : 'This update will make the view incompatible with search indexes and will cause all search indexes to fail. Only views containing $addFields, $set or $match stages with the $expr operator are compatible with search indexes.',
+        buttonText: 'Update',
+        variant: pipelineIsSearchQueryable ? 'primary' : 'danger',
+      });
+
+      if (!confirmed) {
+        return;
+      }
+    }
+
     const options = {
       viewOn: toNS(state.namespace).collection,
       pipeline: viewPipeline,

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

@@ -66,6 +66,24 @@ describe('utils', function () {
         expect(searchMeta.length).to.be.equal(1);
       });
 
+      it('returns $search stage for a view', function () {
+        const search = filterStageOperators({
+          ...filter,
+          sourceName: 'simple.sample',
+        }).filter((o) => o.name === '$search');
+
+        expect(search.length).to.be.equal(1);
+      });
+
+      it('returns $searchMeta stage for a view', function () {
+        const searchMeta = filterStageOperators({
+          ...filter,
+          sourceName: 'simple.sample',
+        }).filter((o) => o.name === '$searchMeta');
+
+        expect(searchMeta.length).to.be.equal(1);
+      });
+
       // $documents only works for db.aggregate, not coll.aggregate
       it('does not return $documents stage for a regular collection', function () {
         const documents = filterStageOperators({ ...filter }).filter(
@@ -97,17 +115,6 @@ describe('utils', function () {
 
         expect(searchStages.length).to.be.equal(0);
       });
-
-      it('does not return full-text search stages for views', function () {
-        const searchStages = filterStageOperators({
-          ...filter,
-          sourceName: 'simple.sample',
-        }).filter((o) =>
-          ['$search', '$searchMeta', '$documents'].includes(o.name)
-        );
-
-        expect(searchStages.length).to.be.equal(0);
-      });
     });
 
     context('when on-prem', function () {