@sentry/junior-dashboard 0.265.0 → 0.266.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/junior-dashboard",
-  "version": "0.265.0",
+  "version": "0.266.0",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -37,8 +37,8 @@
     "recharts": "3.8.0",
     "shiki": "4.1.0",
     "zod": "^4.5.4",
-    "@sentry/junior": "0.265.0",
-    "@sentry/junior-plugin-api": "0.265.0"
+    "@sentry/junior": "0.266.0",
+    "@sentry/junior-plugin-api": "0.266.0"
   },
   "devDependencies": {
     "@oxlint/plugins": "1.79.0",

package/src/client/components/ActorAvatar.tsx

@@ -1,5 +1,4 @@
-import { useState } from "react";
-import { cn } from "../styles";
+import { Avatar, AvatarFallback, AvatarImage } from "./ui/avatar";
 
 /** Show an actor's image, with initials when the image is missing or fails. */
 export function ActorAvatar(props: {
@@ -7,7 +6,6 @@ export function ActorAvatar(props: {
   imageUrl?: string;
   size: "detail" | "list";
 }) {
-  const [failedUrl, setFailedUrl] = useState<string>();
   const words = props.name.trim().split(/\s+/).filter(Boolean);
   // The compact size fits one initial at the 12px type floor.
   const initials =
@@ -17,24 +15,14 @@ export function ActorAvatar(props: {
         ? `${words[0]![0]}${words.at(-1)![0]}`
         : props.name.slice(0, 2);
   return (
-    <span
-      aria-hidden="true"
-      className={cn(
-        "relative inline-grid shrink-0 place-items-center overflow-hidden rounded-full border-2 border-dashboard-bg bg-dashboard-focus font-sans font-semibold leading-none tracking-tight text-dashboard-text-inverse shadow-sm",
-        props.size === "list" ? "size-6 text-2xs" : "size-7 text-xs",
-      )}
-    >
-      {initials.toUpperCase()}
-      {props.imageUrl && props.imageUrl !== failedUrl ? (
-        <img
-          alt=""
-          className="absolute inset-0 size-full object-cover"
-          loading="lazy"
-          onError={() => setFailedUrl(props.imageUrl)}
-          referrerPolicy="no-referrer"
-          src={props.imageUrl}
-        />
-      ) : null}
-    </span>
+    <Avatar aria-hidden="true" size={props.size === "list" ? "sm" : "default"}>
+      <AvatarImage
+        alt=""
+        loading="lazy"
+        referrerPolicy="no-referrer"
+        src={props.imageUrl}
+      />
+      <AvatarFallback>{initials.toUpperCase()}</AvatarFallback>
+    </Avatar>
   );
 }

package/src/client/components/ConnectionBanner.tsx

@@ -1,4 +1,5 @@
 import { useDashboardOnline } from "../connection";
+import { Alert } from "./ui/alert";
 
 /** Explain that cached data remains visible while network reads are paused. */
 export function ConnectionBanner() {
@@ -6,12 +7,12 @@ export function ConnectionBanner() {
   if (online) return null;
 
   return (
-    <div
+    <Alert
       aria-live="polite"
-      className="border-t border-amber-300/15 bg-amber-300/[0.07] px-3 py-1.5 text-center font-sans text-xs text-amber-100/80"
+      className="block rounded-none border-0 border-t border-amber-300/15 bg-amber-300/[0.07] px-3 py-1.5 text-center font-sans text-xs text-amber-100/80"
       role="status"
     >
       You’re offline. Drafts stay on this device.
-    </div>
+    </Alert>
   );
 }

package/src/client/components/CopyButton.tsx

@@ -44,16 +44,15 @@ export function CopyButton(props: {
 
   if (props.layout === "menu") {
     return (
-      <button
+      <Button
         aria-label={label}
-        className="flex w-full cursor-pointer items-center gap-2.5 rounded-md border-0 bg-transparent px-2.5 py-2 text-left text-sm font-semibold text-dashboard-text transition-colors hover:bg-dashboard-fill-strong focus-visible:bg-dashboard-fill-strong focus-visible:outline-none disabled:cursor-not-allowed disabled:opacity-50"
         disabled={!props.getText || status === "copying"}
         onClick={() => void copyText()}
-        type="button"
+        size="menu"
       >
         <Icon aria-hidden="true" size={16} strokeWidth={2} />
         <span>{label}</span>
-      </button>
+      </Button>
     );
   }
 

package/src/client/components/EmptyTelemetry.tsx

@@ -1,10 +1,12 @@
 import type { ReactNode } from "react";
 
+import { Empty, EmptyDescription } from "./ui/empty";
+
 /** Render the dashboard empty-state block with quiet warning context. */
 export function EmptyTelemetry(props: { children: ReactNode }) {
   return (
-    <div className="min-w-0 rounded-lg border border-dashboard-border-strong bg-dashboard-bg px-4 py-3 text-base leading-relaxed text-dashboard-text-muted">
-      {props.children}
-    </div>
+    <Empty>
+      <EmptyDescription>{props.children}</EmptyDescription>
+    </Empty>
   );
 }

package/src/client/components/ImageAttachment.tsx

@@ -1,17 +1,13 @@
 import { ExternalLink, X } from "lucide-react";
 import {
   type MouseEvent,
-  useEffect,
-  useId,
-  useRef,
   useState,
   type ComponentPropsWithoutRef,
 } from "react";
 
-import {
-  acquireBodyScrollLock,
-  releaseBodyScrollLock,
-} from "../bodyScrollLock";
+import { Button } from "./ui/button";
+import { Dialog, DialogClose, DialogContent, DialogTitle } from "./ui/dialog";
+import { useSheet } from "./ui/useSheet";
 
 /** Return true when an image link click should open the inline preview. */
 export function shouldPreviewImageAttachment(
@@ -45,9 +41,6 @@ export function ImageAttachment(
     triggerClassName?: string;
   } & Omit<ComponentPropsWithoutRef<"img">, "alt" | "className" | "src">,
 ) {
-  const dialogRef = useRef<HTMLDialogElement>(null);
-  const bodyScrollLockedRef = useRef(false);
-  const titleId = useId();
   const [open, setOpen] = useState(false);
   const {
     context,
@@ -58,20 +51,6 @@ export function ImageAttachment(
     ...imageProps
   } = props;
 
-  const unlockBodyScroll = () => {
-    if (!bodyScrollLockedRef.current) return;
-    releaseBodyScrollLock();
-    bodyScrollLockedRef.current = false;
-  };
-
-  const close = () => {
-    dialogRef.current?.close();
-    setOpen(false);
-    unlockBodyScroll();
-  };
-
-  useEffect(() => () => unlockBodyScroll(), []);
-
   return (
     <>
       <a
@@ -80,12 +59,7 @@ export function ImageAttachment(
         onClick={(event) => {
           if (!shouldPreviewImageAttachment(event)) return;
           event.preventDefault();
-          const dialog = dialogRef.current;
-          if (!dialog || dialog.open) return;
-          acquireBodyScrollLock();
-          bodyScrollLockedRef.current = true;
           setOpen(true);
-          dialog.showModal();
         }}
         rel="noreferrer"
         target="_blank"
@@ -97,62 +71,74 @@ export function ImageAttachment(
           src={src}
         />
       </a>
-      <dialog
-        aria-labelledby={titleId}
-        className="m-0 h-dvh max-h-none w-screen max-w-none overflow-hidden border-0 bg-dashboard-ink p-0 text-dashboard-text backdrop:bg-dashboard-overlay md:m-auto md:h-[min(90dvh,960px)] md:w-[min(92vw,1440px)] md:rounded-lg md:border md:border-dashboard-border-emphasis"
-        onClick={(event) => {
-          if (event.target === event.currentTarget) close();
-        }}
-        onClose={() => {
-          setOpen(false);
-          unlockBodyScroll();
-        }}
-        ref={dialogRef}
+      {open ? (
+        <ImagePreview
+          context={context}
+          filename={filename}
+          onClose={() => setOpen(false)}
+          src={src}
+        />
+      ) : null}
+    </>
+  );
+}
+
+/**
+ * Show one image above the page. `useSheet` shares the document scroll lock
+ * with sheets and keeps Escape and backdrop dismissal on the front modal.
+ */
+function ImagePreview(props: {
+  context?: string;
+  filename: string;
+  onClose(): void;
+  src: string;
+}) {
+  const sheet = useSheet(props.onClose);
+  return (
+    <Dialog {...sheet.rootProps}>
+      <DialogContent
+        {...sheet.contentProps}
+        className="top-0 left-0 h-dvh w-screen max-w-none translate-x-0 translate-y-0 gap-0 overflow-hidden rounded-none border-0 bg-dashboard-ink p-0 shadow-none sm:max-w-none md:top-1/2 md:left-1/2 md:h-[min(90dvh,960px)] md:w-[min(92vw,1440px)] md:-translate-x-1/2 md:-translate-y-1/2 md:rounded-lg md:border"
       >
         <div className="grid h-full min-h-0 grid-rows-[auto_minmax(0,1fr)] bg-dashboard-bg-elevated">
           <header className="grid grid-cols-[minmax(0,1fr)_auto_auto] items-center gap-1.5 border-b border-dashboard-border-strong bg-dashboard-surface-raised px-3 py-2 pt-[max(0.5rem,env(safe-area-inset-top))] md:px-4">
             <div className="min-w-0">
-              <div
-                className="truncate font-mono text-xs text-dashboard-text"
-                id={titleId}
-              >
-                {filename}
-              </div>
-              {context ? (
+              <DialogTitle className="truncate font-mono text-xs leading-normal font-normal">
+                {props.filename}
+              </DialogTitle>
+              {props.context ? (
                 <div className="truncate font-mono text-2xs text-dashboard-text-muted">
-                  {context}
+                  {props.context}
                 </div>
               ) : null}
             </div>
             <a
               className="inline-flex min-h-10 items-center gap-1.5 rounded-md px-2.5 text-xs font-medium text-dashboard-text-muted no-underline transition-colors hover:bg-dashboard-fill-hover hover:text-dashboard-text"
-              href={src}
+              href={props.src}
               rel="noreferrer"
               target="_blank"
             >
               <ExternalLink aria-hidden="true" size={14} />
               <span className="max-sm:sr-only">Open raw file</span>
             </a>
-            <button
-              className="inline-flex min-h-10 items-center gap-1.5 rounded-md px-2.5 text-xs font-medium text-dashboard-text-muted transition-colors hover:bg-dashboard-fill-hover hover:text-dashboard-text"
-              onClick={close}
-              type="button"
+            <DialogClose
+              render={
+                <Button className="h-10 gap-1.5 border-0 bg-transparent px-2.5 text-xs text-dashboard-text-muted hover:bg-dashboard-fill-hover hover:text-dashboard-text" />
+              }
             >
               <X aria-hidden="true" size={16} />
               <span>Close</span>
-            </button>
+            </DialogClose>
           </header>
           <div className="grid min-h-0 place-items-center overflow-auto p-3 pb-[max(0.75rem,env(safe-area-inset-bottom))] md:p-6">
-            {open ? (
-              <img
-                alt={filename}
-                className="max-h-full max-w-full object-contain"
-                src={src}
-              />
-            ) : null}
+            <img
+              alt={props.filename}
+              className="max-h-full max-w-full object-contain"
+              src={props.src}
+            />
           </div>
         </div>
-      </dialog>
-    </>
+      </DialogContent>
+    </Dialog>
   );
 }

package/src/client/components/InlineError.tsx

@@ -1,6 +1,7 @@
 import type { ReactNode } from "react";
 
 import { cn } from "../styles";
+import { Alert } from "./ui/alert";
 
 /** Render a compact error message with consistent alert semantics. */
 export function InlineError(props: {
@@ -8,11 +9,14 @@ export function InlineError(props: {
   className?: string;
 }) {
   return (
-    <p
-      className={cn("m-0 text-sm text-rose-300", props.className)}
-      role="alert"
+    <Alert
+      className={cn(
+        "block rounded-none border-0 bg-transparent p-0 text-rose-300",
+        props.className,
+      )}
+      variant="destructive"
     >
       {props.children}
-    </p>
+    </Alert>
   );
 }

package/src/client/components/Notice.tsx

@@ -1,7 +1,9 @@
-import type { ButtonHTMLAttributes, ReactNode } from "react";
+import type { ComponentProps, ReactNode } from "react";
 import type { LucideIcon } from "lucide-react";
 
 import { cn } from "../styles";
+import { Alert, AlertDescription, AlertTitle } from "./ui/alert";
+import { Button } from "./ui/button";
 
 type NoticeTone = "default" | "error";
 
@@ -14,7 +16,7 @@ type NoticeProps = {
   tone?: NoticeTone;
 };
 
-type NoticeActionProps = ButtonHTMLAttributes<HTMLButtonElement> & {
+type NoticeActionProps = ComponentProps<typeof Button> & {
   tone?: NoticeTone;
 };
 
@@ -29,11 +31,10 @@ export function Notice({
 }: NoticeProps) {
   const isError = tone === "error";
   return (
-    <div
-      className={cn(
-        "overflow-hidden rounded-xl border bg-dashboard-surface-raised shadow-[0_16px_40px_rgba(0,0,0,0.5)]",
-        isError ? "border-rose-300/25" : "border-dashboard-border-strong",
-      )}
+    <Alert
+      className="block gap-0 overflow-hidden rounded-xl p-0 shadow-[0_16px_40px_rgba(0,0,0,0.5)]"
+      role={isError ? "alert" : "status"}
+      variant={isError ? "destructive" : "default"}
     >
       <div className="flex min-w-0 items-center gap-3 px-3 py-3">
         {Icon ? (
@@ -48,31 +49,26 @@ export function Notice({
             <Icon aria-hidden="true" size={16} />
           </div>
         ) : null}
-        <div className="min-w-0 flex-1" role={isError ? "alert" : "status"}>
-          <div
-            className={cn(
-              "font-display text-sm font-medium leading-snug",
-              isError ? "text-rose-50" : "text-dashboard-text",
-            )}
-          >
+        <div className="min-w-0 flex-1">
+          <AlertTitle className="font-display text-sm leading-snug">
             {title}
             {detail ? " " : null}
-          </div>
+          </AlertTitle>
           {detail ? (
-            <div
+            <AlertDescription
               className={cn(
                 "mt-0.5 truncate font-sans text-xs",
-                isError ? "text-rose-100/70" : "text-dashboard-text-muted",
+                isError && "text-rose-100/70",
               )}
             >
               {detail}
-            </div>
+            </AlertDescription>
           ) : null}
         </div>
         {action}
       </div>
       {children}
-    </div>
+    </Alert>
   );
 }
 
@@ -80,22 +76,19 @@ export function Notice({
 export function NoticeAction({
   className,
   tone = "default",
-  type = "button",
   ...props
 }: NoticeActionProps) {
   const isError = tone === "error";
   return (
-    <button
+    <Button
       {...props}
       className={cn(
-        "shrink-0 cursor-pointer px-3 py-1.5 text-xs transition focus:outline-none focus:ring-2 disabled:cursor-not-allowed disabled:opacity-40",
-        isError ? "focus:ring-rose-300/40" : "focus:ring-dashboard-focus/35",
+        "h-auto shrink-0 rounded-lg border-0 px-3 py-1.5 text-xs leading-normal",
         isError
-          ? "rounded-lg bg-rose-200/15 font-sans font-medium text-rose-50 hover:bg-rose-200/25"
-          : "rounded-lg bg-dashboard-accent/10 font-sans font-medium text-dashboard-accent-text hover:bg-dashboard-accent/20",
+          ? "bg-rose-200/15 text-rose-50 hover:bg-rose-200/25 hover:text-rose-50 focus-visible:outline-rose-300/55 data-disabled:hover:bg-rose-200/15 data-disabled:hover:text-rose-50"
+          : "bg-dashboard-accent/10 text-dashboard-accent-text hover:bg-dashboard-accent/20 hover:text-dashboard-accent-text data-disabled:hover:bg-dashboard-accent/10 data-disabled:hover:text-dashboard-accent-text",
         className,
       )}
-      type={type}
     />
   );
 }

package/src/client/components/Pagination.tsx

@@ -1,7 +1,12 @@
-import { ChevronLeft, ChevronRight } from "lucide-react";
-
 import { cn } from "../styles";
 import { Button } from "./ui/button";
+import {
+  Pagination,
+  PaginationContent,
+  PaginationItem,
+  PaginationNext,
+  PaginationPrevious,
+} from "./ui/pagination";
 
 /** Render a centered load-more control for cursor or progressive lists. */
 export function LoadMorePagination(props: {
@@ -54,10 +59,9 @@ export function PagePagination(props: {
   const end = Math.min(props.total, current * props.pageSize);
 
   return (
-    <div
-      aria-label="Pagination"
+    <Pagination
       className={cn(
-        "flex flex-wrap items-center justify-between gap-3",
+        "flex-wrap items-center justify-between gap-3",
         props.className,
       )}
     >
@@ -65,27 +69,23 @@ export function PagePagination(props: {
         Showing {start.toLocaleString("en-US")}-{end.toLocaleString("en-US")} of{" "}
         {props.total.toLocaleString("en-US")}
       </p>
-      <div className="flex items-center gap-2">
-        <Button
-          aria-label="Previous page"
-          disabled={current <= 1}
-          onClick={() => props.onPageChange(current - 1)}
-          size="icon"
-        >
-          <ChevronLeft aria-hidden="true" size={16} />
-        </Button>
-        <span className="min-w-16 text-center font-sans tabular-nums text-xs text-dashboard-text-muted">
+      <PaginationContent>
+        <PaginationItem>
+          <PaginationPrevious
+            disabled={current <= 1}
+            onClick={() => props.onPageChange(current - 1)}
+          />
+        </PaginationItem>
+        <PaginationItem className="min-w-16 text-center font-sans tabular-nums text-xs text-dashboard-text-muted">
           {current} / {props.pageCount}
-        </span>
-        <Button
-          aria-label="Next page"
-          disabled={current >= props.pageCount}
-          onClick={() => props.onPageChange(current + 1)}
-          size="icon"
-        >
-          <ChevronRight aria-hidden="true" size={16} />
-        </Button>
-      </div>
-    </div>
+        </PaginationItem>
+        <PaginationItem>
+          <PaginationNext
+            disabled={current >= props.pageCount}
+            onClick={() => props.onPageChange(current + 1)}
+          />
+        </PaginationItem>
+      </PaginationContent>
+    </Pagination>
   );
 }

package/src/client/components/ParticipantAvatarStack.tsx

@@ -1,9 +1,9 @@
 import type { ActorIdentity } from "@sentry/junior/api/schema";
 
 import { actorLabel } from "../format";
-import { cn } from "../styles";
 import type { Conversation } from "../types";
 import { ActorAvatar } from "./ActorAvatar";
+import { Avatar, AvatarFallback } from "./ui/avatar";
 import { Tooltip, TooltipContent, TooltipTrigger } from "./ui/tooltip";
 
 const MAX_VISIBLE_PARTICIPANTS = 3;
@@ -51,7 +51,7 @@ function ParticipantTooltipContent(props: { participant: ActorIdentity }) {
   );
 }
 
-function Avatar(props: {
+function ParticipantAvatar(props: {
   participant: ActorIdentity;
   size: ParticipantAvatarStackProps["size"];
 }) {
@@ -101,7 +101,7 @@ export function ParticipantAvatarStack(props: ParticipantAvatarStackProps) {
       role="group"
     >
       {visible.map((participant, index) => (
-        <Avatar
+        <ParticipantAvatar
           key={participantKey(participant, index)}
           participant={participant}
           size={props.size}
@@ -111,19 +111,19 @@ export function ParticipantAvatarStack(props: ParticipantAvatarStackProps) {
         <Tooltip>
           <TooltipTrigger
             render={
-              <span
+              <Avatar
                 aria-label={`${hidden.length} more participants: ${hidden
                   .map(participantDescription)
                   .join("; ")}`}
-                className={cn(
-                  "relative -ml-1 inline-grid shrink-0 pointer-events-none [@media(hover:hover)_and_(pointer:fine)]:pointer-events-auto place-items-center rounded-full border-2 border-dashboard-bg bg-dashboard-surface-active font-sans font-semibold tabular-nums leading-none text-dashboard-text-muted focus-visible:z-10 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-1 focus-visible:outline-dashboard-focus",
-                  props.size === "list" ? "size-6 text-2xs" : "size-7 text-xs",
-                )}
+                className="-ml-1 pointer-events-none bg-dashboard-surface-active shadow-none focus-visible:z-10 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-1 focus-visible:outline-dashboard-focus [@media(hover:hover)_and_(pointer:fine)]:pointer-events-auto"
                 onKeyDown={(event) => event.stopPropagation()}
+                size={props.size === "list" ? "sm" : "default"}
                 tabIndex={0}
               >
-                +{hidden.length}
-              </span>
+                <AvatarFallback className="tabular-nums tracking-normal text-dashboard-text-muted">
+                  +{hidden.length}
+                </AvatarFallback>
+              </Avatar>
             }
           />
           <TooltipContent>

package/src/client/components/README.md

@@ -35,6 +35,11 @@ file when every caller needs the same style or variant. App components compose
 | `Drawer`, `BottomSheet`        | `ui/sheet`        |
 | `layout/DashboardHeader` (nav) | `ui/sheet`        |
 | `ProfileMenu` (desktop)        | `ui/popover`      |
+| `ActorAvatar`                  | `ui/avatar`       |
+| `ParticipantAvatarStack`       | `ui/avatar`       |
+| `SearchInput`                  | `ui/input-group`  |
+| `PagePagination`               | `ui/pagination`   |
+| `EmptyTelemetry`               | `ui/empty`        |
 | `TimeRangeSelector`            | `ui/toggle-group` |
 | `ThemeSwitcher`                | `ui/toggle-group` |
 | `TranscriptViewToggle`         | `ui/toggle-group` |
@@ -42,6 +47,9 @@ file when every caller needs the same style or variant. App components compose
 | `charts/ActivityChartTooltip`  | `ui/tooltip`      |
 | `conversations/ToolFrame`      | `ui/trace`        |
 | `TranscriptReasoningView`      | `ui/trace`        |
+| `ImageAttachment` (preview)    | `ui/dialog`       |
+| `Notice`, `InlineError`        | `ui/alert`        |
+| App banners                    | `ui/alert`        |
 | Recharts activity charts       | `ui/chart`        |
 
 Rules for adopted components:
@@ -54,6 +62,11 @@ Rules for adopted components:
 - Use `ui/alert-dialog` for every confirmation. Do not use `window.confirm`.
   Keep the confirmed item in state after close, so the closing dialog keeps its
   text.
+- Use `useSheet` for each app `ui/dialog`, as for sheets. It shares the
+  document scroll lock. Only the front modal closes on Escape or a backdrop
+  tap. Mount the dialog only while it is open.
+- `ui/alert` sets `role="alert"`. Pass `role="status"` for a message that is
+  not an error.
 - `ui/toast` has no global notification manager. Features call
   `useToastManager` inside the mounted `Toaster` provider.
 - Put a checkbox or radio inside its `<label>`. Give each `RadioGroup` an
@@ -68,6 +81,8 @@ Rules for adopted components:
 - Build a form field from `ui/field`. Give `FieldError` the id
   `<control id>-error` and `FieldDescription` the id `<control id>-help`. The
   control sets `aria-invalid` and `aria-describedby`.
+- Use `Button size="menu"` for a full-width icon and label row in a mobile
+  overflow menu. A pressed row adds `bg-dashboard-fill-strong`.
 - For a link that looks like a button, put `buttonVariants()` on the router
   `Link`. A form page has one `Button variant="primary"` for its main save
   action.
@@ -141,7 +156,8 @@ Shared class strings live in `src/client/styles.ts`.
 - **Loading.** Use `ui/skeleton` for placeholder leaves and `PageContentSkeleton`
   variants for page bodies. Use `PageRouteLoading` when a page keeps its real
   header above a body skeleton. Use a feature-owned loading component for
-  distinct geometry, such as conversations.
+  distinct geometry, such as conversations. Use `ui/spinner` for an inline
+  pending icon.
 
 ## Navigation