Add frontend proposal cards for prioritize_failure_hotspots and preview_correction_impact

client/package.json

@@ -29,7 +29,8 @@
   "scripts": {
     "start": "react-scripts start",
     "build": "cross-env CI=false react-scripts build",
-    "electron": "electron ."
+    "electron": "electron .",
+    "test": "react-scripts test"
   },
   "eslintConfig": {
     "extends": [

client/src/__tests__/agentProposalCards.test.js

@@ -0,0 +1,71 @@
+import React from "react";
+import { fireEvent, render, screen } from "@testing-library/react";
+import AgentProposalCard from "../components/chat/AgentProposalCard";
+
+describe("AgentProposalCard", () => {
+  it("renders prioritize_failure_hotspots with approve/reject", () => {
+    const onApprove = jest.fn();
+    const onReject = jest.fn();
+    const proposal = {
+      type: "prioritize_failure_hotspots",
+      rationale: "Focus annotation time where model uncertainty is highest.",
+      target_dataset: "set-a",
+      hotspots: ["slice_01", "slice_10"],
+      priority_metric: "uncertainty",
+      min_failure_rate: 0.2,
+    };
+
+    render(
+      <AgentProposalCard
+        proposal={proposal}
+        onApprove={onApprove}
+        onReject={onReject}
+      />,
+    );
+
+    expect(screen.getByText("Prioritize Failure Hotspots")).toBeTruthy();
+    expect(
+      screen.getByText(/Focus annotation time where model uncertainty is highest./),
+    ).toBeTruthy();
+    expect(screen.getByText("set-a")).toBeTruthy();
+
+    fireEvent.click(screen.getByRole("button", { name: "Approve" }));
+    fireEvent.click(screen.getByRole("button", { name: "Reject" }));
+
+    expect(onApprove).toHaveBeenCalledWith(proposal);
+    expect(onReject).toHaveBeenCalledWith(proposal);
+  });
+
+  it("renders preview_correction_impact with compact rationale", () => {
+    const proposal = {
+      proposal_type: "preview_correction_impact",
+      rationale:
+        "A".repeat(200),
+      target_metric: "f1",
+      expected_delta: "+0.05",
+      sample_size: 128,
+      confidence: "high",
+    };
+
+    render(<AgentProposalCard proposal={proposal} />);
+
+    expect(screen.getByText("Preview Correction Impact")).toBeTruthy();
+    expect(screen.getByText(/A{157}…/)).toBeTruthy();
+    expect(screen.getByText("f1")).toBeTruthy();
+    expect(screen.getByText("+0.05")).toBeTruthy();
+    expect(screen.getByRole("button", { name: "Approve" })).toBeTruthy();
+    expect(screen.getByRole("button", { name: "Reject" })).toBeTruthy();
+  });
+
+  it("keeps fallback proposal rendering for existing types", () => {
+    render(
+      <AgentProposalCard
+        proposal={{ type: "existing_type", rationale: "Keep behavior stable", foo: "bar" }}
+      />,
+    );
+
+    expect(screen.getByText("Agent Proposal")).toBeTruthy();
+    expect(screen.getByText("Keep behavior stable")).toBeTruthy();
+    expect(screen.getByText("bar")).toBeTruthy();
+  });
+});

client/src/components/chat/AgentProposalCard.js

@@ -0,0 +1,48 @@
+import React from "react";
+import { getProposalCardContent } from "../../contexts/workflow/proposalCardConfig";
+
+function AgentProposalCard({ proposal, onApprove, onReject, disabled = false }) {
+  const content = getProposalCardContent(proposal);
+
+  return (
+    <section
+      aria-label={`proposal-${content.type}`}
+      style={{
+        border: "1px solid #d9d9d9",
+        borderRadius: 8,
+        padding: 12,
+        background: "#fff",
+      }}
+    >
+      <div style={{ fontWeight: 600, marginBottom: 8 }}>{content.title}</div>
+      <div style={{ marginBottom: 8, fontSize: 13 }}>{content.rationale}</div>
+      <dl
+        style={{
+          margin: 0,
+          display: "grid",
+          gridTemplateColumns: "auto 1fr",
+          columnGap: 8,
+          rowGap: 4,
+          fontSize: 12,
+        }}
+      >
+        {content.fields.map((field) => (
+          <React.Fragment key={field.key}>
+            <dt style={{ textTransform: "capitalize", color: "#666" }}>{field.label}</dt>
+            <dd style={{ margin: 0 }}>{field.value}</dd>
+          </React.Fragment>
+        ))}
+      </dl>
+      <div style={{ display: "flex", gap: 8, marginTop: 12 }}>
+        <button type="button" onClick={() => onApprove?.(proposal)} disabled={disabled}>
+          Approve
+        </button>
+        <button type="button" onClick={() => onReject?.(proposal)} disabled={disabled}>
+          Reject
+        </button>
+      </div>
+    </section>
+  );
+}
+
+export default AgentProposalCard;

client/src/components/chat/ChatTimelineMessage.js

@@ -0,0 +1,18 @@
+import React from "react";
+import AgentProposalCard from "./AgentProposalCard";
+
+function ChatTimelineMessage({ item, onApprove, onReject }) {
+  if (item?.kind === "agent_proposal") {
+    return (
+      <AgentProposalCard
+        proposal={item.proposal}
+        onApprove={onApprove}
+        onReject={onReject}
+      />
+    );
+  }
+
+  return <div>{item?.text}</div>;
+}
+
+export default ChatTimelineMessage;

client/src/contexts/workflow/proposalCardConfig.js

@@ -0,0 +1,76 @@
+const toDisplayValue = (value) => {
+  if (value === null || value === undefined || value === "") {
+    return "—";
+  }
+
+  if (Array.isArray(value)) {
+    return value.length ? value.join(", ") : "—";
+  }
+
+  if (typeof value === "object") {
+    return JSON.stringify(value);
+  }
+
+  return String(value);
+};
+
+const compactRationale = (rationale) => {
+  if (!rationale) return "No rationale provided.";
+  const trimmed = String(rationale).trim();
+  return trimmed.length > 160 ? `${trimmed.slice(0, 157)}…` : trimmed;
+};
+
+const pickEntries = (proposal, keys) =>
+  keys.map((key) => ({ key, label: key.replace(/_/g, " "), value: toDisplayValue(proposal[key]) }));
+
+export const getProposalCardContent = (proposal = {}) => {
+  const type = proposal.type || proposal.proposal_type || "proposal";
+
+  if (type === "prioritize_failure_hotspots") {
+    return {
+      type,
+      title: "Prioritize Failure Hotspots",
+      rationale: compactRationale(proposal.rationale || proposal.why),
+      fields: [
+        ...pickEntries(proposal, [
+          "target_dataset",
+          "hotspots",
+          "priority_metric",
+          "min_failure_rate",
+        ]),
+      ],
+    };
+  }
+
+  if (type === "preview_correction_impact") {
+    return {
+      type,
+      title: "Preview Correction Impact",
+      rationale: compactRationale(proposal.rationale || proposal.why),
+      fields: [
+        ...pickEntries(proposal, [
+          "target_metric",
+          "expected_delta",
+          "sample_size",
+          "confidence",
+        ]),
+      ],
+    };
+  }
+
+  const fallbackFields = Object.entries(proposal)
+    .filter(([key]) => !["type", "proposal_type", "rationale", "why"].includes(key))
+    .slice(0, 4)
+    .map(([key, value]) => ({
+      key,
+      label: key.replace(/_/g, " "),
+      value: toDisplayValue(value),
+    }));
+
+  return {
+    type,
+    title: "Agent Proposal",
+    rationale: compactRationale(proposal.rationale || proposal.why),
+    fields: fallbackFields,
+  };
+};