@asteby/metacore-runtime-react 13.5.1 → 13.5.2

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @asteby/metacore-runtime-react
 
+## 13.5.2
+
+### Patch Changes
+
+- bc99aec: Gate per-row table actions by the row's `status` against the action's `requiresState`.
+
+  Row actions that declare a non-empty `requiresState` (camelCase or the snake_case
+  `requires_state` served by the backend) are now hidden in the row-action dropdown
+  unless the row's `status` value is one of the declared states. For example, an
+  "Iniciar trabajo" action with `requiresState: ['reception']` no longer appears on an
+  order already in `in_progress`.
+
+  Additive and null-safe: actions without `requiresState` (or an empty array) are always
+  shown, and rows without a `status` field surface every action, so there is no
+  regression for existing models.
+
 ## 13.5.1
 
 ### Patch Changes

package/dist/dynamic-columns.d.ts

@@ -14,6 +14,19 @@ export interface DynamicColumnsHelpers {
      */
     apiBaseUrl?: string;
 }
+/**
+ * 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`.
+ *
+ * Null-safe & non-regressive:
+ *   - action without requiresState (or empty array)  → always shown.
+ *   - row with no `status` field                      → all actions shown.
+ */
+export declare const isActionAllowedForRowState: (action: any, row: any) => boolean;
 /**
  * Builds the canonical column factory used by `<DynamicTable>` when the host
  * does not supply its own. Pass `{ getImageUrl, apiBaseUrl }` to wire avatar

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

@@ -1 +1 @@
-{"version":3,"file":"dynamic-columns.d.ts","sourceRoot":"","sources":["../src/dynamic-columns.tsx"],"names":[],"mappings":"AAsCA,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;AAwHD;;;;GAIG;AACH,wBAAgB,4BAA4B,CACxC,OAAO,GAAE,qBAA0B,GACpC,iBAAiB,CAqXnB;AAED;;;;GAIG;AACH,eAAO,MAAM,wBAAwB,EAAE,iBACL,CAAA"}
+{"version":3,"file":"dynamic-columns.d.ts","sourceRoot":"","sources":["../src/dynamic-columns.tsx"],"names":[],"mappings":"AAsCA,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;AAOD;;;;;;;;;;;GAWG;AACH,eAAO,MAAM,0BAA0B,GAAI,QAAQ,GAAG,EAAE,KAAK,GAAG,KAAG,OAMlE,CAAA;AAmHD;;;;GAIG;AACH,wBAAgB,4BAA4B,CACxC,OAAO,GAAE,qBAA0B,GACpC,iBAAiB,CAsXnB;AAED;;;;GAIG;AACH,eAAO,MAAM,wBAAwB,EAAE,iBACL,CAAA"}

package/dist/dynamic-columns.js

@@ -21,6 +21,27 @@ import { DynamicIcon } from './dynamic-icon';
 import { isColumnVisibleInTable } from './column-visibility';
 const defaultGetImageUrl = (path) => path;
 const getNestedValue = (obj, path) => path.split('.').reduce((acc, part) => acc && acc[part], obj);
+/**
+ * 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`.
+ *
+ * Null-safe & non-regressive:
+ *   - action without requiresState (or empty array)  → always shown.
+ *   - row with no `status` field                      → 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 === '')
+        return true;
+    return requires.map(String).includes(String(status));
+};
 const lowerFirst = (value) => {
     if (!value)
         return value;
@@ -311,6 +332,7 @@ export function makeDefaultGetDynamicColumns(helpers = {}) {
                 maxSize: 80,
                 meta: {},
                 cell: ({ row }) => (_jsx("div", { className: "flex items-center justify-end", children: _jsxs(DropdownMenu, { children: [_jsx(DropdownMenuTrigger, { asChild: true, children: _jsxs(Button, { variant: "ghost", className: "h-8 w-8 p-0", children: [_jsx("span", { className: "sr-only", children: "Abrir men\u00FA" }), _jsx(MoreHorizontal, { className: "h-4 w-4" })] }) }), _jsx(DropdownMenuContent, { align: "end", children: resolvedActions
+                                    .filter((action) => isActionAllowedForRowState(action, row.original))
                                     .filter((action) => {
                                     if (!action.condition)
                                         return true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asteby/metacore-runtime-react",
-  "version": "13.5.1",
+  "version": "13.5.2",
   "description": "React runtime for metacore hosts — renders addon contributions dynamically",
   "repository": {
     "type": "git",
@@ -61,8 +61,8 @@
     "typescript": "^6.0.0",
     "vitest": "^4.0.0",
     "zustand": "^5.0.0",
-    "@asteby/metacore-sdk": "3.1.0",
-    "@asteby/metacore-ui": "2.1.0"
+    "@asteby/metacore-ui": "2.1.0",
+    "@asteby/metacore-sdk": "3.1.0"
   },
   "scripts": {
     "build": "tsc -p tsconfig.json",

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

@@ -0,0 +1,51 @@
+import { describe, it, expect } from 'vitest'
+
+import { isActionAllowedForRowState } from '../dynamic-columns'
+
+describe('isActionAllowedForRowState', () => {
+    it('hides the action when row.status is NOT in requiresState', () => {
+        const action = { key: 'start', requiresState: ['reception'] }
+        const row = { id: 1, status: 'in_progress' }
+        expect(isActionAllowedForRowState(action, row)).toBe(false)
+    })
+
+    it('shows the action when row.status IS in requiresState', () => {
+        const action = { key: 'start', requiresState: ['reception'] }
+        const row = { id: 1, status: 'reception' }
+        expect(isActionAllowedForRowState(action, row)).toBe(true)
+    })
+
+    it('shows the action when row.status matches one of several requiresState entries', () => {
+        const action = { key: 'finish', requiresState: ['in_progress', 'paused'] }
+        expect(isActionAllowedForRowState(action, { status: 'paused' })).toBe(true)
+    })
+
+    it('always shows the action when requiresState is empty', () => {
+        const action = { key: 'view', requiresState: [] as string[] }
+        expect(isActionAllowedForRowState(action, { status: 'whatever' })).toBe(true)
+    })
+
+    it('always shows the action when requiresState is absent (no regression)', () => {
+        const action = { key: 'view' }
+        expect(isActionAllowedForRowState(action, { status: 'in_progress' })).toBe(true)
+    })
+
+    it('shows all actions when the row has no status field (no regression)', () => {
+        const action = { key: 'start', requiresState: ['reception'] }
+        expect(isActionAllowedForRowState(action, { id: 1 })).toBe(true)
+        expect(isActionAllowedForRowState(action, { id: 1, status: null })).toBe(true)
+        expect(isActionAllowedForRowState(action, { id: 1, status: '' })).toBe(true)
+    })
+
+    it('tolerates the snake_case requires_state served by the backend', () => {
+        const action = { key: 'start', requires_state: ['reception'] }
+        expect(isActionAllowedForRowState(action, { status: 'reception' })).toBe(true)
+        expect(isActionAllowedForRowState(action, { status: 'in_progress' })).toBe(false)
+    })
+
+    it('coerces numeric status / state values via String() comparison', () => {
+        const action = { key: 'advance', requiresState: [1, 2] as unknown as string[] }
+        expect(isActionAllowedForRowState(action, { status: 2 })).toBe(true)
+        expect(isActionAllowedForRowState(action, { status: '3' })).toBe(false)
+    })
+})

package/src/dynamic-columns.tsx

@@ -62,6 +62,26 @@ const defaultGetImageUrl = (path: string) => path
 const getNestedValue = (obj: any, path: string) =>
     path.split('.').reduce((acc, part) => acc && acc[part], obj)
 
+/**
+ * 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`.
+ *
+ * Null-safe & non-regressive:
+ *   - action without requiresState (or empty array)  → always shown.
+ *   - row with no `status` field                      → 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
+    return requires.map(String).includes(String(status))
+}
+
 const lowerFirst = (value?: string) => {
     if (!value) return value
     return value.charAt(0).toLowerCase() + value.slice(1)
@@ -519,6 +539,7 @@ export function makeDefaultGetDynamicColumns(
                             </DropdownMenuTrigger>
                             <DropdownMenuContent align="end">
                                 {resolvedActions
+                                    .filter((action) => isActionAllowedForRowState(action, row.original))
                                     .filter((action) => {
                                         if (!action.condition) return true
                                         const { field, operator, value } = action.condition