Skip to content

Conversation

@pgayvallet
Copy link
Contributor

@pgayvallet pgayvallet commented Sep 12, 2025

@pgayvallet
Copy link
Contributor Author

/ci

@pgayvallet pgayvallet added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v9.2.0 labels Sep 12, 2025
@pgayvallet pgayvallet marked this pull request as ready for review September 12, 2025 11:30
@pgayvallet pgayvallet requested review from a team as code owners September 12, 2025 11:30
Copy link
Member

@jedrazb jedrazb left a comment

Choose a reason for hiding this comment

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

LGTM, You missed renaming apis in our readme

export const ONECHAT_TITLE = 'Chat';
export const ONECHAT_APP_ID = 'agent_builder';
export const ONECHAT_PATH = '/app/agent_builder';
export const ONECHAT_TITLE = 'Agent Builder';
Copy link
Member

Choose a reason for hiding this comment

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

i18n? this will be used for rendering labels in some parts of UI

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, now is the right time to do it 👍

{
link: { navId: 'agent_builder' },
breadcrumbs: ['Conversations'],
link: { deepLinkId: 'agent_builder' },
Copy link
Contributor

Choose a reason for hiding this comment

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

this needs to be navId so it can click the parent item. which isn't a deep link.

@pgayvallet pgayvallet enabled auto-merge (squash) September 14, 2025 13:50
@pgayvallet pgayvallet merged commit 9e2261d into elastic:main Sep 14, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #86 / Observability AI Assistant Functional tests knowledge_base_management/index.spec.ts Knowledge management tab Bulk import knowledge base entries successfully imports multiple entries from a NDJSON file

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
onechat 505 506 +1

Async chunks

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

id before after diff
enterpriseSearch 1.1MB 1.1MB +29.0B
onechat 257.5KB 257.6KB +19.0B
total +48.0B

Page load bundle

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

id before after diff
onechat 13.0KB 13.2KB +158.0B
serverlessSearch 29.9KB 29.9KB +36.0B
total +194.0B

History

KodeRad pushed a commit to KodeRad/kibana that referenced this pull request Sep 15, 2025
## Summary

Part of elastic/search-team#10961

(sibling of elastic#234304)

---------

Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Sep 24, 2025
## Summary

Part of elastic/search-team#10961

(sibling of elastic#234304)

---------

Co-authored-by: kibanamachine <[email protected]>
niros1 pushed a commit that referenced this pull request Sep 30, 2025
## Summary

Part of elastic/search-team#10961

(sibling of #234304)

---------

Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants