@ai-matrx/records-ui 0.69.0 → 0.70.0

package/CHANGELOG.md

@@ -1,5 +1,32 @@
 # Changelog — @ai-matrx/records-ui
 
+## 0.70.0
+
+**The bar that said four now opens four.** (lane DRILL.)
+
+* **`TablePage` takes a `filter`** — the store's one `RecordFilter` — and hands it to the grid
+  AND the board, so switching layout after a drill-through does not quietly widen back to the
+  whole table. It is a question for this visit, like `activeView` and `activeGroupField`, and
+  is never written onto the saved view. `Grid` and `ViewSwitcher` (and `useViewRecords`) take
+  it too; the narrowing happens in the STORE, never in the browser, because this screen holds
+  one page and a browser-side filter would drop every match on page two.
+* **`cameFromLine` says the narrow thing, and means it.** TAILS-6 shipped *"This is every
+  record grouped into the same columns the chart used — not only the ones that number
+  counted"*, which was TRUE and useless: no door took a filter. It now reads *"These are only
+  the records that number counted: Status is Cancelled. Everything else in this table is being
+  held back"*, and it still refuses to claim narrowing when no question came with the click.
+* **`filterInWords`** turns the question into the words of the person whose business it is —
+  the Field's own name through the same `fieldName` the column headers use, never a column
+  key. `null` becomes "Status is not filled in"; a window becomes "from … up to …", because
+  the store's windows are half-open and "to 2026-02-01" would read as including February 1st.
+* **`DigestScheduler.filters` is a `RecordFilter`**, and `DashboardCanvas`'s
+  `as Record<string, unknown>` is gone — that cast was the seam where the dashboard's filter,
+  the saved view it schedules over and the drill-through it opens could have drifted apart.
+* **Guard `a-number-opens-only-its-own-rows.test.tsx`** asserts the DOOR the page opened and
+  the argument it carried, and refuses by name the moment anything opens the unfiltered
+  `read_records` while holding a filter. Shown RED on the pre-fix bytes (four clauses,
+  including the old sentence) and green on these.
+
 ## 0.69.0
 
 **She can make the one that is missing, from where she is looking for it.** (lane TAILS-7.)

package/dist/index.cjs

@@ -3557,6 +3557,7 @@ var NO_COLUMNS_WHY = "A column is what a record holds \u2014 a name, a date, an
 function Grid({
   tableId,
   pageSize = 50,
+  filter,
   onOpenRecord,
   onAddField,
   onNewRecordForm,
@@ -3572,7 +3573,7 @@ function Grid({
   const table = (0, import_react21.useTable)(tableId);
   const fields = (0, import_react21.useFields)(tableId);
   const [page, setPage] = (0, import_react20.useState)(0);
-  const records = (0, import_react21.useRecords)(tableId, { pageSize, page });
+  const records = (0, import_react21.useRecords)(tableId, { pageSize, page, ...filter ? { filter } : {} });
   const rights = useTableRights(table.data);
   const rows = (0, import_react20.useMemo)(() => records.data?.rows ?? [], [records.data]);
   const rowIds = (0, import_react20.useMemo)(() => rows.map((row) => row.id), [rows]);
@@ -7321,9 +7322,12 @@ function unplacedName(record, field, labels) {
 // src/ViewSwitcher.tsx
 var import_jsx_runtime26 = require("react/jsx-runtime");
 var NO_FIELDS = [];
-function useViewRecords(view, pageSize = 200) {
+function useViewRecords(view, pageSize = 200, filter) {
   const client = (0, import_react49.useRecordsClient)();
-  const table = (0, import_react49.useRecords)(view.ruleId ? null : view.subject, { pageSize });
+  const table = (0, import_react49.useRecords)(view.ruleId ? null : view.subject, {
+    pageSize,
+    ...filter ? { filter } : {}
+  });
   const [ruled, setRuled] = (0, import_react48.useState)({
     rows: [],
     loading: Boolean(view.ruleId),
@@ -7378,6 +7382,7 @@ function ViewSwitcher({
   onNewRecordForm,
   onAskWhoChanged,
   pageSize = 200,
+  filter,
   className
 }) {
   const [layout, setLayout] = (0, import_react48.useState)(view.layout);
@@ -7415,6 +7420,7 @@ function ViewSwitcher({
       {
         tableId: view.subject,
         pageSize,
+        ...filter ? { filter } : {},
         ...local.presentation ?? view.presentation ? { presentation: local.presentation ?? view.presentation } : {},
         ...onViewChange ? { onPresentationChange: (next) => change({ presentation: next }) } : {},
         ...onOpenRecord ? { onOpenRecord } : {},
@@ -7426,6 +7432,7 @@ function ViewSwitcher({
       {
         view: { ...view, ...local, layout },
         pageSize,
+        ...filter ? { filter } : {},
         onChange: change,
         remembered: Boolean(onViewChange),
         ...onOpenRecord ? { onOpenRecord } : {},
@@ -7473,13 +7480,14 @@ function boardColumnChoice(fields, chosenKey, stage) {
 function Board({
   view,
   pageSize,
+  filter,
   onChange,
   remembered,
   onOpenRecord,
   onMoved
 }) {
   const fields = (0, import_react49.useFields)(view.subject);
-  const records = useViewRecords(view, pageSize);
+  const records = useViewRecords(view, pageSize, filter);
   const stage = useStageField(view.subject);
   if (fields.error) return /* @__PURE__ */ (0, import_jsx_runtime26.jsx)(RefusalNotice, { error: fields.error });
   if (records.error) return /* @__PURE__ */ (0, import_jsx_runtime26.jsx)(RefusalNotice, { error: records.error });
@@ -13000,14 +13008,20 @@ function DashboardCanvas({ tableId, activeDashboardId, filter, className }) {
         rights.admin ? /* @__PURE__ */ (0, import_jsx_runtime47.jsx)(import_design_system44.Button, { size: "sm", variant: "ghost", onClick: () => void remove(board), children: "Delete" }) : null
       ] }) : null
     ] }),
-    board && scheduling ? /* @__PURE__ */ (0, import_jsx_runtime47.jsx)(
-      DigestScheduler,
-      {
-        tableId,
-        subjectName: board.name,
-        ...filter ? { filters: filter } : {},
-        onClose: () => setScheduling(false)
-      }
+    board && scheduling ? (
+      // ONE SHAPE, SO NO CAST. The dashboard's own filter, the saved view it
+      // schedules over and the drill-through it opens are all the same
+      // `RecordFilter`; the `as Record<string, unknown>` that used to be here
+      // was the seam where they could have drifted.
+      /* @__PURE__ */ (0, import_jsx_runtime47.jsx)(
+        DigestScheduler,
+        {
+          tableId,
+          subjectName: board.name,
+          ...filter ? { filters: filter } : {},
+          onClose: () => setScheduling(false)
+        }
+      )
     ) : null,
     board ? /* @__PURE__ */ (0, import_jsx_runtime47.jsxs)("div", { className: "flex flex-col gap-1", children: [
       /* @__PURE__ */ (0, import_jsx_runtime47.jsxs)("div", { className: "flex items-center gap-1", children: [
@@ -15007,9 +15021,35 @@ var import_react111 = require("react");
 var import_react112 = require("@ai-matrx/records/react");
 var import_design_system54 = require("@ai-matrx/design-system");
 var import_jsx_runtime58 = require("react/jsx-runtime");
-function cameFromLine(label, groupField) {
-  const grouped = groupField ? `grouped into the same columns the chart used` : `for this table`;
-  return `You came here from \u201C${label}\u201D. This is every record ${grouped} \u2014 not only the ones that number counted \u2014 so the column you clicked is here with the rest beside it.`;
+function filterInWords(filter, fields) {
+  const nameOf = (key) => {
+    const field = fields.find((f) => f.key === key);
+    return field ? fieldName(field) : humanize(key);
+  };
+  const parts = Object.keys(filter).map((key) => {
+    const value = filter[key];
+    if (value === null || value === void 0) return `${nameOf(key)} is not filled in`;
+    if (typeof value === "object") {
+      const from = (value.from ?? "").toString().trim();
+      const to = (value.to ?? "").toString().trim();
+      if (from !== "" && to !== "") return `${nameOf(key)} from ${from} up to ${to}`;
+      if (from !== "") return `${nameOf(key)} from ${from} onwards`;
+      if (to !== "") return `${nameOf(key)} up to ${to}`;
+      return `${nameOf(key)} in a period nobody named`;
+    }
+    return `${nameOf(key)} is ${String(value)}`;
+  });
+  if (parts.length === 0) return "";
+  if (parts.length === 1) return parts[0];
+  return `${parts.slice(0, -1).join(", ")} and ${parts[parts.length - 1]}`;
+}
+function cameFromLine(label, groupField, question) {
+  const asked = (question ?? "").trim();
+  if (asked === "") {
+    const grouped = groupField ? " grouped into the same columns the chart used" : "";
+    return `You came here from \u201C${label}\u201D. This is every record in this table${grouped} \u2014 that number was counted over all of them.`;
+  }
+  return `You came here from \u201C${label}\u201D. These are only the records that number counted: ${asked}. Everything else in this table is being held back \u2014 clear the filter to see all of it.`;
 }
 var TABLE_NOT_REACHABLE = "This table is not in the organization you are working in, so there is nothing here to show. Switch to the organization that owns it and open it again \u2014 or it may have been deleted.";
 var DEFAULT_VIEW_NAME = "All records";
@@ -15056,11 +15096,14 @@ function TablePage({
   onViewChanged,
   activeGroupField,
   cameFrom,
+  filter,
   className
 }) {
   const client = (0, import_react112.useRecordsClient)();
   const table = (0, import_react112.useTable)(tableId);
   const rights = useTableRights(table.data);
+  const pageFields = (0, import_react112.useFields)(filter ? tableId : null);
+  const askedInWords = filter ? filterInWords(filter, pageFields.data ?? []) : null;
   const organizationId = (0, import_react112.useRecordsClient)().config.organizationId;
   const [view, setView] = (0, import_react111.useState)(null);
   const opening = openingRail(activeRecordId);
@@ -15254,7 +15297,7 @@ function TablePage({
           {
             "data-testid": "came-from-a-number",
             className: "rounded-md border px-3 py-2 text-xs leading-relaxed text-muted-foreground",
-            children: cameFromLine(cameFrom, activeGroupField ?? null)
+            children: cameFromLine(cameFrom, activeGroupField ?? null, askedInWords)
           }
         ) : null,
         /* @__PURE__ */ (0, import_jsx_runtime58.jsx)(
@@ -15268,6 +15311,7 @@ function TablePage({
               ...activeGroupField ? { groupField: activeGroupField } : {}
             },
             pageSize,
+            ...filter ? { filter } : {},
             onLayoutChange: (layout) => {
               setLayoutFromLink(layout);
               onViewChanged?.(layout);