@bearei/nest 2.0.0-dev.2 → 2.0.0-dev.4

package/CHANGELOG.md

@@ -4,6 +4,24 @@ All notable changes to this project are documented in this file. The format is b
 [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 2.0.0-dev.4 — 2026-10-27
+
+Development snapshot published under the `next` dist-tag for downstream service refactors. It
+includes the 409 conflict mapping for writes that lose the pre-validation race and depends on
+`@bearei/contracts@^2.1.0`. Its contents otherwise match the 2.0.0 scope documented below; it does
+not replace the final 2.0.0 release notes.
+
+Install it with `yarn add @bearei/nest@next` or pin `@bearei/nest@2.0.0-dev.4`.
+
+## 2.0.0-dev.3 — 2026-10-27
+
+Development snapshot published under the `next` dist-tag to verify the unified release pipeline
+across all Bearei repositories: the artifact is pulled back from the registry, re-verified and
+attached to the GitHub release. Its contents match the 2.0.0 scope documented below; it does not
+replace the final 2.0.0 release notes.
+
+Install it with `yarn add @bearei/nest@next` or pin `@bearei/nest@2.0.0-dev.3`.
+
 ## 2.0.0-dev.2 — 2026-10-27
 
 Development snapshot published under the `next` dist-tag to verify the npm trusted publishing
@@ -61,6 +79,7 @@ baseline; because the npm identity is new, the Breaking list below is also the m
 - `CommonEntity`: nanoid public `id`, internal numeric id, optimistic-locking `version`, UTC timestamps and idempotent inserts.
 - `ListQueryDto`, `ByIdDto`, `ByEmailDto`, `ExcludeIdDto`, plus `PaginatedResult`, `ReservedRequestHeaders`, `AuthenticatedRequest` and string-literal union types for the shared enums.
 - `ApiErrorBody` and `ReservedRequestHeaderName`, re-exported from `@bearei/contracts`.
+- `HTTP_EXCEPTION.CONFLICT` for concurrent writes that collide with existing data, together with `HTTP_EXCEPTION_CODE.CONFLICT` (`409000`) from `@bearei/contracts`.
 - `createResponseSanitizer` (replaces `createSanitizeResponseEntity`), `formatMobileNumber` (replaces `buildMobileNumber`) and `generateRandomNumberString` (replaces `generateRandomNumber`).
 - `runWithDatabaseRetry` for transient database connection failures.
 - `AllowAccess`, `RequirePermissions` and `RequireSubscriptionFeatures` decorators, alongside the existing validators.
@@ -82,3 +101,7 @@ baseline; because the npm identity is new, the Breaking list below is also the m
 - The `crypto` runtime dependency; signing uses Node's built-in `crypto`.
 - `ENTITY_STATUS`, `EntityStatus`, `entityStatus`, `beforeSoftRemove` and `beforeRecover`.
 - The old `access/`, `app.*` and `authentication.service` modules.
+
+### Fixed
+
+- Writes that lose the pre-validation race now return `409 Conflict` with `CONFLICT` (`409000`) instead of leaking a PostgreSQL `23505` unique-constraint error as `500`. Covers `create`, `createMany`, `update` and `updateAssociation`; the failure is logged when action logging is enabled.

package/README.md

@@ -84,7 +84,7 @@ yarn lint        # ESLint, no fixes
 yarn typecheck   # tsc --noEmit
 yarn test        # Jest with coverage thresholds
 yarn build       # compile to dist/
-yarn verify      # lint + typecheck + test + build
+yarn verify      # format + lint + typecheck + test + build
 ```
 
 Packaging:

package/dist/base-service/base-service.service.d.ts

@@ -23,5 +23,6 @@ export declare class BaseService {
     }, actionLogOptions?: ActionLogOptions): Promise<void>;
     createSuccessActionLog<T extends CommonEntity>(entities: T | T[], actionLogOptions: ActionLogOptions): Promise<void>;
     throwError({ httpExceptionKind, ...restOptions }: ThrowBaseServiceExceptionOptions, actionLogOptions?: ActionLogOptions): Promise<never>;
+    private throwUniqueViolation;
     private preValidateUpdate;
 }

package/dist/base-service/base-service.service.js

@@ -35,7 +35,21 @@ const ASSOCIATION_FIND_OPTION_KEYS = [
     'withTotal'
 ];
 const ASSOCIATION_BATCH_SIZE = 1024;
+const UNIQUE_VIOLATION_ERROR_CODE = '23505';
 const isNotFoundException = (error) => error instanceof common_1.HttpException && error.getStatus() === common_1.HttpStatus.NOT_FOUND;
+const isUniqueViolationError = (error) => error instanceof typeorm_1.QueryFailedError &&
+    error.driverError?.code === UNIQUE_VIOLATION_ERROR_CODE;
+const mapUniqueViolation = async (write, onConflict) => {
+    try {
+        return await write();
+    }
+    catch (error) {
+        if (!isUniqueViolationError(error)) {
+            throw error;
+        }
+        return onConflict();
+    }
+};
 const normalizePagination = (page, pageSize) => {
     const normalizedPage = Math.max(1, Math.trunc(Number(page) || 1));
     const normalizedPageSize = Math.min(contracts_1.MAX_PAGE_SIZE, Math.max(1, Math.trunc(Number(pageSize) || contracts_1.DEFAULT_PAGE_SIZE)));
@@ -98,7 +112,7 @@ let BaseService = class BaseService {
             }
             const createPayload = omitKeys(entity, SERVER_MANAGED_CREATE_KEYS);
             const createdEntity = repository.create(createPayload);
-            const savedEntity = await (manager ? manager.save(createdEntity) : createdEntity.save());
+            const savedEntity = await mapUniqueViolation(() => (manager ? manager.save(createdEntity) : createdEntity.save()), () => this.throwUniqueViolation(actionLogOptions));
             void this.createSuccessActionLog(savedEntity, actionLogOptions);
             return savedEntity;
         });
@@ -106,7 +120,7 @@ let BaseService = class BaseService {
     createMany(repository, entities, manager) {
         const createPayloads = entities.map(entity => omitKeys(entity, SERVER_MANAGED_CREATE_KEYS));
         const createdEntities = repository.create(createPayloads);
-        return manager ? manager.save(createdEntities) : repository.save(createdEntities);
+        return mapUniqueViolation(() => (manager ? manager.save(createdEntities) : repository.save(createdEntities)), () => this.throwUniqueViolation());
     }
     delete(repository, { headers, isLogEnabled = true, serviceName, where }, manager) {
         const actionLogOptions = {
@@ -165,7 +179,7 @@ let BaseService = class BaseService {
             }
             const updatePayload = omitKeys(entity, IMMUTABLE_ENTITY_KEYS);
             repository.merge(currentEntity, updatePayload);
-            await (manager ? manager.save(currentEntity) : currentEntity.save());
+            await mapUniqueViolation(() => (manager ? manager.save(currentEntity) : currentEntity.save()), () => this.throwUniqueViolation(actionLogOptions));
             void this.createSuccessActionLog(currentEntity, actionLogOptions);
             return currentEntity;
         });
@@ -373,7 +387,7 @@ let BaseService = class BaseService {
             }
             const manager = (entityManager ?? repository);
             if (entitiesToAdd.length) {
-                await manager.save(entitiesToAdd);
+                await mapUniqueViolation(() => manager.save(entitiesToAdd), () => this.throwUniqueViolation(actionLogOptions));
             }
             if (entitiesToRecover.length) {
                 await manager.recover(entitiesToRecover);
@@ -481,6 +495,17 @@ let BaseService = class BaseService {
         }
         throw error;
     }
+    async throwUniqueViolation(actionLogOptions) {
+        const serviceName = actionLogOptions?.serviceName;
+        const message = serviceName ?
+            `The ${serviceName} fields already exists.`
+            : 'A record with the same unique fields already exists.';
+        return this.throwError({
+            code: exception_1.HTTP_EXCEPTION_CODE.CONFLICT,
+            httpExceptionKind: exception_1.HTTP_EXCEPTION.CONFLICT,
+            message
+        }, actionLogOptions);
+    }
     async preValidateUpdate({ actionLogOptions, currentEntity, preValidationOptions, repository, serviceName, where }) {
         const { validationKeys = [], where: preValidationWhere, ...restOptions } = preValidationOptions;
         const validationKeysSet = new Set(validationKeys.map(String));

package/dist/exception/exception.enum.d.ts

@@ -2,6 +2,7 @@ export { HTTP_EXCEPTION_CODE } from '@bearei/contracts';
 export declare const HTTP_EXCEPTION: {
     readonly BAD_GATEWAY: "BAD_GATEWAY";
     readonly BAD_REQUEST: "BAD_REQUEST";
+    readonly CONFLICT: "CONFLICT";
     readonly FORBIDDEN: "FORBIDDEN";
     readonly GENERIC: "GENERIC";
     readonly INTERNAL_SERVER_ERROR: "INTERNAL_SERVER_ERROR";

package/dist/exception/exception.enum.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "HTTP_EXCEPTION_CODE", { enumerable: true, get: f
 exports.HTTP_EXCEPTION = {
     BAD_GATEWAY: 'BAD_GATEWAY',
     BAD_REQUEST: 'BAD_REQUEST',
+    CONFLICT: 'CONFLICT',
     FORBIDDEN: 'FORBIDDEN',
     GENERIC: 'GENERIC',
     INTERNAL_SERVER_ERROR: 'INTERNAL_SERVER_ERROR',

package/dist/exception/http.exception.js

@@ -15,6 +15,7 @@ const EXCEPTION_FACTORY_MAP = {
         message: buildMessage(message, service),
         statusCode: common_1.HttpStatus.BAD_REQUEST
     }),
+    [exception_enum_1.HTTP_EXCEPTION.CONFLICT]: ({ code = exception_enum_1.HTTP_EXCEPTION_CODE.CONFLICT, message = 'Conflict', service }) => new common_1.ConflictException({ code, message: buildMessage(message, service), statusCode: common_1.HttpStatus.CONFLICT }),
     [exception_enum_1.HTTP_EXCEPTION.FORBIDDEN]: ({ code = exception_enum_1.HTTP_EXCEPTION_CODE.FORBIDDEN, message = 'Forbidden', service }) => new common_1.ForbiddenException({ code, message: buildMessage(message, service), statusCode: common_1.HttpStatus.FORBIDDEN }),
     [exception_enum_1.HTTP_EXCEPTION.GENERIC]: ({ code = exception_enum_1.HTTP_EXCEPTION_CODE.INTERNAL_SERVER_ERROR, message = 'Internal Server Error', service, status = common_1.HttpStatus.INTERNAL_SERVER_ERROR }) => new common_1.HttpException({ code, message: buildMessage(message, service), statusCode: status }, status),
     [exception_enum_1.HTTP_EXCEPTION.INTERNAL_SERVER_ERROR]: ({ code = exception_enum_1.HTTP_EXCEPTION_CODE.INTERNAL_SERVER_ERROR, message = 'Internal Server Error', service }) => new common_1.InternalServerErrorException({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bearei/nest",
-  "version": "2.0.0-dev.2",
+  "version": "2.0.0-dev.4",
   "description": "Shared NestJS and TypeORM building blocks for Bearei services",
   "keywords": [
     "bearei",
@@ -38,7 +38,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@bearei/contracts": "^2.0.0",
+    "@bearei/contracts": "^2.1.0",
     "bcrypt": "^5.1.1",
     "dayjs": "^1.11.13",
     "ioredis": "^5.4.1",
@@ -59,9 +59,9 @@
     "typeorm": "^1.0.0"
   },
   "devDependencies": {
-    "@bearei/eslint-config": "^2.0.0",
-    "@bearei/jest-preset": "^2.0.0",
-    "@bearei/tsconfig": "^2.0.0",
+    "@bearei/eslint-config": "^2.1.0-dev.2",
+    "@bearei/jest-preset": "^2.1.0-dev.2",
+    "@bearei/tsconfig": "^2.1.0-dev.2",
     "@nestjs/axios": "^4.0.1",
     "@nestjs/cli": "^11.0.0",
     "@nestjs/common": "^11.1.27",
@@ -119,11 +119,10 @@
     "prepack": "yarn build",
     "test": "jest",
     "test:coverage": "jest --coverage",
-    "test:cov": "jest --coverage",
     "test:debug": "node --inspect-brk -r tsconfig-paths/register -r ts-node/register node_modules/.bin/jest --runInBand",
     "test:watch": "jest --watch",
     "typecheck": "tsc --noEmit -p tsconfig.json",
-    "verify": "yarn lint && yarn typecheck && yarn test:coverage && yarn build"
+    "verify": "yarn prettier --check . && yarn lint && yarn typecheck && yarn test:coverage && yarn build"
   },
   "packageManager": "yarn@4.18.0+sha512.fcb8716fe7cd0eece141ffc18b92193a9df9204c1ba83189c288835223fc0bbe64af473bab0d5e9927a7daeb5caf2bb07eb2787cc9338ca040ea125f2a1f2f7e"
 }