fix(web): fix hydration mismatch in KeyboardShortcutHint (#1041)

* fix(web): fix hydration mismatch in KeyboardShortcutHint

Replace module-level IS_MAC constant with a useIsMac hook that defers
platform detection to after mount via useEffect, ensuring server and
initial client renders agree before updating to the correct value.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* chore: update CHANGELOG for #1041

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: brendan-kellam

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 - Fixed line numbers being selectable in Safari in the lightweight code highlighter. [#1037](https://github.com/sourcebot-dev/sourcebot/pull/1037)
 - Fixed GitLab sync deleting repos when the API returns a non-404 error (e.g. 500) during group/user/project fetch. [#1039](https://github.com/sourcebot-dev/sourcebot/pull/1039)
+- Fixed React hydration mismatch in `KeyboardShortcutHint` caused by platform detection running at module load time during SSR. [#1041](https://github.com/sourcebot-dev/sourcebot/pull/1041)
 
 ### Added
 - Added optional copy button to the lightweight code highlighter (`isCopyButtonVisible` prop), shown on hover. [#1037](https://github.com/sourcebot-dev/sourcebot/pull/1037)

packages/web/src/app/components/keyboardShortcutHint.tsx

@@ -1,7 +1,8 @@
 'use client';
 
-import { cn, IS_MAC } from '@/lib/utils'
-import React, { useMemo } from 'react'
+import { cn } from '@/lib/utils'
+import { useIsMac } from '@/hooks/useIsMac'
+import { useMemo } from 'react'
 
 interface KeyboardShortcutHintProps {
     shortcut: string
@@ -81,16 +82,17 @@ function mapKey(key: string, keyMap: Record<string, string>): string {
  * Converts react-hotkeys syntax to platform-appropriate keyboard shortcut display.
  * Accepts formats like: "mod+b", "alt+shift+f12", "ctrl enter"
  */
-function getPlatformShortcut(shortcut: string): string {
+function getPlatformShortcut(shortcut: string, isMac: boolean): string {
     // Split by + or space to handle both "mod+b" and "⌘ B" formats
     const keys = shortcut.split(/[+\s]+/).filter(Boolean);
-    const keyMap = IS_MAC ? MAC_KEY_MAP : WINDOWS_KEY_MAP;
-    
+    const keyMap = isMac ? MAC_KEY_MAP : WINDOWS_KEY_MAP;
+
     return keys.map(key => mapKey(key, keyMap)).join(' ');
 }
 
 export function KeyboardShortcutHint({ shortcut, label, className }: KeyboardShortcutHintProps) {
-    const platformShortcut = useMemo(() => getPlatformShortcut(shortcut), [shortcut]);
+    const isMac = useIsMac();
+    const platformShortcut = useMemo(() => getPlatformShortcut(shortcut, isMac), [shortcut, isMac]);
 
     return (
         <div className={cn("inline-flex items-center", className)} aria-label={label || `Keyboard shortcut: ${platformShortcut}`}>

packages/web/src/features/chat/components/chatBox/chatBox.tsx

@@ -5,7 +5,8 @@ import { Button } from "@/components/ui/button";
 import { Tooltip, TooltipContent, TooltipTrigger } from "@/components/ui/tooltip";
 import { CustomEditor, LanguageModelInfo, MentionElement, RenderElementPropsFor, SearchScope } from "@/features/chat/types";
 import { insertMention, slateContentToString } from "@/features/chat/utils";
-import { cn, IS_MAC } from "@/lib/utils";
+import { cn } from "@/lib/utils";
+import { useIsMac } from "@/hooks/useIsMac";
 import { computePosition, flip, offset, shift, VirtualElement } from "@floating-ui/react";
 import { ArrowUp, Loader2, StopCircleIcon } from "lucide-react";
 import { Fragment, KeyboardEvent, memo, useCallback, useEffect, useMemo, useRef, useState } from "react";
@@ -367,6 +368,7 @@ const MentionComponent = ({
 }: RenderElementPropsFor<MentionElement>) => {
     const selected = useSelected();
     const focused = useFocused();
+    const isMac = useIsMac();
 
     if (data.type === 'file') {
         return (
@@ -384,7 +386,7 @@ const MentionComponent = ({
                     >
                         <span contentEditable={false} className="flex flex-row items-center select-none">
                             {/* @see: https://github.com/ianstormtaylor/slate/issues/3490 */}
-                            {IS_MAC ? (
+                            {isMac ? (
                                 <Fragment>
                                     {children}
                                     <VscodeFileIcon fileName={data.name} className="w-3 h-3 mr-1" />

packages/web/src/hooks/useIsMac.ts

@@ -0,0 +1,13 @@
+'use client';
+
+import { useState, useEffect } from 'react';
+
+export function useIsMac(): boolean {
+    const [isMac, setIsMac] = useState(false);
+
+    useEffect(() => {
+        setIsMac(/Mac OS X/.test(navigator.userAgent));
+    }, []);
+
+    return isMac;
+}

packages/web/src/lib/utils.ts

@@ -619,8 +619,6 @@ export const getOrgMetadata = (org: Org): OrgMetadata | null => {
     return currentMetadata.success ? currentMetadata.data : null;
 }
 
-export const IS_MAC = typeof navigator !== 'undefined' && /Mac OS X/.test(navigator.userAgent);
-
 
 export const isHttpError = (error: unknown, status: number): boolean => {
     return error !== null