Skip to content

[9.4] [Agent Builder] Prevent conversation overwrite via auto-create (#280189)#280877

Open
brunofarache wants to merge 2 commits into
elastic:9.4from
brunofarache:backport/9.4/pr-280189
Open

[9.4] [Agent Builder] Prevent conversation overwrite via auto-create (#280189)#280877
brunofarache wants to merge 2 commits into
elastic:9.4from
brunofarache:backport/9.4/pr-280189

Conversation

@brunofarache

Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 9.4:

This is a manual backport of #280189. The 9.4 conversation client still uses the older hasAccess-based shape, so the fix was adapted by hand rather than cherry-picked cleanly. Only the conversation client (client.ts) and its unit test (client.test.ts) are changed:

  • exists() now reports physical document existence and no longer applies the access gate.
  • create() uses op_type: 'create' so an existing conversation (e.g. owned by another user) is never overwritten.

Security fix. Please review carefully before merging.

Questions ?

Please refer to the Backport tool documentation

…ic#280189)

Manual backport of elastic#280189 adapted to the 9.4 hasAccess-based conversation
client. Makes exists() report physical document existence and create()
non-destructive via op_type: 'create'.
@brunofarache brunofarache added the backport This PR is a backport of another PR label Jul 24, 2026
@brunofarache brunofarache self-assigned this Jul 24, 2026
@brunofarache
brunofarache marked this pull request as ready for review July 24, 2026 19:20
@brunofarache
brunofarache requested a review from machadoum July 24, 2026 19:21
@brunofarache
brunofarache enabled auto-merge (squash) July 24, 2026 19:22
@kibanamachine

Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #17 / lens app - group 4 lens dashboard tests should add a drilldown to a Lens by-value chart
  • [job] [logs] Scout Lane #7 - stateful-classic / default / local-stateful-classic - Discover app - errors - should render invalid scripted field error

Metrics [docs]

✅ unchanged

History

cc @brunofarache

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants