@cueloop/daemon 0.1.0-alpha.98 → 0.1.0-alpha.99

package/src/client.ts

@@ -57,9 +57,11 @@ import {
   PendingDeliverySchema,
 } from "./validate";
 import type { WorkingTreeDiff } from "./working-tree";
+import { DaemonTransportError, type DaemonRequestOptions } from "./client-errors";
+export { DaemonTransportError, type DaemonRequestOptions } from "./client-errors";
 import { DAEMON_VERSION } from "./version";
 import { ThreadAgentSchema } from "./thread-agent-validation";
-import type { AgentComment, ThreadAgentState } from "@cueloop/schema";
+import type { AgentComment, ThreadAgentState, AgentPromptRequest } from "@cueloop/schema";
 
 export type { EventFrame } from "./protocol";
 
@@ -226,7 +228,8 @@ export class DaemonClient implements ThreadClient {
   private daemonVersion: string | undefined;
   private nextId = 1;
   private eventListeners = new Set<(event: EventFrame) => void>();
-  private disconnectListeners = new Set<() => void>();
+  private disconnectListeners = new Set<(reason: DaemonTransportError) => void>();
+  private disconnectReason: DaemonTransportError | undefined;
   private closed = false;
   private role: DaemonRole = "owner";
   private author: string | undefined;
@@ -255,7 +258,15 @@ export class DaemonClient implements ThreadClient {
       if (!options.autostart || err instanceof DaemonClientError) {
         client.close();
 
-        throw err;
+        throw err instanceof DaemonClientError || err instanceof DaemonTransportError
+          ? err
+          : new DaemonTransportError(
+              "connection",
+              "Daemon connection failed",
+              "not_sent",
+              undefined,
+              { cause: err },
+            );
       }
     }
 
@@ -270,7 +281,9 @@ export class DaemonClient implements ThreadClient {
     this.writer = null;
     this.closed = false;
     for (const pendingRequest of this.pending.values())
-      pendingRequest.reject(new Error("daemon connection replaced"));
+      pendingRequest.reject(
+        new DaemonTransportError("connection", "daemon connection replaced", "unknown"),
+      );
     this.pending.clear();
   }
 
@@ -310,7 +323,11 @@ export class DaemonClient implements ThreadClient {
       const output = readFileSync(startupLogPath, "utf8").trim();
       const detail = output ? `\ndaemon startup output:\n${output.slice(-4_096)}` : "";
 
-      throw new Error(`daemon did not come up at ${path}: ${String(lastError)}${detail}`);
+      throw new DaemonTransportError(
+        "connection",
+        `Daemon startup failed at ${path}: ${String(lastError)}${detail}`,
+        "not_sent",
+      );
     } finally {
       rmSync(startupLogPath, { force: true });
     }
@@ -318,15 +335,23 @@ export class DaemonClient implements ThreadClient {
 
   private async dial(path: string): Promise<void> {
     const buffer = new LineBuffer();
+
+    this.disconnectReason = undefined;
     const epoch = ++this.connectionEpoch;
 
     const close = () => {
       if (this.connectionEpoch !== epoch) return;
       this.closed = true;
       for (const pendingRequest of this.pending.values())
-        pendingRequest.reject(new Error("daemon connection closed"));
+        pendingRequest.reject(
+          new DaemonTransportError("connection", "daemon connection closed", "unknown"),
+        );
       this.pending.clear();
-      for (const listener of this.disconnectListeners) listener();
+      const reason =
+        this.disconnectReason ??
+        new DaemonTransportError("connection", "daemon connection closed", "unknown");
+
+      for (const listener of this.disconnectListeners) listener(reason);
     };
 
     if (typeof Bun !== "undefined") {
@@ -399,7 +424,7 @@ export class DaemonClient implements ThreadClient {
   }
 
   /** Notify an adapter when its daemon socket closes so it can reconnect and replay Messages. */
-  onDisconnect(listener: () => void): () => void {
+  onDisconnect(listener: (reason: DaemonTransportError) => void): () => void {
     this.disconnectListeners.add(listener);
 
     return () => this.disconnectListeners.delete(listener);
@@ -410,7 +435,27 @@ export class DaemonClient implements ThreadClient {
 
     try {
       frame = parseInboundFrame(line);
-    } catch {
+    } catch (cause) {
+      this.disconnectReason = new DaemonTransportError(
+        "protocol",
+        "daemon sent a malformed frame",
+        "unknown",
+        undefined,
+        { cause },
+      );
+      for (const pendingRequest of this.pending.values())
+        pendingRequest.reject(
+          new DaemonTransportError(
+            "protocol",
+            "daemon sent a malformed frame",
+            "unknown",
+            undefined,
+            { cause },
+          ),
+        );
+      this.pending.clear();
+      this.close();
+
       return;
     }
     if ("event" in frame) {
@@ -431,49 +476,109 @@ export class DaemonClient implements ThreadClient {
     method: string,
     params: Request["params"],
     resultSchema: v.GenericSchema<unknown, TOutput>,
-    timeoutMs = 30_000,
+    options: number | DaemonRequestOptions = 30_000,
   ): Promise<TOutput> {
-    if (this.closed || !this.socket) return Promise.reject(new Error("not connected"));
+    const { timeoutMs = 30_000, signal } = v.is(v.number(), options)
+      ? { timeoutMs: options }
+      : options;
+
+    if (signal?.aborted)
+      return Promise.reject(
+        new DaemonTransportError("cancelled", `request ${method} cancelled`, "not_sent", method),
+      );
+    if (this.closed || !this.socket)
+      return Promise.reject(
+        new DaemonTransportError("connection", "not connected", "not_sent", method),
+      );
     const id = this.nextId++;
 
     return new Promise<TOutput>((resolve, reject) => {
-      const timer = setTimeout(() => {
+      const cleanup = () => {
+        clearTimeout(timer);
+        signal?.removeEventListener("abort", abort);
         this.pending.delete(id);
-        reject(new Error(`request ${method} timed out`));
+      };
+      const fail = (error: Error) => {
+        cleanup();
+        reject(error);
+      };
+      const abort = () =>
+        fail(
+          new DaemonTransportError("cancelled", `request ${method} cancelled`, "unknown", method),
+        );
+      const timer = setTimeout(() => {
+        fail(new DaemonTransportError("timeout", `request ${method} timed out`, "unknown", method));
       }, timeoutMs);
 
       this.pending.set(id, {
         resolve: (value) => {
-          clearTimeout(timer);
-          resolve(v.parse(resultSchema, value));
-        },
-        reject: (error) => {
-          clearTimeout(timer);
-          reject(error);
+          cleanup();
+          try {
+            const parsed = v.safeParse(resultSchema, value);
+
+            if (!parsed.success)
+              throw new DaemonTransportError(
+                "protocol",
+                `request ${method} returned an invalid result`,
+                "unknown",
+                method,
+                { cause: parsed.issues },
+              );
+            resolve(parsed.output);
+          } catch (cause) {
+            reject(
+              cause instanceof DaemonTransportError
+                ? cause
+                : new DaemonTransportError(
+                    "protocol",
+                    `request ${method} returned an invalid result`,
+                    "unknown",
+                    method,
+                    { cause },
+                  ),
+            );
+          }
         },
+        reject: fail,
       });
-      this.writer!.write(JSON.stringify({ id, method, params }) + "\n");
+      signal?.addEventListener("abort", abort, { once: true });
+      try {
+        this.writer!.write(JSON.stringify({ id, method, params }) + "\n");
+      } catch (cause) {
+        fail(
+          new DaemonTransportError(
+            "connection",
+            `request ${method} could not be written`,
+            "unknown",
+            method,
+            { cause },
+          ),
+        );
+      }
     });
   }
 
   close(): void {
     this.closed = true;
+    for (const pendingRequest of this.pending.values())
+      pendingRequest.reject(
+        new DaemonTransportError("connection", "daemon connection closed", "unknown"),
+      );
+    this.pending.clear();
     this.socket?.end();
   }
 
   // ── typed primitives ─────────────────────────────
   /** Read the agent transcript without starting a model request. */
-  agentGet(id: string): Promise<ThreadAgentState> {
-    return this.request("agent.get", { id }, ThreadAgentSchema);
+  agentGet(id: string, options?: DaemonRequestOptions): Promise<ThreadAgentState> {
+    return this.request("agent.get", { id }, ThreadAgentSchema, options);
   }
   /** Submit a question and optional selected passage to the daemon-owned agent. */
-  agentPrompt(params: {
-    id: string;
-    text: string;
-    context?: string;
-    retry?: string;
-  }): Promise<ThreadAgentState> {
-    return this.request("agent.prompt", params, ThreadAgentSchema);
+  agentPrompt(
+    params: AgentPromptRequest,
+    options?: DaemonRequestOptions,
+  ): Promise<ThreadAgentState> {
+    return this.request("agent.prompt", params, ThreadAgentSchema, options);
   }
   /** Cancel the active turn; the daemon retains partial output. */
   agentCancel(id: string): Promise<ThreadAgentState> {
@@ -681,11 +786,14 @@ export class DaemonClient implements ThreadClient {
     outcome: MessageOutcome,
     summary: string,
     actionBodies?: Record<string, string>,
+    operationId?: string,
+    options?: DaemonRequestOptions,
   ): Promise<Thread> {
     return this.request(
       "session.sendMessage",
-      { id, outcome, summary, actionBodies },
+      { id, outcome, summary, actionBodies, operationId },
       ThreadRecordSchema,
+      options,
     );
   }
   harnessBind(
@@ -764,6 +872,7 @@ export class DaemonClient implements ThreadClient {
 }
 
 export class DaemonClientError extends Error {
+  readonly _tag = "DaemonClientError";
   constructor(
     readonly code: string,
     message: string,

package/src/daemon-sdk-effect.ts

@@ -0,0 +1,113 @@
+import { Context, Effect, Layer, Stream, Queue, Cause, type Scope } from "effect";
+import {
+  connectOwnerSdk,
+  normalizeSdkError,
+  type OwnerDaemonSdk,
+  type DaemonSdkError,
+  type SdkThread,
+  type SdkThreadId,
+} from "./daemon-sdk";
+import type { ConnectOptions } from "./client";
+import type { DaemonRequestOptions } from "./client-errors";
+
+type EffectMethods<Methods> = {
+  readonly [Key in keyof Methods]: Methods[Key] extends (
+    ...args: infer Arguments
+  ) => Promise<infer Result>
+    ? (...args: Arguments) => Effect.Effect<Result, DaemonSdkError>
+    : never;
+};
+
+/** The Effect service owns a scoped connection while daemon-owned work outlives local waits. */
+export class CueloopSdk extends Context.Service<
+  CueloopSdk,
+  {
+    readonly ids: OwnerDaemonSdk["ids"];
+    readonly threads: EffectMethods<OwnerDaemonSdk["threads"]> & {
+      watch(threadId: SdkThreadId): Stream.Stream<SdkThread, DaemonSdkError>;
+    };
+    readonly comments: EffectMethods<OwnerDaemonSdk["comments"]>;
+    readonly sessions: EffectMethods<OwnerDaemonSdk["sessions"]>;
+    readonly agents: EffectMethods<OwnerDaemonSdk["agents"]>;
+  }
+>()("@cueloop/daemon/CueloopSdk") {
+  static ownerLayer(
+    options: Omit<ConnectOptions, "role"> = {},
+  ): Layer.Layer<CueloopSdk, DaemonSdkError> {
+    return Layer.effect(CueloopSdk, acquireOwnerSdk(options).pipe(Effect.map(effectSdkService)));
+  }
+}
+
+function acquireOwnerSdk(
+  options: Omit<ConnectOptions, "role">,
+): Effect.Effect<OwnerDaemonSdk, DaemonSdkError, Scope.Scope> {
+  return Effect.acquireRelease(
+    Effect.tryPromise({ try: () => connectOwnerSdk(options), catch: normalizeSdkError }),
+    (sdk) => Effect.sync(() => sdk.close()),
+  );
+}
+
+function sdkEffect<Result>(
+  request: (options: DaemonRequestOptions) => Promise<Result>,
+  options?: DaemonRequestOptions,
+): Effect.Effect<Result, DaemonSdkError> {
+  return Effect.tryPromise({
+    try: (signal) =>
+      request({
+        ...options,
+        signal: options?.signal ? AbortSignal.any([signal, options.signal]) : signal,
+      }),
+    catch: normalizeSdkError,
+  });
+}
+
+function effectSdkService(sdk: OwnerDaemonSdk): CueloopSdk["Service"] {
+  return {
+    ids: sdk.ids,
+    threads: {
+      watch: (threadId) =>
+        Stream.callback<SdkThread, DaemonSdkError>(
+          Effect.fn(function* (queue) {
+            yield* Effect.acquireRelease(
+              Effect.sync(() =>
+                sdk.observeThread({
+                  threadId,
+                  onValue: (thread) => {
+                    Queue.offerUnsafe(queue, thread);
+                  },
+                  onError: (error) => {
+                    if (error._tag === "DaemonClientError" || error.kind === "protocol")
+                      Queue.failCauseUnsafe(queue, Cause.fail(error));
+                  },
+                }),
+              ),
+              (stop) => Effect.sync(stop),
+            );
+          }),
+          { bufferSize: 1, strategy: "sliding" },
+        ),
+      get: (id, options) => sdkEffect((request) => sdk.threads.get(id, request), options),
+      create: (input, options) =>
+        sdkEffect((request) => sdk.threads.create(input, request), options),
+    },
+    comments: {
+      add: (input, options) => sdkEffect((request) => sdk.comments.add(input, request), options),
+      list: (id, options) => sdkEffect((request) => sdk.comments.list(id, request), options),
+      reply: (input, options) =>
+        sdkEffect((request) => sdk.comments.reply(input, request), options),
+    },
+    sessions: {
+      sendMessage: (input, options) =>
+        sdkEffect((request) => sdk.sessions.sendMessage(input, request), options),
+    },
+    agents: {
+      get: (id, options) => sdkEffect((request) => sdk.agents.get(id, request), options),
+      prompt: (input, options) =>
+        sdkEffect((request) => sdk.agents.prompt(input, request), options),
+      replyToComment: (input, options) =>
+        sdkEffect((request) => sdk.agents.replyToComment(input, request), options),
+      wait: (input, options) => sdkEffect((request) => sdk.agents.wait(input, request), options),
+      cancel: (id, options) => sdkEffect((request) => sdk.agents.cancel(id, request), options),
+    },
+  };
+}

package/src/daemon-sdk.test.ts

@@ -0,0 +1,306 @@
+import { expect, test } from "bun:test";
+import { mkdtempSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { Effect, Stream } from "effect";
+import { DaemonServer } from "./server";
+import { DaemonClient } from "./client";
+import { connectOwnerSdk, connectReviewSdk } from "./daemon-sdk";
+import { CueloopSdk } from "./daemon-sdk-effect";
+
+import { createTestSdkHarness } from "../../../test/helpers/sdk-harness";
+
+test("SDK accepts once, waits for its own submission and cancelling a wait preserves work", async () => {
+  const home = mkdtempSync(join(tmpdir(), "cueloop-sdk-"));
+  const harness = createTestSdkHarness();
+  const server = new DaemonServer({
+    home,
+    idleExitMs: 0,
+    threadAgent: { enabled: true, adapter: harness.adapter },
+  });
+
+  server.start();
+  const sdk = await connectOwnerSdk({ home });
+
+  try {
+    const thread = await sdk.threads.create({
+      workspace: { repoRoot: home, branch: "main" },
+      artifact: { type: "plan", content: "Explain this", meta: {} },
+    });
+    const input = {
+      threadId: thread.id,
+      operationId: sdk.ids.operation("ask-once"),
+      text: "Explain",
+    };
+    const accepted = await sdk.agents.prompt(input);
+    const duplicate = await sdk.agents.prompt(input);
+
+    expect(duplicate).toEqual(accepted);
+    expect((await sdk.agents.get(thread.id)).submissions).toHaveLength(1);
+    await expect(sdk.agents.prompt({ ...input, text: "Different" })).rejects.toMatchObject({
+      code: "operation_conflict",
+    });
+    const cancel = new AbortController();
+    const abandonedWait = sdk.agents.wait(accepted, { signal: cancel.signal });
+
+    cancel.abort();
+    await expect(abandonedWait).rejects.toMatchObject({ kind: "cancelled" });
+    expect(harness.cancelled()).toBe(0);
+    const completed = sdk.agents.wait(accepted);
+
+    harness.complete();
+    expect((await completed).outcome).toBe("completed");
+    const message = {
+      threadId: thread.id,
+      operationId: sdk.ids.operation("approve-once"),
+      outcome: "approved" as const,
+      summary: "Good",
+    };
+    const sent = await sdk.sessions.sendMessage(message);
+
+    expect(await sdk.sessions.sendMessage(message)).toEqual(sent);
+  } finally {
+    sdk.close();
+    server.stop();
+    rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("Effect scope supports review comments and typed owner-only capabilities", async () => {
+  const home = mkdtempSync(join(tmpdir(), "cueloop-sdk-effect-"));
+  const server = new DaemonServer({ home, idleExitMs: 0 });
+
+  server.start();
+
+  try {
+    const program = Effect.gen(function* () {
+      const sdk = yield* CueloopSdk;
+      const thread = yield* sdk.threads.create({
+        workspace: { repoRoot: home, branch: "main" },
+        artifact: { type: "plan", content: "Discuss this", meta: {} },
+      });
+      const snapshots = yield* sdk.threads.watch(thread.id).pipe(Stream.take(1), Stream.runCollect);
+
+      expect(snapshots[0]?.id).toBe(thread.id);
+      const client = yield* Effect.promise(() => DaemonClient.connect({ home }));
+
+      try {
+        yield* Effect.promise(() =>
+          client.sessionComment(thread.id, {
+            id: "comment",
+            kind: "comment",
+            anchor: { quote: "Discuss", prefix: "", suffix: " this" },
+            body: "Why?",
+          }),
+        );
+      } finally {
+        client.close();
+      }
+      yield* sdk.comments.reply({
+        threadId: thread.id,
+        commentId: "comment",
+        body: "Because",
+        replyId: "reply",
+      });
+      const comments = yield* sdk.comments.list(thread.id);
+
+      expect(comments.at(-1)).toMatchObject({ id: "reply", replyTo: "comment", body: "Because" });
+
+      return thread.id;
+    });
+    const id = await Effect.runPromise(
+      program.pipe(Effect.provide(CueloopSdk.ownerLayer({ home }))),
+    );
+    const review = await connectReviewSdk({ home, role: "agent", author: "review-agent" });
+
+    try {
+      expect((await review.threads.get(id)).id).toBe(id);
+      expect("agents" in review).toBe(false);
+      const capped = await DaemonClient.connect({ home, role: "agent", author: "review-agent" });
+
+      try {
+        await expect(capped.agentPrompt({ id, text: "Forbidden" })).rejects.toMatchObject({
+          code: "forbidden",
+        });
+      } finally {
+        capped.close();
+      }
+    } finally {
+      review.close();
+    }
+  } finally {
+    server.stop();
+    rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("lost response after message commit is uncertain and an explicit retry survives server restart", async () => {
+  const home = mkdtempSync(join(tmpdir(), "cueloop-sdk-lost-response-"));
+  let server = new DaemonServer({ home, idleExitMs: 0 });
+
+  server.start();
+  const connection = await DaemonClient.connect({ home });
+
+  try {
+    const thread = await connection.sessionCreate(
+      { repoRoot: home, branch: "main" },
+      { type: "plan", content: "Approve this", meta: {} },
+    );
+    const send = server.core.sessionSendMessage.bind(server.core);
+
+    server.core.sessionSendMessage = (...args) => {
+      const result = send(...args);
+
+      connection.close();
+
+      return result;
+    };
+    await expect(
+      connection.sessionSendMessage(thread.id, "approved", "Done", undefined, "lost-ack"),
+    ).rejects.toMatchObject({ kind: "connection", certainty: "unknown" });
+    const original = server.core.sessionGet(thread.id).message;
+
+    server.stop();
+    server = new DaemonServer({ home, idleExitMs: 0 });
+    server.start();
+    const sdk = await connectOwnerSdk({ home });
+
+    try {
+      expect(
+        await sdk.sessions.sendMessage({
+          threadId: sdk.ids.thread(thread.id),
+          operationId: sdk.ids.operation("lost-ack"),
+          outcome: "approved",
+          summary: "Done",
+        }),
+      ).toEqual(original!);
+      expect(
+        (await sdk.threads.get(sdk.ids.thread(thread.id))).history?.entries.filter(
+          (entry) => entry.type === "message",
+        ),
+      ).toHaveLength(1);
+    } finally {
+      sdk.close();
+    }
+  } finally {
+    connection.close();
+    server.stop();
+    rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("Effect interruption aborts the completion wait without stopping the harness", async () => {
+  const { Fiber } = await import("effect");
+  const home = mkdtempSync(join(tmpdir(), "cueloop-sdk-interrupt-"));
+  const harness = createTestSdkHarness();
+  const server = new DaemonServer({
+    home,
+    idleExitMs: 0,
+    threadAgent: { enabled: true, adapter: harness.adapter },
+  });
+
+  server.start();
+
+  try {
+    const program = Effect.gen(function* () {
+      const sdk = yield* CueloopSdk;
+      const thread = yield* sdk.threads.create({
+        workspace: { repoRoot: home, branch: "main" },
+        artifact: { type: "plan", content: "Explain", meta: {} },
+      });
+      const accepted = yield* sdk.agents.prompt({
+        threadId: thread.id,
+        operationId: sdk.ids.operation("interrupt"),
+        text: "Explain",
+      });
+      const fiber = yield* Effect.forkChild(sdk.agents.wait(accepted));
+
+      yield* Effect.yieldNow;
+      yield* Fiber.interrupt(fiber);
+      expect(harness.cancelled()).toBe(0);
+      yield* Effect.sync(() => harness.complete());
+      const completed = yield* sdk.agents.wait(accepted);
+
+      expect(completed.outcome).toBe("completed");
+    });
+
+    await Effect.runPromise(program.pipe(Effect.provide(CueloopSdk.ownerLayer({ home }))));
+  } finally {
+    server.stop();
+    rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("lost prompt acknowledgement replays original acceptance after restart without a harness", async () => {
+  const home = mkdtempSync(join(tmpdir(), "cueloop-sdk-lost-prompt-"));
+  const harness = createTestSdkHarness();
+  let server = new DaemonServer({
+    home,
+    idleExitMs: 0,
+    threadAgent: { enabled: true, adapter: harness.adapter },
+  });
+
+  server.start();
+  const connection = await DaemonClient.connect({ home });
+
+  try {
+    const thread = await connection.sessionCreate(
+      { repoRoot: home, branch: "main" },
+      { type: "plan", content: "Explain", meta: {} },
+    );
+
+    await connection.subscribe();
+    connection.onEvent((event) => {
+      if (event.event === "agent.updated") connection.close();
+    });
+    await expect(
+      connection.agentPrompt({ id: thread.id, text: "Once", operationId: "lost-prompt" }),
+    ).rejects.toMatchObject({ kind: "connection", certainty: "unknown" });
+    const inspect = await DaemonClient.connect({ home });
+    const accepted = await inspect.agentGet(thread.id);
+
+    inspect.close();
+    server.stop();
+    server = new DaemonServer({ home, idleExitMs: 0, threadAgent: { enabled: true } });
+    server.start();
+    const sdk = await connectOwnerSdk({ home });
+
+    try {
+      const replay = await sdk.agents.prompt({
+        threadId: sdk.ids.thread(thread.id),
+        operationId: sdk.ids.operation("lost-prompt"),
+        text: "Once",
+      });
+
+      expect(replay.submissionIds).toEqual(
+        accepted.promptOperations![0]!.result.map(sdk.ids.submission),
+      );
+      expect((await sdk.agents.get(sdk.ids.thread(thread.id))).submissions).toHaveLength(1);
+      expect((await sdk.agents.wait(replay)).outcome).toBe("failed");
+    } finally {
+      sdk.close();
+    }
+  } finally {
+    connection.close();
+    server.stop();
+    rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("SDK identity types distinguish Thread, operation, and submission capabilities", () => {
+  type ThreadId = import("./daemon-sdk").SdkThreadId;
+  type OperationId = import("./daemon-sdk").SdkOperationId;
+  type SubmissionId = import("./daemon-sdk").SdkSubmissionId;
+  type Review = import("./daemon-sdk").ReviewDaemonSdk;
+  const operationIsThread: OperationId extends ThreadId ? true : false = false;
+  const submissionIsOperation: SubmissionId extends OperationId ? true : false = false;
+  const reviewHasAgent: "agents" extends keyof Review ? true : false = false;
+  const reviewCreatesThread: "create" extends keyof Review["threads"] ? true : false = false;
+
+  expect([operationIsThread, submissionIsOperation, reviewHasAgent, reviewCreatesThread]).toEqual([
+    false,
+    false,
+    false,
+    false,
+  ]);
+});