[Workspace]Fix maximum call stack error in use case service

changelogs/fragments/7817.yml

@@ -0,0 +1,2 @@
+fix:
+- [Workspace] maximum call stack error in use case service ([#7817](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/7817))

src/plugins/workspace/public/types.ts

@@ -14,11 +14,16 @@ export type Services = CoreStart & {
   navigationUI?: NavigationPublicPluginStart['ui'];
 };
 
+export interface WorkspaceUseCaseFeature {
+  id: string;
+  title?: string;
+}
+
 export interface WorkspaceUseCase {
   id: string;
   title: string;
   description: string;
-  features: Array<{ id: string; title?: string }>;
+  features: WorkspaceUseCaseFeature[];
   systematic?: boolean;
   order?: number;
 }

src/plugins/workspace/public/utils.test.ts

@@ -558,6 +558,23 @@ describe('workspace utils: isEqualWorkspaceUseCase', () => {
       })
     ).toEqual(false);
   });
+  it('should return false for duplicate features', () => {
+    expect(
+      isEqualWorkspaceUseCase(
+        { ...useCaseMock, features: [useCaseMock.features[0], useCaseMock.features[0]] },
+        {
+          ...useCaseMock,
+          features: [
+            useCaseMock.features[0],
+            {
+              id: 'another',
+              title: 'Another',
+            },
+          ],
+        }
+      )
+    ).toEqual(false);
+  });
   it('should return true when all properties equal', () => {
     expect(
       isEqualWorkspaceUseCase(useCaseMock, {

src/plugins/workspace/public/utils.ts

@@ -24,7 +24,7 @@ import {
   WorkspaceAvailability,
 } from '../../../core/public';
 import { DEFAULT_SELECTED_FEATURES_IDS, WORKSPACE_DETAIL_APP_ID } from '../common/constants';
-import { WorkspaceUseCase } from './types';
+import { WorkspaceUseCase, WorkspaceUseCaseFeature } from './types';
 import { formatUrlWithWorkspaceId } from '../../../core/public/utils';
 import { SigV4ServiceName } from '../../../plugins/data_source/common/data_sources';
 
@@ -260,6 +260,24 @@ export const convertNavGroupToWorkspaceUseCase = ({
   order,
 });
 
+const compareFeatures = (
+  features1: WorkspaceUseCaseFeature[],
+  features2: WorkspaceUseCaseFeature[]
+) => {
+  const featureSerialize = ({ id, title }: WorkspaceUseCaseFeature) => `${id}-${title}`;
+  const features1Set = new Set(features1.map(featureSerialize));
+  const features2Set = new Set(features2.map(featureSerialize));
+  if (features1Set.size !== features2Set.size) {
+    return false;
+  }
+  for (const feature of features1Set) {
+    if (!features2Set.has(feature)) {
+      return false;
+    }
+  }
+  return true;
+};
+
 export const isEqualWorkspaceUseCase = (a: WorkspaceUseCase, b: WorkspaceUseCase) => {
   if (a.id !== b.id) {
     return false;
@@ -276,14 +294,7 @@ export const isEqualWorkspaceUseCase = (a: WorkspaceUseCase, b: WorkspaceUseCase
   if (a.order !== b.order) {
     return false;
   }
-  if (
-    a.features.length !== b.features.length ||
-    a.features.some((aFeature) =>
-      b.features.some(
-        (bFeature) => aFeature.id !== bFeature.id || aFeature.title !== bFeature.title
-      )
-    )
-  ) {
+  if (a.features.length !== b.features.length || !compareFeatures(a.features, b.features)) {
     return false;
   }
   return true;