make deserializer and serializer optional and replace by a default on…

.github/workflows/test.yml

@@ -18,15 +18,15 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [18.x]
+        node-version: [18.x, 20.x]
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v3
+      uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
 
-    - uses: pnpm/action-setup@v2.2.4
+    - uses: pnpm/action-setup@v2
       name: Install pnpm
       id: pnpm-install
       with:
@@ -42,10 +42,10 @@ jobs:
     runs-on: ubuntu-latest
     needs: [test]
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Use Node.js
-      uses: actions/setup-node@v3
-    - uses: pnpm/action-setup@v2.2.4
+      uses: actions/setup-node@v4
+    - uses: pnpm/action-setup@v2
       name: Install pnpm
       id: pnpm-install
       with:

packages/http/src/decorators/use-guard.ts

@@ -1,4 +1,3 @@
-
 import { container } from '@triptyk/nfw-core';
 import type { Class } from 'type-fest';
 import type { GuardInterface } from '../interfaces/guard.js';

packages/http/src/decorators/use-middleware.ts

@@ -1,4 +1,3 @@
-
 import { container } from '@triptyk/nfw-core';
 import type { Class } from 'type-fest';
 import { MetadataStorage } from '../storages/metadata-storage.js';

packages/http/src/index.ts

@@ -1,4 +1,3 @@
-
 export * from './builders/default.js';
 export * from './decorators/verbs.js';
 export * from './decorators/controller.js';

packages/http/src/storages/metadata/use-middleware.ts

@@ -1,4 +1,3 @@
-
 import type { Class } from 'type-fest';
 import type { AnyMiddlewareType } from '../../types/any-middleware.js';
 

packages/http/src/storages/metadata/use-response-handler.ts

@@ -1,4 +1,3 @@
-
 import type { Class } from 'type-fest';
 import type { ResponseHandlerInterface } from '../../interfaces/response-handler.js';
 

packages/http/src/utils/numerical-sort.ts

@@ -1,4 +1,3 @@
-
 export function numericalSortOnKeyASC<O extends Record<string, any>> (array : O[], propertyName: keyof O) {
   return [...array].sort((a, b) => a[propertyName] - b[propertyName]);
 }

packages/http/src/utils/resolve-params.ts

@@ -1,4 +1,3 @@
-
 import { ParamResolver } from '../resolvers/param-resolver.js';
 import type { UseParamsMetadataArgs } from '../storages/metadata/use-param.js';
 import type { ControllerContextType } from '../types/controller-context.js';

packages/http/tests/integration/controller-action-storage.test.ts

@@ -12,7 +12,7 @@ const rejects = createCustomDecorator(() => Promise.reject(new Error('test')), '
 })
 class C {
   @POST('/')
-  public test (@rejects reject: string) {
+  public test (@rejects _reject: string) {
     return 'test';
   };
 }

packages/resource-jsonapi/src/query/validators/sort.validator.ts

@@ -24,7 +24,7 @@ export class SortValidator {
     }
   }
 
-  // eslint-disable-next-line max-statements
+  // eslint-disable-next-line max-statements, complexity
   private recursiveSortValidation (sort: SortQuery, field: string, schema: ResourceSchema<Resource>) {
     const directionOrSortQuery = sort[field];
 

packages/resource-jsonapi/src/registry/registry.ts

@@ -1,10 +1,13 @@
+/* eslint-disable max-classes-per-file */
 /* eslint-disable class-methods-use-this */
 import { container, singleton } from '@triptyk/nfw-core';
 import type { Class } from 'type-fest';
 import type { ResourceDeserializer } from '../interfaces/deserializer.js';
 import type { Resource } from '../interfaces/resource.js';
 import type { ResourceSchema } from '../interfaces/schema.js';
 import type { ResourceSerializer } from '../interfaces/serializer.js';
+import { JsonApiResourceSerializer } from '../serialization/serializer.js';
+import { JsonApiResourceDeserializer } from '../serialization/deserializer.js';
 
 export interface BaseConfig {
   host: string,
@@ -35,11 +38,30 @@ export class ResourcesRegistryImpl implements ResourcesRegistry {
     return container.resolve(`deserializer:${type}`);
   }
 
+  // eslint-disable-next-line max-statements
   register<T extends Resource> (type: string, classes: {
-    serializer: Class<ResourceSerializer>,
-    deserializer: Class<ResourceDeserializer<T>>,
+    serializer?: Class<ResourceSerializer>,
+    deserializer?: Class<ResourceDeserializer<T>>,
     schema: ResourceSchema<T>,
   }): void {
+    const registry = this;
+
+    if (!classes.serializer) {
+      classes.serializer = class DefaultSerializer extends JsonApiResourceSerializer {
+        public constructor () {
+          super(registry);
+        }
+      };
+    }
+
+    if (!classes.deserializer) {
+      classes.deserializer = class DefaultDeserializer extends JsonApiResourceDeserializer<T> {
+        public constructor () {
+          super(classes.schema.resourceType, registry);
+        }
+      };
+    }
+
     container.register(`serializer:${type}`, { useClass: classes.serializer });
     container.register(`deserializer:${type}`, { useClass: classes.deserializer });
     container.register(`schema:${type}`, {

packages/resource-jsonapi/src/serialization/document.ts

@@ -52,13 +52,13 @@ export class DocumentSerializer {
     return doc;
   }
 
-  private getSchemaFor(resource: Resource) {
+  private getSchemaFor (resource: Resource) {
     if (!resource.resourceType) {
       throw new Error(`Resource ${resource} has no resourceType assigned`);
     }
 
     try {
-     return this.registry.getSchemaFor(resource.resourceType);
+      return this.registry.getSchemaFor(resource.resourceType);
     } catch (e) {
       throw new Error(`No schema found for resource type ${resource.resourceType}`);
     }