linting fix

moleculerjs · Apr 27, 2024 · e059b8e · e059b8e
1 parent 25e849b
commit e059b8e
Show file tree

Hide file tree

Showing 12 changed files with 28 additions and 22 deletions.
diff --git a/eslint.config.js b/eslint.config.js
@@ -1,13 +1,13 @@
 const js = require("@eslint/js");
 const globals = require("globals");
-const pluginSecurity = require('eslint-plugin-security');
+const pluginSecurity = require("eslint-plugin-security");
+const eslintPluginPrettierRecommended = require("eslint-plugin-prettier/recommended");
 
 /** @type {import('eslint').Linter.FlatConfig[]} */
 module.exports = [
 	js.configs.recommended,
 	pluginSecurity.configs.recommended,
-	//"plugin:security/recommended-legacy",
-	//"plugin:prettier/recommended",
+	eslintPluginPrettierRecommended,
 	{
 		files: ["**/*.js", "**/*.mjs"],
 		languageOptions: {
@@ -39,28 +39,32 @@ module.exports = [
 			"no-process-exit": ["off"],
 			"node/no-unpublished-require": 0
 		},
-		ignores: ["benchmark/test.js", "test/typescript/hello-world/out/*.js"],
-	}, {
+		ignores: ["benchmark/test.js", "test/typescript/hello-world/out/*.js"]
+	},
+	{
 		files: ["test/typescript/hello-world/out/*.js"],
 		languageOptions: {
 			globals: {
 				exports: "readonly",
 				require: "readonly",
 				__dirname: "readonly",
 				console: "readonly",
-				process: "readonly",
+				process: "readonly"
 			}
 		},
 		rules: {
-			"no-unused-vars": ["off"]
+			"no-unused-vars": ["off"],
+			"prettier/prettier": ["off"]
 		}
-	}, {
+	},
+	{
 		files: ["test/**/*.js"],
 		rules: {
 			"no-console": ["off"],
 			"no-unused-vars": ["off"]
 		}
-	}, {
+	},
+	{
 		files: ["dev/**/*.js", "benchmark/**/*.js", "examples/**/*.js"],
 		rules: {
 			"no-console": ["off"],

diff --git a/src/cachers/redis.js b/src/cachers/redis.js
@@ -117,7 +117,7 @@ class RedisCacher extends BaseCacher {
 			let Redlock;
 			try {
 				Redlock = require("redlock");
-			} catch (err) {
+			} catch {
 				Redlock = null;
 			}
 			if (Redlock != null) {

diff --git a/src/errors.js b/src/errors.js
@@ -556,7 +556,7 @@ class Regenerator {
 	 *
 	 * @memberof Regenerator
 	 */
-	extractPlainError(plainErr, payload) {
+	extractPlainError(plainErr /*, payload*/) {
 		return {
 			name: plainErr.name,
 			message: plainErr.message,
@@ -578,7 +578,7 @@ class Regenerator {
 	 *
 	 * @memberof Regenerator
 	 */
-	restoreCustomError(plainError, payload) {
+	restoreCustomError(/*plainError, payload*/) {
 		return undefined;
 	}
 

diff --git a/src/health.js b/src/health.js
@@ -57,7 +57,7 @@ const getMemoryInfo = () => {
 const getUserInfo = () => {
 	try {
 		return os.userInfo();
-	} catch (e) {
+	} catch {
 		return {};
 	}
 };

diff --git a/src/lock.js b/src/lock.js
@@ -18,7 +18,7 @@ class Lock {
 		this.locked = new Map();
 	}
 
-	acquire(key, ttl) {
+	acquire(key /*, ttl*/) {
 		let locked = this.locked.get(key);
 		if (!locked) {
 			// not locked

diff --git a/src/metrics/commons.js b/src/metrics/commons.js
@@ -15,7 +15,7 @@ let v8;
 // Load `v8` module for heap metrics.
 try {
 	v8 = require("v8");
-} catch (e) {
+} catch {
 	// silent
 }
 
@@ -476,7 +476,7 @@ function updateCommonMetrics() {
 						this.set(METRIC.OS_CPU_INFO_TIMES_USER, cpu.times.user, { index });
 						this.set(METRIC.OS_CPU_INFO_TIMES_SYS, cpu.times.sys, { index });
 					});
-				} catch (err) {
+				} catch {
 					// silent
 				}
 			})
@@ -497,7 +497,7 @@ function updateCommonMetrics() {
 function getUserInfo() {
 	try {
 		return os.userInfo();
-	} catch (e) {
+	} catch {
 		/* istanbul ignore next */
 		return {};
 	}

diff --git a/src/middlewares/circuit-breaker.js b/src/middlewares/circuit-breaker.js
@@ -4,6 +4,8 @@
  * MIT Licensed
  */
 
+/* eslint-disable no-unused-vars */
+
 "use strict";
 
 /**

diff --git a/src/middlewares/debugging/action-logger.js b/src/middlewares/debugging/action-logger.js
@@ -47,7 +47,7 @@ module.exports = function ActionLoggerMiddleware(opts) {
 					payload instanceof Error ? Object.getOwnPropertyNames(payload) : null,
 					4
 				);
-			} catch (err) {
+			} catch {
 				data = JSON.stringify(
 					safetyObject(payload),
 					payload instanceof Error ? Object.getOwnPropertyNames(payload) : null,

diff --git a/src/middlewares/debugging/transit-logger.js b/src/middlewares/debugging/transit-logger.js
@@ -42,7 +42,7 @@ module.exports = function TransitLoggerMiddleware(opts) {
 				payload instanceof Error ? Object.getOwnPropertyNames(payload) : null,
 				4
 			);
-		} catch (err) {
+		} catch {
 			data = JSON.stringify(
 				safetyObject(payload),
 				payload instanceof Error ? Object.getOwnPropertyNames(payload) : null,

diff --git a/src/tracing/exporters/datadog-simple.js b/src/tracing/exporters/datadog-simple.js
@@ -117,7 +117,7 @@ class DatadogSimpleTraceExporter extends BaseTraceExporter {
 
 		try {
 			return parseInt(str.substring(0, 8), 16);
-		} catch (err) {
+		} catch {
 			this.logger.warn(`Unable to convert '${str}' to number.`);
 			return null;
 		}

diff --git a/src/transporters/amqp10.js b/src/transporters/amqp10.js
@@ -617,7 +617,7 @@ class Amqp10Transporter extends Transporter {
 	 *
 	 * @returns {Promise}
 	 */
-	send(topic, data, { balanced, packet }) {
+	send(/*topic, data, { balanced, packet }*/) {
 		// Doesn't used
 		return this.broker.Promise.resolve();
 	}

diff --git a/src/transporters/kafka.js b/src/transporters/kafka.js
@@ -64,7 +64,7 @@ class KafkaTransporter extends Transporter {
 							: null,
 					logLevel: 1,
 					logCreator:
-						logLevel =>
+						(/*logLevel*/) =>
 						({ level, log }) => {
 							const { message, ...extra } = log;
 							if (log.error == "Topic creation errors") return;