@cueloop/schema 0.1.0-alpha.69 → 0.1.0-alpha.70

package/package.json

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

package/src/anchor.test.ts

@@ -218,7 +218,7 @@ describe("anchors spanning blocks", () => {
 All persistence lives in a new module,
 so a crash can never leave a torn file on disk.
 
-- server/storage/store.ts - the SessionStore class
+- server/storage/store.ts - the ThreadStore class
 - server/storage/schema.ts - the on-disk record shape
 
 ## Phase 2
@@ -235,7 +235,7 @@ so a crash can never leave a torn file on disk.
     expect(anchor.quote).toBe(
       [
         "so a crash can never leave a torn file on disk.",
-        "server/storage/store.ts - the SessionStore class",
+        "server/storage/store.ts - the ThreadStore class",
         "server/storage/schema.ts",
       ].join(BLOCK_SEPARATOR),
     );
@@ -282,7 +282,7 @@ so a crash can never leave a torn file on disk.
 
   test("re-binds fuzzily after a light edit inside the span", () => {
     // Arrange
-    const edited = parseBlocks(SPANNING_DOC.replace("SessionStore class", "SessionStore type"));
+    const edited = parseBlocks(SPANNING_DOC.replace("ThreadStore class", "ThreadStore type"));
 
     // Act
     const resolved = resolveAnchor(anchor, edited);
@@ -308,7 +308,7 @@ so a crash can never leave a torn file on disk.
     // simulate the stale case directly
     const staleAnchor = {
       ...anchor,
-      quote: `${BLOCK_SEPARATOR}server/storage/store.ts - the SessionStore class`,
+      quote: `${BLOCK_SEPARATOR}server/storage/store.ts - the ThreadStore class`,
     };
 
     // Act

package/src/feedback.ts

@@ -11,7 +11,7 @@ import {
   isAgentNote,
   type Annotation,
   type ArtifactType,
-  type ReviewSession,
+  type Thread,
   type VerdictKind,
 } from "./types";
 import { parseBlocks, sectionOf, type Block } from "./markdown";
@@ -246,7 +246,7 @@ function annotationSectionLines(
 }
 
 export function feedbackForSession(
-  session: ReviewSession,
+  session: Thread,
   verdictKind: VerdictKind,
   summary: string,
   actionBodies?: Record<string, string>,

package/src/history.ts

@@ -10,7 +10,7 @@
  * feedback document read what this module derives.
  */
 
-import type { Annotation, ReviewSession, Verdict } from "./types";
+import type { Annotation, Thread, Verdict } from "./types";
 
 export type EntryAuthor = "agent" | "reviewer";
 
@@ -361,7 +361,7 @@ export function removalEntries(
  * Deterministic: the same record migrates to the same ids.
  */
 export function historyFromLinear(
-  session: Pick<ReviewSession, "id" | "revisions" | "annotations" | "verdict" | "createdAt">,
+  session: Pick<Thread, "id" | "revisions" | "annotations" | "verdict" | "createdAt">,
 ): SessionHistory {
   const entries: SessionEntry[] = [];
   let parentId: string | null = null;

package/src/participants.test.ts

@@ -1,8 +1,8 @@
 import { describe, expect, test } from "bun:test";
 import { registerParticipant } from "./participants";
-import { SCHEMA_VERSION, type ReviewSession } from "./types";
+import { SCHEMA_VERSION, type Thread } from "./types";
 
-function emptySession(): ReviewSession {
+function emptySession(): Thread {
   return {
     schemaVersion: SCHEMA_VERSION,
     id: "ses_1",

package/src/participants.ts

@@ -5,7 +5,7 @@
  * "author name" lands here.
  */
 
-import type { Identity, ReviewSession } from "./types";
+import type { Identity, Thread } from "./types";
 
 /**
  * Return the session with `author` present in the participant registry, setting
@@ -13,11 +13,7 @@ import type { Identity, ReviewSession } from "./types";
  * rail renders anonymous) and never erases a name a past visit set. Immutable:
  * the input session is not mutated.
  */
-export function registerParticipant(
-  session: ReviewSession,
-  author: string,
-  name?: string,
-): ReviewSession {
+export function registerParticipant(session: Thread, author: string, name?: string): Thread {
   const participants = session.participants ?? [];
   const existing = participants.find((participant) => participant.id === author);
   const trimmed = name?.trim();

package/src/path-view.test.ts

@@ -8,7 +8,7 @@ import {
   type SessionHistory,
 } from "./history";
 import { applyPathView, viewFollowing, viewOfPath } from "./path-view";
-import type { Annotation, ReviewSession } from "./types";
+import type { Annotation, Thread } from "./types";
 
 const AT = "2026-09-01T10:00:00.000Z";
 
@@ -101,7 +101,7 @@ describe("viewOfPath", () => {
 describe("applyPathView", () => {
   test("writes the view into the record and removes the fields the view leaves empty", () => {
     // Arrange
-    const session: ReviewSession = {
+    const session: Thread = {
       schemaVersion: "1",
       id: "ses_1",
       workspace: { repoRoot: "/repo", branch: "main" },
@@ -139,7 +139,7 @@ describe("viewFollowing", () => {
     history = navigateTo(history, history.entries[1]!.id);
     history = createBranch(history, "alt");
     history = appendEntry(history, { type: "comment", annotationId: "b1", createdAt: AT }).history;
-    const session: ReviewSession = {
+    const session: Thread = {
       schemaVersion: "1",
       id: "ses_1",
       workspace: { repoRoot: "/repo", branch: "main" },
@@ -177,7 +177,7 @@ describe("viewFollowing", () => {
       verdict: null,
       createdAt: AT,
     });
-    const session: ReviewSession = {
+    const session: Thread = {
       schemaVersion: "1",
       id: "ses_1",
       workspace: { repoRoot: "/repo", branch: "main" },

package/src/path-view.ts

@@ -7,7 +7,7 @@
  */
 
 import { derivePath, followBranch, MAIN_BRANCH, pathOf, type SessionHistory } from "./history";
-import type { Annotation, ReviewSession } from "./types";
+import type { Annotation, Thread } from "./types";
 
 export interface PathView {
   /** The last agent revision on the path: what the artifact shows. */
@@ -51,7 +51,7 @@ export function viewOfPath(history: SessionHistory, known: Annotation[]): PathVi
 }
 
 /** Write a path view back into a record, in place; absent fields are removed. */
-export function applyPathView(session: ReviewSession, view: PathView): void {
+export function applyPathView(session: Thread, view: PathView): void {
   session.artifact = { ...session.artifact, content: view.content };
   session.annotations = view.annotations;
   if (view.workingCopy === undefined) delete session.workingCopy;
@@ -66,12 +66,12 @@ export function applyPathView(session: ReviewSession, view: PathView): void {
  * owner's. A followed branch that no longer exists falls back to main. A record
  * without a history travels as it is.
  */
-export function viewFollowing(session: ReviewSession, branch?: string): ReviewSession {
+export function viewFollowing(session: Thread, branch?: string): Thread {
   const asked = branch ?? session.shareBranch ?? MAIN_BRANCH;
   // a branch that no longer exists falls back to main, matching the daemon
   const followed =
     session.history && session.history.tips[asked] === undefined ? MAIN_BRANCH : asked;
-  const shared: ReviewSession = { ...session, shareBranch: followed };
+  const shared: Thread = { ...session, shareBranch: followed };
 
   if (!session.history) return shared;
   const history = followBranch(session.history, followed);

package/src/types.ts

@@ -1,5 +1,5 @@
 /**
- * The ReviewSession primitive. Everything in cueloop renders, annotates, or
+ * The Thread primitive. Everything in cueloop renders, annotates, or
  * resolves this one noun. This module is pure data shapes - no IO, no
  * dependencies beyond the history shapes.
  */
@@ -230,7 +230,7 @@ export interface Identity {
   handle?: string;
 }
 
-export interface ReviewSession {
+export interface Thread {
   schemaVersion: string;
   id: string;
   workspace: WorkspaceKey;
@@ -284,6 +284,9 @@ export interface ReviewSession {
   participants?: Identity[];
 }
 
+/** @deprecated use Thread */
+export type ReviewSession = Thread;
+
 /** comment and request_changes both map to deny in agent-native contracts. */
 export function verdictAllows(kind: VerdictKind): boolean {
   return kind === "approve";