@cueloop/schema 0.1.0-alpha.79 → 0.1.0-alpha.80

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/schema",
-  "version": "0.1.0-alpha.79",
+  "version": "0.1.0-alpha.80",
   "description": "The Thread primitive: types, markdown block model, quote-primary anchors, feedback serialization",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {

package/src/feedback.test.ts

@@ -29,7 +29,7 @@ describe("renderFeedback", () => {
   test("plan edits section carries the one unified diff, directive framing first", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "Tighten the storage section.",
       artifactContent: PLAN,
       workingCopy: PLAN.replace("one JSON document", "one JSON record"),
@@ -38,7 +38,7 @@ describe("renderFeedback", () => {
     });
 
     // Assert
-    expect(feedback).toContain("# Review: request changes");
+    expect(feedback).toContain("# Review: changes requested");
     expect(feedback).toContain("Tighten the storage section.");
     expect(feedback).toContain("## Plan edits");
     expect(feedback).toContain("Apply this exact diff first");
@@ -50,7 +50,7 @@ describe("renderFeedback", () => {
   test("reply edits section is labelled for the reply artifact, not a plan", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       workingCopy: PLAN.replace("one JSON document", "one JSON record"),
@@ -68,7 +68,7 @@ describe("renderFeedback", () => {
   test("a reply review with no artifact path references reply.md, never plan.md", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: "# Findings\n\nThe cache is fine.\n",
       artifactType: "reply",
@@ -101,7 +101,7 @@ describe("renderFeedback", () => {
 
     // Act
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: submittedPatch,
       workingCopy: curatedPatch,
@@ -120,7 +120,7 @@ describe("renderFeedback", () => {
   test("a diff working copy that rejected everything says so", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: "--- a/x\n+++ b/x\n@@ -1 +1 @@\n-a\n+b\n",
       workingCopy: "",
@@ -136,7 +136,7 @@ describe("renderFeedback", () => {
   test("annotations are located by quote and section", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "comment",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       annotations: [
@@ -159,7 +159,7 @@ describe("renderFeedback", () => {
 
     // Act
     const feedback = renderFeedback({
-      verdictKind: "comment",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       annotations: [
@@ -199,7 +199,7 @@ describe("renderFeedback", () => {
 
     // Act
     const feedback = renderFeedback({
-      verdictKind: "comment",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       annotations: [
@@ -223,7 +223,7 @@ describe("renderFeedback", () => {
   test("orphaned anchors are flagged, never dropped", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "comment",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       annotations: [
@@ -239,7 +239,7 @@ describe("renderFeedback", () => {
   test("empty review says so explicitly", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "approve",
+      outcome: "approved",
       summary: "",
       artifactContent: PLAN,
       annotations: [],
@@ -259,7 +259,7 @@ describe("renderFeedback", () => {
 
     // Act
     const noteOnly = renderFeedback({
-      verdictKind: "approve",
+      outcome: "approved",
       summary: "",
       artifactContent: PLAN,
       annotations: [note],
@@ -269,7 +269,7 @@ describe("renderFeedback", () => {
     expect(noteOnly).toContain("_No edits or annotations._");
     expect(noteOnly).not.toContain("The agent's own explanation of the change.");
     const mixed = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       annotations: [
@@ -291,7 +291,7 @@ describe("renderFeedback", () => {
 
     // Act
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       workingCopy: working,
@@ -309,7 +309,7 @@ describe("renderFeedback", () => {
   test("a prototype annotation locates by its selector and is never orphan-flagged", () => {
     // Act
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactType: "prototype",
       artifactContent: "<main><div class='card'>Pricing</div></main>",
@@ -333,7 +333,7 @@ describe("renderFeedback", () => {
 describe("renderFeedback target grouping", () => {
   test("file notes group under their own path section, apart from the artifact", () => {
     const feedback = renderFeedback({
-      verdictKind: "request_changes",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       artifactPath: "docs/plan.md",
@@ -371,7 +371,7 @@ describe("renderFeedback target grouping", () => {
 
   test("a quick-action reference expands to its body; an unknown skill reference passes through", () => {
     const feedback = renderFeedback({
-      verdictKind: "comment",
+      outcome: "changes_requested",
       summary: "",
       artifactContent: PLAN,
       annotations: [

package/src/feedback.ts

@@ -12,14 +12,14 @@ import {
   type Annotation,
   type ArtifactType,
   type Thread,
-  type VerdictKind,
+  type MessageOutcome,
 } from "./types";
 import { parseBlocks, sectionOf, type Block } from "./markdown";
 import { resolveAnchor } from "./anchor";
 import { unifiedDiffText } from "./diff";
 
 export interface FeedbackInput {
-  verdictKind: VerdictKind;
+  outcome: MessageOutcome;
   summary: string;
   /** The submitted artifact content (latest revision). */
   artifactContent: string;
@@ -80,7 +80,7 @@ export function renderFeedback(input: FeedbackInput): string {
   );
   const lines: string[] = [];
 
-  lines.push("# Review: " + input.verdictKind.replace("_", " "));
+  lines.push("# Review: " + input.outcome.replaceAll("_", " "));
   lines.push("");
   if (input.summary.trim()) {
     lines.push(input.summary.trim());
@@ -247,12 +247,12 @@ function annotationSectionLines(
 
 export function feedbackForSession(
   session: Thread,
-  verdictKind: VerdictKind,
+  outcome: MessageOutcome,
   summary: string,
   actionBodies?: Record<string, string>,
 ): string {
   return renderFeedback({
-    verdictKind,
+    outcome,
     summary,
     artifactContent: session.artifact.content,
     workingCopy: session.workingCopy,

package/src/history.test.ts

@@ -18,10 +18,16 @@ import {
   validateHistory,
   type SessionHistory,
 } from "./history";
-import type { Annotation, Verdict } from "./types";
+import type { Annotation, Message } from "./types";
 
 const AT = "2026-09-01T10:00:00.000Z";
-const VERDICT: Verdict = { kind: "approve", summary: "", feedback: "", resolvedAt: AT };
+const MESSAGE: Message = {
+  id: "msg_1",
+  outcome: "approved",
+  summary: "",
+  body: "",
+  sentAt: AT,
+};
 
 function annotation(id: string, createdAt: string): Annotation {
   return {
@@ -39,13 +45,13 @@ function root(): SessionHistory {
     id: "ses_1",
     revisions: [{ revision: 1, content: "Plan v1", submittedAt: AT }],
     annotations: [],
-    verdict: null,
+    message: null,
     createdAt: AT,
   });
 }
 
 describe("appendEntry and derivePath", () => {
-  test("entries chain on the current tip and the path derives the head, comments, and verdicts", () => {
+  test("entries chain on the current tip and the path derives the head, comments, and messages", () => {
     // Arrange
     let history = root();
 
@@ -57,7 +63,7 @@ describe("appendEntry and derivePath", () => {
       annotationId: "a1",
       createdAt: AT,
     }).history;
-    history = appendEntry(history, { type: "verdict", verdict: VERDICT, createdAt: AT }).history;
+    history = appendEntry(history, { type: "message", message: MESSAGE, createdAt: AT }).history;
     history = appendEntry(history, {
       type: "revision",
       by: "agent",
@@ -70,14 +76,14 @@ describe("appendEntry and derivePath", () => {
 
     expect(derived.head.content).toBe("Plan v2");
     expect(derived.annotationIds).toEqual(["a2"]);
-    expect(derived.verdicts).toEqual([VERDICT]);
+    expect(derived.messages).toEqual([MESSAGE]);
     expect(derived.rounds).toBe(2);
     expect(pathOf(history).map((entry) => entry.type)).toEqual([
       "revision",
       "comment",
       "comment",
       "comment-removed",
-      "verdict",
+      "message",
       "revision",
     ]);
   });
@@ -209,12 +215,12 @@ describe("validateHistory and cycles", () => {
 });
 
 describe("forkHistory", () => {
-  test("copies the path with its comments and labels, drops verdicts, starts at the head", () => {
+  test("copies the path with its comments and labels, drops messages, starts at the head", () => {
     // Arrange
     let history = root();
 
     history = appendEntry(history, { type: "comment", annotationId: "a1", createdAt: AT }).history;
-    history = appendEntry(history, { type: "verdict", verdict: VERDICT, createdAt: AT }).history;
+    history = appendEntry(history, { type: "message", message: MESSAGE, createdAt: AT }).history;
     history = labelTip(history, "approved v1");
     const approvedTip = tipOf(history);
 
@@ -229,16 +235,16 @@ describe("forkHistory", () => {
     expect(fork.entries.map((entry) => entry.type)).toEqual(["revision", "comment"]);
     expect(fork.tips).toEqual({ [MAIN_BRANCH]: fork.entries[1]!.id });
     expect(derivePath(fork).annotationIds).toEqual(["a1"]);
-    // the label sat on the dropped verdict entry, so it does not travel
+    // the label sat on the dropped message entry, so it does not travel
     expect(fork.labels).toEqual({});
     expect(history.labels[approvedTip]).toBe("approved v1");
   });
 
   test("entries after a dropped one are chained to the kept entry before it", () => {
-    // Arrange: rev1, verdict, rev2 by the agent, a reviewer edit, a comment on top
+    // Arrange: rev1, message, rev2 by the agent, a reviewer edit, a comment on top
     let history = root();
 
-    history = appendEntry(history, { type: "verdict", verdict: VERDICT, createdAt: AT }).history;
+    history = appendEntry(history, { type: "message", message: MESSAGE, createdAt: AT }).history;
     history = appendEntry(history, {
       type: "revision",
       by: "agent",
@@ -289,7 +295,7 @@ describe("recaptureMainHead", () => {
 });
 
 describe("historyFromLinear", () => {
-  test("chains revisions on main, files comments after the revision they were made on, ends with the verdict", () => {
+  test("chains revisions on main, files comments after the revision they were made on, ends with the message", () => {
     // Act
     const history = historyFromLinear({
       id: "ses_9",
@@ -301,7 +307,7 @@ describe("historyFromLinear", () => {
         annotation("late", "2026-09-01T13:00:00.000Z"),
         annotation("early", "2026-09-01T11:00:00.000Z"),
       ],
-      verdict: VERDICT,
+      message: MESSAGE,
       createdAt: "2026-09-01T09:00:00.000Z",
     });
 
@@ -310,14 +316,14 @@ describe("historyFromLinear", () => {
       history.entries.map(
         (entry) => `${entry.type}:${"annotationId" in entry ? entry.annotationId : ""}`,
       ),
-    ).toEqual(["revision:", "comment:early", "revision:", "comment:late", "verdict:"]);
+    ).toEqual(["revision:", "comment:early", "revision:", "comment:late", "message:"]);
     expect(history.entries[0]!.id).toBe("ses_9_rev1");
     expect(history.branch).toBe(MAIN_BRANCH);
     const derived = derivePath(history);
 
     expect(derived.head.content).toBe("v2");
     expect(derived.annotationIds).toEqual(["early", "late"]);
-    expect(derived.verdicts).toEqual([VERDICT]);
+    expect(derived.messages).toEqual([MESSAGE]);
     expect(derived.rounds).toBe(2);
   });
 
@@ -327,7 +333,7 @@ describe("historyFromLinear", () => {
       id: "ses_2",
       revisions: [{ revision: 1, content: "v1", submittedAt: AT }],
       annotations: [annotation("a", AT)],
-      verdict: null,
+      message: null,
       createdAt: AT,
     };
 
@@ -343,7 +349,7 @@ describe("followBranch", () => {
       id: "ses_1",
       revisions: [{ revision: 1, content: "v1", submittedAt: AT }],
       annotations: [],
-      verdict: null,
+      message: null,
       createdAt: AT,
     });
 
@@ -373,7 +379,7 @@ describe("removalEntries", () => {
       id: "ses_1",
       revisions: [{ revision: 1, content: "v1", submittedAt: AT }],
       annotations: [],
-      verdict: null,
+      message: null,
       createdAt: AT,
     });
 

package/src/history.ts

@@ -4,13 +4,13 @@
  * from the current branch's tip back to the root. Nothing is ever deleted:
  * navigating moves a tip, a comment removal is its own entry, a fork copies a
  * path into a new history. Every view of the session - the artifact text,
- * the open comments, the verdict history - derives from the active path.
+ * the open comments, the message history - derives from the active path.
  *
  * Pure: no store, no daemon. The daemon owns persistence; the client and the
  * feedback document read what this module derives.
  */
 
-import type { Annotation, Thread, Verdict } from "./types";
+import type { Annotation, Thread, Message } from "./types";
 
 export type EntryAuthor = "agent" | "reviewer";
 
@@ -25,7 +25,7 @@ export type SessionEntry =
   | (EntryBase & { type: "revision"; by: EntryAuthor; content: string })
   | (EntryBase & { type: "comment"; annotationId: string })
   | (EntryBase & { type: "comment-removed"; annotationId: string })
-  | (EntryBase & { type: "verdict"; verdict: Verdict })
+  | (EntryBase & { type: "message"; message: Message })
   | (EntryBase & { type: "branch-summary"; text: string; abandoned: string[] });
 
 export type EntryType = SessionEntry["type"];
@@ -161,8 +161,8 @@ export interface DerivedPath {
    * is addressed by a revision is the annotation's own state, not the path's.
    */
   annotationIds: string[];
-  /** Verdicts on the path, oldest first. */
-  verdicts: Verdict[];
+  /** Messages on the path, oldest first. */
+  messages: Message[];
   summaries: Array<SessionEntry & { type: "branch-summary" }>;
   /** Agent revisions on the path: one per round. */
   rounds: number;
@@ -176,20 +176,20 @@ export function derivePath(history: SessionHistory, fromId?: string): DerivedPat
 
   if (!head) throw new HistoryError("not-a-revision", "a path without a revision has no head");
   const open = new Set<string>();
-  const verdicts: Verdict[] = [];
+  const messages: Message[] = [];
   const summaries: Array<SessionEntry & { type: "branch-summary" }> = [];
 
   for (const entry of path) {
     if (entry.type === "comment") open.add(entry.annotationId);
     if (entry.type === "comment-removed") open.delete(entry.annotationId);
-    if (entry.type === "verdict") verdicts.push(entry.verdict);
+    if (entry.type === "message") messages.push(entry.message);
     if (entry.type === "branch-summary") summaries.push(entry);
   }
 
   return {
     head,
     annotationIds: [...open],
-    verdicts,
+    messages,
     summaries,
     rounds: revisions.filter((entry) => entry.by === "agent").length,
   };
@@ -301,7 +301,7 @@ export function navigateTo(
 
 /**
  * A fork's history: the current path copied as one branch, keeping comments
- * and labels on it, dropping verdicts and the reviewer's edits, with the tip
+ * and labels on it, dropping messages and the reviewer's edits, with the tip
  * at the copied head. The kept entries are chained anew so no entry points at
  * one that was dropped.
  */
@@ -310,7 +310,7 @@ export function forkHistory(history: SessionHistory): SessionHistory {
   const labels: Record<string, string> = {};
 
   for (const entry of pathOf(history)) {
-    if (entry.type === "verdict" || (entry.type === "revision" && entry.by === "reviewer"))
+    if (entry.type === "message" || (entry.type === "revision" && entry.by === "reviewer"))
       continue;
     kept.push({ ...entry, parentId: kept.at(-1)?.id ?? null });
     const label = history.labels[entry.id];
@@ -357,11 +357,11 @@ export function removalEntries(
 /**
  * A linear session as a one-branch tree: its revisions chained on `main` as
  * agent revisions, its comments attached after the revision that was current
- * when they were made, its verdict (when resolved) after the last revision.
+ * when they were made, its message (when resolved) after the last revision.
  * Deterministic: the same record migrates to the same ids.
  */
 export function historyFromLinear(
-  session: Pick<Thread, "id" | "revisions" | "annotations" | "verdict" | "createdAt">,
+  session: Pick<Thread, "id" | "revisions" | "annotations" | "message" | "createdAt">,
 ): SessionHistory {
   const entries: SessionEntry[] = [];
   let parentId: string | null = null;
@@ -403,12 +403,12 @@ export function historyFromLinear(
       nextComment++;
     }
   });
-  if (session.verdict) {
+  if (session.message) {
     push({
-      id: `${session.id}_verdict`,
-      type: "verdict",
-      verdict: session.verdict,
-      createdAt: session.verdict.resolvedAt,
+      id: `${session.id}_message`,
+      type: "message",
+      message: session.message,
+      createdAt: session.message.sentAt,
     });
   }
   const tip = entries.at(-1);

package/src/participants.test.ts

@@ -10,7 +10,7 @@ function emptySession(): Thread {
     artifact: { type: "plan", content: "# Plan\n", meta: {} },
     revisions: [],
     annotations: [],
-    verdict: null,
+    message: null,
     status: "pending",
     createdAt: "2026-01-01T00:00:00.000Z",
   };

package/src/path-view.test.ts

@@ -28,7 +28,7 @@ function threaded(): SessionHistory {
     id: "ses_1",
     revisions: [{ revision: 1, content: "Plan v1", submittedAt: AT }],
     annotations: [],
-    verdict: null,
+    message: null,
     createdAt: AT,
   });
 
@@ -110,7 +110,7 @@ describe("applyPathView", () => {
       annotations: [annotation("a2")],
       shelvedAnnotations: [annotation("a1")],
       workingCopy: "edited",
-      verdict: null,
+      message: null,
       status: "pending",
       createdAt: AT,
     };
@@ -149,7 +149,7 @@ describe("viewFollowing", () => {
       shelvedAnnotations: [annotation("a2")],
       workingCopy: "Plan v1 edited",
       history,
-      verdict: null,
+      message: null,
       status: "pending",
       createdAt: AT,
     };
@@ -174,7 +174,7 @@ describe("viewFollowing", () => {
       id: "ses_1",
       revisions: [{ revision: 1, content: "Plan v1", submittedAt: AT }],
       annotations: [],
-      verdict: null,
+      message: null,
       createdAt: AT,
     });
     const session: Thread = {
@@ -186,7 +186,7 @@ describe("viewFollowing", () => {
       annotations: [],
       shareBranch: "gone",
       history,
-      verdict: null,
+      message: null,
       status: "pending",
       createdAt: AT,
     };

package/src/share-links.test.ts

@@ -10,7 +10,7 @@ function thread(overrides: Partial<Thread> = {}): Thread {
     artifact: { type: "plan", content: "# Plan\n", meta: {} },
     revisions: [{ revision: 1, content: "# Plan\n", submittedAt: "2026-01-01T00:00:00.000Z" }],
     annotations: [],
-    verdict: null,
+    message: null,
     status: "pending",
     createdAt: "2026-01-01T00:00:00.000Z",
     ...overrides,

package/src/types.test.ts

@@ -2,10 +2,11 @@ import { describe, expect, test } from "bun:test";
 import {
   annotationTarget,
   ARTIFACT_TYPES,
+  WORKFLOW_KINDS,
   isArtifactType,
   isMarkdownArtifact,
   newAnnotationId,
-  verdictAllows,
+  messageAllows,
 } from "./types";
 
 describe("annotationTarget", () => {
@@ -33,11 +34,19 @@ describe("ARTIFACT_TYPES", () => {
   });
 });
 
-describe("verdictAllows", () => {
+describe("WORKFLOW_KINDS", () => {
+  test("review and refine are workflows, not artifact types", () => {
+    expect(WORKFLOW_KINDS).toEqual(["plan", "reply", "prototype", "diff", "review", "refine"]);
+    expect(isArtifactType("review")).toBe(false);
+    expect(isArtifactType("refine")).toBe(false);
+  });
+});
+
+describe("messageAllows", () => {
   test("only approve maps to allow", () => {
-    expect(verdictAllows("approve")).toBe(true);
-    expect(verdictAllows("comment")).toBe(false);
-    expect(verdictAllows("request_changes")).toBe(false);
+    expect(messageAllows("approved")).toBe(true);
+    expect(messageAllows("changes_requested")).toBe(false);
+    expect(messageAllows("changes_requested")).toBe(false);
   });
 });
 

package/src/types.ts

@@ -1,6 +1,6 @@
 /**
  * The Thread primitive. Everything in cueloop renders, annotates, or
- * resolves this one noun. This module is pure data shapes - no IO, no
+ * extends this one noun. This module is pure data shapes - no IO, no
  * dependencies beyond the history shapes.
  */
 
@@ -33,6 +33,11 @@ export const ARTIFACT_TYPES = ["plan", "diff", "prototype", "reply"] as const;
 
 export type ArtifactType = (typeof ARTIFACT_TYPES)[number];
 
+/** Harness workflows; review and refine are compositions, never artifact types. */
+export const WORKFLOW_KINDS = ["plan", "reply", "prototype", "diff", "review", "refine"] as const;
+
+export type WorkflowKind = (typeof WORKFLOW_KINDS)[number];
+
 /** Trust-boundary guard: is this string one of the artifact primitives? */
 export function isArtifactType(value: string): value is ArtifactType {
   return ARTIFACT_TYPES.some((candidate) => candidate === value);
@@ -51,6 +56,8 @@ export function isMarkdownArtifact(type: ArtifactType): boolean {
 }
 
 export interface ArtifactMeta {
+  /** The workflow that submitted this artifact, distinct from its artifact type. */
+  workflow?: WorkflowKind;
   cwd?: string;
   agent?: string;
   /** Agent-native session id, for resume/fork context. */
@@ -59,7 +66,7 @@ export interface ArtifactMeta {
   planPath?: string;
   /** Path to the prototype's entry HTML file on disk. */
   prototypePath?: string;
-  /** Pull request reference the diff came from, so the verdict can be posted back. */
+  /** Pull request reference the diff came from, so the message can be posted back. */
   pr?: string;
   /** herdr pane the submitting agent runs in - the review returns focus there. */
   herdrPane?: string;
@@ -198,14 +205,43 @@ export function isAddressed(annotation: Annotation): boolean {
   return annotation.resolution !== undefined;
 }
 
-export type VerdictKind = "comment" | "approve" | "request_changes";
+/** A harness session bound to a Thread. Harness identity stays outside the Thread itself. */
+export interface HarnessBinding {
+  id: string;
+  threadId: string;
+  harness: string;
+  harnessSessionId: string;
+  createdAt: string;
+  /** Approval whose one unchanged plan resubmission this harness has consumed. */
+  approvedRetryMessageId?: string;
+}
+
+/** Durable routing state for one Message sent to one harness binding. */
+export interface Delivery {
+  id: string;
+  messageId: string;
+  bindingId: string;
+  status: "pending" | "acknowledged";
+  createdAt: string;
+  acknowledgedAt?: string;
+}
 
-export interface Verdict {
-  kind: VerdictKind;
+/** What a harness adapter pulls: routing state plus its immutable Message payload. */
+export interface PendingDelivery {
+  delivery: Delivery;
+  message: Message;
+}
+
+export type MessageOutcome = "approved" | "changes_requested";
+
+export interface Message {
+  /** Stable identity used to deduplicate at-least-once harness delivery. */
+  id: string;
+  outcome: MessageOutcome;
   summary: string;
-  /** The one structured feedback document sent to the agent. */
-  feedback: string;
-  resolvedAt: string;
+  /** The structured review document sent to the harness. */
+  body: string;
+  sentAt: string;
 }
 
 export interface Revision {
@@ -287,7 +323,7 @@ export interface Thread {
    * progress. Undefined = the walk never started.
    */
   viewedPaths?: string[];
-  verdict: Verdict | null;
+  message: Message | null;
   status: SessionStatus;
   createdAt: string;
   /**
@@ -316,9 +352,16 @@ export interface Thread {
   participants?: Identity[];
 }
 
-/** comment and request_changes both map to deny in agent-native contracts. */
-export function verdictAllows(kind: VerdictKind): boolean {
-  return kind === "approve";
+/** Whether a Message releases a harness gate. */
+export function messageAllows(outcome: MessageOutcome): boolean {
+  return outcome === "approved";
+}
+
+let messageSeq = 0;
+
+/** A process-unique, time-sortable Message id for delivery deduplication. */
+export function newMessageId(): string {
+  return `msg_${Date.now().toString(36)}${(messageSeq++).toString(36)}_${Math.random().toString(36).slice(2, 8)}`;
 }
 
 /**