Merge pull request #1127 from jetstreamapp/chore/update-eslint-to-fla…

…t-config

Chore/update-eslint-to-flat-config

apps/api/eslint.config.js

@@ -0,0 +1,24 @@
+const baseConfig = require('../../eslint.config.js');
+
+module.exports = [
+  ...baseConfig,
+  {
+    ignores: ['src/assets/**/*'],
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
+    rules: {
+      'no-empty-pattern': 'off',
+    },
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+];

apps/api/src/app/controllers/auth.controller.ts

@@ -739,9 +739,7 @@ const verifyEmailViaLink = createRoute(routeDefinition.verification.validators,
 
     const { code } = query;
 
-    const pendingVerification = req.session.pendingVerification.find(({ type }) => {
-      type === 'email';
-    });
+    const pendingVerification = req.session.pendingVerification.find(({ type }) => type === 'email');
 
     if (!pendingVerification) {
       throw new InvalidSession('Missing pending verification');

apps/api/src/app/controllers/socket.controller.ts

@@ -100,7 +100,9 @@ export function initSocketServer(app: express.Express, middlewareFns: express.Re
       // TODO: should we distinguish specific reason for disconnect before unsubscribing from cometd?
       // If browser did not really disconnect, how will it know that it is no longer subscribed to cometd?
       Object.values(userSocketState.cometdConnections).forEach(({ cometd, subscriptions }) => {
-        cometd && socketUtils.disconnectCometD(cometd, socket, user);
+        if (cometd) {
+          socketUtils.disconnectCometD(cometd, socket, user);
+        }
         subscriptions.clear();
       });
       userSocketState.cometdConnections = {};

apps/cron-tasks/eslint.config.js

@@ -0,0 +1,20 @@
+const baseConfig = require('../../eslint.config.js');
+
+module.exports = [
+  ...baseConfig,
+  {
+    files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+];

apps/cron-tasks/project.json

@@ -3,6 +3,7 @@
   "$schema": "../../node_modules/nx/schemas/project-schema.json",
   "sourceRoot": "apps/cron-tasks/src",
   "projectType": "application",
+  "tags": ["scope:server"],
   "targets": {
     "build": {
       "executor": "@nx/webpack:webpack",
@@ -62,6 +63,5 @@
         "jestConfig": "apps/cron-tasks/jest.config.ts"
       }
     }
-  },
-  "tags": ["scope:server"]
+  }
 }

apps/docs/eslint.config.js

@@ -0,0 +1,28 @@
+const baseConfig = require('../../eslint.config.js');
+
+module.exports = [
+  ...baseConfig,
+  {
+    ignores: ['.docusaurus', 'build', 'node_modules'],
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+    languageOptions: {
+      parserOptions: {
+        project: ['apps/docs/tsconfig.*?.json'],
+      },
+    },
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+];

apps/download-zip-sw/eslint.config.js

@@ -0,0 +1,20 @@
+const baseConfig = require('../../eslint.config.js');
+
+module.exports = [
+  ...baseConfig,
+  {
+    files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+];

apps/download-zip-sw/project.json

@@ -3,6 +3,7 @@
   "$schema": "../../node_modules/nx/schemas/project-schema.json",
   "projectType": "application",
   "sourceRoot": "apps/download-zip-sw/src",
+  "tags": ["sw"],
   "targets": {
     "build": {
       "executor": "@nx/webpack:webpack",
@@ -73,6 +74,5 @@
         "jestConfig": "apps/download-zip-sw/jest.config.ts"
       }
     }
-  },
-  "tags": ["sw"]
+  }
 }

apps/download-zip-sw/src/app/Zip.ts

@@ -48,11 +48,19 @@ class Zip {
 
   // To also work with the node version of readable stream (for testing)
   enqueue = (data) => {
-    this.outputController ? this.outputController.enqueue(data) : this.outputStream.push(data);
+    if (this.outputController) {
+      this.outputController.enqueue(data);
+    } else {
+      this.outputStream.push(data);
+    }
   };
 
   close = () => {
-    this.outputController ? this.outputController.close() : this.outputStream.destroy();
+    if (this.outputController) {
+      this.outputController.close();
+    } else {
+      this.outputStream.destroy();
+    }
   };
 
   // Generators

apps/download-zip-sw/src/main.sw.ts

@@ -44,7 +44,7 @@ const zipMap: Map<
     files: DownZipFile[];
     name: string;
     zip: Zip;
-    sizeBig: BigInt;
+    sizeBig: bigint;
   }
 > = new Map();
 

apps/geo-ip-api/eslint.config.js

@@ -0,0 +1,20 @@
+const baseConfig = require('../../eslint.config.js');
+
+module.exports = [
+  ...baseConfig,
+  {
+    files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    // Override or add rules here
+    rules: {},
+  },
+  {
+    files: ['**/*.js', '**/*.jsx'],
+    // Override or add rules here
+    rules: {},
+  },
+];