[Navigation] Update deep link visibility settings to exclude 'classicSideNav'

This commit modifies the default visibility settings for management plugins to exclude 'classicSideNav', ensuring that management sub-apps do not appear as top-level entries in the classic hamburger navigation. The changes also update related unit tests to reflect the new expectations for deep link visibility, specifically for applications without an explicit 'visibleIn' setting.

Additionally, the test cases for file upload, index data visualizer, and data drift have been adjusted to confirm their visibility in both 'globalSearch' and 'solutionSideNav', while breadcrumb-only page variants are now correctly set to be hidden everywhere.

Author: paulinashakirova

src/platform/plugins/shared/management/public/plugin.tsx

@@ -83,10 +83,11 @@ export class ManagementPlugin
             title: mgmtApp.title,
             path: mgmtApp.basePath,
             keywords: mgmtApp.keywords,
-            // Default includes both 'classicSideNav' and 'solutionSideNav' so all management sections
-            // appear in both navigation modes. Sections that explicitly set visibleIn
-            // (e.g. to hide from globalSearch) must also include the relevant nav flags.
-            visibleIn: mgmtApp.visibleIn ?? ['globalSearch', 'classicSideNav', 'solutionSideNav'],
+            // Default excludes 'classicSideNav' so management sub-apps do not surface as
+            // top-level entries in the classic hamburger nav (the "Stack Management" app
+            // is the entry point there). 'solutionSideNav' is included so solution
+            // navigation trees can keep referencing them by id (e.g. 'management:dataViews').
+            visibleIn: mgmtApp.visibleIn ?? ['globalSearch', 'solutionSideNav'],
           })),
       }));
 

src/platform/plugins/shared/management/public/plugin_deep_links.test.ts

@@ -31,7 +31,7 @@ function getDeepLinksFromUpdater(plugin: ManagementPlugin): AppDeepLink[] {
 }
 
 describe('ManagementPlugin appUpdater deep link visibleIn', () => {
-  it('defaults visibleIn to globalSearch and classicSideNav/solutionSideNav for apps without an explicit visibleIn', () => {
+  it('defaults visibleIn to globalSearch + solutionSideNav for apps without an explicit visibleIn (excludes classicSideNav to avoid duplicating Stack Management entries in the classic hamburger nav)', () => {
     const plugin = createPlugin();
     const setup = plugin.setup(coreMock.createSetup(), { share: mockShare });
 
@@ -46,7 +46,7 @@ describe('ManagementPlugin appUpdater deep link visibleIn', () => {
     const app = kibana?.deepLinks?.find((a) => a.id === 'test-no-visible-in');
 
     expect(app).toBeDefined();
-    expect(app?.visibleIn).toEqual(['globalSearch', 'classicSideNav', 'solutionSideNav']);
+    expect(app?.visibleIn).toEqual(['globalSearch', 'solutionSideNav']);
   });
 
   it('preserves explicit visibleIn when set to classicSideNav/solutionSideNav-only', () => {

x-pack/platform/plugins/shared/ml/public/register_helper/register_search_links/search_deep_links.test.ts

@@ -45,30 +45,36 @@ describe('getDeepLinks', () => {
     ]);
   });
 
-  it('hidden nav nodes (breadcrumb-only) use solutionSideNav-only visibleIn', () => {
+  it('file upload / index data visualizer / data drift remain in globalSearch + solutionSideNav', () => {
     const links = getDeepLinks(true, fullCapabilities, false);
-    const aiOps = links.find((l) => l.id === 'aiOps');
 
-    expect(links.find((l) => l.id === 'fileUpload')?.visibleIn).toEqual(['solutionSideNav']);
-    expect(links.find((l) => l.id === 'indexDataVisualizer')?.visibleIn).toEqual([
-      'solutionSideNav',
-    ]);
-    expect(links.find((l) => l.id === 'indexDataVisualizerPage')?.visibleIn).toEqual([
-      'solutionSideNav',
-    ]);
-    expect(links.find((l) => l.id === 'dataDrift')?.visibleIn).toEqual(['solutionSideNav']);
-    expect(links.find((l) => l.id === 'dataDriftPage')?.visibleIn).toEqual(['solutionSideNav']);
-    expect(aiOps?.deepLinks?.find((l) => l.id === 'logRateAnalysisPage')?.visibleIn).toEqual([
+    expect(links.find((l) => l.id === 'fileUpload')?.visibleIn).toEqual([
+      'globalSearch',
       'solutionSideNav',
     ]);
-    expect(aiOps?.deepLinks?.find((l) => l.id === 'logPatternAnalysisPage')?.visibleIn).toEqual([
+    expect(links.find((l) => l.id === 'indexDataVisualizer')?.visibleIn).toEqual([
+      'globalSearch',
       'solutionSideNav',
     ]);
-    expect(aiOps?.deepLinks?.find((l) => l.id === 'changePointDetectionsPage')?.visibleIn).toEqual([
+    expect(links.find((l) => l.id === 'dataDrift')?.visibleIn).toEqual([
+      'globalSearch',
       'solutionSideNav',
     ]);
   });
 
+  it('breadcrumb-only *Page variants are hidden everywhere (visibleIn: [])', () => {
+    const links = getDeepLinks(true, fullCapabilities, false);
+    const aiOps = links.find((l) => l.id === 'aiOps');
+
+    expect(links.find((l) => l.id === 'indexDataVisualizerPage')?.visibleIn).toEqual([]);
+    expect(links.find((l) => l.id === 'dataDriftPage')?.visibleIn).toEqual([]);
+    expect(aiOps?.deepLinks?.find((l) => l.id === 'logRateAnalysisPage')?.visibleIn).toEqual([]);
+    expect(aiOps?.deepLinks?.find((l) => l.id === 'logPatternAnalysisPage')?.visibleIn).toEqual([]);
+    expect(aiOps?.deepLinks?.find((l) => l.id === 'changePointDetectionsPage')?.visibleIn).toEqual(
+      []
+    );
+  });
+
   it('omits links when capabilities or license are insufficient', () => {
     expect(
       getDeepLinks(false, fullCapabilities, false).find((l) => l.id === 'overview')