@e22m4u/js-repository 0.6.4 → 0.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@e22m4u/js-repository",
-  "version": "0.6.4",
+  "version": "0.7.0",
   "description": "Реализация репозитория для работы с базами данных",
   "author": "Mikhail Evstropov <e22m4u@yandex.ru>",
   "license": "MIT",
@@ -41,7 +41,7 @@
   "dependencies": {
     "@e22m4u/js-empty-values": "~0.1.3",
     "@e22m4u/js-format": "~0.2.1",
-    "@e22m4u/js-service": "~0.4.5"
+    "@e22m4u/js-service": "~0.4.6"
   },
   "devDependencies": {
     "@commitlint/cli": "~20.1.0",
@@ -51,21 +51,21 @@
     "@types/chai-spies": "~1.0.6",
     "@types/mocha": "~10.0.10",
     "c8": "~10.1.3",
-    "chai": "~6.2.0",
+    "chai": "~6.2.1",
     "chai-as-promised": "~8.0.2",
     "chai-spies": "~1.1.0",
-    "esbuild": "~0.25.12",
+    "esbuild": "~0.27.0",
     "eslint": "~9.39.1",
     "eslint-config-prettier": "~10.1.8",
     "eslint-plugin-chai-expect": "~3.1.0",
-    "eslint-plugin-jsdoc": "~61.1.12",
+    "eslint-plugin-jsdoc": "~61.4.1",
     "eslint-plugin-mocha": "~11.2.0",
     "husky": "~9.1.7",
     "mocha": "~11.7.5",
     "prettier": "~3.6.2",
-    "rimraf": "~6.1.0",
+    "rimraf": "~6.1.2",
     "ts-node": "~10.9.2",
     "typescript": "~5.9.3",
-    "typescript-eslint": "~8.46.3"
+    "typescript-eslint": "~8.48.0"
   }
 }

package/src/adapter/adapter.d.ts

@@ -12,11 +12,9 @@ import {ServiceContainer} from '@e22m4u/js-service';
  */
 export declare class Adapter extends Service {
   /**
-   * Kind.
-   *
-   * @type {string}
+   * Kinds.
    */
-  static kind: string;
+  static kinds: string[];
 
   /**
    * Settings.

package/src/adapter/adapter.js

@@ -4,10 +4,8 @@ import {Service} from '@e22m4u/js-service';
 import {NotImplementedError} from '../errors/index.js';
 import {InclusionDecorator} from './decorator/index.js';
 import {DefaultValuesDecorator} from './decorator/index.js';
-import {DataValidationDecorator} from './decorator/index.js';
 import {DataSanitizingDecorator} from './decorator/index.js';
 import {FieldsFilteringDecorator} from './decorator/index.js';
-import {DataTransformationDecorator} from './decorator/index.js';
 import {PropertyUniquenessDecorator} from './decorator/index.js';
 
 /**
@@ -22,9 +20,9 @@ export const ADAPTER_CLASS_NAME = 'Adapter';
  */
 export class Adapter extends Service {
   /**
-   * Kind.
+   * Kinds.
    *
-   * @type {string}
+   * @type {string[]}
    */
   static kinds = [...Service.kinds, ADAPTER_CLASS_NAME];
 
@@ -57,8 +55,6 @@ export class Adapter extends Service {
     if (this.constructor !== Adapter) {
       this.getService(DataSanitizingDecorator).decorate(this);
       this.getService(DefaultValuesDecorator).decorate(this);
-      this.getService(DataTransformationDecorator).decorate(this);
-      this.getService(DataValidationDecorator).decorate(this);
       this.getService(PropertyUniquenessDecorator).decorate(this);
       this.getService(FieldsFilteringDecorator).decorate(this);
       this.getService(InclusionDecorator).decorate(this);

package/src/adapter/adapter.spec.js

@@ -7,10 +7,8 @@ import {ServiceContainer} from '@e22m4u/js-service';
 import {DatabaseSchema} from '../database-schema.js';
 import {InclusionDecorator} from './decorator/index.js';
 import {DefaultValuesDecorator} from './decorator/index.js';
-import {DataValidationDecorator} from './decorator/index.js';
 import {DataSanitizingDecorator} from './decorator/index.js';
 import {FieldsFilteringDecorator} from './decorator/index.js';
-import {DataTransformationDecorator} from './decorator/index.js';
 import {PropertyUniquenessDecorator} from './decorator/index.js';
 
 const sandbox = chai.spy.sandbox();
@@ -45,11 +43,9 @@ describe('Adapter', function () {
       const dbs = new DatabaseSchema();
       const dec1 = dbs.getService(DataSanitizingDecorator);
       const dec2 = dbs.getService(DefaultValuesDecorator);
-      const dec3 = dbs.getService(DataTransformationDecorator);
-      const dec4 = dbs.getService(DataValidationDecorator);
-      const dec5 = dbs.getService(PropertyUniquenessDecorator);
-      const dec6 = dbs.getService(FieldsFilteringDecorator);
-      const dec7 = dbs.getService(InclusionDecorator);
+      const dec3 = dbs.getService(PropertyUniquenessDecorator);
+      const dec4 = dbs.getService(FieldsFilteringDecorator);
+      const dec5 = dbs.getService(InclusionDecorator);
       const order = [];
       const decorate = function (ctx) {
         expect(ctx).to.be.instanceof(Adapter);
@@ -60,8 +56,6 @@ describe('Adapter', function () {
       sandbox.on(dec3, 'decorate', decorate);
       sandbox.on(dec4, 'decorate', decorate);
       sandbox.on(dec5, 'decorate', decorate);
-      sandbox.on(dec6, 'decorate', decorate);
-      sandbox.on(dec7, 'decorate', decorate);
       new Adapter(dbs.container);
       expect(order).to.be.empty;
       expect(dec1.decorate).to.be.not.called;
@@ -69,8 +63,6 @@ describe('Adapter', function () {
       expect(dec3.decorate).to.be.not.called;
       expect(dec4.decorate).to.be.not.called;
       expect(dec5.decorate).to.be.not.called;
-      expect(dec6.decorate).to.be.not.called;
-      expect(dec7.decorate).to.be.not.called;
       class ExtendedAdapter extends Adapter {}
       new ExtendedAdapter(dbs.container);
       expect(order[0]).to.be.eql(dec1);
@@ -78,15 +70,11 @@ describe('Adapter', function () {
       expect(order[2]).to.be.eql(dec3);
       expect(order[3]).to.be.eql(dec4);
       expect(order[4]).to.be.eql(dec5);
-      expect(order[5]).to.be.eql(dec6);
-      expect(order[6]).to.be.eql(dec7);
       expect(dec1.decorate).to.be.called.once;
       expect(dec2.decorate).to.be.called.once;
       expect(dec3.decorate).to.be.called.once;
       expect(dec4.decorate).to.be.called.once;
       expect(dec5.decorate).to.be.called.once;
-      expect(dec6.decorate).to.be.called.once;
-      expect(dec7.decorate).to.be.called.once;
     });
   });
 

package/src/adapter/builtin/memory-adapter.js

@@ -50,10 +50,9 @@ export class MemoryAdapter extends Adapter {
    * @returns {number}
    */
   _genNextIdValue(modelName, propName) {
-    const propType = this.getService(
-      ModelDefinitionUtils,
-    ).getDataTypeByPropertyName(modelName, propName);
-    if (propType !== DataType.ANY && propType !== DataType.NUMBER)
+    const modelUtils = this.getService(ModelDefinitionUtils);
+    const propType = modelUtils.getDataTypeByPropertyName(modelName, propName);
+    if (propType !== DataType.ANY && propType !== DataType.NUMBER) {
       throw new InvalidArgumentError(
         'The memory adapter able to generate only Number identifiers, ' +
           'but the primary key %v of the model %v is defined as %s. ' +
@@ -65,18 +64,36 @@ export class MemoryAdapter extends Adapter {
         capitalize(propType),
         propName,
       );
-    const tableName =
-      this.getService(ModelDefinitionUtils).getTableNameByModelName(modelName);
-    const lastId = this._lastIds.get(tableName) ?? 0;
-    const nextId = lastId + 1;
-    this._lastIds.set(tableName, nextId);
+    }
+    const tableName = modelUtils.getTableNameByModelName(modelName);
     const table = this._getTableOrCreate(modelName);
-    const existedIds = Array.from(table.keys());
-    if (existedIds.includes(nextId))
-      return this._genNextIdValue(modelName, propName);
+    let nextId = this._lastIds.get(tableName) ?? 0;
+    do {
+      nextId++;
+    } while (table.has(nextId));
+    this._lastIds.set(tableName, nextId);
     return nextId;
   }
 
+  /**
+   * Update last id value if needed.
+   *
+   * Если переданное значение последнего использованного
+   * идентификатора больше текущего, то текущее значение
+   * перезаписывается полученным.
+   *
+   * @param {string} modelName
+   * @param {number} idValue
+   */
+  _updateLastIdValueIfNeeded(modelName, idValue) {
+    const tableName =
+      this.getService(ModelDefinitionUtils).getTableNameByModelName(modelName);
+    const currentLastId = this._lastIds.get(tableName) ?? 0;
+    if (idValue > currentLastId) {
+      this._lastIds.set(tableName, idValue);
+    }
+  }
+
   /**
    * Create
    *
@@ -95,6 +112,12 @@ export class MemoryAdapter extends Adapter {
     if (idValue == null || idValue === '' || idValue === 0) {
       idValue = this._genNextIdValue(modelName, pkPropName);
     }
+    // если идентификатор передан вручную и является числом,
+    // то значение последнего использованного идентификатора
+    // обновляется на полученное
+    else if (typeof idValue === 'number') {
+      this._updateLastIdValueIfNeeded(modelName, idValue);
+    }
 
     const table = this._getTableOrCreate(modelName);
     if (table.has(idValue))
@@ -174,6 +197,12 @@ export class MemoryAdapter extends Adapter {
     if (idValue == null || idValue === '' || idValue === 0) {
       idValue = this._genNextIdValue(modelName, pkPropName);
     }
+    // если идентификатор передан вручную и является числом,
+    // то значение последнего использованного идентификатора
+    // обновляется на полученное
+    else if (typeof idValue === 'number') {
+      this._updateLastIdValueIfNeeded(modelName, idValue);
+    }
 
     const table = this._getTableOrCreate(modelName);
     modelData = cloneDeep(modelData);

package/src/adapter/builtin/memory-adapter.spec.js

@@ -56,6 +56,53 @@ describe('MemoryAdapter', function () {
       expect(id2).to.be.eq(2);
       expect(id3).to.be.eq(3);
     });
+
+    it('increments from the the last known identifier', function () {
+      const dbs = new DatabaseSchema();
+      dbs.defineModel({name: 'model'});
+      const A = dbs.getService(MemoryAdapter);
+      A._lastIds.set('model', 10);
+      const id1 = A._genNextIdValue('model', DEF_PK);
+      const id2 = A._genNextIdValue('model', DEF_PK);
+      const id3 = A._genNextIdValue('model', DEF_PK);
+      expect(id1).to.be.eq(11);
+      expect(id2).to.be.eq(12);
+      expect(id3).to.be.eq(13);
+    });
+  });
+
+  describe('_updateLastIdValueIfNeeded', function () {
+    it('does nothing when the given identifier is lower or equal', function () {
+      const dbs = new DatabaseSchema();
+      dbs.defineModel({name: 'model'});
+      const A = dbs.getService(MemoryAdapter);
+      A._lastIds.set('model', 10);
+      A._updateLastIdValueIfNeeded('model', 9);
+      A._updateLastIdValueIfNeeded('model', 10);
+      const res = A._lastIds.get('model');
+      expect(res).to.be.eq(10);
+    });
+
+    it('updates the last known identifier if the given identifier is greater', function () {
+      const dbs = new DatabaseSchema();
+      dbs.defineModel({name: 'model'});
+      const A = dbs.getService(MemoryAdapter);
+      A._lastIds.set('model', 10);
+      A._updateLastIdValueIfNeeded('model', 15);
+      const res = A._lastIds.get('model');
+      expect(res).to.be.eq(15);
+    });
+
+    it('correctly resolves table name from model name', function () {
+      const dbs = new DatabaseSchema();
+      dbs.defineModel({name: 'MyModel', tableName: 'custom_table_name'});
+      const A = dbs.getService(MemoryAdapter);
+      A._lastIds.set('custom_table_name', 10);
+      A._updateLastIdValueIfNeeded('MyModel', 20);
+      const res = A._lastIds.get('custom_table_name');
+      expect(res).to.be.eq(20);
+      expect(A._lastIds.has('MyModel')).to.be.false;
+    });
   });
 
   describe('create', function () {

package/src/adapter/decorator/index.d.ts

@@ -1,7 +1,5 @@
 export * from './inclusion-decorator.js';
 export * from './default-values-decorator.js';
 export * from './data-sanitizing-decorator.js';
-export * from './data-validation-decorator.js';
 export * from './fields-filtering-decorator.js';
-export * from './data-transformation-decorator.js';
 export * from './property-uniqueness-decorator.js';

package/src/adapter/decorator/index.js

@@ -1,7 +1,5 @@
 export * from './inclusion-decorator.js';
 export * from './default-values-decorator.js';
 export * from './data-sanitizing-decorator.js';
-export * from './data-validation-decorator.js';
 export * from './fields-filtering-decorator.js';
-export * from './data-transformation-decorator.js';
 export * from './property-uniqueness-decorator.js';

package/src/definition/model/index.d.ts

@@ -1,8 +1,6 @@
 export * from './relations/index.js';
 export * from './properties/index.js';
 export * from './model-definition.js';
-export * from './model-data-validator.js';
 export * from './model-data-sanitizer.js';
-export * from './model-data-transformer.js';
 export * from './model-definition-utils.js';
 export * from './model-definition-validator.js';

package/src/definition/model/index.js

@@ -1,8 +1,6 @@
 export * from './relations/index.js';
 export * from './properties/index.js';
 export * from './model-definition.js';
-export * from './model-data-validator.js';
 export * from './model-data-sanitizer.js';
-export * from './model-data-transformer.js';
 export * from './model-definition-utils.js';
 export * from './model-definition-validator.js';

package/src/definition/model/properties/index.d.ts

@@ -1,8 +1,6 @@
 export * from './data-type.js';
 export * from './property-definition.js';
 export * from './property-uniqueness.js';
-export * from './property-validator/index.js';
-export * from './property-transformer/index.js';
 export * from './property-uniqueness-validator.js';
 export * from './properties-definition-validator.js';
 export * from './primary-keys-definition-validator.js';

package/src/definition/model/properties/index.js

@@ -1,8 +1,6 @@
 export * from './data-type.js';
 export * from './property-definition.js';
 export * from './property-uniqueness.js';
-export * from './property-validator/index.js';
-export * from './property-transformer/index.js';
 export * from './property-uniqueness-validator.js';
 export * from './properties-definition-validator.js';
 export * from './primary-keys-definition-validator.js';

package/src/definition/model/properties/properties-definition-validator.js

@@ -3,8 +3,6 @@ import {DataType as Type} from './data-type.js';
 import {capitalize} from '../../../utils/index.js';
 import {PropertyUniqueness} from './property-uniqueness.js';
 import {InvalidArgumentError} from '../../../errors/index.js';
-import {PropertyValidatorRegistry} from './property-validator/index.js';
-import {PropertyTransformerRegistry} from './property-transformer/index.js';
 import {PrimaryKeysDefinitionValidator} from './primary-keys-definition-validator.js';
 
 /**
@@ -228,173 +226,6 @@ export class PropertiesDefinitionValidator extends Service {
         propName,
         modelName,
       );
-    // если определена опция "validate", то проверяется значение
-    // опции, которое может являться строкой, массивом или объектом
-    if (propDef.validate != null) {
-      const propertyValidatorRegistry = this.getService(
-        PropertyValidatorRegistry,
-      );
-      // если опция "validate" содержит строку, то проверяется
-      // наличие зарегистрированного валидатора по названию
-      if (propDef.validate && typeof propDef.validate === 'string') {
-        // если название валидатора не зарегистрировано,
-        // то выбрасывается ошибка
-        if (!propertyValidatorRegistry.hasValidator(propDef.validate))
-          throw new InvalidArgumentError(
-            'The property validator %v is not found.',
-            propDef.validate,
-          );
-      }
-      // если опция "validate" содержит функцию, то данная функция
-      // воспринимается как пользовательский валидатор
-      else if (propDef.validate && typeof propDef.validate === 'function') {
-        // (допустимо)
-      }
-      // если опция "validate" содержит массив, то проверяется
-      // каждый элемент массива
-      else if (Array.isArray(propDef.validate)) {
-        for (const validatorOrName of propDef.validate) {
-          // если элемент массива является строкой, то проверяется
-          // наличие зарегистрированного валидатора по названию
-          if (validatorOrName && typeof validatorOrName === 'string') {
-            if (!propertyValidatorRegistry.hasValidator(validatorOrName))
-              throw new InvalidArgumentError(
-                'The property validator %v is not found.',
-                validatorOrName,
-              );
-          }
-          // если элемент массива является функцией, то данная функция
-          // воспринимается как пользовательский валидатор
-          else if (validatorOrName && typeof validatorOrName === 'function') {
-            // (допустимо)
-          }
-          // если элемент массива не является названием зарегистрированного
-          // валидатора или функцией-валидатором, то выбрасывается ошибка
-          else {
-            throw new InvalidArgumentError(
-              'The provided option "validate" for the property %v in the model %v ' +
-                'has an Array value that should contain validator names or validator ' +
-                'functions, but %v was given.',
-              propName,
-              modelName,
-              validatorOrName,
-            );
-          }
-        }
-      }
-      // если опция "validate" содержит объект, то проверяются ключи данного
-      // объекта, которые должны являться названиями зарегистрированных
-      // валидаторов, а значения их аргументами
-      else if (typeof propDef.validate === 'object') {
-        Object.keys(propDef.validate).forEach(validatorName => {
-          // если ключ объекта не является названием зарегистрированного
-          // валидатора, то выбрасывается ошибка
-          if (!propertyValidatorRegistry.hasValidator(validatorName))
-            throw new InvalidArgumentError(
-              'The property validator %v is not found.',
-              validatorName,
-            );
-        });
-      }
-      // если опция "validate" не является не пустой строкой, функцией,
-      // массивом и объектом, то выбрасывается ошибка
-      else {
-        throw new InvalidArgumentError(
-          'The provided option "validate" for the property %v in the model %v ' +
-            'should be either a validator name, a validator function, an array ' +
-            'of validator names or functions, or an object mapping validator ' +
-            'names to their arguments, but %v was given.',
-          propName,
-          modelName,
-          propDef.validate,
-        );
-      }
-    }
-    // если определена опция "transform", то проверяется значение
-    // опции, которое может являться строкой, массивом или объектом
-    if (propDef.transform != null) {
-      const propertyTransformerRegistry = this.getService(
-        PropertyTransformerRegistry,
-      );
-      // если опция "transform" содержит строку, то проверяется
-      // наличие зарегистрированного трансформера по названию
-      if (propDef.transform && typeof propDef.transform === 'string') {
-        // если название трансформера не зарегистрировано,
-        // то выбрасывается ошибка
-        if (!propertyTransformerRegistry.hasTransformer(propDef.transform))
-          throw new InvalidArgumentError(
-            'The property transformer %v is not found.',
-            propDef.transform,
-          );
-      }
-      // если опция "transform" содержит функцию, то данная функция
-      // воспринимается как пользовательский трансформер
-      else if (propDef.transform && typeof propDef.transform === 'function') {
-        // (допустимо)
-      }
-      // если опция "transform" содержит массив, то проверяется
-      // каждый элемент массива
-      else if (Array.isArray(propDef.transform)) {
-        for (const transformerOrName of propDef.transform) {
-          // если элемент массива является строкой, то проверяется
-          // наличие зарегистрированного трансформера по названию
-          if (transformerOrName && typeof transformerOrName === 'string') {
-            if (!propertyTransformerRegistry.hasTransformer(transformerOrName))
-              throw new InvalidArgumentError(
-                'The property transformer %v is not found.',
-                transformerOrName,
-              );
-          }
-          // если элемент массива является функцией, то данная функция
-          // воспринимается как пользовательский трансформер
-          else if (
-            transformerOrName &&
-            typeof transformerOrName === 'function'
-          ) {
-            // (допустимо)
-          }
-          // если элемент массива не является названием зарегистрированного
-          // трансформера или функцией-трансформером, то выбрасывается ошибка
-          else {
-            throw new InvalidArgumentError(
-              'The provided option "transform" for the property %v in the model %v ' +
-                'has an Array value that should contain transformer names or transformer ' +
-                'functions, but %v was given.',
-              propName,
-              modelName,
-              transformerOrName,
-            );
-          }
-        }
-      }
-      // если опция "transform" содержит объект, то проверяются ключи данного
-      // объекта, которые должны являться названиями зарегистрированных
-      // трансформеров, а значения их аргументами
-      else if (typeof propDef.transform === 'object') {
-        Object.keys(propDef.transform).forEach(transformerName => {
-          // если ключ объекта не является названием зарегистрированного
-          // валидатора, то выбрасывается ошибка
-          if (!propertyTransformerRegistry.hasTransformer(transformerName))
-            throw new InvalidArgumentError(
-              'The property transformer %v is not found.',
-              transformerName,
-            );
-        });
-      }
-      // если опция "transform" не является не пустой строкой, функцией,
-      // массивом и объектом, то выбрасывается ошибка
-      else {
-        throw new InvalidArgumentError(
-          'The provided option "transform" for the property %v in the model %v ' +
-            'should be either a transformer name, a transformer function, an array ' +
-            'of transformer names or functions, or an object mapping transformer ' +
-            'names to their arguments, but %v was given.',
-          propName,
-          modelName,
-          propDef.transform,
-        );
-      }
-    }
     if (propDef.unique) {
       if (
         typeof propDef.unique !== 'boolean' &&