@ai-matrx/records-ui 0.66.0 → 0.68.0

package/CHANGELOG.md

@@ -1,5 +1,70 @@
 # Changelog — @ai-matrx/records-ui
 
+## 0.68.0
+
+**Two things a screen was still doing behind a person's back.** (lane TAILS-6, found while
+proving 0.66.0 on the live seat.)
+
+* **The engineer's line was being READ ALOUD.** `sr-only` does not hide a line — it hides it
+  from people with EYES and reads it to everybody else. So the one person who could not skim
+  past *"@ai-matrx/records never reached the store from checklistTemplates: TypeError: Failed
+  to fetch"* was the blind owner of the business, who had it announced in full underneath
+  "We could not reach your data". `RefusalNotice` now marks that line `aria-hidden`; it stays
+  in the DOM with the SQLSTATE on it, and on the notice's `title`, for whoever debugs it.
+  The guard asserts on what a screen reader would SAY, not on what the DOM contains.
+* **The Group by picker contradicted the board beside it.** `offerableFields` answers what is
+  a SENSIBLE column to group by — Choice columns — and the select listed only those. A view,
+  or a `?group=` link from a dashboard number, can name a column outside that set: the board
+  grouped by it happily and the select fell back to its blank "Choose a field". Watched on
+  Rincon Plumbing Co's Jobs board with six columns headed RPC-1000, RPC-1001 … over a picker
+  saying nothing was chosen. The chosen column is now always in the list, named, and marked
+  "an unusual column to use here".
+
+**Consumer action:** none.
+
+## 0.67.0
+
+**The office can copy a waiting client's own link, from the panel where it invited them.**
+(lane TAILS-5.)
+
+* **`PortalsPanel` renders `CopyLink` beside a pending principal**, fed from that person's
+  `accept_path` (new in `@ai-matrx/records` 0.48.0). It is the SAME `CopyLink` this panel
+  already uses for the portal's own address — the one that shows the address to copy by hand
+  when a browser refuses the clipboard — not a second copy control.
+* **A person with no link gets a sentence, not a missing button.** Somebody who is waiting and
+  has no `accept_path` was asked some other way (the server's magic-link route binds without an
+  invitation) or their link expired, and the panel says exactly that with what to do: invite
+  them again and a fresh one is made. Drawing nothing would read as a broken control; drawing a
+  dead button would be worse.
+* **`PortalCardView` takes an optional `origin`** so the bare-mountable view can build a whole
+  URL, falling back to the browser's own origin. It still holds no client and no network, so
+  the blocking gate can keep mounting it against a recorded `custom.portal_card` answer.
+
+**And the two buttons that read alike no longer do.** The Jobs toolbar's control that OPENS the
+add-a-column panel read exactly `"Add field"`, and the panel's own SAVE button read exactly
+`"Add field"` too — `FieldEditor` said `"Save field"` only when EDITING. The toolbar control is
+a TOGGLE, so pressing it while the panel is open CLOSES the panel and throws away everything
+typed into it: a person who had typed a column name, chosen a type and picked the table it
+points at could lose all of it by pressing the words they were told to press, silently, with no
+request ever sent. Measured on 2026-09-21 by a headless walk that lost the column twice and
+spent a round blaming the store.
+
+* The words now live once, in `fieldTypes.ts` beside `FIELD_TYPE_CHOICES` — the module that
+  already owns what a person READS about a column — as `ADD_FIELD_OPEN_LABEL` ("Add field"),
+  `ADD_FIELD_SAVE_LABEL` ("Create field") and `EDIT_FIELD_SAVE_LABEL` ("Save field"). Both
+  surfaces import them; neither writes the literal any more.
+* **They are NOT in `@ai-matrx/design-system`, deliberately:** "field" and "column" are records
+  vocabulary and the design system knows nothing about either, so a labels module there would
+  be a domain word in a generic package that the next person adding a button would not find.
+* `the-opener-and-the-committer-never-read-alike.test.tsx` is a forcing function, not a style
+  note: it asserts the strings differ, that the committer never says "add", and that neither
+  file carries the literal. Shown RED (2 of 3 failing, `expected 'Add field' not to be 'Add
+  field'`) with the collision planted, then GREEN.
+
+**Consumer action:** none for a host that mounts `PortalsPanel`. A host that mounts
+`PortalCardView` directly and serves the app from a known origin should pass `origin`; without
+it the view reads `window.location.origin`, which is right in a browser and empty in SSR.
+
 ## 0.66.0
 
 **A screen stops talking to a developer.** (lane TAILS-6.)

package/dist/index.cjs

@@ -868,6 +868,9 @@ function isRelationFieldType(id) {
 function parityTypesWithNoExplanation() {
   return import_records2.PARITY_FIELD_TYPES.map((row) => row.parity_type).filter((type) => !BY_ID.has(type));
 }
+var ADD_FIELD_OPEN_LABEL = "Add field";
+var ADD_FIELD_SAVE_LABEL = "Create field";
+var EDIT_FIELD_SAVE_LABEL = "Save field";
 
 // src/parity.ts
 var kernelId = (name) => import_records3.KERNEL_TABLES.find((t) => t.name === name)?.id ?? "";
@@ -1489,7 +1492,16 @@ function RefusalNotice({
         /* @__PURE__ */ (0, import_jsx_runtime6.jsxs)(import_design_system5.AlertDescription, { className: "space-y-1 text-xs", children: [
           plain.sentence.replace(/\.$/, "") === plain.title ? null : /* @__PURE__ */ (0, import_jsx_runtime6.jsx)("p", { children: plain.sentence }),
           /* @__PURE__ */ (0, import_jsx_runtime6.jsx)("p", { className: "opacity-80", children: plain.remedy }),
-          plain.forEngineers ? /* @__PURE__ */ (0, import_jsx_runtime6.jsx)("p", { className: "sr-only", ...error.sqlstate ? { "data-sqlstate": error.sqlstate } : {}, children: plain.forEngineers }) : null,
+          plain.forEngineers ? /* @__PURE__ */ (0, import_jsx_runtime6.jsx)(
+            "p",
+            {
+              className: "sr-only",
+              "aria-hidden": "true",
+              "data-for-engineers": "",
+              ...error.sqlstate ? { "data-sqlstate": error.sqlstate } : {},
+              children: plain.forEngineers
+            }
+          ) : null,
           actions
         ] })
       ]
@@ -5021,7 +5033,7 @@ function FieldEditor({ tableId, field, onSaved, onCancel, onRemoved, className }
           size: "sm",
           disabled: mutation.saving || missing !== null,
           onClick: () => void save(),
-          children: mutation.saving ? "Saving\u2026" : field ? "Save field" : "Add field"
+          children: mutation.saving ? "Saving\u2026" : field ? EDIT_FIELD_SAVE_LABEL : ADD_FIELD_SAVE_LABEL
         }
       ),
       onCancel ? /* @__PURE__ */ (0, import_jsx_runtime17.jsx)(import_design_system14.Button, { size: "sm", variant: "ghost", onClick: onCancel, children: "Cancel" }) : null,
@@ -7426,7 +7438,9 @@ function FieldPicker({
 }) {
   const want = LAYOUT_FIELD_KIND[layout];
   const offerable = want ? offerableFields(all, want) : [];
-  if (offerable.length === 0) {
+  const chosenField = chosen ? all.find((field) => field.key === chosen) : void 0;
+  const unusual = chosenField && !offerable.some((field) => field.key === chosenField.key) ? chosenField : void 0;
+  if (offerable.length === 0 && !unusual) {
     return /* @__PURE__ */ (0, import_jsx_runtime26.jsx)("p", { className: "text-xs text-muted-foreground", children: LAYOUT_NO_FIELD[layout] });
   }
   return /* @__PURE__ */ (0, import_jsx_runtime26.jsxs)("div", { className: "flex items-center gap-1.5 text-xs", children: [
@@ -7441,7 +7455,11 @@ function FieldPicker({
         onChange: (event) => onPick(event.target.value === "" ? null : event.target.value),
         children: [
           /* @__PURE__ */ (0, import_jsx_runtime26.jsx)("option", { value: "", children: "Choose a field" }),
-          offerable.map((field) => /* @__PURE__ */ (0, import_jsx_runtime26.jsx)("option", { value: field.key, children: fieldName(field) }, field.key))
+          offerable.map((field) => /* @__PURE__ */ (0, import_jsx_runtime26.jsx)("option", { value: field.key, children: fieldName(field) }, field.key)),
+          unusual ? /* @__PURE__ */ (0, import_jsx_runtime26.jsxs)("option", { value: unusual.key, children: [
+            fieldName(unusual),
+            " \u2014 an unusual column to use here"
+          ] }, unusual.key) : null
         ]
       }
     ),
@@ -9973,6 +9991,7 @@ function PortalsPanel({ tableId, className }) {
           {
             portalId: portal.portal_id,
             ...tableId ? { tableId } : {},
+            origin,
             onChanged: () => void load()
           }
         ) : null
@@ -9980,7 +9999,7 @@ function PortalsPanel({ tableId, className }) {
     }) })
   ] });
 }
-function CopyLink({ url }) {
+function CopyLink({ url, what }) {
   const [copied, setCopied] = (0, import_react68.useState)(false);
   const [shown, setShown] = (0, import_react68.useState)(false);
   return /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)(import_jsx_runtime36.Fragment, { children: [
@@ -10001,7 +10020,7 @@ function CopyLink({ url }) {
             }
           })();
         },
-        children: copied ? "Copied" : "Copy link"
+        children: copied ? "Copied" : what ? `Copy ${what}` : "Copy link"
       }
     ),
     shown ? /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)("p", { className: "mt-1.5 w-full break-all rounded border border-dashed border-border px-2 py-1 text-xs", children: [
@@ -10013,6 +10032,7 @@ function CopyLink({ url }) {
 function PortalDetail({
   portalId,
   tableId,
+  origin,
   onChanged
 }) {
   const client = (0, import_react69.useRecordsClient)();
@@ -10062,6 +10082,7 @@ function PortalDetail({
     PortalCardView,
     {
       card,
+      origin,
       ...tableId ? { tableId } : {},
       onRevoke: revoke,
       onPreview: preview,
@@ -10084,6 +10105,7 @@ function PortalCardView({
   onRevoke,
   onPreview,
   invite,
+  origin,
   className
 }) {
   return /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)("div", { className: (0, import_design_system33.cn)("mt-2.5 flex flex-col gap-2.5 border-t border-border pt-2.5", className), children: [
@@ -10114,7 +10136,7 @@ function PortalCardView({
       ] }, exposed.table_id))
     ] }),
     /* @__PURE__ */ (0, import_jsx_runtime36.jsx)(import_design_system33.Separator, {}),
-    /* @__PURE__ */ (0, import_jsx_runtime36.jsx)(People, { card, onRevoke, onPreview }),
+    /* @__PURE__ */ (0, import_jsx_runtime36.jsx)(People, { card, onRevoke, onPreview, origin }),
     invite ? /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)(import_jsx_runtime36.Fragment, { children: [
       /* @__PURE__ */ (0, import_jsx_runtime36.jsx)(import_design_system33.Separator, {}),
       invite
@@ -10124,8 +10146,10 @@ function PortalCardView({
 function People({
   card,
   onRevoke,
-  onPreview
+  onPreview,
+  origin
 }) {
+  const here = origin ?? (typeof window === "undefined" ? "" : window.location.origin);
   const [askingToRevoke, setAskingToRevoke] = (0, import_react68.useState)(null);
   const [busy, setBusy] = (0, import_react68.useState)(false);
   const [said, setSaid] = (0, import_react68.useState)(null);
@@ -10167,6 +10191,7 @@ function People({
             /* @__PURE__ */ (0, import_jsx_runtime36.jsx)("p", { className: "mt-0.5 break-all text-muted-foreground", children: person.email }),
             /* @__PURE__ */ (0, import_jsx_runtime36.jsx)("p", { className: "mt-0.5 text-muted-foreground", children: state.why }),
             person.is_active ? /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)("div", { className: "mt-1.5 flex flex-wrap gap-1.5", children: [
+              person.accept_path ? /* @__PURE__ */ (0, import_jsx_runtime36.jsx)(CopyLink, { url: `${here}${person.accept_path}`, what: "their link" }) : null,
               /* @__PURE__ */ (0, import_jsx_runtime36.jsx)(
                 import_design_system33.Button,
                 {
@@ -10178,6 +10203,11 @@ function People({
               ),
               /* @__PURE__ */ (0, import_jsx_runtime36.jsx)(import_design_system33.Button, { size: "sm", variant: "ghost", onClick: () => setAskingToRevoke(person), children: "Remove their access" })
             ] }) : null,
+            person.is_active && !person.signed_in && !person.accept_path ? /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)("p", { className: "mt-1.5 text-muted-foreground", children: [
+              "There is no link to copy for ",
+              person.client || person.email,
+              ": they were asked some other way, or the link they were sent has expired. Invite them again and a fresh one is made."
+            ] }) : null,
             asking ? /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)("div", { className: "mt-1.5 flex flex-col gap-1.5 rounded border border-destructive/40 px-2 py-1.5", children: [
               /* @__PURE__ */ (0, import_jsx_runtime36.jsx)("span", { className: "text-destructive", children: revokeConsequence(person, card.title) }),
               /* @__PURE__ */ (0, import_jsx_runtime36.jsxs)("div", { className: "flex flex-wrap gap-1.5", children: [
@@ -14999,7 +15029,7 @@ function TablePage({
               variant: surfaceChosen(surface, { rail: "field" }) ? "secondary" : "outline",
               "aria-pressed": surfaceChosen(surface, { rail: "field" }),
               onClick: () => press({ rail: "field" }),
-              children: "Add field"
+              children: ADD_FIELD_OPEN_LABEL
             }
           ),
           /* @__PURE__ */ (0, import_jsx_runtime58.jsx)(