masterrecord 0.3.40 → 0.3.41

package/Migrations/cli.js

@@ -259,8 +259,11 @@ program.option('-V', 'output the version');
         }
         let contextInstance;
         try{
+          process.env.MASTERRECORD_SCHEMA_ONLY = '1';
           contextInstance = new ContextCtor();
+          delete process.env.MASTERRECORD_SCHEMA_ONLY;
         }catch(err){
+          delete process.env.MASTERRECORD_SCHEMA_ONLY;
           console.error(`\n❌ Error - Failed to construct Context from '${contextAbs}'`);
           console.error(`\nThis usually happens when:`);
           console.error(`  • Environment configuration is missing or invalid`);
@@ -291,27 +294,16 @@ program.option('-V', 'output the version');
         }
         var migrationDate = Date.now();
         var outputFile = `${migBase}/${migrationDate}_${name}_migration.js`
-        fs.writeFile(outputFile, newEntity, 'utf8', async function (err) {
+        fs.writeFile(outputFile, newEntity, 'utf8', function (err) {
           if (err) {
             console.log("--- Error running cammand, re-run command add-migration ---- ", err);
-            if (contextInstance && typeof contextInstance.close === 'function') {
-              await contextInstance.close();
-            }
             process.exit(1);
           }
           console.log(`✓ Migration '${name}' created successfully at ${outputFile}`);
-
-          // Close database connections to prevent hanging
-          if (contextInstance && typeof contextInstance.close === 'function') {
-            await contextInstance.close();
-          }
           process.exit(0);
         });
        }catch (e){
          console.log("Error - Cannot read or find file ", e);
-         if (contextInstance && typeof contextInstance.close === 'function') {
-           await contextInstance.close();
-         }
          process.exit(1);
       }
   });

package/context.js

@@ -675,6 +675,15 @@ class context {
 
             const type = String(options.type || '').toLowerCase();
 
+            // Schema-only mode: CLI commands like add-migration only need entity metadata,
+            // not a live database connection. Skip DB initialization entirely.
+            if (process.env.MASTERRECORD_SCHEMA_ONLY === '1') {
+                this.isSQLite = (type === DB_TYPES.SQLITE || type === DB_TYPES.BETTER_SQLITE3);
+                this.isMySQL = (type === DB_TYPES.MYSQL);
+                this.isPostgres = (type === DB_TYPES.POSTGRES || type === DB_TYPES.POSTGRESQL);
+                return this;
+            }
+
             // SQLite initialization
             if (type === DB_TYPES.SQLITE || type === DB_TYPES.BETTER_SQLITE3) {
                 this.isSQLite = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "masterrecord",
-  "version": "0.3.40",
+  "version": "0.3.41",
   "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/readme.md

@@ -3369,6 +3369,20 @@ user.name = null;  // Error if name is { nullable: false }
 
 ## Changelog
 
+### Version 0.3.41 (2026-02-20) - FIX: add-migration No Longer Creates Unnecessary Database Connections
+
+#### Bug Fixed: `add-migration` creating real database connections and files
+- **FIXED**: Running `masterrecord add-migration` would open a real database connection (creating SQLite files on disk, or connecting to MySQL/PostgreSQL) even though it only needs entity schemas and seed data to generate migration files
+  - **Problem**: On production servers, `add-migration` created unwanted `.sqlite3` files and printed misleading "SQLite database closed" messages
+  - **Root Cause**: The context constructor calls `env()` which initializes a full database connection; `add-migration` only needs entity metadata
+  - **Solution**: Added schema-only mode (`MASTERRECORD_SCHEMA_ONLY` env var) that sets database type flags for correct entity registration but skips all database initialization (no file creation, no connections)
+  - **Impact**: `add-migration` is now faster, creates no side effects, and works safely on production servers
+
+#### Files Modified
+1. **`Migrations/cli.js`** - Set `MASTERRECORD_SCHEMA_ONLY=1` before context construction, removed unnecessary `close()` calls
+2. **`context.js`** - Added schema-only mode check in `env()` that returns early after setting type flags
+3. **`package.json`** - Updated to v0.3.41
+
 ### Version 0.3.39 (2026-02-09) - CRITICAL BUG FIX: Foreign Key String Values
 
 #### Bug Fixed: Foreign Key Fields Silently Ignoring String Values