Update linting setup

Author: jdalton

.config/tsconfig.base.json

@@ -27,6 +27,7 @@
     "noUncheckedIndexedAccess": true,
 
     /* Additional checks */
+    "importsNotUsedAsValues": "remove",
     "noUnusedLocals": true,
     "noUnusedParameters": true
   }

.eslintignore

@@ -0,0 +1,15 @@
+{
+  "settings": {},
+  "rules": {
+    "consistent-function-scoping": "allow",
+    "no-async-endpoint-handlers": "allow",
+    "no-await-in-loop": "allow",
+    "no-control-regex": "allow",
+    "no-empty-file": "allow",
+    "no-extraneous-class": "allow",
+    "no-new": "allow",
+    "no-new-array": "allow",
+    "no-this-alias": ["deny"],
+    "no-unused-vars": "allow"
+  }
+}

.eslintrc.js

@@ -1,5 +1,4 @@
 .github/
 coverage/
 **/dist
-scripts/
 package-lock.json

.oxlintrc.json

@@ -2,7 +2,6 @@
   "arrowParens": "avoid",
   "semi": false,
   "singleQuote": true,
-  "jsxSingleQuote": true,
   "trailingComma": "none",
   "printWidth": 80
 }

.prettierignore

@@ -4,7 +4,7 @@ color: true
 include:
   - test/**/*.test.ts
 exclude:
-  - dist/**
+  - test/dist/**
 tsconfig: .config/tsconfig.tap.json
 typecheck: false
 allow-incomplete-coverage: true

.prettierrc

@@ -0,0 +1,60 @@
+'use strict'
+
+const path = require('node:path')
+
+const { includeIgnoreFile } = require('@eslint/compat')
+const tsEslint = require('typescript-eslint')
+const tsParser = require('@typescript-eslint/parser')
+
+const gitignorePath = path.resolve(__dirname, '.gitignore')
+const prettierignorePath = path.resolve(__dirname, '.prettierignore')
+
+module.exports = [
+  includeIgnoreFile(gitignorePath),
+  includeIgnoreFile(prettierignorePath),
+  {
+    files: ['src/**/*.ts', 'test/**/*.ts'],
+    languageOptions: {
+      parser: tsParser,
+      parserOptions: {
+        projectService: {
+          allowDefaultProject: ['test/*.ts'],
+          defaultProject: 'tsconfig.json',
+          tsconfigRootDir: __dirname
+        }
+      }
+    },
+    plugins: {
+      '@typescript-eslint': tsEslint.plugin
+    },
+    linterOptions: {
+      reportUnusedDisableDirectives: 'off'
+    },
+    rules: {
+      '@typescript-eslint/no-floating-promises': ['error'],
+      '@typescript-eslint/no-misused-promises': ['error'],
+      // Returning unawaited promises in a try/catch/finally is dangerous
+      // (the `catch` won't catch if the promise is rejected, and the `finally`
+      // won't wait for the promise to resolve). Returning unawaited promises
+      // elsewhere is probably fine, but this lint rule doesn't have a way
+      // to only apply to try/catch/finally (the 'in-try-catch' option *enforces*
+      // not awaiting promises *outside* of try/catch/finally, which is not what
+      // we want), and it's nice to await before returning anyways, since you get
+      // a slightly more comprehensive stack trace upon promise rejection.
+      '@typescript-eslint/return-await': ['error', 'always'],
+      'no-warning-comments': ['warn', { terms: ['fixme'] }]
+    }
+  },
+  {
+    files: ['test/**/*.ts'],
+    rules: {
+      '@typescript-eslint/no-floating-promises': 'off'
+    }
+  },
+  {
+    files: ['scripts/**/*.js', 'test/**/*.cjs'],
+    rules: {
+      'no-warning-comments': ['warn', { terms: ['fixme'] }]
+    }
+  }
+]