docs(intro): rewrite overview — ground every claim in concrete examples#2326
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughThis PR rewrites docs/core/introduction.mdx: it repositions Wren AI as an “open context layer,” replaces prior narratives, and adds sections for What Wren AI provides, open-core components, correctness via ChangesIntroduction content rewrite
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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.
🧹 Nitpick comments (1)
docs/core/introduction.mdx (1)
3-3: ⚡ Quick winRemove the legacy tagline to keep the opening crisp.
Line 3 is a long, jargon-heavy sentence that dilutes the stronger concise framing introduced at Line 7. Dropping it will make the lede align with the rewrite goal and reduce repetition.
🤖 Prompt for 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. In `@docs/core/introduction.mdx` at line 3, Remove the legacy tagline sentence "Turn any AI Agents into world-class data analysts through the open context layer that gives AI agents grounded, governed memory, context, and SQL across 20+ data sources, that helps you build GenBI, dashboards, and agentic analytics." from docs/core/introduction.mdx so the opening is shorter and aligns with the new concise lede; simply delete that full line/paragraph (it is the long jargon-heavy sentence at the top) and ensure surrounding whitespace/formatting remains correct.
🤖 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.
Nitpick comments:
In `@docs/core/introduction.mdx`:
- Line 3: Remove the legacy tagline sentence "Turn any AI Agents into
world-class data analysts through the open context layer that gives AI agents
grounded, governed memory, context, and SQL across 20+ data sources, that helps
you build GenBI, dashboards, and agentic analytics." from
docs/core/introduction.mdx so the opening is shorter and aligns with the new
concise lede; simply delete that full line/paragraph (it is the long
jargon-heavy sentence at the top) and ensure surrounding whitespace/formatting
remains correct.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 1201ff6f-0376-4708-bfb3-f99f8d072a1f
📒 Files selected for processing (1)
docs/core/introduction.mdx
785eebf to
c9ee479
Compare
Senior DE + teammate review both flagged the overview and the correctness concept page for "too much telling, not enough showing" and "feels like marketing instead of dev docs". This rewrite addresses both. ## Overview (docs/core/introduction.mdx) - Lead with the tagline, then a tight "Why Wren AI exists" using the existing concrete examples (status = 4, loyalty_v3, Project Lighthouse, MAU exclusions) and closing on the open-source line about flat files that outlive your tools - Rebuild "What Wren AI provides" with the 5-layer table still framing the problem, then a concrete MDL YAML snippet for the `customers` model, a `wren --sql` shell session showing output, and a 5-step explanation of what runs behind the scenes - Keep the verbose "What is in the open core" bullets but move the MDL deep-link from the bullet into the provides lead-in - Replace "Where this is going" 3 bullets with one sentence + link to GitHub Discussions - Replace "Learn more" with "Dig deeper" — 6 question-titled Concept links so every concept page has an overview entry point - Delete "Correctness is a system" section, "How you use it" section, "When to use Wren AI" section, "How it fits your stack" section — redundant with sidebar / start-here / per-page links, and the marketing density was the main reviewer complaint ## Correctness concept (docs/core/concepts/correctness.md) - Replace the abstract "six pillars" framing with a simplified architecture walk that directly answers the page title's question - New mermaid showing 5 layers (Skills → CLI → MDL/Memory → Plan/Validate → Data) instead of the full 7-component diagram - For each layer, name the specific failure mode it blocks (MDL blocks invented tables, memory blocks schema dumps, dry-plan blocks silent prod errors, skills block skipped steps) - End with a concrete 6-step agent loop trace (recall → fetch → write → dry-plan → execute → store) - Link to the full architecture reference for depth Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
c9ee479 to
75913b6
Compare
Summary
Two independent reviews (Senior DE + teammate) converged on the same critique of the overview: too much telling, not enough showing. Specifically:
The fix is not to strip framing — overview pages need a 30-second pitch. The fix is to anchor every claim in a snippet a reader can verify: an MDL YAML file, a CLI shell session, a
dry-planexpansion.What changed
Net length is roughly the same (~140 → ~130 lines), but signal density is up significantly — every claim now has a snippet next to it.
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit