Update to typescript-eslint v8

.changeset/lemon-cats-judge.md

@@ -0,0 +1,6 @@
+---
+"@definitelytyped/eslint-plugin": patch
+"@definitelytyped/dtslint": patch
+---
+
+Update to typescript-eslint v8

package.json

@@ -18,8 +18,8 @@
     "@types/jest": "^29.5.11",
     "@types/node": "^18.19.7",
     "@types/yargs": "^17.0.32",
-    "@typescript-eslint/eslint-plugin": "^7.14.1",
-    "@typescript-eslint/parser": "^7.14.1",
+    "@typescript-eslint/eslint-plugin": "^8.10.0",
+    "@typescript-eslint/parser": "^8.10.0",
     "eslint": "^8.57.0",
     "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-jsdoc": "^48.0.2",

packages/dtslint/package.json

@@ -28,11 +28,11 @@
     "@definitelytyped/typescript-packages": "workspace:*",
     "@definitelytyped/typescript-versions": "workspace:*",
     "@definitelytyped/utils": "workspace:*",
-    "@typescript-eslint/eslint-plugin": "^7.14.1",
-    "@typescript-eslint/parser": "^7.14.1",
-    "@typescript-eslint/types": "^7.14.1",
-    "@typescript-eslint/typescript-estree": "^7.14.1",
-    "@typescript-eslint/utils": "^7.14.1",
+    "@typescript-eslint/eslint-plugin": "^8.10.0",
+    "@typescript-eslint/parser": "^8.10.0",
+    "@typescript-eslint/types": "^8.10.0",
+    "@typescript-eslint/typescript-estree": "^8.10.0",
+    "@typescript-eslint/utils": "^8.10.0",
     "eslint": "^8.57.0",
     "eslint-plugin-import": "^2.29.1",
     "semver": "^7.5.4",

packages/eslint-plugin/package.json

@@ -19,18 +19,18 @@
   },
   "scripts": {
     "build": "tsc -b .",
-    "test": "../../node_modules/.bin/jest --config ../../jest.config.js packages/dtslint"
+    "test": "../../node_modules/.bin/jest --config ../../jest.config.js packages/eslint-plugin"
   },
   "dependencies": {
     "@definitelytyped/utils": "workspace:*",
-    "@typescript-eslint/types": "^7.14.1",
-    "@typescript-eslint/utils": "^7.14.1",
+    "@typescript-eslint/types": "^8.10.0",
+    "@typescript-eslint/utils": "^8.10.0",
     "semver": "^7.5.4"
   },
   "peerDependencies": {
-    "@typescript-eslint/eslint-plugin": "^7.14.1",
-    "@typescript-eslint/parser": "^7.14.1",
-    "eslint": "^8.40.0",
+    "@typescript-eslint/eslint-plugin": "^8.10.0",
+    "@typescript-eslint/parser": "^8.10.0",
+    "eslint": "^8.57.0",
     "eslint-plugin-jsdoc": "^44.0.0",
     "typescript": ">= 3.0.0-dev || >= 3.1.0-dev || >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.7.0-dev || >= 3.8.0-dev || >= 3.9.0-dev || >= 4.0.0-dev || >=5.0.0-dev"
   },

packages/eslint-plugin/src/configs/all.ts

@@ -154,13 +154,8 @@ export const all: Linter.BaseConfig = {
           },
         ],
         "@typescript-eslint/adjacent-overload-signatures": "error",
-        "@typescript-eslint/ban-types": [
-          "error",
-          {
-            types: { "{}": false },
-            extendDefaults: true,
-          },
-        ],
+        "@typescript-eslint/no-wrapper-object-types": "error",
+        "@typescript-eslint/no-unsafe-function-type": "error",
         "@typescript-eslint/prefer-namespace-keyword": "error",
         "@typescript-eslint/triple-slash-reference": ["error", { types: "prefer-import", path: "always" }],
         "@typescript-eslint/no-empty-interface": "error",

packages/eslint-plugin/src/rules/no-type-only-packages.ts

@@ -32,7 +32,11 @@ const rule = createRule({
     let hasNonEmptyFile = false;
 
     function isOutsidePackage(fileName: string): boolean {
-      return fileName.includes("node_modules") || path.relative(pkg!.dir, fileName).startsWith("..");
+      if (fileName.includes("node_modules") || path.relative(pkg!.dir, fileName).startsWith("..")) {
+        return true;
+      }
+      const other = findTypesPackage(fileName);
+      return !other || other.dir !== pkg!.dir;
     }
 
     for (const sourceFile of program.getSourceFiles()) {

packages/eslint-plugin/test/__snapshots__/plugin.test.ts.snap

@@ -48,15 +48,6 @@ exports[`plugin should have the expected exports 1`] = `
                 "ts-nocheck": true,
               },
             ],
-            "@typescript-eslint/ban-types": [
-              "error",
-              {
-                "extendDefaults": true,
-                "types": {
-                  "{}": false,
-                },
-              },
-            ],
             "@typescript-eslint/consistent-type-definitions": "error",
             "@typescript-eslint/explicit-member-accessibility": [
               "error",
@@ -84,6 +75,8 @@ exports[`plugin should have the expected exports 1`] = `
               },
             ],
             "@typescript-eslint/no-misused-new": "error",
+            "@typescript-eslint/no-unsafe-function-type": "error",
+            "@typescript-eslint/no-wrapper-object-types": "error",
             "@typescript-eslint/prefer-namespace-keyword": "error",
             "@typescript-eslint/triple-slash-reference": [
               "error",

packages/eslint-plugin/test/eslint.test.ts

@@ -25,23 +25,15 @@ function getAllExpectedLintSnapshots() {
   return new Set(allFixtures.map(getLintSnapshotPath));
 }
 
-let eslint: ESLint;
-
-beforeAll(() => {
-  eslint = new ESLint({
-    cwd: fixtureRoot,
-    plugins: { [plugin.meta.name]: plugin },
-  });
-});
-
-afterAll(() => {
-  eslint = undefined as any;
-});
-
 // Force one test per fixture so we can see when a file has no errors.
 for (const fixture of allFixtures) {
   describe(`fixture ${fixture}`, () => {
     it("should lint", async () => {
+      const eslint = new ESLint({
+        cwd: fixtureRoot,
+        plugins: { [plugin.meta.name]: plugin },
+      });
+
       const results = await eslint.lintFiles([fixture]);
       for (const result of results) {
         result.filePath = path.relative(fixtureRoot, result.filePath);