@remit/web-client 0.0.171 → 0.0.172

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/web-client",
-  "version": "0.0.171",
+  "version": "0.0.172",
   "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/compose/ComposeProvider.tsx

@@ -59,7 +59,12 @@ export const ComposeProvider = ({
 		enabled: !!pollingMessageId,
 		refetchInterval: (query) => {
 			const status = query.state.data?.status;
-			if (status === "sent" || status === "failed" || status === "blocked")
+			if (
+				status === "sent" ||
+				status === "unfiled" ||
+				status === "failed" ||
+				status === "blocked"
+			)
 				return false;
 			if (Date.now() - startedAtRef.current > MAX_POLL_DURATION_MS)
 				return false;
@@ -72,6 +77,7 @@ export const ComposeProvider = ({
 
 		const terminal =
 			polledMessage.status === "sent" ||
+			polledMessage.status === "unfiled" ||
 			polledMessage.status === "failed" ||
 			polledMessage.status === "blocked";
 

package/src/components/mail/OutboxPane.tsx

@@ -59,7 +59,7 @@ import {
 	matchesOutboxSearch,
 	outboxQueryIsUnsupported,
 } from "@/lib/outbox-search";
-import { isOutboxListRow, isUnsendableStatus } from "@/lib/outbox-status";
+import { isOutboxListRow, showsLastError } from "@/lib/outbox-status";
 import { normalizeSearchQuery } from "@/lib/search-query";
 import { parseSearchTokens } from "@/lib/search-tokens";
 import { useEditDraft, useRetainOpenPanels } from "@/routing";
@@ -87,6 +87,11 @@ const STATUS_CONFIG: Record<
 		label: "Sent",
 		className: "text-positive",
 	},
+	unfiled: {
+		icon: AlertTriangle,
+		label: "Sent, not filed",
+		className: "text-warning",
+	},
 	failed: {
 		icon: AlertCircle,
 		label: "Failed",
@@ -310,7 +315,7 @@ function OutboxMessageRow({
 	>;
 	if (!STATUS_CONFIG[status]) return null;
 
-	const showError = isUnsendableStatus(message.status);
+	const showError = showsLastError(message.status);
 
 	return (
 		<OutboxRow
@@ -414,7 +419,7 @@ function OutboxMessageDetail({ message, onBack }: OutboxMessageDetailProps) {
 							<span className={cn("text-xs font-medium", config.className)}>
 								{config.label}
 							</span>
-							{isUnsendableStatus(message.status) && message.lastError && (
+							{showsLastError(message.status) && message.lastError && (
 								<span className="text-xs text-fg-muted">
 									— {message.lastError}
 								</span>

package/src/lib/outbox-status.test.ts

@@ -3,7 +3,7 @@ import { describe, test } from "node:test";
 import {
 	describeOutboxStatus,
 	isOutboxListRow,
-	isUnsendableStatus,
+	showsLastError,
 } from "./outbox-status.js";
 
 describe("describeOutboxStatus", () => {
@@ -25,6 +25,13 @@ describe("describeOutboxStatus", () => {
 		assert.equal(desc.label, "Sent");
 	});
 
+	test("unfiled says the message was sent but not filed", () => {
+		const desc = describeOutboxStatus("unfiled");
+		assert.ok(desc);
+		assert.equal(desc.tone, "warning");
+		assert.match(desc.label, /not filed/);
+	});
+
 	test("failed is error tone (distinct from blocked)", () => {
 		const desc = describeOutboxStatus("failed");
 		assert.ok(desc);
@@ -33,19 +40,23 @@ describe("describeOutboxStatus", () => {
 	});
 });
 
-describe("isUnsendableStatus", () => {
-	test("sent must NOT be flagged as unsendable — never show error subtitle on success (issue #192)", () => {
-		assert.equal(isUnsendableStatus("sent"), false);
+describe("showsLastError", () => {
+	test("sent must NOT surface an error — never show an error subtitle on success (issue #192)", () => {
+		assert.equal(showsLastError("sent"), false);
 	});
 
-	test("failed and blocked are unsendable", () => {
-		assert.equal(isUnsendableStatus("failed"), true);
-		assert.equal(isUnsendableStatus("blocked"), true);
+	test("failed and blocked surface their error", () => {
+		assert.equal(showsLastError("failed"), true);
+		assert.equal(showsLastError("blocked"), true);
 	});
 
-	test("queued and sending are not unsendable", () => {
-		assert.equal(isUnsendableStatus("queued"), false);
-		assert.equal(isUnsendableStatus("sending"), false);
+	test("unfiled surfaces its reason — the row exists only to carry it", () => {
+		assert.equal(showsLastError("unfiled"), true);
+	});
+
+	test("queued and sending carry no error", () => {
+		assert.equal(showsLastError("queued"), false);
+		assert.equal(showsLastError("sending"), false);
 	});
 });
 
@@ -61,4 +72,8 @@ describe("isOutboxListRow", () => {
 		assert.equal(isOutboxListRow("failed"), true);
 		assert.equal(isOutboxListRow("blocked"), true);
 	});
+
+	test("keeps an unfiled row listed — it is the only copy the user has", () => {
+		assert.equal(isOutboxListRow("unfiled"), true);
+	});
 });

package/src/lib/outbox-status.ts

@@ -29,6 +29,7 @@ const STATUS_DESCRIPTORS: Record<OutboxDisplayStatus, OutboxStatusDescriptor> =
 		queued: { label: "Queued", tone: "neutral" },
 		sending: { label: "Sending…", tone: "info" },
 		sent: { label: "Sent", tone: "success" },
+		unfiled: { label: "Sent, not filed", tone: "warning" },
 		failed: { label: "Failed", tone: "error" },
 		blocked: { label: "Blocked", tone: "warning" },
 	};
@@ -56,11 +57,12 @@ export const describeOutboxStatus = (
  *
  * Critically: `sent` is NOT included. A successful send must never display
  * a stale lastError — that is the contradictory "green check + 'SMTP not
- * configured'" UI bug from issue #192.
+ * configured'" UI bug from issue #192. `unfiled` is: the message was
+ * delivered but could not be filed, and the reason is the whole point of
+ * the row still being there.
  */
-export const isUnsendableStatus = (
-	status: RemitImapOutboxMessageStatus,
-): boolean => status === "failed" || status === "blocked";
+export const showsLastError = (status: RemitImapOutboxMessageStatus): boolean =>
+	status === "failed" || status === "blocked" || status === "unfiled";
 
 /**
  * Whether the outbox list view should render this row.
@@ -68,7 +70,9 @@ export const isUnsendableStatus = (
  * `draft` belongs in the Drafts view. `sent` rows are deleted by the IMAP
  * APPEND handler (issue #178) — but until that happens we hide them from
  * the Outbox list so a successfully-sent message never appears in two
- * places at once (issue #193).
+ * places at once (issue #193). `unfiled` is the settled failure of that
+ * APPEND: the message is on no server folder, so the Outbox row is the only
+ * copy the user has and it stays listed.
  */
 export const isOutboxListRow = (
 	status: RemitImapOutboxMessageStatus,