Fixed webworker startup bc elkjs hidden dependency (#78)

Fixes eclipse-glsp/glsp#1018
eclipse-glsp · Apr 15, 2024 · 9634f64 · 9634f64
1 parent 4e5a7a9
commit 9634f64
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -16,3 +16,5 @@ report.xml
 
 wf-glsp-server-node.js
 wf-glsp-server-node.js.map
+wf-glsp-server-webworker.js
+wf-glsp-server-webworker.js.map
diff --git a/examples/workflow-server-bundled/package.json b/examples/workflow-server-bundled/package.json
@@ -41,7 +41,7 @@
     "wf-glsp-server-node.js.map"
   ],
   "scripts": {
-    "clean": "rimraf wf-glsp-server-node.js wf-glsp-server-node.js.map",
+    "clean": "rimraf wf-glsp-server-node.js wf-glsp-server-node.js.map wf-glsp-server-webworker.js wf-glsp-server-webworker.js.map",
     "start": "node --enable-source-maps ./wf-glsp-server-node.js --port 5007",
     "start:websocket": "node --enable-source-maps ./wf-glsp-server-node.js -w --port 8081",
     "watch": "tsc -w"

diff --git a/packages/layout-elk/src/di.config.ts b/packages/layout-elk/src/di.config.ts
@@ -82,7 +82,11 @@ export function configureELKLayoutModule(options: ElkModuleOptions): ContainerMo
         const elkFactory: ElkFactory = () =>
             new ElkConstructor({
                 algorithms: options.algorithms,
-                defaultLayoutOptions: options.defaultLayoutOptions
+                defaultLayoutOptions: options.defaultLayoutOptions,
+                // The node implementation relied on elkjs' `FakeWorker` to set the `workerFactory`.
+                // However, since the required file is dynamically loaded and not available in a web-worker context,
+                // it needs to be mocked manually.
+                workerFactory: () => ({ postMessage: () => {} }) as unknown as Worker
             });
 
         bind(ElkFactory).toConstantValue(elkFactory);