@ai-matrx/records-ui 0.3.2 → 0.5.4

package/CHANGELOG.md

@@ -1,5 +1,115 @@
 # Changelog — @ai-matrx/records-ui
 
+## 0.5.4
+
+Makes the live FormBuilder certification wait for the editor state that follows
+the saved-form tab, instead of asserting synchronously between the two React
+effects.
+
+### Consumer action
+
+None. This changes only the live certification harness.
+
+## 0.5.3
+
+Makes the certification harness's raw record selector hide soft-deleted rows
+by default, matching the active-record behavior of the store's public read
+doors. An explicit `deleted_at` filter still allows a test to inspect
+tombstones intentionally.
+
+### Consumer action
+
+None. This changes only the live certification harness.
+
+## 0.5.2
+
+Resets records inside the test organization's harness-owned `records_ui_*`
+package Tables through the real delete door when certification starts. A view,
+form, or action edited by an earlier run can no longer override the next run's
+declared seed while product behavior continues to preserve real user edits.
+
+### Consumer action
+
+None. This changes only the live certification harness.
+
+## 0.5.1
+
+Builds the 0.5.0 system-table reconciliation against the refreshed 240-door
+`@ai-matrx/records` contract after the dependency's live release guard caught
+two newly registered store doors.
+
+### Consumer action
+
+None. The public API is unchanged.
+
+## 0.5.0
+
+**A package Table that changed SHAPE under an organization that already had a
+copy.** `ActionInbox` could not record an outcome in one organization: its copy
+of `records_ui_action` held `status` as a uuid-shaped Field while this package
+declares it `text`, so every outcome write was refused `22P02 invalid input
+syntax for type uuid: "open"`. The store was right; this package was the one out
+of date. `ensureSystemTable` reconciled a MISSING Field on every load and never
+looked at a Field it already had, so the organization that got its copy first
+was the one where the package quietly stopped working.
+
+Reconciliation is now two questions, not one — which Fields are missing, and
+which are there with the wrong shape. Shape means exactly the two properties
+that decide what a value is MADE OF, and therefore what the store will accept:
+the behaviour (FLD-1) and whether it is multi (FLD-2). Label, sort, required and
+config are deliberately left alone — they are presentation and policy, an
+organization may legitimately have moved them, and none of them can make a write
+fail. Nothing is ever renamed or removed.
+
+Also fixed alongside it: a refusal from the FIRST declaration of a package Table
+was both returned as this package's never-throw envelope and left as an
+unhandled promise rejection, so a refusal that had in fact been reported
+properly also produced a red console (and, under a test runner, a failed run).
+
+Proved against the LIVE store in `demo/system-table-shape.test.tsx`: the Table
+is declared with the old behaviour, the module is reloaded — which is what a
+page load is, and the only way an organization ever meets a newer package — and
+the new spec is ensured over it. RED without the fix: `expected 'range' to be
+'text'`.
+
+### Consumer action
+
+None. The public API is unchanged, and the reconciliation is automatic on the
+next load of any screen that opens a package-owned Table. An organization whose
+copy drifted is levelled the first time a person opens that screen — there is
+nothing to run and nothing to migrate.
+
+## 0.4.3
+
+Resets the harness-owned demo table through the store's real read and delete
+doors before a certification run. Repeated releases can no longer push the
+three canonical Grid rows beyond the first page. The demo port is also
+configurable for isolated local proof while remaining `3005` by default.
+
+### Consumer action
+
+None. The public package API is unchanged.
+
+## 0.4.2
+
+Uses the same bounded live-store wait as `@ai-matrx/records`, so a legitimate
+overlapping rollback-only verification cannot make the demo proof fail after
+twenty seconds.
+
+### Consumer action
+
+None beyond taking `latest`; the public API is unchanged.
+
+## 0.4.1
+
+Restores monotonic versioning after a concurrent release plan overwrote the
+already-staged 0.4.0 version, and shares the serialized record-store release
+lane with `@ai-matrx/records`.
+
+### Consumer action
+
+None beyond taking `latest`; the 0.4.0 API remains intact.
+
 ## 0.3.2
 
 Automatic changed-only republish (docs/metadata drift since the last tag — see
@@ -28,6 +138,33 @@ table page's records plus its five admin rails, with the settings rail really
 opening the panel; and the same page giving a plain member the records and NONE
 of the administering controls — absent, not disabled.
 
+**THE WALK, and the three defects walking it found.** `demo/walk.test.tsx` runs
+the whole product against the live store in one file — declare a table with three
+field types, write a record through the generated form, open the table page and
+peek the record, save a view, file an agent's field proposal and accept it,
+export the CSV, then walk the same table as a plain member and be offered nothing
+to administer. Seven cases, green, and the table it makes is soft-deleted after.
+It found:
+
+· **A table nobody had saved a view for showed NO RECORDS.** A view is a record,
+  so a brand-new table has none, and `TablePage` then rendered its own honest
+  "Waiting for this table's views" and nothing else — a dead end on the first
+  table anybody makes. The page now seeds `DEFAULT_VIEW_NAME` ("All records",
+  grid), an ordinary saved-view record a person can rename or delete, never a
+  hidden built-in mode. A host with its own idea passes `seedViews`.
+
+· **`ProposedChange.table` was required for all three acts and unfillable for
+  two.** `update` and `remove` name a record and ignore `table`, so a proposer
+  wanting to change an existing record had to supply the id of the kernel Table
+  that record lives in — a fact the client has no door to ask for. It is optional
+  now, and an `add` without it is refused by name instead of writing nowhere.
+
+· **The demo harness's pages were dead in a browser while `pnpm test` was
+  green.** Two copies of React in one vite graph (the demo's own dependency tree
+  plus the workspace's, through design-system and alchemy): React reported
+  "Invalid hook call … more than one copy of React" and the page rendered
+  nothing. `demo/vite.config.ts` now dedupes react and react-dom.
+
 **Consumer action:** none. `recordsDataSource(client)` is unchanged at the call
 site; a host that deliberately wants a different fallback passes it second.
 

package/dist/index.cjs

@@ -980,17 +980,18 @@ async function ensureSystemTable(client, spec) {
   if (found) {
     const grown = await addMissingFields(client, spec, found.id);
     if (!grown.ok) return grown;
+    const levelled = await reconcileFieldShapes(client, spec, found.id);
+    if (!levelled.ok) return levelled;
     return { ok: true, data: found.id };
   }
   let settle = () => void 0;
   let fail = () => void 0;
-  inFlight.set(
-    cacheKey,
-    new Promise((resolve, reject) => {
-      settle = resolve;
-      fail = reject;
-    })
-  );
+  const pending = new Promise((resolve, reject) => {
+    settle = resolve;
+    fail = reject;
+  });
+  pending.catch(() => void 0);
+  inFlight.set(cacheKey, pending);
   const declared = await declare(client, spec);
   if (declared.ok) settle(declared.data);
   else {
@@ -1019,6 +1020,22 @@ async function addMissingFields(client, spec, tableId) {
   }
   return { ok: true, data: true };
 }
+async function reconcileFieldShapes(client, spec, tableId) {
+  const held = await client.fields({ table_id: tableId });
+  if (!held.ok) return held;
+  const byKey = new Map(held.data.map((f) => [f.key, f]));
+  for (const field of spec.fields) {
+    const have = byKey.get(field.key);
+    if (!have) continue;
+    const patch = {};
+    if (have.type !== field.type) patch.type = field.type;
+    if (have.multi !== (field.multi ?? false)) patch.multi = field.multi ?? false;
+    if (Object.keys(patch).length === 0) continue;
+    const written = await client.recordUpdate({ record_id: have.id, patch });
+    if (!written.ok) return written;
+  }
+  return { ok: true, data: true };
+}
 function fieldDocument(field, tableId) {
   return {
     key: field.key,
@@ -2190,6 +2207,12 @@ function ProposalRow({
   const [confirming, setConfirming] = (0, import_react32.useState)(false);
   async function applyThroughTheStore() {
     if (change.act === "add") {
+      if (!change.table) {
+        return {
+          settled: "refused",
+          sentence: "This proposal asks to add a record but does not name the table it goes in, so nothing was written."
+        };
+      }
       const written = await client.recordWrite({ table_id: change.table, data: change.data ?? {} });
       return written.ok ? { settled: "accepted", sentence: "Written." } : { settled: "refused", sentence: written.error.message };
     }
@@ -5895,7 +5918,11 @@ var import_react77 = require("react");
 var import_react78 = require("@ai-matrx/records/react");
 var import_design_system36 = require("@ai-matrx/design-system");
 var import_jsx_runtime38 = require("react/jsx-runtime");
-function TablePage({ tableId, pageSize = 100, className }) {
+var DEFAULT_VIEW_NAME = "All records";
+function defaultView(tableId) {
+  return { name: DEFAULT_VIEW_NAME, subject: tableId, layout: "grid", isDefault: true };
+}
+function TablePage({ tableId, pageSize = 100, seedViews, className }) {
   const table = (0, import_react78.useTable)(tableId);
   const rights = useTableRights(table.data);
   const [view, setView] = (0, import_react77.useState)(null);
@@ -5907,7 +5934,15 @@ function TablePage({ tableId, pageSize = 100, className }) {
   return /* @__PURE__ */ (0, import_jsx_runtime38.jsxs)("div", { className: (0, import_design_system36.cn)("flex min-h-0 gap-4", className), children: [
     /* @__PURE__ */ (0, import_jsx_runtime38.jsxs)("div", { className: "flex min-w-0 flex-1 flex-col gap-2", children: [
       /* @__PURE__ */ (0, import_jsx_runtime38.jsxs)("div", { className: "flex flex-wrap items-center gap-1.5", children: [
-        /* @__PURE__ */ (0, import_jsx_runtime38.jsx)(ViewBar, { tableId, onActiveView: setView, className: "min-w-0 flex-1" }),
+        /* @__PURE__ */ (0, import_jsx_runtime38.jsx)(
+          ViewBar,
+          {
+            tableId,
+            onActiveView: setView,
+            seed: seedViews ?? [defaultView(tableId)],
+            className: "min-w-0 flex-1"
+          }
+        ),
         rights.write ? /* @__PURE__ */ (0, import_jsx_runtime38.jsx)(import_design_system36.Button, { size: "sm", onClick: () => show("new-record"), children: "New record" }) : null,
         rights.admin ? /* @__PURE__ */ (0, import_jsx_runtime38.jsxs)(import_jsx_runtime38.Fragment, { children: [
           /* @__PURE__ */ (0, import_jsx_runtime38.jsx)(import_design_system36.Button, { size: "sm", variant: "outline", onClick: () => show("field"), children: "Add field" }),