@remit/web-client 0.0.321 → 0.0.323

package/src/hooks/useMoveMessages.test.ts

@@ -1,63 +0,0 @@
-import assert from "node:assert";
-import { describe, test } from "node:test";
-import type { RemitImapThreadMessageResponse } from "@remit/api-http-client/types.gen.ts";
-import { removeMovedMessagesFromItems } from "./useMoveMessages.js";
-
-const make = (
-	overrides: Partial<RemitImapThreadMessageResponse> & {
-		messageId: string;
-		threadMessageId: string;
-	},
-): RemitImapThreadMessageResponse =>
-	({
-		threadId: "t1",
-		mailboxId: "mb1",
-		accountConfigId: "acc-1",
-		subject: "s",
-		fromName: "n",
-		fromEmail: "e",
-		sentDate: "2025-01-01T00:00:00Z",
-		snippet: "",
-		hasAttachment: false,
-		hasStars: false,
-		isRead: true,
-		...overrides,
-	}) as RemitImapThreadMessageResponse;
-
-describe("removeMovedMessagesFromItems (#236)", () => {
-	test("drops items whose messageId is in the set", () => {
-		const items = [
-			make({ messageId: "m1", threadMessageId: "tm1" }),
-			make({ messageId: "m2", threadMessageId: "tm2" }),
-			make({ messageId: "m3", threadMessageId: "tm3" }),
-		];
-		const got = removeMovedMessagesFromItems(items, new Set(["m2"]));
-		assert.deepStrictEqual(
-			got.map((i) => i.messageId),
-			["m1", "m3"],
-		);
-	});
-
-	test("returns the input unchanged when nothing matches", () => {
-		const items = [
-			make({ messageId: "m1", threadMessageId: "tm1" }),
-			make({ messageId: "m2", threadMessageId: "tm2" }),
-		];
-		const got = removeMovedMessagesFromItems(items, new Set(["nope"]));
-		assert.equal(got.length, 2);
-	});
-
-	test("returns an empty array when all items match", () => {
-		const items = [
-			make({ messageId: "m1", threadMessageId: "tm1" }),
-			make({ messageId: "m2", threadMessageId: "tm2" }),
-		];
-		const got = removeMovedMessagesFromItems(items, new Set(["m1", "m2"]));
-		assert.deepStrictEqual(got, []);
-	});
-
-	test("returns an empty array for empty input", () => {
-		const got = removeMovedMessagesFromItems([], new Set(["m1"]));
-		assert.deepStrictEqual(got, []);
-	});
-});

package/src/hooks/useOrganizeWiden.test.ts

@@ -1,217 +0,0 @@
-/**
- * The widen step's two-phase preview. On a semantic-capable deployment it runs
- * the anchor preview once and stops. On a deployment that reports
- * `semanticUnavailable` (no vector pipeline — semantic-capability.ts) it
- * re-previews with what the whole selection agrees on (#458) — one `From` or
- * `FromDomain` clause when senders agree, the shared subject when they don't,
- * combined with `Or`, no anchor — and reports the literal match count, so
- * every commit scope acts on the same set the fallback derived.
- */
-
-import assert from "node:assert/strict";
-import { afterEach, describe, it } from "node:test";
-import { createElement, useEffect } from "react";
-import { createDomHarness, type DomHarness } from "../test-support/dom";
-import { type HttpMock, mockFetch } from "../test-support/http";
-import { useOrganizeWiden } from "./useOrganizeWiden";
-
-let harness: DomHarness | undefined;
-let http: HttpMock | undefined;
-
-afterEach(() => {
-	harness?.close();
-	harness = undefined;
-	http?.restore();
-	http = undefined;
-});
-
-type Widen = ReturnType<typeof useOrganizeWiden>;
-type Responder = Parameters<typeof mockFetch>[0];
-
-const PREVIEW_PATH = "/organize/preview";
-
-const mount = (
-	senders: string[],
-	responder: Responder,
-	subjects: string[] = [],
-) => {
-	http = mockFetch(responder);
-	harness = createDomHarness();
-	const holder: { current: Widen | undefined } = { current: undefined };
-	function Probe() {
-		const widen = useOrganizeWiden("acc-1", "msg-1", senders, subjects);
-		holder.current = widen;
-		const { preview } = widen;
-		useEffect(() => {
-			preview();
-		}, [preview]);
-		return null;
-	}
-	harness.renderApp(createElement(Probe));
-	return holder;
-};
-
-async function settle(minCalls: number): Promise<void> {
-	const dom = harness;
-	const mock = http;
-	if (!dom || !mock) throw new Error("not mounted");
-	for (let attempt = 0; attempt < 40; attempt += 1) {
-		await dom.flush();
-		if (mock.to(PREVIEW_PATH).length >= minCalls) {
-			await dom.flush();
-			await dom.wait(1);
-			await dom.flush();
-			return;
-		}
-		await dom.wait(1);
-	}
-}
-
-describe("useOrganizeWiden — semantic-capable deployment", () => {
-	it("previews the anchor once and reports the semantic match, no fallback", async () => {
-		const holder = mount([], () => ({
-			matchedCount: 42,
-			messageIds: ["a", "b"],
-			semanticUnavailable: false,
-		}));
-		await settle(1);
-
-		assert.equal(http?.to(PREVIEW_PATH).length, 1);
-		const [call] = http?.to(PREVIEW_PATH) ?? [];
-		assert.equal(call.body?.anchorMessageId, "msg-1");
-		assert.deepEqual(call.body?.literalClauses, []);
-
-		const widen = holder.current;
-		assert.equal(widen?.matchedCount, 42);
-		assert.equal(widen?.semanticUnavailable, false);
-		assert.equal(widen?.senderFallback, false);
-		assert.deepEqual(widen?.matchPredicate, {
-			anchorMessageId: "msg-1",
-			matchOperator: "And",
-			literalClauses: [],
-		});
-	});
-});
-
-describe("useOrganizeWiden — no vector pipeline, senders present", () => {
-	it("re-previews with distinct sender From clauses combined with Or and no anchor", async () => {
-		const holder = mount(
-			["npm@github.com", "npm@github.com", "a@x.com"],
-			(call) =>
-				call.body?.anchorMessageId
-					? { matchedCount: 0, messageIds: [], semanticUnavailable: true }
-					: {
-							matchedCount: 128,
-							messageIds: ["m"],
-							semanticUnavailable: false,
-						},
-		);
-		await settle(2);
-
-		const calls = http?.to(PREVIEW_PATH) ?? [];
-		assert.equal(calls.length, 2);
-		// The second preview is the literal fallback: no anchor, Or, distinct
-		// sender From clauses.
-		const second = calls[1];
-		assert.equal(second.body?.anchorMessageId, undefined);
-		assert.equal(second.body?.matchOperator, "Or");
-		assert.deepEqual(second.body?.literalClauses, [
-			{ field: "From", value: "npm@github.com" },
-			{ field: "From", value: "a@x.com" },
-		]);
-
-		const widen = holder.current;
-		assert.equal(widen?.matchedCount, 128);
-		assert.equal(widen?.semanticUnavailable, true);
-		assert.equal(widen?.senderFallback, true);
-		assert.deepEqual(widen?.senders, ["npm@github.com", "a@x.com"]);
-		assert.deepEqual(widen?.matchPredicate, {
-			matchOperator: "Or",
-			literalClauses: [
-				{ field: "From", value: "npm@github.com" },
-				{ field: "From", value: "a@x.com" },
-			],
-		});
-	});
-});
-
-describe("useOrganizeWiden — no vector pipeline, mixed senders sharing a subject", () => {
-	it("re-previews with the shared subject rather than one From clause per sender (#458)", async () => {
-		const holder = mount(
-			["billing@acme.test", "accounts@globex.test"],
-			(call) =>
-				call.body?.anchorMessageId
-					? { matchedCount: 0, messageIds: [], semanticUnavailable: true }
-					: {
-							matchedCount: 64,
-							messageIds: ["m"],
-							semanticUnavailable: false,
-						},
-			["Invoice 1841", "Invoice 1902"],
-		);
-		await settle(2);
-
-		const calls = http?.to(PREVIEW_PATH) ?? [];
-		assert.equal(calls.length, 2);
-		const second = calls[1];
-		assert.equal(second.body?.anchorMessageId, undefined);
-		assert.equal(second.body?.matchOperator, "Or");
-		assert.deepEqual(second.body?.literalClauses, [
-			{ field: "Subject", value: "Invoice" },
-		]);
-
-		const widen = holder.current;
-		assert.deepEqual(widen?.matchPredicate, {
-			matchOperator: "Or",
-			literalClauses: [{ field: "Subject", value: "Invoice" }],
-		});
-	});
-});
-
-describe("useOrganizeWiden — no vector pipeline, no senders", () => {
-	it("does not re-preview and stays capability-absent with the anchor predicate", async () => {
-		const holder = mount([], () => ({
-			matchedCount: 0,
-			messageIds: [],
-			semanticUnavailable: true,
-		}));
-		await settle(1);
-		// Give any spurious second preview a chance to fire before asserting none did.
-		await harness?.flush();
-
-		assert.equal(http?.to(PREVIEW_PATH).length, 1);
-		const widen = holder.current;
-		assert.equal(widen?.semanticUnavailable, true);
-		assert.equal(widen?.senderFallback, false);
-		assert.deepEqual(widen?.senders, []);
-		assert.equal(widen?.matchPredicate.anchorMessageId, "msg-1");
-	});
-
-	it("still re-previews on a shared subject alone (#458)", async () => {
-		const holder = mount(
-			[],
-			(call) =>
-				call.body?.anchorMessageId
-					? { matchedCount: 0, messageIds: [], semanticUnavailable: true }
-					: {
-							matchedCount: 12,
-							messageIds: ["m"],
-							semanticUnavailable: false,
-						},
-			["Invoice 1841", "Invoice 1902"],
-		);
-		await settle(2);
-
-		const calls = http?.to(PREVIEW_PATH) ?? [];
-		assert.equal(calls.length, 2);
-		const second = calls[1];
-		assert.equal(second.body?.anchorMessageId, undefined);
-		assert.deepEqual(second.body?.literalClauses, [
-			{ field: "Subject", value: "Invoice" },
-		]);
-
-		const widen = holder.current;
-		assert.equal(widen?.semanticUnavailable, true);
-		assert.equal(widen?.senderFallback, true);
-	});
-});

package/src/hooks/useOrganizeWiden.ts

@@ -1,112 +0,0 @@
-import { organizeOperationsPreviewOrganizeMutation } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import { derivePropertyClauses, distinctSenders } from "@remit/ui";
-import { useMutation } from "@tanstack/react-query";
-import { useCallback, useEffect, useMemo, useState } from "react";
-import { buildOrganizeInput } from "@/lib/organize/organize-model";
-import {
-	buildSenderFallbackDraft,
-	type OrganizeMatchPredicate,
-} from "@/lib/organize/sender-fallback";
-
-/**
- * "Select similar messages" (the widen step) with a selection-agreement
- * fallback for deployments without the vector pipeline.
- *
- * The first preview runs the semantic anchor (POST /organize/preview). When the
- * server reports `semanticUnavailable` — no vector pipeline on this deployment
- * (semantic-capability.ts) — the widen re-previews with the literal vocabulary
- * that matches vector-free: what the whole selection agrees on (#458) — one
- * sender/domain clause, or the subject the selection shares when senders don't
- * — combined with `Or`, no anchor. The counts shown, the one-time back-apply,
- * and any standing filter then all carry that same literal predicate, so the
- * previewed set equals the set every commit scope acts on.
- *
- * A semantic-capable deployment never fires the second preview and keeps exactly
- * the anchor behaviour.
- */
-export const useOrganizeWiden = (
-	accountId: string | undefined,
-	anchorMessageId: string | undefined,
-	senders: readonly string[],
-	subjects: readonly string[],
-) => {
-	const mutation = useMutation(organizeOperationsPreviewOrganizeMutation());
-	const { mutate, reset: mutationReset } = mutation;
-	const [fellBack, setFellBack] = useState(false);
-
-	const distinct = useMemo(() => distinctSenders(senders), [senders]);
-	const agreementClauses = useMemo(
-		() => derivePropertyClauses(distinct, subjects),
-		[distinct, subjects],
-	);
-
-	const preview = useCallback(() => {
-		if (!accountId || !anchorMessageId) return;
-		setFellBack(false);
-		mutate({
-			path: { accountId },
-			body: buildOrganizeInput({
-				anchorMessageId,
-				matchOperator: "And",
-				literalClauses: [],
-			}),
-		});
-	}, [accountId, anchorMessageId, mutate]);
-
-	const reset = useCallback(() => {
-		setFellBack(false);
-		mutationReset();
-	}, [mutationReset]);
-
-	const data = mutation.data;
-	// The first preview came back capability-absent and the selection agrees on
-	// something to match on — a sender, a domain, or just a shared subject, not
-	// senders alone (#458): re-preview with the literal clauses before showing
-	// any count, so the count the user sees is the agreement's match count,
-	// never a flash of the empty semantic result.
-	const willFallBack =
-		data?.semanticUnavailable === true &&
-		!fellBack &&
-		agreementClauses.length > 0;
-
-	useEffect(() => {
-		if (!willFallBack || !accountId) return;
-		setFellBack(true);
-		mutate({
-			path: { accountId },
-			body: buildOrganizeInput(buildSenderFallbackDraft(distinct, subjects)),
-		});
-	}, [willFallBack, accountId, distinct, subjects, mutate]);
-
-	// The literal re-preview has no anchor, so its response reports
-	// `semanticUnavailable: false`; `fellBack` is what remembers the capability
-	// was absent once the fallback has run.
-	const capabilityAbsent = fellBack || (data?.semanticUnavailable ?? false);
-	const isPending = mutation.isPending || willFallBack;
-
-	const matchPredicate: OrganizeMatchPredicate = fellBack
-		? { matchOperator: "Or", literalClauses: agreementClauses }
-		: {
-				...(anchorMessageId ? { anchorMessageId } : {}),
-				matchOperator: "And",
-				literalClauses: [],
-			};
-
-	return {
-		preview,
-		reset,
-		matchedCount: isPending ? undefined : data?.matchedCount,
-		messageIds: isPending ? undefined : data?.messageIds,
-		// The deployment ships no vector pipeline — the widen matched on senders
-		// (or, with no senders, on nothing) rather than semantic similarity.
-		semanticUnavailable: capabilityAbsent,
-		// The widen fell back to sender matching and produced a usable literal
-		// match set: the commit scopes reach it, and the copy names the senders.
-		senderFallback: fellBack,
-		senders: distinct,
-		matchPredicate,
-		isPending,
-		isError: mutation.isError,
-		error: mutation.error,
-	};
-};

package/src/hooks/useSemanticSearch.ts

@@ -1,132 +0,0 @@
-import { semanticSearchOperationsSemanticSearchOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import type { RemitImapSemanticSearchResult } from "@remit/api-http-client/types.gen.ts";
-import { MessageCategory } from "@remit/domain-enums";
-import { useQuery } from "@tanstack/react-query";
-import { useMailContext } from "@/lib/mail-context";
-import { normalizeSearchQuery } from "@/lib/search-query";
-import { semanticMailboxScope } from "@/lib/search-scope";
-import { parseSearchTokens, type SearchToken } from "@/lib/search-tokens";
-import { useBrowsedList } from "@/routing";
-import { useSearchTokenContext } from "./useSearchTokenContext";
-
-/** Cap the "Related" section; the literal "Top matches" is the primary surface. */
-const SEMANTIC_RESULT_LIMIT = 20;
-
-const MESSAGE_CATEGORIES = new Set<string>(Object.values(MessageCategory));
-
-/** Narrow a filter-chip id to a real category, or `undefined` for `"all"` / any unknown value. */
-const toCategoryParam = (
-	filterCategory: string | undefined,
-): (typeof MessageCategory)[keyof typeof MessageCategory] | undefined =>
-	filterCategory !== undefined && MESSAGE_CATEGORIES.has(filterCategory)
-		? (filterCategory as (typeof MessageCategory)[keyof typeof MessageCategory])
-		: undefined;
-
-/**
- * `isRead` for the read-state tokens. `is:unread` and `is:read` are opposites,
- * so a query carrying both asks for nothing; unread wins, being the narrower
- * ask and the one the chips also offer.
- */
-const readStateParam = (tokens: SearchToken[]): boolean | undefined => {
-	if (tokens.some((t) => t.type === "isUnread")) return false;
-	if (tokens.some((t) => t.type === "isRead")) return true;
-	return undefined;
-};
-
-interface UseSemanticSearchParams {
-	/**
-	 * Restrict results to a single mailbox. A mailbox route pins the scope to its
-	 * own mailbox regardless of what is passed here, so this only carries a scope
-	 * a route does not already imply.
-	 */
-	mailboxId?: string;
-	/**
-	 * Restrict results to a single header-derived category. The sentinel `"all"`
-	 * means no category scope — the param is omitted from the request entirely.
-	 */
-	filterCategory?: string;
-}
-
-/**
- * Fetch semantic-search hits for the active query (one source of truth: the
- * debounced `searchQuery` in `MailContext`). Scoping mirrors the literal search
- * and is taken from the route (`semanticMailboxScope`): no chip means global, a
- * chip means this engine respects it like every other. Disabled until the query
- * is non-empty so an empty field issues no request.
- *
- * Filter tokens (`has:attachment`, `is:unread`/`is:read`, `is:starred`,
- * `category:`, `before:`/`after:`) parsed from the query map onto the search
- * API's own filter params. `from:`, `subject:` and `account:` have no
- * equivalent on `GET /search/semantic` (no sender, subject or
- * account filter) — all still render as chips and narrow the literal engine,
- * but never reach the semantic request (`account:` is a documented gap, see
- * doc/design/flows/06-search.md — the semantic index is per account config).
- * `in:` resolves to a mailboxId, so typing `in:archive` re-scopes the search
- * from the unscoped daily brief. It cannot contradict a scoped view because
- * `useSearchTokenContext` does not resolve the term there at all — the term
- * stays free text and no engine, and no chip, treats it as a filter.
- *
- * With no resolved mailbox the request spans every mailbox of every account the
- * caller owns: the backend partitions the vector index by accountConfigId (one
- * per signed-in user, not per mail account), so unscoped is genuinely global.
- */
-export function useSemanticSearch({
-	mailboxId,
-	filterCategory,
-}: UseSemanticSearchParams = {}): {
-	hits: RemitImapSemanticSearchResult[];
-	isLoading: boolean;
-} {
-	const { searchQuery } = useMailContext();
-	const tokenContext = useSearchTokenContext();
-	const browsed = useBrowsedList();
-	const normalizedQuery = normalizeSearchQuery(searchQuery);
-	const { freeText, tokens } = parseSearchTokens(normalizedQuery, tokenContext);
-	// `freeText`, not the raw query: a query of nothing but tokens
-	// (`has:attachment`, `in:archive`) parses to empty free text, and asking a
-	// vector index what an empty string means has no answer to give. The literal
-	// engines still apply those tokens; the semantic section simply has nothing
-	// to rank.
-	const enabled = freeText.length > 0;
-
-	// The chip wins over the token: it is the visible narrowing, and the two can
-	// only disagree when the user set both.
-	const categoryToken = tokens.find((t) => t.type === "category");
-	const category = toCategoryParam(filterCategory) ?? categoryToken?.category;
-	const hasAttachment = tokens.some((t) => t.type === "hasAttachment")
-		? true
-		: undefined;
-	const hasStars = tokens.some((t) => t.type === "isStarred")
-		? true
-		: undefined;
-	const isRead = readStateParam(tokens);
-	const afterToken = tokens.find((t) => t.type === "after");
-	const beforeToken = tokens.find((t) => t.type === "before");
-	const inToken = tokens.find((t) => t.type === "in");
-	// The route decides the scope, not the call site — see `semanticMailboxScope`.
-	const effectiveMailboxId = semanticMailboxScope({
-		browsed,
-		callerMailboxId: mailboxId,
-		inTokenMailboxId: inToken?.mailboxId,
-	});
-
-	const { data, isLoading } = useQuery({
-		...semanticSearchOperationsSemanticSearchOptions({
-			query: {
-				query: freeText,
-				mailboxId: effectiveMailboxId,
-				limit: SEMANTIC_RESULT_LIMIT,
-				...(category !== undefined ? { category } : {}),
-				...(hasAttachment !== undefined ? { hasAttachment } : {}),
-				...(hasStars !== undefined ? { hasStars } : {}),
-				...(isRead !== undefined ? { isRead } : {}),
-				...(afterToken ? { sentDateFrom: afterToken.epochSeconds } : {}),
-				...(beforeToken ? { sentDateTo: beforeToken.epochSeconds } : {}),
-			},
-		}),
-		enabled,
-		staleTime: 60_000,
-	});
-
-	return { hits: data?.items ?? [], isLoading: enabled && isLoading };
-}

package/src/hooks/useSemanticSearchEnabled.ts

@@ -1,23 +0,0 @@
-import { configOperationsGetConfigOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import { useQuery } from "@tanstack/react-query";
-
-/**
- * Whether this instance stores message vectors (#1068).
- *
- * A property of the deployment, so it rides the config read every surface
- * already makes rather than a probe of its own: the semantic surfaces need the
- * answer before they have a query to send, and an empty semantic result is
- * indistinguishable from a mailbox with nothing similar in it.
- *
- * `undefined` while the config is in flight, and for a server that predates the
- * field. Callers must therefore branch on `=== false`, so an instance whose
- * answer has not arrived shows what it showed before rather than an off state
- * it may not be in.
- */
-export const useSemanticSearchEnabled = (): boolean | undefined => {
-	const { data } = useQuery({
-		...configOperationsGetConfigOptions(),
-		staleTime: Infinity,
-	});
-	return data?.semanticSearchEnabled;
-};

package/src/hooks/useVisualViewport.test.ts

@@ -1,126 +0,0 @@
-import assert from "node:assert";
-import { beforeEach, describe, test } from "node:test";
-
-/**
- * Unit test for useVisualViewport hook logic.
- * Tests the core behaviour without React.
- */
-
-const KEYBOARD_THRESHOLD = 150;
-
-interface ViewportState {
-	viewportHeight: number;
-	isKeyboardOpen: boolean;
-}
-
-/**
- * Simulates the viewport-tracking logic from useVisualViewport.
- */
-class VisualViewportSimulator {
-	private innerHeight: number;
-	private viewportHeight: number;
-	private listeners: Array<() => void> = [];
-
-	constructor(innerHeight: number) {
-		this.innerHeight = innerHeight;
-		this.viewportHeight = innerHeight;
-	}
-
-	getState(): ViewportState {
-		return {
-			viewportHeight: this.viewportHeight,
-			isKeyboardOpen:
-				this.innerHeight - this.viewportHeight > KEYBOARD_THRESHOLD,
-		};
-	}
-
-	/** Simulate a visualViewport resize (e.g. keyboard appearing). */
-	resize(newViewportHeight: number): ViewportState {
-		this.viewportHeight = newViewportHeight;
-		for (const fn of this.listeners) fn();
-		return this.getState();
-	}
-
-	addListener(fn: () => void): void {
-		this.listeners.push(fn);
-	}
-
-	removeListener(fn: () => void): void {
-		this.listeners = this.listeners.filter((l) => l !== fn);
-	}
-
-	get listenerCount(): number {
-		return this.listeners.length;
-	}
-}
-
-describe("useVisualViewport logic", () => {
-	let sim: VisualViewportSimulator;
-
-	beforeEach(() => {
-		// iPhone-like viewport: 844px tall
-		sim = new VisualViewportSimulator(844);
-	});
-
-	test("initial state has keyboard closed", () => {
-		const state = sim.getState();
-		assert.strictEqual(state.viewportHeight, 844);
-		assert.strictEqual(state.isKeyboardOpen, false);
-	});
-
-	test("keyboard open when viewport shrinks by more than threshold", () => {
-		// Typical iOS keyboard: viewport shrinks to ~400px
-		const state = sim.resize(400);
-		assert.strictEqual(state.isKeyboardOpen, true);
-		assert.strictEqual(state.viewportHeight, 400);
-	});
-
-	test("keyboard stays closed for small viewport changes", () => {
-		// Address bar collapsing — only ~50px change
-		const state = sim.resize(794);
-		assert.strictEqual(state.isKeyboardOpen, false);
-	});
-
-	test("keyboard closes when viewport restores", () => {
-		sim.resize(400);
-		const state = sim.resize(844);
-		assert.strictEqual(state.isKeyboardOpen, false);
-		assert.strictEqual(state.viewportHeight, 844);
-	});
-
-	test("exactly at threshold boundary — not open", () => {
-		// innerHeight - viewportHeight === 150, not > 150
-		const state = sim.resize(844 - KEYBOARD_THRESHOLD);
-		assert.strictEqual(state.isKeyboardOpen, false);
-	});
-
-	test("one pixel past threshold — keyboard open", () => {
-		const state = sim.resize(844 - KEYBOARD_THRESHOLD - 1);
-		assert.strictEqual(state.isKeyboardOpen, true);
-	});
-
-	test("listeners are called on resize", () => {
-		let callCount = 0;
-		sim.addListener(() => {
-			callCount++;
-		});
-		sim.resize(400);
-		sim.resize(844);
-		assert.strictEqual(callCount, 2);
-	});
-
-	test("removeListener stops notifications", () => {
-		let callCount = 0;
-		const fn = () => {
-			callCount++;
-		};
-		sim.addListener(fn);
-		sim.resize(400);
-		assert.strictEqual(callCount, 1);
-
-		sim.removeListener(fn);
-		sim.resize(844);
-		assert.strictEqual(callCount, 1);
-		assert.strictEqual(sim.listenerCount, 0);
-	});
-});