@asteby/metacore-runtime-react 29.2.7 → 29.2.8

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @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

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asteby/metacore-runtime-react",
-  "version": "29.2.7",
+  "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/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