[Streams] Enable UI by default

[mohamedhamed-ahmed]

closes #232455

Summary

This PR removes the below enable streams UI advanced setting option.
The setting is enabled by default for observability serverless deployment as well as for stateful.

[elasticmachine]

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

[elasticmachine]

Pinging @elastic/streams-program-team (Team:streams-program)

[shahzad31]

Obs UX changes LGTM !!

[flash1293]

Tested and works fine.

The

    return from(
      repositoryClient
        .fetch('GET /api/streams/_status', {
          signal: new AbortController().signal,
        })
        .then(
          (response) => (response.enabled ? ENABLED_STATUS : DISABLED_STATUS),
          (error) => {
            logger.error(error);
            return UNKNOWN_STATUS;
          }
        )
    ).pipe(startWith(UNKNOWN_STATUS), shareReplay(1));

doesn't matter anymore, since it's never triggered (serverless security/search don't show the nav item ever, and that would be the only case that's still affected).

After that LGTM

[mohamedhamed-ahmed]

doesn't matter anymore, since it's never triggered (serverless security/search don't show the nav item ever, and that would be the only case that's still affected).

Makes sense, will remove it and return Disabled otherwise 👍

[Copilot]

Pull Request Overview

This PR enables the Streams UI by default for both observability serverless and stateful deployments by removing the observability:enableStreamsUI advanced setting option. The UI is now always enabled instead of being controlled by a configurable setting.

Key Changes:

• Removed the OBSERVABILITY_ENABLE_STREAMS_UI setting and its associated configuration
• Hardcoded the Streams UI to be enabled (enabled: true)
• Updated status determination logic to use deployment type checks instead of setting values

Reviewed Changes

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

Show a summary per file

File	Description
use_streams_privileges.ts	Removes UI setting check and hardcodes UI as enabled
plugin.ts	Updates registerFeatureFlags function call to remove plugins parameter
feature_flags.ts	Removes UI setting registration and serverless detection logic
public/plugin.ts	Simplifies status observable to use deployment type checks instead of settings
oss_platform.json	Removes telemetry schema entry for the deleted setting
types.ts	Removes usage stats type definition for the deleted setting
schema.ts	Removes telemetry collection schema for the deleted setting
observability_project/index.ts	Removes setting from serverless project configuration
setting_ids/index.ts	Removes the setting ID constant definition

[kapral18]

LGTM for kibana-management

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: 79825dc

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
streams	23	22	-1

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/management-settings-ids	133	132	-1

Async chunks

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

id	before	after	diff
streamsApp	668.9KB	668.8KB	-40.0B

Page load bundle

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

id	before	after	diff
streams	7.5KB	7.4KB	-111.0B

Unknown metric groups

API count

id	before	after	diff
@kbn/management-settings-ids	134	133	-1

History

• 💔 Build #336232 failed 0054b12
• 💚 Build #335827 succeeded de6462c
• 💔 Build #335783 failed 0bd44c5

[TinaHeiligers]

LGTM