@xh/hoist 73.0.0-SNAPSHOT.1741714715638 → 73.0.0-SNAPSHOT.1741790055811

package/CHANGELOG.md

@@ -6,6 +6,13 @@
 
 * Modify `TabContainerModel` to make more methods `protected`, improving extensibility for advanced
 use-cases.
+* Enhance exception handling in `FetchService` to capture messages returned as raw strings, or without
+explicit names.
+
+### 🐞 Bug Fixes
+
+* Prevent native browser context menu on Dash Canvas surfaces. It can hide the Dash Canvas custom
+  context menu when an app's `showBrowserContextMenu` flag is `true`.
 
 ## v72.1.0 - 2025-02-13
 

package/core/exception/Exception.ts

@@ -7,7 +7,7 @@
 import {PlainObject, XH} from '@xh/hoist/core';
 import {FetchOptions} from '@xh/hoist/svc';
 import {pluralize} from '@xh/hoist/utils/js';
-import {isPlainObject} from 'lodash';
+import {isPlainObject, truncate} from 'lodash';
 import {FetchException, HoistException, TimeoutException, TimeoutExceptionConfig} from './Types';
 
 /**
@@ -90,17 +90,16 @@ export class Exception {
         // Try to "smart" decode as server provided JSON Exception (with a name)
         try {
             const cType = headers.get('Content-Type');
-            if (cType && cType.includes('application/json')) {
-                const serverDetails = JSON.parse(responseText);
-                if (serverDetails?.name) {
-                    return this.createFetchException({
-                        ...defaults,
-                        name: serverDetails.name,
-                        message: serverDetails.message,
-                        isRoutine: serverDetails.isRoutine ?? false,
-                        serverDetails
-                    });
-                }
+            if (cType?.includes('application/json')) {
+                const obj = safeParseJson(responseText),
+                    message = obj ? obj.message : truncate(responseText?.trim(), {length: 255});
+                return this.createFetchException({
+                    ...defaults,
+                    name: obj?.name ?? defaults.name,
+                    message: message ?? statusText,
+                    isRoutine: obj?.isRoutine ?? false,
+                    serverDetails: obj ?? responseText
+                });
             }
         } catch (ignored) {}
 
@@ -222,6 +221,14 @@ export class Exception {
     }
 }
 
+function safeParseJson(txt: string): PlainObject {
+    try {
+        return JSON.parse(txt);
+    } catch (ignored) {
+        return null;
+    }
+}
+
 export function isHoistException(src: unknown): src is HoistException {
     return src?.['isHoistException'];
 }

package/desktop/cmp/dash/canvas/DashCanvas.ts

@@ -18,7 +18,7 @@ import {
 import {dashCanvasAddViewButton} from '@xh/hoist/desktop/cmp/button/DashCanvasAddViewButton';
 import '@xh/hoist/desktop/register';
 import {Classes, overlay} from '@xh/hoist/kit/blueprint';
-import {TEST_ID} from '@xh/hoist/utils/js';
+import {consumeEvent, TEST_ID} from '@xh/hoist/utils/js';
 import classNames from 'classnames';
 import ReactGridLayout, {WidthProvider} from 'react-grid-layout';
 import {DashCanvasModel} from './DashCanvasModel';
@@ -125,6 +125,7 @@ const onContextMenu = (e, model) => {
             x = clientX + model.ref.current.scrollLeft,
             y = clientY + model.ref.current.scrollTop;
 
+        consumeEvent(e);
         showContextMenu(
             dashCanvasContextMenu({
                 dashCanvasModel: model,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "73.0.0-SNAPSHOT.1741714715638",
+  "version": "73.0.0-SNAPSHOT.1741790055811",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": "github:xh/hoist-react",
   "homepage": "https://xh.io",