[Unified rules] Hide stack rules from global search#260088
[Unified rules] Hide stack rules from global search#260088baileycash-elastic merged 8 commits intoelastic:mainfrom
Conversation
|
Pinging @elastic/response-ops (Team:ResponseOps) |
ApprovabilityVerdict: Needs human review This PR modifies visibility behavior for management items in global search by introducing a new No code changes detected at You can customize Macroscope's approvability policy. Learn more. |
c14d467 to
e1e1cf1
Compare
| id: PLUGIN_ID, | ||
| title: featureTitle, | ||
| order: 1, | ||
| visibleIn: [], |
There was a problem hiding this comment.
added visibleIn here to avoid changing the intentions behind hideFromGlobalSearch filter in management plugin. We still need the app in deep links to retain sidenav appearance
| icon?: string; // URL to image file; fallback if no `euiIconType` | ||
| hideFromSidebar?: boolean; | ||
| hideFromGlobalSearch?: boolean; // Hide from global search results | ||
| visibleIn?: AppDeepLinkLocations[]; // Controls deep link visibility; takes precedence over hideFromGlobalSearch |
There was a problem hiding this comment.
Is there a reason t have both hideFromGlobalSearch and visibleIn? Could visibleIn be more descriptive, such as deepLinkVisibility?
There was a problem hiding this comment.
hideFromGlobalSearch is being used in a couple of other places. This is a last minute change as part of the unified rules effort next week, trying to get approvals for its removal may delay us. I also didn't want to nuke a use case I'm not aware of.
visibleIn is a newer toggle used elsewhere in Kibana for deep link visibility management. I agree it's vague, but I didn't want to break away from convention in one place and have a discrepancy with how we manage visibility
There was a problem hiding this comment.
I also didn't want to nuke a use case I'm not aware of.
I just wanted to verify my understanding, any cleanup could happen later.
visibleIn is a newer toggle used elsewhere in Kibana for deep link visibility management
Looks like its used in the core app interface, good reason to reuse it.
mattkime
left a comment
There was a problem hiding this comment.
Making sure this is an easy to understand addition to the api we expose. We should probably talk this over just to make sure I understand everything - while I understand how the management app works, I'm less clear about how things are exposed externally.
💔 Build Failed
Failed CI StepsTest Failures
Metrics [docs]Page load bundle
History
|
…heck * commit '6f040b29a5220ce12886a9731f656613e50aff06': (34 commits) [Entity Analytics] Add entity resolution UI to service flyout (elastic#260504) [Dashboard] Fix setState in embeddables (elastic#260082) [EDR Workflows] Unskip FTR tests that failed due to transient Fleet service unavailability (elastic#260519) [Observability:Streams] Fix query streams error handling test (elastic#260777) [Alerting v2] Dispatcher grouping modes, throttle strategies, and matcher autosuggestion (elastic#260249) [Dashboard] State extraction as a consistent override (elastic#259839) [Alerting v2] [Rule authoring] Fix rule name validation and error visibility in create/edit flow (elastic#260337) [Fix] re-introduce sln breadcrumbs to unified rules (elastic#260289) [Security Solution][Endpoint] Updated kibana docs to include `xpack.securitySolution.maxEndpointScriptFileSize` as configurable in cloud (elastic#260568) [Alerting v2] updated the alerting-v2-constants package with artifacts constants, fix to the runbook max characters (elastic#260342) [Automatic Import V2] Provide user tooltips (elastic#260725) [One Workflow] Deduplicate step types by base type in workflow list (elastic#260763) [Security Solution] Execution results UI: Enable the feature flag (elastic#260711) [Metrics][Discover] internal/search/esql_async returns 200 but METRICS_INFO responds with error (elastic#260746) Collapse redundant anyOf/oneOf array unions in OAS query params (elastic#260585) [Unified rules] Hide stack rules from global search (elastic#260088) [Agent Builder] Sidebar navigation updates (elastic#260728) [* As Code] Use PUT for upserts (elastic#260318) Update EUI to v114.0.0 (elastic#259497) [Entity Resolution] Add contextual-security-apps as co-owner of resolution paths (elastic#260659) ... # Conflicts: # src/platform/plugins/shared/dashboard/public/index.ts
## Summary Closes elastic/kibana-team#3108 Removes stack management rules from global search to avoid redundancy with unified rules ## Before https://github.com/user-attachments/assets/16f50bdc-1dfe-4922-9645-2af301facd67 ## After https://github.com/user-attachments/assets/5902d1aa-4107-4b51-b2cc-457254383dc6
## Summary Closes elastic/kibana-team#3108 Removes stack management rules from global search to avoid redundancy with unified rules ## Before https://github.com/user-attachments/assets/16f50bdc-1dfe-4922-9645-2af301facd67 ## After https://github.com/user-attachments/assets/5902d1aa-4107-4b51-b2cc-457254383dc6
Summary
Closes https://github.com/elastic/kibana-team/issues/3108
Removes stack management rules from global search to avoid redundancy with unified rules
Before
Screen.Recording.2026-03-27.at.2.32.18.PM.mov
After
Screen.Recording.2026-03-27.at.2.33.52.PM.mov