@communecter/cocolight-api-client 1.0.60 → 1.0.61

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@communecter/cocolight-api-client",
-    "version": "1.0.60",
+    "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,19 +735,8 @@ export class BaseEntity<TServerData = any> {
   private _removeUnserializables(obj: any, seen = new WeakSet<object>()): any {
     if (obj === null || typeof obj !== "object") return obj;
 
-    if (seen.has(obj)) return null;
-    seen.add(obj);
-
-    // Ignore les proxys réactifs
-    if (obj.__isReactive && typeof obj.__raw === "object") {
-      return this._removeUnserializables(obj.__raw, seen);
-    }
-
-    if (Array.isArray(obj)) {
-      return obj.map((el) => this._removeUnserializables(el, seen));
-    }
-
-    // Preserve native types that EJSON handles natively
+    // 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;
     }
@@ -765,6 +755,25 @@ export class BaseEntity<TServerData = any> {
       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);
+
+    // Ignore les proxys réactifs
+    if (obj.__isReactive && typeof obj.__raw === "object") {
+      return this._removeUnserializables(obj.__raw, seen);
+    }
+
+    if (Array.isArray(obj)) {
+      return obj.map((el) => this._removeUnserializables(el, seen));
+    }
+
     const clean: Record<string, any> = {};
     for (const key of Object.keys(obj)) {
       const val = obj[key];
@@ -790,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;
   }
 
   /**
@@ -816,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/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.
      *