Skip to content

[Unified rules] Hide stack rules from global search#260088

Merged
baileycash-elastic merged 8 commits intoelastic:mainfrom
baileycash-elastic:kt-3108-hide-stack-rules-from-search
Apr 1, 2026
Merged

[Unified rules] Hide stack rules from global search#260088
baileycash-elastic merged 8 commits intoelastic:mainfrom
baileycash-elastic:kt-3108-hide-stack-rules-from-search

Conversation

@baileycash-elastic
Copy link
Copy Markdown
Contributor

@baileycash-elastic baileycash-elastic commented Mar 27, 2026

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

@baileycash-elastic baileycash-elastic requested a review from a team as a code owner March 27, 2026 18:37
@baileycash-elastic baileycash-elastic added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.4.0 labels Mar 27, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@github-actions github-actions bot added the author:actionable-obs PRs authored by the actionable obs team label Mar 27, 2026
@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp bot commented Mar 27, 2026

Approvability

Verdict: Needs human review

This PR modifies visibility behavior for management items in global search by introducing a new visibleIn property that takes precedence over existing hideFromGlobalSearch logic. The author does not own any of the changed files, which are owned by @elastic/kibana-management and @elastic/response-ops teams who should review this behavioral change.

No code changes detected at c80d47c. Prior analysis still applies.

You can customize Macroscope's approvability policy. Learn more.

@baileycash-elastic baileycash-elastic requested a review from a team as a code owner March 27, 2026 20:59
@baileycash-elastic baileycash-elastic force-pushed the kt-3108-hide-stack-rules-from-search branch from c14d467 to e1e1cf1 Compare March 27, 2026 21:22
id: PLUGIN_ID,
title: featureTitle,
order: 1,
visibleIn: [],
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Copy Markdown
Member

@umbopepato umbopepato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RO code changes LGTM!

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
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason t have both hideFromGlobalSearch and visibleIn? Could visibleIn be more descriptive, such as deepLinkVisibility?

Copy link
Copy Markdown
Contributor Author

@baileycash-elastic baileycash-elastic Apr 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Copy Markdown
Contributor

@mattkime mattkime Apr 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Contributor

@mattkime mattkime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Contributor

@mattkime mattkime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 🚀 🚀

@baileycash-elastic baileycash-elastic enabled auto-merge (squash) April 1, 2026 16:28
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 1, 2026

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #201 / Agent Builder sidebar Sidebar Conversation Flow sends a message and receives a response

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
management 12.4KB 12.5KB +123.0B
triggersActionsUi 109.6KB 109.6KB +13.0B
total +136.0B

History

@baileycash-elastic baileycash-elastic merged commit bb6117d into elastic:main Apr 1, 2026
21 checks passed
mbondyra added a commit to mbondyra/kibana that referenced this pull request Apr 1, 2026
…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
eokoneyo pushed a commit to davismcphee/kibana that referenced this pull request Apr 2, 2026
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this pull request Apr 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author:actionable-obs PRs authored by the actionable obs team backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants