Fix linting/syntax

Author: magicus

.eslintrc.json

@@ -9,16 +9,70 @@
     "@typescript-eslint"
   ],
   "rules": {
-    "@typescript-eslint/naming-convention": "warn",
+    // Whitespace
+    "indent": ["error", 2],
+    "max-len": ["warn", { "code": 120 }],
+    "array-bracket-newline": ["warn", "consistent"],
+    "array-element-newline": ["warn", "consistent"],
+    "function-paren-newline": ["warn", "consistent"],
+    "object-curly-spacing": ["warn", "always", { "objectsInObjects": false }],
+    "function-call-argument-newline": "off",
+    "lines-between-class-members": "off",
+    "padded-blocks": "off",
+    "space-before-function-paren": ["warn", {"anonymous": "always", "named": "never", "asyncArrow": "always"}],
+
+    // Formatting
+    "curly": "error",
+    "arrow-parens": ["warn", "as-needed"],
+    "dot-location": ["warn", "property"],
+    "quote-props": "off",
+    "quotes": ["warn", "single"],
+    "semi": "warn",
     "@typescript-eslint/semi": "warn",
-    "curly": "warn",
+
+    // Style and pitfalls
+    "class-methods-use-this": "off",
     "eqeqeq": "warn",
+    "func-style": ["warn", "declaration"],
+    "multiline-ternary": "off",
+    "no-console": "off",
+    "no-else-return": "off",
+    "no-invalid-this": "off",
+    "no-magic-numbers": ["warn", { "ignore": [0, 1] }],
+    "no-negated-condition": "off",
+    "no-promise-executor-return": "off",
+    "no-ternary": "off",
     "no-throw-literal": "warn",
-    "semi": "off"
+    "no-undef-init": "off",
+    "no-undefined": "off",
+    "no-underscore-dangle": "off",
+    "no-unused-vars": ["warn", { "args": "none" }],
+    "no-use-before-define": "off",
+    "one-var": "off",
+    "prefer-destructuring": "off",
+    "prefer-template": "off",
+    "sort-keys": "off",
+    "@typescript-eslint/naming-convention": "warn",
+
+    // Stay out of my comments
+    "capitalized-comments": "off",
+    "line-comment-position": "off",
+    "lines-around-comment": "off",
+    "multiline-comment-style": "off",
+    "no-inline-comments": "off",
+    "no-warning-comments": "off",
+
+    // No limits
+    "max-classes-per-file":  "off",
+    "max-lines-per-function": "off",
+    "max-lines": "off",
+    "max-params": "off",
+    "max-statements": "off"
   },
   "ignorePatterns": [
     "out",
     "dist",
     "**/*.d.ts"
+ //   "webpack.config.js"
   ]
-}
+}

tsconfig.json

@@ -7,11 +7,13 @@
     ],
     "sourceMap": true,
     "rootDir": "src",
-    "strict": true   /* enable all strict type-checking options */
-    /* Additional Checks */
-    // "noImplicitReturns": true, /* Report error when not all code paths in function return a value. */
-    // "noFallthroughCasesInSwitch": true, /* Report errors for fallthrough cases in switch statement. */
-    // "noUnusedParameters": true,  /* Report errors on unused parameters. */
+    "strict": true,
+    "noUnusedLocals": true,
+    "noImplicitReturns": true,
+    "noFallthroughCasesInSwitch": true,
+    "noUncheckedIndexedAccess": true,
+    "esModuleInterop": true,
+    "forceConsistentCasingInFileNames": true
   },
   "exclude": [
     "node_modules",

webpack.config.js

@@ -1,4 +1,6 @@
 //@ts-check
+/* eslint-disable max-len */
+
 
 'use strict';
 
@@ -10,7 +12,7 @@ const path = require('path');
 /** @type WebpackConfig */
 const extensionConfig = {
   target: 'node', // vscode extensions run in a Node.js-context 📖 -> https://webpack.js.org/configuration/node/
-	mode: 'none', // this leaves the source code as close as possible to the original (when packaging we set this to 'production')
+  mode: 'none', // this leaves the source code as close as possible to the original (when packaging we set this to 'production')
 
   entry: './src/extension.ts', // the entry point of this extension, 📖 -> https://webpack.js.org/configuration/entry-context/
   output: {