Conversation
|
@shinokada is attempting to deploy a commit to the Themesberg Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughThis PR removes "Key Props" subsections and CompoAttributesViewer blocks from many component docs, replaces Seealso component invocations with explicit Markdown links, adjusts a few relatedLinks arrays, adds small "See also" headers in some pages, and bumps devDependency svelte-doc-llm to ^0.6.0. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes
Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (5)
🚧 Files skipped from review as they are similar to previous changes (1)
🧰 Additional context used🧠 Learnings (1)📚 Learning: 2025-08-31T12:56:05.074ZApplied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
🔇 Additional comments (7)
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: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (21)
src/routes/docs/components/alert.md(0 hunks)src/routes/docs/components/avatar.md(0 hunks)src/routes/docs/components/badge.md(0 hunks)src/routes/docs/components/banner.md(0 hunks)src/routes/docs/components/bottom-navigation.md(0 hunks)src/routes/docs/components/breadcrumb.md(0 hunks)src/routes/docs/components/button-group.md(0 hunks)src/routes/docs/components/buttons.md(0 hunks)src/routes/docs/components/card.md(0 hunks)src/routes/docs/components/carousel.md(0 hunks)static/llm/components/alert.md(0 hunks)static/llm/components/avatar.md(0 hunks)static/llm/components/badge.md(0 hunks)static/llm/components/banner.md(0 hunks)static/llm/components/bottom-navigation.md(0 hunks)static/llm/components/breadcrumb.md(0 hunks)static/llm/components/button-group.md(0 hunks)static/llm/components/buttons.md(0 hunks)static/llm/components/card.md(0 hunks)static/llm/components/carousel.md(0 hunks)static/llm/context-full.txt(1 hunks)
💤 Files with no reviewable changes (20)
- static/llm/components/breadcrumb.md
- static/llm/components/buttons.md
- static/llm/components/carousel.md
- src/routes/docs/components/buttons.md
- static/llm/components/bottom-navigation.md
- static/llm/components/alert.md
- static/llm/components/avatar.md
- static/llm/components/banner.md
- src/routes/docs/components/alert.md
- src/routes/docs/components/card.md
- static/llm/components/button-group.md
- src/routes/docs/components/banner.md
- src/routes/docs/components/carousel.md
- src/routes/docs/components/breadcrumb.md
- static/llm/components/card.md
- src/routes/docs/components/avatar.md
- src/routes/docs/components/badge.md
- src/routes/docs/components/button-group.md
- static/llm/components/badge.md
- src/routes/docs/components/bottom-navigation.md
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2025-03-20T12:47:33.307Z
Learnt from: superstes
Repo: themesberg/flowbite-svelte PR: 0
File: :0-0
Timestamp: 2025-03-20T12:47:33.307Z
Learning: In the Flowbite Svelte library's composite form components like MultiInput, when implementing focus styles, ensure both the container (with focus-within) and the inner input element (with direct focus) have dynamic styling based on the validation state. The input element needs its own focus styles that change to red when inputInvalid=true.
Applied to files:
static/llm/context-full.txt
📚 Learning: 2025-08-31T12:56:05.074Z
Learnt from: jjagielka
Repo: themesberg/flowbite-svelte PR: 1745
File: src/lib/types.ts:677-686
Timestamp: 2025-08-31T12:56:05.074Z
Learning: HTMLInputAttributes includes a `group` property that can be used for grouping form controls like checkboxes and radio buttons, so interfaces extending HTMLInputAttributes don't need to explicitly declare `group` unless they want to override its type.
Applied to files:
static/llm/context-full.txt
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Playwright and Lint
🔇 Additional comments (1)
static/llm/context-full.txt (1)
2-2: Verify the timestamp regeneration is correct.The timestamp on line 2 indicates the file was regenerated on 2025-11-21T13:12:21.604Z. Please confirm this reflects the actual content changes and that the file generation process executed correctly.
| # Flowbite Svelte Documentation - Full Context | ||
| # Generated on 2025-11-21T12:26:03.543Z | ||
| # Generated on 2025-11-21T13:12:21.604Z | ||
| # This file contains all documentation concatenated in logical order | ||
|
|
||
|
|
There was a problem hiding this comment.
🧩 Analysis chain
PR description is required to understand the intent and scope of documentation removal.
The PR body is empty—it contains only the contribution template with no actual description of changes, rationale, breaking changes, or verification steps. Given that this PR removes extensive documentation from an LLM context file, clarification is essential before merging.
Please provide:
- Rationale: Why is removing Key Props documentation from the LLM context file appropriate? Is this documentation duplicated elsewhere?
- Impact: How will this affect users or tools relying on this context file (e.g., AI-assisted documentation, code generation)?
- Scope confirmation: Verify that all removals are intentional and not accidental deletions.
- Breaking changes: Is this a breaking change for any downstream consumers of this file?
I'll verify the concerns raised in this review comment by examining the file and understanding the actual changes.
🏁 Script executed:
#!/bin/bash
# Check if the file exists and show its content
if [ -f "static/llm/context-full.txt" ]; then
echo "=== File exists ==="
wc -l static/llm/context-full.txt
echo ""
echo "=== First 20 lines ==="
head -20 static/llm/context-full.txt
else
echo "File does not exist at static/llm/context-full.txt"
fiLength of output: 996
🏁 Script executed:
#!/bin/bash
# Search for references to the LLM context file in the codebase
echo "=== References to context-full.txt ==="
rg -n "context-full" --type=ts --type=js --type=json --type=md 2>/dev/null | head -20
echo ""
echo "=== Files in static/llm directory ==="
find static/llm -type f 2>/dev/null | head -20Length of output: 1885
🏁 Script executed:
#!/bin/bash
# Check git history to understand what was changed
echo "=== Recent commits affecting context-full.txt ==="
git log --oneline -10 -- static/llm/context-full.txt 2>/dev/null || echo "Git history not available"
echo ""
echo "=== Current git status ==="
git diff HEAD -- static/llm/context-full.txt 2>/dev/null | head -100 || echo "Git diff not available"Length of output: 309
I need to check the PR metadata and understand the scope of changes more clearly.
🏁 Script executed:
#!/bin/bash
# Get PR information using GitHub CLI
echo "=== Checking PR metadata ==="
gh pr view 2>/dev/null || echo "GitHub CLI not available or not in PR context"
echo ""
echo "=== Show detailed git diff for context-full.txt (first 200 lines) ==="
git diff --no-color HEAD~1 HEAD -- static/llm/context-full.txt 2>/dev/null | head -200Length of output: 6672
🏁 Script executed:
#!/bin/bash
# Check if there's a commit message with more detail
echo "=== Commit message details ==="
git show --format=fuller 32ab8194 2>/dev/null | head -50
echo ""
echo "=== Look for any documentation about the removal ==="
rg -i "key props|remove.*props|remove.*documentation" --type=md --type=txt static/llm 2>/dev/null | head -20Length of output: 2361
🏁 Script executed:
#!/bin/bash
# Check llm.config.js to understand how context-full.txt is generated
echo "=== LLM Config File (shows generation order) ==="
cat llm.config.js 2>/dev/null | head -60Length of output: 1138
Provide context and rationale for Key Props documentation removal before merging.
Verification confirms extensive removal of "Key Props" documentation from ~15+ components (Alert, Avatar, Badge, Banner, BottomNav, Breadcrumb, ButtonGroup, Button, Card, Carousel, and more) in the LLM context file. The commit message provides only the title "docs: remove Key Props" with no explanation of intent or impact.
Since this file is actively used for LLM context integration (documented in src/routes/docs/pages/ai-integration.md), removal of these convenience property summaries without justification poses a risk to users and tools relying on this context.
Please provide:
- Rationale: Why are "Key Props" summaries being removed? Are they redundant with
<CompoAttributesViewer />? - Impact: How does this affect LLM context quality for code generation and documentation discovery?
- Scope verification: Confirm all removals are intentional across all affected components.
- Breaking changes: Document if this constitutes a breaking change for downstream consumers of the context file.
There was a problem hiding this comment.
Actionable comments posted: 3
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
static/llm/forms/number-input.md (1)
178-181: Remove stray>character.Line 180 contains a leftover
>character that appears to be an incomplete markdown code fence. This should be removed.</form> - >
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (33)
package.json(1 hunks)static/llm/components/accordion.md(1 hunks)static/llm/components/alert.md(1 hunks)static/llm/components/avatar.md(1 hunks)static/llm/components/badge.md(1 hunks)static/llm/components/banner.md(1 hunks)static/llm/components/bottom-navigation.md(1 hunks)static/llm/components/breadcrumb.md(1 hunks)static/llm/components/button-group.md(1 hunks)static/llm/components/buttons.md(1 hunks)static/llm/components/card.md(1 hunks)static/llm/components/carousel.md(1 hunks)static/llm/components/list-group.md(1 hunks)static/llm/components/progress.md(1 hunks)static/llm/components/tooltip.md(1 hunks)static/llm/context-full.txt(32 hunks)static/llm/extend/button-toggle.md(1 hunks)static/llm/extend/clipboard-manager.md(1 hunks)static/llm/extend/command-palette.md(1 hunks)static/llm/extend/kanban-board.md(1 hunks)static/llm/extend/progressradial.md(1 hunks)static/llm/extend/scroll-spy.md(1 hunks)static/llm/extend/split-pane.md(1 hunks)static/llm/extend/tags.md(1 hunks)static/llm/extend/tour.md(1 hunks)static/llm/extend/virtual-list.md(1 hunks)static/llm/extend/virtual-masonry.md(1 hunks)static/llm/forms/floating-label.md(1 hunks)static/llm/forms/input-field.md(1 hunks)static/llm/forms/number-input.md(1 hunks)static/llm/forms/phone-input.md(1 hunks)static/llm/typography/blockquote.md(1 hunks)static/llm/utilities/close-button.md(1 hunks)
✅ Files skipped from review due to trivial changes (8)
- static/llm/context-full.txt
- static/llm/extend/command-palette.md
- static/llm/extend/split-pane.md
- static/llm/extend/scroll-spy.md
- static/llm/extend/tour.md
- static/llm/components/accordion.md
- static/llm/extend/kanban-board.md
- static/llm/extend/virtual-list.md
🚧 Files skipped from review as they are similar to previous changes (11)
- static/llm/components/button-group.md
- static/llm/extend/button-toggle.md
- static/llm/components/progress.md
- static/llm/components/bottom-navigation.md
- static/llm/components/avatar.md
- static/llm/extend/progressradial.md
- static/llm/forms/floating-label.md
- static/llm/components/badge.md
- static/llm/components/buttons.md
- static/llm/forms/phone-input.md
- static/llm/typography/blockquote.md
🧰 Additional context used
🧠 Learnings (6)
📚 Learning: 2025-03-20T12:47:33.307Z
Learnt from: superstes
Repo: themesberg/flowbite-svelte PR: 0
File: :0-0
Timestamp: 2025-03-20T12:47:33.307Z
Learning: In the Flowbite Svelte library's composite form components like MultiInput, when implementing focus styles, ensure both the container (with focus-within) and the inner input element (with direct focus) have dynamic styling based on the validation state. The input element needs its own focus styles that change to red when inputInvalid=true.
Applied to files:
static/llm/forms/input-field.mdstatic/llm/forms/number-input.mdstatic/llm/extend/tags.md
📚 Learning: 2025-03-20T11:11:53.953Z
Learnt from: superstes
Repo: themesberg/flowbite-svelte PR: 0
File: :0-0
Timestamp: 2025-03-20T11:11:53.953Z
Learning: In the Flowbite Svelte library's MultiInput component, the 'inputInvalid' property should be exported and passed to the Wrapper component to properly style the component's ring with red color when validation fails, similar to how the Input component handles validation states.
Applied to files:
static/llm/forms/input-field.mdstatic/llm/forms/number-input.mdstatic/llm/extend/tags.md
📚 Learning: 2025-03-20T11:52:09.106Z
Learnt from: superstes
Repo: themesberg/flowbite-svelte PR: 0
File: :0-0
Timestamp: 2025-03-20T11:52:09.106Z
Learning: In the Flowbite Svelte library's form components like MultiInput, when handling focus states for validation, avoid hardcoding focus-within classes like 'focus-within:border-primary-500'. Instead, create a dynamic mapping object (e.g., focusWithinClasses) that changes the focus ring color based on the current color state, especially when inputInvalid=true.
Applied to files:
static/llm/forms/input-field.mdstatic/llm/forms/number-input.mdstatic/llm/extend/tags.md
📚 Learning: 2025-03-20T11:11:53.952Z
Learnt from: superstes
Repo: themesberg/flowbite-svelte PR: 0
File: :0-0
Timestamp: 2025-03-20T11:11:53.952Z
Learning: In the Flowbite Svelte library, when implementing components with validation states, the 'inputInvalid' property should be explicitly defined and passed to the Wrapper component. When inputInvalid=true, the color should be set to "red" to ensure the ring/border styling is applied correctly.
Applied to files:
static/llm/forms/input-field.mdstatic/llm/extend/tags.md
📚 Learning: 2025-03-20T12:47:33.307Z
Learnt from: superstes
Repo: themesberg/flowbite-svelte PR: 0
File: :0-0
Timestamp: 2025-03-20T12:47:33.307Z
Learning: In the Flowbite Svelte library's composite form components like MultiInput, when implementing focus styles with focus-within, create a dedicated focusWithinClasses object with appropriate colors for each state (base, primary, green, red). Using just 'focus-within:ring-1' without color specification won't correctly apply color-specific styling when the input receives focus directly.
Applied to files:
static/llm/forms/input-field.mdstatic/llm/extend/tags.md
📚 Learning: 2024-11-22T08:37:27.996Z
Learnt from: mrh1997
Repo: themesberg/flowbite-svelte PR: 1442
File: src/lib/utils/Popper.svelte:144-145
Timestamp: 2024-11-22T08:37:27.996Z
Learning: In `src/lib/utils/Popper.svelte`, inconsistencies in event handling (e.g., using both `mouseenter`/`mouseleave` and `pointerenter`/`pointerleave`) may be intentional to minimize modifications and reduce the risk of new problems.
Applied to files:
static/llm/components/tooltip.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Playwright and Lint
🔇 Additional comments (13)
static/llm/extend/virtual-masonry.md (1)
16-19: Clean conversion of Seealso component to explicit markdown link.The change successfully replaces the Seealso component with a direct markdown link, aligning with the PR's objective. The link format is correct and the documentation structure is maintained.
static/llm/components/tooltip.md (2)
1-205: Verify completeness of changes per PR title.The PR is titled "docs: remove Key Props" and the AI summary indicates Key Props subsections and CompoAttributesViewer components are being removed across documentation files. However, the only visible changes in this file are the "See also" section updates (lines 177–179).
This file does not appear to have a distinct "Key Props" subsection or CompoAttributesViewer component visible in the provided content. Please confirm:
- Whether this file was expected to have Key Props/CompoAttributesViewer changes
- Whether the changes to this file are complete, or if sections are truncated in the provided excerpt
175-180: See also section correctly references existing component documentation.All three markdown links point to existing files in the repository. The conversion from component invocations to explicit links is clean and properly formatted.
package.json (1)
92-92: Dependency bump aligns with documentation refactor goals.The
svelte-doc-llmminor version bump from ^0.5.1 to ^0.6.0 likely enables the Key Props removal and Seealso component replacement patterns implemented throughout the PR's documentation changes.static/llm/components/list-group.md (1)
175-182: See also section properly converted to explicit markdown links.The transition from Seealso component to direct markdown links is clean and consistent. All links follow the established pattern with proper URLs.
static/llm/components/card.md (1)
602-609: See also section properly updated with explicit links.The conversion to markdown links is consistent and properly formatted with all external URLs using the flowbite-svelte.com/llm pattern.
static/llm/utilities/close-button.md (1)
29-37: See also section properly converted to explicit markdown links.All links follow the consistent absolute URL pattern and are properly formatted.
static/llm/components/carousel.md (1)
452-458: See also section properly converted with consistent link formatting.All links follow the established absolute URL pattern and maintain proper spacing.
static/llm/components/alert.md (1)
368-374: See also section properly converted to explicit markdown links.All links follow the consistent absolute URL pattern established across the documentation.
static/llm/components/banner.md (1)
315-321: See also section properly converted with consistent formatting.All links follow the established absolute URL pattern.
static/llm/extend/clipboard-manager.md (1)
18-18: This review comment is incorrect.The link uses an absolute URL to
flowbite-svelte.combecause that is the established pattern throughout the entire codebase. A search reveals 178 instances offlowbite-svelte.com/llm/URLs instatic/llm/, all consistently used in "See also" sections across multiple files (close-button.md, blockquote.md, and others). The code change in clipboard-manager.md follows this convention. Converting to relative paths would break consistency, not improve it.Likely an incorrect or invalid review comment.
static/llm/forms/input-field.md (1)
664-668: Verify scope of CompoAttributesViewer removal.According to the AI summary, this PR removes CompoAttributesViewer components as part of the Key Props subsections cleanup. However, line 668 still contains
<CompoAttributesViewer {components}/>. Clarify whether:
- CompoAttributesViewer removal was incomplete, or
- The component should remain and the AI summary overstated the scope.
Verify the PR scope and confirm whether line 668 should be removed or retained as part of this change.
static/llm/forms/number-input.md (1)
210-216: Disregard this review comment—the "[Input field]" reference is intentional and consistent with documentation patterns.The "See also" section in number-input.md contains the same "[Input field]" link found in floating-label.md and input-field.md itself. This is a standardized cross-reference pattern used consistently across form documentation files, not a mislabeling. Even input-field.md references itself as "[Input field]" in its own "See also" section, confirming this is the established convention. Changing it to "[Number Input]" would create a redundant self-reference and break consistency with related documentation.
Likely an incorrect or invalid review comment.
| ## See also | ||
|
|
||
| <Seealso links={relatedLinks} /> | ||
| - [Navbar](https://flowbite-svelte.com/llm/components/navbar.md) | ||
| - [Sidebar](https://flowbite-svelte.com/llm/components/sidebar.md) | ||
| - [Pagination](https://flowbite-svelte.com/llm/components/pagination.md) | ||
| - [Heading](docs/typography/heading.md) | ||
|
|
There was a problem hiding this comment.
URL format inconsistency in See also section.
Line 220 uses a relative path docs/typography/heading.md while lines 217–219 use absolute URLs. For consistency and reliability across the documentation, use the full absolute URL pattern like other See also sections:
- [Heading](docs/typography/heading.md)
+ [Heading](https://flowbite-svelte.com/llm/typography/heading.md)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ## See also | |
| <Seealso links={relatedLinks} /> | |
| - [Navbar](https://flowbite-svelte.com/llm/components/navbar.md) | |
| - [Sidebar](https://flowbite-svelte.com/llm/components/sidebar.md) | |
| - [Pagination](https://flowbite-svelte.com/llm/components/pagination.md) | |
| - [Heading](docs/typography/heading.md) | |
| ## See also | |
| - [Navbar](https://flowbite-svelte.com/llm/components/navbar.md) | |
| - [Sidebar](https://flowbite-svelte.com/llm/components/sidebar.md) | |
| - [Pagination](https://flowbite-svelte.com/llm/components/pagination.md) | |
| - [Heading](https://flowbite-svelte.com/llm/typography/heading.md) | |
🤖 Prompt for AI Agents
In static/llm/components/breadcrumb.md around lines 215 to 221, the "See also"
entry for Heading uses a relative path ("docs/typography/heading.md") while the
other links use absolute URLs; update that line to the full absolute URL
consistent with the other entries (e.g. replace the relative path with the
site's absolute URL for the heading page such as
https://flowbite-svelte.com/docs/typography/heading.md or the correct absolute
site path) so all links follow the same absolute URL pattern.
| ## See also | ||
|
|
||
| <Seealso links={relatedLinks} /> | ||
| - [Input field](https://flowbite-svelte.com/llm/forms/input-field.md) | ||
| - [Floating label](https://flowbite-svelte.com/llm/forms/floating-label.md) | ||
| - [Tags](https://flowbite-svelte.com/llm/extend/tags.md) |
There was a problem hiding this comment.
Remove self-referential link from See also section.
The See also section includes "Tags" pointing to the current page. This is unconventional; See also sections typically reference related documentation, not the current page itself.
## See also
- [Input field](https://flowbite-svelte.com/llm/forms/input-field.md)
- [Floating label](https://flowbite-svelte.com/llm/forms/floating-label.md)
- - [Tags](https://flowbite-svelte.com/llm/extend/tags.md)🤖 Prompt for AI Agents
In static/llm/extend/tags.md around lines 104 to 108, the "See also" list
contains a self-referential entry linking to the current page; remove the "-
[Tags](https://flowbite-svelte.com/llm/extend/tags.md)" bullet (or replace it
with a different related doc if intended) so the See also section only links to
other pages.
| ## See also | ||
|
|
||
| <Seealso links={relatedLinks} /> | ||
| - [Input field](https://flowbite-svelte.com/llm/forms/input-field.md) | ||
| - [Floating label](https://flowbite-svelte.com/llm/forms/floating-label.md) | ||
| - [Tags](https://flowbite-svelte.com/llm/extend/tags.md) |
There was a problem hiding this comment.
Remove self-referential link from See also section.
The See also section includes "Input field" linking to the current page (input-field.md). This is unconventional; See also sections should reference related but distinct documentation pages, not the current page itself.
## See also
- - [Input field](https://flowbite-svelte.com/llm/forms/input-field.md)
- [Floating label](https://flowbite-svelte.com/llm/forms/floating-label.md)
- [Tags](https://flowbite-svelte.com/llm/extend/tags.md)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ## See also | |
| <Seealso links={relatedLinks} /> | |
| - [Input field](https://flowbite-svelte.com/llm/forms/input-field.md) | |
| - [Floating label](https://flowbite-svelte.com/llm/forms/floating-label.md) | |
| - [Tags](https://flowbite-svelte.com/llm/extend/tags.md) | |
| ## See also | |
| - [Floating label](https://flowbite-svelte.com/llm/forms/floating-label.md) | |
| - [Tags](https://flowbite-svelte.com/llm/extend/tags.md) |
🤖 Prompt for AI Agents
In static/llm/forms/input-field.md around lines 657 to 661, the "See also" list
includes a self-referential link to "Input field" (the current page); remove
that bullet (or replace it with a different related page) so the See also
section only links to distinct, related documentation pages (keep the "Floating
label" and "Tags" entries).
🔗 Related issue (optional)
Closes #
📑 Description
🔍 PR Type
🚦 PR Status
✅ Checklist
pnpm check && pnpm test:e2eand all tests passmainbranch (not the published npm version)🧪 Screenshots / Demos (optional)
ℹ️ Additional Information
Summary by CodeRabbit
Documentation
Chores
✏️ Tip: You can customize this high-level summary in your review settings.