feat: add ChangelogCard component and update homepage:#242
Conversation
- Introduced ChangelogCard component to display recent changelog entries. - Updated homepage to include ChangelogCard, enhancing visibility of recent updates. - Added structured changelog entries for versions 0.1.90, 0.1.89, and 0.1.88 with relevant details.
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughThe changelog page now includes releases v0.1.90 through v0.1.78. The homepage renders a client-side ChangesChangelog experience
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant Visitor
participant Home
participant ChangelogCard
participant ChangelogPage
Visitor->>Home: Open homepage
Home->>ChangelogCard: Render changelog card
Visitor->>ChangelogCard: Select READ FULL CHANGELOG
ChangelogCard->>ChangelogPage: Navigate to /changelog
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration. 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
apps/web/components/homepage/changelog-card.tsx (1)
1-1: 🚀 Performance & Scalability | 🔵 Trivial | ⚡ Quick winKeep this component as a Server Component.
Nothing in
ChangelogCardrequires client-side behavior. Remove"use client"to avoid shipping and hydrating unnecessary JavaScript;Linkand the icon can remain in a Server Component.🤖 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 `@apps/web/components/homepage/changelog-card.tsx` at line 1, Remove the "use client" directive from the ChangelogCard component so it remains a Server Component; leave its existing Link and icon usage unchanged.Source: Coding guidelines
🤖 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 `@apps/web/components/homepage/changelog-card.tsx`:
- Around line 6-10: Update the changelogEntries data used by the homepage card
to reuse the changelog page’s shared release source, preserving chronological
ordering and each entry’s version/date metadata. Remove the duplicated hardcoded
release list and ensure the card reflects the same 49 release sections as the
linked changelog page.
---
Nitpick comments:
In `@apps/web/components/homepage/changelog-card.tsx`:
- Line 1: Remove the "use client" directive from the ChangelogCard component so
it remains a Server Component; leave its existing Link and icon usage unchanged.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 9b5d3702-31b2-4d39-84ee-26bee26e931f
⛔ Files ignored due to path filters (1)
bun.lockis excluded by!**/*.lock
📒 Files selected for processing (3)
apps/web/app/(pages)/changelog/page.tsxapps/web/app/page.tsxapps/web/components/homepage/changelog-card.tsx
| const changelogEntries = [ | ||
| { type: "feat" as const, text: "Claude Opus 5 added as a new model — included in high-value models and pricing" }, | ||
| { type: "feat" as const, text: "New skills management — commands to add, remove, and list agent skills" }, | ||
| { type: "fix" as const, text: "Hallucination guard in agent runner — detects when model claims actions without tool calls" }, | ||
| { type: "feat" as const, text: "Usage event migration — new usage_event table for detailed event tracking" }, |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win
Keep the homepage changelog metadata consistent with the changelog page.
The card duplicates release data and is already inconsistent: the entries are not chronological, lack per-entry version/date metadata, and the linked page renders 49 release sections while the card claims 56. Prefer a shared changelog data source, or update the card to match the page.
Also applies to: 36-36
🤖 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 `@apps/web/components/homepage/changelog-card.tsx` around lines 6 - 10, Update
the changelogEntries data used by the homepage card to reuse the changelog
page’s shared release source, preserving chronological ordering and each entry’s
version/date metadata. Remove the duplicated hardcoded release list and ensure
the card reflects the same 49 release sections as the linked changelog page.
1 participant
Description
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
bun testpassesbun run typecheckpassesbun run lintpasses (if applicable)Checklist:
Summary by CodeRabbit