docs: rework both front doors — README and the book introduction - #527
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
…e current front door Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
|
Warning Review limit reached
Next review available in: 14 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughREADME and introduction documentation are refreshed with updated project positioning, architecture and implementation descriptions, thesis-gate language, navigation links, pre-release status, and development-process details. ChangesDocumentation refresh
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Line 285: Update the repository-layout fenced code block in README.md to
specify the text language identifier, changing the opening fence to use text
while preserving the ASCII tree contents.
- Around line 221-229: Update the README quick-start example around the OTLP
query to align ingest and query tenancy: configure the example's ingest-side
service.name as checkout, or explicitly state that X-Ourios-Tenant must match
the service.name used during ingestion, while preserving the existing query
example.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: b30361b4-f47f-41c6-b358-70475fe3dbe5
📒 Files selected for processing (2)
README.mddocs/introduction.md
There was a problem hiding this comment.
Pull request overview
Reworks the project’s two primary entry points (root README.md and the mdBook docs/introduction.md) to reflect the current implementation status, updated benchmarks framing, and the current “front door” navigation for new readers.
Changes:
- Refreshes README status/architecture description, adds quickstart instructions, and documents the MCP surface.
- Updates the README’s “What is implemented” and “How it measures” sections to align with current RFCs and benchmark documentation.
- Replaces the mdBook introduction’s outdated “greenfield” text with an updated thesis/status blurb and a navigational map of the book.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| README.md | Updates badges, status/architecture narrative, implementation/benchmarks summary, adds quickstart + MCP + development-process sections. |
| docs/introduction.md | Updates the book landing page to reflect current project status and adds organized navigation links into the book. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…, rfcs link Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
ec6bb1f to
1bd8854
Compare
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
What
Both front doors aligned with the current state of the project — the maintainer's "a lot of water has run into the lake" request. Two commits, docs only.
README.mdrework. Stale claims fixed: "no packaged release yet" (v0.2.x signed releases + images + Helm exist), "Parquet on local filesystem" (all seams on S3 per RFC 0013/0019), "served querier pending" (RFC 0016 green), WAL checkpointing "still open" (RFC 0008 accepted), "ten RFCs" (33). Benchmark section rewritten with channel-qualified language cross-checked againstdocs/benchmarks.md: B1/B2 pass on the spec-described baseline, A1 framed as the RFC 0011 diagnostic, and the RFC 0031 comparative program summarized honestly (frozen storage-bytes must-wins ~77×/~22×, needle latency ~75 ms vs 23–24 s, window browses a documented storage-bytes loss, gates asserting on dispatch). New sections: quick start (verified against the clap CLI and the quickstart guide), the MCP surface (tool/resource names verified in source), and the development process incl. the honest fully-AI-assisted-under-human-review stance. Badges: added OpenSSF Best Practices + a?filter=v*Release badge (the unfiltered form shows the corpus tag), docs badge now points at the live Pages site; all kept badges verified live. DSL example verified against the RFC 0002 test corpus; every relative link script-checked.docs/introduction.md. The published book's landing page still said "Greenfield. No code has been written yet." Replaced with a short page: the §2 thesis wording, honest pre-release status pointing at Benchmarks, and a map of all five book parts with working links (including a real caught bug: mdbook rendersrfcs/README.mdasrfcs/index.htmlwithout rewriting inline links — the link is written against the rendered form; all 14 internal links verified againstbook/).Checks run
mdbook buildclean (re-run after rebasing onto today's main); rendered introduction page eyeballed; link targets script-verified. Docs only, no Rust touched.🤖 Generated with Claude Code
https://claude.ai/code/session_01WQY9wfrfRggqSpMLH8Xj3Y
Summary by CodeRabbit