@sentry/junior-dashboard 0.276.0 → 0.278.0

package/src/client/pages/memory/MemoryTimeline.tsx

@@ -5,7 +5,6 @@ import {
   ActivityChartXAxis,
   ActivityChartYAxis,
 } from "../../components/charts/ActivityChart";
-import { ChartLegend } from "../../components/charts/ChartLegend";
 import {
   type TimeRangeBucketUnit,
   type TimeRangeDays,
@@ -14,16 +13,11 @@ import { Card } from "../../components/layout/Card";
 import { ChartContainer, ChartTooltip } from "../../components/ui/chart";
 import type { MemoryDay } from "./memoryDashboard";
 
-const series = [
-  { color: "var(--color-cyan-300)", key: "personal", label: "Personal" },
-  { color: "var(--color-emerald-300)", key: "public", label: "Public" },
-] as const;
+const chartConfig = {
+  count: { color: "var(--color-emerald-300)", label: "Memories" },
+};
 
-const chartConfig = Object.fromEntries(
-  series.map((item) => [item.key, { color: item.color, label: item.label }]),
-);
-
-/** Render viewer memory creation as a stacked personal/public timeline. */
+/** Render memory creation as a timeline. */
 export function MemoryTimeline(props: {
   bucketUnit?: TimeRangeBucketUnit;
 
@@ -33,7 +27,7 @@ export function MemoryTimeline(props: {
   const bucketUnit = props.bucketUnit ?? "day";
 
   const days = props.days;
-  const totals = days.map((day) => day.personal + day.public);
+  const totals = days.map((day) => day.count);
   const maximum = Math.max(1, ...totals);
   const hasMemories = totals.some((total) => total > 0);
 
@@ -45,15 +39,13 @@ export function MemoryTimeline(props: {
         </h2>
         <p className="mt-1 mb-0 font-sans text-xs leading-relaxed text-dashboard-text-muted">
           {bucketUnit === "hour"
-            ? "Stacked personal + public memories created each hour."
+            ? "Memories created each hour."
             : bucketUnit === "6hour"
-              ? "Stacked personal + public memories created each 6 hours."
-              : "Stacked personal + public memories created each day."}
+              ? "Memories created each 6 hours."
+              : "Memories created each day."}
         </p>
       </div>
 
-      <ChartLegend ariaLabel="Memory visibility legend" items={series} />
-
       <div className="relative mt-3 overflow-hidden">
         <ChartContainer
           className="aspect-auto h-50 w-full"
@@ -71,25 +63,17 @@ export function MemoryTimeline(props: {
             <ChartTooltip
               content={
                 <ActivityChartTooltipContent
-                  rows={(day: MemoryDay) => [
-                    ["personal", day.personal],
-                    ["public", day.public],
-                    ["total", day.personal + day.public],
-                  ]}
+                  rows={(day: MemoryDay) => [["memories", day.count]]}
                 />
               }
             />
-            {series.map((item) => (
-              <Bar
-                dataKey={item.key}
-                fill={`var(--color-${item.key})`}
-                fillOpacity={0.82}
-                isAnimationActive={false}
-                key={item.key}
-                maxBarSize={13}
-                stackId="visibility"
-              />
-            ))}
+            <Bar
+              dataKey="count"
+              fill="var(--color-count)"
+              fillOpacity={0.82}
+              isAnimationActive={false}
+              maxBarSize={13}
+            />
           </BarChart>
         </ChartContainer>
         {!hasMemories ? (

package/src/client/pages/memory/memoryDashboard.ts

@@ -1,4 +1,4 @@
-import { keepPreviousData, useQuery } from "@tanstack/react-query";
+import { useQuery } from "@tanstack/react-query";
 import { z } from "zod";
 
 import { fetchDashboardJson } from "../../http";
@@ -7,9 +7,8 @@ const memoryBucketSchema = z.string().regex(/^\d{4}-\d{2}-\d{2}(T\d{2})?$/);
 
 const memoryDaySchema = z
   .object({
+    count: z.number().int().min(0),
     date: memoryBucketSchema,
-    personal: z.number().int().min(0),
-    public: z.number().int().min(0),
   })
   .strict();
 
@@ -38,10 +37,8 @@ export const memoryDashboardSchema = z
         embedded: z.number().int().min(0),
         explicit: z.number().int().min(0),
         knowledge: z.number().int().min(0),
-        personal: z.number().int().min(0),
         preference: z.number().int().min(0),
         procedure: z.number().int().min(0),
-        public: z.number().int().min(0),
       })
       .strict(),
   })
@@ -64,36 +61,3 @@ export function useMemoryDashboardData() {
     retry: false,
   });
 }
-
-const memoryCountsSchema = z
-  .object({
-    active: z.number().int().min(0),
-    personal: z.number().int().min(0),
-    public: z.number().int().min(0),
-  })
-  .strict();
-
-/**
- * Load viewer-scoped memory totals that match a search query. The query stays
- * idle without a search, because the dashboard summary owns unfiltered totals.
- */
-export function useMemorySearchCounts(searchQuery: string) {
-  return useQuery({
-    enabled: Boolean(searchQuery),
-    placeholderData: keepPreviousData,
-    queryFn: ({ signal }) =>
-      fetchDashboardJson(
-        memoryCountsSchema,
-        `/api/plugins/memory/counts?${new URLSearchParams({ q: searchQuery })}`,
-        signal,
-      ),
-    queryKey: [
-      "dashboard",
-      "plugin-user-page",
-      "memory",
-      "counts",
-      searchQuery,
-    ],
-    retry: false,
-  });
-}

package/src/client/pages/memory/memoryRecord.ts

@@ -14,7 +14,6 @@ const memoryRecordSchema = z
     observedAt: z.iso.datetime(),
     origin: z.enum(["automatic", "explicit", "other"]),
     sourcePlatform: z.enum(["local", "slack", "web"]),
-    visibility: z.enum(["private", "public"]),
   })
   .strict();
 
@@ -43,10 +42,7 @@ export function memoryPageRecord(memory: MemoryRecord): PluginUserPageRecord {
   return {
     actions: [
       {
-        confirmation:
-          memory.visibility === "public"
-            ? "Forget this memory for everyone?"
-            : "Forget this memory?",
+        confirmation: "Forget this memory for everyone?",
         href: `/api/plugins/memory/memories/${encodeURIComponent(memory.id)}`,
         label: "Forget",
         method: "DELETE",
@@ -58,7 +54,6 @@ export function memoryPageRecord(memory: MemoryRecord): PluginUserPageRecord {
       { label: "Type", value: titleCase(memory.kind) },
       { label: "Learned", value: learned },
       { label: "Source", value: titleCase(memory.sourcePlatform) },
-      { label: "Visibility", value: titleCase(memory.visibility) },
       { label: "Remembered", value: formattedDate(memory.createdAt) },
       { label: "Observed", value: formattedDate(memory.observedAt) },
       {

package/src/client/pages/settings/AutoArchiveSettings.tsx

@@ -0,0 +1,145 @@
+import { useMutation, useQueryClient } from "@tanstack/react-query";
+
+import {
+  AUTO_ARCHIVE_MAX_GRACE_HOURS,
+  dashboardIdentitySchema,
+  type DashboardIdentity,
+} from "../../../api/schema";
+import { Button } from "../../components/ui/button";
+import { Checkbox } from "../../components/ui/checkbox";
+import { Field, FieldDescription, FieldLabel } from "../../components/ui/field";
+import { NativeSelect } from "../../components/ui/native-select";
+import { InlineError } from "../../components/InlineError";
+import { Card } from "../../components/layout/Card";
+import { useArchiveDoneConversations } from "../../conversations/autoArchiveQueries";
+import { patch } from "../../http";
+import type { DashboardCoreData } from "../../types";
+
+const dashboardCoreQueryKey = ["dashboard", "core"] as const;
+
+/** Grace periods offered in Settings, in hours. */
+const AUTO_ARCHIVE_GRACE_OPTIONS = [1, 6, 24, 72, AUTO_ARCHIVE_MAX_GRACE_HOURS];
+
+function formatArchivedCount(count: number): string {
+  if (count === 0) return "No done conversations to archive.";
+  return count === 1
+    ? "Archived 1 conversation."
+    : `Archived ${count} conversations.`;
+}
+
+function formatGraceHours(hours: number): string {
+  if (hours % 24 !== 0) return hours === 1 ? "1 hour" : `${hours} hours`;
+  const days = hours / 24;
+  if (days === 7) return "1 week";
+  return days === 1 ? "1 day" : `${days} days`;
+}
+
+/**
+ * Let the user turn auto-archive on for their conversations and pick how long
+ * finished work must stay quiet before Junior archives it.
+ */
+export function AutoArchiveSettings({
+  identity,
+}: {
+  identity: DashboardIdentity;
+}) {
+  const queryClient = useQueryClient();
+  const update = useMutation({
+    mutationFn: (
+      settings: Pick<
+        DashboardIdentity,
+        "autoArchiveDefault" | "autoArchiveGraceHours"
+      >,
+    ) => patch(dashboardIdentitySchema, "/api/me", settings),
+    onSuccess: (updated) => {
+      queryClient.setQueryData<DashboardCoreData>(
+        dashboardCoreQueryKey,
+        (current) => (current ? { ...current, me: updated } : current),
+      );
+    },
+  });
+  const archiveDone = useArchiveDoneConversations();
+  const enabled = identity.autoArchiveDefault ?? false;
+  const graceHours = identity.autoArchiveGraceHours ?? 24;
+  // Keep a saved value that is not a preset, so the select shows the truth.
+  const graceOptions = AUTO_ARCHIVE_GRACE_OPTIONS.includes(graceHours)
+    ? AUTO_ARCHIVE_GRACE_OPTIONS
+    : [...AUTO_ARCHIVE_GRACE_OPTIONS, graceHours].sort((a, b) => a - b);
+
+  return (
+    <Card className="mt-6 mb-0" padding="md" variant="raised">
+      <h2 className="m-0 text-lg font-bold">Conversations</h2>
+      <label className="mt-4 flex cursor-pointer items-start gap-3">
+        <Checkbox
+          checked={enabled}
+          className="mt-0.5"
+          disabled={update.isPending}
+          onCheckedChange={(checked) =>
+            update.mutate({ autoArchiveDefault: checked })
+          }
+        />
+        <span className="grid gap-0.5">
+          <span className="text-sm font-medium">
+            Auto-archive conversations when done
+          </span>
+          <span className="text-xs text-dashboard-text-muted">
+            Archive a conversation for you when all of its linked work, such as
+            pull requests, is merged or closed and nobody replies during the
+            wait time. You can change this for each conversation in its details.
+          </span>
+        </span>
+      </label>
+      <Field className="mt-5 max-w-xs">
+        <FieldLabel htmlFor="auto-archive-grace">Wait time</FieldLabel>
+        <NativeSelect
+          aria-describedby="auto-archive-grace-help"
+          disabled={update.isPending}
+          id="auto-archive-grace"
+          onChange={(event) =>
+            update.mutate({ autoArchiveGraceHours: Number(event.target.value) })
+          }
+          value={graceHours}
+        >
+          {graceOptions.map((hours) => (
+            <option key={hours} value={hours}>
+              {formatGraceHours(hours)}
+            </option>
+          ))}
+        </NativeSelect>
+        <FieldDescription id="auto-archive-grace-help">
+          How long finished work and the conversation stay quiet before archive.
+        </FieldDescription>
+      </Field>
+      {update.isError ? (
+        <InlineError className="mt-3">
+          Could not save this setting. Try again.
+        </InlineError>
+      ) : null}
+      <div className="mt-5 grid gap-1.5">
+        <div className="flex flex-wrap items-center gap-3">
+          <Button
+            disabled={archiveDone.isPending}
+            onClick={() => archiveDone.mutate()}
+            type="button"
+          >
+            {archiveDone.isPending ? "Archiving…" : "Archive done now"}
+          </Button>
+          {archiveDone.isSuccess ? (
+            <p className="m-0 text-sm text-dashboard-text-muted" role="status">
+              {formatArchivedCount(archiveDone.data.archived)}
+            </p>
+          ) : null}
+          {archiveDone.isError ? (
+            <InlineError>
+              Could not archive conversations. Try again.
+            </InlineError>
+          ) : null}
+        </div>
+        <p className="m-0 text-xs text-dashboard-text-muted">
+          Archive every done conversation once, without the wait time. Skips
+          conversations where you turned auto-archive off.
+        </p>
+      </div>
+    </Card>
+  );
+}

package/src/client/pages/settings/SettingsPage.tsx

@@ -24,6 +24,7 @@ import { InlineError } from "../../components/InlineError";
 import { Card } from "../../components/layout/Card";
 import { DashboardApiError, fetchDashboardJson, patch, put } from "../../http";
 import type { DashboardCoreData } from "../../types";
+import { AutoArchiveSettings } from "./AutoArchiveSettings";
 import { SettingsPageLayout } from "./SettingsPageLayout";
 
 const dashboardCoreQueryKey = ["dashboard", "core"] as const;
@@ -108,6 +109,7 @@ export function SettingsPage({ identity }: SettingsPageProps) {
       </form>
       <ConversationStartSetting />
       <ComposerAutocorrectSetting />
+      <AutoArchiveSettings identity={identity} />
       <CustomizationForm />
     </SettingsPageLayout>
   );

package/src/client/review-desk/ConversationReviewItems.tsx

@@ -3,6 +3,7 @@ import { useEffect, useRef, useState } from "react";
 import type { ReviewItemPayload } from "@sentry/junior/api/schema";
 
 import { LoadMorePagination } from "../components/Pagination";
+import { Collapsible, CollapsibleContent } from "../components/ui/collapsible";
 import {
   DetailsCard,
   DetailsList,
@@ -74,9 +75,9 @@ function ReviewItemRow(props: {
     if (props.defaultExpanded) setExpanded(true);
   }, [props.defaultExpanded]);
   return (
-    <>
+    <Collapsible className="min-w-0" onOpenChange={setExpanded} open={expanded}>
       <DetailsRow
-        expanded={expanded}
+        disclosure
         icon={<Inbox aria-hidden="true" className="size-3.5" />}
         meta={
           <span className="grid size-5 place-items-center">
@@ -86,14 +87,11 @@ function ReviewItemRow(props: {
             />
           </span>
         }
-        onToggle={() => setExpanded((value) => !value)}
         title={props.item.title}
       />
-      {expanded ? (
-        <div className="px-1.5 pb-1">
-          <ReviewItemCard item={props.item} />
-        </div>
-      ) : null}
-    </>
+      <CollapsibleContent className="px-1.5 pb-1">
+        <ReviewItemCard item={props.item} />
+      </CollapsibleContent>
+    </Collapsible>
   );
 }

package/src/client/review-desk/queries.ts

@@ -20,7 +20,8 @@ import {
 import { conversationDetailTitle } from "../format";
 import { fetchDashboardJson, post } from "../http";
 
-const REVIEW_ITEMS_QUERY_KEY = ["dashboard", "review-items"] as const;
+/** Prefix of every Review item read, for refreshes after a change. */
+export const REVIEW_ITEMS_QUERY_KEY = ["dashboard", "review-items"] as const;
 const COUNT_REFRESH_MS = 60_000;
 
 /**

package/src/client/types.ts

@@ -56,32 +56,22 @@ export type TranscriptViewSubagentPart = {
 export type TranscriptViewContextEventPart = {
   /** People mentioned in the summary, resolved by the API. */
   mentions?: ActorIdentity[];
-  event:
-    | {
-        createdAt: string;
-        type: "compaction";
-        modelId?: string;
-        modelProfile?: string;
-        summary?: string;
-        details?: {
-          reason: "capacity";
-          estimatedInputTokens: number;
-          replacementInputTokens?: number;
-          triggerTokens: number;
-          inputLimitTokens: number;
-          inputMessageCount: number;
-          retainedMessageCount: number;
-          summaryChars: number;
-        };
-      }
-    | {
-        createdAt: string;
-        modelId: string;
-        modelProfile: string;
-        reasoningLevel?: string;
-        summary?: string;
-        type: "handoff";
-      };
+  event: {
+    createdAt: string;
+    type: "compaction";
+    modelId?: string;
+    summary?: string;
+    details?: {
+      reason: "capacity";
+      estimatedInputTokens: number;
+      replacementInputTokens?: number;
+      triggerTokens: number;
+      inputLimitTokens: number;
+      inputMessageCount: number;
+      retainedMessageCount: number;
+      summaryChars: number;
+    };
+  };
   type: "context_event";
 };
 
@@ -138,6 +128,8 @@ export type TranscriptViewMessage = {
   events?: ConversationReportMessageEvent[];
   /** Whether the source message addressed Junior directly. */
   explicitMention?: boolean;
+  /** Quiet message: stored, but not delivered to the Location. */
+  quiet?: true;
   /** Person changes to Review items that this message records. */
   reviewItemChanges?: ReviewItemChangeRecord[];
   /** People mentioned in the event text, resolved by the API. */
@@ -148,13 +140,6 @@ export type TranscriptViewMessage = {
   messageId?: string;
   /** True while the message is accepted in the mailbox but not yet in history. */
   pending?: boolean;
-  route?: {
-    confidence?: number;
-    modelId: string;
-    modelProfile: string;
-    reasoningLevel: string;
-    source: "configured" | "inherited" | "router";
-  };
   eventId?: string;
   failureCode?: ConversationTurnFailureCode;
   failureReason?: ConversationTurnFailureReason;
@@ -176,6 +161,7 @@ export type Conversation = {
   annotations?: ConversationSummaryReport["annotations"];
   sidebarAnnotations?: ConversationSummaryReport["sidebarAnnotations"];
   archivedAt?: string | null;
+  pinnedAt?: string | null;
   auxiliaryCosts?: ConversationSummaryReport["auxiliaryCosts"];
   channel?: string;
   channelName?: string;

package/src/conversation-cache.ts

@@ -1,7 +1,8 @@
 import { createHash } from "node:crypto";
+import { CONVERSATION_DETAIL_VERSION_HEADER } from "@sentry/junior/api/schema";
 import type { MiddlewareHandler } from "hono";
 
-/** Revalidate dashboard detail reads only after the route checks viewer access. */
+/** Revalidate mock detail reads on content, after the route checks viewer access. */
 export const revalidateConversation: MiddlewareHandler = async (
   context,
   next,
@@ -15,15 +16,12 @@ export const revalidateConversation: MiddlewareHandler = async (
   // generatedAt is the read time, not a change to the Conversation. The mailbox
   // has its own live watermark and is never handled by this middleware.
   const { generatedAt: _generatedAt, ...content } = report;
-  const etag = `W/"${createHash("sha256").update(JSON.stringify(content)).digest("hex")}"`;
+  const version = createHash("sha256")
+    .update(JSON.stringify(content))
+    .digest("hex");
   context.header("cache-control", "private, no-store");
-  context.header("etag", etag);
-  const validators = context.req.header("if-none-match")?.split(",");
-  if (
-    validators?.some(
-      (value) => value.trim().replace(/^W\//, "") === etag.slice(2),
-    )
-  ) {
+  context.header(CONVERSATION_DETAIL_VERSION_HEADER, version);
+  if (context.req.header(CONVERSATION_DETAIL_VERSION_HEADER) === version) {
     context.res = new Response(null, {
       status: 304,
       headers: context.res.headers,