@mikro-orm/core 7.1.4-dev.0 → 7.1.4-dev.1

package/metadata/MetadataDiscovery.js

@@ -1391,7 +1391,9 @@ export class MetadataDiscovery {
                     meta.root.uniques.push({ properties });
                 }
             }
-            newProp.nullable = true;
+            // The primary key column is shared by every subtype, so it stays NOT NULL —
+            // only subtype-specific columns become nullable for the rows of other subtypes.
+            newProp.nullable = !newProp.primary;
             newProp.inherited = !rootProp;
             // For narrowed relation overrides, keep the root's declaration intact so
             // the full target union (e.g. `Food`) is preserved for populates from the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/core",
-  "version": "7.1.4-dev.0",
+  "version": "7.1.4-dev.1",
   "description": "TypeScript ORM for Node.js based on Data Mapper, Unit of Work and Identity Map patterns. Supports MongoDB, MySQL, PostgreSQL and SQLite databases as well as usage with vanilla JavaScript.",
   "keywords": [
     "data-mapper",

package/utils/EntityComparator.js

@@ -393,7 +393,25 @@ export class EntityComparator {
         else {
             mapEntityProperties(meta);
         }
-        lines.push(`  for (let k in result) { if (Object.hasOwn(result, k) && !mapped[k] && ret[k] === undefined) ret[k] = result[k]; }`);
+        // pass through any unmapped columns (e.g. extra selections or virtual props). for embeddables we restrict
+        // this to declared keys, otherwise JSON keys not part of the schema leak into the original data while
+        // hydration drops them from the entity, diverging the snapshot and triggering spurious updates.
+        let knownKeysGuard = '';
+        if (meta.embeddable) {
+            const knownKeys = new Set();
+            for (const m of meta.polymorphs?.length ? meta.polymorphs : [meta]) {
+                for (const prop of m.props) {
+                    knownKeys.add(prop.name);
+                    if (prop.embedded) {
+                        knownKeys.add(prop.embedded[1]);
+                    }
+                    prop.fieldNames?.forEach(field => knownKeys.add(field));
+                }
+            }
+            context.set('knownKeys', knownKeys);
+            knownKeysGuard = ' && knownKeys.has(k)';
+        }
+        lines.push(`  for (let k in result) { if (Object.hasOwn(result, k) && !mapped[k] && ret[k] === undefined${knownKeysGuard}) ret[k] = result[k]; }`);
         const code = `// compiled mapper for entity ${meta.className}\n` +
             `return function(result) {\n  const ret = {};\n${lines.join('\n')}\n  return ret;\n}`;
         const fnKey = `resultMapper-${meta.uniqueName}`;

package/utils/Utils.js

@@ -141,7 +141,7 @@ export function parseJsonSafe(value) {
 /** Collection of general-purpose utility methods used throughout the ORM. */
 export class Utils {
     static PK_SEPARATOR = '~~~';
-    static #ORM_VERSION = '7.1.4-dev.0';
+    static #ORM_VERSION = '7.1.4-dev.1';
     /**
      * Checks if the argument is instance of `Object`. Returns false for arrays.
      */