@asteby/metacore-runtime-react 29.2.15 → 30.0.0

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @asteby/metacore-runtime-react
 
+## 30.0.0
+
+### Patch Changes
+
+- Updated dependencies [30fe202]
+  - @asteby/metacore-ui@2.14.0
+
+## 29.2.16
+
+### Patch Changes
+
+- f46d82e: Fix `ImportDialog` swallowing the real per-row error report when the backend answers a validate/import failure with a non-2xx status (e.g. 422). Previously any non-2xx response fell into a generic "Error al importar los datos" / "Error al validar el archivo" toast, discarding the `data.failures`/`data.errors` detail the backend had already computed. Now the dialog renders that report when present, and only falls back to the generic toast when the response carries no usable payload.
+
 ## 29.2.15
 
 ### Patch Changes

package/dist/dialogs/import.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"import.d.ts","sourceRoot":"","sources":["../../src/dialogs/import.tsx"],"names":[],"mappings":"AA0BA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,UAAU,CAAA;AAG7C,UAAU,iBAAiB;IACvB,IAAI,EAAE,OAAO,CAAA;IACb,YAAY,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAA;IACrC,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,aAAa,CAAA;IACvB,UAAU,CAAC,EAAE,MAAM,IAAI,CAAA;CAC1B;AAqCD,wBAAgB,YAAY,CAAC,EACzB,IAAI,EACJ,YAAY,EACZ,KAAK,EACL,QAAQ,EACR,UAAU,GACb,EAAE,iBAAiB,+BAkWnB"}
+{"version":3,"file":"import.d.ts","sourceRoot":"","sources":["../../src/dialogs/import.tsx"],"names":[],"mappings":"AA0BA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,UAAU,CAAA;AAG7C,UAAU,iBAAiB;IACvB,IAAI,EAAE,OAAO,CAAA;IACb,YAAY,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAA;IACrC,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,aAAa,CAAA;IACvB,UAAU,CAAC,EAAE,MAAM,IAAI,CAAA;CAC1B;AAqCD,wBAAgB,YAAY,CAAC,EACzB,IAAI,EACJ,YAAY,EACZ,KAAK,EACL,QAAQ,EACR,UAAU,GACb,EAAE,iBAAiB,+BAgYnB"}

package/dist/dialogs/import.js

@@ -82,8 +82,23 @@ export function ImportDialog({ open, onOpenChange, model, metadata, onImported,
             setStep('validation');
         }
         catch (err) {
-            const message = err?.response?.data?.message || 'Error al validar el archivo';
-            toast.error(message);
+            // Backends may answer a validation failure with a non-2xx status
+            // (e.g. 422) while still carrying the real per-row report in the
+            // body. Render it instead of a body-less "something went wrong"
+            // toast whenever that shape is present.
+            const body = err?.response?.data;
+            const payload = body?.data ?? body;
+            if (payload && (Array.isArray(payload.errors) || typeof payload.rowCount === 'number' || typeof payload.valid === 'number')) {
+                setValidationResult({
+                    valid: payload.rowCount ?? payload.valid ?? 0,
+                    skipped: payload.skipped ?? 0,
+                    errors: payload.errors ?? [],
+                });
+                setStep('validation');
+            }
+            else {
+                toast.error(body?.message || 'Error al validar el archivo');
+            }
         }
         finally {
             setValidating(false);
@@ -117,8 +132,27 @@ export function ImportDialog({ open, onOpenChange, model, metadata, onImported,
             }
         }
         catch (err) {
-            const message = err?.response?.data?.message || 'Error al importar los datos';
-            toast.error(message);
+            // A partial or total import failure is still a real, displayable
+            // result — the backend computes per-row reasons in `data.failures`
+            // even when it answers with a non-2xx status (e.g. 422 when zero
+            // rows were created). Show that report instead of discarding it
+            // behind a generic toast.
+            const body = err?.response?.data;
+            const payload = body?.data ?? body;
+            if (payload && (Array.isArray(payload.failures) || Array.isArray(payload.errors) || typeof payload.created === 'number')) {
+                setImportResult({
+                    created: payload.created ?? 0,
+                    skipped: payload.skipped ?? 0,
+                    errors: payload.failures ?? payload.errors ?? [],
+                });
+                setStep('results');
+                if ((payload.created ?? 0) > 0) {
+                    onImported?.();
+                }
+            }
+            else {
+                toast.error(body?.message || 'Error al importar los datos');
+            }
         }
         finally {
             setImporting(false);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asteby/metacore-runtime-react",
-  "version": "29.2.15",
+  "version": "30.0.0",
   "description": "React runtime for metacore hosts — renders addon contributions dynamically",
   "repository": {
     "type": "git",
@@ -38,7 +38,7 @@
     "sonner": ">=1.7",
     "zustand": ">=5",
     "@asteby/metacore-sdk": "^3.4.0",
-    "@asteby/metacore-ui": "^2.13.0"
+    "@asteby/metacore-ui": "^2.14.0"
   },
   "peerDependenciesMeta": {
     "@tanstack/react-router": {
@@ -68,7 +68,7 @@
     "vitest": "^4.0.0",
     "zustand": "^5.0.0",
     "@asteby/metacore-sdk": "3.4.0",
-    "@asteby/metacore-ui": "2.13.0"
+    "@asteby/metacore-ui": "2.14.0"
   },
   "scripts": {
     "build": "tsc -p tsconfig.json",

package/src/dialogs/import.tsx

@@ -146,9 +146,22 @@ export function ImportDialog({
             })
             setStep('validation')
         } catch (err: any) {
-            const message =
-                err?.response?.data?.message || 'Error al validar el archivo'
-            toast.error(message)
+            // Backends may answer a validation failure with a non-2xx status
+            // (e.g. 422) while still carrying the real per-row report in the
+            // body. Render it instead of a body-less "something went wrong"
+            // toast whenever that shape is present.
+            const body = err?.response?.data
+            const payload = body?.data ?? body
+            if (payload && (Array.isArray(payload.errors) || typeof payload.rowCount === 'number' || typeof payload.valid === 'number')) {
+                setValidationResult({
+                    valid: payload.rowCount ?? payload.valid ?? 0,
+                    skipped: payload.skipped ?? 0,
+                    errors: payload.errors ?? [],
+                })
+                setStep('validation')
+            } else {
+                toast.error(body?.message || 'Error al validar el archivo')
+            }
         } finally {
             setValidating(false)
         }
@@ -187,9 +200,26 @@ export function ImportDialog({
                 onImported?.()
             }
         } catch (err: any) {
-            const message =
-                err?.response?.data?.message || 'Error al importar los datos'
-            toast.error(message)
+            // A partial or total import failure is still a real, displayable
+            // result — the backend computes per-row reasons in `data.failures`
+            // even when it answers with a non-2xx status (e.g. 422 when zero
+            // rows were created). Show that report instead of discarding it
+            // behind a generic toast.
+            const body = err?.response?.data
+            const payload = body?.data ?? body
+            if (payload && (Array.isArray(payload.failures) || Array.isArray(payload.errors) || typeof payload.created === 'number')) {
+                setImportResult({
+                    created: payload.created ?? 0,
+                    skipped: payload.skipped ?? 0,
+                    errors: payload.failures ?? payload.errors ?? [],
+                })
+                setStep('results')
+                if ((payload.created ?? 0) > 0) {
+                    onImported?.()
+                }
+            } else {
+                toast.error(body?.message || 'Error al importar los datos')
+            }
         } finally {
             setImporting(false)
             setProgress(0)