chore: renew NodeJs linting

Fix found errors with new linting system

Author: aaltat

.gitattributes

@@ -1,2 +1,3 @@
+* text=auto eol=lf
 *_pb2.py        eol=lf
 *_pb*.[tj]s        eol=lf

node/.prettierrc.js

@@ -1,7 +1,8 @@
 module.exports = {
     semi: true,
-    trailingComma: "all",
+    trailingComma: 'all',
     singleQuote: true,
     printWidth: 120,
-    tabWidth: 4
-};
+    tabWidth: 4,
+    endOfLine: 'lf',
+};

node/build.testapp.js

@@ -1,38 +1,37 @@
 #!/usr/bin/env node
 
 const esbuild = require('esbuild');
-const { nodeExternalsPlugin } = require('esbuild-node-externals');
 const fs = require('fs');
-const path = require('path')
+const path = require('path');
 
-const distPath = path.resolve(__dirname, './dynamic-test-app/dist')
+const distPath = path.resolve(__dirname, './dynamic-test-app/dist');
 
 if (!fs.existsSync(distPath)) {
-  fs.mkdirSync(distPath);
+    fs.mkdirSync(distPath);
 }
 
-const indexHtmlSource = path.resolve(__dirname, './dynamic-test-app/static/index.html')
-const indexHtmlTarget = path.resolve(distPath, './index.html')
-fs.copyFileSync(indexHtmlSource,  indexHtmlTarget)
+const indexHtmlSource = path.resolve(__dirname, './dynamic-test-app/static/index.html');
+const indexHtmlTarget = path.resolve(distPath, './index.html');
+fs.copyFileSync(indexHtmlSource, indexHtmlTarget);
 
 /* Build testApp frontend */
-esbuild.build(
-  {
-    logLevel: "info",
-    entryPoints: ['./node/dynamic-test-app/src/index.tsx'],
-    bundle: true,
-    platform: "browser",
-    outfile: "./node/dynamic-test-app/dist/index.js",
-  }
-).catch(() => process.exit(1));
+esbuild
+    .build({
+        logLevel: 'info',
+        entryPoints: ['./node/dynamic-test-app/src/index.tsx'],
+        bundle: true,
+        platform: 'browser',
+        outfile: './node/dynamic-test-app/dist/index.js',
+    })
+    .catch(() => process.exit(1));
 /* Build testApp backend */
-esbuild.build(
-  {
-    logLevel: "info",
-    entryPoints: ["./node/dynamic-test-app/src/server.ts"],
-    bundle: true,
-    platform: "node",
-    outfile: "./node/dynamic-test-app/dist/server.js",
-    /* plugins: [nodeExternalsPlugin()], */
-  }
-).catch(() => process.exit(1));
+esbuild
+    .build({
+        logLevel: 'info',
+        entryPoints: ['./node/dynamic-test-app/src/server.ts'],
+        bundle: true,
+        platform: 'node',
+        outfile: './node/dynamic-test-app/dist/server.js',
+        /* plugins: [nodeExternalsPlugin()], */
+    })
+    .catch(() => process.exit(1));

node/build.wrapper.js

@@ -2,20 +2,20 @@
 const esbuild = require('esbuild');
 const { nodeExternalsPlugin } = require('esbuild-node-externals');
 
-esbuild.build(
-  {
-    logLevel: "info",
-    entryPoints: ["./node/playwright-wrapper/index.ts"],
-    bundle: true,
-    platform: "node",
-    outfile: "./Browser/wrapper/index.js",
-    plugins: [nodeExternalsPlugin({
-      // Allow UUID to be bundled instead of external
-      // Needed when building with pkg
-      allowList: ['uuid']
-    })],
-    external: [
-      'playwright-core/*',
-    ],
-  }
-).catch(() => process.exit(1));
+esbuild
+    .build({
+        logLevel: 'info',
+        entryPoints: ['./node/playwright-wrapper/index.ts'],
+        bundle: true,
+        platform: 'node',
+        outfile: './Browser/wrapper/index.js',
+        plugins: [
+            nodeExternalsPlugin({
+                // Allow UUID to be bundled instead of external
+                // Needed when building with pkg
+                allowList: ['uuid'],
+            }),
+        ],
+        external: ['playwright-core/*'],
+    })
+    .catch(() => process.exit(1));

node/dynamic-test-app/src/app.tsx

@@ -1,8 +1,9 @@
+import React from 'react';
 import { HashRouter, Route, Routes } from 'react-router-dom';
+
 import DragGame from './draggame';
 import EventsPage from './events';
 import Login from './login';
-import React from 'react';
 
 const App: React.FC = () => {
     return (

node/dynamic-test-app/src/draggame.tsx

@@ -1,8 +1,8 @@
 // Created by Copilot
 // This code is a simple drag-and-drop game using React and react-dnd.
+import React, { useCallback, useEffect, useState } from 'react';
 import { DndProvider, useDrag, useDrop } from 'react-dnd';
 import { HTML5Backend } from 'react-dnd-html5-backend';
-import React, { useCallback, useEffect, useState } from 'react';
 
 const ItemTypes = {
     CIRCLE: 'circle',

node/dynamic-test-app/src/events.tsx

@@ -150,7 +150,9 @@ function EventLogger() {
         const handler = (e: Event) => {
             const data = buildEventSummary(e);
             // Show both wall-clock and monotonic in the visual log
-            pushVisual(`[${data.time} | ${Math.round(data.mono as number)}ms] ${data.type} on #${data.targetId}`);
+            pushVisual(
+                `[${String(data.time)} | ${Math.round(data.mono as number)}ms] ${String(data.type)} on #${String(data.targetId)}`,
+            );
             // Persist full JSON line
             pushText(data);
         };

node/dynamic-test-app/src/index.tsx

@@ -1,5 +1,6 @@
 import * as React from 'react';
 import { createRoot } from 'react-dom/client';
+
 import App from './app';
 
 const container = document.getElementById('root');

node/dynamic-test-app/src/login.tsx

@@ -76,16 +76,16 @@ function fileUploaded(uploadResultElement: React.RefObject<HTMLElement>, event:
                 fileNames = files[i].name;
             }
         }
-        uploadResultElement.current!.innerHTML = fileNames;
+        uploadResultElement.current.innerHTML = fileNames;
     } else {
-        uploadResultElement.current!.innerHTML = 'no uploaded file';
+        uploadResultElement.current.innerHTML = 'no uploaded file';
     }
 }
 
 function testPrompt(promptResultElement: React.RefObject<HTMLElement>) {
     const input = prompt('Enter a value');
-    if (input) promptResultElement.current!.innerHTML = input;
-    else promptResultElement.current!.innerHTML = 'prompt_not_filled';
+    if (input) promptResultElement.current.innerHTML = input;
+    else promptResultElement.current.innerHTML = 'prompt_not_filled';
 }
 
 export default function Site() {
@@ -117,7 +117,9 @@ export default function Site() {
         mouseButton.current!.innerHTML = '';
         const mouseButtons = ['left', 'middle', 'right'];
         mouseButton.current!.innerHTML = mouseButtons[e.button];
-        console.log(`Mouse button: ${mouseButtons[e.button]}, X: ${e.pageX}, Y: ${e.pageY}, Time: ${new Date()}`);
+        console.log(
+            `Mouse button: ${mouseButtons[e.button]}, X: ${e.pageX}, Y: ${e.pageY}, Time: ${new Date().toISOString()}`,
+        );
         altKey.current!.innerHTML = e.altKey.toString();
         shiftKey.current!.innerHTML = e.shiftKey.toString();
         ctrlKey.current!.innerHTML = e.ctrlKey.toString();
@@ -217,10 +219,10 @@ export default function Site() {
                 <button id="pops_up" onClick={popup}>
                     Pops up a window
                 </button>
-                <button id="delayed_request" onClick={delayedRequest}>
+                <button id="delayed_request" onClick={() => void delayedRequest()}>
                     Fires a request in 200ms
                 </button>
-                <button id="delayed_request_big" onClick={delayedRequestBig}>
+                <button id="delayed_request_big" onClick={() => void delayedRequestBig()}>
                     Fires a big request in 250ms
                 </button>
                 <button id="alerts" onClick={() => alert('Am an alert')}>

node/dynamic-test-app/src/server.ts

@@ -1,11 +1,11 @@
+import { Command } from 'commander';
 import * as express from 'express';
 import * as fs from 'fs';
 import * as https from 'https';
 import * as path from 'path';
-import { Command } from 'commander';
 
 const app = express.default();
-// eslint-disable-next-line
+
 app.use(express.json());
 
 const program = new Command();
@@ -173,6 +173,7 @@ if (tls || mutualTls) {
     };
 
     https
+        // eslint-disable-next-line @typescript-eslint/no-misused-promises
         .createServer(serverOptions, app)
         .listen(port, () => console.log(`Successfully started server on https://localhost:${port}`));
 } else {