@ai-matrx/records-ui 0.3.2 → 0.5.4

package/dist/index.d.cts

@@ -521,8 +521,16 @@ declare function ViewBar({ tableId, activeViewId, onActiveView, seed, className
 interface ProposedChange {
     id: string;
     act: "add" | "update" | "remove";
-    /** The Table the change lands in. */
-    table: Uuid;
+    /**
+     * The Table a NEW record lands in. Required for `add` and unused by `update`
+     * and `remove`, which name a record instead — and it is optional for exactly
+     * that reason: it used to be required for all three, so a proposer wanting to
+     * change an existing record had to supply the id of the kernel Table that
+     * record lives in, which is a fact the client has no door to ask for. A
+     * required field a caller cannot fill is a field that gets filled with
+     * something untrue.
+     */
+    table?: Uuid | undefined;
     /** The record it changes. Required for `update` and `remove`. */
     record?: Uuid | undefined;
     /** The document for `add`, or the patch for `update`. */
@@ -1087,9 +1095,16 @@ interface TablePageProps {
     tableId: Uuid$1;
     /** Rows per page for every layout. */
     pageSize?: number | undefined;
+    /**
+     * The views this table should have. Each one whose NAME the table does not
+     * hold yet is saved the first time the bar loads, and a name a person has
+     * since tweaked is never overwritten. Left out, `DEFAULT_VIEW_NAME` below is
+     * used — see why that default is not optional.
+     */
+    seedViews?: SavedViewSpec[] | undefined;
     className?: string | undefined;
 }
-declare function TablePage({ tableId, pageSize, className }: TablePageProps): react.JSX.Element;
+declare function TablePage({ tableId, pageSize, seedViews, className }: TablePageProps): react.JSX.Element;
 
 /** One Field a person asked for, in the store's own behaviour words (FLD-1). */
 interface NewFieldSpec {

package/dist/index.d.ts

@@ -521,8 +521,16 @@ declare function ViewBar({ tableId, activeViewId, onActiveView, seed, className
 interface ProposedChange {
     id: string;
     act: "add" | "update" | "remove";
-    /** The Table the change lands in. */
-    table: Uuid;
+    /**
+     * The Table a NEW record lands in. Required for `add` and unused by `update`
+     * and `remove`, which name a record instead — and it is optional for exactly
+     * that reason: it used to be required for all three, so a proposer wanting to
+     * change an existing record had to supply the id of the kernel Table that
+     * record lives in, which is a fact the client has no door to ask for. A
+     * required field a caller cannot fill is a field that gets filled with
+     * something untrue.
+     */
+    table?: Uuid | undefined;
     /** The record it changes. Required for `update` and `remove`. */
     record?: Uuid | undefined;
     /** The document for `add`, or the patch for `update`. */
@@ -1087,9 +1095,16 @@ interface TablePageProps {
     tableId: Uuid$1;
     /** Rows per page for every layout. */
     pageSize?: number | undefined;
+    /**
+     * The views this table should have. Each one whose NAME the table does not
+     * hold yet is saved the first time the bar loads, and a name a person has
+     * since tweaked is never overwritten. Left out, `DEFAULT_VIEW_NAME` below is
+     * used — see why that default is not optional.
+     */
+    seedViews?: SavedViewSpec[] | undefined;
     className?: string | undefined;
 }
-declare function TablePage({ tableId, pageSize, className }: TablePageProps): react.JSX.Element;
+declare function TablePage({ tableId, pageSize, seedViews, className }: TablePageProps): react.JSX.Element;
 
 /** One Field a person asked for, in the store's own behaviour words (FLD-1). */
 interface NewFieldSpec {

package/dist/index.js

@@ -872,17 +872,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 {
@@ -911,6 +912,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,
@@ -2097,6 +2114,12 @@ function ProposalRow({
   const [confirming, setConfirming] = useState14(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 };
     }
@@ -5814,7 +5837,11 @@ import { useState as useState35 } from "react";
 import { useTable as useTable21 } from "@ai-matrx/records/react";
 import { Button as Button31, Separator as Separator7, Skeleton as Skeleton25, cn as cn36 } from "@ai-matrx/design-system";
 import { Fragment as Fragment8, jsx as jsx38, jsxs as jsxs35 } from "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 = useTable21(tableId);
   const rights = useTableRights(table.data);
   const [view, setView] = useState35(null);
@@ -5826,7 +5853,15 @@ function TablePage({ tableId, pageSize = 100, className }) {
   return /* @__PURE__ */ jsxs35("div", { className: cn36("flex min-h-0 gap-4", className), children: [
     /* @__PURE__ */ jsxs35("div", { className: "flex min-w-0 flex-1 flex-col gap-2", children: [
       /* @__PURE__ */ jsxs35("div", { className: "flex flex-wrap items-center gap-1.5", children: [
-        /* @__PURE__ */ jsx38(ViewBar, { tableId, onActiveView: setView, className: "min-w-0 flex-1" }),
+        /* @__PURE__ */ jsx38(
+          ViewBar,
+          {
+            tableId,
+            onActiveView: setView,
+            seed: seedViews ?? [defaultView(tableId)],
+            className: "min-w-0 flex-1"
+          }
+        ),
         rights.write ? /* @__PURE__ */ jsx38(Button31, { size: "sm", onClick: () => show("new-record"), children: "New record" }) : null,
         rights.admin ? /* @__PURE__ */ jsxs35(Fragment8, { children: [
           /* @__PURE__ */ jsx38(Button31, { size: "sm", variant: "outline", onClick: () => show("field"), children: "Add field" }),