@alexkroman1/aai-cli 8.2.0 → 9.0.0

package/dist/templates/recap-workflow/agent.test.ts

@@ -34,7 +34,6 @@ import {
   parseSchemaInput,
   schemaInputIssues,
   toolRunner,
-  withDiscoveredTools,
 } from "@alexkroman1/aai/testing";
 import {
   installStubStepFetch,
@@ -44,7 +43,6 @@ import {
 import type { WorkflowRunSnapshot } from "@alexkroman1/aai/workflow-api";
 import { beforeEach, describe, expect, test, vi } from "vitest";
 import { createHook, type Hook, sleep } from "workflow";
-import authoredAgent from "./agent.ts";
 import { recap } from "./shared.ts";
 import {
   askWhetherToKeep,
@@ -72,18 +70,8 @@ vi.mock("workflow", async (importActual) => ({
   createHook: vi.fn(),
 }));
 
-/**
- * The def a DEPLOYED agent runs: authored, plus what `tools/` declares.
- *
- * The glob is written HERE rather than reached for from a shared helper because
- * this file SHIPS: it is what a scaffolded project runs, so it may not import
- * anything outside its own template, and `import.meta.glob` is expanded against
- * the file containing it either way. This is the pattern a user writes.
- */
-const agentDef = withDiscoveredTools(
-  authoredAgent,
-  import.meta.glob("./tools/*.ts", { eager: true }),
-);
+/** The def a DEPLOYED agent runs: authored, plus what `tools/` declares. */
+import agentDef from "virtual:aai/agent";
 
 /**
  * Every tool here is driven through the agent's own table, by the name the model
@@ -151,10 +139,7 @@ describe("the agent declares its workflow", () => {
     expect(agentDef.requiredEnv).toContain("ASSEMBLYAI_API_KEY");
   });
 
-  test("as a VOICE agent, not a workflow app", () => {
-    // The distinction this template exists on: a caller is on the line, so the
-    // run is a handoff. `link-digest` is the other shape.
-    expect(agentDef.page).toBeUndefined();
+  test("discovers every tool in tools/, by file name", () => {
     // Discovered, not declared: every name here is a file in `tools/`.
     expect(Object.keys(agentDef.tools).sort()).toEqual([
       "cancel_recap",

package/dist/templates/recap-workflow/agent.ts

@@ -65,11 +65,13 @@
  * the run really submits the recording, really polls it, and really deletes it
  * when it has to.
  *
- * Storage is what makes it DURABLE (`aai storage enable`, Settings → Database
- * in the studio, or `DATABASE_URL` under `aai dev`) — runs and the
- * correlation-key index both live there. Without it both live in the process:
- * the flow still runs, and a recap parked on a callback does not survive the
- * agent restarting, redeploying or going idle.
+ * Runs are DURABLE on the platform with nothing to configure, so a recap parked
+ * on a callback survives a restart, a redeploy and an idle sandbox.
+ *
+ * A `DATABASE_URL` you supply moves the correlation-key index out of memory,
+ * which is what lets `find()` resolve that parked run by key afterwards. Under
+ * `aai dev` with none, both the runs and the index go with the process: the flow
+ * still runs, it just does not outlive the server.
  */
 
 import { agent } from "@alexkroman1/aai";

package/dist/templates/redline/agent.test.ts

@@ -42,19 +42,6 @@ const INPUT: RedlineInput = {
 // ─── 1. The declaration ──────────────────────────────────────────────────────
 
 describe("the agent declares itself a workflow app", () => {
-  test("its front door is a page, not a microphone", () => {
-    // The port's own decision: seven long-form model calls in sequence is not
-    // something anyone holds a phone for.
-    expect(agentDef.page).toBe("static");
-  });
-
-  test("it declares no voice pipeline and no tools, because nothing talks", () => {
-    expect(agentDef.stt).toBeUndefined();
-    expect(agentDef.llm).toBeUndefined();
-    expect(agentDef.tts).toBeUndefined();
-    expect(agentDef.tools).toEqual({});
-  });
-
   test("under the name the page starts a run by", () => {
     // `useWorkflowSubmit("redline")` in client.tsx names this key. Nothing else
     // records it, so a rename here is a 400 there rather than a compile error.

package/dist/templates/redline/agent.ts

@@ -25,8 +25,9 @@
  *   `ToolContext`, so it reads that key with `requireStepEnv` (inside
  *   `stepGenerate`); `requiredEnv` below is what makes a deploy check for it
  *   rather than letting the first run find out.
- * - **Storage** (`aai storage enable`, or `DATABASE_URL` under `aai dev`) — runs
- *   live there.
+ * - **Nothing for durability.** A deployed app's runs live on the platform. A
+ *   `DATABASE_URL` (yours, if you set one) only moves the correlation-key index
+ *   out of memory.
  */
 
 import { workflow, workflowApp } from "@alexkroman1/aai";

package/dist/templates/redline/client.tsx

@@ -28,7 +28,7 @@
 
 import "@alexkroman1/aai-ui/styles.css";
 import { plural } from "@alexkroman1/aai/utils";
-import type { WorkflowOutputOf } from "@alexkroman1/aai/workflow-api";
+import type { WorkflowInputOf, WorkflowOutputOf } from "@alexkroman1/aai/workflow-api";
 import {
   Form,
   type FormValues,
@@ -62,10 +62,15 @@ const WORKFLOW = "redline";
  * thing that otherwise gets half-done in three places. Blank lines go, so a
  * trailing newline is not a requirement to cover "".
  */
-export function toInput(values: FormValues): FormValues {
+export function toInput(values: FormValues): WorkflowInputOf<typeof redline> {
   const raw = typeof values.mustCover === "string" ? values.mustCover : "";
+  // The scalars ride through as the form collected them — strings from the DOM,
+  // which the WORKFLOW's schema coerces and validates server-side. Only
+  // `mustCover` is reshaped here, because no control renders a `string[]`.
+  // The assertion is on the scalars alone and is what `submitForm` exists to
+  // avoid needing anywhere a page is not doing this reshaping deliberately.
   return {
-    ...values,
+    ...(values as Omit<WorkflowInputOf<typeof redline>, "mustCover">),
     mustCover: raw
       .split("\n")
       .map((line) => line.trim())
@@ -74,7 +79,7 @@ export function toInput(values: FormValues): FormValues {
 }
 
 function RedlineDesk() {
-  const { submit, run, pending, error, reset } = useWorkflowSubmit<Redline>(WORKFLOW);
+  const { submit, run, pending, error, reset } = useWorkflowSubmit<typeof redline>(WORKFLOW);
 
   return (
     <main className="mx-auto flex max-w-2xl flex-col gap-8 p-8">

package/dist/templates/research-workflow/agent.eval.test.ts

@@ -1,5 +1,3 @@
-/// <reference types="vite/client" />
-
 // An EVAL: does this desk actually behave? Run it with `aai eval`.
 //
 // `agent.test.ts` drives the four tools against a STUBBED `ctx.workflows` and
@@ -29,14 +27,6 @@
 //
 // And what no eval here can see at all: anything below the audio boundary —
 // endpointing, barge-in, whether two sentences merged into one turn.
-import { withDiscoveredTools } from "@alexkroman1/aai/testing";
-import { installStubStepFetch } from "@alexkroman1/aai/testing/vitest";
-import type { EvalToolCall, EvalWorkflows } from "@alexkroman1/aai-runtime/eval";
-import { describeEval } from "@alexkroman1/aai-runtime/eval/vitest";
-import { expect } from "vitest";
-import { z } from "zod";
-import authoredAgent from "./agent.ts";
-import { research } from "./shared.ts";
 
 /**
  * The def a DEPLOYED agent runs: authored, plus what `tools/` declares.
@@ -50,10 +40,13 @@ import { research } from "./shared.ts";
  * The glob is written here rather than reached for from a shared helper because
  * this file SHIPS — see `agent.test.ts`.
  */
-const agentDef = withDiscoveredTools(
-  authoredAgent,
-  import.meta.glob("./tools/*.ts", { eager: true }),
-);
+import agentDef from "virtual:aai/agent";
+import { installStubStepFetch } from "@alexkroman1/aai/testing/vitest";
+import type { EvalToolCall, EvalWorkflows } from "@alexkroman1/aai-runtime/eval";
+import { describeEval } from "@alexkroman1/aai-runtime/eval/vitest";
+import { expect } from "vitest";
+import { z } from "zod";
+import { research } from "./shared.ts";
 
 /**
  * The key the run's steps read with `requireStepEnv`.

package/dist/templates/research-workflow/agent.test.ts

@@ -28,14 +28,12 @@ import {
   type StubGatewayCall,
   schemaInputIssues,
   toolRunner,
-  withDiscoveredTools,
 } from "@alexkroman1/aai/testing";
 import { mockWorkflows, installStubGateway as stubGateway } from "@alexkroman1/aai/testing/vitest";
 import { visitWebpage, webSearch } from "@alexkroman1/aai/tools";
 import type { WorkflowRunSnapshot } from "@alexkroman1/aai/workflow-api";
 import { beforeEach, describe, expect, test, vi } from "vitest";
 import { FatalError, RetryableError } from "workflow";
-import authoredAgent from "./agent.ts";
 import { research } from "./shared.ts";
 import {
   countSources,
@@ -63,18 +61,8 @@ vi.mock("@alexkroman1/aai/tools", () => ({
   visitWebpage: vi.fn(async () => ({ content: "The page body." })),
 }));
 
-/**
- * The def a DEPLOYED agent runs: authored, plus what `tools/` declares.
- *
- * The glob is written HERE rather than reached for from a shared helper because
- * this file SHIPS: it is what a scaffolded project runs, so it may not import
- * anything outside its own template, and `import.meta.glob` is expanded against
- * the file containing it either way. This is the pattern a user writes.
- */
-const agentDef = withDiscoveredTools(
-  authoredAgent,
-  import.meta.glob("./tools/*.ts", { eager: true }),
-);
+/** The def a DEPLOYED agent runs: authored, plus what `tools/` declares. */
+import agentDef from "virtual:aai/agent";
 
 /**
  * Every tool here is driven through the agent's own table, by the name the model
@@ -404,7 +392,7 @@ describe("the steps that research", () => {
 
     const note = await investigate(brief, "Tool use");
 
-    expect(webSearch).toHaveBeenCalledWith({ query: "otter tool use", max_results: 5 });
+    expect(webSearch).toHaveBeenCalledWith({ query: "otter tool use", maxResults: 5 });
     expect(visitWebpage).toHaveBeenCalledWith("https://otters.example/tools");
     expect(note.findings).toContain("crack shellfish");
     expect(note.sources).toEqual([{ title: "Otters", url: "https://otters.example/tools" }]);

package/dist/templates/research-workflow/agent.ts

@@ -69,11 +69,13 @@
  * thing that changes under `aai dev` (the key has to be in `.env`, not just your
  * shell).
  *
- * Storage is what makes it DURABLE (`aai storage enable`, Settings → Database
- * in the studio, or `DATABASE_URL` under `aai dev`) — runs and the key index
- * both live there. Without it both live in the process, so a run in flight is
- * lost when the agent restarts, redeploys or goes idle; everything below still
- * works, which is what lets you try it before provisioning anything.
+ * Runs are DURABLE on the platform with nothing to configure — they live on the
+ * platform's own database and survive a restart, a redeploy and an idle sandbox.
+ *
+ * A `DATABASE_URL` you supply (a secret when deployed, `.env` under `aai dev`)
+ * buys the key index, which is what lets `find()` resolve a run by key across a
+ * restart. Under `aai dev` with none, the runs go with the process too —
+ * everything below still works, which is what lets you try it first.
  */
 
 import { agent } from "@alexkroman1/aai";

package/dist/templates/research-workflow/workflows/research.ts

@@ -65,7 +65,13 @@ import {
   RESEARCH_SYSTEM,
 } from "./prompts.ts";
 
-/** Angles investigated at once. The far side of every one is a rate limit. */
+/**
+ * Angles investigated at once. The far side of every one is a rate limit.
+ *
+ * Inside the workflow world's default worker concurrency (three on the
+ * `DATABASE_URL` path), so this width is what really runs — see "The WINDOW is
+ * not the concurrency" in `mapConcurrent`.
+ */
 const ANGLE_CONCURRENCY = 2;
 
 /**
@@ -424,7 +430,7 @@ async function search(query: string): Promise<{ summary: string; sources: Source
   try {
     const results = await webSearch<{ results?: { title?: string; url?: string }[] }>({
       query,
-      max_results: SEARCH_RESULTS,
+      maxResults: SEARCH_RESULTS,
     });
     // The `catch` below was written for exactly this and could not reach it:
     // `webSearch` ANSWERS with `{ error }` rather than throwing, so a refused

package/dist/templates/retail/agent.eval.test.ts

@@ -1,8 +1,14 @@
-/// <reference types="vite/client" />
-
-import { withSystemPrompt } from "@alexkroman1/aai/manifest";
+/**
+ * The def a DEPLOYED agent runs: authored, plus what `tools/` declares, plus
+ * what `system-prompt.md` says.
+ *
+ * Driving the raw default export instead would measure a seventeen-tool desk
+ * with no tools and the FRAMEWORK DEFAULT prompt — and for this template that
+ * is the whole subject: the authenticate-first discipline, the three-step
+ * readback and the one-customer-per-call rule all live in that file.
+ */
+import retailAgent from "virtual:aai/agent";
 import type { SessionEvent } from "@alexkroman1/aai/protocol";
-import { withDiscoveredTools } from "@alexkroman1/aai/testing";
 // An EVAL: does this desk actually behave? Run it with `aai eval`.
 //
 // `agent.test.ts` drives each tool directly and asserts about its result;
@@ -29,27 +35,6 @@ import { type EvalSession, type EvalTurn, lastStateIn } from "@alexkroman1/aai-r
 import { describeEval } from "@alexkroman1/aai-runtime/eval/vitest";
 import { expect } from "vitest";
 import { z } from "zod";
-import authoredAgent from "./agent.ts";
-import systemPrompt from "./system-prompt.md?raw";
-
-/**
- * The def a DEPLOYED agent runs: authored, plus what `tools/` declares, plus
- * what `system-prompt.md` says.
- *
- * BOTH wrappers are load-bearing and neither is applied by `agent()` — they are
- * applied by the BUILD (`aai build` enumerates `tools/` and resolves the prompt
- * file), so an eval driving the raw default export would measure a
- * seventeen-tool desk with no tools and the FRAMEWORK DEFAULT prompt. For this
- * template that is the whole subject: the authenticate-first discipline, the
- * three-step readback and the one-customer-per-call rule are all in that file.
- *
- * The glob is written here rather than reached for from a shared helper because
- * this file SHIPS — see `registry.test.ts`.
- */
-const retailAgent = withSystemPrompt(
-  withDiscoveredTools(authoredAgent, import.meta.glob("./tools/*.ts", { eager: true })),
-  systemPrompt,
-);
 
 /** Harper Brown: exactly ONE pending order, which is what makes "cancel my
  *  pending order" unambiguous — `resolveOrder` refuses a reference that matches

package/dist/templates/retail/registry.test.ts

@@ -1,9 +1,6 @@
-/// <reference types="vite/client" />
-
 import { isToolFailure, type ToolContext } from "@alexkroman1/aai";
-import { createToolContext, withDiscoveredTools } from "@alexkroman1/aai/testing";
+import { createToolContext, deployedAgent } from "@alexkroman1/aai/testing";
 import { describe, expect, test } from "vitest";
-import authoredAgent from "./agent.ts";
 import { callFlow, retailSlot } from "./store.ts";
 
 /** Tools that legitimately run before the caller is identified — the six
@@ -20,17 +17,23 @@ const PUBLIC_TOOLS = new Set([
 ]);
 
 /**
- * The def a DEPLOYED agent runs: authored, plus what `tools/` declares.
+ * The def a DEPLOYED agent runs, lowered BY HAND — the one place in the
+ * templates that still does.
  *
- * The glob is written HERE rather than reached for from a shared helper because
- * this file SHIPS: it is what a scaffolded project runs, so it may not import
- * anything outside its own template, and `import.meta.glob` is expanded against
- * the file containing it either way. This is the pattern a user writes.
+ * `virtual:aai/agent` is what every other spec imports and what a user should
+ * reach for. This file is the exception on purpose: its whole subject is the
+ * tool REGISTRY, so doing the discovery explicitly is the thing under test
+ * rather than setup around it. It is also the worked example for a project
+ * whose runner is not vitest, and so cannot register the plugin.
  */
-const retailAgent = withDiscoveredTools(
-  authoredAgent,
-  import.meta.glob("./tools/*.ts", { eager: true }),
-);
+import authoredAgent from "./agent.ts";
+import systemPrompt from "./system-prompt.md?raw";
+
+const retailAgent = deployedAgent(authoredAgent, {
+  tools: import.meta.glob("./tools/*.ts", { eager: true }),
+  systemPrompt,
+});
+
 const registry = Object.entries(retailAgent.tools);
 
 // `createToolContext()` rather than a cast: it carries a real slot store (the

package/dist/templates/solo-rpg/agent.eval.test.ts

@@ -1,5 +1,3 @@
-/// <reference types="vite/client" />
-
 // An EVAL: does the story machine actually hold? Run it with `aai eval`.
 //
 // `agent.test.ts` drives each tool directly against a context it made itself,
@@ -25,15 +23,6 @@
 // What no eval here can see: anything below the audio boundary — endpointing,
 // barge-in, a sentence split across two turns. Those need real paced audio.
 
-import { withSystemPrompt } from "@alexkroman1/aai/manifest";
-import { withDiscoveredTools } from "@alexkroman1/aai/testing";
-import { type EvalTurn, toolResultIn } from "@alexkroman1/aai-runtime/eval";
-import { describeEval } from "@alexkroman1/aai-runtime/eval/vitest";
-import { expect } from "vitest";
-import { z } from "zod";
-import authoredAgent from "./agent.ts";
-import systemPrompt from "./system-prompt.md?raw";
-
 /**
  * The def a DEPLOYED agent runs, assembled the way the build assembles it: the
  * authored export, plus what `tools/` declares, plus `system-prompt.md`.
@@ -43,10 +32,11 @@ import systemPrompt from "./system-prompt.md?raw";
  * eval run against the framework default prompt would measure an agent nobody
  * deployed.
  */
-const agentDef = withSystemPrompt(
-  withDiscoveredTools(authoredAgent, import.meta.glob("./tools/*.ts", { eager: true })),
-  systemPrompt,
-);
+import agentDef from "virtual:aai/agent";
+import { type EvalTurn, toolResultIn } from "@alexkroman1/aai-runtime/eval";
+import { describeEval } from "@alexkroman1/aai-runtime/eval/vitest";
+import { expect } from "vitest";
+import { z } from "zod";
 
 /**
  * What each tool this file drives answers, off the wire.

package/dist/templates/solo-rpg/agent.test.ts

@@ -1,4 +1,4 @@
-import type { Db, ToolContext, ToolDef, ToolInputSchema } from "@alexkroman1/aai";
+import type { ToolContext, ToolDef, ToolInputSchema } from "@alexkroman1/aai";
 import { isToolFailure } from "@alexkroman1/aai";
 import { createToolContext, ok } from "@alexkroman1/aai/testing";
 import { describe, expect, test, vi } from "vitest";
@@ -18,44 +18,17 @@ import {
 import actionRoll from "./tools/action_roll.ts";
 import burnMomentum from "./tools/burn_momentum.ts";
 import checkState from "./tools/check_state.ts";
-import loadGame from "./tools/load_game.ts";
 import oracle from "./tools/oracle.ts";
-import saveGame from "./tools/save_game.ts";
 import setupCharacter from "./tools/setup_character.ts";
 import updateState from "./tools/update_state.ts";
 
 // ── Test doubles ─────────────────────────────────────────────────────────────
 
-/**
- * Map-backed fake of the app database, implementing exactly the three SQL
- * statements the shared save-slot helpers emit (create table / select /
- * upsert). Values are stored parsed, the way a postgres driver returns jsonb.
- */
-function makeDb(): { db: Db; rows: Map<string, unknown> } {
-  const rows = new Map<string, unknown>();
-  const db: Db = {
-    async query<T>(sql: string, params: unknown[] = []): Promise<T[]> {
-      if (sql.startsWith("create table if not exists app_state")) return [];
-      if (sql.startsWith("select value from app_state")) {
-        const key = params[0] as string;
-        return rows.has(key) ? ([{ value: structuredClone(rows.get(key)) }] as T[]) : [];
-      }
-      if (sql.startsWith("insert into app_state")) {
-        const [key, json] = params as [string, string];
-        rows.set(key, JSON.parse(json)); // $2::jsonb — parsed like postgres would
-        return [];
-      }
-      throw new Error(`unexpected SQL in test: ${sql}`);
-    },
-  };
-  return { db, rows };
-}
-
 /** `send` is a spy rather than the recorder `createToolContext` installs,
  *  because this suite asserts call counts on it. Each call gets its own slot
  *  store, which is what makes two contexts two games. */
-function makeCtx(db: Db = makeDb().db): ToolContext {
-  return createToolContext({ db, send: vi.fn() });
+function makeCtx(): ToolContext {
+  return createToolContext({ send: vi.fn() });
 }
 
 const SETUP_ARGS = {
@@ -588,65 +561,13 @@ describe("update_state", () => {
     expect(
       params.parse({ addClockSegments: 6, updateNpcBond: 4, timeOfDay: "night" }),
     ).toBeTruthy();
-
-    const slotParams = saveGame.inputSchema!;
-    expect(() => slotParams.parse({ slot: "../../etc" })).toThrow();
-    expect(() => slotParams.parse({ slot: "a".repeat(40) })).toThrow();
-    expect(slotParams.parse({ slot: "chapter-2" })).toBeTruthy();
   });
 });
 
-// ── save_game / load_game: cross-session persistence via ctx.db ──────────────
-
-describe("save_game / load_game", () => {
-  test("a save made in one session loads in a later session", async () => {
-    const { db, rows } = makeDb();
-
-    // Session A plays and saves.
-    const sessionA = makeCtx(db);
-    const played = playingState();
-    played.playerName = "Kael";
-    played.sceneCount = 7;
-    seedPlaying(sessionA, played);
-    const saved = ok<{ saved: boolean; slot: string }>(
-      await saveGame.execute({ slot: "chapter-2" }, sessionA),
-    );
-    expect(saved.saved).toBe(true);
-    expect(saved.slot).toBe("chapter-2");
-    expect(rows.get("save:chapter-2")).toMatchObject({ playerName: "Kael", sceneCount: 7 });
-
-    // Session B (a fresh game slot, the same app db) resumes it.
-    const sessionB = makeCtx(db);
-    // `load_game` is a plain `tool()`, so its result is the union its body
-    // writes rather than a position envelope — the guard is what picks the
-    // arm, where the old `as Record<string, unknown>` picked neither.
-    const loaded = await loadGame.execute({ slot: "chapter-2" }, sessionB);
-    if (isToolFailure(loaded)) throw new Error(`load refused: ${loaded.error}`);
-    expect(loaded.loaded).toBe(true);
-    expect(loaded.playerName).toBe("Kael");
-    expect(loaded.sceneCount).toBe(7);
-    expect(gameSlot.get(sessionB).playerName).toBe("Kael");
-  });
-
-  test("loading a missing slot reports an error instead of resetting the game", async () => {
-    const ctx = makeCtx();
-    const result = await loadGame.execute({ slot: "nope" }, ctx);
-    expect(isToolFailure(result) && result.error).toMatch(/No save found/);
-  });
-
-  test("saving twice to one slot upserts — the newer save wins", async () => {
-    const { db, rows } = makeDb();
-    const ctx = makeCtx(db);
-    seedPlaying(ctx);
-    await saveGame.execute({}, ctx); // autosave
-    gameSlot.update(ctx, (game) => {
-      game.sceneCount = 9;
-    });
-    await saveGame.execute({}, ctx);
-    expect(rows.size).toBe(1);
-    expect(rows.get("save:autosave")).toMatchObject({ sceneCount: 9 });
-  });
-});
+// A `save_game / load_game` suite stood here, driving cross-session persistence
+// through a map-backed fake of `ctx.db`. Both tools are gone: `ctx.db` is gone,
+// and a shipped template cannot reach a database (see `shared.ts`). This
+// adventure is single-session now.
 
 // ── the story flow ───────────────────────────────────────────────────────────
 
@@ -655,10 +576,11 @@ describe("the story flow", () => {
     const ctx = makeCtx();
     expect(storyFlow.position(ctx).state).toBe("awaitingSetup");
 
-    // All three of these used to RUN before a character existed: `action_roll`
-    // rolled 2d6 against the stats of nobody and applied consequences to a game
-    // that was not there, and `save_game` wrote an empty campaign to a slot
-    // `load_game` would later restore over a real one.
+    // All of these used to RUN before a character existed: `action_roll` rolled
+    // 2d6 against the stats of nobody and applied consequences to a game that was
+    // not there. (`save_game` was in this list too, writing an empty campaign to a
+    // slot `load_game` would later restore over a real one — both are gone with
+    // `ctx.db`.)
     for (const call of [
       actionRoll.execute(
         { move: "clash", stat: "iron", position: "risky", effect: "standard", purpose: "swing" },
@@ -666,7 +588,6 @@ describe("the story flow", () => {
       ),
       updateState.execute({ location: "Nowhere" }, ctx),
       callNoArgs(burnMomentum, ctx),
-      saveGame.execute({}, ctx),
     ]) {
       const refusal = await call;
       expect(isToolFailure(refusal)).toBe(true);
@@ -740,37 +661,9 @@ describe("the story flow", () => {
     expect(restarted.state).toBe("playing.awaitingRoll");
   });
 
-  test("a loaded game resumes in play rather than awaiting setup", async () => {
-    const { db } = makeDb();
-    const sessionA = makeCtx(db);
-    seedPlaying(sessionA);
-    ok(await saveGame.execute({ slot: "resume" }, sessionA));
-
-    const sessionB = makeCtx(db);
-    expect(storyFlow.position(sessionB).state).toBe("awaitingSetup");
-    const loaded = await loadGame.execute({ slot: "resume" }, sessionB);
-    if (isToolFailure(loaded)) throw new Error(`load refused: ${loaded.error}`);
-    expect(loaded.state).toBe("playing.awaitingRoll");
-
-    // And the play tools are available in the resumed session.
-    ok(await updateState.execute({ location: "Back at the Docks" }, sessionB));
-  });
-
-  test("a save whose game was over resumes as over", async () => {
-    const { db } = makeDb();
-    const sessionA = makeCtx(db);
-    const dead = playingState();
-    dead.health = 0;
-    dead.spirit = 0;
-    dead.gameOver = true;
-    dead.crisisMode = true;
-    seedPlaying(sessionA, dead);
-    ok(await saveGame.execute({ slot: "ended" }, sessionA));
-
-    const sessionB = makeCtx(db);
-    const loaded = await loadGame.execute({ slot: "ended" }, sessionB);
-    if (isToolFailure(loaded)) throw new Error(`load refused: ${loaded.error}`);
-    expect(loaded.state).toBe("gameOver");
-    expect(storyFlow.position(sessionB).done).toBe(true);
-  });
+  // Two resume tests stood here — a saved game reopening in play, and one saved
+  // after the ending reopening as over. Both drove `save_game`/`load_game`, which
+  // are gone with `ctx.db`. What they proved about the FLOW (a resumed position is
+  // restored rather than recomputed) has no path left to exercise it: a session
+  // is the whole life of a game now.
 });