v7.0.12

changelog.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 2025-11-28 - 7.0.12 - fix(collection)
+Ensure TC39 decorator metadata is initialized on both original and decorated constructors/prototypes and add debug logging
+
+- Initialize metadata-driven prototype properties (globalSaveableProperties, saveableProperties, uniqueIndexes, regularIndexes) on both the decorated class prototype and the original constructor prototype to avoid closure/compatibility issues
+- Initialize searchableFields on both the decorated constructor and the original constructor so text-index creation and searches see the fields correctly
+- Forward and initialize _svDbOptions from decorator metadata onto the original constructor to preserve custom serialization options
+- Add debug logging in the Collection decorator and in createSavableObject to surface metadata and saveable-property counts for easier troubleshooting
+
 ## 2025-11-28 - 7.0.9 - fix(classes.collection)
 Fix closure bug in Collection decorator by defining collection getter on original constructor and prototype
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@push.rocks/smartdata",
-  "version": "7.0.10",
+  "version": "7.0.12",
   "private": false,
   "description": "An advanced library for NoSQL data organization and manipulation using TypeScript with support for MongoDB, data validation, collections, and custom data types.",
   "exports": {

ts/00_commitinfo_data.ts

@@ -3,6 +3,6 @@
  */
 export const commitinfo = {
   name: '@push.rocks/smartdata',
-  version: '7.0.9',
+  version: '7.0.12',
   description: 'An advanced library for NoSQL data organization and manipulation using TypeScript with support for MongoDB, data validation, collections, and custom data types.'
 }

ts/classes.collection.ts

@@ -93,38 +93,62 @@ export function Collection(dbArg: SmartdataDb | TDelayed<SmartdataDb>) {
     // Initialize prototype properties from context.metadata (TC39 decorator metadata)
     // This ensures prototype properties are available before any instance is created
     const metadata = context.metadata as any;
-    logger.log('debug', `Collection decorator: metadata keys = ${metadata ? Object.keys(metadata).join(', ') : 'null'}`);
-    logger.log('debug', `Collection decorator: saveableProperties in metadata = ${metadata?.saveableProperties?.length ?? 0}`);
-    logger.log('debug', `Collection decorator: globalSaveableProperties in metadata = ${metadata?.globalSaveableProperties?.length ?? 0}`);
+    logger.log('debug', `Collection decorator for ${constructor.name}: metadata.saveableProperties = ${metadata?.saveableProperties?.length ?? 'undefined'}`);
     if (metadata) {
       const proto = decoratedClass.prototype;
+      const origProto = constructor.prototype;
 
-      // Initialize globalSaveableProperties
-      if (metadata.globalSaveableProperties && !proto.globalSaveableProperties) {
+      // Initialize globalSaveableProperties on BOTH prototypes
+      if (metadata.globalSaveableProperties) {
+        if (!proto.globalSaveableProperties) {
           proto.globalSaveableProperties = [...metadata.globalSaveableProperties];
-        logger.log('debug', `Collection decorator: initialized globalSaveableProperties with ${proto.globalSaveableProperties.length} properties`);
+        }
+        if (!origProto.globalSaveableProperties) {
+          origProto.globalSaveableProperties = [...metadata.globalSaveableProperties];
+        }
       }
 
-      // Initialize saveableProperties
-      if (metadata.saveableProperties && !proto.saveableProperties) {
+      // Initialize saveableProperties on BOTH prototypes (closure fix)
+      if (metadata.saveableProperties) {
+        if (!proto.saveableProperties) {
           proto.saveableProperties = [...metadata.saveableProperties];
-        logger.log('debug', `Collection decorator: initialized saveableProperties with ${proto.saveableProperties.length} properties`);
+        }
+        // Also set on original constructor's prototype for closure references
+        if (!origProto.saveableProperties) {
+          origProto.saveableProperties = [...metadata.saveableProperties];
+          logger.log('debug', `Collection decorator: set saveableProperties on original prototype (${origProto.saveableProperties.length} props)`);
+        }
       }
 
-      // Initialize uniqueIndexes
-      if (metadata.uniqueIndexes && !proto.uniqueIndexes) {
+      // Initialize uniqueIndexes on BOTH prototypes
+      if (metadata.uniqueIndexes) {
+        if (!proto.uniqueIndexes) {
           proto.uniqueIndexes = [...metadata.uniqueIndexes];
         }
-
-      // Initialize regularIndexes
-      if (metadata.regularIndexes && !proto.regularIndexes) {
-        proto.regularIndexes = [...metadata.regularIndexes];
+        if (!origProto.uniqueIndexes) {
+          origProto.uniqueIndexes = [...metadata.uniqueIndexes];
+        }
       }
 
-      // Initialize searchableFields on constructor (not prototype)
-      if (metadata.searchableFields && !Array.isArray((decoratedClass as any).searchableFields)) {
+      // Initialize regularIndexes on BOTH prototypes
+      if (metadata.regularIndexes) {
+        if (!proto.regularIndexes) {
+          proto.regularIndexes = [...metadata.regularIndexes];
+        }
+        if (!origProto.regularIndexes) {
+          origProto.regularIndexes = [...metadata.regularIndexes];
+        }
+      }
+
+      // Initialize searchableFields on BOTH constructors
+      if (metadata.searchableFields) {
+        if (!Array.isArray((decoratedClass as any).searchableFields)) {
           (decoratedClass as any).searchableFields = [...metadata.searchableFields];
         }
+        if (!Array.isArray((constructor as any).searchableFields)) {
+          (constructor as any).searchableFields = [...metadata.searchableFields];
+        }
+      }
 
       // Initialize _svDbOptions from metadata
       if (metadata._svDbOptions && !originalConstructor._svDbOptions) {
@@ -249,31 +273,57 @@ export function managed<TManager extends IManager>(managerArg?: TManager | TDela
     const metadata = context.metadata as any;
     if (metadata) {
       const proto = decoratedClass.prototype;
+      const origProto = constructor.prototype;
 
-      // Initialize globalSaveableProperties
-      if (metadata.globalSaveableProperties && !proto.globalSaveableProperties) {
+      // Initialize globalSaveableProperties on BOTH prototypes
+      if (metadata.globalSaveableProperties) {
+        if (!proto.globalSaveableProperties) {
           proto.globalSaveableProperties = [...metadata.globalSaveableProperties];
         }
+        if (!origProto.globalSaveableProperties) {
+          origProto.globalSaveableProperties = [...metadata.globalSaveableProperties];
+        }
+      }
 
-      // Initialize saveableProperties
-      if (metadata.saveableProperties && !proto.saveableProperties) {
+      // Initialize saveableProperties on BOTH prototypes (closure fix)
+      if (metadata.saveableProperties) {
+        if (!proto.saveableProperties) {
           proto.saveableProperties = [...metadata.saveableProperties];
         }
+        if (!origProto.saveableProperties) {
+          origProto.saveableProperties = [...metadata.saveableProperties];
+        }
+      }
 
-      // Initialize uniqueIndexes
-      if (metadata.uniqueIndexes && !proto.uniqueIndexes) {
+      // Initialize uniqueIndexes on BOTH prototypes
+      if (metadata.uniqueIndexes) {
+        if (!proto.uniqueIndexes) {
           proto.uniqueIndexes = [...metadata.uniqueIndexes];
         }
-
-      // Initialize regularIndexes
-      if (metadata.regularIndexes && !proto.regularIndexes) {
-        proto.regularIndexes = [...metadata.regularIndexes];
+        if (!origProto.uniqueIndexes) {
+          origProto.uniqueIndexes = [...metadata.uniqueIndexes];
+        }
       }
 
-      // Initialize searchableFields on constructor (not prototype)
-      if (metadata.searchableFields && !Array.isArray((decoratedClass as any).searchableFields)) {
+      // Initialize regularIndexes on BOTH prototypes
+      if (metadata.regularIndexes) {
+        if (!proto.regularIndexes) {
+          proto.regularIndexes = [...metadata.regularIndexes];
+        }
+        if (!origProto.regularIndexes) {
+          origProto.regularIndexes = [...metadata.regularIndexes];
+        }
+      }
+
+      // Initialize searchableFields on BOTH constructors
+      if (metadata.searchableFields) {
+        if (!Array.isArray((decoratedClass as any).searchableFields)) {
           (decoratedClass as any).searchableFields = [...metadata.searchableFields];
         }
+        if (!Array.isArray((constructor as any).searchableFields)) {
+          (constructor as any).searchableFields = [...metadata.searchableFields];
+        }
+      }
 
       // Initialize _svDbOptions from metadata
       if (metadata._svDbOptions && !originalConstructor._svDbOptions) {

ts/classes.doc.ts

@@ -951,6 +951,7 @@ export class SmartDataDbDoc<T extends TImplements, TImplements, TManager extends
     const globalProps = this.globalSaveableProperties || [];
     const specificProps = this.saveableProperties || [];
     const saveableProperties = [...globalProps, ...specificProps];
+    logger.log('debug', `createSavableObject: globalProps=${globalProps.length}, specificProps=${specificProps.length}, total=${saveableProperties.length}`);
     // apply custom serialization if configured
     const optionsMap = (this.constructor as any)._svDbOptions || {};
     for (const propertyNameString of saveableProperties) {