@remit/web-client 0.0.24 → 0.0.26

package/src/lib/escalation-label.test.ts

@@ -0,0 +1,49 @@
+import assert from "node:assert/strict";
+import { describe, test } from "node:test";
+import {
+	describeSearchScope,
+	escalatedStatusLabel,
+	escalationActionLabel,
+} from "./escalation-label.js";
+
+describe("describeSearchScope", () => {
+	test("free text query wins and is quoted", () => {
+		assert.equal(describeSearchScope({ query: "npm" }), 'matching "npm"');
+	});
+
+	test("falls back to a from: filter when there's no free text", () => {
+		assert.equal(
+			describeSearchScope({ from: "billing@example.com" }),
+			'from "billing@example.com"',
+		);
+	});
+
+	test("free text wins over a from: filter when both are set", () => {
+		assert.equal(
+			describeSearchScope({ query: "npm", from: "noreply@npmjs.com" }),
+			'matching "npm"',
+		);
+	});
+
+	test("neither present falls back to a generic scope rather than throwing", () => {
+		assert.equal(describeSearchScope({}), "matching your search");
+	});
+});
+
+describe("escalationActionLabel", () => {
+	test("names the scope, never a bare Select all", () => {
+		assert.equal(
+			escalationActionLabel({ query: "npm" }),
+			'Select all matching "npm"',
+		);
+	});
+});
+
+describe("escalatedStatusLabel", () => {
+	test("thousands-separates the total and names the scope", () => {
+		assert.equal(
+			escalatedStatusLabel({ query: "npm" }, 3412),
+			'All 3,412 matching "npm" selected',
+		);
+	});
+});

package/src/lib/escalation-label.ts

@@ -0,0 +1,32 @@
+import { formatNumber } from "./format";
+
+/** The subset of the search predicate that names the scope in escalation copy. */
+export interface EscalationScopeQuery {
+	query?: string;
+	from?: string;
+}
+
+/**
+ * Names what an escalated selection covers, for the notice text and the
+ * confirm dialog — never a bare "Select all" (issue #92 requirement 4). Free
+ * text wins when present since it's what the user actually typed; `from:`
+ * alone still names a scope. Neither present is unreachable in practice (the
+ * escalation control only ever renders for an active search), but the
+ * fallback keeps the label honest instead of throwing.
+ */
+export const describeSearchScope = (query: EscalationScopeQuery): string => {
+	if (query.query) return `matching "${query.query}"`;
+	if (query.from) return `from "${query.from}"`;
+	return "matching your search";
+};
+
+/** "Select all matching "npm"" — the escalation notice's action label. */
+export const escalationActionLabel = (query: EscalationScopeQuery): string =>
+	`Select all ${describeSearchScope(query)}`;
+
+/** "All 3,412 matching "npm" selected" — the bar's status label once escalated. */
+export const escalatedStatusLabel = (
+	query: EscalationScopeQuery,
+	total: number,
+): string =>
+	`All ${formatNumber(total)} ${describeSearchScope(query)} selected`;

package/src/lib/format.test.ts

@@ -112,4 +112,11 @@ describe("formatDeleteToTrashTitle", () => {
 			"Move 0 messages to Trash?",
 		);
 	});
+
+	test("thousands-separates an escalated-selection-scale count", () => {
+		assert.strictEqual(
+			formatDeleteToTrashTitle(3412),
+			"Move 3,412 messages to Trash?",
+		);
+	});
 });

package/src/lib/format.ts

@@ -193,6 +193,10 @@ export const formatList = (
 /**
  * Confirmation title for the move-to-Trash delete flow. Reflects that delete
  * moves messages to Trash (not a permanent delete) and pluralizes on count.
+ * Thousands-separated: at escalated-selection scale this is exactly the digit
+ * count someone checks against what they meant to select.
  */
 export const formatDeleteToTrashTitle = (count: number): string =>
-	count === 1 ? "Move 1 message to Trash?" : `Move ${count} messages to Trash?`;
+	count === 1
+		? "Move 1 message to Trash?"
+		: `Move ${formatNumber(count)} messages to Trash?`;