eslint-config-isaacscript 2.0.0 → 3.0.2

package/base.js

@@ -1,5 +1,5 @@
 // This is a shared configuration file for ESLint:
-// https://eslint.org/docs/user-guide/configuring
+// https://eslint.org/docs/latest/user-guide/configuring
 module.exports = {
   extends: [
     /**
@@ -20,15 +20,17 @@ module.exports = {
     "airbnb-typescript/base",
 
     /**
-     * We extend the Airbnb rules with the "recommended" and "recommended-requiring-type-checking"
-     * rules from the "typescript-eslint" plugin, which is also recommended by Matt Turnbull, the
-     * author of "airbnb-typescript/base":
+     * We extend the Airbnb rules with the "recommended", "recommended-requiring-type-checking", and
+     * "strict" rules from the "typescript-eslint" plugin, which is also recommended by Matt
+     * Turnbull, the author of "airbnb-typescript/base":
      * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/README.md#recommended
      * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/recommended.ts
      * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/recommended-requiring-type-checking.ts
+     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/strict.ts
      */
     "plugin:@typescript-eslint/recommended",
     "plugin:@typescript-eslint/recommended-requiring-type-checking",
+    "plugin:@typescript-eslint/strict",
 
     /**
      * This provides extra miscellaneous rules to keep code safe:
@@ -73,7 +75,7 @@ module.exports = {
   rules: {
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/array-type.md
+     * https://typescript-eslint.io/rules/array-type
      *
      * Not defined in the parent configs.
      *
@@ -88,7 +90,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/explicit-module-boundary-types.md
+     * https://typescript-eslint.io/rules/explicit-module-boundary-types
      *
      * Not defined in the parent configs.
      *
@@ -99,7 +101,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/lines-between-class-members.md
+     * https://typescript-eslint.io/rules/lines-between-class-members
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js
@@ -117,7 +119,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/naming-convention.md
+     * https://typescript-eslint.io/rules/naming-convention
      *
      * Defined at:
      * https://github.com/iamturns/eslint-config-airbnb-typescript/blob/master/lib/shared.js
@@ -151,7 +153,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/no-unnecessary-boolean-literal-compare.md
+     * https://typescript-eslint.io/rules/no-unnecessary-boolean-literal-compare
      *
      * Not defined in the parent configs.
      *
@@ -161,7 +163,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/no-unused-vars.md
+     * https://typescript-eslint.io/rules/no-unused-vars
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/variables.js
@@ -181,7 +183,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/no-use-before-define.md
+     * https://typescript-eslint.io/rules/no-use-before-define
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/variables.js
@@ -192,7 +194,20 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/quotes.md
+     * https://typescript-eslint.io/rules/prefer-optional-chain
+     *
+     * Defined at:
+     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/strict.ts
+     *
+     * This can modify the type of boolean declarations to "boolean | undefined", which is undesired
+     * in some circumstances:
+     * https://github.com/typescript-eslint/typescript-eslint/issues/5269
+     */
+    "@typescript-eslint/prefer-optional-chain": "off",
+
+    /**
+     * Documentation:
+     * https://typescript-eslint.io/rules/quotes
      *
      * Defined at:
      * https://github.com/prettier/eslint-config-prettier/blob/main/%40typescript-eslint.js
@@ -212,7 +227,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/restrict-template-expressions.md
+     * https://typescript-eslint.io/rules/restrict-template-expressions
      *
      * Defined at:
      * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/recommended-requiring-type-checking.ts
@@ -224,7 +239,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/restrict-template-expressions.md
+     * https://typescript-eslint.io/rules/strict-boolean-expressions
      *
      * Not defined in the parent configs.
      *
@@ -243,18 +258,58 @@ module.exports = {
       },
     ],
 
+    /**
+     * Documentation:
+     * https://typescript-eslint.io/rules/switch-exhaustiveness-check/
+     *
+     * Not defined in the parent configs.
+     *
+     * This rule ensures type-safety with switch statements, which can be especially helpful when
+     * values are added or removed from an enum.
+     */
+    "@typescript-eslint/switch-exhaustiveness-check": "warn",
+
+    /**
+     * Documentation:
+     * https://eslint.org/docs/latest/rules/consistent-return
+     *
+     * Defined at:
+     * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/best-practices.js
+     *
+     * We prefer the "noImplicitReturns" compiler flag over the "consistent-return" ESLint rule,
+     * since it is type-aware:
+     * https://github.com/typescript-eslint/typescript-eslint/issues/5254
+     */
+    "consistent-return": ["off"],
+
     /**
      * Documentation:
      * https://eslint.org/docs/latest/rules/curly
      *
-     * Defined at: ?
+     * Defined at:
+     * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/best-practices.js
      *
      * Always requiring curly braces can partially ward against Apple-style if statement bugs:
      * https://www.imperialviolet.org/2014/02/22/applebug.html
-     * ?
+     *
+     * Additionally, this rule needs to be set to "all" to work properly with
+     * `eslint-prettier-config`:
+     * https://github.com/prettier/eslint-config-prettier#curly
      */
     curly: ["warn", "all"],
 
+    /**
+     * Documentation:
+     * https://eslint.org/docs/latest/rules/default-case
+     *
+     * Defined at:
+     * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/best-practices.js
+     *
+     * This rule is generally bad to have on in TypeScript projects:
+     * https://github.com/typescript-eslint/typescript-eslint/issues/5254
+     */
+    "default-case": "off",
+
     /**
      * Documentation:
      * https://github.com/mysticatea/eslint-plugin-eslint-comments/blob/master/docs/rules/disable-enable-pair.md
@@ -308,7 +363,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/no-console
+     * https://eslint.org/docs/latest/rules/no-console
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/errors.js
@@ -330,7 +385,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/no-continue
+     * https://eslint.org/docs/latest/rules/no-continue
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js
@@ -341,24 +396,19 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/no-plusplus
+     * https://eslint.org/docs/latest/rules/no-plusplus
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js
      *
-     * Airbnb disallows these because it can lead to errors with minified code. We don't have to
-     * worry about this in for loops though.
+     * Airbnb disallows these because it can lead to errors with minified code. When using Prettier,
+     * it adds semi-colons everywhere, so we don't have to worry about this.
      */
-    "no-plusplus": [
-      "warn",
-      {
-        allowForLoopAfterthoughts: true,
-      },
-    ],
+    "no-plusplus": "off",
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/no-restricted-syntax
+     * https://eslint.org/docs/latest/rules/no-restricted-syntax
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js
@@ -392,7 +442,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/prefer-destructuring
+     * https://eslint.org/docs/latest/rules/prefer-destructuring
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/es6.js

package/jsdoc.js

@@ -1,5 +1,5 @@
 // This is a shared configuration file for ESLint:
-// https://eslint.org/docs/user-guide/configuring
+// https://eslint.org/docs/latest/user-guide/configuring
 module.exports = {
   plugins: [
     /**

package/mod.js

@@ -1,5 +1,5 @@
 // This is a shared configuration file for ESLint:
-// https://eslint.org/docs/user-guide/configuring
+// https://eslint.org/docs/latest/user-guide/configuring
 module.exports = {
   extends: [
     /**
@@ -12,7 +12,7 @@ module.exports = {
   rules: {
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/naming-convention.md
+     * https://typescript-eslint.io/rules/naming-convention
      *
      * Defined at:
      * https://github.com/iamturns/eslint-config-airbnb-typescript/blob/master/lib/shared.js
@@ -54,7 +54,41 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/no-loop-func.md
+     * https://typescript-eslint.io/rules/no-base-to-string/
+     *
+     * Defined at:
+     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/strict.ts
+     *
+     * The `Vector` object has a `tostring` meta-method, so it can be properly printed without
+     * explicitly specifying the X and Y values.
+     */
+    "@typescript-eslint/no-base-to-string": [
+      "warn",
+      {
+        ignoredTypeNames: ["Vector"],
+      },
+    ],
+
+    /**
+     * Documentation:
+     * https://typescript-eslint.io/rules/no-invalid-void-type
+     *
+     * Defined at:
+     * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/strict.ts
+     *
+     * TSTL has special behavior with respect to "this: void", so we need to configure this rule to
+     * allow the "this" parameter.
+     */
+    "@typescript-eslint/no-invalid-void-type": [
+      "warn",
+      {
+        allowAsThisParameter: true,
+      },
+    ],
+
+    /**
+     * Documentation:
+     * https://typescript-eslint.io/rules/no-loop-func
      *
      * Defined at:
      * https://github.com/iamturns/eslint-config-airbnb-typescript/blob/master/lib/shared.js
@@ -87,7 +121,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/eslint/eslint/blob/master/docs/rules/no-bitwise.md
+     * https://eslint.org/docs/latest/rules/no-bitwise
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js
@@ -98,7 +132,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/no-param-reassign
+     * https://eslint.org/docs/latest/rules/no-param-reassign
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/best-practices.js
@@ -109,7 +143,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/no-restricted-globals
+     * https://eslint.org/docs/latest/rules/no-restricted-globals
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/variables.js
@@ -120,7 +154,7 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://eslint.org/docs/rules/no-underscore-dangle
+     * https://eslint.org/docs/latest/rules/no-underscore-dangle
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js

package/monorepo.js

@@ -3,7 +3,7 @@ const path = require("path"); // eslint-disable-line @typescript-eslint/no-var-r
 const REPO_ROOT = path.join(__dirname, "..", "..");
 
 // This is a shared configuration file for ESLint:
-// https://eslint.org/docs/user-guide/configuring
+// https://eslint.org/docs/latest/user-guide/configuring
 module.exports = {
   plugins: ["@nrwl/nx"],
 
@@ -12,8 +12,7 @@ module.exports = {
   parserOptions: {
     ecmaVersion: 2020,
     sourceType: "module",
-    /* eslint-disable-next-line isaacscript/complete-sentences-line-comments */
-    // This cannot be simplified to "./../.." because of relative path shenanigans.
+    // This cannot be simplified to `./../..` because of relative path shenanigans.
     tsconfigRootDir: REPO_ROOT,
   },
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-isaacscript",
-  "version": "2.0.0",
+  "version": "3.0.2",
   "description": "A sharable ESLint config for TypeScript and IsaacScript projects.",
   "keywords": [
     "eslint",