@asteby/metacore-runtime-react 13.10.0 → 13.10.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @asteby/metacore-runtime-react
 
+## 13.10.1
+
+### Patch Changes
+
+- 9107b10: Fix create-placement action submit hitting `/me/undefined/action/...` (400 Invalid record ID). `buildActionUrl` now omits the record segment when there is no record, posting to the collection route `/data/:model/me/action/:action`, so create modals declared as `placement:create` actions work.
+
 ## 13.10.0
 
 ### Minor Changes

package/dist/action-modal-dispatcher.js

@@ -35,7 +35,16 @@ export function ActionModalDispatcher({ open, onOpenChange, action, model, recor
     return null;
 }
 function buildActionUrl(endpoint, model, recordId, actionKey) {
-    return endpoint ? `${endpoint}/${recordId}/action/${actionKey}` : `/data/${model}/me/${recordId}/action/${actionKey}`;
+    // A create-placement (collection) action has no record yet, so `recordId`
+    // is undefined. Omit the `/{id}` segment so the request hits the collection
+    // route (`/data/:model/me/action/:action`) instead of the per-record route
+    // (`/data/:model/me/:id/action/:action`), which would reject the literal
+    // "undefined" as an invalid record ID (ops dynamic.go ExecuteAction → 400).
+    const hasRecord = recordId != null && recordId !== '' && recordId !== 'undefined';
+    if (endpoint) {
+        return hasRecord ? `${endpoint}/${recordId}/action/${actionKey}` : `${endpoint}/action/${actionKey}`;
+    }
+    return hasRecord ? `/data/${model}/me/${recordId}/action/${actionKey}` : `/data/${model}/me/action/${actionKey}`;
 }
 function ConfirmActionDialog({ open, onOpenChange, action, model, record, endpoint, onSuccess }) {
     const { t } = useTranslation();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asteby/metacore-runtime-react",
-  "version": "13.10.0",
+  "version": "13.10.1",
   "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-ui": "2.1.2",
-    "@asteby/metacore-sdk": "3.1.0"
+    "@asteby/metacore-sdk": "3.1.0",
+    "@asteby/metacore-ui": "2.1.2"
   },
   "scripts": {
     "build": "tsc -p tsconfig.json",

package/src/action-modal-dispatcher.tsx

@@ -112,8 +112,17 @@ export function ActionModalDispatcher({
     return null
 }
 
-function buildActionUrl(endpoint: string | undefined, model: string, recordId: string, actionKey: string) {
-    return endpoint ? `${endpoint}/${recordId}/action/${actionKey}` : `/data/${model}/me/${recordId}/action/${actionKey}`
+function buildActionUrl(endpoint: string | undefined, model: string, recordId: string | undefined, actionKey: string) {
+    // A create-placement (collection) action has no record yet, so `recordId`
+    // is undefined. Omit the `/{id}` segment so the request hits the collection
+    // route (`/data/:model/me/action/:action`) instead of the per-record route
+    // (`/data/:model/me/:id/action/:action`), which would reject the literal
+    // "undefined" as an invalid record ID (ops dynamic.go ExecuteAction → 400).
+    const hasRecord = recordId != null && recordId !== '' && recordId !== 'undefined'
+    if (endpoint) {
+        return hasRecord ? `${endpoint}/${recordId}/action/${actionKey}` : `${endpoint}/action/${actionKey}`
+    }
+    return hasRecord ? `/data/${model}/me/${recordId}/action/${actionKey}` : `/data/${model}/me/action/${actionKey}`
 }
 
 function ConfirmActionDialog({ open, onOpenChange, action, model, record, endpoint, onSuccess }: ActionModalProps) {