@remit/web-client 0.0.321 → 0.0.323

package/src/lib/organize/sender-fallback.test.ts

@@ -1,52 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import { buildSenderFallbackDraft } from "./sender-fallback";
-
-describe("buildSenderFallbackDraft", () => {
-	it("combines the sender clauses with Or and carries no anchor", () => {
-		const draft = buildSenderFallbackDraft(
-			["npm@github.com", "a@x.com"],
-			["Deploy failed", "Standup notes"],
-		);
-		assert.equal(draft.matchOperator, "Or");
-		assert.equal(draft.anchorMessageId, undefined);
-		assert.deepEqual(draft.literalClauses, [
-			{ field: "From", value: "npm@github.com" },
-			{ field: "From", value: "a@x.com" },
-		]);
-	});
-
-	it("carries a single FromDomain clause when the senders share a domain", () => {
-		const draft = buildSenderFallbackDraft(
-			["npm@github.com", "ci@github.com"],
-			["Deploy failed", "Standup notes"],
-		);
-		assert.equal(draft.matchOperator, "Or");
-		assert.deepEqual(draft.literalClauses, [
-			{ field: "FromDomain", value: "github.com" },
-		]);
-	});
-
-	it("falls back to a shared subject when mixed senders share no domain (#458)", () => {
-		const draft = buildSenderFallbackDraft(
-			["billing@acme.test", "accounts@globex.test"],
-			["Invoice 1841", "Invoice 1902"],
-		);
-		assert.equal(draft.matchOperator, "Or");
-		assert.deepEqual(draft.literalClauses, [
-			{ field: "Subject", value: "Invoice" },
-		]);
-	});
-
-	it("offers one sender chip each when mixed senders share no subject either", () => {
-		const draft = buildSenderFallbackDraft(
-			["billing@acme.test", "accounts@globex.test"],
-			["Invoice 1841", "Standup notes"],
-		);
-		assert.equal(draft.matchOperator, "Or");
-		assert.deepEqual(draft.literalClauses, [
-			{ field: "From", value: "billing@acme.test" },
-			{ field: "From", value: "accounts@globex.test" },
-		]);
-	});
-});

package/src/lib/organize/sender-fallback.ts

@@ -1,38 +0,0 @@
-import type { RemitImapOrganizeInput } from "@remit/api-http-client/types.gen.ts";
-import { derivePropertyClauses } from "@remit/ui";
-import type { OrganizeDraft } from "./organize-model";
-
-/**
- * The widen fallback for a deployment that ships no vector pipeline (self-host
- * sqlite — semantic-capability.ts). The semantic anchor matches nothing there,
- * so a widen degrades to the literal vocabulary RFC 031 already matches
- * vector-free: what the whole selection agrees on, combined with `Or`, no
- * anchor (#458) — the same evidence order the properties door prefills from,
- * so a selection with nothing in common on sender still falls back to a
- * subject it shares rather than one `From` chip per message. The same
- * predicate matches at index time (RFC 034), so a standing filter built from
- * it keeps working on future mail.
- */
-
-/**
- * The literal predicate that stands in for the semantic anchor: the clauses
- * the selection agrees on, combined with `Or` and no anchor. The preview, the
- * one-time back-apply, and the standing filter all carry exactly this.
- */
-export const buildSenderFallbackDraft = (
-	senders: readonly string[],
-	subjects: readonly string[],
-): OrganizeDraft => ({
-	matchOperator: "Or",
-	literalClauses: derivePropertyClauses(senders, subjects),
-});
-
-/**
- * The predicate the widen previewed, handed to the organize sentence so the set
- * it previews equals the set every commit scope acts on. Either the semantic
- * anchor or the sender-derived literal fallback, never both.
- */
-export type OrganizeMatchPredicate = Pick<
-	RemitImapOrganizeInput,
-	"anchorMessageId" | "matchOperator" | "literalClauses"
->;

package/src/lib/rescue-telemetry.test.ts

@@ -1,75 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import type { Telemetry } from "@/lib/telemetry";
-import {
-	recordRescueCandidatesSurfaced,
-	recordRescueCommitted,
-	recordRescueFlowOpened,
-	recordRescueSentToJunk,
-} from "./rescue-telemetry";
-
-interface RecordedEvent {
-	name: string;
-	attributes?: Record<string, string>;
-}
-
-function stubTelemetry(): { telemetry: Telemetry; events: RecordedEvent[] } {
-	const events: RecordedEvent[] = [];
-	const telemetry: Telemetry = {
-		recordPageView: () => undefined,
-		recordError: () => undefined,
-		recordTiming: () => undefined,
-		recordEvent: (name, attributes) => events.push({ name, attributes }),
-	};
-	return { telemetry, events };
-}
-
-describe("rescue telemetry", () => {
-	it("emits candidates surfaced with the count", () => {
-		const { telemetry, events } = stubTelemetry();
-		recordRescueCandidatesSurfaced(telemetry, 7);
-		assert.deepEqual(events, [
-			{ name: "rescue.candidates_surfaced", attributes: { count: "7" } },
-		]);
-	});
-
-	it("emits flow opened with the count", () => {
-		const { telemetry, events } = stubTelemetry();
-		recordRescueFlowOpened(telemetry, 3);
-		assert.deepEqual(events, [
-			{ name: "rescue.flow_opened", attributes: { count: "3" } },
-		]);
-	});
-
-	it("emits committed with selected vs total and an inbox destination", () => {
-		const { telemetry, events } = stubTelemetry();
-		recordRescueCommitted(telemetry, { selected: 2, total: 5, toInbox: true });
-		assert.deepEqual(events, [
-			{
-				name: "rescue.committed",
-				attributes: { selected: "2", total: "5", destination: "inbox" },
-			},
-		]);
-	});
-
-	it("marks a non-inbox destination as other", () => {
-		const { telemetry, events } = stubTelemetry();
-		recordRescueCommitted(telemetry, { selected: 1, total: 1, toInbox: false });
-		assert.equal(events[0]?.attributes?.destination, "other");
-	});
-
-	it("emits the reverse signal with trust and rescuable flag", () => {
-		const { telemetry, events } = stubTelemetry();
-		recordRescueSentToJunk(telemetry, {
-			count: 1,
-			senderTrust: "vip",
-			wasRescuable: true,
-		});
-		assert.deepEqual(events, [
-			{
-				name: "rescue.sent_to_junk",
-				attributes: { count: "1", senderTrust: "vip", wasRescuable: "true" },
-			},
-		]);
-	});
-});

package/src/router.test.ts

@@ -1,81 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import type { Telemetry } from "./lib/telemetry";
-
-interface SubscribeCall {
-	eventType: string;
-	fn: (event: { toLocation: { pathname: string } }) => void;
-}
-
-interface RouterStub {
-	subscribeCalls: SubscribeCall[];
-	subscribe(
-		eventType: string,
-		fn: (event: { toLocation: { pathname: string } }) => void,
-	): () => void;
-}
-
-function makeRouterStub(): RouterStub {
-	const stub: RouterStub = {
-		subscribeCalls: [],
-		subscribe(eventType, fn) {
-			stub.subscribeCalls.push({ eventType, fn });
-			return () => undefined;
-		},
-	};
-	return stub;
-}
-
-function makeTelemetryStub(): Telemetry & { pageViews: string[] } {
-	const stub = {
-		pageViews: [] as string[],
-		recordPageView(path: string) {
-			stub.pageViews.push(path);
-		},
-		recordError: () => undefined,
-		recordEvent: () => undefined,
-		recordTiming: () => undefined,
-	};
-	return stub;
-}
-
-function wirePageViews(router: RouterStub, telemetry: Telemetry): void {
-	router.subscribe("onResolved", (event) => {
-		telemetry.recordPageView(event.toLocation.pathname);
-	});
-}
-
-describe("router page-view telemetry wiring", () => {
-	it("subscribes to onResolved", () => {
-		const router = makeRouterStub();
-		const telemetry = makeTelemetryStub();
-		wirePageViews(router, telemetry);
-		assert.equal(router.subscribeCalls.length, 1);
-		assert.equal(router.subscribeCalls[0]?.eventType, "onResolved");
-	});
-
-	it("calls recordPageView with pathname on each navigation", () => {
-		const router = makeRouterStub();
-		const telemetry = makeTelemetryStub();
-		wirePageViews(router, telemetry);
-
-		const handler = router.subscribeCalls[0]?.fn;
-		assert.ok(handler, "Expected a subscribe handler");
-
-		handler({ toLocation: { pathname: "/mail" } });
-		handler({ toLocation: { pathname: "/settings" } });
-
-		assert.deepEqual(telemetry.pageViews, ["/mail", "/settings"]);
-	});
-
-	it("is a no-op when telemetry is noopTelemetry (does not throw)", () => {
-		const router = makeRouterStub();
-		const telemetry = makeTelemetryStub();
-		telemetry.recordPageView = () => undefined;
-		wirePageViews(router, telemetry);
-
-		const handler = router.subscribeCalls[0]?.fn;
-		assert.ok(handler);
-		assert.doesNotThrow(() => handler({ toLocation: { pathname: "/mail" } }));
-	});
-});