adityamparikh opened a new pull request, #196:
URL: https://github.com/apache/solr-mcp/pull/196

   ### Summary
   Two small changes to the inline indexing tools (`index-json-documents`, 
`index-csv-documents`, `index-xml-documents`, `index-markdown-documents`).
   
   - **Payload-splitting guidance.** Inline payloads are generated by the model 
as tool arguments, so one oversized call is truncated at the model's output 
limit and fails to parse. The server instructions now tell the model to split 
inputs larger than a few hundred KB across several calls, and each inline tool 
description carries a one-clause reminder. The guidance is a size rather than a 
document count because record sizes vary widely. The JSON description also 
notes that CSV costs fewer tokens for large flat records.
   - **Soft commit per call.** Splitting multiplies commits, so 
`indexDocuments` now ends with a soft commit (`waitFlush=false, 
waitSearcher=true, softCommit=true`) instead of a hard commit. Documents are 
still searchable when the tool returns; the segment fsync is left to Solr's 
`autoCommit`, which the `_default` configset enables at 15 s. The transaction 
log covers durability in between. README notes that a configset with 
`autoCommit` disabled should enable it.
   
   ### Tests
   - `IndexingServiceTest` verifies the soft-commit overload is used and the 
hard-commit overload is not.
   - `McpToolRegistrationTest` fails if any `index-*` tool description drops 
the splitting guidance.
   - New `McpServerInstructionsTest` guards the instructions text in 
`application.properties`.
   
   ### Verification
   `./gradlew build` on Java 25: 407 tests, 0 failures, 7 skipped. Spotless 
applied.
   
   ### Notes
   - Touches the same tool descriptions as #194 and #195; whichever merges 
second needs a trivial rebase.
   - A follow-up PR consolidates the four inline tools into one 
`index-documents` tool with a `format` argument.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   
   https://claude.ai/code/session_019zJ9WA8nNyA5Yxb8ueM7vV
   


-- 
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]

Reply via email to