Skip to content

Add document management tool with CRUD operations #69

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AnilPuram
Copy link

@AnilPuram AnilPuram commented May 24, 2025

I added tool to do CRUD on documents.

  • Implemented new document_manage tool for document CRUD operations
  • Added support for creating, reading, updating, and deleting documents by ID
  • Updated README.md with documentation for the new tool
  • Added mock data script for testing with sample indices and documents

- Implemented new document_manage tool for document CRUD operations
- Added support for creating, reading, updating, and deleting documents by ID
- Updated README.md with documentation for the new tool
- Added mock data script for testing with sample indices and documents
@AnilPuram AnilPuram requested a review from a team as a code owner May 24, 2025 23:43
@AnilPuram
Copy link
Author

@JoshMock @Anaethelion

@AnilPuram
Copy link
Author

@gose who can review this?
its been pending for a while now

@JoshMock
Copy link
Member

JoshMock commented Jun 2, 2025

@AnilPuram We have some reservations about giving the MCP server the ability to modify or delete data, so we're not merging PRs that introduce that functionality until we have established more guidelines for the project regarding security, data loss, etc.

@JoshMock JoshMock added the tool:read-write Requests to add tools that can modify Elasticsearch data label Jun 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool:read-write Requests to add tools that can modify Elasticsearch data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants