@remit/web-client 0.0.92 → 0.0.94

package/src/components/mail/organize/OrganizeRuleEditor.tsx

@@ -1,285 +0,0 @@
-import { mailboxOperationsListMailboxesOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import {
-	type FilterRule,
-	FilterRuleEditor,
-	type FolderOption,
-	type LabelOption,
-	type RuleMatchMode,
-	type RuleScope,
-} from "@remit/ui";
-import { useQuery } from "@tanstack/react-query";
-import { useEffect, useMemo, useRef, useState } from "react";
-import { useClauseSuggestions } from "@/hooks/useClauseSuggestions";
-import { useCreateMailbox } from "@/hooks/useCreateMailbox";
-import { useCreateFilter } from "@/hooks/useFilters";
-import { useCreateLabel, useLabelList } from "@/hooks/useLabels";
-import { useOrganizeJob } from "@/hooks/useOrganizeJob";
-import { useRuleEditorState } from "@/hooks/useRuleEditorState";
-import { useRulePreview } from "@/hooks/useRulePreview";
-import { useSelectedSubjects } from "@/hooks/useSelectedSubjects";
-import { getMailboxDisplayName } from "@/lib/folder-roles";
-import { buildMoveTargets } from "@/lib/move-targets";
-import {
-	canBackApplyDraft,
-	type OrganizeDraft,
-} from "@/lib/organize/organize-model";
-import {
-	buildInitialRule,
-	defaultMatchMode,
-	matchersForMode,
-	rulePredicate,
-	ruleToDraft,
-	SUPPORTED_CLAUSE_FIELDS,
-} from "@/lib/organize/rule-model";
-import {
-	BackApplyError,
-	CommitError,
-	FilterSaved,
-	JobProgress,
-	SavingState,
-} from "./rule-editor-states";
-
-interface OrganizeRuleEditorProps {
-	accountId: string;
-	selectedMessageIds: string[];
-	/** The widen probe's matched total — seeds the live count without a re-fetch. */
-	seedCount: number;
-	/**
-	 * The deployment ships no vector pipeline, so the widen cannot run. The rule
-	 * opens on the sender-fallback `From` chips instead of the anchor.
-	 */
-	semanticUnavailable?: boolean;
-	/** Distinct sender addresses, for the fallback clauses and the progress copy. */
-	senders?: string[];
-	/**
-	 * The mode the editor opens on. Omitted, it opens on the semantic widen
-	 * wherever the deployment can run it — a caller passes `properties` only to
-	 * enter the flow on a rule built from the selection's own properties.
-	 */
-	seedMatchMode?: RuleMatchMode;
-	/** A folder a "Something else" shortcut pre-picked. */
-	seedMailboxId?: string;
-	/** A scope a "Something else" shortcut pre-picked. */
-	seedScope?: RuleScope;
-	onClose: () => void;
-}
-
-/**
- * The Organize surface as the chip editor (RFC 038 D1). The rule is rendered and
- * edited over the existing preview/apply endpoints: clause chips, a
- * match-operator toggle, a move action, and a scope that maps one-time apply to
- * a back-apply job and standing/until to a `Filter`. Creating a filter also runs
- * the back-apply once, so the rule reaches the mail already in the mailbox and
- * not only the mail that arrives next. The count is live and the
- * commit gate holds apply until it settles, so the set the editor shows is the
- * set a commit acts on. Rendered inside the desktop dialog and the mobile sheet
- * alike, so the two cannot drift.
- */
-export function OrganizeRuleEditor({
-	accountId,
-	selectedMessageIds,
-	seedCount,
-	semanticUnavailable = false,
-	senders = [],
-	seedMatchMode,
-	seedMailboxId,
-	seedScope,
-	onClose,
-}: OrganizeRuleEditorProps) {
-	const anchorMessageId = selectedMessageIds[0];
-	const senderFallback = semanticUnavailable && senders.length > 0;
-	const subjects = useSelectedSubjects(selectedMessageIds);
-
-	// Semantic stays the default wherever the deployment can serve it; the
-	// properties mode is the third way in, not a replacement (RFC 038 D2).
-	const [matchMode, setMatchMode] = useState<RuleMatchMode>(
-		() => seedMatchMode ?? defaultMatchMode(semanticUnavailable),
-	);
-
-	const [initialRule] = useState<FilterRule>(() =>
-		buildInitialRule({
-			anchorMessageId,
-			semanticUnavailable,
-			matchMode,
-			senders,
-			subjects,
-			selectionCount: selectedMessageIds.length,
-			seedMailboxId,
-			seedScope,
-		}),
-	);
-	const { rule, setRule, handlers } = useRuleEditorState({
-		initialRule,
-		widenAnchorCount: selectedMessageIds.length,
-	});
-
-	// Switching mode rebuilds only what the rule matches on — the derived chips
-	// and the widen. The destination, label, scope, name, and any clause the user
-	// typed themselves survive, so the choice stays reversible.
-	const changeMatchMode = (next: RuleMatchMode) => {
-		if (next === matchMode) return;
-		setMatchMode(next);
-		setRule((current) => ({
-			...current,
-			...matchersForMode(next, {
-				anchorMessageId,
-				senders,
-				subjects,
-				selectionCount: selectedMessageIds.length,
-				keepClauses: current.clauses.filter((clause) => !clause.derived),
-				currentOperator: current.matchOperator,
-			}),
-		}));
-	};
-
-	// The selection's own senders are the likeliest values for an address clause
-	// and are already in hand, so they lead the list before anything is typed.
-	const clauseSuggestions = useClauseSuggestions(
-		handlers.clauseEdit?.draft.field,
-		handlers.clauseEdit?.draft.value ?? "",
-		senders,
-	);
-
-	const { data: mailboxesData } = useQuery({
-		...mailboxOperationsListMailboxesOptions({ path: { accountId } }),
-		staleTime: Number.POSITIVE_INFINITY,
-	});
-
-	const folders: FolderOption[] = useMemo(
-		() =>
-			buildMoveTargets(mailboxesData?.items ?? []).map((mailbox) => ({
-				id: mailbox.mailboxId,
-				label: getMailboxDisplayName(mailbox.fullPath),
-			})),
-		[mailboxesData?.items],
-	);
-
-	const { labels: labelItems } = useLabelList(accountId);
-	const labels: LabelOption[] = useMemo(
-		() =>
-			labelItems.map((label) => ({
-				id: label.labelId,
-				name: label.name,
-				color: label.color,
-			})),
-		[labelItems],
-	);
-	const { createLabel } = useCreateLabel(accountId);
-	const onCreateLabel = async (name: string): Promise<LabelOption> => {
-		const label = await createLabel(name);
-		return { id: label.labelId, name: label.name, color: label.color };
-	};
-
-	const preview = useRulePreview(
-		accountId,
-		rulePredicate(rule, anchorMessageId),
-		seedCount,
-	);
-
-	const organizeJob = useOrganizeJob(accountId);
-	const createFilter = useCreateFilter(accountId);
-	const { createFolder } = useCreateMailbox(accountId);
-
-	// Creating a filter also moves the mail that already matches, not only the
-	// mail that arrives next: the same retroactive back-apply the one-time scope
-	// runs. The filter is created first so the rule is live before the pass, then
-	// the pass runs over the existing corpus. `backApplyDraft` is the predicate
-	// to run once the create succeeds, and is undefined when the rule cannot be
-	// back-applied (a `HasWords` clause the vector-free pass can't evaluate — the
-	// filter still saves and applies to incoming mail). It is kept, not cleared,
-	// so a failed start can be retried; `backApplyStarted` guards the one-shot
-	// auto-start against re-firing on re-render.
-	const [backApplyDraft, setBackApplyDraft] = useState<OrganizeDraft>();
-	const backApplyStarted = useRef(false);
-
-	const commit = () => {
-		const draft = ruleToDraft(rule, anchorMessageId);
-		if (rule.scope === "once") {
-			organizeJob.start(draft);
-			return;
-		}
-		backApplyStarted.current = false;
-		setBackApplyDraft(canBackApplyDraft(draft) ? draft : undefined);
-		createFilter.createFilter(
-			draft,
-			rule.scope === "standing" ? "standing" : "temporary",
-			(rule.name ?? "").trim(),
-		);
-	};
-
-	useEffect(() => {
-		if (!backApplyDraft || backApplyStarted.current || !createFilter.isSuccess)
-			return;
-		backApplyStarted.current = true;
-		organizeJob.start(backApplyDraft);
-	}, [backApplyDraft, createFilter.isSuccess, organizeJob.start]);
-
-	const retryBackApply = () => {
-		if (backApplyDraft) organizeJob.start(backApplyDraft);
-	};
-
-	if (organizeJob.isStarting || organizeJob.isRunning || organizeJob.isDone) {
-		return (
-			<JobProgress
-				progress={organizeJob.progress}
-				isDone={organizeJob.isDone}
-				runningLabel={
-					senderFallback
-						? "Organizing mail from these senders…"
-						: matchMode === "properties"
-							? "Organizing matching mail…"
-							: "Organizing similar mail…"
-				}
-				onClose={onClose}
-			/>
-		);
-	}
-
-	// The filter saved, but the back-apply's own request failed (a 5xx or dropped
-	// connection on start, or a failed retry) — no job id was ever assigned, so no
-	// JobProgress state holds. Surface it distinctly instead of falling through to
-	// "Filter saved" and swallowing it: the rule is live, the move is what to
-	// retry.
-	if (createFilter.isSuccess && organizeJob.isError) {
-		return <BackApplyError onRetry={retryBackApply} onClose={onClose} />;
-	}
-
-	if (createFilter.isPending) {
-		return <SavingState />;
-	}
-
-	if (createFilter.isSuccess) {
-		// The filter is saved; the back-apply is about to start (the effect hands
-		// off to the job on the next tick). Keep the saving state until it takes
-		// over so the success screen never flashes between them. When the rule
-		// can't be back-applied, `backApplyDraft` is undefined and this is the
-		// terminal state — saved, applying to incoming mail only.
-		if (backApplyDraft) return <SavingState />;
-		return <FilterSaved onClose={onClose} />;
-	}
-
-	if (createFilter.isError) {
-		return <CommitError onRetry={createFilter.reset} onClose={onClose} />;
-	}
-
-	return (
-		<FilterRuleEditor
-			rule={rule}
-			folders={folders}
-			labels={labels}
-			preview={preview}
-			semanticAvailable={!semanticUnavailable}
-			// A deployment without the vector pipeline has only one mode to be in,
-			// so it gets no choice to make — the rule already matches on properties.
-			matchMode={semanticUnavailable ? undefined : matchMode}
-			onChangeMatchMode={changeMatchMode}
-			clauseFields={SUPPORTED_CLAUSE_FIELDS}
-			clauseSuggestions={clauseSuggestions}
-			{...handlers}
-			onCreateFolder={createFolder}
-			onCreateLabel={onCreateLabel}
-			onCommit={commit}
-			onCancel={onClose}
-		/>
-	);
-}

package/src/components/mail/organize/SearchFilterDialog.render.test.ts

@@ -1,47 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
-import React, { createElement } from "react";
-import { renderToString } from "react-dom/server";
-import { parseSearchTokens } from "@/lib/search-tokens";
-import { SearchFilterDialog } from "./SearchFilterDialog";
-
-// The node test loader transpiles remit-ui's `.tsx` with the classic JSX
-// runtime, which references a global `React`.
-(globalThis as { React?: typeof React }).React = React;
-
-const render = (open: boolean, query = "from:receipts@stripe.com") =>
-	renderToString(
-		createElement(
-			QueryClientProvider,
-			{ client: new QueryClient() },
-			createElement(SearchFilterDialog, {
-				open,
-				accountId: "acc-1",
-				parsed: parseSearchTokens(query, {}),
-				searchHadSemanticReach: true,
-				onClose: () => undefined,
-			}),
-		) as never,
-	);
-
-describe("SearchFilterDialog", () => {
-	it("renders nothing when closed", () => {
-		assert.equal(render(false), "");
-	});
-
-	it("shows the conversion step while the seed preview is in flight", () => {
-		assert.match(render(true), /Turning your search into a filter/);
-	});
-
-	it("opens the editor for a free-text search instead of failing to count it", () => {
-		// The free text converts to a `HasWords` clause, which the vector-free
-		// matcher refuses outright. Asking for a count is a 500, so nothing is
-		// asked: the editor opens and the count region carries the reason.
-		const html = render(true, "receipts");
-		assert.doesNotMatch(html, /Turning your search into a filter/);
-		assert.doesNotMatch(html, /Couldn't build the filter/);
-		assert.match(html, /These chips are the whole rule/);
-		assert.match(html, /reads message bodies/);
-	});
-});

package/src/components/mail/organize/SearchFilterDialog.tsx

@@ -1,90 +0,0 @@
-import { Button, buildSearchRule, Dialog } from "@remit/ui";
-import { Loader2 } from "lucide-react";
-import { useMemo } from "react";
-import { useSearchFilterSeed } from "@/hooks/useSearchFilterSeed";
-import { rulePredicate } from "@/lib/organize/rule-model";
-import { convertSearchToRule } from "@/lib/organize/search-to-rule";
-import type { ParsedSearchQuery } from "@/lib/search-tokens";
-import { SearchFilterEditor } from "./SearchFilterEditor";
-
-interface SearchFilterDialogProps {
-	open: boolean;
-	/** The account the filter is created for (an `account:` facet, else the active account). */
-	accountId: string;
-	/** The current search, already split into free text and facets. */
-	parsed: ParsedSearchQuery;
-	/**
-	 * The search surfaced semantically-similar mail (a non-empty "Related"
-	 * section). The literal filter cannot reproduce that reach, so the conversion
-	 * states it — read from the search's own results, never probed (RFC 038 D5).
-	 */
-	searchHadSemanticReach: boolean;
-	onClose: () => void;
-}
-
-/**
- * "Make this a filter" (RFC 038 D5). Converts the current search to clauses and
- * hands off to the shared chip editor pre-filled. No new endpoint: the seed
- * count rides `POST /organize/preview` and the commit drives the existing filter
- * CRUD.
- */
-export function SearchFilterDialog({
-	open,
-	accountId,
-	parsed,
-	searchHadSemanticReach,
-	onClose,
-}: SearchFilterDialogProps) {
-	const conversion = useMemo(
-		() => convertSearchToRule(parsed, { searchHadSemanticReach }),
-		[parsed, searchHadSemanticReach],
-	);
-	const literalPredicate = useMemo(
-		() => rulePredicate(buildSearchRule(conversion)),
-		[conversion],
-	);
-
-	const { seedCount, isPending, isError, retry } = useSearchFilterSeed(
-		open ? accountId : undefined,
-		literalPredicate,
-	);
-	// A search kept as a `HasWords` clause has no seed count and never will; the
-	// editor opens on the uncountable reason rather than a dead end.
-
-	if (!open) return null;
-
-	return (
-		<Dialog open={open} onClose={onClose} title="Filter rule">
-			{isError ? (
-				<div className="flex flex-col items-center gap-3 px-5 py-8 text-center">
-					<p className="text-sm font-medium text-danger">
-						Couldn't build the filter
-					</p>
-					<p className="max-w-xs text-xs text-fg-muted">Please try again.</p>
-					<div className="mt-2 flex gap-2">
-						<Button variant="primary" onClick={retry}>
-							Try again
-						</Button>
-						<Button variant="ghost" onClick={onClose}>
-							Not now
-						</Button>
-					</div>
-				</div>
-			) : isPending ? (
-				<div className="flex flex-col items-center gap-3 px-5 py-10 text-center">
-					<Loader2 className="size-8 animate-spin text-accent-2" />
-					<p className="text-sm font-medium text-fg">
-						Turning your search into a filter…
-					</p>
-				</div>
-			) : (
-				<SearchFilterEditor
-					accountId={accountId}
-					conversion={conversion}
-					seedCount={seedCount}
-					onClose={onClose}
-				/>
-			)}
-		</Dialog>
-	);
-}

package/src/components/mail/organize/SearchFilterEditor.render.test.ts

@@ -1,260 +0,0 @@
-/**
- * The filter-from-search chip editor (RFC 038 D5) over the live preview/apply
- * endpoints. The contract these tests pin: the converted clauses open pre-filled,
- * the conversion states honestly what the search carried that the filter cannot,
- * and the count on screen is the literal set a commit acts on — the commit is
- * blocked until the count settles and carries exactly the previewed predicate.
- */
-
-import assert from "node:assert/strict";
-import { afterEach, describe, it } from "node:test";
-import { mailboxOperationsListMailboxesQueryKey } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import { createElement } from "react";
-import { convertSearchToRule } from "@/lib/organize/search-to-rule";
-import {
-	parseSearchTokens,
-	type SearchTokenContext,
-} from "@/lib/search-tokens";
-import { createDomHarness, type DomHarness } from "../../../test-support/dom";
-import { makeMailbox } from "../../../test-support/fixtures";
-import {
-	type HttpCall,
-	type HttpMock,
-	mockFetch,
-} from "../../../test-support/http";
-import { SearchFilterEditor } from "./SearchFilterEditor";
-
-let harness: DomHarness | undefined;
-let http: HttpMock | undefined;
-
-afterEach(() => {
-	harness?.close();
-	harness = undefined;
-	http?.restore();
-	http = undefined;
-});
-
-const ACCOUNT_ID = "acc-1";
-
-const MAILBOXES = [
-	makeMailbox({ mailboxId: "mbx-inbox", fullPath: "INBOX" }),
-	makeMailbox({ mailboxId: "mbx-archive", fullPath: "Archive" }),
-];
-
-const CONTEXT: SearchTokenContext = {
-	mailboxesByName: new Map([["archive", "mbx-archive"]]),
-	accountsByName: new Map(),
-};
-
-const conversion = (query: string, searchHadSemanticReach = false) =>
-	convertSearchToRule(parseSearchTokens(query, CONTEXT), {
-		searchHadSemanticReach,
-	});
-
-type Responder = (call: HttpCall) => unknown;
-
-const previewCounts = (counts: number[]): Responder => {
-	let index = 0;
-	return (call) => {
-		if (call.path.endsWith("/organize/preview")) {
-			const count = counts[Math.min(index, counts.length - 1)];
-			index += 1;
-			return { matchedCount: count, messageIds: [] };
-		}
-		if (call.path.endsWith("/filters")) {
-			return { filterId: "filter-1", name: "R", scope: "Standing" };
-		}
-		if (call.path.endsWith("/organize") && call.method === "POST") {
-			return { organizeJobId: "job-1", state: "Running" };
-		}
-		if (call.path.endsWith("/organize/job-1")) {
-			return {
-				organizeJobId: "job-1",
-				state: "Complete",
-				matchedCount: 3,
-				appliedCount: 3,
-				failedCount: 0,
-			};
-		}
-		return {};
-	};
-};
-
-const mount = (
-	query: string,
-	responder: Responder = previewCounts([12]),
-	searchHadSemanticReach = false,
-): DomHarness => {
-	http = mockFetch(responder);
-	harness = createDomHarness();
-	harness.queryClient.setQueryData(
-		mailboxOperationsListMailboxesQueryKey({ path: { accountId: ACCOUNT_ID } }),
-		{ items: MAILBOXES },
-	);
-	harness.renderApp(
-		createElement(SearchFilterEditor, {
-			accountId: ACCOUNT_ID,
-			conversion: conversion(query, searchHadSemanticReach),
-			seedCount: 12,
-			onClose: () => undefined,
-		}),
-	);
-	return harness;
-};
-
-async function settlePreview(dom: DomHarness): Promise<void> {
-	await dom.flush();
-	await dom.wait(400);
-	await dom.flush();
-	await dom.flush();
-}
-
-const primaryButton = (dom: DomHarness, label: string): HTMLButtonElement =>
-	dom.byText("button", label) as HTMLButtonElement;
-
-describe("SearchFilterEditor — pre-filled conversion", () => {
-	it("opens on the converted clauses", () => {
-		const dom = mount("from:alerts@github.com pull request");
-		assert.match(dom.text(), /alerts@github\.com/);
-		assert.match(dom.text(), /pull request/);
-	});
-
-	it("seeds the live count from the converted predicate", () => {
-		const dom = mount("from:receipts@stripe.com");
-		assert.match(dom.text(), /12 messages match/);
-	});
-
-	it("never asks the matcher to count a body-text rule it cannot evaluate", async () => {
-		// A free-text search converts to a `HasWords` clause and carries no anchor,
-		// so the vector-free matcher rejects it outright (organize.ts
-		// `assertNoBodyContentClause`). The count is unavailable and says so; the
-		// request is never sent.
-		const dom = mount("receipts");
-		await settlePreview(dom);
-		assert.match(dom.text(), /can't count matches/i);
-		assert.equal((http?.to("/organize/preview") ?? []).length, 0);
-	});
-
-	it("does not offer the semantic widen on a search-derived rule", () => {
-		const dom = mount("receipts");
-		assert.doesNotMatch(dom.text(), /and similar/i);
-	});
-});
-
-describe("SearchFilterEditor — conversion honesty (RFC 038 D5)", () => {
-	it("states a folder-scoped search is kept out of the filter", () => {
-		const dom = mount("in:archive receipts");
-		assert.match(dom.text(), /limited to archive/i);
-		assert.match(dom.text(), /any folder/i);
-	});
-
-	it("names the dropped attribute facets", () => {
-		const dom = mount("invoice has:attachment is:unread");
-		assert.match(dom.text(), /Has attachment/);
-		assert.match(dom.text(), /Unread/);
-		assert.match(dom.text(), /left out/);
-	});
-
-	it("states the dropped semantic reach when the search surfaced similar mail", () => {
-		const dom = mount("things like this", previewCounts([12]), true);
-		assert.match(dom.text(), /matches these words literally/i);
-		assert.match(dom.text(), /similar mail/i);
-		// The widen chip is never offered on a search-derived rule.
-		assert.doesNotMatch(dom.text(), /and anything similar/i);
-	});
-
-	it("says nothing about semantics when the search had no similar mail", () => {
-		const dom = mount("things like this", previewCounts([12]), false);
-		assert.doesNotMatch(dom.text(), /similar mail/i);
-	});
-});
-
-describe("SearchFilterEditor — commit gate", () => {
-	it("blocks the save until a folder and name are set, then commits the previewed predicate", async () => {
-		const dom = mount("from:alerts@github.com receipts");
-
-		// Standing is the default scope: it needs a folder and a name.
-		assert.equal(primaryButton(dom, "Save rule").disabled, true);
-		dom.select(dom.byLabel("Destination folder"), "mbx-archive");
-		await dom.flush();
-		dom.type(dom.byLabel("Rule name"), "GitHub receipts");
-		await dom.flush();
-		assert.equal(primaryButton(dom, "Save rule").disabled, false);
-
-		dom.click(primaryButton(dom, "Save rule"));
-		await dom.flush();
-
-		const filters = http?.to("/filters") ?? [];
-		assert.equal(filters.length, 1);
-		assert.equal(filters[0].body?.scope, "Standing");
-		assert.deepEqual(filters[0].body?.literalClauses, [
-			{ field: "From", value: "alerts@github.com" },
-			{ field: "HasWords", value: "receipts" },
-		]);
-		assert.match(dom.text(), /Filter saved/);
-	});
-
-	it("stales the count on an edit and holds the save until it settles", async () => {
-		// The seed count (12) is passed directly; the first network preview is the
-		// re-count after the edit, so it returns 40.
-		const dom = mount("from:receipts@stripe.com", previewCounts([40]));
-		dom.select(dom.byLabel("Destination folder"), "mbx-archive");
-		await dom.flush();
-		dom.type(dom.byLabel("Rule name"), "Receipts");
-		await dom.flush();
-
-		dom.click(primaryButton(dom, "Add clause"));
-		dom.select(dom.byLabel("Clause field"), "Subject");
-		dom.type(dom.byLabel("Clause value"), "paid");
-		dom.click(primaryButton(dom, "Add"));
-		await dom.flush();
-
-		// The count is stale and the save is blocked until the re-preview lands.
-		assert.match(dom.text(), /recounting/i);
-		assert.equal(primaryButton(dom, "Save rule").disabled, true);
-
-		await settlePreview(dom);
-		assert.match(dom.text(), /40 messages match/);
-		assert.equal(primaryButton(dom, "Save rule").disabled, false);
-	});
-
-	it("runs a one-time apply as a back-apply job when the scope is 'just once'", async () => {
-		const dom = mount("from:receipts@stripe.com");
-		dom.select(dom.byLabel("Destination folder"), "mbx-archive");
-		await dom.flush();
-		const radio = dom.query('input[name="rule-scope"][value="once"]');
-		if (!radio) throw new Error("no once scope option");
-		dom.click(radio);
-		await dom.flush();
-
-		dom.click(primaryButton(dom, "Apply now"));
-		await dom.flush();
-		const jobs = (http?.to("/organize") ?? []).filter(
-			(call) => call.method === "POST",
-		);
-		assert.equal(jobs.length, 1);
-	});
-
-	it("holds the one-time apply for a body-text rule, offering the saved rule instead", async () => {
-		// The one-time apply runs the same vector-free matcher the count does. A
-		// `HasWords` clause has no reader there, so the apply is held with the
-		// reason stated — saved as a standing rule the same clause works, because
-		// the index-time matcher reads the whole body.
-		const dom = mount("receipts");
-		dom.select(dom.byLabel("Destination folder"), "mbx-archive");
-		await dom.flush();
-		const radio = dom.query('input[name="rule-scope"][value="once"]');
-		if (!radio) throw new Error("no once scope option");
-		dom.click(radio);
-		await dom.flush();
-
-		assert.equal(primaryButton(dom, "Apply now").disabled, true);
-		assert.match(dom.text(), /can't read message bodies/i);
-		assert.equal(
-			(http?.calls ?? []).filter(
-				(call) => call.path.endsWith("/organize") && call.method === "POST",
-			).length,
-			0,
-		);
-	});
-});