Fix error on WS client closure (#110)

* Fix error on WS client closure

* Add hawk catcher

* Add env files to ignore

* Remove hawk token from build args

* fix lint

* Run all files mutations tests only if any are changed in the package

Author: gohabereg

.github/actions/mutation-tests-all-files/action.yml

@@ -19,7 +19,20 @@ runs:
       - name: Setup environment
         uses: ./.github/actions/setup
 
+      - name: Get changed files
+        uses: tj-actions/[email protected]
+        id: changed-files
+        with:
+          files_yaml: |
+            src:
+              - 'src/**/*.ts'
+              - '!src/**/*.spec.ts'
+              - '!src/**/__mocks__/**'
+          separator: "','"
+          path: ${{ inputs.working-directory }}
+
       - name: Run mutation tests
+        if: steps.changed-files.outputs.src_any_changed == 'true'
         env:
           STRYKER_DASHBOARD_API_KEY: ${{ inputs.stryker_dashboard_api_key }}
         shell: bash

.gitignore

@@ -25,3 +25,6 @@ reports/
 *.tsbuildinfo
 
 .DS_Store
+
+# ENV
+**/.env

packages/ot-server/.env packages/ot-server/.env.examplepackages/ot-server/.env renamed to packages/ot-server/.env.example

@@ -1 +1,2 @@
 WSS_PORT=8080
+HAWK_TOKEN=

packages/ot-server/package.json

@@ -16,6 +16,7 @@
   "dependencies": {
     "@editorjs/collaboration-manager": "workspace:^",
     "@editorjs/model": "workspace:^",
+    "@hawk.so/nodejs": "^3.1.5",
     "dotenv": "^16.4.7",
     "ws": "^8.18.1"
   },

packages/ot-server/src/OTServer.ts

@@ -74,7 +74,8 @@ export class OTServer {
    * @param ws - client websocket
    */
   #onClose(ws: WebSocket): void {
-    const [documentId, documentClient] = this.#clients.entries().find(([, clients]) => clients.has(ws)) ?? [];
+    const [documentId, documentClient] = Array.from(this.#clients.entries())
+      .find(([, clients]) => clients.has(ws)) ?? [];
 
     if (documentId === undefined || documentClient === undefined) {
       return;

packages/ot-server/src/index.ts

@@ -1,12 +1,18 @@
 import { OTServer } from './OTServer.js';
 import { config } from 'dotenv';
+import HawkCatcher from '@hawk.so/nodejs';
+import process from 'process';
 
 config();
 
 /**
  * main function
  */
 function main(): void {
+  if (process.env.NODE_ENV === 'production' && process.env.HAWK_TOKEN !== undefined) {
+    HawkCatcher.init(process.env.HAWK_TOKEN);
+  }
+
   const otServer = new OTServer();
 
   otServer.start();

yarn.lock

@@ -1214,6 +1214,7 @@ __metadata:
   dependencies:
     "@editorjs/collaboration-manager": "workspace:^"
     "@editorjs/model": "workspace:^"
+    "@hawk.so/nodejs": "npm:^3.1.5"
     "@types/eslint": "npm:^9.6.1"
     "@types/jest": "npm:^29.5.14"
     "@types/ws": "npm:^8.18.1"
@@ -1732,6 +1733,26 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@hawk.so/nodejs@npm:^3.1.5":
+  version: 3.1.5
+  resolution: "@hawk.so/nodejs@npm:3.1.5"
+  dependencies:
+    "@hawk.so/types": "npm:^0.1.15"
+    axios: "npm:^0.21.1"
+    stack-trace: "npm:^0.0.10"
+  checksum: 9fcf55f6bec4ebbd3ff14a7be4145359dbf48c6c26d76e61245c9e9fcae5580cbbd891bb053d75b813131b0b1cb22445f64d7f5178f0414ae18880bf0043fb2b
+  languageName: node
+  linkType: hard
+
+"@hawk.so/types@npm:^0.1.15":
+  version: 0.1.28
+  resolution: "@hawk.so/types@npm:0.1.28"
+  dependencies:
+    "@types/mongodb": "npm:^3.5.34"
+  checksum: 6bc8fd78eaea5c4e8c59c4d9ff5028ea17e7d7d259798ad33955b3f735080f5b7ab3bf3800ec20ff5fe9feff05979d81359e14646ee4c4d1afc2cbfc596dc2c8
+  languageName: node
+  linkType: hard
+
 "@humanfs/core@npm:^0.19.1":
   version: 0.19.1
   resolution: "@humanfs/core@npm:0.19.1"
@@ -2682,6 +2703,15 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@types/bson@npm:*":
+  version: 4.0.5
+  resolution: "@types/bson@npm:4.0.5"
+  dependencies:
+    "@types/node": "npm:*"
+  checksum: b3e94a2832de82ea6d101fe2efcc65f32de131d25b22b1ff585cb74bc904fbcdb6cb976fc21adf608b8fcb43075e100693f5f321ffa170ff9a7847a3d1e8fdeb
+  languageName: node
+  linkType: hard
+
 "@types/eslint@npm:^8":
   version: 8.56.12
   resolution: "@types/eslint@npm:8.56.12"
@@ -2767,6 +2797,16 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@types/mongodb@npm:^3.5.34":
+  version: 3.6.20
+  resolution: "@types/mongodb@npm:3.6.20"
+  dependencies:
+    "@types/bson": "npm:*"
+    "@types/node": "npm:*"
+  checksum: b85770dfa009cd919f680bdf7e152f9cab21d0ea6aab3d3a4133a9e78a1a2dfd966ed1503594191051985cb7157df6e830740d8bdc2b45d4b31050849f84f33d
+  languageName: node
+  linkType: hard
+
 "@types/node@npm:*":
   version: 22.14.0
   resolution: "@types/node@npm:22.14.0"
@@ -3598,6 +3638,15 @@ __metadata:
   languageName: node
   linkType: hard
 
+"axios@npm:^0.21.1":
+  version: 0.21.4
+  resolution: "axios@npm:0.21.4"
+  dependencies:
+    follow-redirects: "npm:^1.14.0"
+  checksum: da644592cb6f8f9f8c64fdabd7e1396d6769d7a4c1ea5f8ae8beb5c2eb90a823e3a574352b0b934ac62edc762c0f52647753dc54f7d07279127a7e5c4cd20272
+  languageName: node
+  linkType: hard
+
 "babel-jest@npm:^29.7.0":
   version: 29.7.0
   resolution: "babel-jest@npm:29.7.0"
@@ -5574,6 +5623,16 @@ __metadata:
   languageName: node
   linkType: hard
 
+"follow-redirects@npm:^1.14.0":
+  version: 1.15.9
+  resolution: "follow-redirects@npm:1.15.9"
+  peerDependenciesMeta:
+    debug:
+      optional: true
+  checksum: e3ab42d1097e90d28b913903841e6779eb969b62a64706a3eb983e894a5db000fbd89296f45f08885a0e54cd558ef62e81be1165da9be25a6c44920da10f424c
+  languageName: node
+  linkType: hard
+
 "for-each@npm:^0.3.3, for-each@npm:^0.3.5":
   version: 0.3.5
   resolution: "for-each@npm:0.3.5"
@@ -9286,6 +9345,13 @@ __metadata:
   languageName: node
   linkType: hard
 
+"stack-trace@npm:^0.0.10":
+  version: 0.0.10
+  resolution: "stack-trace@npm:0.0.10"
+  checksum: 7bd633f0e9ac46e81a0b0fe6538482c1d77031959cf94478228731709db4672fbbed59176f5b9a9fd89fec656b5dae03d084ef2d1b0c4c2f5683e05f2dbb1405
+  languageName: node
+  linkType: hard
+
 "stack-utils@npm:^2.0.3":
   version: 2.0.6
   resolution: "stack-utils@npm:2.0.6"