@sentry/junior-dashboard 0.91.0 → 0.92.0

package/src/client/pages/PeoplePage.tsx

@@ -2,7 +2,7 @@ import { useState } from "react";
 import { Link, useParams } from "react-router";
 import { Mail, UserRound } from "lucide-react";
 
-import { useRequesterDirectoryData, useRequesterProfileData } from "../api";
+import { useActorDirectoryData, useActorProfileData } from "../api";
 import { Button } from "../components/Button";
 import { ConversationList } from "../components/ConversationList";
 import { ConversationSearchInput } from "../components/ConversationListControls";
@@ -23,37 +23,31 @@ import {
 import { cn } from "../styles";
 import type {
   ConversationStatsItem,
-  RequesterActivityDay,
-  RequesterDirectory,
-  RequesterProfile,
-  RequesterSummary,
-  RequesterTotals,
+  ActorActivityDay,
+  ActorDirectory,
+  ActorProfile,
+  ActorSummary,
+  ActorTotals,
 } from "../types";
 
 type PeopleSort = "activeDays" | "conversations" | "recent" | "runtime";
 
-function requesterName(person: Pick<RequesterSummary, "requester">): string {
+function actorName(person: Pick<ActorSummary, "actor">): string {
   return (
-    person.requester.fullName ??
-    person.requester.slackUserName ??
-    person.requester.email
+    person.actor.fullName ?? person.actor.slackUserName ?? person.actor.email
   );
 }
 
-function personMeta(person: RequesterSummary): string {
+function personMeta(person: ActorSummary): string {
   const pieces = [
-    person.requester.email,
-    person.requester.slackUserName
-      ? `@${person.requester.slackUserName}`
-      : undefined,
+    person.actor.email,
+    person.actor.slackUserName ? `@${person.actor.slackUserName}` : undefined,
     `last ${formatRelativeTime(person.lastSeenAt)}`,
   ];
   return pieces.filter(Boolean).join(" / ");
 }
 
-function sampleLabel(
-  data: Pick<RequesterDirectory, "sampleSize" | "truncated">,
-) {
+function sampleLabel(data: Pick<ActorDirectory, "sampleSize" | "truncated">) {
   return `${formatCompactNumber(data.sampleSize)} sampled conversations${
     data.truncated ? " / limited sample" : ""
   }`;
@@ -64,18 +58,18 @@ function runtimeLabel(durationMs: number, conversations: number): string {
   return formatMs(durationMs);
 }
 
-/** Render the requester directory from dashboard reporting data. */
+/** Render the actor directory from dashboard reporting data. */
 export function PeoplePage() {
-  const query = useRequesterDirectoryData();
+  const query = useActorDirectoryData();
   return <PeoplePageContent data={query.data} error={query.error} />;
 }
 
-/** Render loaded, failed, and empty requester directory states. */
+/** Render loaded, failed, and empty actor directory states. */
 export function PeoplePageContent({
   data,
   error,
 }: {
-  data: RequesterDirectory | undefined;
+  data: ActorDirectory | undefined;
   error: unknown;
 }) {
   const [peopleSearch, setPeopleSearch] = useState("");
@@ -92,7 +86,7 @@ export function PeoplePageContent({
             <SectionTitle>People</SectionTitle>
             <div className="mt-1 break-words text-[0.82rem] leading-relaxed text-[#b8b8b8]">
               {data
-                ? `${people.length} of ${data.people.length} requesters / ${sampleLabel(data)}`
+                ? `${people.length} of ${data.people.length} actors / ${sampleLabel(data)}`
                 : "People failed to load"}
             </div>
           </div>
@@ -116,7 +110,7 @@ export function PeoplePageContent({
         ) : (
           <div className="p-3">
             <EmptyTelemetry>
-              No requester telemetry with trusted email.
+              No actor telemetry with trusted email.
             </EmptyTelemetry>
           </div>
         )}
@@ -126,18 +120,18 @@ export function PeoplePageContent({
 }
 
 function filterPeople(
-  people: RequesterSummary[],
+  people: ActorSummary[],
   query: string,
   sort: PeopleSort,
-): RequesterSummary[] {
+): ActorSummary[] {
   const normalized = query.trim().toLowerCase();
   const filtered = normalized
     ? people.filter((person) =>
         [
-          person.requester.email,
-          person.requester.fullName,
-          person.requester.slackUserId,
-          person.requester.slackUserName,
+          person.actor.email,
+          person.actor.fullName,
+          person.actor.slackUserId,
+          person.actor.slackUserName,
         ]
           .filter(Boolean)
           .join(" ")
@@ -196,7 +190,7 @@ function PeopleToolbar(props: {
   );
 }
 
-function PeopleDirectory(props: { people: RequesterSummary[] }) {
+function PeopleDirectory(props: { people: ActorSummary[] }) {
   if (props.people.length === 0) {
     return (
       <div className="p-3">
@@ -210,7 +204,7 @@ function PeopleDirectory(props: { people: RequesterSummary[] }) {
         className="sticky top-0 z-[1] grid grid-cols-[minmax(14rem,1fr)_repeat(3,minmax(5rem,auto))] items-center gap-3 border-b border-white/10 bg-[#050505] px-3 py-2 text-[0.76rem] font-semibold uppercase leading-none text-[#888] max-md:hidden"
         role="row"
       >
-        <div>Requester</div>
+        <div>Actor</div>
         <div className="justify-self-end">Conversations</div>
         <div className="justify-self-end">Active days</div>
         <div className="justify-self-end">Runtime</div>
@@ -218,8 +212,8 @@ function PeopleDirectory(props: { people: RequesterSummary[] }) {
       {props.people.map((person) => (
         <Link
           className="grid min-w-0 grid-cols-[minmax(14rem,1fr)_repeat(3,minmax(5rem,auto))] items-center gap-3 border-b border-l-4 border-b-white/10 border-l-[#22a06b] bg-[#0b0b0b] px-3 py-3 text-inherit no-underline transition-colors hover:bg-[#151515] max-md:grid-cols-1 max-md:px-4 max-md:py-4"
-          key={person.requester.email}
-          to={peoplePath(person.requester.email)}
+          key={person.actor.email}
+          to={peoplePath(person.actor.email)}
         >
           <div className="min-w-0">
             <div className="flex min-w-0 items-center gap-2">
@@ -228,7 +222,7 @@ function PeopleDirectory(props: { people: RequesterSummary[] }) {
               </span>
               <div className="min-w-0">
                 <div className="truncate text-[1.02rem] font-bold leading-tight text-white">
-                  {requesterName(person)}
+                  {actorName(person)}
                 </div>
                 <div className="mt-1 truncate text-[0.82rem] leading-tight text-[#b8b8b8]">
                   {personMeta(person)}
@@ -255,11 +249,11 @@ function DirectoryNumber(props: { value: number }) {
   );
 }
 
-/** Render one requester's profile and recent conversation history. */
+/** Render one actor's profile and recent conversation history. */
 export function PersonProfilePage() {
   const params = useParams();
   const email = params.email ? decodeURIComponent(params.email) : undefined;
-  const query = useRequesterProfileData(email);
+  const query = useActorProfileData(email);
   if (!query.data && !query.error) {
     return <LoadingView label="Loading profile" />;
   }
@@ -279,7 +273,7 @@ export function PersonProfilePage() {
   );
 }
 
-export function Profile(props: { profile: RequesterProfile }) {
+export function Profile(props: { profile: ActorProfile }) {
   const profile = props.profile;
   const [recentSearch, setRecentSearch] = useState("");
   const conversations = buildConversations(profile.recentConversations);
@@ -297,14 +291,14 @@ export function Profile(props: { profile: RequesterProfile }) {
               </span>
               <div className="min-w-0">
                 <h2 className="m-0 truncate text-2xl font-extrabold leading-tight tracking-normal text-white">
-                  {profile.requester.fullName ??
-                    profile.requester.slackUserName ??
-                    profile.requester.email}
+                  {profile.actor.fullName ??
+                    profile.actor.slackUserName ??
+                    profile.actor.email}
                 </h2>
                 <div className="mt-1 break-words text-[0.88rem] leading-relaxed text-[#b8b8b8]">
-                  {profile.requester.email}
-                  {profile.requester.slackUserName
-                    ? ` / @${profile.requester.slackUserName}`
+                  {profile.actor.email}
+                  {profile.actor.slackUserName
+                    ? ` / @${profile.actor.slackUserName}`
                     : ""}
                 </div>
               </div>
@@ -368,7 +362,7 @@ export function Profile(props: { profile: RequesterProfile }) {
   );
 }
 
-function ProfileMetrics(props: { totals: RequesterTotals }) {
+function ProfileMetrics(props: { totals: ActorTotals }) {
   const totals = props.totals;
   const metrics = [
     ["conversations", formatCompactNumber(totals.conversations)],
@@ -395,22 +389,19 @@ function ProfileMetrics(props: { totals: RequesterTotals }) {
   );
 }
 
-function ContributionGrid(props: { days: RequesterActivityDay[] }) {
+function ContributionGrid(props: { days: ActorActivityDay[] }) {
   const max = Math.max(1, ...props.days.map((day) => day.conversations));
-  const weeks: Array<Array<RequesterActivityDay | null>> = [];
+  const weeks: Array<Array<ActorActivityDay | null>> = [];
   const leadingDays = props.days[0]
     ? new Date(`${props.days[0].date}T00:00:00Z`).getUTCDay()
     : 0;
-  const cells: Array<RequesterActivityDay | null> = [
+  const cells: Array<ActorActivityDay | null> = [
     ...Array.from({ length: leadingDays }, () => null),
     ...props.days,
   ];
   while (cells.length > 0 && cells.length % 7 !== 0) cells.push(null);
   for (let index = 0; index < cells.length; index += 7) {
-    const week: Array<RequesterActivityDay | null> = cells.slice(
-      index,
-      index + 7,
-    );
+    const week: Array<ActorActivityDay | null> = cells.slice(index, index + 7);
     while (week.length < 7) week.push(null);
     weeks.push(week);
   }

package/src/client/types.ts

@@ -16,14 +16,14 @@ import type {
   SkillReport,
 } from "@sentry/junior/reporting";
 import type {
-  RequesterDirectoryReport,
-  RequesterIdentity as ApiRequesterIdentity,
-  RequesterSummaryReport,
-  RequesterTotalsReport,
+  ActorDirectoryReport,
+  ActorIdentity as ApiActorIdentity,
+  ActorSummaryReport,
+  ActorTotalsReport,
 } from "@sentry/junior/api/people/list";
 import type {
-  RequesterActivityDayReport,
-  RequesterProfileReport,
+  ActorActivityDayReport,
+  ActorProfileReport,
 } from "@sentry/junior/api/people/profile";
 
 export type Health = HealthReport;
@@ -45,17 +45,17 @@ export type ConversationSubagentTranscript =
 
 export type ConversationStatsItem = ReportingConversationStatsItem;
 
-export type RequesterIdentity = ApiRequesterIdentity;
+export type ActorIdentity = ApiActorIdentity;
 
-export type RequesterActivityDay = RequesterActivityDayReport;
+export type ActorActivityDay = ActorActivityDayReport;
 
-export type RequesterDirectory = RequesterDirectoryReport;
+export type ActorDirectory = ActorDirectoryReport;
 
-export type RequesterProfile = RequesterProfileReport;
+export type ActorProfile = ActorProfileReport;
 
-export type RequesterSummary = RequesterSummaryReport;
+export type ActorSummary = ActorSummaryReport;
 
-export type RequesterTotals = RequesterTotalsReport;
+export type ActorTotals = ActorTotalsReport;
 
 export type TurnUsage = ConversationUsage;
 
@@ -136,7 +136,7 @@ export type Conversation = {
   id: string;
   lastProgressAt: string;
   lastSeenAt: string;
-  requesterIdentity?: RequesterIdentity;
+  actorIdentity?: ActorIdentity;
   sentryTraceUrl?: string;
   startedAt: string;
   status: ConversationSummary["status"];

package/src/mock-conversations.ts

@@ -3,7 +3,7 @@ import type {
   ConversationStatsItem as DashboardConversationStatsItem,
   ConversationStatsReport as DashboardConversationStatsReport,
   ConversationSubagentTranscriptReport as DashboardConversationSubagentTranscriptReport,
-  RequesterIdentity as DashboardRequesterIdentity,
+  ActorIdentity as DashboardActorIdentity,
   ConversationFeed as DashboardConversationFeed,
   ConversationSummaryReport as DashboardConversationSummary,
   ConversationUsage as DashboardRunUsage,
@@ -90,7 +90,7 @@ function publicIncidentConversation(
           totalTokens: 9700,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           email: "avery@sentry.io",
           fullName: "Avery Stone",
           slackUserId: "UQA111",
@@ -189,7 +189,7 @@ function publicIncidentConversation(
           totalTokens: 9250,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           email: "morgan@sentry.io",
           fullName: "Morgan Lee",
           slackUserId: "UQA222",
@@ -284,7 +284,7 @@ function activeConversation(nowMs: number): DashboardConversationReport {
           totalTokens: 8420,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           email: "sam@sentry.io",
           fullName: "Sam Rivera",
           slackUserId: "UQA333",
@@ -355,7 +355,7 @@ function privateConversation(nowMs: number): DashboardConversationReport {
           totalTokens: 3540,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           email: "private-user@sentry.io",
           slackUserId: "UQA444",
           slackUserName: "private-user",
@@ -462,7 +462,7 @@ function hungConversation(nowMs: number): DashboardConversationReport {
           totalTokens: 12_000,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           email: "dana@sentry.io",
           fullName: "Dana Chen",
           slackUserId: "UQA555",
@@ -540,7 +540,7 @@ function failedConversation(nowMs: number): DashboardConversationReport {
           totalTokens: 4890,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           email: "riley@sentry.io",
           fullName: "Riley Patel",
           slackUserId: "UQA666",
@@ -774,7 +774,7 @@ function dashboardQaConversation(nowMs: number): DashboardConversationReport {
                 name: "advisor",
                 input: {
                   question:
-                    "Review the dashboard plan before editing. Focus on whether requester email can be trusted, what profile metrics are useful, and what UI risks to avoid.",
+                    "Review the dashboard plan before editing. Focus on whether actor email can be trusted, what profile metrics are useful, and what UI risks to avoid.",
                 },
               }),
             ],
@@ -789,7 +789,7 @@ function dashboardQaConversation(nowMs: number): DashboardConversationReport {
                 output: {
                   verdict: "proceed",
                   summary:
-                    "Use trusted requesterIdentity.email, keep metrics to conversations/runtime/tokens, and make profile activity scannable before adding heavier analytics.",
+                    "Use trusted actorIdentity.email, keep metrics to conversations/runtime/tokens, and make profile activity scannable before adding heavier analytics.",
                 },
               }),
             ],
@@ -823,7 +823,7 @@ function dashboardQaConversation(nowMs: number): DashboardConversationReport {
                     },
                   ],
                   summary:
-                    "Add requester activity grid, recent conversations, and email profile links.",
+                    "Add actor activity grid, recent conversations, and email profile links.",
                 },
               }),
             ],
@@ -846,7 +846,7 @@ function dashboardQaConversation(nowMs: number): DashboardConversationReport {
                   risks: {
                     auth: "dashboard routes remain authenticated",
                     privacy:
-                      "requester emails are trusted from normalized reporting identity",
+                      "actor emails are trusted from normalized reporting identity",
                   },
                 },
               }),
@@ -911,7 +911,7 @@ function dashboardQaConversation(nowMs: number): DashboardConversationReport {
             parts: [
               {
                 type: "text",
-                text: "Implemented the people profile route, linked requester emails, and tightened the dashboard widgets based on the advisor review.",
+                text: "Implemented the people profile route, linked actor emails, and tightened the dashboard widgets based on the advisor review.",
               },
             ],
           }),
@@ -925,7 +925,7 @@ function dashboardQaConversation(nowMs: number): DashboardConversationReport {
             status: "completed",
             args: {
               question:
-                "Review the dashboard plan before editing. Focus on whether requester email can be trusted, what profile metrics are useful, and what UI risks to avoid.",
+                "Review the dashboard plan before editing. Focus on whether actor email can be trusted, what profile metrics are useful, and what UI risks to avoid.",
             },
             subagents: [
               mockSubagentActivity({
@@ -1014,7 +1014,7 @@ function dashboardQaAdvisorTranscript(
       parts: [
         {
           type: "text",
-          text: "Review the dashboard plan before editing. Focus on whether requester email can be trusted, what profile metrics are useful, and what UI risks to avoid.",
+          text: "Review the dashboard plan before editing. Focus on whether actor email can be trusted, what profile metrics are useful, and what UI risks to avoid.",
         },
       ],
     }),
@@ -1024,7 +1024,7 @@ function dashboardQaAdvisorTranscript(
       parts: [
         {
           type: "text",
-          text: "Requester identity email is a reasonable profile key because reporting already normalizes trusted identities. Keep the first cut narrow: total conversations, runtime, token volume, recent conversations, and a contribution-style activity grid. Avoid attention widgets until there is an explicit operator workflow.",
+          text: "Actor identity email is a reasonable profile key because reporting already normalizes trusted identities. Keep the first cut narrow: total conversations, runtime, token volume, recent conversations, and a contribution-style activity grid. Avoid attention widgets until there is an explicit operator workflow.",
         },
       ],
     }),
@@ -1137,7 +1137,7 @@ function conversationStatsReportFromSummaries(
   summaries: DashboardConversationSummary[],
 ): DashboardConversationStatsReport {
   const conversations = recentConversationGroups(nowMs, summaries);
-  const requesters = new Map<string, DashboardConversationStatsItem>();
+  const actors = new Map<string, DashboardConversationStatsItem>();
   const locations = new Map<string, DashboardConversationStatsItem>();
   let durationMs = 0;
   let tokens: number | undefined;
@@ -1155,29 +1155,25 @@ function conversationStatsReportFromSummaries(
     failed += signals.failed ? 1 : 0;
     hung += signals.hung ? 1 : 0;
 
-    const requesterRuns = new Map<string, RunContribution[]>();
+    const actorRuns = new Map<string, RunContribution[]>();
     for (const contribution of contributions) {
-      const requester =
-        requesterLabel(contribution.run.requesterIdentity) ?? "Unknown";
-      requesterRuns.set(requester, [
-        ...(requesterRuns.get(requester) ?? []),
-        contribution,
-      ]);
+      const actor = actorLabel(contribution.run.actorIdentity) ?? "Unknown";
+      actorRuns.set(actor, [...(actorRuns.get(actor) ?? []), contribution]);
     }
 
-    for (const [requester, requesterContributions] of requesterRuns) {
-      const item = requesters.get(requester) ?? emptyStatsItem(requester);
-      const requesterSignals = statusSignals(
-        requesterContributions.map((contribution) => contribution.run),
+    for (const [actor, actorContributions] of actorRuns) {
+      const item = actors.get(actor) ?? emptyStatsItem(actor);
+      const actorSignals = statusSignals(
+        actorContributions.map((contribution) => contribution.run),
       );
       item.conversations += 1;
-      item.runs += requesterContributions.length;
-      item.durationMs += contributionDurationTotal(requesterContributions);
-      item.active += requesterSignals.active ? 1 : 0;
-      item.failed += requesterSignals.failed ? 1 : 0;
-      item.hung += requesterSignals.hung ? 1 : 0;
-      addItemTokens(item, contributionTokenTotal(requesterContributions));
-      requesters.set(requester, item);
+      item.runs += actorContributions.length;
+      item.durationMs += contributionDurationTotal(actorContributions);
+      item.active += actorSignals.active ? 1 : 0;
+      item.failed += actorSignals.failed ? 1 : 0;
+      item.hung += actorSignals.hung ? 1 : 0;
+      addItemTokens(item, contributionTokenTotal(actorContributions));
+      actors.set(actor, item);
     }
 
     const location = locationLabel(newestRun(runs));
@@ -1200,7 +1196,7 @@ function conversationStatsReportFromSummaries(
     generatedAt: iso(nowMs),
     hung,
     locations: statsItems(locations),
-    requesters: statsItems(requesters),
+    actors: statsItems(actors),
     sampleLimit: summaries.length,
     sampleSize: summaries.length,
     source: "conversation_index",
@@ -1332,13 +1328,13 @@ function contributionTokenTotal(
   );
 }
 
-function requesterLabel(
-  requester: DashboardRequesterIdentity | undefined,
+function actorLabel(
+  actor: DashboardActorIdentity | undefined,
 ): string | undefined {
-  const email = requester?.email?.trim() || undefined;
-  const fullName = requester?.fullName?.trim() || undefined;
-  const slackUserName = requester?.slackUserName?.trim() || undefined;
-  return email ?? fullName ?? slackUserName ?? requester?.slackUserId;
+  const email = actor?.email?.trim() || undefined;
+  const fullName = actor?.fullName?.trim() || undefined;
+  const slackUserName = actor?.slackUserName?.trim() || undefined;
+  return email ?? fullName ?? slackUserName ?? actor?.slackUserId;
 }
 
 function locationLabel(turn: DashboardConversationSummary): string {

package/src/mock-release-conversation.ts

@@ -34,7 +34,7 @@ function mockSystemPrompt(): string {
     "",
     "- Default application repo: acme/junior-demo.",
     "- Default package namespace: @acme/junior.",
-    "- Open pull requests as drafts unless the requester asks for ready review.",
+    "- Open pull requests as drafts unless the actor asks for ready review.",
     "",
     "# Slack Output",
     "",
@@ -555,7 +555,7 @@ export function longReleaseConversation(
           outputTokens: 1037,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           fullName: "Jordan Blake",
           slackUserId: "UQA777",
           slackUserName: "jordan",
@@ -585,7 +585,7 @@ export function longReleaseConversation(
           outputTokens: 5765,
         },
         surface: "slack",
-        requesterIdentity: {
+        actorIdentity: {
           fullName: "Jordan Blake",
           slackUserId: "UQA777",
           slackUserName: "jordan",

package/src/mock-reporting/conversation.ts

@@ -5,7 +5,7 @@ import type {
   ConversationRunReport,
   ConversationSurface,
   ConversationUsage,
-  RequesterIdentity,
+  ActorIdentity,
   TranscriptMessage,
 } from "@sentry/junior/reporting";
 
@@ -24,7 +24,7 @@ export type MockRunOptions = {
   id?: string;
   lastProgressAt?: string;
   lastSeenAt?: string;
-  requesterIdentity?: RequesterIdentity;
+  actorIdentity?: ActorIdentity;
   sentryTraceUrl?: string;
   startedAt?: string;
   status?: ConversationReportStatus;
@@ -64,8 +64,8 @@ export function mockRun(options: MockRunOptions = {}): ConversationRunReport {
     ...(options.cumulativeUsage !== undefined
       ? { cumulativeUsage: options.cumulativeUsage }
       : {}),
-    ...(options.requesterIdentity !== undefined
-      ? { requesterIdentity: options.requesterIdentity }
+    ...(options.actorIdentity !== undefined
+      ? { actorIdentity: options.actorIdentity }
       : {}),
     ...(options.sentryTraceUrl !== undefined
       ? { sentryTraceUrl: options.sentryTraceUrl }