[Unified Rules] Reroute users to new rules app by baileycash-elastic · Pull Request #250493 · elastic/kibana

baileycash-elastic · 2026-01-26T21:58:09Z

Summary

This PR introduces a feature flag-based redirect for the existing triggersAlerts (rules) app (url: /app/management/insightsAndAlerting/triggersActions). When the flag is enabled, all web traffic is rerouted to app/rules.

Because of this change, a restructure of FTR rules tests was required. Tests for the new app (nearly identical to the old tests, with some small tweaks) will exist in the directory: x-pack/platform/test/functional_with_es_ssl/apps/rules.

Previous PRs modified existing rules tests to utilize the new rules app; these have been partially reverted and the unifiedRulesPage feature flag has been removed from the config.

# without this, the unified rules UI will not appear, and the redirect will not occur
xpack.trigger_actions_ui.enableExperimental:
  - unifiedRulesPage

Testing changes

Navigation has been reverted in the old tests from rules to triggersActions to preserve originally intended flows
Some actions have been tweaked in new tests to address flakiness and to adhere to new layout
"View in app" tests were not carried forward to new rules tests as this functionality was removed

Notes

The issue in the branch name is here. We determined it is simpler to implement a redirect for now than to incorporate feature flags into nav menus. The menus will be updated when we decide to remove the flag.
With the redirect in place, we may be able to rollback feature flag conditionals previously implemented in a future PR. These efforts were not included here for brevity, and to reduce risk.

trying to push

Copilot

Pull request overview

This PR implements a feature flag-based redirect that routes users from the legacy triggersActions app (/app/management/insightsAndAlerting/triggersActions) to the new unified rules app (/app/rules) when the unifiedRulesPage feature flag is enabled. This allows for a gradual migration to the new rules interface.

Changes:

Implemented redirect logic in the plugin registration to route triggersActions traffic to the rules app
Restructured FTR tests to separate tests for the new rules app from the legacy triggersActions tests
Updated test configurations and paths to support both app versions
Made the rules app visible in navigation when the feature flag is enabled

Reviewed changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
x-pack/platform/test/functional_with_es_ssl/config.base.ts	Removed unifiedRulesPage from default feature flags
x-pack/platform/test/functional_with_es_ssl/apps/triggers_actions_ui/rules_page/*.ts	Updated tests to use triggersActions paths instead of rules paths
x-pack/platform/test/functional_with_es_ssl/apps/triggers_actions_ui/index.ts	Removed rules_page test loading
x-pack/platform/test/functional_with_es_ssl/apps/triggers_actions_ui/index.rules.ts	Added rules_page test loading
x-pack/platform/test/functional_with_es_ssl/apps/rules/*/.ts	New test files for the unified rules app with /app/rules paths
x-pack/platform/test/functional_with_es_ssl/apps/rules/config.ts	Configuration file that enables unifiedRulesPage feature flag for rules tests
x-pack/platform/plugins/shared/triggers_actions_ui/public/plugin.ts	Moved rules app registration and added redirect logic from triggersActions to rules app
x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/rules_page/rules_page.tsx	Changed test subject from rulesPageTitle to appTitle
x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/rules_list/components/rules_list_table.tsx	Updated routing logic to use feature flag check instead of current app detection
src/platform/packages/shared/kbn-rule-data-utils/src/routes/stack_rule_paths.ts	Added rulesAppRoute constant
src/platform/packages/shared/kbn-management/cards_navigation/src/types.ts	Changed RULES AppId from triggersActions to rules
src/platform/packages/shared/deeplinks/management/deep_links.ts	Added RulesAppId type and updated AppId union
src/platform/packages/shared/deeplinks/management/constants.ts	Added RULES_APP_ID constant
packages/kbn-optimizer/limits.yml	Increased triggersActionsUi bundle size limit
.github/CODEOWNERS	Added ownership for new rules test directory
.buildkite/ftr_platform_stateful_configs.yml	Added rules config to CI build

ElenaStoeva

Management changes lgtm

This commit: adds the option to disable url checks in navigateToApp, in order for the test to execute removes app registration checks and relies solely on feature flag values

adcoelho · 2026-02-02T09:40:53Z

src/platform/packages/shared/kbn-management/cards_navigation/src/types.ts

  REPORTING = 'reporting',
  CONNECTORS = 'triggersActionsConnectors',
-  RULES = 'triggersActions',
+  RULES = 'rules',


What is this change about?

adcoelho · 2026-02-02T09:45:33Z

...d/triggers_actions_ui/public/application/sections/rules_list/components/rules_list_table.tsx

@@ -258,11 +255,6 @@ export const RulesListTable = (props: RulesListTableProps) => {
  const { euiTheme } = useEuiTheme();

  // Detect current app to determine the correct path format


nit: this can also be removed

baileycash-elastic · 2026-02-03T02:06:32Z

@elasticmachine merge upstream

elasticmachine · 2026-02-03T03:49:38Z

💚 Build Succeeded

Buildkite Build
Commit: 00a8e22

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id	before	after	diff
`@kbn/rule-data-utils`	213	214	+1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
`triggersActionsUi`	1.6MB	1.6MB	-2.3KB

Page load bundle

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

id	before	after	diff
`triggersActionsUi`	108.3KB	108.6KB	+329.0B

Unknown metric groups

API count

id	before	after	diff
`@kbn/rule-data-utils`	226	227	+1

History

💛 Build #388199 was flaky 291d313
💚 Build #387746 succeeded c1adf86
💔 Build #387684 failed a297bd8
💔 Build #387216 failed a297bd8
💔 Build #387199 failed d3a8ecc
💔 Build #387135 failed 9e70ec4

csr

LGTM

Resolves: elastic/ingest-dev#6716 Adds a `getInAppUrl` definition for the rule saved object, which defines where to view the rule in the UI. This is used by the Saved Objects management view, and the integrations assets view. This relies on the re-routing behavior from `/app/management/insightsAndAlerting/triggersActions` to `/app/rules` controlled by the `unifiedRulesPage` flag as implemented in: #250493

Resolves: elastic/ingest-dev#6716 Adds a `getInAppUrl` definition for the rule saved object, which defines where to view the rule in the UI. This is used by the Saved Objects management view, and the integrations assets view. This relies on the re-routing behavior from `/app/management/insightsAndAlerting/triggersActions` to `/app/rules` controlled by the `unifiedRulesPage` flag as implemented in: elastic#250493

## Release Notes Observability rules and stack management rules experiences have merged into a new interface. Unified rules gives users a place to more easily manage rules. Any existing uses of the legacy rules experiences will be redirected to unified rules. ## Summary Closes elastic/kibana-team#2926 Closes elastic/kibana-team#2939 (final change) This PR: - Removes a feature flag for the unified rules feature, officially letting it replace observability and stack management rules. For the most part, this affects navigation. - Removes outdated/redundant stack management rules tests - Evolves the "view in app" button in stack management rule details view to "view in discover," now that experiences have been merged. (viewInAppRelativeUrl value for alerts will be stripped out in a future PR) - Fixes a discrepancy with the alert summary widget where clicking on alert counts would not reveal the alerts tab in unified rules ## Background Epic: elastic/rna-program#152 In this [PR](#250493), we copied functional tests from stack management to a new directory specifically for testing unified rules flows and paths. The test flows are mostly identical, with only some minor differences to navigation. The new tests also included a feature flag specifically for unified rules. Existing tests did not use this flag. Over time, we implemented redirects to observability and stack management rules apps to take users to the unified rules page when the feature flag is on. ## Testing - Stack and observability rules should appear together at `/app/rules` - searchSource (not ES|QL discover mode-based rules) should render "View in Discover" button on the rule details page, "View in App" will no longer appear on details page header - Stack management rules should redirect to unified rules (always) - Observability rules should redirect to unified rules (always) - All apps that pointed to observability rules should point to unified rules (i.e. synthetics, SLO, etc) ## Out of Scope With few exceptions, most links throughout Kibana to obs & stack rules have not been updated so far, we introduced redirects at the plugin level in previous commits, that acts as catch-alls for incoming traffic. We'll handle updates to links and the remainder of test & story updates in future cleanup PRs. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

…258214) ## Release Notes Observability rules and stack management rules experiences have merged into a new interface. Unified rules gives users a place to more easily manage rules. Any existing uses of the legacy rules experiences will be redirected to unified rules. ## Summary Closes elastic/kibana-team#2926 Closes elastic/kibana-team#2939 (final change) This PR: - Removes a feature flag for the unified rules feature, officially letting it replace observability and stack management rules. For the most part, this affects navigation. - Removes outdated/redundant stack management rules tests - Evolves the "view in app" button in stack management rule details view to "view in discover," now that experiences have been merged. (viewInAppRelativeUrl value for alerts will be stripped out in a future PR) - Fixes a discrepancy with the alert summary widget where clicking on alert counts would not reveal the alerts tab in unified rules ## Background Epic: elastic/rna-program#152 In this [PR](elastic#250493), we copied functional tests from stack management to a new directory specifically for testing unified rules flows and paths. The test flows are mostly identical, with only some minor differences to navigation. The new tests also included a feature flag specifically for unified rules. Existing tests did not use this flag. Over time, we implemented redirects to observability and stack management rules apps to take users to the unified rules page when the feature flag is on. ## Testing - Stack and observability rules should appear together at `/app/rules` - searchSource (not ES|QL discover mode-based rules) should render "View in Discover" button on the rule details page, "View in App" will no longer appear on details page header - Stack management rules should redirect to unified rules (always) - Observability rules should redirect to unified rules (always) - All apps that pointed to observability rules should point to unified rules (i.e. synthetics, SLO, etc) ## Out of Scope With few exceptions, most links throughout Kibana to obs & stack rules have not been updated so far, we introduced redirects at the plugin level in previous commits, that acts as catch-alls for incoming traffic. We'll handle updates to links and the remainder of test & story updates in future cleanup PRs. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

baileycash-elastic added 2 commits January 26, 2026 15:53

general improvements, some boilerplate

8a3d999

Merge branch 'main' into unified-rules-239608

f8d102f

github-actions bot added the author:actionable-obs PRs authored by the actionable obs team label Jan 26, 2026

add redirect, roll back added feature flag check

74beb5b

baileycash-elastic force-pushed the unified-rules-239608 branch from 785b166 to 74beb5b Compare January 27, 2026 17:08

include state with redirect

baccade

baileycash-elastic force-pushed the unified-rules-239608 branch 2 times, most recently from 14a076d to baccade Compare January 28, 2026 16:15

baileycash-elastic added 3 commits January 28, 2026 13:13

duplicate rules tests with a new config, reorganize existing tests

4fa3ba9

unblock ci

9e70ec4

maintain triggers actions nav in old test files, update codeowners

a297bd8

baileycash-elastic force-pushed the unified-rules-239608 branch from d3a8ecc to a297bd8 Compare January 29, 2026 05:19

baileycash-elastic added 3 commits January 29, 2026 09:53

align page titles across apps

ecaf43e

remove view in app tests, update settings test

594b954

trying to push

merge in main

c1adf86

baileycash-elastic marked this pull request as ready for review January 29, 2026 23:44

baileycash-elastic requested review from a team as code owners January 29, 2026 23:44

baileycash-elastic requested a review from Copilot January 29, 2026 23:55

Copilot AI reviewed Jan 29, 2026

View reviewed changes

jbudz approved these changes Jan 30, 2026

View reviewed changes

ElenaStoeva approved these changes Jan 30, 2026

View reviewed changes

Add ftrs to redirect from triggers actions

291d313

This commit: adds the option to disable url checks in navigateToApp, in order for the test to execute removes app registration checks and relies solely on feature flag values

baileycash-elastic requested a review from a team as a code owner January 30, 2026 19:43

adcoelho reviewed Feb 2, 2026

View reviewed changes

Merge branch 'main' into unified-rules-239608

19f4e45

adcoelho approved these changes Feb 2, 2026

View reviewed changes

baileycash-elastic added 2 commits February 2, 2026 11:31

Merge branch 'main' into unified-rules-239608

401a699

revert hard coded cards nav change

f0e463e

Merge branch 'main' into unified-rules-239608

00a8e22

csr approved these changes Feb 3, 2026

View reviewed changes

baileycash-elastic merged commit 03c059b into elastic:main Feb 3, 2026
16 checks passed

kibanamachine added the v9.4.0 label Feb 3, 2026

ana-davydova mentioned this pull request Feb 4, 2026

[One Management] Update Fleet Integration rule creation to direct to app/rules #239623

Open

MichelLosier mentioned this pull request Feb 11, 2026

Implement getInAppUrl definition for rules saved object #252782

Merged

10 tasks

MichelLosier mentioned this pull request Feb 17, 2026

[Fleet] Make Alerts embedded in integrations more easily disvoverable #249798

Closed

baileycash-elastic mentioned this pull request Mar 19, 2026

[Unified Rules] [Release] Remove unified rules feature flag #258214

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Unified Rules] Reroute users to new rules app#250493

[Unified Rules] Reroute users to new rules app#250493
baileycash-elastic merged 15 commits intoelastic:mainfrom
baileycash-elastic:unified-rules-239608

baileycash-elastic commented Jan 26, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

ElenaStoeva left a comment

Uh oh!

adcoelho Feb 2, 2026

Uh oh!

adcoelho Feb 2, 2026

Uh oh!

baileycash-elastic commented Feb 3, 2026

Uh oh!

elasticmachine commented Feb 3, 2026

API count

Uh oh!

csr left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

		@@ -258,11 +255,6 @@ export const RulesListTable = (props: RulesListTableProps) => {
		const { euiTheme } = useEuiTheme();

		// Detect current app to determine the correct path format

Conversation

baileycash-elastic commented Jan 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Testing changes

Notes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

ElenaStoeva left a comment

Choose a reason for hiding this comment

Uh oh!

adcoelho Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

adcoelho Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

baileycash-elastic commented Feb 3, 2026

Uh oh!

elasticmachine commented Feb 3, 2026

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Async chunks

Page load bundle

API count

History

Uh oh!

csr left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

baileycash-elastic commented Jan 26, 2026 •

edited

Loading