fix(webapp,dashboard-agent): wake quotes the right note and links its subject; 12px chat code blocks

Customize kept the recommendation's original note across condition/threshold
changes, so the wake quoted a condition the user never watched — the note is
now restated whenever the condition or its number changes. The wake prompt
also hands the model a ready trigger:// markdown link to the watched object.
Chat code examples drop to 12px.

Author: kathiekiwi

apps/webapp/app/components/dashboard-agent/watch-card.test.ts

@@ -131,17 +131,26 @@ describe("condition variants (§3)", () => {
     expect(variantsOf(watchDraftFor(healthWatchRecommendation("warn")))).toHaveLength(1);
   });
 
-  it("carries the subject, window and note across a swap", () => {
+  it("carries the subject and window across a swap, and restates the note", () => {
     const draft = withWindow(runDraft(), 6);
     const failed = withVariant(draft, "run_failed");
+    // The note is "why the user asked for it" and the wake quotes it — words
+    // describing the OLD condition must not survive a condition change.
     expect(failed.spec).toMatchObject({
       kind: "run_failed",
       runId: "run_abc123",
       maxHours: 6,
-      note: draft.spec.note,
+      note: "tell me if run run_abc123 fails",
     });
   });
 
+  it("restates the note when the threshold number changes", () => {
+    const above = withThreshold(withVariant(queueDraft(), "queue_depth_above"), 500);
+    expect(above.spec.note).toBe("tell me if the email-sends queue grows above 500");
+    const age = withAgeMinutes(withVariant(queueDraft(), "queue_oldest_age"), 90);
+    expect(age.spec.note).toBe("tell me if runs in email-sends wait longer than 90 minutes");
+  });
+
   it("gives the threshold variant a usable default", () => {
     const above = withVariant(queueDraft(), "queue_depth_above");
     expect(above.spec).toMatchObject({ kind: "queue_depth_above", queue: "email-sends" });

apps/webapp/app/components/dashboard-agent/watch-card.ts

@@ -45,8 +45,48 @@ export function clampCadence(kind: WatchKind, minutes: number): number {
   return options.find((option) => option >= minutes) ?? options[options.length - 1]!;
 }
 
-/** Swap the condition for its sibling variant (§3), carrying everything else. */
+/**
+ * The note, restated from the spec. The note is "why the user asked for it" and
+ * the wake narration quotes it — so once the user CHANGES the condition (or its
+ * number), the original words no longer describe the watch and must be rewritten.
+ * Edits that keep the condition (window, cadence) keep the user's own words.
+ */
+export function noteFor(spec: WatchSpec): string {
+  switch (spec.kind) {
+    case "run_start":
+      return `tell me when run ${spec.runId} starts`;
+    case "run_finished":
+      return `tell me when run ${spec.runId} finishes`;
+    case "run_failed":
+      return `tell me if run ${spec.runId} fails`;
+    case "backlog_drain":
+      return `tell me when the ${spec.queue} queue drains`;
+    case "queue_depth_above":
+      return `tell me if the ${spec.queue} queue grows above ${spec.threshold}`;
+    case "queue_depth_below":
+      return `tell me when the ${spec.queue} queue is back below ${spec.threshold}`;
+    case "queue_stalled":
+      return `tell me if the ${spec.queue} queue stops moving`;
+    case "queue_oldest_age":
+      return `tell me if runs in ${spec.queue} wait longer than ${spec.thresholdMinutes} minutes`;
+    case "error_recurrence":
+      return `ping me if error ${spec.fingerprint} happens again`;
+    case "health_recovery":
+      return "tell me when health is back to normal";
+  }
+}
+
+/**
+ * Swap the condition for its sibling variant (§3), carrying everything else —
+ * except the note, which is restated to describe the NEW condition.
+ */
 export function withVariant(draft: WatchDraft, kind: WatchKind): WatchDraft {
+  const next = variantSpec(draft, kind);
+  if (next === draft.spec) return draft;
+  return { ...draft, spec: { ...next, note: noteFor(next) } as WatchSpec };
+}
+
+function variantSpec(draft: WatchDraft, kind: WatchKind): WatchSpec {
   const { spec } = draft;
   const common = {
     note: spec.note,
@@ -59,36 +99,36 @@ export function withVariant(draft: WatchDraft, kind: WatchKind): WatchDraft {
     case "run_failed":
     case "run_start": {
       const runId = "runId" in spec ? spec.runId : "";
-      return { ...draft, spec: { ...common, kind, runId } as WatchSpec };
+      return { ...common, kind, runId } as WatchSpec;
     }
     case "backlog_drain": {
       const queue = "queue" in spec ? spec.queue : "";
-      return { ...draft, spec: { ...common, kind, queue } as WatchSpec };
+      return { ...common, kind, queue } as WatchSpec;
     }
     case "queue_depth_above":
     case "queue_depth_below": {
       const queue = "queue" in spec ? spec.queue : "";
       // The number carries across the two threshold questions: someone who typed
       // 500 for "above" means the same 500 when they flip to "back below".
       const threshold = "threshold" in spec ? spec.threshold : WATCH_DEFAULT_QUEUE_THRESHOLD;
-      return { ...draft, spec: { ...common, kind, queue, threshold } as WatchSpec };
+      return { ...common, kind, queue, threshold } as WatchSpec;
     }
     case "queue_stalled": {
       const queue = "queue" in spec ? spec.queue : "";
       // K is not user-facing in this iteration (§3): the default is the product
       // decision, and the card never shows a field for it.
       const ticks = "ticks" in spec ? spec.ticks : WATCH_STALL_TICKS_DEFAULT;
-      return { ...draft, spec: { ...common, kind, queue, ticks } as WatchSpec };
+      return { ...common, kind, queue, ticks } as WatchSpec;
     }
     case "queue_oldest_age": {
       const queue = "queue" in spec ? spec.queue : "";
       const thresholdMinutes =
         "thresholdMinutes" in spec ? spec.thresholdMinutes : WATCH_DEFAULT_QUEUE_AGE_MINUTES;
-      return { ...draft, spec: { ...common, kind, queue, thresholdMinutes } as WatchSpec };
+      return { ...common, kind, queue, thresholdMinutes } as WatchSpec;
     }
     // The kinds with no second question keep the draft untouched.
     default:
-      return draft;
+      return draft.spec;
   }
 }
 
@@ -125,7 +165,9 @@ export function withThreshold(draft: WatchDraft, threshold: number): WatchDraft
   if (draft.spec.kind !== "queue_depth_above" && draft.spec.kind !== "queue_depth_below") {
     return draft;
   }
-  return { ...draft, spec: { ...draft.spec, threshold } };
+  // The note quotes the number, so a new number restates the note.
+  const spec = { ...draft.spec, threshold };
+  return { ...draft, spec: { ...spec, note: noteFor(spec) } };
 }
 
 /**
@@ -134,7 +176,9 @@ export function withThreshold(draft: WatchDraft, threshold: number): WatchDraft
  */
 export function withAgeMinutes(draft: WatchDraft, thresholdMinutes: number): WatchDraft {
   if (draft.spec.kind !== "queue_oldest_age") return draft;
-  return { ...draft, spec: { ...draft.spec, thresholdMinutes } };
+  // The note quotes the number, so a new number restates the note.
+  const spec = { ...draft.spec, thresholdMinutes };
+  return { ...draft, spec: { ...spec, note: noteFor(spec) } };
 }
 
 /**

apps/webapp/app/tailwind.css

@@ -746,7 +746,8 @@
     @apply scrollbar-thin scrollbar-track-transparent scrollbar-thumb-charcoal-600;
   }
   & [data-code-block] pre code {
-    @apply bg-transparent;
+    /* 12px, chat-only: code examples read denser than the 14px prose around them. */
+    @apply bg-transparent text-xs;
   }
   & [data-code-block] .line {
     @apply leading-relaxed;

internal-packages/dashboard-agent/src/dashboard-agent.ts

@@ -351,6 +351,7 @@ async function generateAndSaveTitle(
 
 import {
   agentPageContextSchema,
+  formatTriggerUri,
   investigationStateSchema,
   isWatchKind,
   resolveWatchResult,
@@ -593,7 +594,48 @@ function investigationInstruction(action: WatchWakeAction): string {
   )}.`;
 }
 
-function wakePrompt(action: WatchWakeAction): string {
+/**
+ * The watched object as a `trigger://` markdown link the narration can embed —
+ * the wake runs with no tools, so the link has to be handed to it ready-made.
+ * Needs the tenancy from the wake's metadata; without it there is no link.
+ */
+function wakeSubjectLink(
+  action: WatchWakeAction,
+  tenancy: { projectRef?: string; environmentId?: string } | undefined
+): string | undefined {
+  const projectRef = tenancy?.projectRef;
+  const environmentId = tenancy?.environmentId;
+  if (!projectRef || !environmentId) return undefined;
+
+  const spec = action.spec;
+  const target =
+    "queue" in spec && spec.queue
+      ? { kind: "queue" as const, projectRef, environmentId, name: spec.queue }
+      : "runId" in spec && spec.runId
+        ? { kind: "run" as const, projectRef, environmentId, runId: spec.runId }
+        : "fingerprint" in spec && spec.fingerprint
+          ? { kind: "error" as const, projectRef, environmentId, fingerprint: spec.fingerprint }
+          : "report" in spec && spec.report
+            ? { kind: "report" as const, projectRef, environmentId, key: spec.report }
+            : undefined;
+  if (!target) return undefined;
+
+  const label =
+    target.kind === "queue"
+      ? target.name
+      : target.kind === "run"
+        ? target.runId
+        : target.kind === "error"
+          ? "this error"
+          : "the report";
+  return `[${label}](${formatTriggerUri(target)})`;
+}
+
+function wakePrompt(
+  action: WatchWakeAction,
+  tenancy?: { projectRef?: string; environmentId?: string }
+): string {
+  const subjectLink = wakeSubjectLink(action, tenancy);
   return [
     WAKE_INSTRUCTION,
     `Resolution: ${wakeResolution(action)} — ${wakeOutcome(action)}.`,
@@ -603,6 +645,9 @@ function wakePrompt(action: WatchWakeAction): string {
       : undefined,
     action.note ? `Why the user asked for it: ${action.note}` : undefined,
     `Facts from the check:\n${JSON.stringify(action.facts, null, 2)}`,
+    subjectLink
+      ? `When you point at the watched object, link it: ${subjectLink} — use this exact markdown link, not a bare name.`
+      : undefined,
     wakeStartsInvestigation(action) ? investigationInstruction(action) : undefined,
   ]
     .filter(Boolean)
@@ -723,7 +768,13 @@ async function narrateWatchWake(args: {
     // reads back the same cached prefix a normal turn would.
     messages: [
       ...withCacheBreakpointOnLast(sanitizeReplayedToolInputs(args.messages)),
-      { role: "user" as const, content: wakePrompt(action) },
+      {
+        role: "user" as const,
+        content: wakePrompt(action, {
+          projectRef: args.clientData?.projectRef,
+          environmentId: args.clientData?.environmentId,
+        }),
+      },
     ],
     ...resolved.toAISDKTelemetry(),
   });