@lenne.tech/nest-server 11.24.0 → 11.24.1

package/migration-guides/11.24.0-to-11.24.1.md

@@ -0,0 +1,67 @@
+# Migration Guide: 11.24.0 → 11.24.1
+
+## Overview
+
+| Category | Details |
+|----------|---------|
+| **Breaking Changes** | None |
+| **New Features** | `pushToArray()`/`pullFromArray()` now accept `ObjectId` and objects with `id`/`_id` property |
+| **Migration Effort** | 0 minutes (no code changes required) |
+
+---
+
+## Quick Migration
+
+No code changes required.
+
+```bash
+# Update package
+pnpm add @lenne.tech/nest-server@11.24.1
+
+# Verify build
+pnpm run build
+
+# Run tests
+pnpm test
+```
+
+---
+
+## What's New in 11.24.1
+
+### Flexible `id` Parameter on `pushToArray()` / `pullFromArray()`
+
+Both methods now accept `string`, `Types.ObjectId`, or any object with an `id`/`_id` property (e.g. a Mongoose Document). IDs are normalized internally via `getStringIds()`, consistent with the rest of the framework.
+
+**Before (11.24.0):** Only `string` was accepted.
+```typescript
+await this.entityService.pushToArray(entity.id, 'logs', newLog);
+await this.entityService.pullFromArray(entity.id.toString(), 'tags', 'old');
+```
+
+**After (11.24.1):** All ID types work directly.
+```typescript
+// String (still works)
+await this.entityService.pushToArray('507f1f77bcf86cd799439011', 'logs', newLog);
+
+// ObjectId
+await this.entityService.pushToArray(new Types.ObjectId(id), 'logs', newLog);
+
+// Object with id property (e.g. Mongoose Document or entity reference)
+await this.entityService.pushToArray(entity, 'logs', newLog);
+
+// Same for pullFromArray
+await this.entityService.pullFromArray(entity, 'tags', 'obsolete');
+```
+
+---
+
+## Compatibility Notes
+
+Existing code using `string` IDs continues to work unchanged. The wider type signature is additive — no existing call sites break.
+
+---
+
+## References
+
+- [Migration Guide 11.23.x → 11.24.0](./11.23.x-to-11.24.0.md) — Full list of changes in 11.24.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lenne.tech/nest-server",
-  "version": "11.24.0",
+  "version": "11.24.1",
   "description": "Modern, fast, powerful Node.js web framework in TypeScript based on Nest with a GraphQL API and a connection to MongoDB (or other databases).",
   "keywords": [
     "node",

package/src/core/common/services/crud.service.ts

@@ -9,6 +9,7 @@ import {
   Query,
   QueryFilter,
   QueryOptions,
+  Types,
 } from 'mongoose';
 
 import { FilterArgs } from '../args/filter.args';
@@ -707,13 +708,13 @@ export abstract class CrudService<
    * await this.pushToArray(id, 'logs', newLog);
    * await this.pushToArray(id, 'logs', newLog, { $slice: -500 }); // Keep last 500
    *
-   * @param id - Document ID
+   * @param id - Document ID (string, ObjectId, or object with id/_id property)
    * @param field - Array field name. MUST be a compile-time constant — never pass user-controlled input.
    * @param items - Item(s) to append
    * @param options - MongoDB $push modifiers ($slice, $position, $sort)
    */
   async pushToArray(
-    id: string,
+    id: string | Types.ObjectId | { id?: any; _id?: any },
     field: string,
     items: any | any[],
     options?: { $slice?: number; $position?: number; $sort?: Record<string, 1 | -1> },
@@ -727,6 +728,7 @@ export abstract class CrudService<
       return;
     }
 
+    const stringId = getStringIds(id);
     const pushOp: any = { $each: itemsArray };
     if (options?.$slice !== undefined) {
       pushOp.$slice = options.$slice;
@@ -739,7 +741,7 @@ export abstract class CrudService<
     }
 
     await this.mainDbModel
-      .findByIdAndUpdate(id, { $push: { [field]: pushOp } } as any)
+      .findByIdAndUpdate(stringId, { $push: { [field]: pushOp } } as any)
       .lean()
       .exec();
   }
@@ -756,17 +758,22 @@ export abstract class CrudService<
    * // Remove by condition
    * await this.pullFromArray(id, 'logs', { level: 'DEBUG' });
    *
-   * @param id - Document ID
+   * @param id - Document ID (string, ObjectId, or object with id/_id property)
    * @param field - Array field name. MUST be a compile-time constant — never pass user-controlled input.
    * @param condition - Match condition for removal. MUST be application-controlled.
    */
-  async pullFromArray(id: string, field: string, condition: any): Promise<void> {
+  async pullFromArray(
+    id: string | Types.ObjectId | { id?: any; _id?: any },
+    field: string,
+    condition: any,
+  ): Promise<void> {
     if (typeof field !== 'string' || field.startsWith('$') || field.includes('\0')) {
       throw new Error(`pullFromArray: invalid field name "${field}"`);
     }
 
+    const stringId = getStringIds(id);
     await this.mainDbModel
-      .findByIdAndUpdate(id, { $pull: { [field]: condition } } as any)
+      .findByIdAndUpdate(stringId, { $pull: { [field]: condition } } as any)
       .lean()
       .exec();
   }