[Observability AI Assistant] Use up / down arrows to select up to 5 previously used prompts

[CoenWarmer]

Summary

This adds storing the last 5 used prompts in local storage for quick reuse.
Triggered by using the up or down arrow keys while the natural language prompt textarea editor has focus.

Screen.Recording.2024-03-29.at.15.46.30.mov

[elasticmachine]

Pinging @elastic/obs-knowledge-team (Team:obs-knowledge)

[apmmachine]

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

• /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
• /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
• run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

[dgieselaar]

nice addition!

[dgieselaar]

Two things I've noticed:

• I can't clear a previously used prompt, even though it looks like there is a clear button. Clearing it actually selects it.
• WDYT about only enabling this when there is no input text? I can imagine people want to use the up/down arrows for other reasons once they've entered text. (e.g. to scroll back)

[boriskirov]

Two more cents here:

1. Let's remove the search input, we can simplify this as much as possible.
2. When you hit delete (cross), does the previous 6th prompt come up? If yes, then maybe it's okay to have that, but if not we can remove it and simplify.

[CoenWarmer]

Two things I've noticed:

• I can't clear a previously used prompt, even though it looks like there is a clear button. Clearing it actually selects it.
• WDYT about only enabling this when there is no input text? I can imagine people want to use the up/down arrows for other reasons once they've entered text. (e.g. to scroll back)

1. Yes
2. Agreed

[CoenWarmer]

Two more cents here:

1. Let's remove the search input, we can simplify this as much as possible.
2. When you hit delete (cross), does the previous 6th prompt come up? If yes, then maybe it's okay to have that, but if not we can remove it and simplify.

1. I quite like it and would like to keep it.
2. No. Only 5 are stored.

[dgieselaar]

Shouldn't onFocus be called when the EuiInputPopover element moves focus back to the text area? I'm assuming it does do that, and if not, should we do that ourselves? Depending on what we do on focus in the consumer of this component, maybe that doesn't matter much.

[CoenWarmer]

handleClosePopover on line 89 also calls onFocus. I removed the onFocus here and the focus is correctly passed back to the text area.

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 0143c3d
• Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-179696-0143c3d22656
• Observability Deployment

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
observabilityAIAssistantApp	240	242	+2

Async chunks

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

id	before	after	diff
observabilityAIAssistantApp	149.3KB	151.0KB	+1.7KB

Canvas Sharable Runtime

The Canvas "shareable runtime" is an bundle produced to enable running Canvas workpads outside of Kibana. This bundle is included in third-party webpages that embed canvas and therefor should be as slim as possible.

id	before	after	diff
module count	-	5407	+5407
total size	-	8.8MB	+8.8MB

Unknown metric groups

ESLint disabled line counts

id	before	after	diff
observabilityAIAssistantApp	12	14	+2

Total ESLint disabled count

id	before	after	diff
observabilityAIAssistantApp	12	14	+2

History

• 💔 Build #210183 failed 4af5f4b
• 💚 Build #209740 succeeded e9bed9f
• 💚 Build #202081 succeeded 901e0e3
• 💚 Build #201809 succeeded 5884e63

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @CoenWarmer