@alexkroman1/aai-cli 13.2.0 → 14.0.0

package/dist/templates/pizza-ordering/client.tsx

@@ -1,6 +1,7 @@
+import { formatMoney } from "@alexkroman1/aai/utils";
 import "@alexkroman1/aai-ui/styles.css";
-import { client, useAgentState } from "@alexkroman1/aai-ui";
-import { formatPrice, orderProjection, pizzaPrice } from "./shared.ts";
+import { mountClient, useAgentState } from "@alexkroman1/aai-ui";
+import { orderProjection, pizzaPrice } from "./shared.ts";
 
 function PizzaIcon({ size }: { size: string }) {
   const dim = size === "small" ? 36 : size === "large" ? 52 : 44;
@@ -67,7 +68,7 @@ function OrderSidebar() {
             </p>
           </div>
           <p className="text-sm font-bold whitespace-nowrap text-aai-primary">
-            {formatPrice(pizzaPrice(p))}
+            {formatMoney(pizzaPrice(p))}
           </p>
         </div>
       ))}
@@ -79,7 +80,7 @@ function OrderSidebar() {
   );
 }
 
-client({
+mountClient({
   name: "Pizza Palace",
   sidebar: OrderSidebar,
   theme: {

package/dist/templates/pizza-ordering/shared.ts

@@ -1,4 +1,5 @@
 import { type DeepReadonly, sessionSlot } from "@alexkroman1/aai";
+import { formatMoney } from "@alexkroman1/aai/utils";
 
 export const SIZES = ["small", "medium", "large"] as const;
 export const CRUSTS = ["thin", "regular", "thick", "stuffed"] as const;
@@ -37,19 +38,13 @@ export function calculateTotal(pizzas: readonly ReadonlyPizza[]): number {
   return pizzas.reduce((total, pizza) => total + pizzaPrice(pizza), 0);
 }
 
-/** The one money format. Tool results, the projection, and the sidebar all
- *  show prices through this, so they can never disagree on rounding. */
-export function formatPrice(amount: number): string {
-  return `$${amount.toFixed(2)}`;
-}
-
 /**
  * The menu as prompt prose, generated from `MENU` so the agent can never
  * quote a price the pricing code doesn't charge.
  */
 export function menuText(): string {
   const price = (amount: number, upcharge = false) =>
-    amount === 0 ? "free" : `${upcharge ? "+" : ""}${formatPrice(amount)}`;
+    amount === 0 ? "free" : `${upcharge ? "+" : ""}${formatMoney(amount)}`;
   const list = (items: Record<string, number>, upcharge = false) =>
     Object.entries(items)
       .map(([name, amount]) => `${name.replaceAll("_", " ")} (${price(amount, upcharge)})`)
@@ -169,7 +164,7 @@ export function orderView(order: FrozenOrderState): OrderView {
   const placed = order.placed;
   return {
     pizzas: order.pizzas,
-    total: placed?.total ?? formatPrice(calculateTotal(order.pizzas)),
+    total: placed?.total ?? formatMoney(calculateTotal(order.pizzas)),
     orderPlaced: Boolean(placed),
     ...(placed
       ? { orderNumber: placed.orderNumber, estimatedMinutes: placed.estimatedMinutes }

package/dist/templates/pizza-ordering/tools/add_pizza.ts

@@ -1,5 +1,6 @@
+import { formatMoney } from "@alexkroman1/aai/utils";
 import { z } from "zod";
-import { CRUSTS, calculateTotal, formatPrice, orderSlot, type Pizza, SIZES } from "../shared.ts";
+import { CRUSTS, calculateTotal, orderSlot, type Pizza, SIZES } from "../shared.ts";
 
 export default orderSlot.updateTool({
   description: "Add a pizza to the order. Use when the customer has decided on a pizza.",
@@ -24,7 +25,7 @@ export default orderSlot.updateTool({
 
     return {
       added: pizza,
-      orderTotal: formatPrice(calculateTotal(order.pizzas)),
+      orderTotal: formatMoney(calculateTotal(order.pizzas)),
       itemCount: order.pizzas.length,
     };
   },

package/dist/templates/pizza-ordering/tools/place_order.ts

@@ -1,5 +1,6 @@
 import { toolFailure } from "@alexkroman1/aai";
-import { calculateTotal, formatPrice, orderSlot, resetOrder } from "../shared.ts";
+import { formatMoney } from "@alexkroman1/aai/utils";
+import { calculateTotal, orderSlot, resetOrder } from "../shared.ts";
 
 export default orderSlot.updateTool({
   description:
@@ -9,7 +10,7 @@ export default orderSlot.updateTool({
     if (pizzas.length === 0) return toolFailure("Cannot place an empty order.");
 
     const customerName = order.customerName ?? "Guest";
-    const total = formatPrice(calculateTotal(pizzas));
+    const total = formatMoney(calculateTotal(pizzas));
     const orderNumber = Math.floor(1000 + Math.random() * 9000);
 
     const estimatedMinutes = 15 + pizzas.length * 5;

package/dist/templates/pizza-ordering/tools/remove_pizza.ts

@@ -1,6 +1,7 @@
 import { toolFailure } from "@alexkroman1/aai";
+import { formatMoney } from "@alexkroman1/aai/utils";
 import { z } from "zod";
-import { calculateTotal, formatPrice, orderSlot } from "../shared.ts";
+import { calculateTotal, orderSlot } from "../shared.ts";
 
 export default orderSlot.updateTool({
   description: "Remove a pizza from the order by its ID.",
@@ -15,7 +16,7 @@ export default orderSlot.updateTool({
 
     return {
       removed,
-      orderTotal: formatPrice(calculateTotal(order.pizzas)),
+      orderTotal: formatMoney(calculateTotal(order.pizzas)),
       itemCount: order.pizzas.length,
     };
   },

package/dist/templates/pizza-ordering/tools/update_pizza.ts

@@ -1,6 +1,7 @@
 import { toolFailure } from "@alexkroman1/aai";
+import { formatMoney } from "@alexkroman1/aai/utils";
 import { z } from "zod";
-import { CRUSTS, calculateTotal, formatPrice, orderSlot, type Pizza, SIZES } from "../shared.ts";
+import { CRUSTS, calculateTotal, orderSlot, type Pizza, SIZES } from "../shared.ts";
 
 export default orderSlot.updateTool({
   description: "Update an existing pizza in the order. Only provided fields are changed.",
@@ -23,6 +24,6 @@ export default orderSlot.updateTool({
 
     order.pizzas[idx] = pizza;
 
-    return { updated: pizza, orderTotal: formatPrice(calculateTotal(order.pizzas)) };
+    return { updated: pizza, orderTotal: formatMoney(calculateTotal(order.pizzas)) };
   },
 });

package/dist/templates/pizza-ordering/tools/view_order.ts

@@ -1,4 +1,5 @@
-import { calculateTotal, formatPrice, orderSlot } from "../shared.ts";
+import { formatMoney } from "@alexkroman1/aai/utils";
+import { calculateTotal, orderSlot } from "../shared.ts";
 
 export default orderSlot.tool({
   description: "View the current order summary with all pizzas and total price.",
@@ -15,7 +16,7 @@ export default orderSlot.tool({
         toppings: p.toppings,
         quantity: p.quantity,
       })),
-      orderTotal: formatPrice(calculateTotal(pizzas)),
+      orderTotal: formatMoney(calculateTotal(pizzas)),
     };
   },
 });

package/dist/templates/plan-and-execute/agent.test.ts

@@ -1,7 +1,12 @@
 /** The def a DEPLOYED agent runs: authored, plus what `tools/` declares. */
 import agentDef from "virtual:aai/agent";
 import type { ToolContext } from "@alexkroman1/aai";
-import { createToolContext, okPosition, stubGenerate, toolRunner } from "@alexkroman1/aai/testing";
+import {
+  createToolContext,
+  expectDialogOk,
+  stubGenerate,
+  toolRunner,
+} from "@alexkroman1/aai/testing";
 import { describe, expect, test } from "vitest";
 
 import { executeStep, MAX_STEP_SEARCHES, normalizeAct, planNode } from "./procedure.ts";
@@ -216,7 +221,7 @@ describe("work_next_step", () => {
     const ctx = createToolContext({ generate });
     await run("start_plan", { objective: "a weekend in Lisbon" }, ctx);
 
-    const first = okPosition<{
+    const first = expectDialogOk<{
       finished: boolean;
       step: string;
       result: string;
@@ -243,7 +248,7 @@ describe("work_next_step", () => {
     });
     const ctx = createToolContext({ generate });
     await run("start_plan", { objective: "a weekend in Lisbon" }, ctx);
-    const answered = okPosition<{ finished: boolean; response: string }>(
+    const answered = expectDialogOk<{ finished: boolean; response: string }>(
       await run("work_next_step", ctx),
     );
 
@@ -327,7 +332,7 @@ describe("revise_plan", () => {
     await run("work_next_step", ctx);
     expect(stateOf(ctx).response).not.toBeNull();
 
-    const revised = okPosition<{ finished: boolean; remaining: string[] }>(
+    const revised = expectDialogOk<{ finished: boolean; remaining: string[] }>(
       await run("revise_plan", { instruction: "make it Porto instead" }, ctx),
     );
     expect(revised.result.finished).toBe(false);
@@ -362,7 +367,7 @@ describe("revise_plan", () => {
     });
     const ctx = createToolContext({ generate });
     await run("start_plan", { objective: "a weekend in Lisbon" }, ctx);
-    const revised = okPosition<{ finished: boolean }>(
+    const revised = expectDialogOk<{ finished: boolean }>(
       await run("revise_plan", { instruction: "never mind, it is booked" }, ctx),
     );
     expect(revised.result.finished).toBe(true);

package/dist/templates/plan-and-execute/client.tsx

@@ -1,5 +1,5 @@
 import "@alexkroman1/aai-ui/styles.css";
-import { AutoScroll, client, useAgentState } from "@alexkroman1/aai-ui";
+import { AutoScroll, mountClient, useAgentState } from "@alexkroman1/aai-ui";
 import { planProjection } from "./shared.ts";
 
 /**
@@ -90,7 +90,7 @@ function PlanSidebar() {
   );
 }
 
-client({
+mountClient({
   name: "Planning Desk",
   sidebar: PlanSidebar,
   theme: {

package/dist/templates/podcast-digest/agent.test.ts

@@ -11,7 +11,7 @@
  *   HTTP handling, its partial-failure policy and its `FatalError` guards are
  *   all reachable — `installStubStepFetch` answers the network and
  *   `stubGateway` answers the model.
- * - The BODY, twice. `createWorkflowCtx` records what it asked for (the digest
+ * - The BODY, twice. `createWorkflowContext` records what it asked for (the digest
  *   loop, the shrinking pending set, the sleep BETWEEN digests and never after
  *   the last), and `runWorkflow` runs it on the real replay engine — which is
  *   what this file used to say a unit test could not do, on the ground that a
@@ -26,8 +26,9 @@
 
 import { type FeedItem, parseFeed } from "@alexkroman1/aai/html";
 import {
-  createWorkflowCtx,
+  createWorkflowContext,
   parseSchemaInput,
+  routeStepFetch,
   schemaInputIssues,
   stubGatewayRoute,
 } from "@alexkroman1/aai/testing";
@@ -103,15 +104,25 @@ const slackInput = (slackWebhookUrl: string) => ({
 
 describe("the declaration", () => {
   test("is a workflow app with one workflow and a static page", () => {
-    expect(agentDef.name).toBe("Podcast Digest");
-    expect(Object.keys(agentDef.workflows ?? {})).toEqual(["dailyDigest"]);
+    // Not the agent's name: renaming it is an invited edit, and pinning the
+    // literal here would fail a build in somebody else's project.
+    expect(agentDef.name).toBeTruthy();
+    // `toContain` rather than an exact key list: adding a second workflow is an
+    // invited edit and must not redden a test the author did not write. The
+    // NAME is still pinned, deliberately — the page starts a run by this
+    // string, so renaming the key is a runtime 400 rather than a compile
+    // error, and this pin is the only thing that says so. Rename it here and
+    // in `client.tsx` together.
+    expect(Object.keys(agentDef.workflows ?? {})).toContain("dailyDigest");
     expect(agentDef.workflows?.dailyDigest).toBe(dailyDigest);
   });
 
   test("names the one credential its steps read", () => {
     // A workflow app declares no providers, so nothing else in the config can
     // name this — which is what makes a deploy able to check for it.
-    expect(agentDef.requiredEnv).toEqual(["ASSEMBLYAI_API_KEY"]);
+    // `toContain`, so a step of your own that reads a second credential can be
+    // declared beside this one without failing a test you did not write.
+    expect(agentDef.requiredEnv).toContain("ASSEMBLYAI_API_KEY");
   });
 });
 
@@ -412,12 +423,20 @@ const THREE_EPISODES = feedXml(
  * the thing that forgets, and a step fetch left published reaches the next file.
  */
 function stubRoutes(routes: Record<string, { status?: number; body?: unknown }>) {
-  return installStubStepFetch((request) => {
-    for (const [fragment, answer] of Object.entries(routes)) {
-      if (request.url.includes(fragment)) return answer;
-    }
-    return { status: 404, body: `no route for ${request.url}` };
-  });
+  return installStubStepFetch(
+    routeStepFetch(
+      Object.entries(routes).map(
+        ([fragment, answer]) =>
+          (request) =>
+            request.url.includes(fragment) ? answer : undefined,
+      ),
+      // A 404 rather than the default throw, and this one is deliberate: several
+      // cases below are ABOUT a feed the run cannot reach, and they express that
+      // by leaving it out of `routes`. Where an unrecognised request would be a
+      // finding instead, the default is what says so at the call.
+      { unmatched: "notFound" },
+    ),
+  );
 }
 
 describe("discoverEpisodes", () => {
@@ -827,7 +846,7 @@ describe("posting the digest", () => {
   });
 
   /**
-   * The 4xx/5xx split is the reason this is not a one-line `stepFetchOk`: a
+   * The 4xx/5xx split is the reason this is not a one-line `stepFetchOrFail`: a
    * revoked webhook answers 4xx identically on every retry, so retrying it
    * burns the DevKit's attempts and delays the real error by minutes.
    */
@@ -866,7 +885,7 @@ describe("the body — the run that IS the schedule", () => {
    * per-step spec can see, and all of which is the template's actual subject.
    */
   function driveTwoDigests(pollResults: unknown) {
-    const ctx = createWorkflowCtx({
+    const ctx = createWorkflowContext({
       runSteps: false,
       results: {
         discoverEpisodes: [EPISODE],
@@ -963,7 +982,7 @@ describe("the body — the run that IS the schedule", () => {
 /**
  * The schedule, on the real replay engine.
  *
- * The block above drives the same body through `createWorkflowCtx`, which
+ * The block above drives the same body through `createWorkflowContext`, which
  * RECORDS a sleep rather than taking one and replays nothing — right for the
  * loop's logic, and silent about the property this template is: a run that
  * sleeps for a day between digests and comes back. `runWorkflow`

package/dist/templates/podcast-digest/client.tsx

@@ -3,7 +3,7 @@
  * The browser half — a form that starts a run, and a panel that watches one
  * that may outlive the browser by a week.
  *
- * Mounted with `page()` rather than `client()`: there is no session to build, so
+ * Mounted with `mountPage()` rather than `mountClient()`: there is no session to build, so
  * no socket, no audio graph, no microphone request. `useWorkflowSubmit()` starts
  * the run, follows its STATUS, and hands back the controls bound to it.
  *
@@ -17,12 +17,13 @@
  * minutes, this one loses the only Stop button a thirty-digest commitment has,
  * and the run is invisible from then on to everything but `curl`.
  *
- * `key` is the handle that survives and `recover: true` is what reads it back,
- * so a later load lands on the same count, the same newest line, and the same
- * Wake and Cancel buttons bound to the same run.
+ * A correlation KEY is the handle that survives, and `useWorkflowSubmit` looks
+ * that key's newest run up as it mounts, so a later load lands on the same
+ * count, the same newest line, and the same Wake and Cancel buttons bound to
+ * the same run.
  *
- * **The key is `useRunKey({ storage: "local" })`, and this is the one workflow
- * app that passes that option.** Its siblings take the default,
+ * **This is the one workflow app that passes a `key` of its own, and it is
+ * `useRunKey({ storage: "local" })`.** Its siblings let the hook mint one into
  * `sessionStorage`, which dies with the tab and covers exactly the interruption
  * they have — a reload, a same-tab navigation, a crashed tab. A schedule
  * outlives all of that by design: closing the browser on Tuesday and coming
@@ -42,9 +43,9 @@
  *   URL would carry a credential into a lookup token — which is why the
  *   platform stopped writing keys to the operator's log.
  *
- * A real app with accounts passes the account's own id instead, and then the
- * schedule follows the person to a new device — a promise only a login can
- * keep.
+ * A real app with accounts passes the account's own id as `key` instead, and
+ * then the schedule follows the person to a new device — a promise only a login
+ * can keep.
  *
  * ## The form is DECLARED, not written
  *
@@ -85,8 +86,9 @@
  */
 
 import {
+  BulletList,
   Form,
-  page,
+  mountPage,
   SubmitButton,
   useRunKey,
   useWorkflowSubmit,
@@ -97,7 +99,6 @@ import "@alexkroman1/aai-ui/styles.css";
 // ERASED at build time, so naming the agent's own type costs the browser bundle
 // nothing — and it is what stops this file restating a shape `workflows/
 // digest.ts` already declares.
-import { useState } from "react";
 import type { dailyDigest } from "./agent.ts";
 
 /** The workflow this page drives. Matches the key in `workflowApp({ workflows })`. */
@@ -130,21 +131,15 @@ export function App() {
   // Did THIS load start the schedule? A later load cannot have, and that is the
   // only way the page can tell "scheduled just now" from "still running from
   // Tuesday" — the hook reports the run, not who asked for it.
-  const [startedHere, setStartedHere] = useState(false);
   // The generic is what makes `run.status === "completed"` narrow to a TYPED
   // `run.output` instead of `unknown`. `error` is the agent's own sentence for a
   // rejected input — better copy than anything this page could write, and the
   // reason there is no `try`/`catch` here.
-  const { submitForm, run, pending, error, wake, cancel } = useWorkflowSubmit<typeof dailyDigest>(
-    WORKFLOW,
-    {
-      // Recorded with the run, and read back on the next load. Neither half is
-      // useful alone: without the key there is nothing to find the run by, and
-      // without `recover` the key is only ever written.
-      key,
-      recover: true,
-    },
-  );
+  // The key REPLACES the tab-scoped one the hook would mint; the lookup that
+  // reads it back on the next load happens either way.
+  const { submitForm, run, pending, error, wake, cancel, startedHere } = useWorkflowSubmit<
+    typeof dailyDigest
+  >(WORKFLOW, { key });
 
   return (
     <main className="mx-auto flex max-w-2xl flex-col gap-6 p-8">
@@ -152,13 +147,7 @@ export function App() {
 
       {/* `submit()` resolves as soon as the run EXISTS — deliberately not when it
           finishes, which here could be a month away. */}
-      <Form
-        onSubmit={(values) => {
-          setStartedHere(true);
-          return submitForm(values);
-        }}
-        error={error}
-      >
+      <Form onSubmit={(values) => submitForm(values)} error={error}>
         <WorkflowFields workflow={WORKFLOW} />
         <SubmitButton pending={pending}>
           {pending ? "Digest scheduled" : "Start digest"}
@@ -198,7 +187,13 @@ export function App() {
         </div>
       )}
 
-      {run?.status === "failed" && <p className="text-red-600">That run failed: {run.error}</p>}
+      {/* `role="alert"`, the same contract `<Form>` gives the submit error
+          above: a digest that fails does so days later, with nobody watching. */}
+      {run?.status === "failed" && (
+        <p role="alert" className="text-red-600">
+          That run failed: {run.error}
+        </p>
+      )}
       {run?.status === "cancelled" && <p>Cancelled — no further digests will be posted.</p>}
 
       {run?.status === "completed" && (
@@ -216,11 +211,7 @@ export function App() {
                 </a>
               </h2>
               <p>{episode.summary}</p>
-              <ul className="flex list-disc flex-col gap-1 pl-5 text-sm">
-                {episode.keyPoints.map((point) => (
-                  <li key={point}>{point}</li>
-                ))}
-              </ul>
+              <BulletList items={episode.keyPoints} size="sm" />
             </section>
           ))}
         </article>
@@ -229,4 +220,4 @@ export function App() {
   );
 }
 
-page({ name: "Podcast Digest", component: App });
+mountPage({ name: "Podcast Digest", component: App });

package/dist/templates/podcast-digest/workflows/digest.ts

@@ -49,13 +49,13 @@
  * @module digest
  */
 
-import type { WorkflowCtx } from "@alexkroman1/aai";
-import { mapConcurrent, report, TRANSCRIBE_API } from "@alexkroman1/aai/step";
+import type { WorkflowContext } from "@alexkroman1/aai";
+import { mapConcurrent, stepReport, TRANSCRIBE_API } from "@alexkroman1/aai/step";
 import {
   FatalError,
-  stepGenerateJsonClassified,
-  stepTranscribePollClassified,
-  stepTranscribeSubmitClassified,
+  stepGenerateJsonOrFail,
+  stepTranscribePollOrFail,
+  stepTranscribeSubmitOrFail,
 } from "@alexkroman1/aai/step-errors";
 import { errorMessage } from "@alexkroman1/aai/utils";
 import type { WorkflowInputOf } from "@alexkroman1/aai/workflow-api";
@@ -160,7 +160,7 @@ export type DailyDigestOutput = {
   } | null;
 };
 
-/** What the model must answer with, and what `stepGenerateJsonClassified` enforces. */
+/** What the model must answer with, and what `stepGenerateJsonOrFail` enforces. */
 const SummaryReply = z.object({
   summary: z.string().trim().min(1),
   keyPoints: z.array(z.string().trim().min(1)).min(1).max(5),
@@ -178,7 +178,7 @@ const SummaryReply = z.object({
  */
 export async function dailyDigestFlow(
   input: DigestInput,
-  ctx: WorkflowCtx,
+  ctx: WorkflowContext,
 ): Promise<DailyDigestOutput> {
   // No `??` fallbacks: {@link DigestInput} is the schema's OUTPUT, so every
   // `.default()` has already run by the time a run reaches this line. The
@@ -269,7 +269,7 @@ export async function dailyDigestFlow(
  */
 async function waitForTranscripts(
   jobs: TranscriptJob[],
-  ctx: WorkflowCtx,
+  ctx: WorkflowContext,
 ): Promise<TranscriptState[]> {
   let pending = jobs;
   // Keyed by episode id rather than appended, and that is what keeps the digest
@@ -287,17 +287,19 @@ async function waitForTranscripts(
     if (pending.length > 0) await ctx.sleep("poll", POLL_DELAY_MS);
   }
 
-  for (const job of pending) settled.set(job.id, gaveUpOn(job));
   // The list `jobs` arrived in, which `discoverEpisodes` sorted newest first.
+  // A job still pending is absent from `settled`, so the `??` IS the
+  // give-up path — `pollTranscript` returns the job unchanged while it is
+  // submitted, so the value given up on is the one that went in.
   return jobs.map((job) => settled.get(job.id) ?? gaveUpOn(job));
 }
 
 /**
  * One episode the run is done waiting for.
  *
- * Its own function because {@link waitForTranscripts} needs it in two places —
- * the jobs still pending when the budget ran out, and the unreachable fallback
- * the ordered rebuild above needs for a lookup that cannot miss.
+ * Its own function because {@link waitForTranscripts} reaches it from the
+ * ordered rebuild, for every job whose budget ran out while it was still
+ * submitted and which is therefore absent from `settled`.
  */
 function gaveUpOn(job: TranscriptJob): TranscriptState {
   return {
@@ -322,7 +324,7 @@ function gaveUpOn(job: TranscriptJob): TranscriptState {
  * an `unavailable` VALUE rather than a throw, because one bad episode must not
  * sink a digest of five.
  *
- * The verdict itself is the SDK's: `stepTranscribeSubmitClassified` reads
+ * The verdict itself is the SDK's: `stepTranscribeSubmitOrFail` reads
  * `TranscribeError`'s own `retryable` AND its `retryAfter`, and throws a
  * `FatalError` or a `RetryableError` accordingly. The hand-written
  * `err instanceof TranscribeError && err.retryable` this replaces read only the
@@ -330,9 +332,9 @@ function gaveUpOn(job: TranscriptJob): TranscriptState {
  * one-second default instead.
  */
 export async function submitTranscript(episode: Episode): Promise<TranscriptJob> {
-  await report(`Submitting ${episode.title} for transcription.`);
+  await stepReport(`Submitting ${episode.title} for transcription.`);
   try {
-    const { id } = await stepTranscribeSubmitClassified(episode.audioUrl, {
+    const { id } = await stepTranscribeSubmitOrFail(episode.audioUrl, {
       // A digest quotes nobody, so who spoke costs time for nothing.
       params: { speaker_labels: false },
     });
@@ -356,12 +358,12 @@ export async function pollTranscript(job: TranscriptJob): Promise<TranscriptStat
   if (job.transcriptStatus === "unavailable") return job;
 
   try {
-    const progress = await stepTranscribePollClassified(job.transcriptId);
+    const progress = await stepTranscribePollOrFail(job.transcriptId);
     // Branch on `done`, never on a status string: a vocabulary this body does
     // not own would otherwise read as "not finished yet" forever.
     if (!progress.done) return job;
 
-    await report(`Transcribed ${job.title}.`);
+    await stepReport(`Transcribed ${job.title}.`);
     return {
       ...job,
       transcriptStatus: "done",
@@ -391,8 +393,8 @@ export async function summarizeTranscript(state: TranscriptState): Promise<Episo
     };
   }
 
-  await report(`Summarizing ${state.title}.`);
-  const parsed = await stepGenerateJsonClassified(
+  await stepReport(`Summarizing ${state.title}.`);
+  const parsed = await stepGenerateJsonOrFail(
     [
       `Podcast: ${state.podcastTitle}`,
       `Episode: ${state.title}`,