s3db.js 7.4.1 → 7.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "s3db.js",
-  "version": "7.4.1",
+  "version": "7.4.2",
   "description": "Use AWS S3, the world's most reliable document storage, as a database with this ORM.",
   "main": "dist/s3db.cjs.js",
   "module": "dist/s3db.es.js",

package/src/database.class.js

@@ -113,6 +113,23 @@ export class Database extends EventEmitter {
       
       if (versionData) {
         // Extract configuration from version data at root level
+        // Restore ID generator configuration
+        let restoredIdGenerator, restoredIdSize;
+        if (versionData.idGenerator !== undefined) {
+          if (versionData.idGenerator === 'custom_function') {
+            // Custom function was used but can't be restored - use default
+            restoredIdGenerator = undefined;
+            restoredIdSize = versionData.idSize || 22;
+          } else if (typeof versionData.idGenerator === 'number') {
+            // Size-based generator
+            restoredIdGenerator = versionData.idGenerator;
+            restoredIdSize = versionData.idSize || versionData.idGenerator;
+          }
+        } else {
+          // Legacy resource without saved ID config
+          restoredIdSize = versionData.idSize || 22;
+        }
+
         this.resources[name] = new Resource({
           name,
           client: this.client,
@@ -131,7 +148,9 @@ export class Database extends EventEmitter {
           autoDecrypt: versionData.autoDecrypt !== undefined ? versionData.autoDecrypt : true,
           hooks: versionData.hooks || {},
           versioningEnabled: this.versioningEnabled,
-          map: versionData.map
+          map: versionData.map,
+          idGenerator: restoredIdGenerator,
+          idSize: restoredIdSize
         });
       }
     }
@@ -334,6 +353,8 @@ export class Database extends EventEmitter {
             autoDecrypt: resource.config.autoDecrypt,
             cache: resource.config.cache,
             hooks: resource.config.hooks,
+            idSize: resource.idSize,
+            idGenerator: resource.idGeneratorType,
             createdAt: isNewVersion ? new Date().toISOString() : existingVersionData?.createdAt
           }
         }

package/src/resource.class.js

@@ -142,6 +142,19 @@ export class Resource extends EventEmitter {
 
     // Configure ID generator
     this.idGenerator = this.configureIdGenerator(customIdGenerator, idSize);
+    
+    // Store ID configuration for persistence
+    // If customIdGenerator is a number, use it as idSize
+    // Otherwise, use the provided idSize or default to 22
+    if (typeof customIdGenerator === 'number' && customIdGenerator > 0) {
+      this.idSize = customIdGenerator;
+    } else if (typeof idSize === 'number' && idSize > 0) {
+      this.idSize = idSize;
+    } else {
+      this.idSize = 22;
+    }
+    
+    this.idGeneratorType = this.getIdGeneratorType(customIdGenerator, this.idSize);
 
     // Store configuration - all at root level
     this.config = {
@@ -220,9 +233,9 @@ export class Resource extends EventEmitter {
    * @private
    */
   configureIdGenerator(customIdGenerator, idSize) {
-    // If a custom function is provided, use it
+    // If a custom function is provided, wrap it to ensure string output
     if (typeof customIdGenerator === 'function') {
-      return customIdGenerator;
+      return () => String(customIdGenerator());
     }
     // If customIdGenerator is a number (size), create a generator with that size
     if (typeof customIdGenerator === 'number' && customIdGenerator > 0) {
@@ -236,6 +249,22 @@ export class Resource extends EventEmitter {
     return defaultIdGenerator;
   }
 
+  /**
+   * Get a serializable representation of the ID generator type
+   * @param {Function|number} customIdGenerator - Custom ID generator function or size
+   * @param {number} idSize - Size for auto-generated IDs
+   * @returns {string|number} Serializable ID generator type
+   * @private
+   */
+  getIdGeneratorType(customIdGenerator, idSize) {
+    // If a custom function is provided
+    if (typeof customIdGenerator === 'function') {
+      return 'custom_function';
+    }
+    // For number generators or default size, return the actual idSize
+    return idSize;
+  }
+
   /**
    * Get resource options (for backward compatibility with tests)
    */