@remit/web-client 0.0.199 → 0.0.201

package/src/components/mail/selection-wizard-run-controls.ts

@@ -0,0 +1,101 @@
+import type { RunState, StepId } from "@remit/ui";
+import type { BulkActionTarget, BulkRunOutcome } from "@/lib/bulk-actions";
+import type { OrganizeScope } from "@/lib/organize/organize-model";
+
+export interface RetryContext {
+	runState: RunState;
+	isEscalated: boolean;
+	committedScope: OrganizeScope | undefined;
+	createFilterFailed: boolean;
+	backApplyPending: boolean;
+	widenRunsAsJob: boolean;
+	failedIds: readonly string[];
+	sent: readonly BulkActionTarget[];
+}
+
+export type RetryIntent =
+	| { kind: "refreshStatus" }
+	| { kind: "rerunEscalated" }
+	| { kind: "resetAndResend" }
+	| { kind: "resend" }
+	| { kind: "startBackApply" }
+	| { kind: "waitOnJob" }
+	| { kind: "rerunBulk"; targets: readonly BulkActionTarget[] };
+
+/**
+ * What pressing the run screen's retry asks for, given where the commit got to.
+ *
+ * A back-apply whose status could not be read is still running on the server
+ * (#526), so that one is decided before anything that could queue a second pass
+ * over the same mail.
+ */
+export const retryIntent = (context: RetryContext): RetryIntent => {
+	if (context.runState === "statusUnknown") return { kind: "refreshStatus" };
+	// The predicate is re-resolved, not resumed: every verb it carries is
+	// idempotent, so the messages the first pass already reached are a no-op.
+	if (context.isEscalated) return { kind: "rerunEscalated" };
+	if (
+		context.committedScope === "standing" ||
+		context.committedScope === "temporary"
+	) {
+		if (context.createFilterFailed) return { kind: "resetAndResend" };
+		return context.backApplyPending
+			? { kind: "startBackApply" }
+			: { kind: "waitOnJob" };
+	}
+	if (context.committedScope === "all-like-these" && context.widenRunsAsJob) {
+		return { kind: "resend" };
+	}
+	// A run hands back ids, and each one's account came from the batch it was
+	// sent in — so a retry re-sends the targets the run was given, filtered to
+	// what it never reached.
+	const outstanding = new Set(context.failedIds);
+	return {
+		kind: "rerunBulk",
+		targets:
+			outstanding.size > 0
+				? context.sent.filter((target) => outstanding.has(target.id))
+				: context.sent,
+	};
+};
+
+/**
+ * What a commit has left paging. A commit that never started, one whose runner
+ * has handed back its outcome, and one that could not start at all all have
+ * nothing to end — so the run screen offers no control that would claim to end
+ * them (#521).
+ */
+export const runIsInFlight = (
+	bulkRun: { outcome?: BulkRunOutcome; failureReason?: string } | undefined,
+): boolean =>
+	bulkRun !== undefined &&
+	bulkRun.outcome === undefined &&
+	bulkRun.failureReason === undefined;
+
+/**
+ * Ends the run rather than leaving it. An escalated predicate is paged by the
+ * list's own runner and a bounded selection by the wizard's, so the stop has to
+ * follow whichever one is paging: a stop that reaches only one of them leaves
+ * the other's control on screen doing nothing.
+ */
+export const stopRunner = (
+	escalated: { stop: () => void } | undefined,
+	stopBulk: () => void,
+): void => {
+	if (escalated) {
+		escalated.stop();
+		return;
+	}
+	stopBulk();
+};
+
+/**
+ * Which of the two movements the wizard's exit is, on the step it is taken
+ * from. Everywhere but the run screen it rewinds the entries the wizard owns
+ * and leaves the selection where it was; on the run screen the action has
+ * already been sent, so leaving walks away from a run that keeps going.
+ */
+export const wizardExit = (
+	step: StepId,
+	movements: { dismiss: () => void; cancel: () => void },
+): (() => void) => (step === "run" ? movements.dismiss : movements.cancel);

package/src/components/mail/BriefPane.selection.test.ts

@@ -1,51 +0,0 @@
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * The phone brief must mount the same selection surface the multi-pane list
- * does (#203). `BriefPane` wires the bulk verbs and the selection bar into
- * `DailyBrief` from the `BriefList` slot; the phone view used to render
- * `DailyBrief` directly, without `onDeleteMessages`, so a selection made in the
- * brief on a phone raised no action bar — selectable, but nothing to act with.
- *
- * As with `MessageList.selection.test.ts`, `BriefPane` weaves routing and
- * several data hooks together, so the rule is enforced by reading the source:
- * the phone view reuses the list slot rather than a bare `DailyBrief`.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const source = readFileSync(resolve(here, "BriefPane.tsx"), "utf8");
-
-const briefPhoneBody = (): string => {
-	const start = source.indexOf("function BriefPhone(");
-	assert.notEqual(start, -1, "BriefPhone is defined");
-	const next = source.indexOf("\nfunction ", start + 1);
-	return source.slice(start, next === -1 ? undefined : next);
-};
-
-describe("BriefPane phone selection surface (#203)", () => {
-	it("the phone brief reuses the list slot, not a bare DailyBrief", () => {
-		const body = briefPhoneBody();
-		assert.match(
-			body,
-			/<BriefList \/>/,
-			"phone brief renders the list slot that wires the bulk verbs",
-		);
-		assert.doesNotMatch(
-			body,
-			/<DailyBrief\b/,
-			"phone brief must not render DailyBrief directly — that drops onDeleteMessages and the selection bar with it",
-		);
-	});
-
-	it("the list slot hands DailyBrief the delete verb the selection bar gates on", () => {
-		const start = source.indexOf("function BriefList(");
-		assert.notEqual(start, -1, "BriefList is defined");
-		const next = source.indexOf("\nfunction ", start + 1);
-		const body = source.slice(start, next === -1 ? undefined : next);
-		assert.match(body, /onDeleteMessages=\{onDeleteMessages\}/);
-	});
-});

package/src/components/mail/DailyBrief.selection.test.ts

@@ -1,75 +0,0 @@
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * The brief raises one selection surface at every width, and select-all is on
- * it — wired into the touch sheet alone, the same list could be selected whole
- * on a phone and not on a desktop (#453).
- *
- * As with `MessageList.selection.test.ts`, `DailyBrief` weaves routing, the
- * cursor and several data hooks together, so the rule is enforced by reading
- * the source. `resolveBriefSelectionScope` carries the decisions that can be
- * unit-tested on their own.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const source = readFileSync(resolve(here, "DailyBrief.tsx"), "utf8");
-
-const chromeBody = (): string => {
-	const start = source.indexOf("function BriefSelectionChrome(");
-	assert.notEqual(start, -1, "BriefSelectionChrome is defined");
-	const next = source.indexOf("\nfunction ", start + 1);
-	return source.slice(start, next === -1 ? undefined : next);
-};
-
-describe("the brief's select-all", () => {
-	it("reaches the one selection bar from one derivation", () => {
-		const body = chromeBody();
-		const derivations = body.match(/const selectAll = useMemo\(/g) ?? [];
-		assert.equal(derivations.length, 1, "one select-all");
-		const bars = body.match(/<SelectionTopBar\b/g) ?? [];
-		assert.equal(bars.length, 1, "one surface, at every width (#480)");
-		const wired = body.match(/selectAll=\{selectAll\}/g) ?? [];
-		assert.equal(wired.length, 1, "the bar takes it");
-	});
-
-	it("is offered only when there are rows to select", () => {
-		assert.match(chromeBody(), /orderedIds\.length > 0/);
-	});
-
-	it("mounts the bar unconditionally — it is the list header (#480)", () => {
-		assert.match(
-			chromeBody(),
-			/const selectionBar = \(chrome: ListHeaderChrome\) => \(\s*<SelectionTopBar/,
-		);
-	});
-
-	it("carries every verb the bar can offer (#477 1.4)", () => {
-		// That each of them opens the wizard is `selection-verbs.test.ts`, over
-		// every surface at once. What this pins is that none of them was dropped
-		// from the bar the brief raises.
-		const body = chromeBody();
-		for (const prop of [
-			"onDelete",
-			"onMove",
-			"onOrganize",
-			"onJunk",
-			"onMarkRead",
-		]) {
-			assert.match(body, new RegExp(`${prop}=`));
-		}
-		assert.doesNotMatch(body, /onSomethingElse/);
-	});
-
-	it("drives the wizard and the keyboard layer from one control", () => {
-		// Two copies of "which verb is the wizard on" is how the brief came to
-		// offer a verb the mailbox list did not, and how its keyboard came to act
-		// behind a wizard the bar had opened.
-		assert.match(source, /const wizard = useSelectionWizard\(\);/);
-		assert.match(source, /onSelectionVerb={wizard.start}/);
-		assert.match(source, /wizardOpen={wizard.isOpen}/);
-	});
-});

package/src/components/mail/MessageList.selection.test.ts

@@ -1,195 +0,0 @@
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * Selection mode has one source of truth and one exit (#115). `MessageList`
- * wires the DOM, the virtualizer, routing and several data hooks together, so
- * — as with this package's other component-level rules (see
- * `../../index.css.test.ts`) — the rule is enforced by reading the source
- * rather than rendering the tree. The decisions themselves are unit-tested in
- * `../../lib/selection-mode.test.ts`.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const source = readFileSync(resolve(here, "MessageList.tsx"), "utf8");
-
-describe("MessageList selection mode", () => {
-	it("keeps no selection-mode flag of its own", () => {
-		assert.doesNotMatch(source, /useState[^\n]*[iI]sMultiSelectMode/);
-		assert.doesNotMatch(source, /setIsMultiSelectMode/);
-	});
-
-	it("derives the mode from the selection count", () => {
-		assert.match(
-			source,
-			/const isMultiSelectMode = deriveIsMultiSelectMode\(\s*selectedCount,/,
-		);
-	});
-
-	it("calls the selection hook's clearSelection from exactly one place", () => {
-		const calls = source.match(/\bclearSelection\(\)/g) ?? [];
-		assert.equal(calls.length, 1);
-		assert.match(
-			source,
-			/const exitSelection = useCallback\(\(\) => \{[^}]*clearSelection\(\);/,
-		);
-	});
-
-	it("exits selection when the mailbox changes", () => {
-		assert.match(source, /previousMailboxIdRef\.current === mailboxId/);
-	});
-
-	it("turns the back gesture into an exit, and only while selecting", () => {
-		assert.match(
-			source,
-			/shouldExitSelectionOnNavigate\(action, hasSelection, wizardStep\)/,
-		);
-		assert.match(source, /disabled: !hasSelection/);
-	});
-
-	it("leaves the back gesture to the wizard while the wizard is open", () => {
-		assert.match(
-			source,
-			/disabled: !hasSelection \|\| wizardStep !== undefined/,
-		);
-	});
-});
-
-/**
- * An escalated selection is a predicate, and it walks the same wizard every
- * other selection walks (#508). The list hands the wizard the predicate — the
- * words it names it with, the count it was escalated to, and the chunked runner
- * that re-resolves it — and keeps no second confirmation of its own.
- */
-describe("MessageList escalated actions", () => {
-	it("hands the escalated predicate to the wizard rather than running it here", () => {
-		assert.match(
-			source,
-			/const escalatedSelection: EscalatedSelection \| undefined =\s*escalation\.phase\.kind === "escalated"/,
-		);
-		assert.match(
-			source,
-			/scope: describeSearchScope\(searchPredicate \?\? \{\}\)/,
-		);
-		assert.match(source, /total: escalation\.phase\.total/);
-		assert.match(
-			source,
-			/run: \(action: EscalatedAction\) => escalation\.runAction\(action\)/,
-		);
-		const host = source.match(/<SelectionWizardHost[\s\S]*?\/>/)?.[0] ?? "";
-		assert.match(host, /escalated=\{escalatedSelection\}/);
-	});
-
-	it("keeps no predicate confirmation of its own", () => {
-		// The review screen names what the predicate covers; a second dialog
-		// asking the same question is the drift the wizard exists to end.
-		assert.doesNotMatch(source, /source: "predicate"/);
-		assert.doesNotMatch(source, /requestEscalatedDelete/);
-		assert.doesNotMatch(source, /pendingDeleteIsEstimate/);
-	});
-
-	it("words the bar's progress per action instead of per delete", () => {
-		assert.match(
-			source,
-			/bulkActionProgressLabel\(\s*escalation\.runningAction\.kind,/,
-		);
-	});
-});
-
-/**
- * Advanced selection is no longer mobile-only (#212): the escalation engine is
- * opened to every width, and the one selection bar renders the
- * offer → counting → escalated → progress → notice states.
- */
-describe("MessageList escalation reaches desktop (#212)", () => {
-	it("no longer gates the escalation engine on the mobile viewport", () => {
-		assert.match(
-			source,
-			/const escalationEnabled = isSearching && !!searchPredicate;/,
-		);
-		assert.doesNotMatch(
-			source,
-			/!isDesktop && isSearching && !!searchPredicate/,
-		);
-	});
-
-	it("feeds the bar the shared escalation state", () => {
-		// Up to the bar's own closing tag, at its own indent — a nested slot
-		// element closes first and would cut the match short.
-		const bar = source.match(/<SelectionTopBar[\s\S]*?\n\t\t\/>/)?.[0] ?? "";
-		assert.match(bar, /statusLabel=\{selectionStatusLabel\}/);
-		assert.match(bar, /notice=\{selectionNotice\}/);
-		assert.match(bar, /progress=\{selectionProgress\}/);
-		assert.match(bar, /selectAll=\{selectionSelectAll\}/);
-		assert.match(bar, /isCounting=\{escalation\.phase\.kind === "counting"\}/);
-	});
-
-	it("raises one selection surface, at every width (#480)", () => {
-		const bars = source.match(/<SelectionTopBar\b/g) ?? [];
-		assert.equal(bars.length, 1, "one bar, so no two surfaces can drift");
-		assert.doesNotMatch(source, /<SelectionSheet\b/);
-	});
-
-	it("mounts that surface unconditionally — it is the list header", () => {
-		// Gated on a selection, the bar's zero-ticked state (the mailbox name,
-		// and select-all inline from 768px up) reaches nothing.
-		assert.match(source, /const activeSelectionBar = \(\s*<SelectionTopBar/);
-		assert.match(source, /title=\{listHeaderChrome\.title \|\| listTitle\}/);
-		assert.match(source, /navSlot=\{listHeaderChrome\.navSlot\}/);
-	});
-
-	it("carries every verb the bar can offer (#477 1.4)", () => {
-		// That each of them opens the wizard is `selection-verbs.test.ts`, over
-		// every surface at once. What this pins is that none of them was dropped
-		// from the one bar the mailbox raises.
-		const bar = source.match(/<SelectionTopBar[\s\S]*?\n\t\t\/>/)?.[0] ?? "";
-		for (const prop of [
-			"onDelete",
-			"onMove",
-			"onOrganize",
-			"onJunk",
-			"onMarkRead",
-		]) {
-			assert.match(bar, new RegExp(`${prop}=`));
-		}
-		// The one selection surface has no second organize entry beside them.
-		assert.doesNotMatch(source, /onSomethingElse/);
-	});
-
-	it("keeps Organize reachable over a predicate rather than dropping it", () => {
-		// A verb that vanishes when a selection escalates leaves the user no route
-		// at all: the make-filter affordance it defers to is hidden while rows are
-		// ticked (#477 1.7).
-		assert.match(source, /if \(escalatedSelection\) \{\s*startFromSearch\(\);/);
-	});
-
-	it("offers the label picker only when the account has labels", () => {
-		assert.match(source, /labels\.length > 0 \? \(/);
-	});
-});
-
-/**
- * A confirmed delete opens the surviving neighbour. On desktop that fills the
- * reading pane beside the list; on a single-pane mobile layout the same
- * navigation replaced the list with a full-screen message, so the delete read as
- * "jumped into a random message" rather than "the row is gone" (#202). Mobile
- * now stays on the list and raises a completion banner instead.
- */
-describe("MessageList confirmed delete stays on the list on mobile (#202)", () => {
-	it("only the desktop two-pane opens the surviving neighbour after a delete", () => {
-		assert.match(
-			source,
-			/if \(isDesktop\) \{\s*openRow\(nextFocus, \{ replace: true \}\);\s*\}/,
-		);
-	});
-
-	it("raises a completion banner on mobile so the delete is not silent", () => {
-		assert.match(
-			source,
-			/if \(!isDesktop\) \{\s*setCompletionBanner\(\s*bulkActionCompletionText\("delete", ids\.length, deleteOutcome\),?\s*\);\s*\}/,
-		);
-	});
-});

package/src/components/mail/SelectionWizardHost.job-status.test.ts

@@ -1,50 +0,0 @@
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * A back-apply whose status could not be read is still running on the server
- * (#526), so the run screen's affordance looks at that job again rather than
- * queuing a second pass over the same mail.
- *
- * The host wires routing, history and several data hooks together, so — as with
- * this package's other component-level rules (see `SelectionWizardHost.run-exit.test.ts`)
- * — the wiring is read off the source. What it decides is proven where it can be
- * run: `./organize-run-state.test.ts` for the ending the screen shows, and
- * `../../hooks/useOrganizeJob.render.test.ts` for the poll it re-issues.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const source = readFileSync(resolve(here, "SelectionWizardHost.tsx"), "utf8");
-
-const retryBody = (): string => {
-	const body = source.match(
-		/const retry = \(\): void => \{([\s\S]*?)\n\t\};/,
-	)?.[1];
-	assert.ok(body, "the run screen has no retry");
-	return body;
-};
-
-describe("retrying from an unknown status", () => {
-	it("looks at the job it already has", () => {
-		const body = retryBody();
-		const unknown = body.indexOf('run.state === "statusUnknown"');
-		assert.ok(unknown >= 0, "retry does not recognise an unknown status");
-		assert.match(body.slice(unknown), /organizeJob\.refreshStatus\(\)/);
-	});
-
-	it("decides that before it can reach anything that queues a second job", () => {
-		const body = retryBody();
-		assert.ok(
-			body.indexOf('run.state === "statusUnknown"') < body.indexOf("startJob("),
-			"a second back-apply can be queued over a job that is still running",
-		);
-	});
-
-	it("reads what the job is doing rather than that something failed", () => {
-		assert.doesNotMatch(source, /organizeJob\.isError/);
-		assert.match(source, /organizeRunState\(\{/);
-	});
-});

package/src/components/mail/SelectionWizardHost.run-exit.test.ts

@@ -1,83 +0,0 @@
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * Every way off the run screen is the same movement, and none of them ends the
- * run the screen promised would keep going (#521). Ending it is a control of
- * its own, and whichever way the user leaves, the run reports where it got to.
- *
- * The host wires routing, history and several data hooks together, so — as with
- * this package's other component-level rules (see `MessageList.selection.test.ts`)
- * — the wiring is read off the source. What the exits then do is proven where it
- * can be run: `../../hooks/escalated-run-lifetime.render.test.ts` for the run's
- * lifetime, and `@remit/ui`'s `selection-wizard.render.test.ts` for the controls.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const source = readFileSync(resolve(here, "SelectionWizardHost.tsx"), "utf8");
-
-describe("leaving the run screen", () => {
-	it("routes the header's X and back arrow through the same dismiss the footer uses", () => {
-		assert.match(source, /onExit=\{current === "run" \? dismiss : cancel\}/);
-	});
-
-	it("routes hardware back through that dismiss too", () => {
-		assert.match(
-			source,
-			/if \(action !== "BACK"\) return false;\s*\n\s*dismiss\(\);/,
-		);
-	});
-
-	it("leaves the run alone on the way out", () => {
-		const dismissBody = source.match(
-			/const dismiss = useCallback\(\(\) => \{([\s\S]*?)\}, \[/,
-		)?.[1];
-		assert.ok(dismissBody, "the run screen has no dismiss");
-		assert.doesNotMatch(dismissBody, /stop/i);
-	});
-});
-
-describe("ending the run", () => {
-	it("is a control of its own, offered only while a run is in flight", () => {
-		assert.match(source, /onCancelRun: runInFlight \? stopRun : undefined/);
-		assert.match(
-			source,
-			/const runInFlight =\s*bulkRun !== undefined &&\s*bulkRun\.outcome === undefined &&\s*bulkRun\.failureReason === undefined;/,
-		);
-	});
-
-	it("stops whichever runner is paging — the list's or the wizard's own", () => {
-		const stopBody = source.match(
-			/const stopRun = useCallback\(\(\) => \{([\s\S]*?)\}, \[/,
-		)?.[1];
-		assert.ok(stopBody, "there is no way to stop a run");
-		assert.match(stopBody, /escalated\.stop\(\)/);
-		assert.match(stopBody, /stopBulk\(\)/);
-	});
-});
-
-/**
- * Both runners report, not just the predicate one: a ticked select-all runs on
- * the wizard's own runner, and the same screen makes the same promise over it.
- */
-describe("stating how a run ended once the screen is gone", () => {
-	it("marks the walk-away where it happens rather than reading it back off the URL", () => {
-		assert.match(source, /walkedAway\.current = true;\s*\n\s*closeWizard\(/);
-	});
-
-	for (const runner of ["runBulk", "runEscalated"]) {
-		it(`reports the run ${runner} drove`, () => {
-			const body =
-				source.match(
-					new RegExp(
-						`const ${runner} = useCallback\\(([\\s\\S]*?)\\n\\t\\}, \\[`,
-					),
-				)?.[1] ?? "";
-			assert.match(body, /if \(walkedAway\.current\)/);
-			assert.match(body, /onRunEnded\?\.\(action\.kind, /);
-		});
-	}
-});

package/src/components/mail/make-filter-entry.test.ts

@@ -1,63 +0,0 @@
-import assert from "node:assert/strict";
-import { readdirSync, readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * "Make this a filter" is the wizard's second entry (#477 clause 1.8), and the
- * step it pushes has to be answered on the view that offered it.
- *
- * The affordance is built once by `MailListHeader` and handed down as
- * `makeFilterSlot`, so any surface can put it on screen — but the wizard is
- * mounted per surface, and a surface that renders the row without one leaves
- * the press on a URL nothing answers. That is the dead button clause 1.7 exists
- * to prevent, and it is invisible in review: the row and the host are two files
- * apart, and the affordance keeps working everywhere else.
- *
- * So the pairing is the assertion. A surface that renders the row names where
- * its step is answered, and that file is checked to mount a host. A fourth
- * surface fails here until it does the same.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-
-/** How the chrome's slot is read — `MailListHeader` writes it, everyone else reads it. */
-const RENDERS_SLOT = /\.makeFilterSlot/;
-const MOUNTS_HOST = /<SelectionWizardHost/;
-
-/** Surface that renders the affordance → the file whose composition answers the step. */
-const ANSWERED_BY: Record<string, string> = {
-	"DailyBrief.tsx": "DailyBrief.tsx",
-	"MessageList.tsx": "MessageList.tsx",
-	// The starred list's bar is shared with the brief's, so its host is mounted
-	// by the view rather than beside the bar.
-	"ThreadListInteraction.tsx": "FlaggedList.tsx",
-};
-
-const read = (file: string): string =>
-	readFileSync(resolve(here, file), "utf8");
-
-const surfacesRenderingTheAffordance = (): string[] =>
-	readdirSync(here)
-		.filter((file) => file.endsWith(".tsx"))
-		.filter((file) => RENDERS_SLOT.test(read(file)));
-
-describe("the make-filter entry", () => {
-	it("is rendered only by surfaces that say where its step is answered", () => {
-		assert.deepEqual(
-			surfacesRenderingTheAffordance().sort(),
-			Object.keys(ANSWERED_BY).sort(),
-		);
-	});
-
-	it("is answered by a mounted wizard on every one of them", () => {
-		for (const [surface, host] of Object.entries(ANSWERED_BY)) {
-			assert.match(
-				read(host),
-				MOUNTS_HOST,
-				`${surface} is answered by ${host}`,
-			);
-		}
-	});
-});