fix(pr-comments): address review feedback for chat and migrations

Author: 0x3EF8

.gitignore

@@ -48,8 +48,6 @@ next-env.d.ts
 # supabase
 supabase/*
 !supabase/migrations
-supabase/.temp/
-.temp/
 
 
 yarn.lock

app/components/Chat.tsx

@@ -1,4 +1,4 @@
-"use client";
+"use client";
 
 import { useCallback, useEffect, useMemo, useRef, useState } from "react";
 import { RealtimeChannel, User } from "@supabase/supabase-js";
@@ -1400,12 +1400,15 @@ export default function Chat({ user }: { user: User }) {
     ? "G" 
     : activeOtherUser?.email?.[0]?.toUpperCase() ?? "?";
 
-  const allMediaAttachments = messages
-    .flatMap((m) => m.attachments || [])
-    .filter(
-      (a) => a?.mimetype?.startsWith("image/") || a?.mimetype?.startsWith("video/")
-    )
-    .reverse();
+  const allMediaAttachments = useMemo(() => {
+    return messages
+      .flatMap((m) => m.attachments || [])
+      .filter(
+        (a) =>
+          a?.mimetype?.startsWith("image/") || a?.mimetype?.startsWith("video/"),
+      )
+      .reverse();
+  }, [messages]);
 
   const currentMediaIndex = mediaViewer ? allMediaAttachments.findIndex((a) => a.public_url === mediaViewer.url) : -1;
   const hasPrevMedia = currentMediaIndex > 0;

app/components/chat/Conversations.tsx

@@ -1,4 +1,4 @@
-import { User } from "@supabase/supabase-js";
+import { User } from "@supabase/supabase-js";
 import { Conversation, TypingState } from "../Chat";
 
 export default function Conversations({

app/components/chat/Messages.tsx

@@ -1,4 +1,4 @@
-"use client";
+"use client";
 
 import { User } from "@supabase/supabase-js";
 import ReactMarkdown from "react-markdown";

app/components/dashboard/Navbar.tsx

@@ -358,7 +358,7 @@ export default function DashboardLayout({
   role: string;
   children: React.ReactNode;
 }) {
-  const [collapsed, setCollapsed] = useState(true);
+  const [collapsed, setCollapsed] = useState(false);
   const [mobileHidden, setMobileHidden] = useState(false);
   const [isMobile, setIsMobile] = useState(false);
 
@@ -397,8 +397,7 @@ export default function DashboardLayout({
         setCollapsed(true);
         setMobileHidden(true);
       } else {
-        // default to collapsed on desktop too
-        setCollapsed(true);
+        setCollapsed(false);
         setMobileHidden(false);
       }
     };

app/lib/supabase/help/user.ts

@@ -1,6 +1,7 @@
+import { cache } from "react";
 import { createClient } from "../server";
 
-export async function getUserWithProfile() {
+export const getUserWithProfile = cache(async () => {
   const supabase = await createClient();
 
   const {
@@ -16,4 +17,4 @@ export async function getUserWithProfile() {
     .single();
 
   return { user, profile };
-}
+});

supabase/migrations/20260320234600_add_user_stats.sql

@@ -13,16 +13,20 @@ create table public.user_stats (
   daily_stats jsonb NOT NULL default '[]'::jsonb,
   last_fetched_at timestamp with time zone NOT NULL default now()
 );
+
 alter table public.user_stats enable row level security;
+
 /* ---- RLS Policy ----- */
 create policy "Users can insert their stats"
 on public.user_stats
 for insert
 with check (auth.uid() = user_id);
+
 create policy "Users can update their stats"
 on public.user_stats
 for update
 using (auth.uid() = user_id);
+
 create policy "Authenticated users can view stats"
 on public.user_stats
 for select

supabase/migrations/20260320234653_add_profiles.sql

@@ -5,20 +5,25 @@ create table public.profiles (
   wakatime_api_key text default null unique,
   created_at timestamp with time zone NOT NULL default now()
 );
+
 /* ---- RLS Policies ----- */
 alter table public.profiles enable row level security;
+
 create policy "Users can view their own profile"
 on public.profiles
 for select
 using (auth.uid() = id);
+
 create policy "Users can update their own profile"
 on public.profiles
 for update
 using (auth.uid() = id);
+
 create policy "Users can insert their own profile"
 on public.profiles
 for insert
 with check (auth.uid() = id);
+
 /* ---- Auto Create Profile on Signup Trigger ----- */
 create or replace function public.handle_new_user()
 returns trigger as $$
@@ -28,6 +33,7 @@ begin
   return new;
 end;
 $$ language plpgsql security definer;
+
 create trigger on_auth_user_created
 after insert on auth.users
 for each row execute procedure public.handle_new_user();

supabase/migrations/20260320234714_add_leaderboards.sql

@@ -9,7 +9,9 @@ create table public.leaderboards (
   join_code text not null unique,
   created_at timestamptz NOT NULL default now()
 );
+
 alter table public.leaderboards enable row level security;
+
 /* ---- Leaderboards Members ----- */
 create table public.leaderboard_members (
   id uuid primary key default gen_random_uuid(),
@@ -19,7 +21,9 @@ create table public.leaderboard_members (
   joined_at timestamptz NOT NULL default now(),
   unique (leaderboard_id, user_id)
 );
+
 alter table public.leaderboard_members enable row level security;
+
 /* ---- Leaderboards Members View ----- */
 create view public.leaderboard_members_view as
 select
@@ -35,32 +39,39 @@ select
 from public.leaderboard_members lm
 join auth.users u on lm.user_id = u.id
 left join public.user_stats us on lm.user_id = us.user_id;
+
 /* ---- RLS Policies ----- */
 create policy "Public leaderboards are viewable"
 on public.leaderboards
 for select
 using (is_public = true OR owner_id = auth.uid());
+
 create policy "Users can create leaderboards"
 on public.leaderboards
 for insert
 with check (auth.uid() = owner_id);
+
 create policy "Owner can update leaderboard"
 on public.leaderboards
 for update
 using (auth.uid() = owner_id);
+
 /* ---- Members Policies ----- */
 create policy "Users can see their own membership"
 on public.leaderboard_members
 for select
 using (user_id = auth.uid());
+
 create policy "Users can join leaderboard"
 on public.leaderboard_members
 for insert
 with check (auth.uid() = user_id);
+
 create policy "Members can leave leaderboard"
 on public.leaderboard_members
 for delete
 using (user_id = auth.uid());
+
 create policy "Owner can manage members"
 on public.leaderboard_members
 for delete

supabase/migrations/20260320234731_add_enforcement_checks.sql

@@ -1,12 +1,15 @@
 alter table public.leaderboards
 add constraint leaderboards_name_length
 check (length(trim(name)) between 3 and 50);
+
 alter table public.leaderboards
 add constraint leaderboards_description_length
 check (description is null or length(description) <= 150);
+
 alter table public.leaderboards
 add constraint leaderboards_join_code_format
 check (join_code ~ '^[A-Za-z0-9]{1,8}$');
+
 create policy "Owner can delete leaderboard"
 on public.leaderboards
 for delete