adityamparikh opened a new pull request, #163: URL: https://github.com/apache/solr-mcp/pull/163
## Motivation On Solr 10, `/admin/mbeans` was removed; `get-collection-stats` already degrades gracefully and returns `null` for `cacheStats`/`handlerStats`. But nothing on the tool surface tells the client this is expected — an LLM client sees unexplained nulls and may retry, misreport a broken server, or go hunting for a workaround. A bare `null` a client must interpret is a documentation gap in the place clients actually look: the tool description. ## Changes - `get-collection-stats` description now states that on Solr 10+ the two stats are always null because the endpoint was removed from Solr, and that this is expected. Deliberately scoped to the description only: PR #111 is already modifying the `fetchCacheMetrics`/`fetchHandlerMetrics` catch clauses, and this avoids conflicting with it. A follow-up could migrate to `/admin/metrics` to restore the stats on Solr 10. Description-only change — no behavior change. `./gradlew build` passes (unit + Testcontainers integration tests). 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
