[BUG] Visual Editor Monitor is saved as Extraction Query Monitor

[xluo-aws]

What is the bug?
I'm testing on future playground with a remote data source using 2.17.1 version. Create a visual editor monitor but it's saved as Extraction Query monitor. Main has the same issue

How can one reproduce the bug?
Steps to reproduce the behavior:

1. Go to '...'
2. Select a workspace with Alert feature
3. Create a monitor using visual editor
4. Once the monitor is created, the monitor definition type becomes Extraction query.

What is the expected behavior?
The monitor definition type should remain as visual editor

What is your host/environment?

• Version [OS2.17.1]
• Plugins [Alert]

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.

Do you have any additional context?
Add any other context about the problem.

[AWSHurneyt]

@xluo-aws I wasn't able to reproduce the issue using the playground link you provided above. I created hurneyt-test monitor which shows visual graph on the details page, and the edit page shows the visual editor display.

I also was not able to reproduce this issue on the current opensearchproject/2.17.1 docker image.

[AWSHurneyt]

I was able to reproduce the issue in the playground link provided above by adjusting the workspace/datasource. Troubleshooting now.

[AWSHurneyt]

After some troubleshooting locally, it looks like the backend is not returning the ui_metadata field of the monitor JSON. If a monitor is created by curl commands/devtools, we do not require the user to supply the ui_metadata, and the frontend will use the extraction query editor when the ui_metadata is not present. Continuing to troubleshoot why this field is not being sent to the frontend.

[songkant-aws]

Backend ui_metadata is probably excluded based on 'User-Agent' header: https://github.com/opensearch-project/alerting/blob/main/alerting/src/main/kotlin/org/opensearch/alerting/util/RestHandlerUtils.kt#L20-L25.
The backend only includes ui_metadata only if the node server requests to opensearch 'User-Agent' header contain 'Opensearch-Dashboards'. However, MDSClient may not have it.