efficient-web-builder 23.1.0 → 23.2.0

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## Unreleased
 
+## 23.2.0 - 2026-10-19
+
+**The show-when helpers are the shell's** (SSOT batch 1, GG 2026-10-19). The website editor and the email designer are one editor, one rule, built once (Lewis). `efficient-shell/format` 7.75.0 holds them, and this package re-exports them under the names it had. Nothing a merchant sees moves.
+
+### Changed for hosts
+
+- `lib/puck/show-when-attach` (and `lib/puck/show-when`) re-export `withRowsShownWhen`, `withRowRules`, `RowRule`, `shownForOption`, `addShowWhenFacts` and `ShowWhenFactData` from `efficient-shell/format`, and add `truthy` (a value read as `if (value)` does). `BlockShowWhenFacts` is the shell's `ShowWhenFacts<ShowWhenFactParams>`. `ShowWhenFactParams`, what this editor hands a resolver, stays here.
+- `resolverContext` and `rowsShownForNode` build their lazy context with the shell's `lazyShowWhenContext` and `showWhenFactReaders`.
+- The efficient-shell peer and dev range rises to `^7.75.0`.
+
+### Guard
+
+- `lib/source-rules/one-show-when` R6 now covers every show-when helper of the shell's (`SHELL_HELPERS`, 11 names), not only `withShowWhen` and `showWhenOf`. A repository that writes one of them again fails. The storefront-blocks' own `truthy` (`commerce/page-rules`) goes with its bump to this release.
+- `scripts/test-show-when-rows.tsx` §10b: the five helpers are the shell's functions.
+
 ## 23.1.0 - 2026-10-19
 
 **A list item's rows read the section's settings, and a rule that hides a whole tab keeps the tab strip** (SSOT batch 1, GG 2026-10-19). Nothing a merchant sees moves: no rule uses either yet. They let the last two hand-written hides in the storefront blocks move (DetailRows' Image shape, ProductHero's dynamic-page cut).

package/dist/lib/puck/show-when-attach.d.ts

@@ -1,13 +1,6 @@
-import { showWhenOf, withShowWhen, type ShowWhenRule, type WithShowWhen } from "efficient-shell/format";
-export { showWhenOf, withShowWhen };
-export type { ShowWhenRule, WithShowWhen };
-export declare function withRowsShownWhen<T extends Record<string, unknown>>(fields: T, keys: readonly string[], rule: ShowWhenRule): T;
-export type RowRule = readonly [keys: readonly string[], rule: ShowWhenRule];
-export declare function withRowRules<T extends Record<string, unknown>>(fields: T, table: readonly RowRule[]): T;
-export declare function shownForOption(field: string, known: readonly string[], fallback: string, shownFor: readonly string[]): ShowWhenRule;
-export type ShowWhenFactData = {
-    props?: Record<string, unknown>;
-};
+import { type ShowWhenFacts } from "efficient-shell/format";
+export { showWhenOf, shownForOption, truthy, withRowRules, withRowsShownWhen, withShowWhen, } from "efficient-shell/format";
+export type { RowRule, ShowWhenFactData, ShowWhenRule, WithShowWhen } from "efficient-shell/format";
 export type ShowWhenFactParams = {
     fields: Record<string, unknown>;
     appState?: unknown;
@@ -15,5 +8,5 @@ export type ShowWhenFactParams = {
         type?: string;
     } | null;
 };
-export type BlockShowWhenFacts = Readonly<Record<string, (data: ShowWhenFactData, params: ShowWhenFactParams) => unknown>>;
-export declare function addShowWhenFacts(block: object, facts: BlockShowWhenFacts): void;
+export type BlockShowWhenFacts = ShowWhenFacts<ShowWhenFactParams>;
+export declare const addShowWhenFacts: (block: object, facts: BlockShowWhenFacts) => void;

package/dist/lib/puck/show-when-attach.js

@@ -1 +1 @@
-import{showWhenOf as u,withShowWhen as p}from"efficient-shell/format";const f=new WeakMap;function W(t,o,n){if(!t||!o.some(i=>i in t))return t;let e=f.get(t);e||f.set(t,e=new WeakMap);let c=e.get(n);c||e.set(n,c=new Map);const r=o.join("\0"),a=c.get(r);if(a)return a;const h={...t};for(const i of o){const s=h[i];if(!s||typeof s!="object")continue;const w=u(s);h[i]=p(s,w?{all:[w,n]}:n)}return c.set(r,h),h}function b(t,o){return o.reduce((n,[e,c])=>W(n,e,c),t)}function y(t,o,n,e){const c={field:t,in:e};return e.includes(n)?{any:[c,{not:{field:t,in:o}}]}:c}function R(t,o){const n=t;n.showWhenFacts={...o,...n.showWhenFacts??{}}}export{R as addShowWhenFacts,u as showWhenOf,y as shownForOption,b as withRowRules,W as withRowsShownWhen,p as withShowWhen};
+import{addShowWhenFacts as o}from"efficient-shell/format";import{showWhenOf as s,shownForOption as e,truthy as a,withRowRules as r,withRowsShownWhen as d,withShowWhen as i}from"efficient-shell/format";const t=o;export{t as addShowWhenFacts,s as showWhenOf,e as shownForOption,a as truthy,r as withRowRules,d as withRowsShownWhen,i as withShowWhen};

package/dist/lib/puck/show-when-resolve.js

@@ -1 +1 @@
-import{isShown as S}from"efficient-shell/format";import{containerOffersPositions as g}from"../content/grid-span.js";import{heroGroupNestDepth as j}from"../hero-group-placement.js";import{BUCKET_KEY as k,reseatBucketTabs as E}from"./field-groups.js";import{readsContext as x,resolverContext as y,showWhenOf as v}from"./show-when.js";import{findParentNode as B}from"./passes/document-position.js";const m=new WeakMap,C=new Set(Object.values(k));function A(e,p,s,w={}){if(!e)return e;const u=p?.props??{},c=u.id,d=Object.fromEntries(Object.entries(w).map(([t,r])=>[t,()=>r(p,s)])),O=y(s,{...d,parentOffersPositions:()=>{const t=B(s.appState,c);return t===void 0?void 0:g(t)},heroGroupDepth:()=>j(s.appState?.data,typeof c=="string"?c:"")}),o=[];for(const[t,r]of Object.entries(e)){const f=v(r);!f||x(f,"device")||S(f,u,O)||o.push(t)}if(o.length===0)return e;let i=m.get(e);i||m.set(e,i=new Map);const h=o.join("\0"),a=i.get(h);if(a)return a;const b=new Set(o);let n={};for(const[t,r]of Object.entries(e))b.has(t)||(n[t]=r);return o.some(t=>C.has(t))&&(n=E(n)),i.set(h,n),n}export{A as rowsShownForNode};
+import{isShown as O,showWhenFactReaders as b}from"efficient-shell/format";import{containerOffersPositions as k}from"../content/grid-span.js";import{heroGroupNestDepth as j}from"../hero-group-placement.js";import{BUCKET_KEY as x,reseatBucketTabs as y}from"./field-groups.js";import{readsContext as E,resolverContext as v,showWhenOf as B}from"./show-when.js";import{findParentNode as C}from"./passes/document-position.js";const m=new WeakMap,D=new Set(Object.values(x));function A(e,p,r,w={}){if(!e)return e;const u=p?.props??{},i=u.id,d=b(w,p,r),S=v(r,{...d,parentOffersPositions:()=>{const t=C(r.appState,i);return t===void 0?void 0:k(t)},heroGroupDepth:()=>j(r.appState?.data,typeof i=="string"?i:"")}),o=[];for(const[t,c]of Object.entries(e)){const f=B(c);!f||E(f,"device")||O(f,u,S)||o.push(t)}if(o.length===0)return e;let s=m.get(e);s||m.set(e,s=new Map);const h=o.join("\0"),a=s.get(h);if(a)return a;const g=new Set(o);let n={};for(const[t,c]of Object.entries(e))g.has(t)||(n[t]=c);return o.some(t=>D.has(t))&&(n=y(n)),s.set(h,n),n}export{A as rowsShownForNode};

package/dist/lib/puck/show-when.d.ts

@@ -1,7 +1,7 @@
 import { type ShowWhenContext, type ShowWhenRule } from "efficient-shell/format";
 import type { SlotConfigView, TreeItem } from "efficient-easel/draw";
 export type { ShowWhenRule, ShowWhenContext };
-export { showWhenOf, shownForOption, withRowRules, withRowsShownWhen, withShowWhen, type RowRule, type WithShowWhen } from "./show-when-attach.js";
+export { showWhenOf, shownForOption, truthy, withRowRules, withRowsShownWhen, withShowWhen, type RowRule, type WithShowWhen } from "./show-when-attach.js";
 export declare function visibleFields<T extends Record<string, unknown>>(fields: T, values: Readonly<Record<string, unknown>> | null | undefined, context: ShowWhenContext): T;
 export type ShowWhenSelector = {
     index: number;

package/dist/lib/puck/show-when.js

@@ -1 +1 @@
-import{isShown as s}from"efficient-shell/format";import{isEditorOnDynamicPage as c}from"./editor-page-context.js";import{parentOfSelection as a}from"./selection-parent.js";import{showWhenOf as W,shownForOption as S,withRowRules as g,withRowsShownWhen as j,withShowWhen as C}from"./show-when-attach.js";import{showWhenOf as u}from"./show-when-attach.js";function w(o,n,t){let e=!1;const r={};for(const[i,p]of Object.entries(o))s(u(p),n??{},t)?r[i]=p:e=!0;return e?r:o}function x(o,n,t,e){const r=a(n,t,e);return{device:o,parentType:r?.item.type,parent:r?.item.props}}function f(o,n){if(!o||typeof o!="object")return!1;const t=o;if(typeof t.context=="string"&&(t.context===n||t.context.startsWith(`${n}.`)))return!0;if("not"in t)return f(t.not,n);for(const e of[t.all,t.any])if(Array.isArray(e)&&e.some(r=>f(r,n)))return!0;return!1}function m(o,n={}){const t={},e=(r,i)=>Object.defineProperty(t,r,{get:i,enumerable:!0});e("parentType",()=>o.parent?.type??void 0),e("parent",()=>o.parent?.props??void 0),e("dynamicPage",()=>c()),e("root",()=>o.appState?.data?.root?.props);for(const[r,i]of Object.entries(n))e(r,i);return t}export{f as readsContext,m as resolverContext,x as showWhenContext,W as showWhenOf,S as shownForOption,w as visibleFields,g as withRowRules,j as withRowsShownWhen,C as withShowWhen};
+import{isShown as s,lazyShowWhenContext as a}from"efficient-shell/format";import{isEditorOnDynamicPage as h}from"./editor-page-context.js";import{parentOfSelection as u}from"./selection-parent.js";import{showWhenOf as C,shownForOption as b,truthy as g,withRowRules as R,withRowsShownWhen as j,withShowWhen as v}from"./show-when-attach.js";import{showWhenOf as c}from"./show-when-attach.js";function d(t,o,e){let r=!1;const n={};for(const[f,i]of Object.entries(t))s(c(i),o??{},e)?n[f]=i:r=!0;return r?n:t}function m(t,o,e,r){const n=u(o,e,r);return{device:t,parentType:n?.item.type,parent:n?.item.props}}function p(t,o){if(!t||typeof t!="object")return!1;const e=t;if(typeof e.context=="string"&&(e.context===o||e.context.startsWith(`${o}.`)))return!0;if("not"in e)return p(e.not,o);for(const r of[e.all,e.any])if(Array.isArray(r)&&r.some(n=>p(n,o)))return!0;return!1}function W(t,o={}){return a({parentType:()=>t.parent?.type??void 0,parent:()=>t.parent?.props??void 0,dynamicPage:()=>h(),root:()=>t.appState?.data?.root?.props,...o})}export{p as readsContext,W as resolverContext,m as showWhenContext,C as showWhenOf,b as shownForOption,g as truthy,d as visibleFields,R as withRowRules,j as withRowsShownWhen,v as withShowWhen};

package/dist/lib/source-rules/one-show-when.js

@@ -1,4 +1,4 @@
-import{ruleChecks as F}from"./rule.js";const W=["lib/puck/show-when.ts","lib/puck/show-when-resolve.ts"],k=(t,l=[])=>/(^|\/)fields\.tsx?$/.test(t)||t.includes("lib/puck/passes/")||l.includes(t),y=new Set(["SettingRow","SettingsField","SettingsSwitch","SettingsSegmented","SettingsDrill","SettingsAction","SettingsGroupSub","InspectorField","FieldText","FieldNumber","FieldSelect","FieldSlider","ListField"]),R=/\b(?:props|values|settings|data\.props)(?:\.|\[|\?\.)/;function g(t,l,c=[]){const i=t.createSourceFile(l.path,l.text,t.ScriptTarget.Latest,!0,t.ScriptKind.TSX),h=e=>i.getLineAndCharacterOfPosition(e.getStart(i)).line+1,r=[];let f=0;const p=W.some(e=>l.path.endsWith(e)),m=e=>t.isPropertyAssignment(e)&&e.name.getText(i)==="resolveFields"||t.isBinaryExpression(e)&&e.operatorToken.kind===t.SyntaxKind.EqualsToken&&t.isPropertyAccessExpression(e.left)&&e.left.name.text==="resolveFields",u=(e,o)=>{const d=o||m(e);if(t.isDeleteExpression(e)&&(t.isPropertyAccessExpression(e.expression)||t.isElementAccessExpression(e.expression))&&(d?r.push({file:l.path,line:h(e),what:"R1 a `delete` in a resolveFields — give the row a showWhen rule"}):k(l.path,c)&&r.push({file:l.path,line:h(e),what:"R2 a `delete` in a field file or a pass — give the row a showWhen rule"})),d&&t.isObjectBindingPattern(e)&&e.elements.some(n=>n.dotDotDotToken)&&r.push({file:l.path,line:h(e),what:"R1 a destructure that leaves a key out, in a resolveFields — give the row a showWhen rule"}),d&&t.isCallExpression(e)&&t.isPropertyAccessExpression(e.expression)&&e.expression.name.text==="filter"&&/^Object\.entries\(/.test(e.expression.expression.getText(i))&&r.push({file:l.path,line:h(e),what:"R1 a filter over Object.entries in a resolveFields — give the row a showWhen rule"}),d&&t.isForOfStatement(e)&&/^Object\.entries\(/.test(e.expression.getText(i))&&r.push({file:l.path,line:h(e),what:"R4 a loop over Object.entries in a resolveFields — give the row a showWhen rule"}),t.isJsxExpression(e)&&e.expression){const n=e.expression,w=a=>t.isParenthesizedExpression(a)?w(a.expression):a,s=a=>{if(!a)return!1;const S=t.isJsxElement(a)?a.openingElement:t.isJsxSelfClosingElement(a)?a:t.isJsxFragment(a)?null:void 0;return S===null?a.getChildren(i).some(v=>v.getChildren(i).some(b=>s(b))):!!S&&y.has(S.tagName.getText(i))};(t.isBinaryExpression(n)&&n.operatorToken.kind===t.SyntaxKind.AmpersandAmpersandToken&&R.test(n.left.getText(i))&&s(w(n.right))||t.isConditionalExpression(n)&&R.test(n.condition.getText(i))&&(s(w(n.whenTrue))||s(w(n.whenFalse))))&&r.push({file:l.path,line:h(e),what:"R5 a settings row drawn by a stored value — use the row's showWhen (SettingRow / ShowWhen)"})}const x=n=>!!n&&t.isIdentifier(n)&&(n.text==="withShowWhen"||n.text==="showWhenOf");if((t.isFunctionDeclaration(e)&&x(e.name)||t.isVariableDeclaration(e)&&x(e.name)&&e.initializer&&(t.isArrowFunction(e.initializer)||t.isFunctionExpression(e.initializer)))&&r.push({file:l.path,line:h(e),what:"R6 a second copy of the shell's withShowWhen / showWhenOf — import them from efficient-shell/format"}),t.isCallExpression(e)&&t.isIdentifier(e.expression)){const n=e.expression.text;n==="isShown"&&!p&&r.push({file:l.path,line:h(e),what:"R3 a second reader of the rule — `isShown(` lives in lib/puck/show-when"}),(n==="withShowWhen"||n==="withRowsShownWhen"||n==="withRowRules")&&f++}t.forEachChild(e,n=>u(n,d))};return u(i,!1),{bad:r,attached:f}}function T(t,l,c){const{check:i,report:h}=F("one show-when rule"),r=c.kept??{};{const e=["export const A = { resolveFields: (data, { fields }) => { const f = { ...fields }; delete f.alt; return f; } };","export const B = { resolveFields: (d, { fields }) => { const { align: _x, ...rest } = fields; return rest; } };",'comp.resolveFields = (d, p) => Object.fromEntries(Object.entries(p.fields).filter(([k]) => k !== "x"));',"function cull(f) { delete f[key]; }","const shown = isShown(rule, values);","// delete f.alt in a comment is not a use",'export const ok = { fields: { alt: withShowWhen(text("Alt"), { field: "src", set: true }) } };',"export const C = { resolveFields: (d, { fields }) => { const out = {}; for (const [k, v] of Object.entries(fields)) if (keep.includes(k)) out[k] = v; return out; } };",'const p = <div>{settings.logo_position === "center" && <SettingRow label="Left items" />}</div>;','const q = <div>{props.on ? (<SettingsField label="x" />) : null}</div>;','const r = <div>{style ? <SettingsField label="Style" /> : null}</div>;',"export function withShowWhen(field, rule) { return { ...field, showWhen: rule }; }","const showWhenOf = (f) => f.showWhen;",'export { showWhenOf, withShowWhen } from "efficient-shell/format";'].join(`
-`),o=g(t,{path:"lib/puck/blocks/general/X/fields.tsx",text:e}),d=o.bad.map(s=>s.line);i("R1 flags a delete in a resolveFields",o.bad.some(s=>s.line===1&&s.what.startsWith("R1"))),i("R1 flags a destructure that leaves a key out",o.bad.some(s=>s.line===2&&s.what.startsWith("R1"))),i("R1 flags a filter over Object.entries",o.bad.some(s=>s.line===3&&s.what.startsWith("R1"))),i("R2 flags a delete in a field file's helper",o.bad.some(s=>s.line===4&&s.what.startsWith("R2"))),i("R3 flags a second isShown(",o.bad.some(s=>s.line===5&&s.what.startsWith("R3"))),i("a comment is not a use, and a rule passes and is counted",!d.includes(6)&&!d.includes(7)&&o.attached===1),i("R4 flags an allow-list loop over Object.entries",o.bad.some(s=>s.line===8&&s.what.startsWith("R4"))),i("R5 flags a row drawn by a stored value (&&)",o.bad.some(s=>s.line===9&&s.what.startsWith("R5"))),i("R5 flags a row drawn by a stored value (? :)",o.bad.some(s=>s.line===10&&s.what.startsWith("R5"))),i("R5 lets a part the host passed through",!d.includes(11)),i("R6 flags a second withShowWhen (a function)",o.bad.some(s=>s.line===12&&s.what.startsWith("R6"))),i("R6 flags a second showWhenOf (a const arrow)",o.bad.some(s=>s.line===13&&s.what.startsWith("R6"))),i("R6 lets a re-export from the shell through",!d.includes(14));const x=g(t,{path:"lib/puck/cull.ts",text:"function cull(f) { delete f.x; }"},["lib/puck/cull.ts"]);i("R2 reads a file a repository names as a field file",x.bad.length===1);const n=g(t,{path:"lib/puck/part-styles.ts",text:"function f(next) { delete next[k]; }"});i("R2 reads only field files and passes (a delete of a VALUE elsewhere is not a hide)",n.bad.length===0);const w=g(t,{path:"lib/puck/show-when.ts",text:"isShown(rule, values);"});i("R3 lets the reader read",w.bad.length===0)}const f=l.filter(e=>/\.tsx?$/.test(e.path)&&!e.path.endsWith(".d.ts")),p=[],m=new Map;let u=0;for(const e of f){const o=g(t,e,c.alsoFieldFiles);u+=o.attached,m.set(e.path,o.bad.length),e.path in r||p.push(...o.bad)}i("no settings row hidden by hand: no delete or key-dropping filter in a resolver, field file or pass; one reader",p.length===0,`${p.length} hit(s):
-      `+p.map(e=>`${e.file}:${e.line}  ${e.what}`).join(`
-      `)),i(`read the source (${f.length} TS/TSX files, at least ${c.minFiles})`,f.length>=c.minFiles),i(`found >= ${c.ruleFloor} rows given a rule (found ${u})`,u>=c.ruleFloor,"fewer means rows went back to a hand-written hide, or the reader broke");for(const e of c.alsoFieldFiles??[])i(`read the named field file ${e}`,f.some(o=>o.path===e));for(const[e,o]of Object.entries(r))i(`the exception is still real: ${e} (${o})`,(m.get(e)??0)>0,"remove the stale entry");return h()}export{T as oneShowWhen};
+import{ruleChecks as b}from"./rule.js";const W=["lib/puck/show-when.ts","lib/puck/show-when-resolve.ts"],k=(s,l=[])=>/(^|\/)fields\.tsx?$/.test(s)||s.includes("lib/puck/passes/")||l.includes(s),y=new Set(["SettingRow","SettingsField","SettingsSwitch","SettingsSegmented","SettingsDrill","SettingsAction","SettingsGroupSub","InspectorField","FieldText","FieldNumber","FieldSelect","FieldSlider","ListField"]),S=/\b(?:props|values|settings|data\.props)(?:\.|\[|\?\.)/,E=new Set(["withShowWhen","showWhenOf","withRowsShownWhen","withRowRules","truthy","shownForOption","addShowWhenFacts","withShowWhenFacts","showWhenFactsContext","showWhenFactReaders","lazyShowWhenContext"]);function g(s,l,c=[]){const i=s.createSourceFile(l.path,l.text,s.ScriptTarget.Latest,!0,s.ScriptKind.TSX),h=e=>i.getLineAndCharacterOfPosition(e.getStart(i)).line+1,a=[];let f=0;const w=W.some(e=>l.path.endsWith(e)),x=e=>s.isPropertyAssignment(e)&&e.name.getText(i)==="resolveFields"||s.isBinaryExpression(e)&&e.operatorToken.kind===s.SyntaxKind.EqualsToken&&s.isPropertyAccessExpression(e.left)&&e.left.name.text==="resolveFields",p=(e,n)=>{const d=n||x(e);if(s.isDeleteExpression(e)&&(s.isPropertyAccessExpression(e.expression)||s.isElementAccessExpression(e.expression))&&(d?a.push({file:l.path,line:h(e),what:"R1 a `delete` in a resolveFields — give the row a showWhen rule"}):k(l.path,c)&&a.push({file:l.path,line:h(e),what:"R2 a `delete` in a field file or a pass — give the row a showWhen rule"})),d&&s.isObjectBindingPattern(e)&&e.elements.some(o=>o.dotDotDotToken)&&a.push({file:l.path,line:h(e),what:"R1 a destructure that leaves a key out, in a resolveFields — give the row a showWhen rule"}),d&&s.isCallExpression(e)&&s.isPropertyAccessExpression(e.expression)&&e.expression.name.text==="filter"&&/^Object\.entries\(/.test(e.expression.expression.getText(i))&&a.push({file:l.path,line:h(e),what:"R1 a filter over Object.entries in a resolveFields — give the row a showWhen rule"}),d&&s.isForOfStatement(e)&&/^Object\.entries\(/.test(e.expression.getText(i))&&a.push({file:l.path,line:h(e),what:"R4 a loop over Object.entries in a resolveFields — give the row a showWhen rule"}),s.isJsxExpression(e)&&e.expression){const o=e.expression,u=r=>s.isParenthesizedExpression(r)?u(r.expression):r,t=r=>{if(!r)return!1;const R=s.isJsxElement(r)?r.openingElement:s.isJsxSelfClosingElement(r)?r:s.isJsxFragment(r)?null:void 0;return R===null?r.getChildren(i).some(F=>F.getChildren(i).some(v=>t(v))):!!R&&y.has(R.tagName.getText(i))};(s.isBinaryExpression(o)&&o.operatorToken.kind===s.SyntaxKind.AmpersandAmpersandToken&&S.test(o.left.getText(i))&&t(u(o.right))||s.isConditionalExpression(o)&&S.test(o.condition.getText(i))&&(t(u(o.whenTrue))||t(u(o.whenFalse))))&&a.push({file:l.path,line:h(e),what:"R5 a settings row drawn by a stored value — use the row's showWhen (SettingRow / ShowWhen)"})}const m=o=>!!o&&s.isIdentifier(o)&&E.has(o.text);if((s.isFunctionDeclaration(e)&&m(e.name)||s.isVariableDeclaration(e)&&m(e.name)&&e.initializer&&(s.isArrowFunction(e.initializer)||s.isFunctionExpression(e.initializer)))&&a.push({file:l.path,line:h(e),what:"R6 a second copy of a show-when helper of the shell's — import it from efficient-shell/format"}),s.isCallExpression(e)&&s.isIdentifier(e.expression)){const o=e.expression.text;o==="isShown"&&!w&&a.push({file:l.path,line:h(e),what:"R3 a second reader of the rule — `isShown(` lives in lib/puck/show-when"}),(o==="withShowWhen"||o==="withRowsShownWhen"||o==="withRowRules")&&f++}s.forEachChild(e,o=>p(o,d))};return p(i,!1),{bad:a,attached:f}}function A(s,l,c){const{check:i,report:h}=b("one show-when rule"),a=c.kept??{};{const e=["export const A = { resolveFields: (data, { fields }) => { const f = { ...fields }; delete f.alt; return f; } };","export const B = { resolveFields: (d, { fields }) => { const { align: _x, ...rest } = fields; return rest; } };",'comp.resolveFields = (d, p) => Object.fromEntries(Object.entries(p.fields).filter(([k]) => k !== "x"));',"function cull(f) { delete f[key]; }","const shown = isShown(rule, values);","// delete f.alt in a comment is not a use",'export const ok = { fields: { alt: withShowWhen(text("Alt"), { field: "src", set: true }) } };',"export const C = { resolveFields: (d, { fields }) => { const out = {}; for (const [k, v] of Object.entries(fields)) if (keep.includes(k)) out[k] = v; return out; } };",'const p = <div>{settings.logo_position === "center" && <SettingRow label="Left items" />}</div>;','const q = <div>{props.on ? (<SettingsField label="x" />) : null}</div>;','const r = <div>{style ? <SettingsField label="Style" /> : null}</div>;',"export function withShowWhen(field, rule) { return { ...field, showWhen: rule }; }","const showWhenOf = (f) => f.showWhen;",'export { showWhenOf, withShowWhen } from "efficient-shell/format";','export function truthy(field) { return { not: { field, in: [false, 0, "", null] } }; }',"const withRowRules = (fields, table) => fields;"].join(`
+`),n=g(s,{path:"lib/puck/blocks/general/X/fields.tsx",text:e}),d=n.bad.map(t=>t.line);i("R1 flags a delete in a resolveFields",n.bad.some(t=>t.line===1&&t.what.startsWith("R1"))),i("R1 flags a destructure that leaves a key out",n.bad.some(t=>t.line===2&&t.what.startsWith("R1"))),i("R1 flags a filter over Object.entries",n.bad.some(t=>t.line===3&&t.what.startsWith("R1"))),i("R2 flags a delete in a field file's helper",n.bad.some(t=>t.line===4&&t.what.startsWith("R2"))),i("R3 flags a second isShown(",n.bad.some(t=>t.line===5&&t.what.startsWith("R3"))),i("a comment is not a use, and a rule passes and is counted",!d.includes(6)&&!d.includes(7)&&n.attached===1),i("R4 flags an allow-list loop over Object.entries",n.bad.some(t=>t.line===8&&t.what.startsWith("R4"))),i("R5 flags a row drawn by a stored value (&&)",n.bad.some(t=>t.line===9&&t.what.startsWith("R5"))),i("R5 flags a row drawn by a stored value (? :)",n.bad.some(t=>t.line===10&&t.what.startsWith("R5"))),i("R5 lets a part the host passed through",!d.includes(11)),i("R6 flags a second withShowWhen (a function)",n.bad.some(t=>t.line===12&&t.what.startsWith("R6"))),i("R6 flags a second showWhenOf (a const arrow)",n.bad.some(t=>t.line===13&&t.what.startsWith("R6"))),i("R6 lets a re-export from the shell through",!d.includes(14)),i("R6 flags a second truthy (a function)",n.bad.some(t=>t.line===15&&t.what.startsWith("R6"))),i("R6 flags a second withRowRules (a const arrow)",n.bad.some(t=>t.line===16&&t.what.startsWith("R6")));const m=g(s,{path:"lib/puck/cull.ts",text:"function cull(f) { delete f.x; }"},["lib/puck/cull.ts"]);i("R2 reads a file a repository names as a field file",m.bad.length===1);const o=g(s,{path:"lib/puck/part-styles.ts",text:"function f(next) { delete next[k]; }"});i("R2 reads only field files and passes (a delete of a VALUE elsewhere is not a hide)",o.bad.length===0);const u=g(s,{path:"lib/puck/show-when.ts",text:"isShown(rule, values);"});i("R3 lets the reader read",u.bad.length===0)}const f=l.filter(e=>/\.tsx?$/.test(e.path)&&!e.path.endsWith(".d.ts")),w=[],x=new Map;let p=0;for(const e of f){const n=g(s,e,c.alsoFieldFiles);p+=n.attached,x.set(e.path,n.bad.length),e.path in a||w.push(...n.bad)}i("no settings row hidden by hand: no delete or key-dropping filter in a resolver, field file or pass; one reader",w.length===0,`${w.length} hit(s):
+      `+w.map(e=>`${e.file}:${e.line}  ${e.what}`).join(`
+      `)),i(`read the source (${f.length} TS/TSX files, at least ${c.minFiles})`,f.length>=c.minFiles),i(`found >= ${c.ruleFloor} rows given a rule (found ${p})`,p>=c.ruleFloor,"fewer means rows went back to a hand-written hide, or the reader broke");for(const e of c.alsoFieldFiles??[])i(`read the named field file ${e}`,f.some(n=>n.path===e));for(const[e,n]of Object.entries(a))i(`the exception is still real: ${e} (${n})`,(x.get(e)??0)>0,"remove the stale entry");return h()}export{A as oneShowWhen};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "efficient-web-builder",
-  "version": "23.1.0",
+  "version": "23.2.0",
   "description": "The EFFICIENT website builder: the website editor, the general sections and the renderer that draws them, compiled. Hostable by any EFFICIENT product; nothing here imports a host.",
   "license": "UNLICENSED",
   "author": "Efficient Pty Ltd",
@@ -45,7 +45,7 @@
   "peerDependencies": {
     "@tabler/icons-react": "^3.46.0",
     "efficient-easel": "^13.1.1",
-    "efficient-shell": "^7.74.0",
+    "efficient-shell": "^7.75.0",
     "isomorphic-dompurify": "^3.18.0",
     "next": ">=16",
     "react": ">=19",
@@ -65,7 +65,7 @@
     "@types/react": "^19",
     "@types/react-dom": "^19",
     "efficient-easel": "^13.1.1",
-    "efficient-shell": "^7.74.0",
+    "efficient-shell": "^7.75.0",
     "esbuild": "^0.28.0",
     "happy-dom": "^20.14.5",
     "isomorphic-dompurify": "^3.18.0",