feature: enable strict typescript features, code clean-up and remove …

package.json

@@ -22,7 +22,7 @@
         "pnpm": ">=7.5.1"
     },
     "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "types": "src/index.ts",
     "scripts": {
         "build": "turbo run build",
         "build:clean": "turbo run clean:build",

packages/authorization/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/authorization/package.json

@@ -15,8 +15,8 @@
     "publishConfig": {
         "access": "public"
     },
-    "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "main": "src/index.js",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/authorization/src/decorator/EnableAuthorization.ts

@@ -10,7 +10,7 @@ export function EnableAuthorization(
     CurrentUserCheckerClass?: new () => CurrentUserChecker,
     AuthorizationCheckerClass?: new () => AuthorizationChecker,
 ): Function {
-    return function (object: Function) {
+    return function () {
         if (AuthorizationCheckerClass) {
             ApplicationContext.get().authorizationChecker = new AuthorizationCheckerClass();
         }

packages/config/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/config/package.json

@@ -16,7 +16,7 @@
         "access": "public"
     },
     "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/context/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/context/package.json

@@ -16,7 +16,7 @@
         "access": "public"
     },
     "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/core/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/core/package.json

@@ -16,7 +16,7 @@
         "access": "public"
     },
     "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/core/src/decorators/Configurations.ts

@@ -1,5 +1,5 @@
 export function Configurations(configurationClasses: (new (...args: any[]) => any)[]): Function {
-    return function (target: any) {
+    return function () {
         configurationClasses.map(ClassConstructor => new ClassConstructor());
     };
 }

packages/core/src/decorators/Controllers.ts

@@ -1,7 +1,7 @@
 import {ApplicationContext} from "@node-boot/context";
 
 export function Controllers(controllers: Function[]): Function {
-    return function (target: any) {
+    return function () {
         ApplicationContext.get().controllerClasses = controllers;
     };
 }

packages/core/src/decorators/EnableClassTransformer.ts

@@ -2,19 +2,19 @@ import {ApplicationContext, TransformerOptions} from "@node-boot/context";
 import {ClassTransformOptions} from "class-transformer";
 
 export function ClassToPlainTransform(options: ClassTransformOptions): Function {
-    return function (target: Function) {
+    return function () {
         ApplicationContext.get().classToPlainTransformOptions = options;
     };
 }
 
 export function PlainToClassTransform(options: ClassTransformOptions): Function {
-    return function (target: Function) {
+    return function () {
         ApplicationContext.get().plainToClassTransformOptions = options;
     };
 }
 
 export function EnableClassTransformer(options?: TransformerOptions): Function {
-    return function (target: Function) {
+    return function () {
         ApplicationContext.get().classTransformer = options?.enabled ?? true;
 
         ApplicationContext.get().classToPlainTransformOptions = options?.classToPlain;

packages/core/src/decorators/EnableComponentScan.ts

@@ -2,7 +2,7 @@ import {ApplicationContext, ComponentScanOptions} from "@node-boot/context";
 import path from "path";
 
 export function EnableComponentScan(options?: ComponentScanOptions): Function {
-    return function (object: Function) {
+    return function () {
         options = options ?? {
             controllerPaths: ["/controllers"],
             middlewarePaths: ["/middlewares"],

packages/core/src/decorators/GlobalMiddlewares.ts

@@ -1,7 +1,7 @@
 import {ApplicationContext} from "@node-boot/context";
 
 export function GlobalMiddlewares(middlewares: Function[]): Function {
-    return function (target: any) {
+    return function () {
         ApplicationContext.get().globalMiddlewares = middlewares;
     };
 }

packages/core/src/decorators/Interceptors.ts

@@ -1,7 +1,7 @@
 import {ApplicationContext} from "@node-boot/context";
 
 export function Interceptors(interceptors: Function[]): Function {
-    return function (target: any) {
+    return function () {
         ApplicationContext.get().interceptorClasses = interceptors;
     };
 }

packages/core/src/decorators/index.ts

@@ -12,7 +12,6 @@ export * from "./CookieParam";
 export * from "./CookieParams";
 export * from "./Ctx";
 export * from "./Delete";
-export * from "./EnableAutoConfiguration";
 export * from "./EnableClassTransformer";
 export * from "./EnableComponentScan";
 export * from "./ErrorHandler";

packages/di/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/di/package.json

@@ -16,7 +16,7 @@
         "access": "public"
     },
     "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/di/src/decorators/EnableDI.ts

@@ -7,7 +7,7 @@ import {ApplicationContext, IocContainer, UseContainerOptions} from "@node-boot/
  * @param options Extra options for the IOC container
  */
 export function EnableDI<TContainer>(iocContainer: IocContainer<TContainer>, options?: UseContainerOptions): Function {
-    return function (object: Function) {
+    return function () {
         ApplicationContext.get().diOptions = {
             iocContainer,
             options,

packages/di/src/decorators/Inject.ts

@@ -16,12 +16,10 @@ export function Inject(options?: InjectionOptions): Function {
     return (target: Object, propertyName: string | Symbol, index?: number) => {
         // Registering metadata for custom filed injection (used for example in the Persistence Event Subscribers)
         if (propertyName && typeof propertyName === "string") {
-            const propertyType = Reflect.getMetadata("design:type", target, propertyName);
             const injectProperties: string[] = Reflect.getMetadata(REQUIRES_FIELD_INJECTION_KEY, target) || [];
             injectProperties.push(propertyName);
             Reflect.defineMetadata(REQUIRES_FIELD_INJECTION_KEY, injectProperties, target);
         }
-
         // Normal injection
         decorateInjection(target, propertyName, index, options);
     };

packages/engine/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/engine/package.json

@@ -16,8 +16,8 @@
     "publishConfig": {
         "access": "public"
     },
-    "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "main": "src/index.js",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/engine/src/core/NodeBootDriver.ts

@@ -79,7 +79,7 @@ export abstract class NodeBootDriver<TServer, TAction extends Action = Action> {
      */
     currentUserChecker?: CurrentUserChecker;
 
-    protected transformResult(result: any, actionMetadata: ActionMetadata, action: TAction): any {
+    protected transformResult<T = unknown>(result: any, actionMetadata: ActionMetadata): T {
         // check if we need to transform result
         const shouldTransform =
             this.useClassTransformer && // transform only if class-transformer is enabled

packages/error/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/error/package.json

@@ -15,8 +15,8 @@
     "publishConfig": {
         "access": "public"
     },
-    "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "main": "src/index.js",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/extension/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/extension/package.json

@@ -16,7 +16,7 @@
         "access": "public"
     },
     "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/extension/src/Optional.ts

@@ -196,11 +196,11 @@ export class Optional<T> {
             return Optional.of(this.value.filter(predicate) as T);
         } else if (this.value instanceof Map || this.value instanceof Set) {
             // Filter map or set
-            const filteredEntries = Array.from(this.value.entries()).filter(([key, value]) => predicate(value));
+            const filteredEntries = Array.from(this.value.entries()).filter(([, value]) => predicate(value));
             if (this.value instanceof Map) {
                 return Optional.of(new Map(filteredEntries) as T);
             } else {
-                return Optional.of(new Set(filteredEntries.map(([key, value]) => value)) as T);
+                return Optional.of(new Set(filteredEntries.map(([, value]) => value)) as T);
             }
         } else {
             // Filter single value

packages/openapi/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",

packages/openapi/package.json

@@ -16,7 +16,7 @@
         "access": "public"
     },
     "main": "dist/index.js",
-    "types": "dist/index.d.ts",
+    "types": "src/index.ts",
     "scripts": {
         "build": "tsc -p tsconfig.build.json",
         "clean:build": "rimraf ./dist",

packages/openapi/src/adapter/ExpressOpenApi.ts

@@ -1,13 +1,14 @@
 import {OpenApiAdapter, OpenApiOptions} from "@node-boot/context";
 import swaggerUi from "swagger-ui-express";
 import {OpenApiSpecAdapter} from "./OpenApiSpecAdapter";
+import {Response} from "express";
 
 export class ExpressOpenApi implements OpenApiAdapter {
     bind(openApiOptions: OpenApiOptions, server: any, router: any): void {
         if (swaggerUi?.serve) {
             const {spec, options} = OpenApiSpecAdapter.adapt(openApiOptions);
 
-            router.get(options.swaggerOptions.url, (req, res) => res.json(spec));
+            router.get(options.swaggerOptions.url, (_: never, res: Response) => res.json(spec));
             server.use("/api-docs", swaggerUi.serve, swaggerUi.setup(spec, options));
         } else {
             throw new Error(

packages/openapi/src/adapter/FastifyOpenApi.ts

@@ -6,7 +6,7 @@ export class FastifyOpenApi implements OpenApiAdapter {
     bind(openApiOptions: OpenApiOptions, server: FastifyInstance, router: FastifyInstance): void {
         const {spec, options} = OpenApiSpecAdapter.adapt(openApiOptions);
 
-        router.get(options.swaggerOptions.url, async (request, reply) => {
+        router.get(options.swaggerOptions.url, async (_, reply) => {
             reply.send(spec);
         });
         server.register(require("@fastify/swagger"), {

packages/openapi/src/decorator/EnableOpenApi.ts

@@ -7,10 +7,10 @@ import {FastifyOpenApi} from "../adapter/FastifyOpenApi";
 /**
  * Defines the configurations to enable Swagger Open API
  *
- * @param openApi The OpenAPI definitions and base config
+ * @param _ The OpenAPI definitions and base config
  */
-export function EnableOpenApi(openApi: Partial<oa.OpenAPIObject> = {}): Function {
-    return function (object: Function) {
+export function EnableOpenApi(_: Partial<oa.OpenAPIObject> = {}): Function {
+    return function () {
         ApplicationContext.get().openApi = new (class implements OpenApiBridgeAdapter {
             bind(serverType: string): OpenApiAdapter {
                 switch (serverType) {

packages/openapi/src/decorator/OpenAPI.ts

@@ -11,7 +11,7 @@ import {OpenAPI as InnerOpenAPI} from "routing-controllers-openapi";
  * returning an updated Operation.
  */
 export function OpenAPI(...args: Parameters<typeof InnerOpenAPI>) {
-    return <TFunction extends Function>(...innerArgs: [Function] | [object, string, PropertyDescriptor]) => {
+    return (...innerArgs: [Function] | [object, string, PropertyDescriptor]) => {
         InnerOpenAPI(...args)(...innerArgs);
     };
 }

samples/sample-express/jest.config.js

@@ -1,4 +1,3 @@
-/** @type {import('jest').Config} */
 module.exports = {
     transform: {
         "^.+\\.(t|j)sx?$": "@swc/jest",