1 files changed, 13 insertions, 10 deletions

diff --git a/.eslintrc.cjs b/.eslintrc.cjs
index 8f1451b..e76ff6f 100644
--- a/.eslintrc.cjs
+++ b/.eslintrc.cjs
@@ -44,8 +44,9 @@ module.exports = {
     },
   },
   rules: {
-    // TODO Flip this to `ignorePackages` once we are on `nodenext` resolution.
-    'import/extensions': ['error', 'never'],
+    // `import/exteions` is buggy with ts files:
+    // https://github.com/import-js/eslint-plugin-import/issues/2111
+    'import/extensions': 'off',
     'import/no-unresolved': 'error',
     'import/order': [
       'error',
@@ -85,7 +86,12 @@ module.exports = {
       files: ['**/stores/**/*.ts'],
       rules: {
         // In a mobx-state-tree action, we assign to the properties of `self` to update the store.
-        'no-param-reassign': 'off',
+        'no-param-reassign': [
+          'error',
+          {
+            props: false,
+          },
+        ],
         // mobx-state-tree uses empty interfaces to speed up typechecking.
         '@typescript-eslint/no-empty-interface': 'off',
       },
@@ -102,17 +108,14 @@ module.exports = {
       },
     },
     {
-      files: ['**/*.js'],
-      rules: {
-        // ESM requires extensions for imports.
-        'import/extensions': ['error', 'ignorePackages'],
-      },
-    },
-    {
       files: ['**/*.cjs'],
       parserOptions: {
         sourceType: 'script',
       },
+      rules: {
+        // We have to use `const foo = require('foo')` syntax in CommonJS files.
+        '@typescript-eslint/no-var-requires': 'off',
+      },
     },
     {
       files: [