@asteby/metacore-runtime-react 29.2.6 → 29.2.8

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @asteby/metacore-runtime-react
 
+## 29.2.8
+
+### Patch Changes
+
+- b705996: Fix row-action `condition` evaluation for nested fields and `equals`/`notEquals` operators. Hosts declaring `{ field: "user.verified", operator: "equals", value: false }` no longer show every action at once.
+
+## 29.2.7
+
+### Patch Changes
+
+- 89e79b0: fix(barcode-scanner): cerrar al escanear en formularios (modal crear/editar)
+
+  `continuous={false}` en dynamic-form y dynamic-record: al llenar SKU/código el
+  overlay se cierra. El POS sigue con continuous (default) para varios productos.
+
 ## 29.2.6
 
 ### Patch Changes

package/dist/dialogs/dynamic-record.js

@@ -1034,7 +1034,10 @@ function ScannableRecordInput({ field, value, onChange, inputType, }) {
     const input = (_jsx(Input, { type: inputType, value: value ?? '', onChange: (e) => onChange(inputType === 'number' ? (e.target.value === '' ? '' : Number(e.target.value)) : e.target.value), placeholder: field.placeholder }));
     if (!scanEnabled)
         return input;
-    return (_jsxs("div", { className: "flex w-full min-w-0 items-center gap-1.5", children: [_jsx("div", { className: "min-w-0 flex-1", children: input }), _jsx(Button, { type: "button", variant: "outline", size: "icon", className: "size-9 shrink-0", onClick: () => setScanOpen(true), title: "Escanear con la c\u00E1mara", "aria-label": "Escanear c\u00F3digo de barras con la c\u00E1mara", children: _jsx(ScanLine, { className: "size-4" }) }), _jsx(BarcodeScanner, { open: scanOpen, onClose: () => setScanOpen(false), onDetected: (code) => onChange(code), continuous: true, position: "fixed", title: `Escanear ${field.label ?? ''}`.trim() })] }));
+    return (_jsxs("div", { className: "flex w-full min-w-0 items-center gap-1.5", children: [_jsx("div", { className: "min-w-0 flex-1", children: input }), _jsx(Button, { type: "button", variant: "outline", size: "icon", className: "size-9 shrink-0", onClick: () => setScanOpen(true), title: "Escanear con la c\u00E1mara", "aria-label": "Escanear c\u00F3digo de barras con la c\u00E1mara", children: _jsx(ScanLine, { className: "size-4" }) }), _jsx(BarcodeScanner, { open: scanOpen, onClose: () => setScanOpen(false), onDetected: (code) => onChange(code), 
+                // Un campo = un código: cerrar al detectar. El POS usa continuous
+                // (default) porque suma varios productos seguidos.
+                continuous: false, position: "fixed", title: `Escanear ${field.label ?? ''}`.trim() })] }));
 }
 function ImageUploadField({ field: _field, value, onChange }) {
     const { t } = useTranslation();

package/dist/dynamic-columns.d.ts

@@ -53,8 +53,11 @@ export declare const formatAggregateTotal: (col: ColumnDefinition, value: unknow
 export declare const isActionAllowedForRowState: (action: any, row: any) => boolean;
 /**
  * Declarative `condition` gate for a per-row action: shows the action only when
- * the row's `field` satisfies the `eq | neq | in | not_in` operator. No
- * condition declared → always shown. Unknown operator → permissive.
+ * the row's `field` satisfies the operator. Supports both the SDK dialect
+ * (`eq` | `neq` | `in` | `not_in`) and the common host dialect
+ * (`equals` | `notEquals` | `not_in`). Nested paths (`user.verified`) are
+ * resolved via `getNestedValue`. No condition → always shown. Unknown
+ * operator → permissive.
  */
 export declare const isActionConditionMet: (action: any, row: any) => boolean;
 /**

package/dist/dynamic-columns.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dynamic-columns.d.ts","sourceRoot":"","sources":["../src/dynamic-columns.tsx"],"names":[],"mappings":"AAcA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAE9B,OAAO,EAAU,KAAK,MAAM,EAAE,MAAM,UAAU,CAAA;AAwC9C,OAAO,KAAK,EAAiB,gBAAgB,EAAE,MAAM,SAAS,CAAA;AAE9D,OAAO,KAAK,EAER,iBAAiB,EACpB,MAAM,wBAAwB,CAAA;AAE/B,qEAAqE;AACrE,MAAM,WAAW,qBAAqB;IAClC;;;;OAIG;IACH,WAAW,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,MAAM,CAAA;IACtC;;;;OAIG;IACH,UAAU,CAAC,EAAE,MAAM,CAAA;CACtB;AA0BD;;;;GAIG;AACH,eAAO,MAAM,eAAe,GAAI,KAAK,gBAAgB,EAAE,cAAc,MAAM,KAAG,MACzB,CAAA;AAQrD;;;;;GAKG;AACH,eAAO,MAAM,WAAW,GAAI,KAAK,gBAAgB,KAAG,MAAM,GAAG,SAG5D,CAAA;AAED;;;;;;GAMG;AACH,eAAO,MAAM,oBAAoB,GAC7B,KAAK,gBAAgB,EACrB,OAAO,OAAO,EACd,WAAW,MAAM,EACjB,SAAS,MAAM,KAChB,MAyBF,CAAA;AAyCD;;;;;;;;;;;GAWG;AACH,eAAO,MAAM,0BAA0B,GAAI,QAAQ,GAAG,EAAE,KAAK,GAAG,KAAG,OAMlE,CAAA;AAED;;;;GAIG;AACH,eAAO,MAAM,oBAAoB,GAAI,QAAQ,GAAG,EAAE,KAAK,GAAG,KAAG,OAiB5D,CAAA;AAED;;;;GAIG;AACH,eAAO,MAAM,kBAAkB,GAAI,QAAQ,GAAG,EAAE,KAAK,GAAG,KAAG,OACqB,CAAA;AAwFhF;;;;;;;GAOG;AACH,eAAO,MAAM,cAAc,GAAI,KAAK,IAAI,CAAC,gBAAgB,EAAE,KAAK,CAAC,KAAG,MAGnE,CAAA;AAED,6EAA6E;AAC7E,eAAO,MAAM,eAAe,2DAA4D,CAAA;AAExF;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,cAAc,CAC1B,KAAK,EAAE,OAAO,EACd,QAAQ,EAAE,MAAM,GAAG,SAAS,EAC5B,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE,MAAM,GAClB;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CA6C5C;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,oBAAoB,GAAI,KAAK,gBAAgB,EAAE,KAAK,GAAG,KAAG,MAWtE,CAAA;AAED;;;;;;GAMG;AACH,eAAO,MAAM,oBAAoB,GAAI,KAAK,gBAAgB,EAAE,KAAK,GAAG,KAAG,MAOtE,CAAA;AAED;;;;;;;;;;;;;;GAcG;AACH,eAAO,MAAM,gBAAgB,GACzB,KAAK,gBAAgB,EACrB,KAAK,GAAG,EACR,OAAO,GAAG,EACV,mBAAe,KAChB,MAAM,GAAG,SAcX,CAAA;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,uBAAuB,GAAI,KAAK,gBAAgB,EAAE,KAAK,GAAG,KAAG,MAOzE,CAAA;AA+ID;;;;GAIG;AACH;;;;;GAKG;AACH,eAAO,MAAM,SAAS,EAAE,KAAK,CAAC,EAAE,CAAC;IAC7B,KAAK,EAAE,OAAO,CAAA;IACd,WAAW,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,MAAM,CAAA;CACxC,CAqBA,CAAA;AAED,wBAAgB,4BAA4B,CACxC,OAAO,GAAE,qBAA0B,GACpC,iBAAiB,CA+mBnB;AAED;;;;GAIG;AACH,eAAO,MAAM,wBAAwB,EAAE,iBACL,CAAA"}
+{"version":3,"file":"dynamic-columns.d.ts","sourceRoot":"","sources":["../src/dynamic-columns.tsx"],"names":[],"mappings":"AAcA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAE9B,OAAO,EAAU,KAAK,MAAM,EAAE,MAAM,UAAU,CAAA;AAwC9C,OAAO,KAAK,EAAiB,gBAAgB,EAAE,MAAM,SAAS,CAAA;AAE9D,OAAO,KAAK,EAER,iBAAiB,EACpB,MAAM,wBAAwB,CAAA;AAE/B,qEAAqE;AACrE,MAAM,WAAW,qBAAqB;IAClC;;;;OAIG;IACH,WAAW,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,MAAM,CAAA;IACtC;;;;OAIG;IACH,UAAU,CAAC,EAAE,MAAM,CAAA;CACtB;AA0BD;;;;GAIG;AACH,eAAO,MAAM,eAAe,GAAI,KAAK,gBAAgB,EAAE,cAAc,MAAM,KAAG,MACzB,CAAA;AAQrD;;;;;GAKG;AACH,eAAO,MAAM,WAAW,GAAI,KAAK,gBAAgB,KAAG,MAAM,GAAG,SAG5D,CAAA;AAED;;;;;;GAMG;AACH,eAAO,MAAM,oBAAoB,GAC7B,KAAK,gBAAgB,EACrB,OAAO,OAAO,EACd,WAAW,MAAM,EACjB,SAAS,MAAM,KAChB,MAyBF,CAAA;AAyCD;;;;;;;;;;;GAWG;AACH,eAAO,MAAM,0BAA0B,GAAI,QAAQ,GAAG,EAAE,KAAK,GAAG,KAAG,OAMlE,CAAA;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,oBAAoB,GAAI,QAAQ,GAAG,EAAE,KAAK,GAAG,KAAG,OAkC5D,CAAA;AAED;;;;GAIG;AACH,eAAO,MAAM,kBAAkB,GAAI,QAAQ,GAAG,EAAE,KAAK,GAAG,KAAG,OACqB,CAAA;AAwFhF;;;;;;;GAOG;AACH,eAAO,MAAM,cAAc,GAAI,KAAK,IAAI,CAAC,gBAAgB,EAAE,KAAK,CAAC,KAAG,MAGnE,CAAA;AAED,6EAA6E;AAC7E,eAAO,MAAM,eAAe,2DAA4D,CAAA;AAExF;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,cAAc,CAC1B,KAAK,EAAE,OAAO,EACd,QAAQ,EAAE,MAAM,GAAG,SAAS,EAC5B,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE,MAAM,GAClB;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CA6C5C;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,oBAAoB,GAAI,KAAK,gBAAgB,EAAE,KAAK,GAAG,KAAG,MAWtE,CAAA;AAED;;;;;;GAMG;AACH,eAAO,MAAM,oBAAoB,GAAI,KAAK,gBAAgB,EAAE,KAAK,GAAG,KAAG,MAOtE,CAAA;AAED;;;;;;;;;;;;;;GAcG;AACH,eAAO,MAAM,gBAAgB,GACzB,KAAK,gBAAgB,EACrB,KAAK,GAAG,EACR,OAAO,GAAG,EACV,mBAAe,KAChB,MAAM,GAAG,SAcX,CAAA;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,uBAAuB,GAAI,KAAK,gBAAgB,EAAE,KAAK,GAAG,KAAG,MAOzE,CAAA;AA+ID;;;;GAIG;AACH;;;;;GAKG;AACH,eAAO,MAAM,SAAS,EAAE,KAAK,CAAC,EAAE,CAAC;IAC7B,KAAK,EAAE,OAAO,CAAA;IACd,WAAW,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,MAAM,CAAA;CACxC,CAqBA,CAAA;AAED,wBAAgB,4BAA4B,CACxC,OAAO,GAAE,qBAA0B,GACpC,iBAAiB,CA+mBnB;AAED;;;;GAIG;AACH,eAAO,MAAM,wBAAwB,EAAE,iBACL,CAAA"}

package/dist/dynamic-columns.js

@@ -129,23 +129,43 @@ export const isActionAllowedForRowState = (action, row) => {
 };
 /**
  * Declarative `condition` gate for a per-row action: shows the action only when
- * the row's `field` satisfies the `eq | neq | in | not_in` operator. No
- * condition declared → always shown. Unknown operator → permissive.
+ * the row's `field` satisfies the operator. Supports both the SDK dialect
+ * (`eq` | `neq` | `in` | `not_in`) and the common host dialect
+ * (`equals` | `notEquals` | `not_in`). Nested paths (`user.verified`) are
+ * resolved via `getNestedValue`. No condition → always shown. Unknown
+ * operator → permissive.
  */
 export const isActionConditionMet = (action, row) => {
     if (!action?.condition)
         return true;
     const { field, operator, value } = action.condition;
-    const rowValue = String(row?.[field] ?? '');
-    const values = Array.isArray(value) ? value : [value];
-    switch (operator) {
+    if (!field)
+        return true;
+    const coerce = (v) => {
+        if (v === null || v === undefined)
+            return '';
+        if (typeof v === 'boolean')
+            return v ? 'true' : 'false';
+        return String(v);
+    };
+    const raw = getNestedValue(row, field);
+    const rowValue = coerce(raw);
+    const values = (Array.isArray(value) ? value : [value]).map(coerce);
+    const op = String(operator ?? '').toLowerCase();
+    switch (op) {
         case 'eq':
+        case 'equals':
+        case '==':
             return rowValue === values[0];
         case 'neq':
+        case 'notequals':
+        case 'not_equals':
+        case '!=':
             return rowValue !== values[0];
         case 'in':
             return values.includes(rowValue);
         case 'not_in':
+        case 'notin':
             return !values.includes(rowValue);
         default:
             return true;

package/dist/dynamic-form.js

@@ -264,7 +264,10 @@ function ScannableInput({ field, value, onChange, type, }) {
     const input = (_jsx(Input, { id: field.key, type: type, value: value ?? '', onChange: (e) => onChange(type === 'number' ? e.target.valueAsNumber || '' : e.target.value), placeholder: field.placeholder }));
     if (!scanEnabled)
         return input;
-    return (_jsxs("div", { className: "flex w-full min-w-0 items-center gap-1.5", children: [_jsx("div", { className: "min-w-0 flex-1", children: input }), _jsx(Button, { type: "button", variant: "outline", size: "icon", className: "size-9 shrink-0", onClick: () => setScanOpen(true), title: "Escanear con la c\u00E1mara", "aria-label": "Escanear c\u00F3digo de barras con la c\u00E1mara", children: _jsx(ScanLine, { className: "size-4" }) }), _jsx(BarcodeScanner, { open: scanOpen, onClose: () => setScanOpen(false), onDetected: (code) => onChange(code), continuous: true, position: "fixed", title: `Escanear ${field.label ?? ''}`.trim() })] }));
+    return (_jsxs("div", { className: "flex w-full min-w-0 items-center gap-1.5", children: [_jsx("div", { className: "min-w-0 flex-1", children: input }), _jsx(Button, { type: "button", variant: "outline", size: "icon", className: "size-9 shrink-0", onClick: () => setScanOpen(true), title: "Escanear con la c\u00E1mara", "aria-label": "Escanear c\u00F3digo de barras con la c\u00E1mara", children: _jsx(ScanLine, { className: "size-4" }) }), _jsx(BarcodeScanner, { open: scanOpen, onClose: () => setScanOpen(false), onDetected: (code) => onChange(code), 
+                // Un campo = un código: cerrar al detectar. El POS usa continuous
+                // (default) porque suma varios productos seguidos.
+                continuous: false, position: "fixed", title: `Escanear ${field.label ?? ''}`.trim() })] }));
 }
 function RefSelect({ field, value, onChange }) {
     const { options, loading } = useOptionsResolver({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asteby/metacore-runtime-react",
-  "version": "29.2.6",
+  "version": "29.2.8",
   "description": "React runtime for metacore hosts — renders addon contributions dynamically",
   "repository": {
     "type": "git",

package/src/__tests__/resolve-row-actions.test.ts

@@ -77,4 +77,19 @@ describe('isRowActionVisible', () => {
         expect(isActionConditionMet({ condition: { field: 's', operator: 'not_in', value: ['a', 'b'] } }, { s: 'c' })).toBe(true)
         expect(isActionConditionMet({ condition: { field: 's', operator: 'neq', value: 'a' } }, { s: 'a' })).toBe(false)
     })
+
+    it('isActionConditionMet resolves nested paths and equals/notEquals aliases', () => {
+        const approve = {
+            condition: { field: 'user.verified', operator: 'equals', value: false },
+        }
+        const revoke = {
+            condition: { field: 'user.verified', operator: 'equals', value: true },
+        }
+        expect(isActionConditionMet(approve, { user: { verified: false } })).toBe(true)
+        expect(isActionConditionMet(approve, { user: { verified: true } })).toBe(false)
+        expect(isActionConditionMet(revoke, { user: { verified: true } })).toBe(true)
+        expect(isActionConditionMet(revoke, { user: { verified: false } })).toBe(false)
+        // Flat lookup must NOT match nested conditions (regression of row[field]).
+        expect(isActionConditionMet(approve, { 'user.verified': false } as any)).toBe(false)
+    })
 })

package/src/dialogs/dynamic-record.tsx

@@ -1886,7 +1886,9 @@ function ScannableRecordInput({
                 open={scanOpen}
                 onClose={() => setScanOpen(false)}
                 onDetected={(code) => onChange(code)}
-                continuous
+                // Un campo = un código: cerrar al detectar. El POS usa continuous
+                // (default) porque suma varios productos seguidos.
+                continuous={false}
                 position="fixed"
                 title={`Escanear ${field.label ?? ''}`.trim()}
             />

package/src/dynamic-columns.tsx

@@ -226,22 +226,42 @@ export const isActionAllowedForRowState = (action: any, row: any): boolean => {
 
 /**
  * Declarative `condition` gate for a per-row action: shows the action only when
- * the row's `field` satisfies the `eq | neq | in | not_in` operator. No
- * condition declared → always shown. Unknown operator → permissive.
+ * the row's `field` satisfies the operator. Supports both the SDK dialect
+ * (`eq` | `neq` | `in` | `not_in`) and the common host dialect
+ * (`equals` | `notEquals` | `not_in`). Nested paths (`user.verified`) are
+ * resolved via `getNestedValue`. No condition → always shown. Unknown
+ * operator → permissive.
  */
 export const isActionConditionMet = (action: any, row: any): boolean => {
     if (!action?.condition) return true
     const { field, operator, value } = action.condition
-    const rowValue = String(row?.[field] ?? '')
-    const values = Array.isArray(value) ? value : [value]
-    switch (operator) {
+    if (!field) return true
+
+    const coerce = (v: unknown): string => {
+        if (v === null || v === undefined) return ''
+        if (typeof v === 'boolean') return v ? 'true' : 'false'
+        return String(v)
+    }
+
+    const raw = getNestedValue(row, field)
+    const rowValue = coerce(raw)
+    const values = (Array.isArray(value) ? value : [value]).map(coerce)
+    const op = String(operator ?? '').toLowerCase()
+
+    switch (op) {
         case 'eq':
+        case 'equals':
+        case '==':
             return rowValue === values[0]
         case 'neq':
+        case 'notequals':
+        case 'not_equals':
+        case '!=':
             return rowValue !== values[0]
         case 'in':
             return values.includes(rowValue)
         case 'not_in':
+        case 'notin':
             return !values.includes(rowValue)
         default:
             return true

package/src/dynamic-form.tsx

@@ -480,7 +480,9 @@ function ScannableInput({
                 open={scanOpen}
                 onClose={() => setScanOpen(false)}
                 onDetected={(code) => onChange(code)}
-                continuous
+                // Un campo = un código: cerrar al detectar. El POS usa continuous
+                // (default) porque suma varios productos seguidos.
+                continuous={false}
                 position="fixed"
                 title={`Escanear ${field.label ?? ''}`.trim()}
             />