@circuitwall/jarela 2.2.1 → 2.4.0

package/components/chat/MessageList.tsx

@@ -1,21 +1,17 @@
 "use client";
 import { memo, useCallback, useEffect, useLayoutEffect, useMemo, useRef, useState } from "react";
 import { Brain, ChevronRight, Hourglass, X, ArrowDown, Eye, EyeOff } from "lucide-react";
-import type { AgentConfig, AutomationActivityMetadata, ContentPart, Message, SummaryTopicSegment, UserProfile } from "@/api/types";
+import type { AgentConfig, AutomationActivityMetadata, ContentPart, Message, SummaryTopicSegment, UserProfile, WarmChannelSummary } from "@/api/types";
 import { ToolList, type ToolEvent } from "./ToolList";
 import { MessageBubble } from "./MessageBubble";
 import { ContextBoundaryDivider, WarmSummaryCard } from "./ContextBoundary";
+import { WorkflowStepRetryContext, workflowStepRetryMessage, type WorkflowStepRetryRequest } from "./workflow-step-retry";
 import { MESSAGE_FILTER_KEYS, type MessageFilterKey } from "@/hooks/useMessageFilters";
 import { CollapseChevron } from "@/components/ui/CollapseChevron";
 import { MetaRow } from "@/components/ui/MetaRow";
 import { StatusDot } from "@/components/ui/StatusDot";
 import { Dialog } from "@/components/ui/Dialog";
 
-interface SystemNotice {
-  id: string;
-  text: string;
-}
-
 interface QueuedMessageView {
   id: string;
   text: string;
@@ -31,7 +27,6 @@ interface Props {
   threadId?: string | null;
   messages: Message[];
   steeredSegments?: SteeredSegment[];
-  notices?: SystemNotice[];
   agentConfig?: AgentConfig | null;
   userProfile?: UserProfile | null;
   streamingContent?: string;
@@ -53,6 +48,8 @@ interface Props {
   warmSummarySourceMessages?: number | null;
   warmSummarySourceChars?: number | null;
   warmSummaryTopics?: SummaryTopicSegment[] | null;
+  /** Automation-channel recaps committed at the current boundary. */
+  warmChannelSummaries?: WarmChannelSummary[];
   warmSummaryPending?: boolean;
   /** An automatic compaction is summarising earlier messages right now. */
   compactionPending?: boolean;
@@ -78,8 +75,9 @@ interface Props {
 const ALL_FILTERS_SHOWN: Record<MessageFilterKey, boolean> = Object.fromEntries(
   MESSAGE_FILTER_KEYS.map((k) => [k, true]),
 ) as Record<MessageFilterKey, boolean>;
+const NO_CHANNEL_SUMMARIES: WarmChannelSummary[] = [];
 
-export function MessageList({ threadId, messages, steeredSegments, notices, agentConfig, userProfile, streamingContent, thinkingContent, toolEvents, hasMore, loadingMore, onLoadMore, queuedMessages, onRemoveQueued, hotSinceSeq, warmSummary, warmSummaryBeforeSeq, warmSummaryComputedAt, warmSummarySourceMessages, warmSummarySourceChars, warmSummaryTopics, warmSummaryPending = false, compactionPending = false, onSetContextPin, streaming, contextWindowTokens, onRetryMessage, filters = ALL_FILTERS_SHOWN, onToggleFilter: toggle = () => {}, onResetFilters: reset = () => {} }: Props) {
+export function MessageList({ threadId, messages, steeredSegments, agentConfig, userProfile, streamingContent, thinkingContent, toolEvents, hasMore, loadingMore, onLoadMore, queuedMessages, onRemoveQueued, hotSinceSeq, warmSummary, warmSummaryBeforeSeq, warmSummaryComputedAt, warmSummarySourceMessages, warmSummarySourceChars, warmSummaryTopics, warmChannelSummaries = NO_CHANNEL_SUMMARIES, warmSummaryPending = false, compactionPending = false, onSetContextPin, streaming, contextWindowTokens, onRetryMessage, filters = ALL_FILTERS_SHOWN, onToggleFilter: toggle = () => {}, onResetFilters: reset = () => {} }: Props) {
   const hostRef = useRef<HTMLDivElement>(null);
   const maskRegionRef = useRef<HTMLDivElement>(null);
   const scrollRef = useRef<HTMLDivElement>(null);
@@ -104,6 +102,13 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
   const [pendingHotSinceSeq, setPendingHotSinceSeq] = useState<number | null>(null);
   const [topControlsOpen, setTopControlsOpen] = useState(false);
   const [summaryPopoverOpen, setSummaryPopoverOpen] = useState(false);
+
+  const retryWorkflowStep = useMemo(
+    () => onRetryMessage
+      ? (request: WorkflowStepRetryRequest) => onRetryMessage(workflowStepRetryMessage(request), [])
+      : null,
+    [onRetryMessage],
+  );
 
   // Apply category filter. Messages with no `category` (NULL = ordinary
   // chat) are always shown; tagged messages are gated by their toggle.
@@ -503,9 +508,15 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
       ? nextHotCount < currentHotCount
       : false;
   const hasWarmSummary = !!warmSummary;
+  // A committed boundary whose chat range had nothing to summarise (e.g. a
+  // bridge-only agent): the recap, if any, lives in the channel summaries.
+  const summaryCommittedEmpty = !hasWarmSummary
+    && effectiveHotSinceSeq !== null
+    && warmSummaryBeforeSeq === effectiveHotSinceSeq;
   const summaryFresh = hasWarmSummary
     && effectiveHotSinceSeq !== null
     && warmSummaryBeforeSeq === effectiveHotSinceSeq;
+  const summaryReady = summaryFresh || summaryCommittedEmpty;
   const summaryUpdating = hasWarmSummary && !summaryFresh && !!streaming;
   const summaryStale = hasWarmSummary && !summaryFresh && !summaryUpdating;
 
@@ -593,10 +604,25 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
     boundaryAnchorRef.current = null;
   }, []);
 
+  // While a programmatic smooth scroll (locate boundary, deep link) animates
+  // away from the bottom, its first frames are still "near bottom". Without
+  // this lock, scroll sync re-arms bottom-follow and the post-render snap
+  // cancels the animation, so the target is never reached.
+  const programmaticScrollUntilRef = useRef(0);
   const syncScrollButton = useCallback((el: HTMLDivElement) => {
     const isAtBottom = el.scrollHeight - el.scrollTop - el.clientHeight < 100;
-    atBottomRef.current = isAtBottom;
-    setShowScrollButton(!isAtBottom && hasScrollTarget);
+    const locked = Date.now() < programmaticScrollUntilRef.current;
+    atBottomRef.current = isAtBottom && !locked;
+    setShowScrollButton(!atBottomRef.current && hasScrollTarget);
+  }, [hasScrollTarget]);
+
+  const scrollTargetIntoView = useCallback((target: HTMLElement) => {
+    programmaticScrollUntilRef.current = Date.now() + PROGRAMMATIC_SCROLL_LOCK_MS;
+    atBottomRef.current = false;
+    setShowScrollButton(hasScrollTarget);
+    target.scrollIntoView({ behavior: "smooth", block: "center" });
+    target.classList.add("jarela-deep-link-flash");
+    window.setTimeout(() => target.classList.remove("jarela-deep-link-flash"), 1600);
   }, [hasScrollTarget]);
 
   // Anchor for older-message pagination. When the user scrolls near the top
@@ -715,11 +741,7 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
         const safe = id.replace(/"/g, '\\"');
         const el = root.querySelector(`[data-message-id="${safe}"]`) as HTMLElement | null;
         if (!el) return;
-        atBottomRef.current = false;
-        setShowScrollButton(hasScrollTarget);
-        el.scrollIntoView({ behavior: "smooth", block: "center" });
-        el.classList.add("jarela-deep-link-flash");
-        setTimeout(() => el.classList.remove("jarela-deep-link-flash"), 1600);
+        scrollTargetIntoView(el);
         // Strip the hash so the next streaming message / list-length change
         // doesn't yank the view back to this anchor. replaceState avoids
         // triggering a hashchange event (which would re-enter this handler).
@@ -730,7 +752,7 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
     scrollToHashTarget();
     window.addEventListener("hashchange", scrollToHashTarget);
     return () => window.removeEventListener("hashchange", scrollToHashTarget);
-  }, [messages.length, hasScrollTarget]);
+  }, [messages.length, scrollTargetIntoView]);
 
   // Handle orientation changes and resizes to maintain scroll position
   useEffect(() => {
@@ -765,12 +787,11 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
       ? document.getElementById(`context-boundary-${effectiveHotSinceSeq}`)
       : root.querySelector("[data-focus-boundary='1']") as HTMLElement | null;
     if (!boundary) return;
-    atBottomRef.current = false;
-    setShowScrollButton(hasScrollTarget);
-    boundary.scrollIntoView({ behavior: "smooth", block: "center" });
+    scrollTargetIntoView(boundary);
   }
 
   return (
+    <WorkflowStepRetryContext.Provider value={retryWorkflowStep}>
     <div ref={hostRef} className="flex-1 flex flex-col min-h-0 relative">
       {visibleMessages.length > 0 && (
         <div className="shrink-0 px-4 pt-[calc(var(--app-safe-top)+3rem)] pb-1">
@@ -1014,13 +1035,6 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
           ))}
         </div>
       )}
-      {notices?.map((n) => (
-        <div key={n.id} className="flex justify-center my-3">
-          <span className="text-xs italic text-fg-faint bg-surface-2 px-3 py-1 rounded-full border border-border">
-            {n.text}
-          </span>
-        </div>
-      ))}
         </div>
 
         <div
@@ -1079,7 +1093,7 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
                 className={[
                   "text-[10px] px-1.5 py-px rounded border inline-flex items-center gap-1",
                   summaryUpdating ? "border-accent/40 bg-accent/10 text-accent" :
-                  summaryFresh ? "border-emerald-500/30 bg-emerald-500/10 text-emerald-600 dark:text-emerald-400" :
+                  summaryReady ? "border-emerald-500/30 bg-emerald-500/10 text-emerald-600 dark:text-emerald-400" :
                   summaryStale ? "border-amber-500/30 bg-amber-500/10 text-amber-700 dark:text-amber-400" :
                   "border-border bg-surface-2 text-fg-faint",
                 ].join(" ")}
@@ -1088,7 +1102,7 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
                   <span className="h-2 w-2 rounded-full bg-accent animate-pulse" aria-label="Summary updating" />
                 )}
                 <span>
-                  {summaryUpdating ? "updating" : summaryFresh ? "ready" : summaryStale ? "needs refresh" : "pending"}
+                  {summaryUpdating ? "updating" : summaryReady ? "ready" : summaryStale ? "needs refresh" : "pending"}
                 </span>
               </span>
             </div>
@@ -1096,6 +1110,10 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
               <p className="text-[11px] text-fg-faint">
                 {summaryFresh
                   ? "Showing the latest summary for messages above the boundary line."
+                  : summaryCommittedEmpty
+                  ? (warmChannelSummaries.length > 0
+                    ? "No chat messages above the boundary line; automation activity there is summarised below."
+                    : "Nothing above the boundary line needed summarising.")
                   : summaryStale
                   ? "This summary is from an older focus and will refresh after the next reply."
                   : "No summary yet. It will appear after the next reply."}
@@ -1109,6 +1127,7 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
               computedAt={warmSummaryComputedAt ?? null}
               streaming={!!streaming}
               topics={warmSummaryTopics ?? null}
+              channelSummaries={warmChannelSummaries}
             />
           </div>
         </Dialog>
@@ -1157,6 +1176,7 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
         </div>
       </Dialog>
     </div>
+    </WorkflowStepRetryContext.Provider>
   );
 }
 
@@ -1165,6 +1185,7 @@ export function MessageList({ threadId, messages, steeredSegments, notices, agen
 // activeChannels + lib/agents/prepare/history-window.ts). The rest
 // (extension, page_capture, synthetic, tool_use, thinking) are display-only.
 const CHANNEL_SCOPED_FILTER_KEYS = new Set<MessageFilterKey>(["scheduled_task", "watcher", "bridge"]);
+const PROGRAMMATIC_SCROLL_LOCK_MS = 1500;
 
 const CHIP_LABELS: Record<MessageFilterKey, string> = {
   scheduled_task: "scheduled",

package/components/chat/ToolList.test.tsx

@@ -1,7 +1,8 @@
 // @vitest-environment jsdom
 import { describe, it, expect, vi } from "vitest";
-import { render, screen, fireEvent, waitFor, within } from "@testing-library/react";
+import { render, screen, fireEvent, waitFor } from "@testing-library/react";
 import { ToolList, type ToolEvent } from "./ToolList";
+import { WorkflowStepRetryContext, workflowStepRetryMessage } from "./workflow-step-retry";
 
 const { codexJobGet, codexJobCancel } = vi.hoisted(() => ({
   codexJobGet: vi.fn(),
@@ -343,21 +344,96 @@ describe("ToolList — live progress (ADR-0073)", () => {
     render(<ToolList events={events} />);
     fireEvent.click(screen.getByText("workflow_progress"));
 
-    expect(screen.getByText("workflow")).toBeTruthy();
-    expect(screen.getAllByText("Fetching changes").length).toBeGreaterThan(0);
     expect(screen.getAllByText("Fetch changes").length).toBeGreaterThan(0);
     expect(screen.getAllByText("Build todo list").length).toBeGreaterThan(0);
-    expect(screen.getByLabelText("done")).toBeTruthy();
-    expect(screen.getByLabelText("checking")).toBeTruthy();
-    const firstContract = screen.getAllByText("Step contract")[0];
-    fireEvent.click(firstContract);
-    const contract = firstContract.closest("details");
-    expect(contract).toBeTruthy();
-    if (!contract) throw new Error("Workflow step contract details were not rendered");
-    expect(within(contract).getByText("Baseline tag and repository HEAD")).toBeTruthy();
-    expect(within(contract).getByText("Paths changed since baseline")).toBeTruthy();
-    expect(within(contract).getByText("Inspect system-computed git diff result")).toBeTruthy();
-    expect(within(contract).getByText("git diff returned the changed paths")).toBeTruthy();
+    expect(screen.getByRole("img", { name: "verified" })).toBeTruthy();
+    expect(screen.getByRole("img", { name: "checking" })).toBeTruthy();
+    expect(screen.queryByText("Step contract")).toBeNull();
+    expect(screen.queryByText("Baseline tag and repository HEAD")).toBeNull();
+    expect(screen.getByText("Fetch changes").getAttribute("title")).toContain("git diff returned the changed paths");
+    expect(screen.queryByRole("button", { name: /^Retry / })).toBeNull();
+    expect(screen.queryByText("result")).toBeNull();
+  });
+
+  it("shows a rejected workflow update as an error instead of a silent checklist", () => {
+    const events: ToolEvent[] = [
+      {
+        id: "w4",
+        phase: "call",
+        name: "workflow_progress",
+        payload: {
+          workflow_id: "release_review",
+          items: [{ id: "run-tests", label: "Run tests", input: "Diff", output: "Results", validation: "Run suite", status: "done" }],
+        },
+      },
+      {
+        id: "w4",
+        phase: "result",
+        name: "workflow_progress",
+        payload: { ok: false, workflow_id: "release_review", error: "Every completed workflow item requires observed validation_evidence." },
+      },
+    ];
+
+    render(<ToolList events={events} />);
+    fireEvent.click(screen.getByText("workflow_progress"));
+
+    expect(screen.getByText("Update rejected: Every completed workflow item requires observed validation_evidence.")).toBeTruthy();
+  });
+
+  it("shows the attempt count on a workflow step that was retried", () => {
+    const snapshot = (id: string, status: string) => ({
+      workflow_id: "release_review",
+      items: [{ id: "run-tests", label: "Run tests", input: "Diff", output: "Results", validation: "Run suite", status }],
+    });
+    const result = (status: string) => ({
+      ok: true,
+      workflow_id: "release_review",
+      state: { phase: null, summary: "", error: null, checklist: snapshot("", status).items },
+    });
+    const events: ToolEvent[] = (["checking", "needs_attention", "checking"] as const).flatMap((status, i) => [
+      { id: `a${i}`, phase: "call" as const, name: "workflow_progress", payload: snapshot("", status) },
+      { id: `a${i}`, phase: "result" as const, name: "workflow_progress", payload: result(status) },
+    ]);
+
+    render(<ToolList events={events} />);
+    const rows = screen.getAllByText("workflow_progress");
+    fireEvent.click(rows[1]);
+    expect(screen.queryByText("attempt 2")).toBeNull();
+    fireEvent.click(rows[2]);
+    expect(screen.getByText("attempt 2")).toBeTruthy();
+  });
+
+  it("offers a per-step retry that asks the agent to redo that step", () => {
+    const onRetry = vi.fn();
+    const events: ToolEvent[] = [{
+      id: "w3",
+      phase: "call",
+      name: "workflow_progress",
+      payload: {
+        workflow_id: "release_review",
+        items: [
+          { id: "check-changes", label: "Review changes", input: "Diff", output: "Findings", validation: "Inspect files", validation_evidence: "Diff reviewed", status: "done" },
+          { id: "run-tests", label: "Run tests", input: "Findings", output: "Results", validation: "Run suite", status: "needs_attention" },
+          { id: "report-result", label: "Report result", input: "Results", output: "Summary", validation: "Cross-check" },
+        ],
+      },
+    }];
+
+    render(
+      <WorkflowStepRetryContext.Provider value={onRetry}>
+        <ToolList events={events} />
+      </WorkflowStepRetryContext.Provider>,
+    );
+    fireEvent.click(screen.getByText("workflow_progress"));
+
+    expect(screen.getByRole("button", { name: "Retry Review changes" })).toBeTruthy();
+    expect(screen.queryByRole("button", { name: "Retry Report result" })).toBeNull();
+    fireEvent.click(screen.getByRole("button", { name: "Retry Run tests" }));
+    expect(onRetry).toHaveBeenCalledWith({ workflowId: "release_review", itemId: "run-tests", label: "Run tests" });
+    expect(workflowStepRetryMessage({ workflowId: "release_review", itemId: "run-tests", label: "Run tests" }))
+      .toContain("complete items snapshot");
+    expect(workflowStepRetryMessage({ workflowId: "version_adoption", itemId: "adapt-settings", label: "Adapt settings" }))
+      .toContain('item_id: "adapt-settings"');
   });
 
   it("previews agent-created workflow items while the tool call is running", () => {
@@ -403,8 +479,8 @@ describe("ToolList — live progress (ADR-0073)", () => {
     expect(screen.getAllByText("Review changes").length).toBeGreaterThan(0);
     expect(screen.getAllByText("Run tests").length).toBeGreaterThan(0);
     expect(screen.getAllByText("Report result").length).toBeGreaterThan(0);
-    expect(screen.getByLabelText("done")).toBeTruthy();
-    expect(screen.getByLabelText("checking")).toBeTruthy();
+    expect(screen.getByRole("img", { name: "verified" })).toBeTruthy();
+    expect(screen.getByRole("img", { name: "checking" })).toBeTruthy();
   });
 
   it("renders an invoke_tool dispatch under the target tool's name, args and result", () => {

package/components/chat/ToolList.tsx

@@ -29,6 +29,7 @@ import {
 import { Badge } from "@/components/ui/Badge";
 import { CollapseChevron } from "@/components/ui/CollapseChevron";
 import { WorkflowChecklist, type WorkflowChecklistItemView, type WorkflowChecklistStatus } from "@/components/ui/WorkflowChecklist";
+import { useWorkflowStepRetry } from "./workflow-step-retry";
 import { StructuredTurnCard } from "@/components/chat/StructuredTurnCard";
 import { MetaRow } from "@/components/ui/MetaRow";
 import { ProviderLogo, brandSlugForToolName } from "@/components/models/ProviderLogo";
@@ -263,6 +264,9 @@ interface ToolCallGroup {
   result: unknown;
   deadlineMs: number;
   status: "running" | "ok" | "error" | "async";
+  // workflow_progress only: per-step attempt counts so far in this message
+  // (how many times each item has entered `checking`).
+  workflowAttempts?: Readonly<Record<string, number>>;
   // True when this group's `result` arrived via a later tool_result_get
   // call rather than the original tool's direct return. Surfaces a small
   // "read" pill so it's obvious the agent reaped a background handoff.
@@ -386,7 +390,33 @@ function groupByCallId(events: ToolEvent[]): ToolCallGroup[] {
     g.absorbed = true;
   }
 
-  return order.map((id) => map.get(id)!).filter((g) => !g.absorbed);
+  const groups = order.map((id) => map.get(id)!).filter((g) => !g.absorbed);
+  annotateWorkflowAttempts(groups);
+  return groups;
+}
+
+// Counts each workflow step's transitions into `checking` across the
+// message's workflow_progress calls, so auto-retried steps show "attempt N".
+// Rejected updates (ok:false) changed nothing and are not counted.
+function annotateWorkflowAttempts(groups: ToolCallGroup[]): void {
+  const lastStatus = new Map<string, WorkflowChecklistStatus>();
+  const attempts = new Map<string, Record<string, number>>();
+  for (const group of groups) {
+    if (group.name !== "workflow_progress") continue;
+    if (coerceObject(unwrapLangChainSerializable(group.result))?.ok === false) continue;
+    const view = workflowViewFor(group);
+    if (!view) continue;
+    const counts = { ...(attempts.get(view.workflow_id) ?? {}) };
+    for (const item of view.items) {
+      const key = `${view.workflow_id}:${item.id}`;
+      if (item.status === "checking" && lastStatus.get(key) !== "checking") {
+        counts[item.id] = (counts[item.id] ?? 0) + 1;
+      }
+      lastStatus.set(key, item.status);
+    }
+    attempts.set(view.workflow_id, counts);
+    group.workflowAttempts = counts;
+  }
 }
 
 function ToolCallCard({ group, startedAt }: { group: ToolCallGroup; startedAt: number }) {
@@ -498,6 +528,7 @@ function ToolCallCard({ group, startedAt }: { group: ToolCallGroup; startedAt: n
       {open && (
         <div className="mt-0.5 rounded border border-border/40 bg-surface-2/30 px-2 py-1.5 space-y-1.5 text-[10px]">
           <WorkflowProgressDetails group={group} />
+          {!hasWorkflowView(group) && (<>
           {hasVisibleArgs(group.args) ? (
             <DetailSection label="arguments" value={group.args} />
           ) : hasArgs ? (
@@ -523,6 +554,7 @@ function ToolCallCard({ group, startedAt }: { group: ToolCallGroup; startedAt: n
               value={group.result}
             />
           )}
+          </>)}
         </div>
       )}
     </div>
@@ -617,19 +649,42 @@ function ToolResultRefDetails({ value }: { value: unknown }) {
   );
 }
 
-function WorkflowProgressDetails({ group }: { group: ToolCallGroup }) {
+function workflowViewFor(group: ToolCallGroup): WorkflowProgressView | null {
   if (group.name !== "workflow_progress") return null;
-  const workflow = workflowProgressFrom(group.result) ?? workflowProgressPreviewFrom(group.args);
+  const view = workflowProgressFrom(group.result) ?? workflowProgressPreviewFrom(group.args);
+  const result = coerceObject(unwrapLangChainSerializable(group.result));
+  const attempts = group.workflowAttempts;
+  const withAttempts = view && attempts
+    ? { ...view, items: view.items.map((item) => (attempts[item.id] ? { ...item, attempts: attempts[item.id] } : item)) }
+    : view;
+  if (withAttempts && result?.ok === false && typeof result.error === "string") {
+    return { ...withAttempts, error: `Update rejected: ${result.error}` };
+  }
+  return withAttempts;
+}
+
+/** Workflow rows show only their work items; raw payloads stay visible when the call failed. */
+function hasWorkflowView(group: ToolCallGroup): boolean {
+  return group.status !== "error" && workflowViewFor(group) !== null;
+}
+
+function WorkflowProgressDetails({ group }: { group: ToolCallGroup }) {
+  const retryStep = useWorkflowStepRetry();
+  const workflow = workflowViewFor(group);
   if (!workflow) return null;
+  const workflowId = workflow.workflow_id;
   return (
     <WorkflowChecklist
       compact
       eyebrow="workflow"
-      title={workflow.workflow_id.replace(/[_-]/g, " ")}
+      title={workflowId.replace(/[_-]/g, " ")}
       phaseLabel={workflow.phase ?? "in progress"}
       summary={workflow.summary ?? ""}
       items={workflow.items}
       error={workflow.error}
+      onRetryItem={retryStep
+        ? (item) => retryStep({ workflowId, itemId: item.id, label: item.label })
+        : undefined}
     />
   );
 }

package/components/chat/chat-helpers.test.tsx

@@ -1,6 +1,6 @@
 import { describe, it, expect } from "vitest";
 import type { Message } from "@/api/types";
-import { appendUnique, applyThreadMeta, type ThreadMetaApplier } from "./chat-helpers";
+import { appendUnique, applyThreadMeta, compactionOutcomeToast, type ThreadMetaApplier } from "./chat-helpers";
 
 // created_at is display-only after the seq-ordering change — every fixture
 // shares one value to prove ordering doesn't depend on it. `seq` is omitted
@@ -115,10 +115,18 @@ describe("applyThreadMeta", () => {
         setWarmSummarySourceChars: push("warmSummarySourceChars") as (v: number | null) => void,
         setContextWindowTokens: push("contextWindowTokens") as (v: number | null) => void,
         setWarmSummaryPending: push("warmSummaryPending") as (v: boolean) => void,
+        setManualCompactionPending: push("manualCompactionPending") as (v: boolean) => void,
       },
     };
   }
 
+  it("restores a server-side manual /compact so a remounted view stays busy", () => {
+    const { meta, calls } = recorder();
+    applyThreadMeta(meta, { hot_since_seq: null, manual_compaction_pending: true });
+    applyThreadMeta(meta, { hot_since_seq: null });
+    expect(calls.manualCompactionPending).toEqual([true, false]);
+  });
+
   it("marks an observed hot_since as pending until its matching summary arrives", () => {
     const { meta, calls } = recorder();
 
@@ -148,3 +156,18 @@ describe("applyThreadMeta", () => {
     expect(calls.warmSummaryPending).toEqual([false]);
   });
 });
+
+describe("compactionOutcomeToast", () => {
+  it("reports success with the summarised message count", () => {
+    expect(compactionOutcomeToast({ compacted: true, message_count: 2 })).toMatchObject({ kind: "success", body: "Summarised 2 messages into warm context." });
+    expect(compactionOutcomeToast({ compacted: true, message_count: 1 }).body).toBe("Summarised 1 message into warm context.");
+    expect(compactionOutcomeToast({ compacted: true, message_count: 0 }).body).toBe("Started a fresh context window.");
+  });
+
+  it("only says the conversation is empty when the server says so", () => {
+    expect(compactionOutcomeToast({ compacted: false, reason: "nothing to compact" }).title).toBe("Nothing to compact");
+    const declined = compactionOutcomeToast({ compacted: false, reason: "no summary produced" });
+    expect(declined.title).toBe("Nothing was compacted");
+    expect(declined.body).not.toMatch(/send some messages/i);
+  });
+});

package/components/chat/chat-helpers.ts

@@ -1,5 +1,5 @@
 "use client";
-import type { ContentPart, Message, SummaryTopicSegment } from "@/api/types";
+import type { ContentPart, Message, SummaryTopicSegment, WarmChannelSummary } from "@/api/types";
 
 export interface QueuedMessage {
   id: string;
@@ -7,9 +7,46 @@ export interface QueuedMessage {
   attachments: ContentPart[];
 }
 
-export interface SystemNotice {
-  id: string;
-  text: string;
+export interface CompactionOutcome {
+  compacted: boolean;
+  reason?: string;
+  message_count?: number;
+}
+
+export interface CompactionToast {
+  kind: "info" | "success" | "error";
+  title: string;
+  body: string;
+}
+
+// Maps the /compact response to the toast the user sees. The server's reason
+// decides the text: "nothing to compact" means the thread is empty, anything
+// else means there were messages but no summary came out of them.
+export function compactionOutcomeToast(result: CompactionOutcome): CompactionToast {
+  if (result.compacted) {
+    const n = result.message_count ?? 0;
+    return {
+      kind: "success",
+      title: "Context compacted",
+      body: n > 0
+        ? `Summarised ${n} message${n === 1 ? "" : "s"} into warm context.`
+        : "Started a fresh context window.",
+    };
+  }
+  if (result.reason === "nothing to compact") {
+    return { kind: "info", title: "Nothing to compact", body: "This conversation has no messages yet." };
+  }
+  if (result.reason === "already compacting") {
+    return { kind: "info", title: "Already compacting", body: "A compaction for this conversation is still running." };
+  }
+  if (result.reason === "timed out") {
+    return { kind: "error", title: "Compaction timed out", body: "The summary took too long and was discarded. Context is unchanged." };
+  }
+  return {
+    kind: "info",
+    title: "Nothing was compacted",
+    body: "There was too little to summarise, or the model returned no usable summary. Context is unchanged.",
+  };
 }
 
 export interface ThreadMetaApplier {
@@ -22,9 +59,11 @@ export interface ThreadMetaApplier {
   setWarmSummarySourceMessages: (v: number | null) => void;
   setWarmSummarySourceChars: (v: number | null) => void;
   setWarmSummaryTopics?: (v: SummaryTopicSegment[] | null) => void;
+  setWarmChannelSummaries?: (v: WarmChannelSummary[]) => void;
   setContextWindowTokens: (v: number | null) => void;
   setWarmSummaryPending?: (v: boolean) => void;
   setCompactionPending?: (v: boolean) => void;
+  setManualCompactionPending?: (v: boolean) => void;
 }
 
 export interface ThreadGetPayload {
@@ -37,8 +76,10 @@ export interface ThreadGetPayload {
   warm_summary_source_messages?: number | null;
   warm_summary_source_chars?: number | null;
   warm_summary_topics?: SummaryTopicSegment[] | null;
+  warm_channel_summaries?: WarmChannelSummary[];
   pending_hot_since?: string | null;
   pending_hot_since_seq?: number | null;
+  manual_compaction_pending?: boolean;
   context_window_tokens?: number | null;
 }
 
@@ -128,10 +169,12 @@ export function applyThreadMeta(meta: ThreadMetaApplier, payload: ThreadGetPaylo
   meta.setWarmSummarySourceMessages(payload.warm_summary_source_messages ?? null);
   meta.setWarmSummarySourceChars(payload.warm_summary_source_chars ?? null);
   meta.setWarmSummaryTopics?.(payload.warm_summary_topics ?? null);
+  meta.setWarmChannelSummaries?.(payload.warm_channel_summaries ?? []);
   meta.setContextWindowTokens(payload.context_window_tokens ?? null);
   const summaryFresh = hotSinceSeq !== null && summaryBeforeSeq === hotSinceSeq;
   meta.setWarmSummaryPending?.(hotSinceSeq !== null && !summaryFresh);
   meta.setCompactionPending?.(payload.pending_hot_since_seq != null);
+  meta.setManualCompactionPending?.(payload.manual_compaction_pending === true);
 }
 
 export function makeQueuedId(prefix = "q"): string {