chore: oxc --type-aware --type-check (@miodec)

.github/workflows/monkey-ci.yml

@@ -161,7 +161,7 @@ jobs:
         run: pnpm install
 
       - name: Check lint
-        run: npm run lint-be
+        run: npm run lint-fast-be && npm run lint-be
 
       - name: Build
         run: npm run build-be
@@ -214,7 +214,7 @@ jobs:
         run: pnpm install
 
       - name: Check lint
-        run: npm run lint-fe
+        run: npm run lint-fast-fe && npm run lint-fe
 
       - name: Build
         run: npm run build-fe
@@ -330,7 +330,7 @@ jobs:
         run: pnpm install
 
       - name: Check lint
-        run: npm run lint-pkg
+        run: npm run lint-fast-pkg && npm run lint-pkg
 
       - name: Build
         run: npm run build-pkg

.oxlintrc.json

@@ -0,0 +1,7 @@
+{
+  "ignorePatterns": ["node_modules", "dist", ".turbo"],
+  "extends": [
+    "./packages/oxlint-config/index.jsonc"
+    // "@monkeytype/oxlint-config"
+  ]
+}

backend/.oxlintrc-plugin.json

@@ -0,0 +1,4 @@
+{
+  "ignorePatterns": ["node_modules", "__migration__", "dist", ".turbo"],
+  "extends": ["../packages/oxlint-config/plugin.jsonc"]
+}

backend/.oxlintrc.json

@@ -1,33 +1,7 @@
 {
+  "ignorePatterns": ["node_modules", "__migration__", "dist", ".turbo"],
   "extends": [
     "../packages/oxlint-config/index.jsonc"
     // "@monkeytype/oxlint-config"
-  ],
-  "overrides": [
-    {
-      "files": ["__tests__/**"],
-      "plugins": [
-        "typescript",
-        "unicorn",
-        "oxc",
-        "import",
-        "node",
-        "promise",
-        "jest",
-        "vitest"
-      ],
-      "rules": {
-        "no-explicit-any": "allow",
-        "explicit-function-return-type": "off",
-        "ban-ts-comment": "off"
-      }
-    },
-    {
-      "files": ["__tests__/**/*.d.ts"],
-      "rules": {
-        "typescript/consistent-type-definitions": "off",
-        "typescript/no-empty-object-type": "off"
-      }
-    }
   ]
 }

backend/__tests__/api/controllers/admin.spec.ts

@@ -10,6 +10,7 @@ import GeorgeQueue from "../../../src/queues/george-queue";
 import * as AuthUtil from "../../../src/utils/auth";
 
 import { enableRateLimitExpects } from "../../__testData__/rate-limit";
+import Test from "supertest/lib/test";
 
 const { mockApp, uid } = setup();
 const configuration = Configuration.getCachedConfiguration();
@@ -558,12 +559,12 @@ describe("AdminController", () => {
     });
   });
 
-  async function expectFailForNonAdmin(call: SuperTest): Promise<void> {
+  async function expectFailForNonAdmin(call: Test): Promise<void> {
     isAdminMock.mockResolvedValue(false);
     const { body } = await call.expect(403);
     expect(body.message).toEqual("You don't have permission to do this.");
   }
-  async function expectFailForDisabledEndpoint(call: SuperTest): Promise<void> {
+  async function expectFailForDisabledEndpoint(call: Test): Promise<void> {
     await enableAdminEndpoints(false);
     const { body } = await call.expect(503);
     expect(body.message).toEqual("Admin endpoints are currently disabled.");

backend/__tests__/api/controllers/user.spec.ts

@@ -33,11 +33,10 @@ import { mockAuthenticateWithApeKey } from "../../__testData__/auth";
 import { randomUUID } from "node:crypto";
 import { MonkeyMail, UserStreak } from "@monkeytype/schemas/users";
 import MonkeyError, { isFirebaseError } from "../../../src/utils/error";
-import { LeaderboardEntry } from "@monkeytype/schemas/leaderboards";
 import * as WeeklyXpLeaderboard from "../../../src/services/weekly-xp-leaderboard";
 import * as ConnectionsDal from "../../../src/dal/connections";
 import { pb } from "../../__testData__/users";
-import { SuperTest } from "supertest";
+import Test from "supertest/lib/test";
 
 const { mockApp, uid, mockAuth } = setup();
 const configuration = Configuration.getCachedConfiguration();
@@ -2979,7 +2978,7 @@ describe("user controller test", () => {
 
       getUserByNameMock.mockResolvedValue(foundUser as any);
 
-      const rank = { rank: 24 } as LeaderboardEntry;
+      const rank = { rank: 24 } as LeaderboardDal.DBLeaderboardEntry;
       leaderboardGetRankMock.mockResolvedValue(rank);
       leaderboardGetCountMock.mockResolvedValue(100);
 
@@ -3041,7 +3040,7 @@ describe("user controller test", () => {
         ...foundUser,
         profileDetails: { showActivityOnPublicProfile: true },
       } as any);
-      const rank = { rank: 24 } as LeaderboardEntry;
+      const rank = { rank: 24 } as LeaderboardDal.DBLeaderboardEntry;
       leaderboardGetRankMock.mockResolvedValue(rank);
       leaderboardGetCountMock.mockResolvedValue(100);
 
@@ -3063,7 +3062,7 @@ describe("user controller test", () => {
         ...foundUser,
         profileDetails: { showActivityOnPublicProfile: false },
       } as any);
-      const rank = { rank: 24 } as LeaderboardEntry;
+      const rank = { rank: 24 } as LeaderboardDal.DBLeaderboardEntry;
       leaderboardGetRankMock.mockResolvedValue(rank);
       leaderboardGetCountMock.mockResolvedValue(100);
 
@@ -3081,7 +3080,7 @@ describe("user controller test", () => {
         banned: true,
       } as any);
 
-      const rank = { rank: 24 } as LeaderboardEntry;
+      const rank = { rank: 24 } as LeaderboardDal.DBLeaderboardEntry;
       leaderboardGetRankMock.mockResolvedValue(rank);
       leaderboardGetCountMock.mockResolvedValue(100);
 
@@ -3132,7 +3131,7 @@ describe("user controller test", () => {
       const uid = foundUser.uid;
       getUserMock.mockResolvedValue(foundUser as any);
 
-      const rank = { rank: 24 } as LeaderboardEntry;
+      const rank = { rank: 24 } as LeaderboardDal.DBLeaderboardEntry;
       leaderboardGetRankMock.mockResolvedValue(rank);
       leaderboardGetCountMock.mockResolvedValue(100);
 
@@ -4050,7 +4049,7 @@ async function enableConnectionsEndpoints(enabled: boolean): Promise<void> {
   );
 }
 
-async function expectFailForDisabledEndpoint(call: SuperTest): Promise<void> {
+async function expectFailForDisabledEndpoint(call: Test): Promise<void> {
   await enableConnectionsEndpoints(false);
   const { body } = await call.expect(503);
   expect(body.message).toEqual("Connections are not available at this time.");

backend/__tests__/tsconfig.json

@@ -1,7 +1,11 @@
 {
   "extends": "@monkeytype/typescript-config/base.json",
   "compilerOptions": {
-    "noEmit": true
+    "noEmit": true,
+    "noImplicitAny": false,
+    "strictFunctionTypes": false,
+    "useUnknownInCatchVariables": false,
+    "strictPropertyInitialization": false
   },
   "files": ["vitest.d.ts"],
   "include": ["./**/*.ts", "./**/*.spec.ts", "./setup-tests.ts"]

backend/package.json

@@ -4,9 +4,8 @@
   "license": "GPL-3.0",
   "private": true,
   "scripts": {
-    "eslint": "eslint \"./src/**/*.ts\"",
-    "oxlint": "oxlint .",
-    "lint": "npm run oxlint && npm run eslint",
+    "lint": "oxlint . --type-aware --type-check && oxlint -c .oxlintrc-plugin.json",
+    "lint-fast": "oxlint .",
     "build": "npm run gen-docs && tsc --build",
     "watch": "tsc --build --watch",
     "clean": "tsc --build --clean",
@@ -62,7 +61,6 @@
     "zod": "3.23.8"
   },
   "devDependencies": {
-    "@monkeytype/eslint-config": "workspace:*",
     "@monkeytype/oxlint-config": "workspace:*",
     "@monkeytype/typescript-config": "workspace:*",
     "@redocly/cli": "2.0.5",
@@ -84,10 +82,9 @@
     "@types/uuid": "10.0.0",
     "@vitest/coverage-v8": "4.0.8",
     "concurrently": "8.2.2",
-    "eslint": "8.57.1",
-    "eslint-watch": "8.0.0",
     "openapi3-ts": "2.0.2",
-    "oxlint": "1.29.0",
+    "oxlint": "1.32.0",
+    "oxlint-tsgolint": "0.8.3",
     "readline-sync": "1.4.10",
     "supertest": "7.1.4",
     "testcontainers": "11.4.0",

backend/scripts/openapi.ts

@@ -219,14 +219,15 @@ function addRateLimit(
   metadata: EndpointMetadata | undefined,
 ): void {
   if (metadata === undefined || metadata.rateLimit === undefined) return;
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+  // oxlint-disable-next-line no-unsafe-assignment
   const okResponse = operation.responses["200"];
   if (okResponse === undefined) return;
 
   operation.description += getRateLimitDescription(metadata.rateLimit);
 
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+  // oxlint-disable-next-line no-unsafe-assignment no-unsafe-member-access
   okResponse["headers"] = {
+    // oxlint-disable-next-line no-unsafe-member-access
     ...okResponse["headers"],
     "x-ratelimit-limit": {
       schema: { type: "integer" },

backend/src/api/controllers/dev.ts

@@ -247,7 +247,7 @@ async function updateUser(uid: string): Promise<void> {
         lbPersonalBests[mode.mode][mode.mode2] = {};
       }
 
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+      // oxlint-disable-next-line no-unsafe-member-access
       lbPersonalBests[mode.mode][mode.mode2][mode.language] = entry;
     }
 

backend/src/api/controllers/user.ts

@@ -553,7 +553,7 @@ export async function getUser(req: MonkeyRequest): Promise<GetUserResponse> {
           uid,
         );
       } catch (e) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+        // oxlint-disable-next-line no-unsafe-member-access
         if (e.code === "auth/user-not-found") {
           throw new MonkeyError(
             404,

backend/src/api/ts-rest-adapter.ts

@@ -11,7 +11,7 @@ export function callController<
   TParams,
   TResponse,
   //ignoring as it might be used in the future
-  // eslint-disable-next-line @typescript-eslint/no-unnecessary-type-parameters
+  // oxlint-disable-next-line no-unnecessary-type-parameters
   TStatus = 200,
 >(
   handler: MonkeyHandler<TQuery, TBody, TParams, TResponse>,

backend/src/api/types.ts

@@ -2,7 +2,7 @@ import { TsRestRequest as TsRestRequestGeneric } from "@ts-rest/express";
 import { Request as ExpressRequest } from "express";
 import { Context } from "../middlewares/context";
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
+// oxlint-disable-next-line no-explicit-any
 export type TsRestRequest = TsRestRequestGeneric<any>;
 
 export type ExpressRequestWithContext = {

backend/src/dal/connections.ts

@@ -74,7 +74,7 @@ export async function create(
 
     return created;
   } catch (e) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     if (e.name === "MongoServerError" && e.code === 11000) {
       const existing = await getCollection().findOne(
         { key },

backend/src/dal/leaderboards.ts

@@ -85,7 +85,7 @@ export async function get(
 
     return leaderboard;
   } catch (e) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     if (e.error === 175) {
       //QueryPlanKilled, collection was removed during the query
       return false;
@@ -162,7 +162,7 @@ export async function getRank(
       return results[0] ?? null;
     }
   } catch (e) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     if (e.error === 175) {
       //QueryPlanKilled, collection was removed during the query
       return false;
@@ -393,7 +393,7 @@ async function createIndex(
       Logger.warning(`Index ${key} not matching, dropping and recreating...`);
 
       const existingIndex = (await getUsersCollection().listIndexes().toArray())
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+        // oxlint-disable-next-line no-unsafe-member-access
         .map((it) => it.name as string)
         .find((it) => it.startsWith(key));
 

backend/src/dal/user.ts

@@ -772,7 +772,7 @@ export async function getPersonalBests(
   ]);
 
   if (mode2 !== undefined) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     return user.personalBests?.[mode]?.[mode2] as PersonalBest;
   }
 

backend/src/middlewares/auth.ts

@@ -203,7 +203,7 @@ async function authenticateWithBearerToken(
       );
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     const errorCode = error?.errorInfo?.code as string | undefined;
 
     if (errorCode?.includes("auth/id-token-expired")) {

backend/src/middlewares/utility.ts

@@ -41,7 +41,7 @@ export function onlyAvailableOnDev(): RequestHandler {
 }
 
 export function getMetadata(req: TsRestRequestWithContext): EndpointMetadata {
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+  // oxlint-disable-next-line no-unsafe-member-access
   return (req.tsRestRoute["metadata"] ?? {}) as EndpointMetadata;
 }
 

backend/src/utils/auth.ts

@@ -103,7 +103,7 @@ export async function sendForgotPasswordEmail(email: string): Promise<void> {
     await emailQueue.sendForgotPasswordEmail(email, name, link);
   } catch (err) {
     if (isFirebaseError(err) && err.errorInfo.code !== "auth/user-not-found") {
-      // eslint-disable-next-line @typescript-eslint/only-throw-error
+      // oxlint-disable-next-line only-throw-error
       throw err;
     }
   }

backend/src/utils/error.ts

@@ -8,18 +8,18 @@ type FirebaseErrorParent = {
   errorInfo: FirebaseError;
 };
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
+// oxlint-disable-next-line no-explicit-any
 export function isFirebaseError(err: any): err is FirebaseErrorParent {
   return (
     typeof err === "object" &&
     "code" in err &&
     "errorInfo" in err &&
     "codePrefix" in err &&
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     typeof err.errorInfo === "object" &&
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     "code" in err.errorInfo &&
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    // oxlint-disable-next-line no-unsafe-member-access
     "message" in err.errorInfo
   );
 }

backend/src/utils/etag.ts

@@ -13,7 +13,7 @@ export function createETagGenerator(options: {
   return function generateETag(body, encoding) {
     const buf = !Buffer.isBuffer(body) ? Buffer.from(body, encoding) : body;
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-call
+    // oxlint-disable-next-line no-unsafe-assignment, no-unsafe-call
     const generatedTag: string = etag(buf, options);
 
     //custom code to add the version number