fix(coach): align mobile sheet height to the responsive-sheet convention

The Coach drawer's phone branch capped at 95 dvh, leaving only a 5 %
slice of the underlying /insights page visible — read as a takeover
instead of a sheet. Every other mobile sheet in the app routes
through <ResponsiveSheet>, whose phone branch caps at 90 dvh so 10 %
of the underlying page stays visible. Match the convention here so
the Coach drawer feels like the same kind of surface as the rest of
the app on phone-class viewports.

src/components/insights/coach-panel/coach-drawer.tsx

@@ -288,11 +288,14 @@ export function CoachDrawer({
           "w-full p-0 sm:max-w-[720px]",
           "lg:!max-w-[min(960px,75vw)] xl:!max-w-[1080px]",
           isPhoneViewport
-            ? // Bottom-sheet caps at 95 dvh so a sliver of the underlying
-              // /insights page remains visible — clear "this is a
-              // sheet, not a takeover" signal. The rounded top corners
-              // match the iOS bottom-sheet feel.
-              "flex h-[95dvh] max-h-[95dvh] flex-col gap-0 rounded-t-2xl"
+            ? // Bottom-sheet caps at 90 dvh so a 10 % slice of the
+              // underlying /insights page stays visible — same
+              // convention as every other mobile sheet on the app via
+              // <ResponsiveSheet>'s phone branch (v1.4.28 R3c BK-M5
+              // alignment; previously 95 dvh, which left only a 5 %
+              // sliver and read as a takeover instead of a sheet).
+              // Rounded top corners match the iOS bottom-sheet feel.
+              "flex h-[90dvh] max-h-[90dvh] flex-col gap-0 rounded-t-2xl"
             : "flex h-[100dvh] flex-col gap-0",
         )}
       >