@ai-matrx/records 0.83.1 → 0.84.0

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog — @ai-matrx/records
 
+## 0.84.0 — an empty box is no value; a link is never sorted by its ids
+
+- **`createTableDataPort` writes "" as null for every column that is not words** (`blankToNull`, exported from
+  `./data/table-port`): a cleared date input sent `post_date: ""` and `record_write` refused it ("Post Date
+  takes a date") with nothing on screen (Applets live test, 2026-10-08). Text columns keep "".
+- **A sort by a link (relation) column is refused by name** ("\"Brand\" is a link, so the store cannot sort
+  by it."): the store would order by the ids in the cell, a sort that looks like it did nothing.
+- Tests red→green in `src/__tests__/an-empty-box-is-no-value-and-a-link-is-never-sorted-by-id.test.ts`.
+- **Consumer action:** a page that sorted a table source by a link field sorts its rows on the page by the
+  links' labels (`@ai-matrx/applets` `sortRows`).
+
+## 0.83.2
+
+Automatic changed-only republish (docs/metadata drift since the last tag — see
+`git diff npm/records/v0.83.1..npm/records/v0.83.2 -- apps/shared/records`).
+No source changes intended and no consumer action required.
+
 ## 0.83.1
 
 Automatic changed-only republish (docs/metadata drift since the last tag — see
@@ -20,7 +37,7 @@ Consumer action: `parseFormula` / `FORMULA_FUNCTIONS` from `@ai-matrx/records`
 ## 0.82.1
 ### lane DESCRIBE-BENCH2 (sweep): the mechanical wrong shapes are read, not refused
 
-- `autofixDescribeSpec` also rewrites, each with an auto-fix note and only when unambiguous: view `filters` written as a list of `{field, op, value}` (→ the store's map; several values → `anyOf`; a condition the store cannot express is dropped and SAID), `sort`/`orderBy` → `sorts`, `group_by`/`date_field`/… → `groupBy`/`dateField`/…, an exact duplicate view dropped; a field typed with `kind`/`type` (or by a plain label) → `parityType`; a formula over bare field names → `{Label}`; a relationship `inverseKey` that is already a field of the target (the store then read the rollup's link as a plain formula) renamed; a rollup named for a status with no filter → the filter when exactly one far-side choice/tick box is that word; a real-looking phone → a fictional one; a copy of the shared team_member block → an ordinary table (name, role, notes) — never a table whose fields are exactly one the person already has (a link-only reuse is not changed).
+- `autofixDescribeSpec` also rewrites, each with an auto-fix note and only when unambiguous: view `filters` written as a list of `{field, op, value}` (→ the store's map; several values → `anyOf`; a condition the store cannot express is dropped and SAID), `sort`/`orderBy` → `sorts`, `group_by`/`date_field`/… → `groupBy`/`dateField`/…, an exact duplicate view dropped; a field typed with `kind`/`type` (or by a plain label) → `parityType`; a formula over bare field names → `{Label}`; a relationship `inverseKey` that is already a field of the target (the store then read the rollup's link as a plain formula) renamed; a rollup named for a status with no filter → the filter when exactly one far-side choice/tick box is that word; a real-looking phone → a fictional one; a copy of the shared team_member block → an ordinary table (name, role, notes) — never a table whose fields are exactly one the person already has (a link-only reuse is not changed, and `describeCheck` no longer calls it a copy of the shared block).
 - `describeVocabulary()` gains `view_filters_shape`, `field_shape`, `formula_shape`, `relationship_shape`, `phone_rule`, `team_rule`.
 
 ## Earlier in this release — a rollup or lookup in the wrong shape is read, not refused

package/dist/data/index.cjs

@@ -214,6 +214,9 @@ function compileSort(sort, shape) {
   for (const s of sort ?? []) {
     const column = shape.columns.find((c) => c.key === s.column);
     if (!column) return { ok: false, error: noSuchColumn(s.column) };
+    if (column.type === "relation") {
+      return { ok: false, error: refusal(`"${column.label}" is a link, so the store cannot sort by it.`, "Sort the rows on the page by each link's label. Nothing was read.") };
+    }
     const as = isNumberColumn(column.type) ? "number" : isDateColumn(column.type) ? "date" : "text";
     out.push({ field: column.key, direction: s.direction === "desc" ? "desc" : "asc", as });
   }
@@ -6072,6 +6075,11 @@ function relationIdsOf(value) {
   }
   return out;
 }
+var WORD_TYPES = /* @__PURE__ */ new Set(["text", "long_text", "rich_text", "email", "url", "phone", "markdown"]);
+function blankToNull(type, value) {
+  if (typeof value === "string" && value.trim() === "" && !WORD_TYPES.has(type)) return null;
+  return value;
+}
 function createTableDataPort({ client, aliases }) {
   const shapes = /* @__PURE__ */ new Map();
   let warnedNotLive = false;
@@ -6145,7 +6153,7 @@ function createTableDataPort({ client, aliases }) {
       if (!column) return refuse2("invalid_argument", `This table has no column "${key}".`, "Ask columns() for the keys this table has now. Nothing was saved.");
       if (column.readOnly) return refuse2("invalid_argument", `"${column.label}" is worked out by the table, so it cannot be written.`, "Nothing was saved.");
       const kinds = column.type === "relation" ? entityKindsOf(shape.fields.get(key)) : [];
-      out[key] = kinds.length ? entityRefsOf2(value, kinds) : column.type === "relation" ? relationIdsOf(value) : value;
+      out[key] = kinds.length ? entityRefsOf2(value, kinds) : column.type === "relation" ? relationIdsOf(value) : blankToNull(column.type, value);
     }
     return { ok: true, data: out };
   }