fix lint

Author: hipstersmoothie

.eslintrc.js

@@ -4,7 +4,12 @@ const project = resolve(process.cwd(), "tsconfig.json");
 
 /** @type {import("eslint").Linter.Config} */
 module.exports = {
-  extends: ["eslint:recommended", "prettier", "eslint-config-turbo"],
+  extends: [
+    "eslint:recommended",
+    "prettier",
+    "eslint-config-turbo",
+    "plugin:@typescript-eslint/recommended",
+  ],
   plugins: ["only-warn"],
   globals: {
     React: true,
@@ -35,4 +40,4 @@ module.exports = {
       files: ["*.js?(x)", "*.ts?(x)"],
     },
   ],
-};
+};

packages/config-eslint/base.js

@@ -3,6 +3,7 @@
   "version": "0.0.0",
   "license": "MIT",
   "private": true,
+  "main": "base.js",
   "scripts": {
     "clean": "rm -rf node_modules"
   },

packages/config-eslint/package.json

@@ -4,13 +4,11 @@
   "private": true,
   "repository": "jimp-dev/jimp",
   "scripts": {
-    "lint": "eslint .",
     "dev": "tshy --watch"
   },
   "author": "Andrew Lisowski <[email protected]>",
   "license": "MIT",
   "devDependencies": {
-    "@jimp/config-eslint": "workspace:*",
     "@jimp/config-typescript": "workspace:*",
     "@jimp/test-utils": "workspace:*",
     "eslint": "^8.57.0",

packages/config-vitest/package.json

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/config-vitest/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

packages/core/.eslintrc.cjs

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/core/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

packages/diff/.eslintrc.cjs

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/diff/tsconfig.json

@@ -4,7 +4,6 @@
   "type": "module",
   "version": "0.0.1",
   "scripts": {
-    "lint": "eslint .",
     "dev": "astro dev",
     "start": "astro dev",
     "build-website": "rm -rf dist && astro check && astro build",

packages/docs/package.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

packages/jimp/.eslintrc copy.cjs

@@ -62,6 +62,7 @@ describe("Callbacks", () => {
       const clone = targetImg.clone();
 
       expect(
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         (clone[op as keyof typeof clone] as any)(...args)
       ).toMatchSnapshot();
     });

packages/jimp/src/callbacks.test.ts

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/jimp/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

packages/read-file/.eslintrc.cjs

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/read-file/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

packages/test-utils/.eslintrc.cjs

@@ -2,12 +2,13 @@ import { expect } from "vitest";
 import { Bitmap } from "@jimp/types";
 import path from "path";
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function hashForEach<Hash extends Record<string, any>>(
   hash: Hash,
   func: (key: keyof Hash, value: Hash[keyof Hash]) => void
 ) {
   for (const key in hash)
-    if (hash.hasOwnProperty(key)) {
+    if (key in hash) {
       func(key, hash[key]);
     }
 }
@@ -134,7 +135,7 @@ export function testImgToStr(testImage: Bitmap) {
     colors2[c] = k;
   });
 
-  let unknownColors = new Set<number>();
+  const unknownColors = new Set<number>();
 
   for (let y = 0; y < testImage.height; y++) {
     for (let x = 0; x < w; x++) {

packages/test-utils/src/index.ts

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/test-utils/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

packages/types/.eslintrc.cjs

@@ -14,6 +14,7 @@ export interface Bitmap {
 
 export interface Format<
   Mime extends string = string,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   ExportOptions extends Record<string, any> | undefined = undefined,
 > {
   mime: Mime;
@@ -51,19 +52,23 @@ export interface JimpClass {
   getPixelColor: (x: number, y: number) => number;
   setPixelColor: (hex: number, x: number, y: number) => JimpClass;
 
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   scan(f: (x: number, y: number, idx: number) => any): JimpClass;
   scan(
     x: number,
     y: number,
     w: number,
     h: number,
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     cb: (x: number, y: number, idx: number) => any
   ): JimpClass;
   scan(
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     x: number | ((x: number, y: number, idx: number) => any),
     y?: number,
     w?: number,
     h?: number,
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     f?: (x: number, y: number, idx: number) => any
   ): JimpClass;
 }

packages/types/src/index.ts

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/types/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

packages/utils/.eslintrc.cjs

@@ -8,11 +8,13 @@ export function clone<I extends JimpClass>(image: I): I {
     data: Buffer.from(image.bitmap.data),
   };
 
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   return new (image.constructor as any)(newBitmap);
 }
 
 export function scan<I extends JimpClass>(
   image: I,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   f: (this: I, x: number, y: number, idx: number) => any
 ): I;
 export function scan<I extends { bitmap: Bitmap }>(
@@ -21,20 +23,24 @@ export function scan<I extends { bitmap: Bitmap }>(
   y: number,
   w: number,
   h: number,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   cb: (x: number, y: number, idx: number) => any
 ): I;
 export function scan<I extends { bitmap: Bitmap }>(
   image: I,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   xArg: number | ((x: number, y: number, idx: number) => any),
   yArg?: number,
   wArg?: number,
   hArg?: number,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   cbArg?: (x: number, y: number, idx: number) => any
 ): I {
   let x: number;
   let y: number;
   let w: number;
   let h: number;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   let cb: (x: number, y: number, idx: number) => any;
 
   if (typeof xArg === "function") {

packages/utils/src/index.ts

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

packages/utils/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

plugins/js-bmp/.eslintrc.cjs

@@ -7,6 +7,7 @@ export type { BmpColor } from "bmp-ts";
 export { BmpCompression } from "bmp-ts";
 
 type Pretty<T> = {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [K in keyof T]: T[K] extends (...args: any[]) => any
     ? T[K]
     : T[K] extends object

plugins/js-bmp/src/index.ts

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

plugins/js-bmp/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

plugins/js-gif/.eslintrc.cjs

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

plugins/js-gif/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

plugins/js-jpeg/.eslintrc.cjs

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

plugins/js-jpeg/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};

plugins/js-png/.eslintrc.cjs

@@ -2,7 +2,5 @@
   "extends": "@jimp/config-typescript/base.json",
   "compilerOptions": {
     "outDir": "dist"
-  },
-  "include": ["src"],
-  "exclude": ["node_modules", "dist"]
-}
+  }
+}

plugins/js-png/tsconfig.json

@@ -0,0 +1,5 @@
+/** @type {import("eslint").Linter.Config} */
+module.exports = {
+  root: true,
+  extends: [require.resolve("@jimp/config-eslint/base.js")],
+};