@ai-matrx/records 0.72.12 → 0.72.14

package/dist/templates/index.js

@@ -3463,6 +3463,12 @@ function validateTemplate(raw, options = {}) {
     seedLooks.add(p);
     problems.push(p);
   };
+  const galleryBar = /* @__PURE__ */ new WeakSet();
+  const sayGallery = (at, says) => {
+    const p = { at, says };
+    galleryBar.add(p);
+    problems.push(p);
+  };
   const tz = raw.business?.timezone || "UTC";
   const tablesRaw = allTables(raw);
   const ISO_DAY = /\b\d{4}-\d{2}-\d{2}\b/;
@@ -3785,7 +3791,7 @@ function validateTemplate(raw, options = {}) {
       say(`${tat}.rows`, `Only ${titles.size} of ${table.rows.length} rows have a title of their own; a list of look-alike names reads as generated.`);
     }
   });
-  if (!hasLookupOrRollup) say("tables", "At least one relationship has to be read across: add a lookup or a rollup through a relation field.");
+  if (!hasLookupOrRollup) sayGallery("tables", "At least one relationship has to be read across: add a lookup or a rollup through a relation field.");
   if (main) {
     const normTitle = (v) => (storeText(v) ?? "").toLowerCase().replace(/\s+/g, " ").trim();
     const rows = main.rows;
@@ -3853,8 +3859,8 @@ function validateTemplate(raw, options = {}) {
     else surfaces.set(token, { at, ...table ? { table } : {} });
   };
   const views = spec.views ?? [];
-  if (views.length < 3) say("views", `A template shows at least three views; this one has ${views.length}.`);
-  if (!views.some((v) => v.kind === "grid")) say("views", "One of the views is a grid \u2014 the table everyone recognises.");
+  if (views.length < 3) sayGallery("views", `A template shows at least three views; this one has ${views.length}.`);
+  if (!views.some((v) => v.kind === "grid")) sayGallery("views", "One of the views is a grid \u2014 the table everyone recognises.");
   views.forEach((view, i) => {
     const at = `views[${i}] (${view.token})`;
     claim(view.token, at, view.table);
@@ -3980,7 +3986,7 @@ function validateTemplate(raw, options = {}) {
     }
   };
   const extras = spec.extras ?? [];
-  if (extras.length < 2) say("extras", `A template carries at least two extras; this one has ${extras.length}.`);
+  if (extras.length < 2) sayGallery("extras", `A template carries at least two extras; this one has ${extras.length}.`);
   const formTokens = new Set(forms.map((f) => f.token));
   const viewTokens = new Set(views.map((v) => v.token));
   const docTokens = new Set(extras.filter((e) => e.kind === "document").map((e) => e.token));
@@ -4329,7 +4335,8 @@ function validateTemplate(raw, options = {}) {
       if (here >= 0 && Number.isFinite(firstWhen) && here < firstWhen && !TERMINAL_STAGE.test(st)) {
         for (const k of rule.filled ?? []) {
           if (!/^(?:datetime|date|checkbox|signature|attachment)$/.test(fieldOf(table, k)?.parityType ?? "")) continue;
-          if (filled(row.values?.[k])) say(at, `"${row.key}" is ${st}, but already has "${fieldOf(table, k)?.label ?? k}", which only a ${rule.when.join("/")} record carries. Leave it empty until the record gets there.`);
+          const v = row.values?.[k];
+          if (fieldOf(table, k)?.parityType === "checkbox" ? v === true || v === "true" : filled(v)) say(at, `"${row.key}" is ${st}, but already has "${fieldOf(table, k)?.label ?? k}", which only a ${rule.when.join("/")} record carries. Leave it empty until the record gets there.`);
         }
       }
       if (!rule.when.includes(st)) continue;
@@ -4667,6 +4674,7 @@ function validateTemplate(raw, options = {}) {
               }
             }
             adds += 1;
+            for (const f of table.fields) if (f.default !== void 0 && !isComputed(f) && values[f.key] === void 0) values[f.key] = f.default;
             const added = { key: `walk-${i}-${ri}`, values };
             const landed = rules ? storeText(values[rules.stageField]) : null;
             for (const k of landed ? rules?.requires?.[landed] ?? [] : []) if (!filled(values[k])) say(`${at}.action.rows[${ri}]`, `The new row "${keyValue}" lands in "${landed}", which needs "${k}" filled; the store would refuse it.`);
@@ -5791,7 +5799,7 @@ ${e.body}`.matchAll(MERGE_TOKEN)].map((m) => m[1]));
       if (bothDates && !datesStated) say(`${at}.childDates`, `"${t.token}" and its ${rel.label.toLowerCase()} ("${parent.token}") both carry dates; say how they agree, e.g. {"relation": "${rel.key}", "field": "<this table's date>", "parentField": "<${parent.token}'s date>"} or say why they are independent in independentOf {"${rel.key}": "<reason>"}.`);
       if (bothStatus && !statusStated) say(`${at}.statusAgrees`, `"${t.token}" and its ${rel.label.toLowerCase()} ("${parent.token}") both carry a status; say which child statuses need which parent status (statusAgrees), e.g. a Closed deal means a Sold listing.`);
     }
-    for (const f of t.fields.filter((x) => isNumber(x) && !isComputed(x) && /(^|_)(count|roster_size|size|number_of|enrolled|headcount|players|members)($|_)/.test(x.key))) {
+    for (const f of t.fields.filter((x) => isNumber(x) && !isComputed(x) && /(^|_)(count|roster_size|team_size|class_size|number_of|enrolled|headcount|players|members)($|_)/.test(x.key))) {
       const linked = own.find((c) => c.fields.some((cf) => isRelation(cf) && cf.relationTarget === t.token));
       const declared = (t.countMatches ?? []).find((c) => c.field === f.key);
       const linkers = own.filter((c) => c.fields.some((cf) => isRelation(cf) && cf.relationTarget === t.token)).map((c) => c.labelPlural.toLowerCase());
@@ -5826,7 +5834,7 @@ ${e.body}`.matchAll(MERGE_TOKEN)].map((m) => m[1]));
     const describeOwes = (p) => {
       const root = p.at.split(/[.\s[(]/)[0];
       if (GALLERY_ONLY_ROOTS.includes(root)) return false;
-      if (seedLooks.has(p)) return false;
+      if (seedLooks.has(p) || galleryBar.has(p)) return false;
       if (/^tables\[\d+\] \([^)]*\)\.rows$/.test(p.at)) return false;
       return true;
     };