@asteby/metacore-runtime-react 29.2.10 → 29.2.11

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @asteby/metacore-runtime-react
 
+## 29.2.11
+
+### Patch Changes
+
+- 9e07da0: Gate `requiresState` on row `status` or `state` so actions like purchases receive_goods hide on draft orders.
+
 ## 29.2.10
 
 ### Patch Changes

package/dist/dynamic-columns.d.ts

@@ -42,13 +42,14 @@ export declare const formatAggregateTotal: (col: ColumnDefinition, value: unknow
  * State-machine gate for per-row actions.
  *
  * An action that declares a non-empty `requiresState` (camelCase) / `requires_state`
- * (snake_case, as served by some backends) is only surfaced for rows whose `status`
- * field value is contained in that array. This hides e.g. an "Iniciar trabajo"
- * action (requiresState: ['reception']) on an order already in `in_progress`.
+ * (snake_case, as served by some backends) is only surfaced for rows whose
+ * lifecycle field (`status` or `state`) is contained in that array. This hides
+ * e.g. "Recibir" (requiresState: ['confirmed','partial']) on a purchase order
+ * still in `draft`.
  *
  * Null-safe & non-regressive:
  *   - action without requiresState (or empty array)  → always shown.
- *   - row with no `status` field                      → all actions shown.
+ *   - row with neither `status` nor `state`          → all actions shown.
  */
 export declare const isActionAllowedForRowState: (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;;;;;;;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"}
+{"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;AAsDD;;;;;;;;;;;;GAYG;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

@@ -106,24 +106,39 @@ const CodeCell = ({ text, maxLength }) => {
     };
     return (_jsxs("div", { className: "group flex items-center gap-1.5", children: [_jsx("code", { className: "rounded bg-muted px-1.5 py-0.5 font-mono text-xs text-foreground/80", title: text, children: display }), _jsx("button", { type: "button", onClick: onCopy, className: "opacity-0 transition-opacity group-hover:opacity-100 text-muted-foreground hover:text-foreground", "aria-label": "Copiar", title: "Copiar", children: copied ? (_jsx(icons.Check, { className: "h-3.5 w-3.5 text-green-500" })) : (_jsx(icons.Copy, { className: "h-3.5 w-3.5" })) })] }));
 };
+/**
+ * Lifecycle column used by `requiresState`: prefer `status` (workshop, vehicles,
+ * …) and fall back to `state` (purchases, inventory transfers, …). Empty string
+ * is treated as missing so a blank `status` does not hide a populated `state`.
+ */
+const rowLifecycleState = (row) => {
+    const status = row?.status;
+    if (status !== undefined && status !== null && status !== '')
+        return status;
+    const state = row?.state;
+    if (state !== undefined && state !== null && state !== '')
+        return state;
+    return undefined;
+};
 /**
  * State-machine gate for per-row actions.
  *
  * An action that declares a non-empty `requiresState` (camelCase) / `requires_state`
- * (snake_case, as served by some backends) is only surfaced for rows whose `status`
- * field value is contained in that array. This hides e.g. an "Iniciar trabajo"
- * action (requiresState: ['reception']) on an order already in `in_progress`.
+ * (snake_case, as served by some backends) is only surfaced for rows whose
+ * lifecycle field (`status` or `state`) is contained in that array. This hides
+ * e.g. "Recibir" (requiresState: ['confirmed','partial']) on a purchase order
+ * still in `draft`.
  *
  * Null-safe & non-regressive:
  *   - action without requiresState (or empty array)  → always shown.
- *   - row with no `status` field                      → all actions shown.
+ *   - row with neither `status` nor `state`          → all actions shown.
  */
 export const isActionAllowedForRowState = (action, row) => {
     const requires = action?.requiresState ?? action?.requires_state;
     if (!Array.isArray(requires) || requires.length === 0)
         return true;
-    const status = row?.status;
-    if (status === undefined || status === null || status === '')
+    const status = rowLifecycleState(row);
+    if (status === undefined)
         return true;
     return requires.map(String).includes(String(status));
 };

package/package.json

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

package/src/__tests__/action-visibility-by-state.test.ts

@@ -48,4 +48,24 @@ describe('isActionAllowedForRowState', () => {
         expect(isActionAllowedForRowState(action, { status: 2 })).toBe(true)
         expect(isActionAllowedForRowState(action, { status: '3' })).toBe(false)
     })
+
+    it('falls back to row.state when status is absent (purchases, transfers)', () => {
+        const receive = { key: 'receive_goods', requiresState: ['confirmed', 'partial'] }
+        expect(isActionAllowedForRowState(receive, { state: 'draft' })).toBe(false)
+        expect(isActionAllowedForRowState(receive, { state: 'confirmed' })).toBe(true)
+        expect(isActionAllowedForRowState(receive, { state: 'partial' })).toBe(true)
+        expect(isActionAllowedForRowState(receive, { state: 'received' })).toBe(false)
+    })
+
+    it('prefers status over state when both are set', () => {
+        const action = { key: 'start', requiresState: ['reception'] }
+        expect(isActionAllowedForRowState(action, { status: 'reception', state: 'draft' })).toBe(true)
+        expect(isActionAllowedForRowState(action, { status: 'in_progress', state: 'reception' })).toBe(false)
+    })
+
+    it('ignores empty status and uses state instead', () => {
+        const action = { key: 'confirm', requiresState: ['draft'] }
+        expect(isActionAllowedForRowState(action, { status: '', state: 'draft' })).toBe(true)
+        expect(isActionAllowedForRowState(action, { status: '', state: 'confirmed' })).toBe(false)
+    })
 })

package/src/dynamic-columns.tsx

@@ -204,23 +204,37 @@ const CodeCell: React.FC<{ text: string; maxLength?: number }> = ({ text, maxLen
     )
 }
 
+/**
+ * Lifecycle column used by `requiresState`: prefer `status` (workshop, vehicles,
+ * …) and fall back to `state` (purchases, inventory transfers, …). Empty string
+ * is treated as missing so a blank `status` does not hide a populated `state`.
+ */
+const rowLifecycleState = (row: any): unknown => {
+    const status = row?.status
+    if (status !== undefined && status !== null && status !== '') return status
+    const state = row?.state
+    if (state !== undefined && state !== null && state !== '') return state
+    return undefined
+}
+
 /**
  * State-machine gate for per-row actions.
  *
  * An action that declares a non-empty `requiresState` (camelCase) / `requires_state`
- * (snake_case, as served by some backends) is only surfaced for rows whose `status`
- * field value is contained in that array. This hides e.g. an "Iniciar trabajo"
- * action (requiresState: ['reception']) on an order already in `in_progress`.
+ * (snake_case, as served by some backends) is only surfaced for rows whose
+ * lifecycle field (`status` or `state`) is contained in that array. This hides
+ * e.g. "Recibir" (requiresState: ['confirmed','partial']) on a purchase order
+ * still in `draft`.
  *
  * Null-safe & non-regressive:
  *   - action without requiresState (or empty array)  → always shown.
- *   - row with no `status` field                      → all actions shown.
+ *   - row with neither `status` nor `state`          → all actions shown.
  */
 export const isActionAllowedForRowState = (action: any, row: any): boolean => {
     const requires: unknown = action?.requiresState ?? action?.requires_state
     if (!Array.isArray(requires) || requires.length === 0) return true
-    const status = row?.status
-    if (status === undefined || status === null || status === '') return true
+    const status = rowLifecycleState(row)
+    if (status === undefined) return true
     return requires.map(String).includes(String(status))
 }