@remit/web-client 0.0.93 → 0.0.94

package/src/lib/wizard-history.ts

@@ -37,6 +37,33 @@ export const wizardStepFromParam = (value: unknown): StepId | undefined => {
 	return parsed.success ? parsed.data : undefined;
 };
 
+/**
+ * Which affordance opened the wizard. Absent is the selection bar; `search` is
+ * the make-filter affordance in the search results header, whose clauses come
+ * from the query rather than from ticked rows (#477 1.8).
+ *
+ * It rides the URL beside the step for the same reason the step does: a
+ * reloaded document has to know which walk it is in the middle of, and the
+ * query it was seeded from is in the URL already. It decides two things and no
+ * more — which step the wizard opens on, and whether the query seeds the
+ * clauses (#477 3.4). One host answers either way.
+ */
+const wizardEntry = z.literal("search");
+
+export type WizardEntry = z.infer<typeof wizardEntry>;
+
+export const wizardEntryFromParam = (
+	value: unknown,
+): WizardEntry | undefined => {
+	const parsed = wizardEntry.safeParse(value);
+	return parsed.success ? parsed.data : undefined;
+};
+
+export const wizardEntryValue = z.unknown().transform(wizardEntryFromParam);
+
+export const useWizardEntryValue = (): WizardEntry | undefined =>
+	useSearch({ from: "/mail", select: (search) => search.wizardFrom });
+
 /**
  * The route's `wizard` field. A value the wizard cannot be on reads as no step
  * rather than as a validation failure, so a truncated or hand-typed link lands
@@ -54,14 +81,21 @@ export const useWizardStepValue = (): StepId | undefined =>
 
 /**
  * Opens the wizard on a step, from a surface that does not drive it — a verb on
- * the selection bar. The push is the wizard's first owned entry, so the back
- * that leaves it lands on the list with the selection still ticked.
+ * the selection bar, or the make-filter affordance on a search. The push is the
+ * wizard's first owned entry, so the back that leaves it lands on the list with
+ * the selection still ticked.
  */
-export const useOpenWizard = (): ((step: StepId) => void) => {
+export const useOpenWizard = (): ((
+	step: StepId,
+	entry?: WizardEntry,
+) => void) => {
 	const navigate = useNavigate();
 	return useCallback(
-		(step: StepId) => {
-			navigate({ to: ".", search: (prev) => ({ ...prev, wizard: step }) });
+		(step: StepId, entry?: WizardEntry) => {
+			navigate({
+				to: ".",
+				search: (prev) => ({ ...prev, wizard: step, wizardFrom: entry }),
+			});
 		},
 		[navigate],
 	);
@@ -78,10 +112,15 @@ export const useWizardStep = (openingStep: StepId): WizardStepNavigation => {
 	const router = useRouter();
 	const navigate = useNavigate();
 	const step = useWizardStepValue();
+	const entry = useWizardEntryValue();
 	// Whether this document loaded already holding a step, which is the one
 	// entrance that leaves the wizard unrooted. A step the app itself pushed
 	// arrives rooted, so re-rooting it would duplicate the entry underneath it.
 	const loadedHoldingStep = useRef(step !== undefined);
+	// The entry that step was reached by, so the root the wizard is put back on
+	// carries neither the step nor the affordance that opened it, and the entry
+	// pushed over it carries both.
+	const loadedEntry = useRef(entry);
 	const pushedTo = useRef<StepId | undefined>(undefined);
 
 	// Two taps on Continue land before the URL settles, and both would push the
@@ -94,15 +133,24 @@ export const useWizardStep = (openingStep: StepId): WizardStepNavigation => {
 	useEffect(() => {
 		if (!loadedHoldingStep.current) return;
 		loadedHoldingStep.current = false;
+		const openingEntry = loadedEntry.current;
 		void (async () => {
 			await navigate({
 				to: ".",
-				search: (prev) => ({ ...prev, wizard: undefined }),
+				search: (prev) => ({
+					...prev,
+					wizard: undefined,
+					wizardFrom: undefined,
+				}),
 				replace: true,
 			});
 			await navigate({
 				to: ".",
-				search: (prev) => ({ ...prev, wizard: openingStep }),
+				search: (prev) => ({
+					...prev,
+					wizard: openingStep,
+					wizardFrom: openingEntry,
+				}),
 			});
 		})();
 	}, [openingStep, navigate]);

package/src/routes/mail.tsx

@@ -43,7 +43,7 @@ import {
 	searchInputForView,
 	shouldMirrorQuery,
 } from "@/lib/search-view";
-import { wizardStepValue } from "@/lib/wizard-history";
+import { wizardEntryValue, wizardStepValue } from "@/lib/wizard-history";
 import "@/lib/client";
 
 // `MailContext` / `useMailContext` live in `@/lib/mail-context` so the provider
@@ -57,6 +57,8 @@ const mailSearchSchema = z.object({
 	// The selection wizard's step (#477 clause 1.6). The router owns history, so
 	// the step is a validated search param rather than a raw pushState entry.
 	wizard: wizardStepValue,
+	// Which affordance opened it, so a reload lands back on the walk it was in.
+	wizardFrom: wizardEntryValue,
 });
 
 export const Route = createFileRoute("/mail")({

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,
-		);
-	});
-});

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

@@ -1,140 +0,0 @@
-import { mailboxOperationsListMailboxesOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import {
-	buildSearchRule,
-	type FilterRule,
-	FilterRuleEditor,
-	type FolderOption,
-	type SearchConversion,
-	SearchConversionNoticeView,
-} from "@remit/ui";
-import { useQuery } from "@tanstack/react-query";
-import { useMemo, useState } from "react";
-import { useCreateFilter } from "@/hooks/useFilters";
-import { useOrganizeJob } from "@/hooks/useOrganizeJob";
-import { useRuleEditorState } from "@/hooks/useRuleEditorState";
-import { useRulePreview } from "@/hooks/useRulePreview";
-import { getMailboxDisplayName } from "@/lib/folder-roles";
-import { buildMoveTargets } from "@/lib/move-targets";
-import {
-	rulePredicate,
-	ruleToDraft,
-	SUPPORTED_CLAUSE_FIELDS,
-} from "@/lib/organize/rule-model";
-import {
-	CommitError,
-	FilterSaved,
-	JobProgress,
-	SavingState,
-} from "./rule-editor-states";
-
-interface SearchFilterEditorProps {
-	/** The account the filter is created for (an `account:` facet, else the active account). */
-	accountId: string;
-	/** The converted search — its clauses seed the rule, its drops seed the notice. */
-	conversion: SearchConversion;
-	/**
-	 * The converted literal predicate's live count, seeding the editor without a
-	 * re-fetch. Absent when the predicate is one the vector-free matcher refuses —
-	 * a `HasWords` clause from the search's free text — in which case the count
-	 * region says so and the one-time apply is held; saving it as a standing rule
-	 * still works, since the index-time matcher does read message bodies.
-	 */
-	seedCount?: number;
-	onClose: () => void;
-}
-
-/**
- * The filter-from-search surface as the chip editor (RFC 038 D5). The current
- * search's literal terms and facets arrive already converted to clauses; this
- * opens the shared rule editor pre-filled on them, over the same preview/apply
- * endpoints as Organize. The conversion notice states what the search carried
- * that the filter cannot — a folder scope, non-clause facets, semantic reach.
- *
- * A search-derived rule has no message anchor, so the semantic widen chip is not
- * offered here (its loss, where the deployment could have served it, is the
- * notice's job). The commit gate is the same: the count on screen is the literal
- * set a commit acts on.
- */
-export function SearchFilterEditor({
-	accountId,
-	conversion,
-	seedCount,
-	onClose,
-}: SearchFilterEditorProps) {
-	const [initialRule] = useState<FilterRule>(() => buildSearchRule(conversion));
-	const { rule, handlers } = useRuleEditorState({ initialRule });
-
-	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 { count: preview } = useRulePreview(
-		accountId,
-		rulePredicate(rule),
-		seedCount,
-	);
-
-	const organizeJob = useOrganizeJob(accountId);
-	const createFilter = useCreateFilter(accountId);
-
-	const commit = () => {
-		const draft = ruleToDraft(rule);
-		if (rule.scope === "once") {
-			organizeJob.start(draft);
-			return;
-		}
-		createFilter.createFilter(
-			draft,
-			rule.scope === "standing" ? "standing" : "temporary",
-			(rule.name ?? "").trim(),
-		);
-	};
-
-	if (organizeJob.isStarting || organizeJob.isRunning || organizeJob.isDone) {
-		return (
-			<JobProgress
-				progress={organizeJob.progress}
-				isDone={organizeJob.isDone}
-				runningLabel="Organizing your mail…"
-				onClose={onClose}
-			/>
-		);
-	}
-
-	if (createFilter.isPending) return <SavingState />;
-	if (createFilter.isSuccess) return <FilterSaved onClose={onClose} />;
-	if (createFilter.isError)
-		return <CommitError onRetry={createFilter.reset} onClose={onClose} />;
-
-	return (
-		<FilterRuleEditor
-			rule={rule}
-			folders={folders}
-			preview={preview}
-			notice={
-				<SearchConversionNoticeView
-					notice={{
-						scopedOutFolder: conversion.scopedOut?.label,
-						droppedFacets: conversion.droppedFacets.map((facet) => facet.label),
-						droppedSemantic: conversion.droppedSemantic,
-					}}
-				/>
-			}
-			semanticAvailable={false}
-			clauseFields={SUPPORTED_CLAUSE_FIELDS}
-			{...handlers}
-			onCommit={commit}
-			onCancel={onClose}
-		/>
-	);
-}