chore: Adds initial support for eslint.

api/eslint.config.js

@@ -0,0 +1,10 @@
+import globals from "globals";
+import pluginJs from "@eslint/js";
+import tseslint from "typescript-eslint";
+
+
+export default [
+  {languageOptions: { globals: globals.node }},
+  pluginJs.configs.recommended,
+  ...tseslint.configs.recommended,
+];

api/package.json

@@ -14,14 +14,18 @@
   "author": "greysoh",
   "license": "BSD-3-Clause",
   "devDependencies": {
+    "@eslint/js": "^9.2.0",
     "@types/bcrypt": "^5.0.2",
     "@types/node": "^20.12.7",
     "@types/ssh2": "^1.15.0",
     "@types/ws": "^8.5.10",
+    "eslint": "^8.57.0",
+    "globals": "^15.2.0",
     "nodemon": "^3.0.3",
     "prettier": "^3.2.5",
     "prisma": "^5.13.0",
-    "typescript": "^5.3.3"
+    "typescript": "^5.3.3",
+    "typescript-eslint": "^7.8.0"
   },
   "dependencies": {
     "@fastify/websocket": "^10.0.1",

api/src/backendimpl/base.ts

@@ -1,3 +1,5 @@
+// @eslint-ignore-file
+
 export type ParameterReturnedValue = {
   success: boolean;
   message?: string;

api/src/backendimpl/index.ts

@@ -10,4 +10,4 @@ export const backendProviders: Record<string, typeof BackendBaseClass> = {
 
 if (process.env.NODE_ENV != "production") {
   backendProviders["dummy"] = BackendBaseClass;
-}
+}

api/src/backendimpl/passyfire-reimpl/index.ts

@@ -216,7 +216,7 @@ export class PassyFireBackendProvider implements BackendBaseClass {
   static checkParametersBackendInstance(data: string): ParameterReturnedValue {
     try {
       parseBackendProviderString(data);
-      // @ts-ignore
+      // @ts-expect-error
     } catch (e: Error) {
       return {
         success: false,

api/src/backendimpl/passyfire-reimpl/routes.ts

@@ -69,7 +69,7 @@ export function route(instance: PassyFireBackendProvider) {
       },
     },
     (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         username: string;
         password: string;
@@ -115,7 +115,7 @@ export function route(instance: PassyFireBackendProvider) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
       } = req.body;
@@ -132,7 +132,7 @@ export function route(instance: PassyFireBackendProvider) {
         req.hostname.indexOf(":") + 1,
       );
 
-      // @ts-ignore
+      // @ts-expect-error
       // parseInt(...) can take a number just fine, at least in Node.JS
       const port = parseInt(unparsedPort == "" ? proxiedPort : unparsedPort);
 

api/src/backendimpl/passyfire-reimpl/socket.ts

@@ -57,7 +57,7 @@ export function requestHandler(
   let state: "authentication" | "data" = "authentication";
   let socket: dgram.Socket | net.Socket | undefined;
 
-  // @ts-ignore
+  // @ts-expect-error
   let connectedClient: ConnectedClientExt = {};
 
   ws.on("close", () => {

api/src/backendimpl/ssh.ts

@@ -92,7 +92,7 @@ export class SSHBackendProvider implements BackendBaseClass {
       this.logs.push(`Failed to start SSHBackendProvider! Error: '${e}'`);
       this.state = "stopped";
 
-      // @ts-ignore
+      // @ts-expect-error
       this.sshInstance = null;
 
       return false;
@@ -112,7 +112,7 @@ export class SSHBackendProvider implements BackendBaseClass {
 
     this.sshInstance.dispose();
 
-    // @ts-ignore
+    // @ts-expect-error
     this.sshInstance = null;
 
     this.logs.push("Successfully stopped SSHBackendProvider.");
@@ -255,7 +255,7 @@ export class SSHBackendProvider implements BackendBaseClass {
   static checkParametersBackendInstance(data: string): ParameterReturnedValue {
     try {
       parseBackendProviderString(data);
-      // @ts-ignore
+      // @ts-expect-error
     } catch (e: Error) {
       return {
         success: false,

api/src/libs/permissions.ts

@@ -27,7 +27,7 @@ export const permissionListDisabled: Record<string, boolean> = {
 };
 
 // FIXME: This solution fucking sucks.
-export let permissionListEnabled: Record<string, boolean> = JSON.parse(
+export const permissionListEnabled: Record<string, boolean> = JSON.parse(
   JSON.stringify(permissionListDisabled),
 );
 

api/src/routes/backends/create.ts

@@ -35,7 +35,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         name: string;

api/src/routes/backends/lookup.ts

@@ -33,7 +33,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         id?: number;

api/src/routes/backends/remove.ts

@@ -30,7 +30,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         id: number;

api/src/routes/forward/connections.ts

@@ -27,7 +27,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         id: number;

api/src/routes/forward/create.ts

@@ -50,7 +50,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
 

api/src/routes/forward/lookup.ts

@@ -41,7 +41,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
 

api/src/routes/forward/remove.ts

@@ -30,7 +30,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         id: number;

api/src/routes/forward/start.ts

@@ -30,7 +30,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         id: number;
@@ -59,7 +59,7 @@ export function route(routeOptions: RouteOptions) {
         });
 
       // Other restrictions in place make it so that it MUST be either TCP or UDP
-      // @ts-ignore
+      // @ts-expect-error
       const protocol: "tcp" | "udp" = forward.protocol;
 
       backends[forward.destProviderID].addConnection(

api/src/routes/forward/stop.ts

@@ -30,7 +30,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         id: number;
@@ -59,7 +59,7 @@ export function route(routeOptions: RouteOptions) {
         });
 
       // Other restrictions in place make it so that it MUST be either TCP or UDP
-      // @ts-ignore
+      // @ts-expect-error
       const protocol: "tcp" | "udp" = forward.protocol;
 
       backends[forward.destProviderID].removeConnection(

api/src/routes/getPermissions.ts

@@ -22,7 +22,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
       } = req.body;

api/src/routes/user/create.ts

@@ -29,7 +29,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         name: string;
         email: string;
@@ -87,9 +87,9 @@ export function route(routeOptions: RouteOptions) {
       }
 
       if (options.allowUnsafeGlobalTokens) {
-        // @ts-ignore
+        // @ts-expect-error
         userData.rootToken = generateRandomData();
-        // @ts-ignore
+        // @ts-expect-error
         userData.isRootServiceAccount = true;
       }
 

api/src/routes/user/login.ts

@@ -26,7 +26,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         email?: string;
         username?: string;

api/src/routes/user/lookup.ts

@@ -31,7 +31,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         id?: number;

api/src/routes/user/remove.ts

@@ -30,7 +30,7 @@ export function route(routeOptions: RouteOptions) {
       },
     },
     async (req, res) => {
-      // @ts-ignore
+      // @ts-expect-error
       const body: {
         token: string;
         uid: number;