@heyramzi/ui 2.1.0 → 2.2.0

package/src/inbox/types.ts

@@ -3,137 +3,159 @@ import type { InboxItem } from "../tasks/inbox/inbox-types.js";
 export type InboxActorKind = "user" | "agent";
 
 export type InboxActor = {
-	kind: InboxActorKind;
-	name: string;
-	/** Image URL for a person. The host's own portrait goes through the `avatar` snippet. */
-	avatar?: string;
+  kind: InboxActorKind;
+  name: string;
+  /** Image URL for a person. The host's own portrait goes through the `avatar` snippet. */
+  avatar?: string;
 };
 
-export type InboxEntryKind = "comment" | "status" | "assigned" | "mention" | "agent_response" | "proposal";
+export type InboxEntryKind =
+  | "comment"
+  | "status"
+  | "assigned"
+  | "mention"
+  | "agent_response"
+  | "proposal";
 
 /** What the thread is about: one task, project or doc. Entries with the same type and id share a thread. */
 export type InboxSubject = {
-	/** "task", "project", "doc" or whatever the host calls it. */
-	type: string;
-	id: string;
-	/** The short reference, such as "TSK-7". */
-	ref?: string;
-	title: string;
-	/** A status id the host draws itself, for a badge in the thread header. */
-	status?: string;
-	href?: string;
+  /** "task", "project", "doc" or whatever the host calls it. */
+  type: string;
+  id: string;
+  /** The short reference, such as "TSK-7". */
+  ref?: string;
+  title: string;
+  /** A status id the host draws itself, for a badge in the thread header. */
+  status?: string;
+  href?: string;
 };
 
 export type InboxEntry = {
-	id: string;
-	kind: InboxEntryKind;
-	/** An entry with no subject id belongs to no thread and is dropped. */
-	subject?: InboxSubject;
-	actor: InboxActor;
-	/** ISO timestamp. A malformed one sorts last. */
-	at: string;
-	read?: boolean;
-	/** Text of a comment, a mention or an agent response. */
-	body?: string;
-	/** A `status` entry: where the subject moved. */
-	status?: { from?: string; to: string };
-	/** An `assigned` entry: who got the work. */
-	assignee?: string;
-	/** A `proposal` entry: the agent's ask, drawn as an InboxCard while it waits. */
-	proposal?: InboxItem;
+  id: string;
+  kind: InboxEntryKind;
+  /** An entry with no subject id belongs to no thread and is dropped. */
+  subject?: InboxSubject;
+  actor: InboxActor;
+  /** ISO timestamp. A malformed one sorts last. */
+  at: string;
+  read?: boolean;
+  /** Text of a comment, a mention or an agent response. */
+  body?: string;
+  /** A `status` entry: where the subject moved. */
+  status?: { from?: string; to: string };
+  /** An `assigned` entry: who got the work. */
+  assignee?: string;
+  /** A `proposal` entry: the agent's ask, drawn as an InboxCard while it waits. */
+  proposal?: InboxItem;
 };
 
 export type InboxThread = {
-	/** `type:id`, unique across subjects. */
-	key: string;
-	subject: InboxSubject;
-	/** Oldest first. */
-	entries: InboxEntry[];
-	unread: number;
-	/** The newest valid timestamp, as written. The raw first one when none parses. */
-	latestAt: string;
-	/** Proposals still waiting for a verdict, newest first. */
-	pending: InboxEntry[];
+  /** `type:id`, unique across subjects. */
+  key: string;
+  subject: InboxSubject;
+  /** Oldest first. */
+  entries: InboxEntry[];
+  unread: number;
+  /** The newest valid timestamp, as written. The raw first one when none parses. */
+  latestAt: string;
+  /** Proposals still waiting for a verdict, newest first. */
+  pending: InboxEntry[];
 };
 
 export type InboxFilter = {
-	/** A subject type, or "all". */
-	type: string;
-	/** An actor name, or "all". */
-	who: string;
-	/** Only threads holding a pending proposal. */
-	needsYou: boolean;
+  /** A subject type, or "all". */
+  type: string;
+  /** An actor name, or "all". */
+  who: string;
+  /** Only threads holding a pending proposal. */
+  needsYou: boolean;
 };
 
 export const ALL = "all";
 
-export type InboxKey = "next" | "previous" | "toggleRead" | "snooze" | "approve" | "decline" | "steer" | "edit";
+export type InboxKey =
+  | "next"
+  | "previous"
+  | "toggleRead"
+  | "snooze"
+  | "approve"
+  | "decline"
+  | "steer"
+  | "edit";
 
 export type InboxSnooze = { thread: InboxThread; until: Date };
 
 export type InboxLabels = {
-	title: string;
-	all: string;
-	anyone: string;
-	needsYou: string;
-	empty: string;
-	emptyNeedsYou: string;
-	emptyFiltered: string;
-	clearFilters: string;
-	unread: string;
-	markRead: string;
-	markUnread: string;
-	snooze: string;
-	snoozeLater: string;
-	snoozeTomorrow: string;
-	snoozeNextWeek: string;
-	selectThread: string;
-	filterType: string;
-	filterWho: string;
-	agent: string;
-	statusMoved: string;
-	statusSet: string;
-	assigned: string;
-	mentioned: string;
-	commented: string;
-	responded: string;
-	proposed: string;
-	/** Keyboard hint words. */
-	hintMove: string;
-	hintRead: string;
-	hintSnooze: string;
-	hintAct: string;
+  title: string;
+  all: string;
+  anyone: string;
+  needsYou: string;
+  empty: string;
+  emptyNeedsYou: string;
+  emptyFiltered: string;
+  clearFilters: string;
+  unread: string;
+  markRead: string;
+  markUnread: string;
+  snooze: string;
+  snoozeLater: string;
+  snoozeTomorrow: string;
+  snoozeNextWeek: string;
+  selectThread: string;
+  filterType: string;
+  filterWho: string;
+  filter: string;
+  kind: string;
+  from: string;
+  open: string;
+  agent: string;
+  statusMoved: string;
+  statusSet: string;
+  assigned: string;
+  mentioned: string;
+  commented: string;
+  responded: string;
+  proposed: string;
+  /** Keyboard hint words. */
+  hintMove: string;
+  hintRead: string;
+  hintSnooze: string;
+  hintAct: string;
 };
 
 export const DEFAULT_INBOX_LABELS: InboxLabels = {
-	title: "Inbox",
-	all: "All",
-	anyone: "Anyone",
-	needsYou: "Needs you",
-	empty: "Nothing here",
-	emptyNeedsYou: "Nothing is waiting on you",
-	emptyFiltered: "No thread matches these filters",
-	clearFilters: "Clear filters",
-	unread: "unread",
-	markRead: "Mark read",
-	markUnread: "Mark unread",
-	snooze: "Snooze",
-	snoozeLater: "Later today",
-	snoozeTomorrow: "Tomorrow",
-	snoozeNextWeek: "Next week",
-	selectThread: "Select a thread",
-	filterType: "Filter by type",
-	filterWho: "Filter by person or agent",
-	agent: "AI",
-	statusMoved: "moved this from {from} to {to}",
-	statusSet: "set this to {to}",
-	assigned: "assigned this to {assignee}",
-	mentioned: "mentioned you",
-	commented: "commented",
-	responded: "replied",
-	proposed: "asks for your call",
-	hintMove: "move",
-	hintRead: "read",
-	hintSnooze: "snooze",
-	hintAct: "approve, decline, steer, edit",
+  title: "Inbox",
+  all: "All",
+  anyone: "Anyone",
+  needsYou: "Needs you",
+  empty: "Nothing here",
+  emptyNeedsYou: "Nothing is waiting on you",
+  emptyFiltered: "No thread matches these filters",
+  clearFilters: "Clear filters",
+  unread: "unread",
+  markRead: "Mark read",
+  markUnread: "Mark unread",
+  snooze: "Snooze",
+  snoozeLater: "Later today",
+  snoozeTomorrow: "Tomorrow",
+  snoozeNextWeek: "Next week",
+  selectThread: "Select a thread",
+  filterType: "Filter by type",
+  filterWho: "Filter by person or agent",
+  filter: "Filter",
+  kind: "Type",
+  from: "From",
+  open: "Open",
+  agent: "AI",
+  statusMoved: "moved this from {from} to {to}",
+  statusSet: "set this to {to}",
+  assigned: "assigned this to {assignee}",
+  mentioned: "mentioned you",
+  commented: "commented",
+  responded: "replied",
+  proposed: "asks for your call",
+  hintMove: "move",
+  hintRead: "read",
+  hintSnooze: "snooze",
+  hintAct: "approve, decline, steer, edit",
 };

package/src/lib/theme-transition.ts

@@ -14,10 +14,8 @@ const DEFAULT_DURATION = 400;
 const STYLE_ID = "heyramzi-reveal-theme";
 
 /**
- * Run `apply` inside a circular wipe that grows from the centre of the clicked
- * element, the element, or a point. Falls back to a plain `apply()` when the
- * browser has no view transitions, the user prefers reduced motion, or `skip`
- * is set. It adds the `::view-transition-*(root)` reset itself.
+ * Run `apply` inside a circular wipe from the clicked element or a point, falling back to
+ * plain `apply()` without view transitions, under reduced motion, or with `skip`.
  */
 export function revealTheme(
   origin: RevealOrigin,

package/src/lib/tone.ts

@@ -1,10 +1,8 @@
 export type Tone = "neutral" | "accent" | "info" | "success" | "warning" | "destructive";
 
 /**
- * The severity scale, as Tailwind classes for a chip and as CSS colors for the
- * dot. Tokens are plain CSS variables, so a host app can retheme them. The
- * accent tone reads `--tone-accent` and falls back to `--color-primary`, for a
- * host whose primary is a neutral and whose accent is a brand color.
+ * The severity scale as chip classes and dot colors. Accent reads `--tone-accent` before
+ * `--color-primary`, for a host whose primary is a neutral.
  */
 
 export const TONE_CLASS: Record<Tone, string> = {

package/src/lib/utils.ts

@@ -1,5 +1,5 @@
-// The class merge and prop types every shadcn primitive is written against. They live here, not in
-// a shared utils package, so this public package installs with nothing private behind it.
+// The class merge and prop types every shadcn primitive uses, kept here, not in a shared utils
+// package, so this public package installs with nothing private behind it.
 import { type ClassValue, clsx } from "clsx";
 import { twMerge } from "tailwind-merge";
 

package/src/modules.test.ts

@@ -5,91 +5,101 @@ import { describe, expect, it, vi } from "vite-plus/test";
 import * as status from "./status/index.js";
 import * as panel from "./panel/index.js";
 import * as record from "./record/index.js";
-import * as taskInbox from "./tasks/inbox/index.js";
 import * as inbox from "./inbox/index.js";
 import * as thread from "./thread/index.js";
 import type {
-	PanelStat as FromTasksPanelStat,
-	SidePanelWidth as FromTasksWidth,
-	StatusDef as FromTasksStatusDef,
-	StatusDotVariant as FromTasksVariant,
-	StatusId as FromTasksId,
-	StatusMap as FromTasksMap,
-	Statuses as FromTasksStatuses,
+  PanelStat as FromTasksPanelStat,
+  SidePanelWidth as FromTasksWidth,
+  StatusDef as FromTasksStatusDef,
+  StatusDotVariant as FromTasksVariant,
+  StatusId as FromTasksId,
+  StatusMap as FromTasksMap,
+  Statuses as FromTasksStatuses,
 } from "./tasks/index.js";
 import type { PanelStat, SidePanelWidth } from "./panel/index.js";
-import type { StatusDef, StatusDotVariant, StatusId, StatusMap, Statuses } from "./status/index.js";
+import type {
+  StatusDef,
+  StatusDotVariant,
+  StatusId,
+  StatusMap,
+  Statuses,
+} from "./status/index.js";
 
 // Both spellings of every moved type have to be the same type.
 type Same<A, B> = [A] extends [B] ? ([B] extends [A] ? true : false) : false;
 const sameTypes: [
-	Same<PanelStat, FromTasksPanelStat>,
-	Same<SidePanelWidth, FromTasksWidth>,
-	Same<StatusDef, FromTasksStatusDef>,
-	Same<StatusDotVariant, FromTasksVariant>,
-	Same<StatusId<{ a: StatusDef }>, FromTasksId<{ a: StatusDef }>>,
-	Same<StatusMap, FromTasksMap>,
-	Same<Statuses, FromTasksStatuses>,
+  Same<PanelStat, FromTasksPanelStat>,
+  Same<SidePanelWidth, FromTasksWidth>,
+  Same<StatusDef, FromTasksStatusDef>,
+  Same<StatusDotVariant, FromTasksVariant>,
+  Same<StatusId<{ a: StatusDef }>, FromTasksId<{ a: StatusDef }>>,
+  Same<StatusMap, FromTasksMap>,
+  Same<Statuses, FromTasksStatuses>,
 ] = [true, true, true, true, true, true, true];
 
-const pkg = JSON.parse(readFileSync(join(dirname(fileURLToPath(import.meta.url)), "..", "package.json"), "utf8")) as {
-	exports: Record<string, string | Record<string, string>>;
+const pkg = JSON.parse(
+  readFileSync(join(dirname(fileURLToPath(import.meta.url)), "..", "package.json"), "utf8"),
+) as {
+  exports: Record<string, string | Record<string, string>>;
 };
 
 describe("module split", () => {
-	it("keeps every moved name on tasks, the same object as on its new module", async () => {
-		// The kanban board reads matchMedia on import, and jsdom has none.
-		vi.stubGlobal("matchMedia", () => ({ matches: false, addEventListener() {}, removeEventListener() {} }));
-		const tasks = await import("./tasks/index.js");
-		expect(sameTypes).toHaveLength(7);
-		expect(tasks.StatusBadge).toBe(status.StatusBadge);
-		expect(tasks.StatusDot).toBe(status.StatusDot);
-		expect(tasks.StatusIcon).toBe(status.StatusIcon);
-		expect(tasks.defineStatuses).toBe(status.defineStatuses);
-		expect(tasks.StatusBadge).toBeDefined();
-		expect(tasks.SidePanel).toBe(panel.SidePanel);
-		expect(Object.keys(taskInbox)).toContain("Inbox");
-	});
+  it("keeps every moved name on tasks, the same object as on its new module", async () => {
+    // The kanban board reads matchMedia on import, and jsdom has none.
+    vi.stubGlobal("matchMedia", () => ({
+      matches: false,
+      addEventListener() {},
+      removeEventListener() {},
+    }));
+    const tasks = await import("./tasks/index.js");
+    expect(sameTypes).toHaveLength(7);
+    expect(tasks.StatusBadge).toBe(status.StatusBadge);
+    expect(tasks.StatusDot).toBe(status.StatusDot);
+    expect(tasks.StatusIcon).toBe(status.StatusIcon);
+    expect(tasks.defineStatuses).toBe(status.defineStatuses);
+    expect(tasks.StatusBadge).toBeDefined();
+    expect(tasks.SidePanel).toBe(panel.SidePanel);
+  });
 
-	it("holds the record page's names", () => {
-		expect(record.RecordShell).toBeDefined();
-		expect(record.DEFAULT_RECORD_LABELS).toBeDefined();
-	});
+  it("holds the record page's names", () => {
+    expect(record.RecordShell).toBeDefined();
+    expect(record.DEFAULT_RECORD_LABELS).toBeDefined();
+  });
 
-	it("names what the new modules export", () => {
-		expect(thread.Thread).toBeDefined();
-		expect(inbox.ThreadInbox).toBeDefined();
-		expect(inbox.InboxCard).toBeDefined();
-		expect(inbox.threadsOf).toBeTypeOf("function");
-	});
+  it("names what the new modules export", () => {
+    expect(thread.Thread).toBeDefined();
+    expect(inbox.ThreadInbox).toBeDefined();
+    expect(inbox.InboxCard).toBeDefined();
+    expect(inbox.threadsOf).toBeTypeOf("function");
+  });
 
-	it("resolves every export path to a file", () => {
-		const root = join(dirname(fileURLToPath(import.meta.url)), "..");
-		const missing: string[] = [];
-		for (const [name, target] of Object.entries(pkg.exports)) {
-			const path = typeof target === "string" ? target : (target.default ?? target.svelte);
-			if (path.includes("*") || path.endsWith(".css")) continue;
-			if (!existsSync(join(root, path))) missing.push(`${name} -> ${path}`);
-		}
-		expect(missing).toEqual([]);
-	});
+  it("resolves every export path to a file", () => {
+    const root = join(dirname(fileURLToPath(import.meta.url)), "..");
+    const missing: string[] = [];
+    for (const [name, target] of Object.entries(pkg.exports)) {
+      const path = typeof target === "string" ? target : (target.default ?? target.svelte);
+      if (path.includes("*") || path.endsWith(".css")) continue;
+      if (!existsSync(join(root, path))) missing.push(`${name} -> ${path}`);
+    }
+    expect(missing).toEqual([]);
+  });
 
-	it("keeps the old subpaths and adds the new ones", () => {
-		for (const name of [
-			"./tasks",
-			"./tasks/status",
-			"./tasks/status-dot",
-			"./tasks/statuses",
-			"./tasks/side-panel",
-			"./tasks/record",
-			"./tasks/inbox",
-			"./status",
-			"./panel",
-			"./record",
-			"./thread",
-			"./inbox",
-		]) {
-			expect(pkg.exports[name], name).toBeDefined();
-		}
-	});
+  it("keeps the old subpaths and adds the new ones", () => {
+    for (const name of [
+      "./tasks",
+      "./tasks/status",
+      "./tasks/status-dot",
+      "./tasks/statuses",
+      "./tasks/side-panel",
+      "./tasks/record",
+      "./tasks/inbox",
+      "./status",
+      "./panel",
+      "./record",
+      "./thread",
+      "./inbox",
+    ]) {
+      expect(pkg.exports[name], name).toBeDefined();
+    }
+  });
 });

package/src/onboarding/define-tours.test.ts

@@ -2,49 +2,64 @@ import { describe, expect, it } from "vite-plus/test";
 import { defineTours } from "./define-tours.js";
 import type { Step, Tour } from "./types.js";
 
-const step = (n: number): Step => ({ target: `[data-tour="s${n}"]`, title: `Title ${n}`, body: `Body ${n}` });
+const step = (n: number): Step => ({
+  target: `[data-tour="s${n}"]`,
+  title: `Title ${n}`,
+  body: `Body ${n}`,
+});
 const steps = (count: number) => Array.from({ length: count }, (_, i) => step(i + 1));
 const board = (over: Partial<Extract<Tour, { kind: "steps" }>> = {}): Tour => ({
-	id: "board",
-	route: "/(app)/projects",
-	kind: "steps",
-	steps: steps(3),
-	...over,
+  id: "board",
+  route: "/(app)/projects",
+  kind: "steps",
+  steps: steps(3),
+  ...over,
 });
 
 describe("defineTours", () => {
-	it("returns the tours it was given", () => {
-		const list = [board(), { id: "home", route: "/(app)", kind: "intro", title: "Hi", body: "Welcome." } as const];
-		expect(defineTours(list)).toBe(list);
-	});
-
-	it("rejects a repeated id", () => {
-		expect(() => defineTours([board(), board({ route: "/(app)/other" })])).toThrow(/"board".*used twice/);
-	});
-
-	it("takes 1 to 6 steps", () => {
-		expect(() => defineTours([board({ steps: steps(1) })])).not.toThrow();
-		expect(() => defineTours([board({ steps: steps(6) })])).not.toThrow();
-		expect(() => defineTours([board({ steps: [] })])).toThrow(/0 steps/);
-		expect(() => defineTours([board({ steps: steps(7) })])).toThrow(/7 steps/);
-	});
-
-	it("rejects a route that isn't a route id", () => {
-		expect(() => defineTours([board({ route: "projects" })])).toThrow(/not a route id/);
-	});
-
-	it("rejects a bad version", () => {
-		expect(() => defineTours([board({ version: 0 })])).toThrow(/version/);
-		expect(() => defineTours([board({ version: 1.5 })])).toThrow(/version/);
-		expect(() => defineTours([board({ version: 2 })])).not.toThrow();
-	});
-
-	it("rejects a step with no target or no words", () => {
-		expect(() => defineTours([board({ steps: [{ ...step(1), target: " " }] })])).toThrow(/step 1 has no target/);
-		expect(() => defineTours([board({ steps: [{ ...step(1), body: "" }] })])).toThrow(/title and a body/);
-	});
-
-	it("rejects an intro with no title", () => {
-		expect(() => defineTours([{ id: "home", route: "/(app)", kind: "intro", title: "", body: "x" }])).toThrow(/intro/);
-	});
+  it("returns the tours it was given", () => {
+    const list = [
+      board(),
+      { id: "home", route: "/(app)", kind: "intro", title: "Hi", body: "Welcome." } as const,
+    ];
+    expect(defineTours(list)).toBe(list);
+  });
+
+  it("rejects a repeated id", () => {
+    expect(() => defineTours([board(), board({ route: "/(app)/other" })])).toThrow(
+      /"board".*used twice/,
+    );
+  });
+
+  it("takes 1 to 6 steps", () => {
+    expect(() => defineTours([board({ steps: steps(1) })])).not.toThrow();
+    expect(() => defineTours([board({ steps: steps(6) })])).not.toThrow();
+    expect(() => defineTours([board({ steps: [] })])).toThrow(/0 steps/);
+    expect(() => defineTours([board({ steps: steps(7) })])).toThrow(/7 steps/);
+  });
+
+  it("rejects a route that isn't a route id", () => {
+    expect(() => defineTours([board({ route: "projects" })])).toThrow(/not a route id/);
+  });
+
+  it("rejects a bad version", () => {
+    expect(() => defineTours([board({ version: 0 })])).toThrow(/version/);
+    expect(() => defineTours([board({ version: 1.5 })])).toThrow(/version/);
+    expect(() => defineTours([board({ version: 2 })])).not.toThrow();
+  });
+
+  it("rejects a step with no target or no words", () => {
+    expect(() => defineTours([board({ steps: [{ ...step(1), target: " " }] })])).toThrow(
+      /step 1 has no target/,
+    );
+    expect(() => defineTours([board({ steps: [{ ...step(1), body: "" }] })])).toThrow(
+      /title and a body/,
+    );
+  });
+
+  it("rejects an intro with no title", () => {
+    expect(() =>
+      defineTours([{ id: "home", route: "/(app)", kind: "intro", title: "", body: "x" }]),
+    ).toThrow(/intro/);
+  });
 });

package/src/onboarding/define-tours.ts

@@ -1,31 +1,31 @@
 import { MAX_STEPS, type Tour } from "./types.js";
 
 function fail(id: string, problem: string): never {
-	throw new Error(`tour "${id}": ${problem}`);
+  throw new Error(`tour "${id}": ${problem}`);
 }
 
 /** Returns the tours as written, after checking what a typo would hide until a user hits it. */
 export function defineTours<const T extends readonly Tour[]>(tours: T): T {
-	const ids = new Set<string>();
-	for (const tour of tours) {
-		const { id } = tour;
-		if (!id || !id.trim()) throw new Error("a tour needs an id");
-		if (ids.has(id)) fail(id, "its id is used twice");
-		ids.add(id);
-		if (!tour.route.startsWith("/")) fail(id, `route "${tour.route}" is not a route id`);
-		const version = tour.version ?? 1;
-		if (!Number.isInteger(version) || version < 1) fail(id, "version is a whole number from 1");
-		if (tour.kind === "intro") {
-			if (!tour.title || !tour.body) fail(id, "an intro needs a title and a body");
-			continue;
-		}
-		if (tour.steps.length < 1 || tour.steps.length > MAX_STEPS) {
-			fail(id, `has ${tour.steps.length} steps, and a tour takes 1 to ${MAX_STEPS}`);
-		}
-		tour.steps.forEach((step, i) => {
-			if (!step.target.trim()) fail(id, `step ${i + 1} has no target`);
-			if (!step.title || !step.body) fail(id, `step ${i + 1} needs a title and a body`);
-		});
-	}
-	return tours;
+  const ids = new Set<string>();
+  for (const tour of tours) {
+    const { id } = tour;
+    if (!id || !id.trim()) throw new Error("a tour needs an id");
+    if (ids.has(id)) fail(id, "its id is used twice");
+    ids.add(id);
+    if (!tour.route.startsWith("/")) fail(id, `route "${tour.route}" is not a route id`);
+    const version = tour.version ?? 1;
+    if (!Number.isInteger(version) || version < 1) fail(id, "version is a whole number from 1");
+    if (tour.kind === "intro") {
+      if (!tour.title || !tour.body) fail(id, "an intro needs a title and a body");
+      continue;
+    }
+    if (tour.steps.length < 1 || tour.steps.length > MAX_STEPS) {
+      fail(id, `has ${tour.steps.length} steps, and a tour takes 1 to ${MAX_STEPS}`);
+    }
+    tour.steps.forEach((step, i) => {
+      if (!step.target.trim()) fail(id, `step ${i + 1} has no target`);
+      if (!step.title || !step.body) fail(id, `step ${i + 1} needs a title and a body`);
+    });
+  }
+  return tours;
 }