From d5e2973dce617f451e4eb07491b3a6874ea6ca47 Mon Sep 17 00:00:00 2001
From: xuatz <xzlow10@gmail.com>
Date: Sun, 22 Jun 2025 20:29:30 +0900
Subject: chore: migrate away from eslint to oxlint (#1642)

* chore: migrate away from eslint to oxlint

* revert turbo task name lint

* it seems like we can remove the seemingly default globals
---
 apps/workers/.oxlintrc.json           | 22 ++++++++++++++++++++++
 apps/workers/package.json             | 11 ++---------
 apps/workers/workers/crawlerWorker.ts |  2 +-
 apps/workers/workers/feedWorker.ts    |  2 +-
 apps/workers/workers/videoWorker.ts   |  2 +-
 5 files changed, 27 insertions(+), 12 deletions(-)
 create mode 100644 apps/workers/.oxlintrc.json

(limited to 'apps/workers')

diff --git a/apps/workers/.oxlintrc.json b/apps/workers/.oxlintrc.json
new file mode 100644
index 00000000..44a3dcef
--- /dev/null
+++ b/apps/workers/.oxlintrc.json
@@ -0,0 +1,22 @@
+{
+  "$schema": "../../node_modules/oxlint/configuration_schema.json",
+  "extends": [
+    "../../tooling/oxlint/oxlint-base.json"
+  ],
+  "categories": {
+    "correctness": "warn"
+  },
+    "env": {
+    "builtin": true,
+    "commonjs": true
+  },
+  "ignorePatterns": [
+    "**/*.config.js",
+    "**/*.config.cjs",
+    "**/.eslintrc.cjs",
+    "**/.next",
+    "**/dist",
+    "**/build",
+    "**/pnpm-lock.yaml"
+  ]
+}
diff --git a/apps/workers/package.json b/apps/workers/package.json
index 7d05e7d6..c888350e 100644
--- a/apps/workers/package.json
+++ b/apps/workers/package.json
@@ -46,7 +46,6 @@
     "zod": "^3.24.2"
   },
   "devDependencies": {
-    "@karakeep/eslint-config": "workspace:^0.2.0",
     "@karakeep/prettier-config": "workspace:^0.1.0",
     "@types/jsdom": "^21.1.6",
     "@types/metascraper": "^5.14.3",
@@ -55,17 +54,11 @@
   "scripts": {
     "start": "tsx watch index.ts",
     "start:prod": "tsx index.ts",
-    "lint": "eslint .",
-    "lint:fix": "eslint . --fix",
+    "lint": "oxlint .",
+    "lint:fix": "oxlint . --fix",
     "format": "prettier . --ignore-path ../../.prettierignore",
     "format:fix": "prettier . --ignore-path ../../.prettierignore --write",
     "typecheck": "tsc --noEmit"
   },
-  "eslintConfig": {
-    "root": true,
-    "extends": [
-      "@karakeep/eslint-config/base"
-    ]
-  },
   "prettier": "@karakeep/prettier-config"
 }
diff --git a/apps/workers/workers/crawlerWorker.ts b/apps/workers/workers/crawlerWorker.ts
index b928e145..d92da208 100644
--- a/apps/workers/workers/crawlerWorker.ts
+++ b/apps/workers/workers/crawlerWorker.ts
@@ -697,7 +697,7 @@ async function crawlAndParseUrl(
     }
     try {
       return new Date(date);
-    } catch (_e) {
+    } catch {
       return null;
     }
   };
diff --git a/apps/workers/workers/feedWorker.ts b/apps/workers/workers/feedWorker.ts
index 1eaba0c3..682889f0 100644
--- a/apps/workers/workers/feedWorker.ts
+++ b/apps/workers/workers/feedWorker.ts
@@ -150,7 +150,7 @@ async function run(req: DequeuedJob<ZFeedRequestSchema>) {
 
   // For feeds that don't have guids, use the link as the id
   feedItems.forEach((item) => {
-    item.guid = item.guid ?? `${item.id}` ?? item.link;
+    item.guid = item.guid ?? item.id ?? item.link;
   });
 
   const exitingEntries = await db.query.rssFeedImportsTable.findMany({
diff --git a/apps/workers/workers/videoWorker.ts b/apps/workers/workers/videoWorker.ts
index ca4168a6..ca591e6f 100644
--- a/apps/workers/workers/videoWorker.ts
+++ b/apps/workers/workers/videoWorker.ts
@@ -193,7 +193,7 @@ async function deleteLeftOverAssetFile(
   );
   try {
     await fs.promises.rm(assetFile);
-  } catch (e) {
+  } catch {
     logger.error(
       `[VideoCrawler][${jobId}] Failed deleting leftover video asset "${assetFile}".`,
     );
-- 
cgit v1.2.3-70-g09d2