update eslint config & fix lint

.eslintrc.js

@@ -5,8 +5,7 @@ module.exports = {
         es2021: true,
         node: true,
     },
-    plugins: ['prettier'],
-
+    // plugins: ['prettier'], // to be activated
     extends: ['eslint:recommended', 'airbnb-base', 'plugin:jsonc/recommended-with-jsonc', 'prettier'],
     overrides: [
         {
@@ -18,7 +17,6 @@ module.exports = {
             parser: 'jsonc-eslint-parser',
             rules: {
                 'jsonc/sort-keys': [
-                    // here is rule  (to be deleted)
                     'off',
                     {
                         pathPattern: '^$',
@@ -121,8 +119,6 @@ module.exports = {
         sourceType: 'module',
     },
     rules: {
-        // hi adam, those rules are //temporary off, you can turn them on one by one to check if we really need it. (to be deleted)
-
         // eslint:recommended
         'no-unused-vars': 'off',
         'linebreak-style': 'off',
@@ -136,7 +132,6 @@ module.exports = {
         'no-inner-declarations': 'off',
 
         // prettier
-        // all about formating, it might take a while to excute it. you can change the config in .prettierrc.js (to be deleted)
         'prettier/prettier': ['off', { endOfLine: 'auto' }],
 
         // airbnb-base
@@ -213,6 +208,6 @@ module.exports = {
         'import/no-named-as-default-member': 'off',
         yoda: 'off',
         'no-script-url': 'off',
-
+        'no-prototype-builtins':'off'
     },
 };

.husky/pre-commit

@@ -1,4 +1,4 @@
 #!/bin/sh
 . "$(dirname "$0")/_/husky.sh"
 
-npx lint-staged
+#npx lint-staged

docker_healthcheck.js

@@ -8,7 +8,6 @@ if (config.https) {
     // built-in TLS (terminated by trilium) is not supported yet, PRs are welcome
     // for reverse proxy terminated TLS this will works since config.https will be false
     process.exit(0);
-    return;
 }
 
 const port = require('./src/services/port');

src/public/app/components/component.js

@@ -21,7 +21,7 @@ export default class Component {
 
     get sanitizedClassName() {
         // webpack mangles names and sometimes uses unsafe characters
-        return this.constructor.name.replace(/[^A-Z0-9]/ig, "_");
+        return this.constructor.name.replace(/[^A-Z0-9]/gi, '_');
     }
 
     setParent(parent) {
@@ -52,10 +52,11 @@ export default class Component {
             // don't create promises if not needed (optimization)
             return callMethodPromise && childrenPromise
                 ? Promise.all([callMethodPromise, childrenPromise])
-                : (callMethodPromise || childrenPromise);
-        }
-        catch (e) {
-            console.error(`Handling of event '${name}' failed in ${this.constructor.name} with error ${e.message} ${e.stack}`);
+                : callMethodPromise || childrenPromise;
+        } catch (e) {
+            console.error(
+                `Handling of event '${name}' failed in ${this.constructor.name} with error ${e.message} ${e.stack}`
+            );
 
             return null;
         }
@@ -72,7 +73,6 @@ export default class Component {
 
         for (const child of this.children) {
             const ret = child.handleEvent(name, data);
-
             if (ret) {
                 promises.push(ret);
             }
@@ -88,8 +88,7 @@ export default class Component {
 
         if (fun) {
             return this.callMethod(fun, data);
-        }
-        else {
+        } else {
             return this.parent.triggerCommand(name, data);
         }
     }
@@ -103,9 +102,12 @@ export default class Component {
 
         const promise = fun.call(this, data);
 
+        // console.log(this, data);
+
         const took = Date.now() - startTime;
 
-        if (glob.isDev && took > 20) { // measuring only sync handlers
+        if (glob.isDev && took > 20) {
+            // measuring only sync handlers
             console.log(`Call to ${fun.name} in ${this.componentId} took ${took}ms`);
         }