@ai-matrx/records-ui 0.66.0 → 0.67.0

package/CHANGELOG.md

@@ -1,5 +1,48 @@
 # Changelog — @ai-matrx/records-ui
 
+## 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 ?? "";
@@ -5021,7 +5024,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,
@@ -9973,6 +9976,7 @@ function PortalsPanel({ tableId, className }) {
           {
             portalId: portal.portal_id,
             ...tableId ? { tableId } : {},
+            origin,
             onChanged: () => void load()
           }
         ) : null
@@ -9980,7 +9984,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 +10005,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 +10017,7 @@ function CopyLink({ url }) {
 function PortalDetail({
   portalId,
   tableId,
+  origin,
   onChanged
 }) {
   const client = (0, import_react69.useRecordsClient)();
@@ -10062,6 +10067,7 @@ function PortalDetail({
     PortalCardView,
     {
       card,
+      origin,
       ...tableId ? { tableId } : {},
       onRevoke: revoke,
       onPreview: preview,
@@ -10084,6 +10090,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 +10121,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 +10131,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 +10176,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 +10188,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 +15014,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)(