@ai-matrx/records-ui 0.109.4 → 0.109.6

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog — @ai-matrx/records-ui
 
+## 0.109.6
+### lane SORT-REACTS (follow-up)
+
+- A link's `?sort=` opens the grid sorted and keeps the address's sort: it outranks where the grid opens (the table's own sort, the saved view's — both read after the columns) until the person sorts for herself or presses Reset view. It used to be let go once the columns were known, so a late view sort re-seated the query and rewrote the address.
+- Enter on an edit that MOVES its row leaves the cursor on the edited cell at its new place (the lit row and the keys agree); Tab keeps its own rule. Layout check 9b.
+
+**Consumer action:** none.
+
+## 0.109.5
+
+Automatic changed-only republish (docs/metadata drift since the last tag — see
+`git diff npm/records-ui/v0.109.4..npm/records-ui/v0.109.5 -- apps/shared/records-ui`).
+No source changes intended and no consumer action required.
+
 ## 0.109.4
 ### lane TABLE-EDITING-EDGES
 

package/dist/index.cjs

@@ -4288,7 +4288,7 @@ function ReferenceBuilder({
     ] }) }),
     target && /* @__PURE__ */ (0, import_jsx_runtime12.jsxs)("div", { className: "flex flex-col gap-0.5", children: [
       /* @__PURE__ */ (0, import_jsx_runtime12.jsx)(import_design_system10.Label, { className: "text-xs", children: "It will read" }),
-      preview.loading ? /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("p", { className: "text-[11px] text-muted-foreground", children: "Reading a few of its records\u2026" }) : (preview.data?.rows.length ?? 0) === 0 ? /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("p", { className: "text-[11px] text-muted-foreground", children: "That table has no records yet, so there is nothing to show here \u2014 this column will still work, and the first record added is the first thing it offers." }) : /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("ul", { className: "flex flex-col gap-0.5", children: (preview.data?.rows ?? []).map((row) => /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("li", { className: "truncate text-[11px] text-foreground", children: previewWords(
+      preview.loading ? /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("p", { className: "text-[11px] text-muted-foreground", children: "Reading a few of its records\u2026" }) : (preview.data?.rows?.length ?? 0) === 0 ? /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("p", { className: "text-[11px] text-muted-foreground", children: "That table has no records yet, so there is nothing to show here \u2014 this column will still work, and the first record added is the first thing it offers." }) : /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("ul", { className: "flex flex-col gap-0.5", children: (preview.data?.rows ?? []).map((row) => /* @__PURE__ */ (0, import_jsx_runtime12.jsx)("li", { className: "truncate text-[11px] text-foreground", children: previewWords(
         row.document,
         display,
         titleField
@@ -22236,7 +22236,7 @@ function useRowFollowsItsMove(args) {
   const lastIds = (0, import_react116.useRef)(args.rowIds);
   const frame = (0, import_react116.useRef)(0);
   (0, import_react116.useEffect)(() => () => cancelAnimationFrame(frame.current), []);
-  const noteWrite = (0, import_react116.useCallback)((rowId) => {
+  const noteWrite = (0, import_react116.useCallback)((rowId, key) => {
     const box = latest.current.box.current;
     if (!box) return;
     const el = rowEl(box, rowId);
@@ -22245,7 +22245,7 @@ function useRowFollowsItsMove(args) {
       held.current = null;
       return;
     }
-    held.current = { rowId, top: el.getBoundingClientRect().top, index, at: Date.now() };
+    held.current = { rowId, key, top: el.getBoundingClientRect().top, index, at: Date.now() };
   }, []);
   (0, import_react116.useLayoutEffect)(() => {
     if (args.rowIds === lastIds.current) return;
@@ -22265,6 +22265,7 @@ function useRowFollowsItsMove(args) {
     }
     if (index === pending.index) return;
     held.current = null;
+    latest.current.onMoved?.(pending.rowId, pending.key);
     const el = rowEl(box, pending.rowId);
     if (!el) {
       bringIntoView(box, pending.rowId, index, args.rowIds);
@@ -23274,6 +23275,9 @@ function GridBody({
   const openingKey = openingSort ? `${openingSort.id}:${openingSort.direction}` : "none";
   const address = useViewAddress();
   const addressSeed = (0, import_react123.useRef)(address?.initial ?? null);
+  const addressSort = (0, import_react123.useRef)(
+    address?.initial?.sort ? { id: address.initial.sort.field, direction: address.initial.sort.direction } : null
+  );
   const [query, setQuery] = (0, import_react123.useState)(() => ({
     page: addressSeed.current?.page ?? 1,
     pageSize,
@@ -23288,11 +23292,15 @@ function GridBody({
     if (seed && fields.data) addressSeed.current = null;
     setQuery((held) => ({
       ...held,
-      sort: seed?.sort ? { id: seed.sort.field, direction: seed.sort.direction } : openingSort,
+      sort: addressSort.current ?? openingSort,
       page: seed?.page ?? 1
     }));
     setSeatedOn(openingKey);
   }, [openingKey]);
+  (0, import_react123.useEffect)(() => {
+    const held = addressSort.current;
+    if (held && (query.sort?.id !== held.id || query.sort?.direction !== held.direction)) addressSort.current = null;
+  }, [query.sort]);
   const columnSort = query.sort;
   const overrideKey = sortOverride ? `${sortOverride.field}:${sortOverride.direction}` : "";
   const overrideSeen = (0, import_react123.useRef)(overrideKey);
@@ -23335,6 +23343,7 @@ function GridBody({
     if (resetSeen.current === resetEpoch) return;
     resetSeen.current = resetEpoch;
     addressSeed.current = null;
+    addressSort.current = null;
     setQuery((held) => ({ ...held, sort: openingSort, search: "", anyOf: "", columnFilters: {}, page: 1 }));
   }, [resetEpoch]);
   const [page, setPage] = (0, import_react123.useState)(0);
@@ -23466,6 +23475,7 @@ function GridBody({
   const canWrite = Boolean(editable) && (rights.write || askPerRow && rowIds.some(mayWriteRow));
   const readOnly = !editable || Boolean(host.rights) && !rights.write && !rights.structure;
   const gridBox = (0, import_react123.useRef)(null);
+  const lastExit = (0, import_react123.useRef)(null);
   const pushWritten = (0, import_react123.useRef)(null);
   const sheetController = (0, import_react123.useRef)(null);
   const versions = useRecordVersions(client, rows);
@@ -23486,7 +23496,10 @@ function GridBody({
      * Enter or Cmd-Z works without a click. Sheets and Airtable.
      */
     ...merged ? {
-      onEnded: (exit, from, how) => sheetController.current?.endEdit(exit, { rowId: from.rowId, columnId: from.key }, how)
+      onEnded: (exit, from, how) => {
+        lastExit.current = { rowId: from.rowId, key: from.key, exit };
+        sheetController.current?.endEdit(exit, { rowId: from.rowId, columnId: from.key }, how);
+      }
     } : {}
   });
   const editingNow = (0, import_react123.useRef)(editing);
@@ -23514,7 +23527,15 @@ function GridBody({
     sort: sortedField ? { key: sortedField.key, name: fieldName(sortedField) } : null,
     say: (line) => undo.say(line, false, true)
   });
-  const rowMove = useRowFollowsItsMove({ box: gridBox, rowIds });
+  const rowMove = useRowFollowsItsMove({
+    box: gridBox,
+    rowIds,
+    onMoved: (rowId, key) => {
+      const exit = lastExit.current;
+      if (!exit || exit.rowId !== rowId || exit.key !== key) return;
+      if (exit.exit === "down" || exit.exit === "stay") sheetController.current?.select({ rowId, columnId: key });
+    }
+  });
   pushWritten.current = (write) => {
     const field2 = (fields.data ?? []).find((f) => f.key === write.key);
     const name = changedRecordName(table.data, rows.find((r) => r.id === write.rowId)?.document ?? {}, write.key, write.after);
@@ -23522,7 +23543,7 @@ function GridBody({
       label: `Changed ${field2 ? fieldName(field2) : (0, import_text_case10.humanizeIdentifier)(write.key)} on ${name}`,
       cells: [{ rowId: write.rowId, key: write.key, before: write.before ?? null, after: write.after ?? null, version: write.version }]
     });
-    rowMove.noteWrite(write.rowId);
+    rowMove.noteWrite(write.rowId, write.key);
     noteWriteForView(write, name);
   };
   const storeLayout = useStoreGridLayout(tableId);
@@ -24944,7 +24965,7 @@ function GridBody({
           }
         ) : null,
         question.refused.length > 0 ? /* @__PURE__ */ (0, import_jsx_runtime72.jsx)("p", { role: "status", "data-records-filter-refused": "", className: "text-xs text-muted-foreground", children: question.refused.join(" ") }) : null,
-        storePaged && (serverRead.data?.sortIgnored.length ?? 0) > 0 ? /* @__PURE__ */ (0, import_jsx_runtime72.jsx)("p", { role: "status", "data-records-sort-ignored": serverRead.data?.sortIgnored.join(" "), className: "text-xs text-muted-foreground", children: "Sort skipped: column not available" }) : null,
+        storePaged && sortIgnoredOf(serverRead.data).length > 0 ? /* @__PURE__ */ (0, import_jsx_runtime72.jsx)("p", { role: "status", "data-records-sort-ignored": sortIgnoredOf(serverRead.data).join(" "), className: "text-xs text-muted-foreground", children: "Sort skipped: column not available" }) : null,
         runner.outcome && !(merged && runner.outcome.kind === "ran" && runner.outcome.undo !== null) ? /* @__PURE__ */ (0, import_jsx_runtime72.jsx)(ActionOutcomeNotice, { outcome: runner.outcome, onUndo: () => void runner.undo(), onDismiss: runner.dismiss }) : null,
         duplicated ? /* @__PURE__ */ (0, import_jsx_runtime72.jsxs)("p", { role: "status", "data-records-duplicated": "", className: "flex items-center gap-2 rounded border border-border bg-muted/40 px-2 py-1 text-xs", children: [
           /* @__PURE__ */ (0, import_jsx_runtime72.jsx)("span", { children: duplicated }),
@@ -25346,6 +25367,9 @@ function withoutGoneColumns(query, liveKeys, openingSort) {
   const fallback = openingSort && liveKeys.has(openingSort.id) ? openingSort : null;
   return { ...query, sort: sortGone ? fallback : query.sort, columnFilters, page: 1 };
 }
+function sortIgnoredOf(answer) {
+  return Array.isArray(answer?.sortIgnored) ? answer.sortIgnored : [];
+}
 
 // src/RecordsTableCard.tsx
 var import_data_table15 = require("@ai-matrx/design-system/data-table");
@@ -26781,7 +26805,7 @@ function columnFromName(label) {
   if (has(/\b(type|category|kind|tier|plan|platform|channel|source|level)\b/)) return { label, type: "select" };
   if (has(/\b(time|at|scheduled|starts|ends|appointment)\b/) && !has(/\b(date|day)\b/)) return { label, type: "datetime" };
   if (has(/\b(date|day|birthday|deadline|due|dob|born|joined|since|until|start|end|published|posted)\b/)) return { label, type: "date" };
-  if (has(/\b(price|cost|amount|revenue|budget|fee|fees|salary|paid|invoice|total|spend|value|rate card|mrr|arr)\b|\$/)) return { label, type: "currency" };
+  if (has(/\b(price|cost|amount|revenue|budget|fee|fees|salary|paid|total|spend|value|rate card|mrr|arr)\b|\$/)) return { label, type: "currency" };
   if (has(/%|\b(percent|percentage|rate|ratio|margin|completion)\b/)) return { label, type: "percent" };
   if (has(/\b(count|qty|quantity|number of|age|score|hours|minutes|units|followers|views|likes|stars|rating|points|#)\b|^#|^no\.? /)) return { label, type: "number" };
   if (has(/\b(done|completed|approved|active|signed|confirmed)\b/)) return { label, type: "checkbox" };