feat(node): drop support for node 16 (#71)

BREAKING CHANGE: Please use Node 18 or higher

Author: nicojs

.eslintrc

@@ -13,7 +13,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                node-version: [16.x, 20.x]
+                node-version: [18.x, 22.x]
         steps:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v2

.github/workflows/ci.yml

@@ -1,4 +1 @@
-{
-  "singleQuote": true,
-  "printWidth": 150
-}
+{ "singleQuote": true }

.prettierrc

@@ -37,6 +37,7 @@
       "args": [
         "--no-timeout",
         "--colors",
+        "--bail",
         "${workspaceFolder}/dist/test/helpers/**/*.js",
         "${workspaceFolder}/dist/test/integration/**/*.js"
       ],

.vscode/launch.json

@@ -1,13 +1,14 @@
 {
   "editor.codeActionsOnSave": {
-    "source.fixAll.eslint": true
+    "source.fixAll.eslint": "explicit"
   },
   "eslint.validate": [
     "javascript",
     "typescript"
   ],
   "typescript.tsdk": "node_modules\\typescript\\lib",
   "cSpell.words": [
+    "tseslint",
     "Typesafe"
   ]
 }

.vscode/settings.json

@@ -0,0 +1,75 @@
+import eslint from '@eslint/js';
+import tseslint from 'typescript-eslint';
+import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended';
+import pluginChaiFriendly from 'eslint-plugin-chai-friendly';
+
+export default tseslint.config(
+  eslint.configs.recommended,
+  tseslint.configs.recommendedTypeChecked,
+  eslintPluginPrettierRecommended,
+  {
+    languageOptions: {
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+    rules: {
+      '@typescript-eslint/no-explicit-any': ['off'],
+      '@typescript-eslint/no-use-before-define': ['off'],
+      '@typescript-eslint/no-empty-object-type': ['off'],
+      '@typescript-eslint/no-unsafe-function-type': 'off',
+      '@typescript-eslint/no-restricted-types': [
+        'error',
+        {
+          types: {
+            String: {
+              message: 'Use string instead',
+              fixWith: 'string',
+            },
+            Boolean: {
+              message: 'Use boolean instead',
+              fixWith: 'boolean',
+            },
+            Number: {
+              message: 'Use number instead',
+              fixWith: 'number',
+            },
+            Symbol: {
+              message: 'Use symbol instead',
+              fixWith: 'symbol',
+            },
+
+            // object typing
+            Object: {
+              message:
+                'The `Object` type actually means "any non-nullish value", so it is marginally better than `unknown`.\n- If you want a type meaning "any object", you probably want `Record<string, unknown>` instead.\n- If you want a type meaning "any value", you probably want `unknown` instead.',
+            },
+            object: {
+              message:
+                'The `object` type is currently hard to use ([see this issue](https://github.com/microsoft/TypeScript/issues/21732)).\nConsider using `Record<string, unknown>` instead, as it allows you to more easily inspect and use the keys.',
+            },
+          },
+        },
+      ],
+    },
+  },
+  {
+    plugins: { 'chai-friendly': pluginChaiFriendly },
+    files: ['test/**/*.@(ts|js|mts|cts)'],
+    rules: {
+      'no-unused-expressions': 'off', // disable original rule
+      '@typescript-eslint/no-unused-expressions': 'off', // disable original rule
+      'chai-friendly/no-unused-expressions': 'error',
+    },
+  },
+  {
+    ignores: [
+      'testResources/**/*.ts',
+      'reports',
+      '.nyc_output',
+      'dist',
+      'eslint.config.js',
+    ],
+  },
+);