@ai-matrx/records 0.83.0 → 0.83.2

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog — @ai-matrx/records
 
+## 0.83.2
+
+Automatic changed-only republish (docs/metadata drift since the last tag — see
+`git diff npm/records/v0.83.1..npm/records/v0.83.2 -- apps/shared/records`).
+No source changes intended and no consumer action required.
+
+## 0.83.1
+
+Automatic changed-only republish (docs/metadata drift since the last tag — see
+`git diff npm/records/v0.83.0..npm/records/v0.83.1 -- apps/shared/records`).
+No source changes intended and no consumer action required.
 
 ## 0.83.0 — the gate parses with the one formula language
 
@@ -15,7 +26,7 @@ Consumer action: `parseFormula` / `FORMULA_FUNCTIONS` from `@ai-matrx/records`
 ## 0.82.1
 ### lane DESCRIBE-BENCH2 (sweep): the mechanical wrong shapes are read, not refused
 
-- `autofixDescribeSpec` also rewrites, each with an auto-fix note and only when unambiguous: view `filters` written as a list of `{field, op, value}` (→ the store's map; several values → `anyOf`; a condition the store cannot express is dropped and SAID), `sort`/`orderBy` → `sorts`, `group_by`/`date_field`/… → `groupBy`/`dateField`/…, an exact duplicate view dropped; a field typed with `kind`/`type` (or by a plain label) → `parityType`; a formula over bare field names → `{Label}`; a relationship `inverseKey` that is already a field of the target (the store then read the rollup's link as a plain formula) renamed; a rollup named for a status with no filter → the filter when exactly one far-side choice/tick box is that word; a real-looking phone → a fictional one; a copy of the shared team_member block → an ordinary table (name, role, notes).
+- `autofixDescribeSpec` also rewrites, each with an auto-fix note and only when unambiguous: view `filters` written as a list of `{field, op, value}` (→ the store's map; several values → `anyOf`; a condition the store cannot express is dropped and SAID), `sort`/`orderBy` → `sorts`, `group_by`/`date_field`/… → `groupBy`/`dateField`/…, an exact duplicate view dropped; a field typed with `kind`/`type` (or by a plain label) → `parityType`; a formula over bare field names → `{Label}`; a relationship `inverseKey` that is already a field of the target (the store then read the rollup's link as a plain formula) renamed; a rollup named for a status with no filter → the filter when exactly one far-side choice/tick box is that word; a real-looking phone → a fictional one; a copy of the shared team_member block → an ordinary table (name, role, notes) — never a table whose fields are exactly one the person already has (a link-only reuse is not changed, and `describeCheck` no longer calls it a copy of the shared block).
 - `describeVocabulary()` gains `view_filters_shape`, `field_shape`, `formula_shape`, `relationship_shape`, `phone_rule`, `team_rule`.
 
 ## Earlier in this release — a rollup or lookup in the wrong shape is read, not refused

package/dist/templates/index.cjs

@@ -7178,13 +7178,15 @@ function fixFictionalPhones(spec, fixes) {
   Object.assign(spec, next);
   fixes.push(`phones: ${n} number(s) that could reach a real phone replaced with fictional ones (555-0100\u20260199, UK Ofcom drama range)`);
 }
-function fixTeamBlockCopy(spec, fixes) {
+function fixTeamBlockCopy(spec, fixes, existing = []) {
   const block = SHARED_BLOCKS.team_member;
   const used = /* @__PURE__ */ new Set();
   for (const v of spec.views ?? []) for (const k of ["groupBy", "dateField", "startField", "endField", "coverField", "swimlaneBy", "measure"]) if (v.table && v[k]) used.add(`${v.table}.${v[k]}`);
   for (const fm of spec.forms ?? []) for (const q2 of fm.fields ?? []) used.add(`${fm.table}.${q2.fieldKey}`);
   for (const e of spec.extras ?? []) for (const q2 of e.questions ?? []) used.add(`${e.table}.${q2.fieldKey}`);
   for (const t of spec.tables) {
+    const keys = t.fields.map((f) => f.key).sort().join("|");
+    if (existing.some((e) => e.fields.map((f) => f.key).sort().join("|") === keys)) continue;
     const meaning = (f) => block.fields.find((bf) => bf.parityType === f.parityType && bf.sameAs.some((w) => same3(w, f.label) || same3(w, f.key)));
     const hits = t.fields.map((f) => ({ f, bf: meaning(f) })).filter((x) => x.bf);
     if (hits.length < block.copyThreshold) continue;
@@ -7538,7 +7540,7 @@ function autofixDescribeSpec(input, ctx = {}) {
   dropDuplicateViews(spec, autoFixes);
   fixInverseKeys(spec, autoFixes);
   fixRollupFilters(spec, autoFixes);
-  fixTeamBlockCopy(spec, autoFixes);
+  fixTeamBlockCopy(spec, autoFixes, ctx.existingTables ?? []);
   fixFictionalPhones(spec, autoFixes);
   fixMissingQuestionFields(spec, autoFixes);
   raiseSensitivity(spec, autoFixes);
@@ -7555,7 +7557,10 @@ function autofixDescribeSpec(input, ctx = {}) {
 function describeCheck(spec, ctx = {}) {
   const fixed = autofixDescribeSpec(spec, ctx);
   const v = validateTemplate(fixed.spec, { profile: "describe" });
-  return { ...fixed, problems: v.problems.map((p) => ({ at: p.at, says: p.says })) };
+  const sameFields = (t) => (ctx.existingTables ?? []).some((e) => e.fields.map((f) => f.key).sort().join("|") === t.fields.map((f) => f.key).sort().join("|"));
+  const reused = new Set(fixed.spec.tables.filter(sameFields).map((t) => t.token));
+  const problems = v.problems.filter((p) => !(/^This table copies the shared /.test(p.says) && reused.has(/\((\w+)\)/.exec(p.at)?.[1] ?? "")));
+  return { ...fixed, problems: problems.map((p) => ({ at: p.at, says: p.says })) };
 }
 
 // src/templates/describe/repair.ts