@remit/web-client 0.0.324 → 0.0.326

package/src/lib/move-options.ts

@@ -1,60 +0,0 @@
-import type {
-	RemitImapFolderAppointment,
-	RemitImapMailboxResponse,
-} from "@remit/api-http-client/types.gen.ts";
-import type { FolderTreeNode } from "@remit/ui";
-import { excludeFolder } from "./delete-folder.js";
-import { buildMailboxRoleMap, labelForMailbox } from "./folder-roles.js";
-import { buildMoveTargets } from "./move-targets.js";
-
-type Translator = (key: string, fallback: string) => string;
-
-interface MoveOptionsInput {
-	mailboxes: readonly RemitImapMailboxResponse[];
-	folderAppointments: readonly RemitImapFolderAppointment[];
-	/** Rendered as a non-selectable row, so the user sees where mail lives now. */
-	currentMailboxId?: string;
-	/** Dropped entirely — the folder being deleted is not its own destination. */
-	excludeMailboxId?: string;
-	translator?: Translator;
-}
-
-/**
- * The account's hierarchy separator, which the tree splits paths on. Every
- * mailbox of an account reports the same one, so the first is the account's.
- */
-export const folderDelimiter = (
-	mailboxes: readonly RemitImapMailboxResponse[],
-): string => mailboxes[0]?.hierarchyDelimiter ?? "/";
-
-/**
- * The single shaping of a mailbox list into Move-to picker options. Labels
- * follow the account's role appointments (RFC 032, #976) and any
- * `displayNameOverride`, so a picker reads `Inbox`/`Trash` rather than the
- * provider's leaf, while the provider path it nests and filters under stays
- * untouched.
- *
- * The message count travels with every option (#887): a folder named `Trash`
- * may be an empty look-alike beside the real one, and the count is the only
- * thing that tells them apart. The Move-to picker shows it for the same reason.
- */
-export const buildMoveOptions = ({
-	mailboxes,
-	folderAppointments,
-	currentMailboxId,
-	excludeMailboxId,
-	translator,
-}: MoveOptionsInput): FolderTreeNode[] => {
-	const targets = buildMoveTargets(mailboxes, folderAppointments);
-	const roleMap = buildMailboxRoleMap(folderAppointments);
-	const destinations = excludeMailboxId
-		? excludeFolder(targets, excludeMailboxId)
-		: targets;
-	return destinations.map((mailbox) => ({
-		id: mailbox.mailboxId,
-		label: labelForMailbox(mailbox, roleMap.get(mailbox.mailboxId), translator),
-		path: mailbox.fullPath,
-		isCurrent: mailbox.mailboxId === currentMailboxId,
-		messageCount: mailbox.messageCount,
-	}));
-};

package/src/lib/move-targets.test.ts

@@ -1,235 +0,0 @@
-import assert from "node:assert";
-import { describe, test } from "node:test";
-import type {
-	RemitImapFolderAppointment,
-	RemitImapMailboxResponse,
-} from "@remit/api-http-client/types.gen.ts";
-import { buildMoveTargets } from "./move-targets.js";
-
-const make = (
-	overrides: Partial<RemitImapMailboxResponse> & {
-		mailboxId: string;
-		fullPath: string;
-	},
-): RemitImapMailboxResponse =>
-	({
-		accountId: "acct-1",
-		namespaceType: "personal",
-		namespacePrefix: "",
-		hierarchyDelimiter: "/",
-		messageCount: 0,
-		unseenCount: 0,
-		deletedCount: 0,
-		createdAt: 0,
-		updatedAt: 0,
-		...overrides,
-	}) as RemitImapMailboxResponse;
-
-const appoint = (
-	role: RemitImapFolderAppointment["role"],
-	mailboxId: string,
-): RemitImapFolderAppointment => ({ role, source: "Appointed", mailboxId });
-
-describe("buildMoveTargets — excluded destinations (#236, #976)", () => {
-	test("drops the account's appointed Drafts and Sent mailboxes", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "m1", fullPath: "INBOX" }),
-				make({ mailboxId: "m2", fullPath: "Drafts" }),
-				make({ mailboxId: "m3", fullPath: "Sent" }),
-			],
-			[appoint("Drafts", "m2"), appoint("Sent", "m3")],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId);
-		assert.deepStrictEqual(ids, ["m1"]);
-	});
-
-	test("keeps an UNappointed Drafts/Sent look-alike — exclusion follows the appointment, not the name", () => {
-		// Hostnet-style: INBOX/Drafts carries the flag but is empty; the user
-		// appointed INBOX/Concepten instead. INBOX/Drafts is now just a plain
-		// folder and a valid move destination.
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "inbox", fullPath: "INBOX" }),
-				make({ mailboxId: "empty-drafts", fullPath: "INBOX/Drafts" }),
-				make({ mailboxId: "concepten", fullPath: "INBOX/Concepten" }),
-			],
-			[appoint("Drafts", "concepten")],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId).sort();
-		assert.deepStrictEqual(ids, ["empty-drafts", "inbox"]);
-	});
-
-	test("with no appointments at all, nothing is excluded by role (only Outbox by name)", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "m1", fullPath: "INBOX" }),
-				make({ mailboxId: "m2", fullPath: "Drafts" }),
-				make({ mailboxId: "m3", fullPath: "Outbox" }),
-				make({ mailboxId: "m4", fullPath: "Sent Mail" }),
-			],
-			[],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId).sort();
-		assert.deepStrictEqual(ids, ["m1", "m2", "m4"]);
-	});
-
-	test("keeps Trash and Spam — both are valid manual destinations", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "trash", fullPath: "Trash" }),
-				make({ mailboxId: "spam", fullPath: "Spam" }),
-				make({ mailboxId: "inbox", fullPath: "INBOX" }),
-			],
-			[
-				appoint("Trash", "trash"),
-				appoint("Junk", "spam"),
-				appoint("Inbox", "inbox"),
-			],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId).sort();
-		assert.deepStrictEqual(ids, ["inbox", "spam", "trash"]);
-	});
-
-	test("keeps user-defined folders", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "m1", fullPath: "INBOX" }),
-				make({ mailboxId: "m2", fullPath: "Project Alpha" }),
-				make({ mailboxId: "m3", fullPath: "Receipts/2025" }),
-			],
-			[],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId);
-		assert.deepStrictEqual(ids.sort(), ["m1", "m2", "m3"]);
-	});
-});
-
-describe("buildMoveTargets — sort order", () => {
-	test("orders appointed system folders by role priority, ahead of plain folders", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "custom", fullPath: "Project Alpha" }),
-				make({ mailboxId: "trash", fullPath: "Trash" }),
-				make({ mailboxId: "archive", fullPath: "Archive" }),
-				make({ mailboxId: "inbox", fullPath: "INBOX" }),
-			],
-			[
-				appoint("Trash", "trash"),
-				appoint("Archive", "archive"),
-				appoint("Inbox", "inbox"),
-			],
-		);
-		assert.deepStrictEqual(
-			result.map((m) => m.mailboxId),
-			["inbox", "archive", "trash", "custom"],
-		);
-	});
-
-	test("plain folders sort alphabetically, case-insensitively", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "b", fullPath: "banana" }),
-				make({ mailboxId: "a", fullPath: "Apple" }),
-			],
-			[],
-		);
-		assert.deepStrictEqual(
-			result.map((m) => m.mailboxId),
-			["a", "b"],
-		);
-	});
-});
-
-describe("buildMoveTargets — Outbox locale exclusion (#290)", () => {
-	// Outbox has no IMAP special-use flag and isn't a canonical role, so we
-	// recognize it by name across the locales the curated list covers.
-	const localizedOutboxNames: readonly string[] = [
-		"Outbox",
-		"Postvak UIT",
-		"Boîte d'envoi",
-		"Postausgang",
-		"Buzón de salida",
-		"Posta in uscita",
-		"送信トレイ",
-		"送件匣",
-		"发件箱",
-		"Skrzynka nadawcza",
-	];
-
-	for (const name of localizedOutboxNames) {
-		test(`drops localized Outbox "${name}"`, () => {
-			const result = buildMoveTargets(
-				[
-					make({ mailboxId: "inbox", fullPath: "INBOX" }),
-					make({ mailboxId: "outbox", fullPath: name }),
-				],
-				[],
-			);
-			const ids = result.map((mailbox) => mailbox.mailboxId);
-			assert.deepStrictEqual(ids, ["inbox"]);
-		});
-	}
-
-	test("matches Outbox locale names case-insensitively", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "inbox", fullPath: "INBOX" }),
-				make({ mailboxId: "m1", fullPath: "POSTVAK UIT" }),
-				make({ mailboxId: "m2", fullPath: "postausgang" }),
-				make({ mailboxId: "m3", fullPath: "BOÎTE D'ENVOI" }),
-			],
-			[],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId);
-		assert.deepStrictEqual(ids, ["inbox"]);
-	});
-
-	test("matches Outbox locale names with surrounding whitespace", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "inbox", fullPath: "INBOX" }),
-				make({ mailboxId: "m1", fullPath: "  Outbox  " }),
-				make({ mailboxId: "m2", fullPath: " Postvak UIT " }),
-			],
-			[],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId);
-		assert.deepStrictEqual(ids, ["inbox"]);
-	});
-
-	test("does not over-match unrelated mailbox names", () => {
-		const result = buildMoveTargets(
-			[
-				make({ mailboxId: "inbox", fullPath: "INBOX" }),
-				make({ mailboxId: "trash", fullPath: "Trash" }),
-				make({ mailboxId: "custom", fullPath: "Custom Folder" }),
-				// Substring of a locale name must not trigger the filter — only
-				// the exact normalized leaf matches.
-				make({ mailboxId: "outboxy", fullPath: "Outbox Archive" }),
-			],
-			[],
-		);
-		const ids = result.map((mailbox) => mailbox.mailboxId).sort();
-		assert.deepStrictEqual(ids, ["custom", "inbox", "outboxy", "trash"]);
-	});
-});
-
-describe("buildMoveTargets — the server’s own delimiter (#877)", () => {
-	test("orders and filters by the leaf under a dot-delimited namespace", () => {
-		const dotted = (mailboxId: string, fullPath: string) =>
-			make({ mailboxId, fullPath, hierarchyDelimiter: "." });
-		const result = buildMoveTargets(
-			[
-				dotted("beta", "INBOX.Projects.Beta"),
-				dotted("outbox", "INBOX.Outbox"),
-				dotted("alpha", "INBOX.Zoo.Alpha"),
-			],
-			[],
-		);
-		assert.deepStrictEqual(
-			result.map((mailbox) => mailbox.mailboxId),
-			["alpha", "beta"],
-		);
-	});
-});

package/src/lib/move-targets.ts

@@ -1,70 +0,0 @@
-import type {
-	RemitImapFolderAppointment,
-	RemitImapMailboxResponse,
-} from "@remit/api-http-client/types.gen.ts";
-import { mailboxLeafName } from "@remit/data-ports/mailbox-name";
-import { buildMailboxRoleMap, ROLE_PRIORITY } from "./folder-roles.js";
-
-// Outbox has no IMAP special-use flag (RFC 6154 doesn't define one) and isn't
-// a canonical role (RFC 032 exclusive-folder-appointment) either — it's
-// Remit's own concept, so it isn't in the account's role map. KISS: covers the
-// languages we expect to encounter; extend as needed. Stored lowercased and
-// trimmed; comparisons normalize the input the same way.
-const OUTBOX_LOCALE_NAMES: ReadonlySet<string> = new Set([
-	"outbox",
-	"postvak uit",
-	"boîte d'envoi",
-	"postausgang",
-	"buzón de salida",
-	"posta in uscita",
-	"送信トレイ",
-	"送件匣",
-	"发件箱",
-	"skrzynka nadawcza",
-]);
-
-const isOutboxDestination = (mailbox: RemitImapMailboxResponse): boolean => {
-	const leaf = mailboxLeafName(mailbox).trim().toLowerCase();
-	return OUTBOX_LOCALE_NAMES.has(leaf);
-};
-
-const NON_SYSTEM_PRIORITY = ROLE_PRIORITY.length;
-
-/**
- * Filter and order a flat mailbox list for the Move-to picker (RFC 032
- * exclusive-folder-appointment, #976).
- *
- * Drops the account's appointed Drafts and Sent mailboxes (issue #236) —
- * compose/SMTP flows manage those, moving an arbitrary message into them
- * would break the invariant that they only hold messages the user authored —
- * and Outbox by locale name. Everything else, including a Drafts/Sent
- * look-alike the user hasn't actually appointed (e.g. an empty `INBOX/Drafts`
- * twin next to the real `INBOX/Concepten`), is a perfectly valid destination:
- * the exclusion follows the account's role appointment, not folder-name or
- * SPECIAL-USE guessing. Sorts by the same role priority the sidebar pins by,
- * then alphabetically.
- */
-export const buildMoveTargets = (
-	mailboxes: readonly RemitImapMailboxResponse[],
-	folderAppointments: readonly RemitImapFolderAppointment[],
-): RemitImapMailboxResponse[] => {
-	const roleMap = buildMailboxRoleMap(folderAppointments);
-	const filtered = mailboxes.filter((mailbox) => {
-		const role = roleMap.get(mailbox.mailboxId);
-		if (role === "drafts" || role === "sent") return false;
-		return !isOutboxDestination(mailbox);
-	});
-	return [...filtered].sort((a, b) => {
-		const aPriority = roleMap.has(a.mailboxId)
-			? ROLE_PRIORITY.indexOf(roleMap.get(a.mailboxId) ?? "inbox")
-			: NON_SYSTEM_PRIORITY;
-		const bPriority = roleMap.has(b.mailboxId)
-			? ROLE_PRIORITY.indexOf(roleMap.get(b.mailboxId) ?? "inbox")
-			: NON_SYSTEM_PRIORITY;
-		if (aPriority !== bPriority) return aPriority - bPriority;
-		return mailboxLeafName(a).localeCompare(mailboxLeafName(b), undefined, {
-			sensitivity: "base",
-			numeric: true,
-		});
-	});
-};

package/src/lib/network-error.ts

@@ -1,74 +0,0 @@
-/**
- * Transport failures are identified where they happen, not guessed at later.
- *
- * `fetch()` rejects for exactly two reasons: the request never completed at the
- * transport level (DNS, TLS, connection lost, captive portal, timeout), or it
- * was aborted. It never rejects for an HTTP status. So the one place that knows
- * for certain that an error is a network error is the call site that wrapped
- * `fetch` — everywhere downstream is guessing.
- *
- * Guessing is what the classifier used to do, first by treating every error
- * without an HTTP status as a network blip (which swallowed our own exceptions),
- * then by matching browser failure strings (which is unbounded: WebKit alone
- * says "Failed to fetch", "Load failed", "The network connection was lost." and
- * "The request timed out.", and undici says "fetch failed"). Tagging at the
- * boundary makes the question decidable and the browser's wording irrelevant.
- */
-
-import { recordRequest } from "./request-breadcrumbs";
-
-/** A request that never reached a server. Always soft — never escalated. */
-export class NetworkError extends Error {
-	constructor(cause: unknown) {
-		super(
-			cause instanceof Error && cause.message
-				? cause.message
-				: "The request could not be completed.",
-			{ cause },
-		);
-		this.name = "NetworkError";
-	}
-}
-
-/**
- * A deliberate cancellation — a route change, a React Query cancellation, a
- * caller's own `AbortController`. Its own category: not a failure at all, so it
- * passes through untagged.
- *
- * `AbortSignal.timeout()` rejects with a `TimeoutError` rather than an
- * `AbortError`, and a timeout IS a transport failure, so it is deliberately not
- * matched here and gets tagged like any other.
- */
-const isDeliberateAbort = (error: unknown): boolean =>
-	typeof error === "object" &&
-	error !== null &&
-	"name" in error &&
-	(error as { name?: unknown }).name === "AbortError";
-
-/**
- * `fetch`, with transport failures tagged. Every app-owned request goes through
- * this — the generated client is configured with it, and the hand-written
- * wrapper calls it — so a `NetworkError` downstream is a fact, not an inference.
- *
- * It is also the one place every request is seen, so it records a metadata-only
- * breadcrumb (method, path, status, duration, correlation id) for bug reports.
- * A deliberate abort is not a request outcome worth reporting, so it is
- * re-thrown before any breadcrumb is written.
- */
-export const taggedFetch: typeof fetch = async (input, init) => {
-	const startedAt = performance.now();
-	try {
-		const response = await fetch(input, init);
-		recordRequest({
-			input,
-			init,
-			response,
-			durationMs: performance.now() - startedAt,
-		});
-		return response;
-	} catch (error) {
-		if (isDeliberateAbort(error)) throw error;
-		recordRequest({ input, init, durationMs: performance.now() - startedAt });
-		throw new NetworkError(error);
-	}
-};

package/src/lib/new-folder.test.ts

@@ -1,127 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import {
-	composeFolderPath,
-	type FolderTarget,
-	validateNewFolderName,
-} from "./new-folder.js";
-
-const parent: FolderTarget = { fullPath: "INBOX", hierarchyDelimiter: "/" };
-
-describe("composeFolderPath", () => {
-	it("returns the trimmed name for a root folder", () => {
-		assert.equal(composeFolderPath("  Receipts  "), "Receipts");
-	});
-
-	it("joins the parent path by the parent's delimiter", () => {
-		assert.equal(composeFolderPath("Receipts", parent), "INBOX/Receipts");
-	});
-
-	it("honours a non-slash delimiter", () => {
-		assert.equal(
-			composeFolderPath("Receipts", {
-				fullPath: "INBOX",
-				hierarchyDelimiter: ".",
-			}),
-			"INBOX.Receipts",
-		);
-	});
-});
-
-describe("validateNewFolderName", () => {
-	const base = { delimiter: "/", existingPaths: ["INBOX", "INBOX/Archive"] };
-
-	it("rejects an empty or whitespace name", () => {
-		assert.match(
-			validateNewFolderName({ ...base, name: "   " }) ?? "",
-			/Enter a folder name/,
-		);
-	});
-
-	it("rejects a name containing the hierarchy delimiter", () => {
-		assert.match(
-			validateNewFolderName({ ...base, name: "a/b" }) ?? "",
-			/can't contain/,
-		);
-	});
-
-	it("rejects a name that would contain a non-slash delimiter", () => {
-		assert.match(
-			validateNewFolderName({
-				delimiter: ".",
-				existingPaths: [],
-				name: "a.b",
-			}) ?? "",
-			/can't contain/,
-		);
-	});
-
-	it("rejects an exact collision with an existing path", () => {
-		assert.match(
-			validateNewFolderName({
-				...base,
-				name: "Archive",
-				parent,
-			}) ?? "",
-			/already exists/,
-		);
-	});
-
-	it("rejects a root name that collides with an existing top-level fullPath", () => {
-		assert.match(
-			validateNewFolderName({
-				delimiter: "/",
-				existingPaths: ["Work"],
-				name: "Work",
-			}) ?? "",
-			/already exists/,
-		);
-	});
-
-	it("treats INBOX case-insensitively for collisions", () => {
-		assert.match(
-			validateNewFolderName({
-				delimiter: "/",
-				existingPaths: ["INBOX"],
-				name: "inbox",
-			}) ?? "",
-			/already exists/,
-		);
-	});
-
-	it("compares non-INBOX segments case-sensitively", () => {
-		assert.equal(
-			validateNewFolderName({
-				delimiter: "/",
-				existingPaths: ["Work"],
-				name: "work",
-			}),
-			undefined,
-		);
-	});
-
-	it("accepts a valid new nested folder", () => {
-		assert.equal(
-			validateNewFolderName({ ...base, name: "Receipts", parent }),
-			undefined,
-		);
-	});
-});
-
-describe("validateNewFolderName in a flat namespace", () => {
-	const flat = { delimiter: "", existingPaths: ["INBOX", "Archive"] };
-
-	it("accepts a name when the server reports no delimiter", () => {
-		assert.equal(
-			validateNewFolderName({ ...flat, name: "Receipts" }),
-			undefined,
-		);
-	});
-
-	it("still catches a collision", () => {
-		assert.match(
-			validateNewFolderName({ ...flat, name: "inbox" }) ?? "",
-			/already exists/,
-		);
-	});
-});

package/src/lib/new-folder.ts

@@ -1,59 +0,0 @@
-export interface FolderTarget {
-	fullPath: string;
-	hierarchyDelimiter: string;
-}
-
-/**
- * The full path of a new folder: the leaf name on its own for a root folder, or
- * the parent's path joined to the name by the parent's hierarchy delimiter.
- */
-export function composeFolderPath(name: string, parent?: FolderTarget): string {
-	const trimmed = name.trim();
-	if (!parent) return trimmed;
-	return `${parent.fullPath}${parent.hierarchyDelimiter}${trimmed}`;
-}
-
-/**
- * IMAP treats the INBOX root case-insensitively; every other path segment is
- * exact. Canonicalizing only a leading `inbox` segment lets the collision check
- * match `inbox/x` against `INBOX/x` without folding case anywhere else.
- */
-const canonicalizePath = (path: string, delimiter: string): string => {
-	const parts = delimiter.length === 0 ? [path] : path.split(delimiter);
-	if (parts[0]?.toLowerCase() === "inbox") parts[0] = "INBOX";
-	return parts.join(delimiter);
-};
-
-/**
- * Why the folder can't be created, or `undefined` when the name is valid. A
- * name must be non-empty, must not contain the hierarchy delimiter (nesting is
- * expressed by the parent select, not by typing a path), and must not collide
- * with an existing folder — INBOX compared case-insensitively, everything else
- * exactly. A server that reports no delimiter has a flat namespace, so there is
- * no separator to forbid and every name would otherwise contain it.
- */
-export function validateNewFolderName({
-	name,
-	delimiter,
-	parent,
-	existingPaths,
-}: {
-	name: string;
-	delimiter: string;
-	parent?: FolderTarget;
-	existingPaths: readonly string[];
-}): string | undefined {
-	const trimmed = name.trim();
-	if (trimmed === "") return "Enter a folder name.";
-	if (delimiter.length > 0 && trimmed.includes(delimiter))
-		return `A folder name can't contain "${delimiter}".`;
-	const candidate = canonicalizePath(
-		composeFolderPath(trimmed, parent),
-		delimiter,
-	);
-	const collides = existingPaths.some(
-		(path) => canonicalizePath(path, delimiter) === candidate,
-	);
-	if (collides) return "A folder with that name already exists.";
-	return undefined;
-}

package/src/lib/onboarding-completion.test.ts

@@ -1,48 +0,0 @@
-/**
- * The onboarding wizard creates the account server-side, then the user clicks
- * "Go to inbox". The /mail first-run guard redirects back to /onboarding while
- * the cached config still reports zero accounts, so completion must refresh
- * config and wait for it before navigating — otherwise the user loops back into
- * the wizard.
- */
-
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import { completeOnboarding } from "./onboarding-completion.js";
-
-describe("completeOnboarding", () => {
-	it("navigates to the inbox only after config is refreshed", async () => {
-		const order: string[] = [];
-		let resolveRefetch: (() => void) | undefined;
-
-		const refetchQueries = () => {
-			order.push("refetch:start");
-			return new Promise<void>((resolve) => {
-				resolveRefetch = () => {
-					order.push("refetch:done");
-					resolve();
-				};
-			});
-		};
-
-		const done = completeOnboarding({
-			queryClient: { refetchQueries },
-			recordCompleted: () => order.push("recorded"),
-			navigateToInbox: () => order.push("navigate"),
-		});
-
-		// Refetch is in flight; navigation must not have happened yet.
-		await Promise.resolve();
-		assert.deepEqual(order, ["recorded", "refetch:start"]);
-
-		resolveRefetch?.();
-		await done;
-
-		assert.deepEqual(order, [
-			"recorded",
-			"refetch:start",
-			"refetch:done",
-			"navigate",
-		]);
-	});
-});