refactor: renable standard JS linter rules

[poolifier.git] / src / worker / abstract-worker.ts

diff --git a/src/worker/abstract-worker.ts b/src/worker/abstract-worker.ts
index 81523d8ca964f5af19308c65e99f36de618699a9..2359758dc72c2a362b60e513890664731fdc7fea 100644 (file)
--- a/src/worker/abstract-worker.ts
+++ b/src/worker/abstract-worker.ts
@@ -277,10 +277,8 @@ export abstract class AbstractWorker<
           'Cannot set the default task function to a non-existing task function'
         )
       }
-      this.taskFunctions.set(
-        DEFAULT_TASK_NAME,
-        this.taskFunctions.get(name) as TaskFunction<Data, Response>
-      )
+      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+      this.taskFunctions.set(DEFAULT_TASK_NAME, this.taskFunctions.get(name)!)
       this.sendTaskFunctionNamesToMainWorker()
       return { status: true }
     } catch (error) {
@@ -328,19 +326,22 @@ export abstract class AbstractWorker<
     switch (taskFunctionOperation) {
       case 'add':
         response = this.addTaskFunction(
-          taskFunctionName as string,
-          // eslint-disable-next-line @typescript-eslint/no-implied-eval, no-new-func
-          new Function(`return ${taskFunction as string}`)() as TaskFunction<
+          // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+          taskFunctionName!,
+          // eslint-disable-next-line @typescript-eslint/no-implied-eval, no-new-func, @typescript-eslint/no-non-null-assertion
+          new Function(`return ${taskFunction!}`)() as TaskFunction<
           Data,
           Response
           >
         )
         break
       case 'remove':
-        response = this.removeTaskFunction(taskFunctionName as string)
+        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+        response = this.removeTaskFunction(taskFunctionName!)
         break
       case 'default':
-        response = this.setDefaultTaskFunction(taskFunctionName as string)
+        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+        response = this.setDefaultTaskFunction(taskFunctionName!)
         break
       default:
         response = { status: false, error: new Error('Unknown task operation') }
@@ -353,7 +354,8 @@ export abstract class AbstractWorker<
       ...(!response.status &&
         response?.error != null && {
         workerError: {
-          name: taskFunctionName as string,
+          // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+          name: taskFunctionName!,
           message: this.handleError(response.error as Error | string)
         }
       })
@@ -488,8 +490,10 @@ export abstract class AbstractWorker<
     if (!this.taskFunctions.has(taskFunctionName)) {
       this.sendToMainWorker({
         workerError: {
-          name: name as string,
-          message: `Task function '${name as string}' not found`,
+          // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+          name: name!,
+          // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+          message: `Task function '${name!}' not found`,
           data
         },
         taskId
@@ -527,7 +531,8 @@ export abstract class AbstractWorker<
     } catch (error) {
       this.sendToMainWorker({
         workerError: {
-          name: name as string,
+          // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+          name: name!,
           message: this.handleError(error as Error | string),
           data
         },
@@ -563,7 +568,8 @@ export abstract class AbstractWorker<
       .catch(error => {
         this.sendToMainWorker({
           workerError: {
-            name: name as string,
+            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+            name: name!,
             message: this.handleError(error as Error | string),
             data
           },