@communecter/cocolight-api-client 1.0.59 → 1.0.61

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@communecter/cocolight-api-client",
-    "version": "1.0.59",
+    "version": "1.0.61",
     "description": "Client Axios simplifié pour l'API cocolight",
     "repository": {
         "type": "git",

package/src/api/BaseEntity.ts

@@ -4,6 +4,7 @@ import objectId from "bson-objectid";
 import * as pkg from "file-type";
 
 import { ApiAuthenticationError, ApiError, ApiResponseError, ApiValidationError } from "../error.js";
+import { fromEntityJSON } from "./EntityRegistry.js";
 import { EJSON } from "../utils/compat.js";
 import { isReactive, isSignal, reactive } from "../utils/reactive.js";
 
@@ -734,6 +735,33 @@ export class BaseEntity<TServerData = any> {
   private _removeUnserializables(obj: any, seen = new WeakSet<object>()): any {
     if (obj === null || typeof obj !== "object") return obj;
 
+    // Check for native types BEFORE adding to WeakSet and checking for reactive proxies
+    // This prevents Date objects from being processed as regular objects
+    if (obj instanceof Date) {
+      return obj;
+    }
+
+    if (obj instanceof RegExp) {
+      return obj;
+    }
+
+    // Preserve ObjectID instances (both bson-objectid and custom ObjectID)
+    if (obj && typeof obj === "object" && "_bsontype" in obj && obj._bsontype === "ObjectID") {
+      return obj;
+    }
+
+    // Preserve binary types (Uint8Array, Buffer, etc.)
+    if (obj instanceof Uint8Array || ArrayBuffer.isView(obj)) {
+      return obj;
+    }
+
+    // If the object is a BaseEntity instance, serialize it via toJSON()
+    // This ensures nested entities (like author, target in News) are properly serialized
+    if (obj instanceof BaseEntity) {
+      return obj.toJSON();
+    }
+
+    // Now check for circular references
     if (seen.has(obj)) return null;
     seen.add(obj);
 
@@ -771,12 +799,57 @@ export class BaseEntity<TServerData = any> {
 
   /**
    * Restaure les données sérialisées en un objet d'origine.
+   * Désérialise récursivement les entités imbriquées qui ont le marqueur __isSerializedEntity.
    *
    * @param obj - L'objet à restaurer.
+   * @param parent - Instance parente optionnelle pour désérialiser les entités imbriquées.
    * @returns L'objet restauré.
    */
-  static _revive(obj: object): any {
-    return EJSON.fromJSONValue(obj);
+  static _revive(obj: object, parent?: ApiClient | BaseEntity<any> | null): any {
+    // D'abord, restaurer les dates EJSON
+    const revived = EJSON.fromJSONValue(obj);
+
+    // Ensuite, parcourir récursivement pour désérialiser les entités imbriquées
+    return this._deserializeNestedEntities(revived, parent);
+  }
+
+  /**
+   * Parcourt récursivement un objet et désérialise les entités imbriquées.
+   *
+   * @param obj - L'objet à parcourir.
+   * @param parent - Instance parente pour les entités.
+   * @returns L'objet avec les entités désérialisées.
+   * @private
+   */
+  private static _deserializeNestedEntities(obj: any, parent?: ApiClient | BaseEntity<any> | null): any {
+    if (obj === null || typeof obj !== "object") {
+      return obj;
+    }
+
+    // Si c'est un objet Date, RegExp, ou autre type natif, le retourner tel quel
+    if (obj instanceof Date || obj instanceof RegExp) {
+      return obj;
+    }
+
+    // Si c'est un array, parcourir chaque élément
+    if (Array.isArray(obj)) {
+      return obj.map(item => this._deserializeNestedEntities(item, parent));
+    }
+
+    // Si c'est une entité sérialisée, la désérialiser
+    // On vérifie les marqueurs d'une entité sérialisée
+    if (obj.__isSerializedEntity && obj.__entityTag && obj.serverData) {
+      // Désérialiser l'entité imbriquée via EntityRegistry
+      return fromEntityJSON(obj, parent);
+    }
+
+    // Sinon, parcourir les propriétés de l'objet et désérialiser récursivement
+    const result: any = {};
+    for (const [key, value] of Object.entries(obj)) {
+      result[key] = this._deserializeNestedEntities(value, parent);
+    }
+
+    return result;
   }
 
   /**
@@ -797,7 +870,7 @@ export class BaseEntity<TServerData = any> {
     }
     const { serverData } = json as { serverData: object };
 
-    const instance = this.fromServerData(this._revive(serverData), parent as ApiClient | ParentLike, deps || {});
+    const instance = this.fromServerData(this._revive(serverData, parent), parent as ApiClient | ParentLike, deps || {});
 
     return instance;
   }

package/src/api/EntityRegistry.ts

@@ -57,10 +57,18 @@ export function fromEntityJSON(json: unknown, parent: ApiClient | BaseEntity | n
   // On gère null / non-objet / pas une entité → on renvoie tel quel (ou null si tu préfères)
   if (!isEntityJSON(json)) return json;
 
-  // Ici, TS sait que json.__entityTag existe
-  if (!json.serverData?.collection) return json;
+  // Récupérer la collection depuis serverData, ou la dériver depuis __entityTag
+  let collection: CollectionKey | null | undefined = json.serverData?.collection;
 
-  const meta = _getEntityMeta(json.serverData.collection, json.__entityTag);
+  // Si collection est absente, essayer de la dériver depuis __entityTag
+  if (!collection) {
+    collection = _getCollectionFromTag(json.__entityTag);
+  }
+
+  // Si on n'a toujours pas de collection, on ne peut pas désérialiser
+  if (!collection) return json;
+
+  const meta = _getEntityMeta(collection, json.__entityTag);
   if (!meta) return json;
 
   if (!meta.entityClass?.fromJSON) {
@@ -69,6 +77,24 @@ export function fromEntityJSON(json: unknown, parent: ApiClient | BaseEntity | n
   return meta.entityClass.fromJSON(json, parent, meta.deps);
 }
 
+/**
+ * Mappe un EntityTag vers son CollectionKey correspondant.
+ */
+function _getCollectionFromTag(tag: EntityTag): CollectionKey | null {
+  const map: Record<EntityTag, CollectionKey> = {
+    "User": "citoyens",
+    "Organization": "organizations",
+    "Project": "projects",
+    "Event": "events",
+    "Poi": "poi",
+    "News": "news",
+    "Badge": "badges",
+    "Comment": "comments",
+    "Answer": "answers"
+  };
+  return map[tag] || null;
+}
+
 /**
  * Récupère la classe d'entité et ses dépendances à partir du type d'entité.
  */

package/src/types/index.ts

@@ -7,6 +7,7 @@ export type {
 } from "../api/EntityRegistry.js";
 
 export type {
+  EntityTypes,
   TransformsMap,
   TransformFunction,
 } from "./entities.js";

package/types/api/BaseEntity.d.ts

@@ -328,11 +328,22 @@ export declare class BaseEntity<TServerData = any> {
     private _removeUnserializables;
     /**
      * Restaure les données sérialisées en un objet d'origine.
+     * Désérialise récursivement les entités imbriquées qui ont le marqueur __isSerializedEntity.
      *
      * @param obj - L'objet à restaurer.
+     * @param parent - Instance parente optionnelle pour désérialiser les entités imbriquées.
      * @returns L'objet restauré.
      */
-    static _revive(obj: object): any;
+    static _revive(obj: object, parent?: ApiClient | BaseEntity<any> | null): any;
+    /**
+     * Parcourt récursivement un objet et désérialise les entités imbriquées.
+     *
+     * @param obj - L'objet à parcourir.
+     * @param parent - Instance parente pour les entités.
+     * @returns L'objet avec les entités désérialisées.
+     * @private
+     */
+    private static _deserializeNestedEntities;
     /**
      * Crée une instance d'entité à partir de données JSON.
      *

package/types/types/index.d.ts

@@ -1,5 +1,5 @@
 export type { CollectionKey, EntityTag } from "../api/EntityRegistry.js";
-export type { TransformsMap, TransformFunction, } from "./entities.js";
+export type { EntityTypes, TransformsMap, TransformFunction, } from "./entities.js";
 export type { SocialNetworkPayload, } from "./payloads.js";
 export type { UserTransforms, OrganizationTransforms, ProjectTransforms, EventTransforms } from "./transforms.js";
 export type { BaseApiResponse, ApiDataResponse, ApiErrorResponse, PaginatedApiResponse, GetElementsKeyResponse, ApiResponse } from "./api-responses.js";