@ai-matrx/records-ui 0.42.0 → 0.44.0

package/CHANGELOG.md

@@ -1,5 +1,59 @@
 # Changelog — @ai-matrx/records-ui
 
+## 0.44.0
+
+**Pressing Open in the inbox landed you on the table with the record still shut.**
+
+`ActionInbox` has always handed its host `onOpenRecord(recordId, tableId)`, and
+the app has always routed that to `/data-v2/<table>?record=<record>` — and
+NOTHING read the parameter. So an approval, an assignment, an agent's proposal
+and (since 0.41.0) a checklist step all offered an Open that did not open
+anything. Measured in the real app on a dental practice's onboarding steps.
+
+`TablePage` takes `activeRecordId` and opens the record rail on it, moving the
+rail when a later link names a different record. Absent, the page opens exactly
+as it did. `openingRail` is the exported seam and `src/record-link.test.tsx`
+fails on 0.43.0's bytes.
+
+**Consumer action:** pass `activeRecordId` from your route's `?record=`
+parameter, beside the `activeDashboardId` you already pass. Without it the Open
+control in the inbox still goes nowhere.
+
+## 0.43.0
+
+**Five headings carrying the store's real counts, over five empty columns.**
+
+Measured in the real app (lane SCREENS-2, 2026-09-20) on a dental practice's
+treatment plans, immediately after 0.42.0 made the board reachable at all: it
+drew *Proposed 4*, *Insurance review 3*, *Scheduled 2 / 2* — and every column was
+empty. That is the worst thing a screen can do. A heading that says four over a
+column that shows none, with nothing anywhere saying the two disagree.
+
+**A real disagreement about what a cell holds.** `PipelineStage.key` is documented
+as "what a cell holds" and `custom.pipeline_board` keys its columns that way
+(`stage_key: "proposed"`). `custom.read_records` answers the LABEL in the cell and
+puts the key BESIDE it:
+
+```json
+{"stage": "Proposed", "_choices": {"stage": {"key": "proposed", "label": "Proposed"}}}
+```
+
+so `cardsByStage.get(stage.key)` matched nothing, for every row, on every pipeline.
+
+`stageOfCard` is the one seam that decides it and it asks the ROW's own resolved
+choice first, never the wording. Two fallbacks follow, because a cell may
+legitimately hold the key (an agent wrote it) or only the label (an import did),
+and neither is a reason to lose a card.
+
+**And a card with nowhere to go is now SAID.** A retired stage, or a value an
+import wrote that is not one of the board's choices, used to vanish — a board
+quietly one card short is the same lie as the heading that disagreed with its
+column. The board names them and says how to move one.
+
+`src/pipeline-cards.test.tsx` — five clauses, all red on 0.42.0's bytes.
+
+**Consumer action:** none.
+
 ## 0.42.0
 
 **A pipeline table opened as "pick a column", and the board it already had was

package/dist/index.cjs

@@ -5029,6 +5029,27 @@ var import_react38 = require("react");
 var import_react39 = require("@ai-matrx/records/react");
 var import_design_system18 = require("@ai-matrx/design-system");
 var import_jsx_runtime20 = require("react/jsx-runtime");
+var UNPLACED = "\0unplaced";
+function stageOfCard(row, stageKey, stages) {
+  const document2 = row.document ?? {};
+  const choices = document2["_choices"];
+  if (choices && typeof choices === "object") {
+    const chosen = choices[stageKey];
+    if (chosen && typeof chosen === "object") {
+      const key = chosen["key"];
+      if (typeof key === "string" && key !== "") {
+        return stages.some((s) => s.key === key) ? key : null;
+      }
+    }
+  }
+  const raw = document2[stageKey];
+  if (raw === null || raw === void 0 || raw === "") return null;
+  const said = String(raw).trim();
+  const byKey = stages.find((s) => s.key === said);
+  if (byKey) return byKey.key;
+  const byLabel = stages.find((s) => s.label.toLowerCase() === said.toLowerCase());
+  return byLabel ? byLabel.key : null;
+}
 function PipelineBoard({
   tableId,
   rows,
@@ -5078,12 +5099,12 @@ function PipelineBoard({
     const map = /* @__PURE__ */ new Map();
     if (!stageKey) return map;
     for (const row of rows) {
-      const raw = (row.document ?? {})[stageKey];
-      const key = raw === null || raw === void 0 || raw === "" ? "" : String(raw);
-      map.set(key, [...map.get(key) ?? [], row]);
+      const key = stageOfCard(row, stageKey, definition?.stages ?? []);
+      map.set(key ?? UNPLACED, [...map.get(key ?? UNPLACED) ?? [], row]);
     }
     return map;
-  }, [rows, stageKey]);
+  }, [rows, stageKey, definition?.stages]);
+  const unplaced = cardsByStage.get(UNPLACED) ?? [];
   const all = fields.data ?? [];
   const titleField = (0, import_react38.useMemo)(
     () => all.find((f) => f.key !== stageKey) ?? all[0],
@@ -5230,7 +5251,14 @@ function PipelineBoard({
         },
         stage.key
       );
-    }) })
+    }) }),
+    unplaced.length > 0 ? /* @__PURE__ */ (0, import_jsx_runtime20.jsxs)("p", { className: "text-xs text-muted-foreground", "data-testid": "pipeline-unplaced", children: [
+      unplaced.length === 1 ? "One record is" : `${unplaced.length} records are`,
+      " not in any of these columns \u2014 ",
+      unplaced.slice(0, 3).map((r) => rowName(r, titleField?.key ?? null, "Untitled")).join(", "),
+      unplaced.length > 3 ? ` and ${unplaced.length - 3} more` : "",
+      ". Their stage is not one this pipeline offers, so nothing here can draw them; open one from the grid to move it."
+    ] }) : null
   ] });
 }
 function formatTotal(total) {
@@ -11873,6 +11901,9 @@ var VIEW_NOT_SAVED_YET = "This table has no saved view yet, so the layout you pi
 function defaultView(tableId) {
   return { name: DEFAULT_VIEW_NAME, subject: tableId, layout: "grid", isDefault: true };
 }
+function openingRail(activeRecordId) {
+  return activeRecordId ? { rail: "record", record: activeRecordId } : { rail: "none", record: null };
+}
 function TablePage({
   tableId,
   pageSize = 100,
@@ -11880,6 +11911,7 @@ function TablePage({
   onLeave,
   leaveLabel = "Back to your tables",
   activeDashboardId,
+  activeRecordId,
   className
 }) {
   const client = (0, import_react98.useRecordsClient)();
@@ -11887,12 +11919,18 @@ function TablePage({
   const rights = useTableRights(table.data);
   const organizationId = (0, import_react98.useRecordsClient)().config.organizationId;
   const [view, setView] = (0, import_react97.useState)(null);
-  const [rail, setRail] = (0, import_react97.useState)("none");
+  const opening = openingRail(activeRecordId);
+  const [rail, setRail] = (0, import_react97.useState)(opening.rail);
   const [asking, setAsking] = (0, import_react97.useState)(null);
   const [main, setMain] = (0, import_react97.useState)(activeDashboardId ? "dashboards" : "records");
-  const [openRecord, setOpenRecord] = (0, import_react97.useState)(null);
+  const [openRecord, setOpenRecord] = (0, import_react97.useState)(opening.record);
   const viewVersion = useRecordVersion(view?.id ?? null);
   const show = (next) => setRail((current) => current === next ? "none" : next);
+  (0, import_react97.useEffect)(() => {
+    if (!activeRecordId) return;
+    setOpenRecord(activeRecordId);
+    setRail("record");
+  }, [activeRecordId]);
   const patchView = (0, import_react97.useCallback)(
     async (patch) => {
       const current = view;