masterrecord 0.3.1 → 0.3.2

package/Migrations/migrationMySQLQuery.js

@@ -152,6 +152,14 @@ class migrationMySQLQuery {
 
 
     addColum(table){
+        // Fixed: Use columnMapping to generate full column definition with constraints
+        // This includes NOT NULL, DEFAULT, UNIQUE, PRIMARY KEY, AUTO_INCREMENT
+        if(table.type && table.tableName && table.name){
+            const def = this.#columnMapping(table);
+            return `ALTER TABLE \`${table.tableName}\`
+        ADD ${def}`;
+        }
+        // Fallback for legacy behavior with just realDataType
         return `ALTER TABLE \`${table.tableName}\`
         ADD \`${table.name}\` ${table.realDataType}`;
 

package/Migrations/migrationPostgresQuery.js

@@ -167,7 +167,14 @@ class migrationPostgresQuery {
     }
 
     addColum(table){
-        // PostgreSQL add column syntax
+        // Fixed: Use columnMapping to generate full column definition with constraints
+        // This includes NOT NULL, DEFAULT, UNIQUE, PRIMARY KEY
+        if(table.type && table.tableName && table.name){
+            const def = this.#columnMapping(table);
+            return `ALTER TABLE "${table.tableName}"
+        ADD COLUMN ${def}`;
+        }
+        // Fallback for legacy behavior with just realDataType
         return `ALTER TABLE "${table.tableName}"
         ADD COLUMN "${table.name}" ${table.realDataType}`;
     }

package/Migrations/migrationSQLiteQuery.js

@@ -108,6 +108,13 @@ class migrationSQLiteQuery {
             return `ALTER TABLE ${table.tableName}
         ADD COLUMN ${def}`;
         }
+        // Fixed: Support direct column definitions (when table itself IS the column spec)
+        // This matches MySQL/PostgreSQL behavior for explicit column definitions
+        if(table.type && table.tableName && table.name){
+            const def = this.#columnMapping(table);
+            return `ALTER TABLE ${table.tableName}
+        ADD COLUMN ${def}`;
+        }
         // Fallback legacy behavior: raw name provided must include full definition if caller wants type/constraints
         return `ALTER TABLE ${table.tableName}
         ADD COLUMN ${table.name}`;

package/Migrations/schema.js

@@ -24,11 +24,11 @@ class schema{
             if(this.context.isSQLite){
                 var sqliteQuery = require("./migrationSQLiteQuery");
                 var queryBuilder = new sqliteQuery();
-                var queryObj = queryBuilder.alterColumn(this.fullTable.new, table);
-                for (var key in queryObj) {
-                    var query = queryObj[key];
-                    this.context._execute(query);
-                }
+                // Fixed: Use addColum (consistent with MySQL/PostgreSQL) instead of alterColumn
+                // This allows explicit column definitions to work, not just CLI-generated migrations
+                table.realDataType = queryBuilder.typeManager(table.type);
+                var query = queryBuilder.addColum(table);
+                this.context._execute(query);
             }
 
             if(this.context.isMySQL){

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "masterrecord",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "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": {