@ai-matrx/records-ui 0.93.4 → 0.93.6

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog — @ai-matrx/records-ui
 
+## 0.93.6
+
+Automatic changed-only republish (docs/metadata drift since the last tag — see
+`git diff npm/records-ui/v0.93.4..npm/records-ui/v0.93.6 -- apps/shared/records-ui`).
+No source changes intended and no consumer action required.
+
+## 0.93.5
+### Unreleased
+### page-pass shared fix (2026-09-27): custom fields — the design-system Select, one-line empty, and an agent twin for "Add field"
+
+- **"Holds" is the design-system `Select`** (it was a native `<select>`); the name box is `BasicInput`.
+- **Empty is one line**: "Custom fields · None yet · Add field" — no "0", no rule under nothing.
+- **New prop `agentDoor`** on `CustomFieldsSection`: the section hands its host one door (`CustomFieldsAgentDoor` — live `state()`, a whole-request `check()` that reports every problem at once, and `addField()` through `custom.entity_field_declare` followed by a read-back). New exports: `CustomFieldsAgentDoor`, `CustomFieldsAgentState`, `CustomFieldAddRequest`, `CustomFieldAddResult`, `customFieldRequestProblems`, `CUSTOM_FIELD_ADDABLE_TYPES`.
+- **Proof:** `custom-fields-empty-select-and-agent-door.test.tsx` (Harbor Lane Dental's message templates).
+
+#### Consumer action
+- None required. matrx-frontend's `EntityCustomFields` passes `agentDoor` to register the platform write target `custom_fields_add` on every page that embeds the section.
+
 ## 0.93.4
 ### Unreleased
 ### lane RENDER-3 (2026-09-27): one cell edit redraws one row — and none of the data table's toolbar, headers or footer

package/dist/index.cjs

@@ -54,6 +54,7 @@ __export(src_exports, {
   CHECKLIST_REQUIREMENT_LABEL: () => CHECKLIST_REQUIREMENT_LABEL,
   CONDITION_OPS: () => CONDITION_OPS,
   CONTEXT_POLICY_LABEL: () => CONTEXT_POLICY_LABEL,
+  CUSTOM_FIELD_ADDABLE_TYPES: () => ADDABLE_TYPES,
   CaptureRun: () => CaptureRun,
   CaptureSheet: () => CaptureSheet,
   ChartBlock: () => ChartBlock,
@@ -246,6 +247,7 @@ __export(src_exports, {
   conditionValue: () => conditionValue,
   controlFor: () => controlFor,
   currencyCodeFor: () => currencyCodeFor,
+  customFieldRequestProblems: () => customFieldRequestProblems,
   dashboardDeclareArgs: () => dashboardDeclareArgs,
   dashboardFromSummary: () => dashboardFromSummary,
   dateFormatFor: () => dateFormatFor,
@@ -13347,6 +13349,32 @@ var ADDABLE_TYPES = ADDABLE_TYPE_IDS.map((value) => ({
   value,
   label: fieldTypeChoice(value)?.label ?? value
 }));
+var ADDABLE_TYPE_SET = new Set(ADDABLE_TYPE_IDS);
+function customFieldRequestProblems(requests, existingLabels) {
+  const problems = [];
+  const taken = new Set(existingLabels.map((l) => l.trim().toLowerCase()));
+  const asked = /* @__PURE__ */ new Set();
+  if (requests.length === 0) problems.push("Name at least one field to add.");
+  requests.forEach((request, index) => {
+    const at = requests.length > 1 ? `Field ${index + 1}: ` : "";
+    const label = typeof request?.label === "string" ? request.label.trim() : "";
+    if (!label) {
+      problems.push(`${at}a field needs a name.`);
+      return;
+    }
+    const type = request.type ?? "text";
+    if (!ADDABLE_TYPE_SET.has(type)) {
+      problems.push(
+        `${at}"${label}" cannot hold "${String(type)}". Choose one of: ${ADDABLE_TYPE_IDS.join(", ")}.`
+      );
+    }
+    const key = label.toLowerCase();
+    if (taken.has(key)) problems.push(`${at}there is already a field named "${label}".`);
+    else if (asked.has(key)) problems.push(`${at}"${label}" is asked for twice.`);
+    asked.add(key);
+  });
+  return problems;
+}
 function asField(f) {
   return {
     id: f.id,
@@ -13367,8 +13395,10 @@ function CustomFieldsSection({
   entityToken,
   recordId,
   title = "Custom fields",
-  className
+  className,
+  agentDoor
 }) {
+  const client = (0, import_react68.useRecordsClient)();
   const read = (0, import_react68.useEntityRecord)(entityToken, recordId);
   const rights = (0, import_react68.useEntityFieldRights)(entityToken);
   const mutation = (0, import_react68.useEntityFieldMutation)();
@@ -13379,14 +13409,70 @@ function CustomFieldsSection({
   const [draft, setDraft] = (0, import_react67.useState)({});
   const fields = (0, import_react67.useMemo)(() => read.data?.fields ?? [], [read.data]);
   const mayAdd = rights.data?.may_declare === true;
+  const live = (0, import_react67.useRef)({ fields, rights: rights.data, loading: rights.loading, reload: read.reload });
+  (0, import_react67.useEffect)(() => {
+    live.current = { fields, rights: rights.data, loading: rights.loading, reload: read.reload };
+  });
+  (0, import_react67.useEffect)(() => {
+    if (!agentDoor) return;
+    const state = () => {
+      const now = live.current;
+      return {
+        entityToken,
+        recordId,
+        entityLabel: now.rights?.label ?? null,
+        mayAdd: now.rights ? now.rights.may_declare === true : now.loading ? null : false,
+        refusal: now.rights && !now.rights.may_declare ? now.rights.reason ?? null : null,
+        fields: now.fields.map((f) => ({ key: f.key, label: f.label, type: f.type })),
+        types: ADDABLE_TYPES
+      };
+    };
+    const check = (requests) => {
+      const now = state();
+      if (now.mayAdd === false) {
+        return [now.refusal ?? "This person may not add a field here."];
+      }
+      if (now.mayAdd === null) return ["The section is still reading who may add a field. Try again in a moment."];
+      return customFieldRequestProblems(requests, now.fields.map((f) => f.label));
+    };
+    const addField = async (request) => {
+      const problems = check([request]);
+      if (problems.length > 0) return { ok: false, message: problems.join(" ") };
+      const label = request.label.trim();
+      const type = request.type ?? "text";
+      const declared2 = await client.entityFieldDeclare({ token: entityToken, spec: { label, type } });
+      if (!declared2.ok) {
+        const hint = hintForAPerson(declared2.error.hint);
+        return { ok: false, message: hint ? `${declared2.error.message} ${hint}` : declared2.error.message };
+      }
+      const back = await client.entityRecordRead({ token: entityToken, record_id: recordId });
+      live.current.reload();
+      if (!back.ok || !(back.data?.fields ?? []).some((f) => f.id === declared2.data)) {
+        return {
+          ok: false,
+          message: `The store took "${label}" but this record does not show it yet. Nothing was lost \u2014 read the page again before adding it a second time.`
+        };
+      }
+      return { ok: true, field_id: declared2.data, label, type };
+    };
+    return agentDoor({ entityToken, recordId, state, check, addField });
+  }, [agentDoor, client, entityToken, recordId]);
   if (!read.loading && !read.error && fields.length === 0 && !mayAdd) return null;
   if (read.error) return /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(RefusalLine, { error: read.error, className });
   const values = read.data?.custom ?? {};
   const written = read.data?.custom_written ?? {};
+  const empty = fields.length === 0;
+  if (empty && !adding) {
+    return /* @__PURE__ */ (0, import_jsx_runtime45.jsxs)("section", { className: (0, import_design_system38.cn)("flex min-w-0 items-center gap-2 text-xs", className), children: [
+      /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("h3", { className: "text-sm font-medium", children: title }),
+      !read.loading ? /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("span", { className: "text-muted-foreground", children: "None yet" }) : null,
+      mayAdd ? /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(import_design_system38.Button, { size: "sm", variant: "ghost", className: "ml-auto", onClick: () => setAdding(true), children: "Add field" }) : null
+    ] });
+  }
   return /* @__PURE__ */ (0, import_jsx_runtime45.jsxs)("section", { className: (0, import_design_system38.cn)("flex min-w-0 flex-col gap-2 text-xs", className), children: [
     /* @__PURE__ */ (0, import_jsx_runtime45.jsxs)("div", { className: "flex items-center gap-2", children: [
       /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("h3", { className: "text-sm font-medium", children: title }),
-      /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("span", { className: "text-muted-foreground", children: fields.length }),
+      !empty ? /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("span", { className: "text-muted-foreground", children: fields.length }) : null,
       fields.length > 0 ? /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(
         import_design_system38.Button,
         {
@@ -13455,9 +13541,9 @@ function CustomFieldsSection({
           /* @__PURE__ */ (0, import_jsx_runtime45.jsxs)("label", { className: "flex flex-col gap-1", children: [
             /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("span", { className: "text-muted-foreground", children: "Name" }),
             /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(
-              "input",
+              import_design_system38.BasicInput,
               {
-                className: "h-8 rounded-md border bg-background px-2",
+                className: "h-8",
                 value: newLabel,
                 onChange: (e) => setNewLabel(e.target.value),
                 placeholder: "Account tier",
@@ -13465,17 +13551,12 @@ function CustomFieldsSection({
               }
             )
           ] }),
-          /* @__PURE__ */ (0, import_jsx_runtime45.jsxs)("label", { className: "flex flex-col gap-1", children: [
+          /* @__PURE__ */ (0, import_jsx_runtime45.jsxs)("div", { className: "flex flex-col gap-1", children: [
             /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("span", { className: "text-muted-foreground", children: "Holds" }),
-            /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(
-              "select",
-              {
-                className: "h-8 rounded-md border bg-background px-2",
-                value: newType,
-                onChange: (e) => setNewType(e.target.value),
-                children: ADDABLE_TYPES.map((t) => /* @__PURE__ */ (0, import_jsx_runtime45.jsx)("option", { value: t.value, children: t.label }, t.value))
-              }
-            )
+            /* @__PURE__ */ (0, import_jsx_runtime45.jsxs)(import_design_system38.Select, { value: newType, onValueChange: setNewType, children: [
+              /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(import_design_system38.SelectTrigger, { size: "sm", "aria-label": "What this field holds", children: /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(import_design_system38.SelectValue, {}) }),
+              /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(import_design_system38.SelectContent, { children: ADDABLE_TYPES.map((t) => /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(import_design_system38.SelectItem, { value: t.value, children: t.label }, t.value)) })
+            ] })
           ] }),
           /* @__PURE__ */ (0, import_jsx_runtime45.jsx)(import_design_system38.Button, { size: "sm", type: "submit", disabled: mutation.saving || newLabel.trim() === "", children: mutation.saving ? "Adding\u2026" : "Add" })
         ]