@remit/web-client 0.0.284 → 0.0.286

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/web-client",
-  "version": "0.0.284",
+  "version": "0.0.286",
   "type": "module",
   "description": "Remit web client, published as composable primitives — the app shell, auth shells, and runtime config. A distributor imports what it composes and bundles it.",
   "exports": {

package/src/components/mail/RoleAppointmentPromptProvider.render.test.ts

@@ -214,7 +214,9 @@ describe("the appointment ceremony", () => {
 					message: "not settled",
 					details: { mailboxId: "mbx-trash", syncStatus: "pending" },
 				}),
-				{ status: 409, headers: { "content-type": "application/json" } },
+				// 422: the folder being appointed has not settled, which is a
+				// different answer from a 409's "somebody else is changing it".
+				{ status: 422, headers: { "content-type": "application/json" } },
 			);
 		});
 

package/src/hooks/escalated-run-refusal.render.test.ts

@@ -165,11 +165,14 @@ describe("a bulk delete refused for its folder role", () => {
 		await mount((path) => {
 			if (path.endsWith("/config")) return CONFIG;
 			if (path.endsWith("/messages/delete"))
+				// A conflict with no code this prompt knows: the status class's own,
+				// which is what every refusal that names nothing more specific
+				// answers with.
 				return new Response(
 					JSON.stringify({
 						status: 409,
 						message: "boom",
-						code: "mailbox_not_settled",
+						code: "conflict",
 					}),
 					{ status: 409, headers: { "content-type": "application/json" } },
 				);

package/src/hooks/useRulePreview.ts

@@ -78,6 +78,7 @@ export const useRulePreview = (
 					setState({
 						count: response.matchedCount,
 						matchedIds: response.messageIds,
+						indexEmpty: response.semanticIndexEmpty === true,
 						previewedSignature: signature,
 					});
 				})

package/src/lib/organize/rule-model.test.ts

@@ -276,6 +276,36 @@ describe("derivePreview", () => {
 		);
 	});
 
+	it("carries the empty-index answer through, so a zero can say why (#452)", () => {
+		assert.deepEqual(
+			derivePreview(
+				{ count: 0, indexEmpty: true, previewedSignature: signature },
+				signature,
+			),
+			{ status: "ready", count: 0, indexEmpty: true },
+		);
+	});
+
+	it("keeps the empty-index answer on a stale count", () => {
+		assert.deepEqual(
+			derivePreview(
+				{ count: 0, indexEmpty: true, previewedSignature: "sig-old" },
+				signature,
+			),
+			{ status: "ready", count: 0, stale: true, indexEmpty: true },
+		);
+	});
+
+	it("leaves the flag off when the index answered", () => {
+		assert.deepEqual(
+			derivePreview(
+				{ count: 0, indexEmpty: false, previewedSignature: signature },
+				signature,
+			),
+			{ status: "ready", count: 0 },
+		);
+	});
+
 	it("surfaces an error raised for the current predicate", () => {
 		assert.deepEqual(
 			derivePreview(

package/src/lib/organize/rule-model.ts

@@ -253,6 +253,8 @@ export interface PreviewState {
 	count?: number;
 	/** The ids that count was counted over, as the server bounded them. */
 	matchedIds?: readonly string[];
+	/** The server said the semantic index had nothing to answer with (#452). */
+	indexEmpty?: boolean;
 	/** The signature the {@link count} was counted for. */
 	previewedSignature?: string;
 	/** The signature the last error was raised for. */
@@ -274,10 +276,11 @@ export const derivePreview = (
 		return { status: "error", reason: state.error };
 	}
 	if (state.count === undefined) return { status: "loading" };
+	const indexEmpty = state.indexEmpty === true ? { indexEmpty: true } : {};
 	if (state.previewedSignature === currentSignature) {
-		return { status: "ready", count: state.count };
+		return { status: "ready", count: state.count, ...indexEmpty };
 	}
-	return { status: "ready", count: state.count, stale: true };
+	return { status: "ready", count: state.count, stale: true, ...indexEmpty };
 };
 
 /** How long a rule change settles before the next preview fires. */