style: update eslint rules and prettier config

Author: Char2sGu

.eslintrc.js

@@ -10,6 +10,7 @@ module.exports = {
     "simple-import-sort",
   ],
   extends: [
+    "eslint:recommended",
     "plugin:@typescript-eslint/recommended",
     "plugin:prettier/recommended",
   ],
@@ -22,10 +23,96 @@ module.exports = {
     "@typescript-eslint/no-explicit-any": "off",
     "@typescript-eslint/no-empty-function": "off",
     "@typescript-eslint/no-inferrable-types": "off",
-    "@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_" }],
     "@typescript-eslint/explicit-module-boundary-types": "error",
-    eqeqeq: "error",
+    "@typescript-eslint/no-duplicate-imports": "error",
+    "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
+    "@typescript-eslint/method-signature-style": ["error", "method"],
+    "@typescript-eslint/explicit-member-accessibility": [
+      "error",
+      { accessibility: "no-public" },
+    ],
+    "@typescript-eslint/consistent-type-assertions": [
+      "error",
+      { assertionStyle: "as", objectLiteralTypeAssertions: "never" },
+    ],
+    "@typescript-eslint/no-unused-vars": [
+      "error",
+      { argsIgnorePattern: "args" },
+    ],
+    "@typescript-eslint/naming-convention": [
+      "error",
+      { selector: "default", format: ["strictCamelCase"] },
+      { selector: "typeLike", format: ["PascalCase"] },
+      {
+        selector: "variable",
+        format: ["strictCamelCase", "UPPER_CASE"],
+      },
+      {
+        selector: "parameter",
+        modifiers: ["unused"],
+        format: ["strictCamelCase"],
+        leadingUnderscore: "allow",
+      },
+      { selector: "enumMember", format: ["PascalCase"] },
+    ],
+    "eqeqeq": "error",
+    "no-promise-executor-return": "error",
+    "no-self-compare": "error",
+    "no-template-curly-in-string": "error",
+    "no-unreachable-loop": "error",
+    "arrow-body-style": "error",
+    "consistent-return": "error",
+    "curly": ["error", "multi", "consistent"],
+    "default-case-last": "error",
+    "grouped-accessor-pairs": "error",
+    "no-alert": "error",
+    "no-bitwise": "error",
+    "no-console": "error",
+    "no-else-return": "error",
+    "no-empty": ["error", { allowEmptyCatch: true }],
+    "no-eval": "error",
+    "no-extend-native": "error",
+    "no-extra-label": "error",
+    "no-implied-eval": "error",
+    "no-label-var": "error",
+    "no-negated-condition": "error",
+    "no-new-wrappers": "error",
+    "no-return-assign": "error",
+    "no-return-await": "error",
+    "no-sequences": "error",
+    "no-throw-literal": "error",
+    "prefer-promise-reject-errors": "error",
+    "no-unneeded-ternary": "error",
+    "@typescript-eslint/no-unused-expressions": "error",
+    "no-useless-call": "error",
+    "no-useless-escape": "error",
+    "no-useless-rename": "error",
+    "no-useless-return": "error",
+    "no-void": "error",
+    "object-shorthand": "error",
+    "one-var": ["error", "never"],
+    "operator-assignment": "error",
+    "prefer-arrow-callback": "error",
+    "prefer-exponentiation-operator": "error",
+    "prefer-numeric-literals": "error",
+    "prefer-object-spread": "error",
+    "prefer-regex-literals": "error",
+    "prefer-spread": "error",
+    "require-unicode-regexp": "error",
+    "max-depth": ["warn", 4],
+    "max-lines": ["warn", 300],
+    "max-lines-per-function": ["warn", 50],
+    "complexity": ["warn", 8],
     "simple-import-sort/imports": "warn",
     "simple-import-sort/exports": "warn",
   },
+  overrides: [
+    {
+      files: ["**/*.{spec,test}.ts"],
+      rules: {
+        "max-lines": "off",
+        "max-lines-per-function": "off",
+      },
+    },
+  ],
 };

.prettierrc

@@ -1,3 +1,4 @@
 {
-  "trailingComma": "all"
+  "trailingComma": "all",
+  "quoteProps": "consistent"
 }

src/database/database-event.ts

@@ -8,13 +8,17 @@ export function forwardDatabaseEvents(
   dispatcher: EventDispatcher,
 ): void {
   database.unitOfWorkEvents.onInsertPre.subscribe(async (event) =>
-    dispatcher.dispatch(DatabasePreInsert, DatabaseUnitOfWorkEvent.from(event)),
+    dispatcher.dispatch(
+      DATABASE_PRE_INSERT,
+      DatabaseUnitOfWorkEvent.from(event),
+    ),
   );
 }
 
-export const DatabasePreInsert = new EventToken<
+export const DATABASE_PRE_INSERT = new EventToken<
   DatabaseUnitOfWorkEvent<unknown>
 >("database:pre-insert");
+
 // @deepkit/orm is built before @deepkit/event, so there are some differences
 // between their interfaces. Database events will migrate to @deepkit/event
 // in a future release.

src/user/user.entity.ts

@@ -9,7 +9,7 @@ import {
   Unique,
 } from "@deepkit/type";
 import { compare, hash } from "bcryptjs";
-import { DatabasePreInsert } from "src/database/database-event";
+import { DATABASE_PRE_INSERT } from "src/database/database-event";
 import { Entity } from "src/shared/entity";
 
 @entity.name("user")
@@ -33,7 +33,7 @@ export class User extends Entity {
 export type UserPassword = string & MinLength<6> & MaxLength<50>;
 
 export class UserEventListener {
-  @eventDispatcher.listen(DatabasePreInsert)
+  @eventDispatcher.listen(DATABASE_PRE_INSERT)
   async preInsert(event: UnitOfWorkEvent<User>): Promise<void> {
     if (event.classSchema.getClassType() !== User) return;
     await Promise.all(event.items.map((user) => user.hashPassword()));