Built for code review
Review your code FASTER
Leave feedback exactly where it matters. Inline threads, live replies, and context that never gets lost in a giant PR description.
hooks/use-auth.ts+42
1export function useAuth() {
2 const session = useSession();
3
4 return { user, signOut };
5}
NX
Nyx Botline 4 · 2m ago
Do we still need this if the layout above already sets the client boundary? Otherwise looks good.
Reply… 3
api/reviews.ts2 threads
1export async function fetchReviews(pr: number) {
2 const res = await get("/reviews", { pr });
3 return res.json();
4}
NXResolved
Nyx Botline 12 · 1h ago
Removed the redundant guard good catch. The fallback in
ReviewProvider already covers it. Reply…
review-thread.tsx
1export default function Thread({ comments }) {
2 const last = useLiveReply(comments);
3 return <List items={comments} />
4}
NX
Nyx Botreply · 5m ago
The virtualization is weak, fix it by using the function present in lib
Reply… 1
NX
Nyx Bothooks/use-auth.ts:4 · 8m ago
The error path here returns a raw
401— the UI expects a normalized AuthError shape so the toast can pick the right message. Reply… 2
api/diff.ts+12/-3
1export function applyDiff(patch) {
2 const hunks = parsePatch(patch);
3 return hunks.map(render);
4}
NX
Nyx Botreview-thread.tsx:3 · just now
Verified — live replies propagate through the thread cache in
<List /> without a refetch. Approving. LGTM. Reply… 5