Improvement/add unused vars rule

.husky/pre-commit

@@ -0,0 +1 @@
+npm run lint-no-fix && npm run check-types

.husky/pre-push

@@ -0,0 +1 @@
+npm run lint-no-fix && npm run check-types

eslint.config.mjs

@@ -20,6 +20,12 @@ export default tseslint.config(
       "@typescript-eslint/prefer-nullish-coalescing": "off",
       "@typescript-eslint/unified-signatures": "off",
       "@typescript-eslint/no-unnecessary-condition": "off",
+      "@typescript-eslint/no-unused-vars": ["error", {
+        vars: "all",
+        args: "after-used",
+        ignoreRestSiblings: true,
+      }],
+      "no-unused-private-class-members": "error",
       "@stylistic/quotes": ["error", "single", { avoidEscape: true }],
       "@stylistic/indent": ["error", 4],
       "@stylistic/semi": ["error", "always"],
@@ -39,4 +45,4 @@ export default tseslint.config(
       "webapp/control/**/*.gen.d.ts"
     ],
   },
-);
+);

package.json

@@ -32,7 +32,8 @@
     "lint": "npx eslint --fix && npm run check-types",
     "serve": "node mock.mjs",
     "start:webserver": "npx http-server dist/ -p 80",
-    "test": "npx ui5-test-runner"
+    "test": "npx ui5-test-runner",
+    "prepare": "husky"
   },
   "devDependencies": {
     "@stylistic/eslint-plugin": "^5.10.0",
@@ -47,6 +48,7 @@
     "dotenv": "^17.4.1",
     "eslint": "^10.2.0",
     "express": "^5.2.1",
+    "husky": "^9.1.7",
     "typescript-eslint": "^8.58.1",
     "ui5-test-runner": "^5.13.1"
   }

tsconfig.json

@@ -10,6 +10,8 @@
 		"noImplicitAny": true,
 		"strictNullChecks": true,
 		"strictPropertyInitialization": false,
+		"noUnusedLocals": true,
+		"noUnusedParameters": true,
 		"sourceMap": false,
 		"inlineSourceMap": false,
 		"declaration": false,

webapp/common/Formatters.ts

@@ -206,5 +206,5 @@ export function formatErrorDetails(status: string, errorMessage: string): string
 }
 
 export function isErrorStatus(status: string): boolean {
-    return status === SystemStatus.FAILED;
+    return (status as SystemStatus) === SystemStatus.FAILED;
 }