adityamparikh commented on PR #194: URL: https://github.com/apache/solr-mcp/pull/194#issuecomment-5650648446
Closing this one. The `index-file` tool only exists in STDIO mode, and there is no equivalent way to get a file into the server over HTTP, so merging it would give the two transports different tool surfaces for the same job. I would rather not ship a capability that half the deployments cannot use. What HTTP parity would need is a way for a client to hand the server a file: either the MCP file-transfer primitive that is still being specified (SEP-2631), or an out-of-band upload endpoint with its own auth and size limits. Both are bigger than this PR and neither belongs in an indexing change, so I am parking the whole feature rather than carrying a STDIO-only version. The branch stays as is. Two transport-neutral pieces could be lifted out later if useful: the `indexBatch` extraction in `IndexingService`, and the schema-first guidance in the tool descriptions and `index-documents` prompt. The search-argument fixes that started here already live in #195. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
