velocious 1.0.444 → 1.0.445

package/package.json

@@ -3,7 +3,7 @@
     "velocious": "build/bin/velocious.js"
   },
   "name": "velocious",
-  "version": "1.0.444",
+  "version": "1.0.445",
   "main": "build/index.js",
   "types": "build/index.d.ts",
   "files": [

package/src/frontend-model-controller.js

@@ -218,6 +218,14 @@ function frontendModelClientMessageForError(error) {
     return error.message
   }
 
+  // Validation failures are expected user-flow errors. Always forward the
+  // validation summary so the client shows the real reason (e.g. "Name can't
+  // be blank") instead of the generic "Request failed." message, regardless of
+  // whether the raising code also attached error.velocious metadata.
+  if (error instanceof ValidationError) {
+    return error.message
+  }
+
   if (frontendModelErrorHasVelociousMetadata(error) && error instanceof Error) {
     return error.message
   }
@@ -2827,6 +2835,24 @@ export default class FrontendModelController extends Controller {
       model: resolvedModel,
       requestId
     }])
+
+    // Surface genuinely unexpected backend failures on the framework-error
+    // channel so process-level bug reporters capture them, instead of the
+    // controller silently swallowing them behind the generic "Request
+    // failed." client message. Developer-annotated user-flow errors
+    // (`error.velocious` metadata — handled by the early return above),
+    // validation errors, and deliberately client-safe VelociousErrors are
+    // expected and must NOT be reported as framework errors.
+    if (!(error instanceof ValidationError) && !(error instanceof VelociousError && error.safeToExpose)) {
+      const errorPayload = {
+        context: {action, commandType, frontendModelEndpoint: true, model: resolvedModel, requestId},
+        error: error instanceof Error ? error : new Error(String(error)),
+        request: this.getRequest()
+      }
+
+      this.getConfiguration().getErrorEvents().emit("framework-error", errorPayload)
+      this.getConfiguration().getErrorEvents().emit("all-error", {...errorPayload, errorType: "framework-error"})
+    }
   }
 
   /**

package/src/frontend-models/base.js

@@ -3967,7 +3967,7 @@ export default class FrontendModelBase {
 
     const error = /**
                    * Narrows the runtime value to the documented type.
-                    @type {Error & {velocious?: Record<string, ?>, errorType?: string, validationErrors?: Record<string, ?>}} */ (new Error(errorMessage))
+                    @type {Error & {velocious?: Record<string, ?>, errorType?: string, validationErrors?: Record<string, ?>, debugErrorClass?: string, debugBacktrace?: string[]}} */ (new Error(errorMessage))
     if (response.velocious && typeof response.velocious === "object") {
       error.velocious = response.velocious
     }
@@ -3977,6 +3977,16 @@ export default class FrontendModelBase {
     if (response.validationErrors && typeof response.validationErrors === "object") {
       error.validationErrors = response.validationErrors
     }
+    // Forward server-provided debug detail (included only when the backend
+    // deems the requester allowed to see it, e.g. an admin) so callers can
+    // render the real error class and stack trace instead of the generic
+    // client-safe message.
+    if (typeof response.debugErrorClass === "string") {
+      error.debugErrorClass = response.debugErrorClass
+    }
+    if (Array.isArray(response.debugBacktrace)) {
+      error.debugBacktrace = response.debugBacktrace
+    }
     throw error
   }