dashboard, inbound messages: support deferred message status

Author: Seryiza

src/ui/pages/dashboard.ts

@@ -32,7 +32,9 @@ interface TickerRow {
   time: string;
   type: string;
   note: string;
-  status: "ok" | "warn" | "err" | "pend";
+  // `held` = manually deferred. Terminal status — distinct from `pend`
+  // (worker hasn't picked up yet) so the chip doesn't read as "pending".
+  status: "ok" | "warn" | "err" | "pend" | "held";
 }
 
 const HARD_ERROR_STATUSES = [
@@ -121,6 +123,7 @@ function toTickerStatus(
   if (!status) {return "pend";}
   if (status === "processed" || status === "warning") {return "ok";}
   if (status === "code_mapping_error") {return "warn";}
+  if (status === "deferred") {return "held";}
   if (status.endsWith("_error")) {return "err";}
   return "pend";
 }
@@ -385,6 +388,7 @@ function statusChip(status: TickerRow["status"]): string {
   if (status === "ok") {return `<span class="chip chip-ok">processed</span>`;}
   if (status === "warn") {return `<span class="chip chip-warn">needs mapping</span>`;}
   if (status === "err") {return `<span class="chip chip-err">error</span>`;}
+  if (status === "held") {return `<span class="chip">deferred</span>`;}
   return `<span class="chip">pending</span>`;
 }
 

src/ui/pages/inbound-detail.ts

@@ -67,6 +67,7 @@ function toneChip(tone: MessageTone): string {
   if (tone === "ok") {return `<span class="chip chip-ok">processed</span>`;}
   if (tone === "warn") {return `<span class="chip chip-warn">needs mapping</span>`;}
   if (tone === "err") {return `<span class="chip chip-err">error</span>`;}
+  if (tone === "held") {return `<span class="chip">deferred</span>`;}
   return `<span class="chip">pending</span>`;
 }
 

src/ui/pages/inbound.ts

@@ -53,7 +53,11 @@ interface TypeChipCount {
   tone?: "accent" | "err";
 }
 
-export type MessageTone = "ok" | "warn" | "err" | "pend";
+// `held` = manually deferred via POST /defer/:id. Distinct from `pend`
+// (received, worker hasn't processed yet — animated, terminal-bound) so the
+// chip doesn't show a misleading "processing" spinner on a status that won't
+// move on its own. Operator must POST /mark-for-retry/:id to resume.
+export type MessageTone = "ok" | "warn" | "err" | "pend" | "held";
 
 // The 4 hard-error statuses the design's "errors" pseudo-chip aggregates.
 // Matches CLAUDE.md's IncomingHL7v2Message status vocabulary.
@@ -216,8 +220,9 @@ export function statusToTone(p: ParsedIncomingMessage): MessageTone {
     case "conversion_error":
     case "sending_error":
       return "err";
-    case "received":
     case "deferred":
+      return "held";
+    case "received":
       return "pend";
     default:
       return assertNever(p);
@@ -235,6 +240,7 @@ export function statusStringToTone(s: string | undefined): MessageTone {
   if (!s) {return "pend";}
   if (s === "processed" || s === "warning") {return "ok";}
   if (s === "code_mapping_error") {return "warn";}
+  if (s === "deferred") {return "held";}
   if (s.endsWith("_error")) {return "err";}
   return "pend";
 }
@@ -243,6 +249,10 @@ function toneDot(tone: MessageTone): string {
   if (tone === "ok") {return "dot ok";}
   if (tone === "warn") {return "dot warn";}
   if (tone === "err") {return "dot err";}
+  // `held` (manually deferred): static neutral dot — terminal state, no
+  // background work in progress. Distinct from `pend` so the row doesn't
+  // pulse like an in-flight message.
+  if (tone === "held") {return "dot";}
   // `pend` (received / queued-for-worker): animated pulse so the user
   // sees the message is actively being worked on, not stuck. The
   // in-process worker polls every ~5s so the flip to "processed" is
@@ -254,6 +264,8 @@ function toneChip(tone: MessageTone): string {
   if (tone === "ok") {return `<span class="chip chip-ok">processed</span>`;}
   if (tone === "warn") {return `<span class="chip chip-warn">needs mapping</span>`;}
   if (tone === "err") {return `<span class="chip chip-err">error</span>`;}
+  // Held = neutral static chip. Terminal — no spinner, no auto-refresh.
+  if (tone === "held") {return `<span class="chip">deferred</span>`;}
   // Active-state chip: inline spinner + "processing" text so the user
   // sees activity rather than a static "pending" label. Per-row polling
   // (see renderRowStatusCell) swaps this into its settled form when the

test/unit/ui/inbound.test.ts

@@ -126,9 +126,11 @@ describe("statusToTone", () => {
     expect(statusToTone(parsed({ status: "conversion_error", error: "x" }))).toBe("err");
     expect(statusToTone(parsed({ status: "sending_error", error: "x" }))).toBe("err");
   });
-  test("maps received/deferred to pend", () => {
+  test("maps received to pend, deferred to held", () => {
+    // Distinct tones: `received` is animated (worker hasn't picked up yet);
+    // `held` is static (operator deferred — terminal until POST /mark-for-retry).
     expect(statusToTone(parsed({ status: "received" }))).toBe("pend");
-    expect(statusToTone(parsed({ status: "deferred" }))).toBe("pend");
+    expect(statusToTone(parsed({ status: "deferred" }))).toBe("held");
   });
 });