feat: stabilize chat realtime, dashboard insights, and account flows by 0x3EF8 · Pull Request #57 · hallofcodes/devpulse

0x3EF8 · 2026-03-26T18:52:24Z

Summary

improve chat reliability and UX: presence accuracy, read/typing signals, unread tracking, media viewer flow, and conversation handling
enhance dashboard and settings experience, including WakaTime sync and account management improvements
refresh landing and leaderboard presentation for clearer user flow and better readability
add and clean Supabase migrations for cascade delete behavior, chat presence/read tracking, realtime publication, global chat membership, and dashboard snapshots

Review Follow-ups Addressed

removed BOM bytes in chat components where required
removed duplicate hideHint declaration and cleaned unused imports
memoized heavy derived media lists and split concatenated state declarations
reverted unintended desktop navbar collapse default
restored cached user/profile helper behavior
improved migration formatting/comments and removed unnecessary .gitignore noise

Commit History

squashed into one clean commit to avoid dirty commit records in history

Copilot

Pull request overview

This PR updates the chat experience with improved conversation ordering and richer UI (badges/media viewing), and adds database-level cascade deletes to support safe conversation deletion.

Changes:

Add DM sorting controls (newest/oldest/A–Z/Z–A), message search, and revamped chat layout with left/right sidebars.
Restore/standardize rank badge styling/animations by moving badge CSS into app/globals.css and using class-based badges + icons.
Add a Supabase migration to enforce ON DELETE CASCADE for messages and conversation_participants when deleting a conversation.

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
supabase/migrations/20260327100000_cascade_conversation_delete.sql	Adds cascade FK constraints for conversation cleanup on delete.
package.json	Bumps `@fortawesome/react-fontawesome` to support updated icon usage.
package-lock.json	Updates lockfile to reflect dependency bump and npm resolution changes.
app/globals.css	Centralizes badge keyframes/styles so they’re shared across UI.
app/components/dashboard/Navbar.tsx	Changes default sidebar collapse behavior (desktop now collapsed).
app/components/chat/Player.tsx	Adds hint-hiding helper (currently introduces a duplicate declaration).
app/components/chat/Messages.tsx	Adds media navigation controls + updated badge rendering with icons.
app/components/chat/Conversations.tsx	Updates conversation list item layout/labels/initials.
app/components/LeaderboardTable.tsx	Removes inline injected badge CSS now that globals provide styles.
app/components/Chat.tsx	Major chat UI refactor: sorting/search, sidebars, media viewer, deletion flow, badge icon mapping.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-29T11:03:06Z

app/components/chat/Player.tsx

+  const hideHint = () => {
+    if (hintTimeoutRef.current) window.clearTimeout(hintTimeoutRef.current);
+    setActiveHint(null);
+    setHintPos(null);
+  };
+


hideHint is declared twice in this component (there is another const hideHint = ... later in the file). This will cause a TypeScript compile error due to redeclaration. Remove one of the declarations and keep a single hideHint implementation.

Suggested change

const hideHint = () => {

if (hintTimeoutRef.current) window.clearTimeout(hintTimeoutRef.current);

setActiveHint(null);

setHintPos(null);

};

Copilot · 2026-03-29T11:03:06Z

app/components/Chat.tsx

@@ -1,23 +1,49 @@
-"use client";
+"use client";


There is a leading BOM/zero-width character before the "use client" directive. Next.js requires the directive to be the very first bytes in the file; the BOM can prevent the component from being treated as a client component. Remove the BOM so the file starts exactly with "use client";.

Suggested change

"use client";

"use client";

Copilot · 2026-03-29T11:03:06Z

app/components/chat/Conversations.tsx

@@ -1,4 +1,4 @@
-import { User } from "@supabase/supabase-js";
+import { User } from "@supabase/supabase-js";


This file starts with a BOM/zero-width character before the first import. While it may not break runtime behavior, it commonly causes noisy diffs and occasional tooling issues. Please remove the BOM so the file begins with a normal import ....

Suggested change

import { User } from "@supabase/supabase-js";

import { User } from "@supabase/supabase-js";

Copilot · 2026-03-29T11:03:07Z

app/components/dashboard/Navbar.tsx

+        // default to collapsed on desktop too
+        setCollapsed(true);
        setMobileHidden(false);


This change makes the sidebar default to collapsed on desktop as well as mobile. Since the PR is focused on chat ordering/cascade delete, this looks like an unrelated UX change; if it is intentional, it should be called out in the PR description (or moved to a separate PR) to avoid surprising behavior changes.

Copilot · 2026-03-29T11:03:07Z

app/components/chat/Messages.tsx

@@ -1,4 +1,4 @@
-"use client";
+"use client";


There is a leading BOM/zero-width character before the "use client" directive. Next.js requires the directive to be the very first bytes in the file; the BOM can prevent the component from being treated as a client component. Remove the BOM so the file starts exactly with "use client";.

Suggested change

"use client";

"use client";

Copilot · 2026-03-29T11:03:07Z

app/components/Chat.tsx

  const [showModal, setShowModal] = useState(false);
-  const [search, setSearch] = useState("");
-  const [allUsers, setAllUsers] = useState<ChatUser[]>([]);
+  const [search, setSearch] = useState("");    const [messageSearch, setMessageSearch] = useState("");  const [allUsers, setAllUsers] = useState<ChatUser[]>([]);


These state declarations appear to have been accidentally concatenated onto a single line, which hurts readability and makes future diffs harder to review. Please split them into separate const declarations on their own lines.

Suggested change

const [search, setSearch] = useState(""); const [messageSearch, setMessageSearch] = useState(""); const [allUsers, setAllUsers] = useState<ChatUser[]>([]);

const [search, setSearch] = useState("");

const [messageSearch, setMessageSearch] = useState("");

const [allUsers, setAllUsers] = useState<ChatUser[]>([]);

Copilot · 2026-03-29T11:03:07Z

app/components/Chat.tsx

+  faEllipsisVertical,
+  faFlag,
+  faLock,
+  faGhost,


Several icons are imported here but never referenced in the file (e.g. faEllipsisVertical, faFlag, faLock). This will fail linting in many configurations and adds noise. Remove unused imports or wire them up where intended.

Suggested change

faEllipsisVertical,

faFlag,

faLock,

faGhost,

Copilot · 2026-03-29T11:03:07Z

app/components/Chat.tsx

+  const allMediaAttachments = messages
+    .flatMap((m) => m.attachments || [])
+    .filter(
+      (a) => a?.mimetype?.startsWith("image/") || a?.mimetype?.startsWith("video/")
+    )
+    .reverse();


allMediaAttachments is rebuilt (including a .reverse()) on every render. With long message histories this can become a noticeable cost. Consider memoizing this derived list with useMemo (and potentially sharing the media-viewer logic with Messages to avoid duplicated implementations).

mrepol742

Please ammend the commit to prevent dirty commits/records from the git history.

app/lib/supabase/help/user.ts

mrepol742 · 2026-04-01T18:14:43Z

supabase/migrations/20260320234600_add_user_stats.sql

the spacing for these files are necessary to ensure its readable

Addressed in commit f49ed2f. I reformatted this migration with clearer spacing/newlines for readability.

mrepol742 · 2026-04-01T18:14:48Z

supabase/migrations/20260320234653_add_profiles.sql

the spacing for these files are necessary to ensure its readable

Addressed in commit f49ed2f. I reformatted this migration with clearer spacing/newlines for readability.

mrepol742 · 2026-04-01T18:14:56Z

supabase/migrations/20260320234714_add_leaderboards.sql

the spacing for these files are necessary to ensure its readable

Addressed in commit f49ed2f. I reformatted this migration with clearer spacing/newlines for readability.

mrepol742 · 2026-04-01T18:15:02Z

supabase/migrations/20260320234731_add_enforcement_checks.sql

the spacing for these files are necessary to ensure its readable

Addressed in commit f49ed2f. I reformatted this migration with clearer spacing/newlines for readability.

mrepol742 · 2026-04-01T18:15:31Z

supabase/migrations/20260325072343_add_role_to_profiles.sql

the spacing for these files are necessary to ensure its readable

Addressed in commit f49ed2f. I reformatted this migration with clearer spacing/newlines for readability.

supabase/migrations/20260329120000_add_user_dashboard_snapshots.sql

supabase/migrations/20260329133000_add_chat_presence_and_read_tracking.sql

supabase/migrations/20260330104500_fix_global_chat_membership.sql

mrepol742 · 2026-04-01T18:18:38Z

.gitignore

the supabase/.temp folder is already ignored and unnecessary to be written.

Addressed in commit f49ed2f. I removed the unnecessary supabase/.temp ignore noise and kept the .gitignore clean.

Consolidates PR hallofcodes#57 changes into a clean commit history. Includes reviewer follow-ups for chat presence, BOM cleanup, migration readability, and lint/build fixes.

Refines PR hallofcodes#57 into a single clean commit covering chat, dashboard, landing, settings, and migration updates. Includes reviewer follow-ups for BOM cleanup, component fixes, migration readability, and history hygiene.

mrepol742 · 2026-04-01T20:41:32Z

app/components/BrowserCheck.tsx

This browser check is required to ensure the client is not using puppeter or dummy browser for automation.

mrepol742 · 2026-04-01T20:41:59Z

supabase/migrations/20260320234731_add_enforcement_checks.sql

the spacing for these files are necessary to ensure its readable

mrepol742 · 2026-04-01T20:44:41Z

app/(auth)/signup/page.tsx

Please make sure to aligned with the project designed and avoid using 3rd party or inlined SVG. For this SVG Back button,please sure the FontAwesome Chevron Left.

mrepol742 · 2026-04-01T20:45:04Z

app/(auth)/login/page.tsx

Please make sure to aligned with the project designed and avoid using 3rd party or inlined SVG. For this SVG Back button,please sure the FontAwesome Chevron Left.

mrepol742 · 2026-04-01T20:49:06Z

app/api/wakatime/sync/route.ts

the route file (e.g. route.ts) should stay focused on handling the request/response lifecycle (GET, POST, etc.), not business logic or data transformation. Please refractor this to util folder.

mrepol742 · 2026-04-01T20:55:12Z

app/(user)/dashboard/settings/page.tsx

the 'best practices' is should not be there as it should be part of FAQ.

<div className="glass-card p-5 border-t-4 border-indigo-500/40"> <h3 className="text-xs font-semibold text-indigo-300 uppercase tracking-widest mb-3"> Best Practices </h3> <div className="space-y-2 text-xs md:text-sm text-gray-400 leading-relaxed"> <p>Rotate API keys periodically and avoid sharing them in screenshots.</p> <p>Use a strong password and reset it immediately if account activity looks unusual.</p> <p>After changing your API key, run a dashboard sync to refresh your metrics.</p> </div> </div>

changing the api should not forced the system to update the coding metrics to avoid API call and charges in vercel and supabase

rotating the api keys periodically is not ideal as if the API key rotates the user needs to update the API key not just in our system but also on his/her devices and other 3rd party services connected or integrated to Wakatime.

in terms of using a strong password, supabase automatically checks the password strength and if the password is common or in data leaks.

Refines PR hallofcodes#57 into a single clean commit covering chat, dashboard, landing, settings, and migration updates. Includes reviewer follow-ups for BOM cleanup, component fixes, migration readability, and history hygiene.

Copilot AI review requested due to automatic review settings March 29, 2026 10:58

Copilot started reviewing on behalf of 0x3EF8 March 29, 2026 10:59 View session

Copilot AI reviewed Mar 29, 2026

View reviewed changes

0x3EF8 changed the title ~~feat(chat): add chat ordering enhancements and cascade delete migration~~ feat: improve chat, dashboard, landing page, settings, and account completion Mar 29, 2026

0x3EF8 mentioned this pull request Apr 1, 2026

Unable to Change WakaTime API Key After Account Merge #60

Open

mrepol742 requested changes Apr 1, 2026

View reviewed changes

0x3EF8 force-pushed the master branch from 58fb719 to a3f5e5f Compare April 1, 2026 20:30

0x3EF8 changed the title ~~feat: improve chat, dashboard, landing page, settings, and account completion~~ feat: stabilize chat realtime, dashboard insights, and account flows Apr 1, 2026

0x3EF8 force-pushed the master branch from a3f5e5f to f49ed2f Compare April 1, 2026 20:41

mrepol742 requested changes Apr 1, 2026

View reviewed changes

0x3EF8 force-pushed the master branch from f49ed2f to 76d560e Compare April 1, 2026 21:05

0x3EF8 force-pushed the master branch from 76d560e to 534862e Compare April 1, 2026 21:05

0x3EF8 force-pushed the master branch from 534862e to 758a8d6 Compare April 1, 2026 21:16

0x3EF8 force-pushed the master branch from 758a8d6 to 62ac99b Compare April 1, 2026 21:25

0x3EF8 force-pushed the master branch from 62ac99b to b04701d Compare April 1, 2026 21:29

0x3EF8 force-pushed the master branch from b04701d to 3de074b Compare April 1, 2026 21:30

0x3EF8 force-pushed the master branch 5 times, most recently from 45f23e4 to 5b9f4cd Compare April 1, 2026 22:39

0x3EF8 force-pushed the master branch from 5b9f4cd to 7331074 Compare April 1, 2026 23:06

feat(ui): unify dashboard animations and FontAwesome icons

72c9356

0x3EF8 force-pushed the master branch from 7331074 to 72c9356 Compare April 1, 2026 23:14

		@@ -1,4 +1,4 @@
		import { User } from "@supabase/supabase-js";
		import { User } from "@supabase/supabase-js";

Conversation

0x3EF8 commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Review Follow-ups Addressed

Commit History

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

Uh oh!

mrepol742 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

0x3EF8 commented Mar 26, 2026 •

edited

Loading