Conversation
WalkthroughReplaced manual session fetching with authClient.useSession-derived state. Restructured the Social Proof section to a responsive horizontal layout, adjusted logo sizes and separators, tweaked alignment classes, and added text-balance to testimonial styling. Updated div structure accordingly. No exported API changes; default export remains unchanged. Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant H as Home(Page)
participant A as authClient
participant UI as UI
U->>H: Navigate to page
H->>A: useSession()
A-->>H: { session | null }
H->>H: hasSession = Boolean(session?.user)
alt hasSession
H->>UI: Render authenticated view
else no session
H->>UI: Render unauthenticated view
end
note over H,UI: Social Proof and Testimonials rendered with updated layout and styling
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
apps/saru/app/page.tsx (2)
153-153: Mark the visual separator as decorative for screen readers.
Addaria-hiddenso it isn’t announced as empty content.- <span className="h-6 md:h-8 w-px bg-border" /> + <span aria-hidden="true" className="h-6 md:h-8 w-px bg-border" />
155-170: Use native anchors for external URLs (or setprefetch={false}) and add aria-labels.
next/linkoffers no prefetch benefit for external sites; anchors are simpler and avoid subtle warnings. Add anaria-labelfor better SR context.- <Link - href="https://x.com/leerob" - target="_blank" - rel="noopener noreferrer" - className="group flex items-center gap-1.5 rounded-full border border-border bg-background pl-2 md:pl-3 pr-3 md:pr-4 h-8 md:h-10 w-36 md:w-48 justify-center text-xs whitespace-nowrap hover:bg-accent hover:text-accent-foreground transition-colors" - > + <a + href="https://x.com/leerob" + target="_blank" + rel="noopener noreferrer" + aria-label="Lee Robinson (@leerob) — VP DX, Cursor" + className="group flex items-center gap-1.5 rounded-full border border-border bg-background pl-2 md:pl-3 pr-3 md:pr-4 h-8 md:h-10 w-36 md:w-48 justify-center text-xs whitespace-nowrap hover:bg-accent hover:text-accent-foreground transition-colors" + > <Image src="/images/leerob.jpg" alt="Lee Robinson" width={24} height={24} className="size-6 rounded-full object-cover shrink-0" /> <span className="font-medium">@leerob</span> <span className="hidden md:inline text-muted-foreground/60">VP DX, Cursor</span> - </Link> + </a>- <Link - href="https://x.com/dps" - target="_blank" - rel="noopener noreferrer" - className="group flex items-center gap-1.5 rounded-full border border-border bg-background pl-2 md:pl-3 pr-3 md:pr-4 h-8 md:h-10 w-36 md:w-48 justify-center text-xs whitespace-nowrap hover:bg-accent hover:text-accent-foreground transition-colors" - > + <a + href="https://x.com/dps" + target="_blank" + rel="noopener noreferrer" + aria-label="David Singleton (@dps) — ex-CTO, Stripe" + className="group flex items-center gap-1.5 rounded-full border border-border bg-background pl-2 md:pl-3 pr-3 md:pr-4 h-8 md:h-10 w-36 md:w-48 justify-center text-xs whitespace-nowrap hover:bg-accent hover:text-accent-foreground transition-colors" + > <Image src="/images/dps.jpg" alt="David Singleton" width={24} height={24} className="size-6 rounded-full object-cover shrink-0" /> <span className="font-medium">@dps</span> <span className="hidden md:inline text-muted-foreground/60">ex-CTO, Stripe</span> - </Link> + </a>Also applies to: 172-187
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
apps/saru/app/page.tsx(5 hunks)
🔇 Additional comments (4)
apps/saru/app/page.tsx (4)
128-130: Compact “Used by” label looks good.
Clear, unobtrusive, and reads well next to the logos.
141-144: Logo size tweak is balanced.
The reduced Vercel dimensions align better with adjacent marks.
234-234: Nice touch withtext-balance.
Improves ragging for multi-line quotes.
24-25: I'll inspect the hook’s signature andauthClientimport/definition to confirm which loading flag is exposed.
1 participant
Summary
authClient.useSessiontext-balanceTesting
pnpm --filter @saru/app lintpnpm test(fails: No tests found)https://chatgpt.com/codex/tasks/task_e_68b82c06d39c8325abb207de90f87da4
Summary by CodeRabbit