masterrecord 0.3.45 → 0.3.47

package/Migrations/schema.js

@@ -16,6 +16,7 @@ class schema{
      * create the database first, then retry the connection.
      */
     async _ensureReady(){
+        if(this._ready){ return; }
         if(this.context && this.context._initPromise){
             try{
                 await this.context._initPromise;
@@ -35,6 +36,7 @@ class schema{
                 }
             }
         }
+        this._ready = true;
     }
 
     /**
@@ -295,8 +297,12 @@ class schema{
         const existingNames = new Set((existing || []).map(c => (c.name || c.COLUMN_NAME))); // both engines map to name
         // Add missing columns only (safe path)
         for (var key in table) {
-            if(typeof table[key] === 'object'){
+            // Skip metadata properties (indexes, __compositeIndexes, __name, etc.)
+            if(key === 'indexes' || key.startsWith('__')) continue;
+            if(typeof table[key] === 'object' && !Array.isArray(table[key])){
                 const col = table[key];
+                // Skip if missing name/type (not a valid column definition)
+                if(!col.name || !col.type) continue;
                 // Skip relationships
                 if(col.type === 'hasOne' || col.type === 'hasMany' || col.type === 'hasManyThrough') continue;
                 const colName = (col.relationshipType === 'belongsTo' && col.foreignKey) ? col.foreignKey : col.name;
@@ -335,8 +341,10 @@ class schema{
         // Detect modifications (nullable/default/type)
         const desiredCols = [];
         for (var key in table) {
-            if(typeof table[key] === 'object'){
+            if(key === 'indexes' || key.startsWith('__')) continue;
+            if(typeof table[key] === 'object' && !Array.isArray(table[key])){
                 const col = table[key];
+                if(!col.name || !col.type) continue;
                 if(col.type === 'hasOne' || col.type === 'hasMany' || col.type === 'hasManyThrough') continue;
                 const colName = (col.relationshipType === 'belongsTo' && col.foreignKey) ? col.foreignKey : col.name;
                 desiredCols.push({ name: colName, col });

package/mySQLEngine.js

@@ -798,6 +798,14 @@ class MySQLEngine {
         }
     }
 
+    /**
+     * Execute raw SQL (DDL statements like CREATE TABLE, ALTER TABLE, etc.)
+     * Used by migration schema for non-parameterized DDL queries.
+     */
+    _execute(query) {
+        return this._runWithParams(query, []);
+    }
+
     /**
      * Execute parameterized query with mysql2/promise
      */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "masterrecord",
-  "version": "0.3.45",
+  "version": "0.3.47",
   "description": "An Object-relational mapping for the Master framework. Master Record connects classes to relational database tables to establish a database with almost zero-configuration ",
   "main": "MasterRecord.js",
   "bin": {

package/postgresEngine.js

@@ -750,6 +750,14 @@ class postgresEngine {
     /**
      * Execute parameterized query with pg library
      */
+    /**
+     * Execute raw SQL (DDL statements like CREATE TABLE, ALTER TABLE, etc.)
+     * Used by migration schema for non-parameterized DDL queries.
+     */
+    _execute(query) {
+        return this._runWithParams(query, []);
+    }
+
     async _runWithParams(query, params = []) {
         try {
             if (process.env.LOG_SQL === 'true' || process.env.NODE_ENV !== 'production') {