docs: slim README to a user "how to use" guide (builds on #147; ports #95) - #151
Merged
epugh merged 10 commits intoJun 15, 2026
Merged
Conversation
Refocuses README.md on end-user "how to use" content and moves developer/operational detail into the docs that own it, so the README stays scannable and the deep material has a single home. README.md - Lean structure: value prop -> Quick start (start Solr, build, connect a client, try it) -> Example prompts -> tools/resources/prompts tables -> essential config -> documentation hub -> community. - Ports the unique end-user content from #95: sample-data blurb (films/books) and the categorized Example prompts. - Defers the full per-client matrix to the in-repo site pages under docs/site/content/pages/mcp/ (from apache#143), security to docs/security/, observability to the site page, and build/Docker/native/SBOM to dev-docs. - Fixes carried over from the apache#147 review: HTTP_SECURITY_ENABLED (secured by default), not the non-existent SECURITY_ENABLED. CONTRIBUTING.md - Convert bare-path references to markdown links for consistency. - De-duplicate "Publishing to Maven Local" -> link the dev-docs section (CONTRIBUTING's own principle is "no duplication; detail lives in dev-docs"). dev-docs/DEPLOYMENT.md - New "Image variants — why three images" section receiving the rationale moved out of the README: the image x mode matrix, run commands, why Jib builds the JVM image and Paketo the native variants (and why Paketo's JVM image breaks STDIO), and the native Claude Desktop config (latest-native-stdio). dev-docs/DEVELOPMENT.md - Move the consumer-facing SBOM content (where it ships, fetch via the actuator endpoint, scan with Trivy/Grype) here next to "Generating the SBOM locally", replacing the now-removed README section it linked to. Companion website-page fixes (.junie/mcp/mcp.json, VS Code type: http, HTTP_SECURITY_ENABLED) flow into apache#143. Depends on apache#143 for the docs/site/* links to resolve on main. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>
adityamparikh
added a commit
to adityamparikh/solr-mcp
that referenced
this pull request
Jun 14, 2026
Three factual fixes surfaced while reviewing the README polish (apache#147/apache#151), which the website source pages share: - clients/jetbrains.md: Junie's project MCP config lives at `.junie/mcp/mcp.json` (nested `mcp/` dir), not `.junie/mcp.json`. - clients/vs-code.md: the `/mcp` endpoint is Streamable HTTP, so the VS Code server entry needs `"type": "http"`, not the legacy `"sse"`. - security.md: HTTP mode is **secured by default** (`http.security.enabled` defaults to true). Corrected the "disabled by default" framing and removed the non-existent `SECURITY_ENABLED` toggle; the issuer (`OAUTH2_ISSUER_URI`) is what you set to use auth, and `HTTP_SECURITY_ENABLED=false` disables it for local dev. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>
3 tasks
- README: add a "Completions" subsection (the `{collection}` resource
argument and the search-collection/index-data/view-schema/design-schema
prompt arguments all autocomplete to live collection names), and a
one-line Solr compatibility note linking the dev-docs section.
- dev-docs/DEVELOPMENT.md: add "Solr Version Compatibility" under Testing —
the `-Dsolr.test.image` override, the tested-version matrix
(8.11, 9.4, 9.9, 9.10, 10), and the Solr 10 caveats (mbeans removed →
null cache/handler stats; SolrJ 10.x not yet on Maven Central). Ported
from AGENTS.md so it lives in the developer docs, not just the agent guide.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>
epugh
approved these changes
Jun 15, 2026
epugh
left a comment
Contributor
There was a problem hiding this comment.
LGTM, I'll close my oroginal one and commit this.
3 tasks
This was referenced Jul 16, 2026
shahzadarain
pushed a commit
to shahzadarain/solr-mcp
that referenced
this pull request
Jul 31, 2026
…#144 conflicts) Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com> Conflicts: README.md: kept main's slimmed flat tools table (PR apache#151) and added the index-markdown-documents row in its concise style src/main/java/org/apache/solr/mcp/server/indexing/documentcreator/IndexingDocumentCreator.java: kept both the markdownDocumentCreator field and main's constructor Javadoc, extended with the markdownDocumentCreator @param
adityamparikh
added a commit
to adityamparikh/solr-mcp
that referenced
this pull request
Jul 31, 2026
…guides Five client guides linked to https://github.com/apache/solr-mcp#running-the-server, an anchor for a README section that no longer exists (removed by the apache#151 slim-down). Absolute self-links also dodge relative-link checkers, which is how this survived the sweep. - claude-desktop.md: point the built-JAR reference at the README's Quick start section via a relative link - claude-code/cursor/vs-code/jetbrains: inline the HTTP-mode start command instead of linking (the current README has no HTTP-mode startup section to link to) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>
adityamparikh
added a commit
to adityamparikh/solr-mcp
that referenced
this pull request
Aug 1, 2026
The README merged in apache#151 links into docs/site/content/pages/mcp/, a tree that only exists on the unmerged apache#143 branch, so every documentation link on main 404s. Restore the linked content from the apache#143 branch into locations that exist on main today, adapted for plain GitHub rendering (Pelican frontmatter converted to headings, site-absolute links repointed): - per-client setup guides (Claude Desktop, Claude Code, VS Code/Copilot, Cursor, JetBrains, MCP Inspector) under docs/clients/ - observability guide at docs/observability.md and repoint the README links there; the Quick start link now targets the README's own section. Also fix three pre-existing broken links found by a repo-wide sweep: - docs/security/http.md and docs/security/stdio.md referenced ../specs/graalvm-native-image.md, which moved to dev-docs/ - docs/security/keycloak.md TOC listed a 'User Federation (LDAP/AD)' section that does not exist This does not preempt the apache#143 discussion about where website source should live; when that lands these files can move wherever dev@ decides. Fixes apache#168 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>
adityamparikh
added a commit
to adityamparikh/solr-mcp
that referenced
this pull request
Aug 1, 2026
…guides Five client guides linked to https://github.com/apache/solr-mcp#running-the-server, an anchor for a README section that no longer exists (removed by the apache#151 slim-down). Absolute self-links also dodge relative-link checkers, which is how this survived the sweep. - claude-desktop.md: point the built-JAR reference at the README's Quick start section via a relative link - claude-code/cursor/vs-code/jetbrains: inline the HTTP-mode start command instead of linking (the current README has no HTTP-mode startup section to link to) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Signed-off-by: adityamparikh <aditya.m.parikh@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Builds on #147 (@epugh's 1.0.0 docs polish — its commits are included here) and finishes the job from the review of that PR. Refocuses the README on end-user "how to use" content, moves developer/operational detail into the docs that own it, ports the unique end-user content from #95, and fixes the factual errors found in #147 — which also exist in the website source pages, so they flow into #143 too (separately).
README.md — now a lean "how to use" guide
docs/site/content/pages/mcp/, from docs(site): host MCP documentation content in-repo, deployed to solr-site at build time #143), security todocs/security/, observability to the site page, and build/Docker/native/SBOM todev-docs/.README.md: −537 / +~50 lines.Fixes the factual errors from the #147 review
SECURITY_ENABLED→HTTP_SECURITY_ENABLED(the real toggle ishttp.security.enabled=${HTTP_SECURITY_ENABLED:true},@ConditionalOnProperty(... matchIfMissing=true)— secured by default, notfalse).solr-mcp:latest-nativereference (no such tag) — the canonical native run/config now lives in DEPLOYMENT.md with:latest-native-stdio.ghcr.io/apache/solr-mcpis 404/403).CONTRIBUTING.md
dev-docs/
DEPLOYMENT.md: new "Image variants — why three images" section (the matrix, run commands, why Jib builds the JVM image and Paketo the native variants and why Paketo's JVM image breaks STDIO, native Claude Desktop config) — relocated out of the README.DEVELOPMENT.md: the consumer-facing SBOM content (where it ships, fetch via the actuator endpoint, scan with Trivy/Grype) now lives next to "Generating the SBOM locally".Companion change in #143
The same fixes apply to the website source pages and are pushed to #143:
clients/jetbrains.md(.junie/mcp/mcp.json),clients/vs-code.md(type: http),security.md(HTTP_SECURITY_ENABLED, secured by default).The README's
docs/site/content/pages/mcp/*links resolve onmainonly after #143 lands (those pages live there). Recommended merge order: #143 → this PR. #143's own description also asks to land first.Relationship to #147 / #95
🤖 Generated with Claude Code