@sentry/junior-dashboard 0.276.0 → 0.278.0

package/src/client/display-text/codeBlocks.ts

@@ -0,0 +1,92 @@
+import type { BundledLanguage } from "shiki/bundle/web";
+
+import { dashboardCodeLanguages } from "../code-languages";
+import { detectOutputLanguage, prettyJsonData } from "../format";
+import type { CodeBlock } from "../types";
+
+// Shiki has no language detection, and agents often leave fences untagged.
+// Only clear signals select a language, so prose in a fence stays plain.
+const SCRIPT_PATTERN =
+  /^javascript:|=>|\b(?:const|let|var)\s+[\w$]+\s*=|\bfunction\b\s*[\w$]*\s*\(|^\s*(?:import\s.+\sfrom\s|export\s+(?:(?:async|class|const|default|enum|function|interface|let|type|var)\s|[{*]))/m;
+const PYTHON_PATTERN =
+  /^\s*(?:(?:async\s+)?def\s+\w+\s*\(.*\)\s*(?:->.*)?:|class\s+\w+(?:\([^)]*\))?\s*:|from\s+[\w.]+\s+import\s|import\s+[\w.]+\s*$)/m;
+const SHELL_PATTERN =
+  /^\s*(?:\$\s|(?:bun|brew|cargo|cat|cd|cp|curl|docker|echo|export|gh|git|go|grep|kubectl|ls|make|mkdir|mv|npm|npx|pip|pnpm|rm|ssh|sudo|uv|wget|yarn)\s)/;
+
+const LANGUAGE_ALIASES: Record<string, BundledLanguage> = {
+  console: "shellscript",
+  htm: "html",
+  js: "javascript",
+  jsonl: "json",
+  md: "markdown",
+  ndjson: "json",
+  sh: "shellscript",
+  text: "markdown",
+  txt: "markdown",
+  xml: "xml",
+  yml: "yaml",
+};
+
+/** Parse transcript prose as Markdown or JSON while preserving fenced code. */
+export function parseMarkdownBlocks(text: string): CodeBlock[] {
+  const blocks: CodeBlock[] = [];
+  // The info string can hold more than the language, such as `c++` or
+  // `js title="x"`. Only its first word names the language.
+  const fence = /^([ \t]*)```([^\n`]*)\n([\s\S]*?)```/gm;
+  let cursor = 0;
+  let match: RegExpExecArray | null;
+  while ((match = fence.exec(text))) {
+    const prose = text.slice(cursor, match.index).trim();
+    if (prose) blocks.push(proseBlock(prose));
+    const tag = match[2]?.trim().split(/\s+/)[0];
+    const code = stripFenceIndent(match[3] ?? "", match[1]?.length ?? 0);
+    const language = tag ? normalizeLanguage(tag) : guessLanguage(code);
+    blocks.push({
+      code: formatCodeBlock(code, language),
+      fenced: true,
+      language,
+    });
+    cursor = match.index + match[0].length;
+  }
+  const rest = text.slice(cursor).trim();
+  if (rest) blocks.push(proseBlock(rest));
+  return blocks.length > 0 ? blocks : [proseBlock(text)];
+}
+
+function proseBlock(text: string): CodeBlock {
+  const language = detectOutputLanguage(text);
+  return { code: formatCodeBlock(text, language), fenced: false, language };
+}
+
+function normalizeLanguage(tag: string): BundledLanguage {
+  const normalized = tag.toLowerCase();
+  const candidate = LANGUAGE_ALIASES[normalized] ?? normalized;
+  return Object.hasOwn(dashboardCodeLanguages, candidate)
+    ? (candidate as BundledLanguage)
+    : "markdown";
+}
+
+function guessLanguage(code: string): BundledLanguage {
+  if (detectOutputLanguage(code) === "json") return "json";
+  if (SCRIPT_PATTERN.test(code)) return "tsx";
+  if (PYTHON_PATTERN.test(code)) return "python";
+  if (SHELL_PATTERN.test(code)) return "shellscript";
+  return "markdown";
+}
+
+// A fence in a list item is indented, and so are its code lines. Remove that
+// indent, as CommonMark does.
+function stripFenceIndent(code: string, indent: number): string {
+  if (indent === 0) return code;
+  return code
+    .split("\n")
+    .map((line) => {
+      const lead = line.length - line.trimStart().length;
+      return line.slice(Math.min(indent, lead));
+    })
+    .join("\n");
+}
+
+function formatCodeBlock(code: string, language: BundledLanguage): string {
+  return language === "json" ? (prettyJsonData(code) ?? code) : code;
+}

package/src/client/format.ts

@@ -1,5 +1,4 @@
 import type { BundledLanguage } from "shiki/bundle/web";
-import { dashboardCodeLanguages } from "./code-languages";
 import type {
   ActorIdentity,
   ConversationAuxiliaryCosts,
@@ -8,7 +7,6 @@ import type {
   ConversationUsage,
 } from "@sentry/junior/api/schema";
 import type {
-  CodeBlock,
   Conversation,
   ConversationTranscript,
   TranscriptViewMessage,
@@ -815,28 +813,6 @@ export function automationPath(
   return tab === "overview" ? base : `${base}/${tab}`;
 }
 
-function normalizeLanguage(language: string | undefined): BundledLanguage {
-  const normalized = language?.trim().toLowerCase();
-  if (!normalized) return "markdown";
-  const aliases: Record<string, BundledLanguage> = {
-    console: "shellscript",
-    htm: "html",
-    js: "javascript",
-    jsonl: "json",
-    md: "markdown",
-    ndjson: "json",
-    sh: "shellscript",
-    text: "markdown",
-    txt: "markdown",
-    xml: "xml",
-    yml: "yaml",
-  };
-  const candidate = aliases[normalized] ?? normalized;
-  return Object.hasOwn(dashboardCodeLanguages, candidate)
-    ? (candidate as BundledLanguage)
-    : "markdown";
-}
-
 function prettyJson(text: string): string | undefined {
   const trimmed = text.trim();
   if (!trimmed) return undefined;
@@ -863,14 +839,11 @@ function prettyJsonl(text: string): string | undefined {
   return formatted.join("\n");
 }
 
-function prettyJsonData(text: string): string | undefined {
+/** Pretty-print JSON or JSONL text, or return undefined for other text. */
+export function prettyJsonData(text: string): string | undefined {
   return prettyJson(text) ?? prettyJsonl(text);
 }
 
-function formatCodeBlock(code: string, language: BundledLanguage): string {
-  return language === "json" ? (prettyJsonData(code) ?? code) : code;
-}
-
 /**
  * Detect the language for LLM text output prose: json if the text is valid
  * JSON or JSONL, markdown otherwise. Never auto-detects XML, HTML, TypeScript,
@@ -889,44 +862,6 @@ export function detectOutputLanguage(text: string): BundledLanguage {
   return "markdown";
 }
 
-/** Parse transcript prose as Markdown or JSON while preserving fenced code. */
-export function parseMarkdownBlocks(text: string): CodeBlock[] {
-  const blocks: CodeBlock[] = [];
-  const fence = /```([A-Za-z0-9_-]+)?\n([\s\S]*?)```/g;
-  let cursor = 0;
-  let match: RegExpExecArray | null;
-  while ((match = fence.exec(text))) {
-    const prose = text.slice(cursor, match.index).trim();
-    if (prose) {
-      const language = detectOutputLanguage(prose);
-      blocks.push({
-        code: formatCodeBlock(prose, language),
-        fenced: false,
-        language,
-      });
-    }
-    const language = normalizeLanguage(match[1]);
-    blocks.push({
-      code: formatCodeBlock(match[2] ?? "", language),
-      fenced: true,
-      language,
-    });
-    cursor = match.index + match[0].length;
-  }
-  const rest = text.slice(cursor).trim();
-  if (rest) {
-    const language = detectOutputLanguage(rest);
-    blocks.push({
-      code: formatCodeBlock(rest, language),
-      fenced: false,
-      language,
-    });
-  }
-  if (blocks.length > 0) return blocks;
-  const language = detectOutputLanguage(text);
-  return [{ code: formatCodeBlock(text, language), fenced: false, language }];
-}
-
 /** Convert SQL conversation summaries into dashboard rows. */
 export function buildConversations(
   summaries: ConversationSummaryReport[],
@@ -937,6 +872,7 @@ export function buildConversations(
       annotations: summary.annotations,
       sidebarAnnotations: summary.sidebarAnnotations,
       archivedAt: summary.archivedAt,
+      pinnedAt: summary.pinnedAt,
       auxiliaryCosts: summary.auxiliaryCosts,
       channel: summary.channel,
       channelName: summary.channelName,

package/src/client/http.ts

@@ -174,21 +174,21 @@ export async function fetchDashboardJson<T>(
   return schema.parse(await response.json());
 }
 
-/** Read an authenticated response, allowing 304 only for an explicit validator. */
+/** Read an authenticated response, allowing 304 only for explicit validator headers. */
 export async function readDashboardResponse(
   path: string,
   signal?: AbortSignal,
-  etag?: string,
+  validator?: Record<string, string>,
 ): Promise<Response> {
   const response = await fetchDashboard(path, {
     credentials: "same-origin",
     ...(signal ? { signal } : undefined),
-    ...(etag
-      ? { cache: "no-store" as const, headers: { "if-none-match": etag } }
+    ...(validator
+      ? { cache: "no-store" as const, headers: validator }
       : undefined),
   });
   if (response.status === 401) restartDashboardSignIn();
-  if (!response.ok && !(etag && response.status === 304)) {
+  if (!response.ok && !(validator && response.status === 304)) {
     await throwDashboardApiError(path, response);
   }
   return response;

package/src/client/markdownExport.ts

@@ -247,53 +247,43 @@ function appendContextEvent(
   timestamp: number | undefined,
 ): void {
   const event = part.event;
-  lines.push(
-    "",
-    event.type === "handoff" ? "### Model handoff" : "### Context compacted",
-  );
+  lines.push("", "### Context compacted");
   addEventMeta(lines, conversationTranscript, timestamp);
-  if (event.type === "handoff") {
-    addMetaLine(lines, "Model", event.modelId);
-    addMetaLine(lines, "Profile", event.modelProfile);
-    addMetaLine(lines, "Reasoning", event.reasoningLevel);
-  } else {
-    addMetaLine(lines, "Profile", event.modelProfile);
-    addMetaLine(lines, "Model", event.modelId);
-    if (event.details) {
-      addMetaLine(
-        lines,
-        "Estimated input tokens",
-        String(event.details.estimatedInputTokens),
-      );
-      if (event.details.replacementInputTokens !== undefined) {
-        addMetaLine(
-          lines,
-          "Replacement input tokens",
-          String(event.details.replacementInputTokens),
-        );
-      }
-      addMetaLine(
-        lines,
-        "Compaction trigger",
-        String(event.details.triggerTokens),
-      );
-      addMetaLine(lines, "Input limit", String(event.details.inputLimitTokens));
-      addMetaLine(
-        lines,
-        "Input messages",
-        String(event.details.inputMessageCount),
-      );
-      addMetaLine(
-        lines,
-        "Retained messages",
-        String(event.details.retainedMessageCount),
-      );
+  addMetaLine(lines, "Model", event.modelId);
+  if (event.details) {
+    addMetaLine(
+      lines,
+      "Estimated input tokens",
+      String(event.details.estimatedInputTokens),
+    );
+    if (event.details.replacementInputTokens !== undefined) {
       addMetaLine(
         lines,
-        "Summary characters",
-        String(event.details.summaryChars),
+        "Replacement input tokens",
+        String(event.details.replacementInputTokens),
       );
     }
+    addMetaLine(
+      lines,
+      "Compaction trigger",
+      String(event.details.triggerTokens),
+    );
+    addMetaLine(lines, "Input limit", String(event.details.inputLimitTokens));
+    addMetaLine(
+      lines,
+      "Input messages",
+      String(event.details.inputMessageCount),
+    );
+    addMetaLine(
+      lines,
+      "Retained messages",
+      String(event.details.retainedMessageCount),
+    );
+    addMetaLine(
+      lines,
+      "Summary characters",
+      String(event.details.summaryChars),
+    );
   }
   if (event.summary) {
     lines.push("", "#### Continuation summary", "", event.summary);

package/src/client/pages/code/RepositoryTabs.tsx

@@ -16,7 +16,7 @@ import {
 } from "../../components/controls/TimeRangeSelector";
 import { Card } from "../../components/layout/Card";
 import { ConversationHomeList } from "../../conversations/ConversationHomeList";
-import { attentionFirst } from "../../conversations/conversationSections";
+import { pinnedAndAttentionFirst } from "../../conversations/conversationSections";
 import { buildConversations, getDashboardTimeZone } from "../../format";
 import { useDebouncedValue } from "../../debouncedValue";
 import { useWaitingConversationIds } from "../../review-desk/queries";
@@ -54,7 +54,7 @@ export function RepositoryConversationsTab(props: { repositoryId: string }) {
       ) : (
         <ConversationHomeList
           conversations={pageItems(
-            attentionFirst(conversations, waitingConversationIds),
+            pinnedAndAttentionFirst(conversations, waitingConversationIds),
             page,
             PAGE_SIZE,
           )}

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

@@ -1,4 +1,4 @@
-import { Globe2, Lock, Trash2 } from "lucide-react";
+import { Trash2 } from "lucide-react";
 import { useState, type ReactNode } from "react";
 
 import { DetailsPanel } from "../../components/DetailsPanel";
@@ -24,14 +24,8 @@ import type {
   PluginUserPageRecordAction,
 } from "../user/pluginUserPageData";
 
-// The heading, origin, and scope rows already show these values.
-const SUMMARIZED_METADATA = new Set([
-  "Learned",
-  "Memory ID",
-  "Source",
-  "Type",
-  "Visibility",
-]);
+// The heading and origin rows already show these values.
+const SUMMARIZED_METADATA = new Set(["Learned", "Memory ID", "Source", "Type"]);
 
 /** Show one memory beside the library, or in a bottom sheet on small screens. */
 export function MemoryDetails(props: {
@@ -43,8 +37,6 @@ export function MemoryDetails(props: {
   const { record } = props;
   const [confirmOpen, setConfirmOpen] = useState(false);
   const heading = record ? memoryHeading(record) : "Memory details";
-  const isPublic = record && metadataValue(record, "Visibility") === "Public";
-  const VisibilityIcon = isPublic ? Globe2 : Lock;
   const forgetAction = record?.actions?.find(
     (recordAction) => recordAction.tone === "danger",
   );
@@ -56,13 +48,6 @@ export function MemoryDetails(props: {
       sheetTitle={heading}
       title={
         <h2 className="m-0 flex items-center gap-2 font-display text-base font-medium text-dashboard-text">
-          {record ? (
-            <VisibilityIcon
-              aria-hidden="true"
-              className="shrink-0 text-dashboard-text-muted"
-              size={14}
-            />
-          ) : null}
           <span className="truncate">{heading}</span>
         </h2>
       }
@@ -147,15 +132,11 @@ function MemoryFacts(props: { record: PluginUserPageRecord }) {
   const { record } = props;
   const learned = metadataValue(record, "Learned");
   const source = metadataValue(record, "Source");
-  const visibility = metadataValue(record, "Visibility");
-  const isPublic = visibility === "Public";
   const origin =
     learned === "Automatic"
       ? `From a ${source ?? "past"} conversation`
       : learned === "Explicit"
-        ? isPublic
-          ? "Someone asked Junior"
-          : "You asked Junior"
+        ? "Someone asked Junior"
         : learned
           ? "Before origin tracking"
           : undefined;
@@ -169,11 +150,6 @@ function MemoryFacts(props: { record: PluginUserPageRecord }) {
   return (
     <dl className="m-0 grid min-w-0 gap-2 rounded-lg border border-dashboard-border-subtle bg-dashboard-surface-panel p-3 text-xs leading-snug">
       {origin ? <MemoryFact label="Learned">{origin}</MemoryFact> : null}
-      {visibility ? (
-        <MemoryFact label="Used in">
-          {isPublic ? "Future workspace channels" : "Your future conversations"}
-        </MemoryFact>
-      ) : null}
       {dates.map((item) => (
         <MemoryFact key={item.label} label={item.label}>
           {item.value}
@@ -194,12 +170,9 @@ function MemoryFact(props: { children: ReactNode; label: string }) {
   );
 }
 
-/** Name a memory by visibility and type, such as "Private preference". */
+/** Name a memory by type, such as "Preference". */
 function memoryHeading(record: PluginUserPageRecord): string {
-  const kind = metadataValue(record, "Type");
-  const visibility = metadataValue(record, "Visibility");
-  if (!kind) return visibility ? `${visibility} memory` : "Memory";
-  return visibility ? `${visibility} ${kind.toLowerCase()}` : kind;
+  return metadataValue(record, "Type") ?? "Memory";
 }
 
 function metadataValue(

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

@@ -3,8 +3,6 @@ import {
   BrainCircuit,
   CircleAlert,
   Database,
-  Globe2,
-  Lock,
   Sparkles,
   UserRound,
 } from "lucide-react";
@@ -12,7 +10,6 @@ import { useEffect, useRef, useState } from "react";
 import { Navigate, useLocation, useNavigate, useParams } from "react-router";
 import type { PluginUserPageLink } from "@sentry/junior-plugin-api";
 
-import { FilterTabList } from "../../components/FilterBar";
 import { InlineError } from "../../components/InlineError";
 import { PageContentSkeleton } from "../../components/PageContentSkeleton";
 import { LoadMorePagination } from "../../components/Pagination";
@@ -36,7 +33,6 @@ import { cn } from "../../styles";
 import {
   type MemoryDashboardData,
   useMemoryDashboardData,
-  useMemorySearchCounts,
 } from "./memoryDashboard";
 import { MemoryDetails } from "./MemoryDetails";
 import { MemoryPageLayout } from "./MemoryPageLayout";
@@ -47,11 +43,9 @@ import { useMemoryRecord } from "./memoryRecord";
 
 /**
  * Leading memory column flexes; type and date columns stay equal fixed
- * widths on `sm` and up. Below `sm` those columns are hidden, so the grid
- * collapses to just the label and trailing visibility icon.
+ * widths on `sm` and up. Below `sm` those columns are hidden.
  */
-const MEMORY_GRID =
-  "grid-cols-[minmax(0,1fr)_auto] sm:grid-cols-[minmax(0,1fr)_7rem_9rem_auto]";
+const MEMORY_GRID = "grid-cols-1 sm:grid-cols-[minmax(0,1fr)_7rem_9rem]";
 
 /** Render the temporary first-class dashboard experience for memory. */
 export function MemoryPage(props: { page: PluginUserPageLink }) {
@@ -111,7 +105,7 @@ function MemoryOverview(props: { range: TimeRangeDays }) {
             days: dashboardQuery.data.days,
             hours: dashboardQuery.data.hours,
             range: props.range,
-            emptySixHour: (date) => ({ date, personal: 0, public: 0 }),
+            emptySixHour: (date) => ({ count: 0, date }),
           })}
           range={props.range}
         />
@@ -148,21 +142,13 @@ function MemoryLibrary(props: {
   const {
     action,
     content,
-    filter,
     query,
     records,
     runAction,
     searchQuery,
     searchText,
-    setFilter,
     setSearchText,
   } = usePluginUserPageData(props.page);
-  const dashboardQuery = useMemoryDashboardData();
-  const searchCountsQuery = useMemorySearchCounts(searchQuery);
-  // Tab totals follow the search. Without a search, the summary owns them.
-  const counts = searchQuery
-    ? searchCountsQuery.data
-    : dashboardQuery.data?.stats;
   const navigate = useNavigate();
   const location = useLocation();
   const { memoryId } = useParams();
@@ -269,29 +255,6 @@ function MemoryLibrary(props: {
               ) : null}
             </div>
 
-            <FilterTabList
-              ariaLabel="Memory collections"
-              items={[
-                {
-                  count: counts?.active,
-                  label: "All",
-                  value: "",
-                },
-                {
-                  count: counts?.personal,
-                  label: "Private",
-                  value: "private",
-                },
-                {
-                  count: counts?.public,
-                  label: "Public",
-                  value: "public",
-                },
-              ]}
-              onChange={setFilter}
-              value={filter}
-            />
-
             {loading ? (
               <PageContentSkeleton label="Loading memories" variant="panel" />
             ) : query.error ? (
@@ -378,7 +341,6 @@ function MemoryListHeader() {
       <div>Memory</div>
       <div>Type</div>
       <div>Learned</div>
-      <div className="sr-only">Visibility</div>
     </div>
   );
 }
@@ -387,20 +349,10 @@ function MemorySummary(props: { data: MemoryDashboardData }) {
   const { stats } = props.data;
   const items = [
     {
-      detail: "personal + public",
+      detail: "shared across the workspace",
       label: "Total active",
-      value: stats.active.toLocaleString("en-US"),
-    },
-    {
-      detail: "private to you",
-      label: "Personal",
-      value: stats.personal.toLocaleString("en-US"),
-    },
-    {
-      detail: "workspace shareable",
-      label: "Public",
       tone: "text-dashboard-accent-text",
-      value: stats.public.toLocaleString("en-US"),
+      value: stats.active.toLocaleString("en-US"),
     },
     {
       detail: "in the last 30 days",
@@ -411,7 +363,7 @@ function MemorySummary(props: { data: MemoryDashboardData }) {
   return (
     <section
       aria-label="Memory summary"
-      className="grid grid-cols-2 gap-px border-y border-dashboard-border-subtle bg-dashboard-fill-hover lg:grid-cols-4"
+      className="grid grid-cols-2 gap-px border-y border-dashboard-border-subtle bg-dashboard-fill-hover"
     >
       {items.map((item) => (
         <div className="bg-dashboard-bg px-4 py-4 sm:px-5" key={item.label}>
@@ -442,12 +394,9 @@ function MemoryKindPanel(props: { data: MemoryDashboardData }) {
       <h2 className="m-0 font-display text-xl font-medium text-dashboard-text">
         What Junior remembers
       </h2>
-      <p className="mt-1 mb-0 font-sans text-xs leading-relaxed text-dashboard-text-muted">
-        Across personal and public scopes.
-      </p>
       <div className="mt-5 grid gap-px overflow-hidden rounded-md border border-dashboard-border-subtle bg-dashboard-fill-hover">
         <OverviewBreakdownRow
-          detail="Almost entirely personal"
+          detail="How people like to work"
           icon={UserRound}
           label="Preferences"
           value={stats.preference}
@@ -541,8 +490,6 @@ function MemoryRow(props: {
   const kind = metadataValue(props.record, "Type");
   const remembered = metadataValue(props.record, "Remembered");
   const source = metadataValue(props.record, "Source");
-  const visibility = metadataValue(props.record, "Visibility");
-  const isPublic = visibility === "Public";
   return (
     <SelectableRow
       className={cn(
@@ -565,11 +512,6 @@ function MemoryRow(props: {
       >
         <div className="min-w-0 overflow-hidden">
           <div className="flex min-w-0 items-center gap-2 overflow-hidden">
-            <VisibilityIcon
-              className="sm:hidden"
-              isPublic={isPublic}
-              size={14}
-            />
             <h3 className="m-0 truncate font-display text-base font-medium leading-snug text-dashboard-text">
               <MemorySearchHighlight
                 query={props.searchQuery}
@@ -591,38 +533,11 @@ function MemoryRow(props: {
         <span className="hidden truncate font-mono text-xs text-dashboard-text sm:block">
           {shortDate(remembered)}
         </span>
-        <VisibilityIcon
-          className="hidden justify-self-center sm:inline-flex"
-          isPublic={isPublic}
-          size={15}
-        />
       </button>
     </SelectableRow>
   );
 }
 
-/** Render the lock (private) or globe (public) mark for one memory's visibility. */
-function VisibilityIcon(props: {
-  className?: string;
-  isPublic: boolean;
-  size: number;
-}) {
-  const Icon = props.isPublic ? Globe2 : Lock;
-  const label = props.isPublic ? "Public" : "Private";
-  return (
-    <span
-      className={cn(
-        "inline-flex shrink-0 items-center text-dashboard-text-muted",
-        props.className,
-      )}
-      title={label}
-    >
-      <Icon aria-hidden="true" size={props.size} />
-      <span className="sr-only">{label}</span>
-    </span>
-  );
-}
-
 function metadataValue(record: PluginUserPageRecord, label: string): string {
   return record.metadata?.find((item) => item.label === label)?.value ?? "—";
 }