masterrecord 0.3.66 → 0.3.67

package/.claude/settings.local.json

@@ -64,7 +64,9 @@
       "Bash(masterrecord add-migration:*)",
       "Bash(masterrecord:*)",
       "Bash(git -C /Users/alexanderrich/Documents/development/bookbaghq/bookbag-training log --oneline --all -- *MASTERRECORD_ISSUE*)",
-      "Bash(npm ls -g masterrecord 2>&1 | head -5)"
+      "Bash(npm ls -g masterrecord 2>&1 | head -5)",
+      "Bash(gh pr:*)",
+      "Read(//tmp/**)"
     ],
     "deny": [],
     "ask": []

package/Migrations/schema.js

@@ -1,4 +1,6 @@
 // version 0.0.6
+const { _poolKey } = require('masterrecord/context');
+
 class schema{
 
     constructor(context){
@@ -6,12 +8,6 @@ class schema{
         this._dbEnsured = false;
     }
 
-    _poolKey(type, cfg) {
-        const host = cfg.host || 'localhost';
-        const port = cfg.port || (type === 'mysql' ? 3306 : 5432);
-        return `${type}:${cfg.user}@${host}:${port}/${cfg.database}`;
-    }
-
     /**
      * Wait for async database initialization (MySQL/PostgreSQL) to complete.
      * The context constructor fires off an async pool init that may not have
@@ -93,9 +89,9 @@ class schema{
 
         // Check global pool cache first -- another context may have already retried
         const _pools = global.__MR_POOLS__;
-        const key = _pools ? this._poolKey('mysql', config) : null;
+        const key = _poolKey('mysql', config);
 
-        if (_pools && key && _pools.has(key)) {
+        if (_pools.has(key)) {
             const cached = _pools.get(key);
             cached.refCount++;
             if (cached.promise) {
@@ -120,9 +116,7 @@ class schema{
         this.context.db = client;
 
         // Register in global pool cache so other contexts can reuse
-        if (_pools && key) {
-            _pools.set(key, { client, engine: this.context._SQLEngine, refCount: 1, dbType: 'mysql' });
-        }
+        _pools.set(key, { client, engine: this.context._SQLEngine, refCount: 1, dbType: 'mysql' });
         console.log('[MySQL] Connection pool ready');
     }
 
@@ -174,9 +168,9 @@ class schema{
 
         // Check global pool cache first -- another context may have already retried
         const _pools = global.__MR_POOLS__;
-        const key = _pools ? this._poolKey('postgres', config) : null;
+        const key = _poolKey('postgres', config);
 
-        if (_pools && key && _pools.has(key)) {
+        if (_pools.has(key)) {
             const cached = _pools.get(key);
             cached.refCount++;
             if (cached.promise) {
@@ -202,9 +196,7 @@ class schema{
         this.context.db = pool;
 
         // Register in global pool cache so other contexts can reuse
-        if (_pools && key) {
-            _pools.set(key, { pool, engine: this.context._SQLEngine, client: connection, refCount: 1, dbType: 'postgres' });
-        }
+        _pools.set(key, { pool, engine: this.context._SQLEngine, client: connection, refCount: 1, dbType: 'postgres' });
         console.log('[PostgreSQL] Connection pool ready');
     }
 

package/context.js

@@ -2402,4 +2402,7 @@ module.exports = context;
 module.exports.ContextError = ContextError;
 module.exports.ConfigurationError = ConfigurationError;
 module.exports.DatabaseConnectionError = DatabaseConnectionError;
-module.exports.EntityValidationError = EntityValidationError;
+module.exports.EntityValidationError = EntityValidationError;
+
+// Export pool key generator for use by schema.js (single source of truth)
+module.exports._poolKey = _poolKey;

package/package.json

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