Skip to content

Comments

[Agent Builder] add confirmation support for MCP server#247831

Draft
pgayvallet wants to merge 6 commits intoelastic:mainfrom
pgayvallet:ab-12274-mcp-confirmation
Draft

[Agent Builder] add confirmation support for MCP server#247831
pgayvallet wants to merge 6 commits intoelastic:mainfrom
pgayvallet:ab-12274-mcp-confirmation

Conversation

@pgayvallet
Copy link
Contributor

@pgayvallet pgayvallet commented Jan 5, 2026

Summary

Fix https://github.com/elastic/search-team/issues/12274

  • Implement stateful MCP server session management
    • Add GET and DELETE route for server->client communication and session cleanup
    • Implement multi-channel support (bi-directional communication, required for server-sent messages such as elicitations)
  • Add support for confirmation prompts for tools called via the MCP endpoint by converting them to MCP's elicitations

Issues and limitations

Stateful MCP capabilities aren't going to properly function with multi-node Kibana deployments (at least as long as we don't have sticky session management), because all requests of a given MCP sessions must be handled/routed to the same node, due to the stateful nature of keeping multiple channels / sockets open for bi-dir communications.

Technically with this PR, that means:

  1. Everything which was working before the PR still work
  2. Elicitations (confirmations) may not work properly on multi-node environments

Demo

Screen.Recording.2026-01-06.at.08.02.05.mov

@elasticmachine
Copy link
Contributor

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!
  • Click to trigger kibana-entity-store-performance-from-pr for this PR!

@pgayvallet
Copy link
Contributor Author

/ci

@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.4.0 labels Jan 6, 2026
@pgayvallet
Copy link
Contributor Author

/ci

@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 6, 2026

💔 Build Failed

Failed CI Steps

History

@leemthompo leemthompo added the feature:agent-builder Identify agent builder functionalities to be grouped together for release notes label Jan 14, 2026
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 feature:agent-builder Identify agent builder functionalities to be grouped together for release notes release_note:skip Skip the PR/issue when compiling release notes v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants