fix: Catch errors when emitting events to prevent breaking entire layout

- Added a try/catch around all events that are emitted from Golden Layout
- Added @deephaven/log to golden layout so we can log the error
- Tested using the steps in https://github.com/deephaven/deephaven-js-plugins/issues/26
- Fixes deephaven#1352

packages/golden-layout/package.json

@@ -11,6 +11,7 @@
   "type": "module",
   "dependencies": {
     "@deephaven/components": "file:../components",
+    "@deephaven/log": "file:../log",
     "jquery": "^3.6.0"
   },
   "peerDependencies": {

packages/golden-layout/src/utils/EventEmitter.ts

@@ -1,3 +1,7 @@
+import Log from '@deephaven/log';
+
+const log = Log.module('EventEmitter');
+
 /**
  * A generic and very fast EventEmitter
  * implementation. On top of emitting the
@@ -65,7 +69,11 @@ class EventEmitter {
     if (subs) {
       for (let i = 0; i < subs.length; i++) {
         const ctx = subs[i].ctx || {};
-        subs[i].fn.apply(ctx, args);
+        try {
+          subs[i].fn.apply(ctx, args);
+        } catch (e) {
+          log.error('Error while emitting event:', e);
+        }
       }
     }
 
@@ -75,7 +83,11 @@ class EventEmitter {
 
     for (let i = 0; i < allEventSubs.length; i++) {
       const ctx = allEventSubs[i].ctx || {};
-      allEventSubs[i].fn.apply(ctx, args);
+      try {
+        allEventSubs[i].fn.apply(ctx, args);
+      } catch (e) {
+        log.error('Error while emitting event to allEventSubs:', e);
+      }
     }
   }
 

packages/golden-layout/tsconfig.json

@@ -4,13 +4,7 @@
     "rootDir": "src/",
     "outDir": "dist/"
   },
-  "include": [
-    "src/**/*.ts",
-    "src/**/*.tsx",
-    "src/**/*.js",
-    "src/**/*.jsx"
-  ],
-  "exclude": [
-    "node_modules"
-  ]
-}
+  "include": ["src/**/*.ts", "src/**/*.tsx", "src/**/*.js", "src/**/*.jsx"],
+  "exclude": ["node_modules"],
+  "references": [{ "path": "../components" }, { "path": "../log" }]
+}