@mikro-orm/core 7.0.15 → 7.0.16-dev.0

package/metadata/MetadataValidator.js

@@ -17,413 +17,368 @@ const DANGEROUS_PROPERTY_NAMES = ['__proto__', 'constructor', 'prototype'];
  * @internal
  */
 export class MetadataValidator {
-  validateEntityDefinition(metadata, name, options) {
-    const meta = metadata.get(name);
-    // View entities (expression with view flag) behave like regular tables but are read-only
-    // They can have primary keys and are created as actual database views
-    if (meta.view) {
-      this.validateViewEntity(meta);
-      return;
-    }
-    // Virtual entities (expression without view flag) have restrictions - no PKs, limited relation types
-    // Note: meta.virtual is set later in sync(), so we check for expression && !view here
-    if (meta.virtual || (meta.expression && !meta.view)) {
-      for (const prop of Utils.values(meta.properties)) {
-        if (
-          ![ReferenceKind.SCALAR, ReferenceKind.EMBEDDED, ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(
-            prop.kind,
-          )
-        ) {
-          throw new MetadataError(
-            `Only scalars, embedded properties and to-many relations are allowed inside virtual entity. Found '${prop.kind}' in ${meta.className}.${prop.name}`,
-          );
+    validateEntityDefinition(metadata, name, options) {
+        const meta = metadata.get(name);
+        // View entities (expression with view flag) behave like regular tables but are read-only
+        // They can have primary keys and are created as actual database views
+        if (meta.view) {
+            this.validateViewEntity(meta);
+            return;
         }
-        if (prop.primary) {
-          throw new MetadataError(
-            `Virtual entity ${meta.className} cannot have primary key ${meta.className}.${prop.name}`,
-          );
+        // Virtual entities (expression without view flag) have restrictions - no PKs, limited relation types
+        // Note: meta.virtual is set later in sync(), so we check for expression && !view here
+        if (meta.virtual || (meta.expression && !meta.view)) {
+            for (const prop of Utils.values(meta.properties)) {
+                if (![ReferenceKind.SCALAR, ReferenceKind.EMBEDDED, ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind)) {
+                    throw new MetadataError(`Only scalars, embedded properties and to-many relations are allowed inside virtual entity. Found '${prop.kind}' in ${meta.className}.${prop.name}`);
+                }
+                if (prop.primary) {
+                    throw new MetadataError(`Virtual entity ${meta.className} cannot have primary key ${meta.className}.${prop.name}`);
+                }
+            }
+            return;
+        }
+        // entities have PK
+        if (!meta.embeddable && (!meta.primaryKeys || meta.primaryKeys.length === 0)) {
+            throw MetadataError.fromMissingPrimaryKey(meta);
+        }
+        this.validateVersionField(meta);
+        this.validateDuplicateFieldNames(meta, options);
+        this.validateIndexes(meta, meta.indexes ?? [], 'index');
+        this.validateIndexes(meta, meta.uniques ?? [], 'unique');
+        this.validatePropertyNames(meta);
+        for (const prop of Utils.values(meta.properties)) {
+            if (prop.kind !== ReferenceKind.SCALAR) {
+                this.validateReference(meta, prop, options);
+                this.validateBidirectional(meta, prop);
+            }
+            else if (metadata.getByClassName(prop.type, false)) {
+                throw MetadataError.propertyTargetsEntityType(meta, prop, metadata.getByClassName(prop.type));
+            }
         }
-      }
-      return;
-    }
-    // entities have PK
-    if (!meta.embeddable && (!meta.primaryKeys || meta.primaryKeys.length === 0)) {
-      throw MetadataError.fromMissingPrimaryKey(meta);
-    }
-    this.validateVersionField(meta);
-    this.validateDuplicateFieldNames(meta, options);
-    this.validateIndexes(meta, meta.indexes ?? [], 'index');
-    this.validateIndexes(meta, meta.uniques ?? [], 'unique');
-    this.validatePropertyNames(meta);
-    for (const prop of Utils.values(meta.properties)) {
-      if (prop.kind !== ReferenceKind.SCALAR) {
-        this.validateReference(meta, prop, options);
-        this.validateBidirectional(meta, prop);
-      } else if (metadata.getByClassName(prop.type, false)) {
-        throw MetadataError.propertyTargetsEntityType(meta, prop, metadata.getByClassName(prop.type));
-      }
-    }
-  }
-  validateDiscovered(discovered, options) {
-    if (discovered.length === 0 && options.warnWhenNoEntities) {
-      throw MetadataError.noEntityDiscovered();
-    }
-    // Validate no mixing of STI and TPT in the same hierarchy
-    this.validateInheritanceStrategies(discovered);
-    const tableNames = discovered.filter(
-      meta =>
-        !meta.abstract &&
-        !meta.embeddable &&
-        meta === meta.root &&
-        (meta.tableName || meta.collection) &&
-        meta.schema !== '*',
-    );
-    const duplicateTableNames = Utils.findDuplicates(
-      tableNames.map(meta => {
-        const tableName = meta.tableName || meta.collection;
-        return (meta.schema ? '.' + meta.schema : '') + tableName;
-      }),
-    );
-    if (duplicateTableNames.length > 0 && options.checkDuplicateTableNames) {
-      throw MetadataError.duplicateEntityDiscovered(duplicateTableNames);
-    }
-    // validate we found at least one entity (not just abstract/base entities)
-    if (discovered.filter(meta => meta.name).length === 0 && options.warnWhenNoEntities) {
-      throw MetadataError.onlyAbstractEntitiesDiscovered();
-    }
-    const unwrap = type =>
-      type
-        .replace(/Array<(.*)>/, '$1') // unwrap array
-        .replace(/\[]$/, '') // remove array suffix
-        .replace(/\((.*)\)/, '$1'); // unwrap union types
-    const name = p => {
-      if (typeof p === 'function' && !p.prototype) {
-        return Utils.className(p());
-      }
-      return Utils.className(p);
-    };
-    const pivotProps = new Map();
-    // check for not discovered entities
-    discovered.forEach(meta =>
-      Object.values(meta.properties).forEach(prop => {
-        if (
-          prop.kind !== ReferenceKind.SCALAR &&
-          !unwrap(prop.type)
-            .split(/ ?\| ?/)
-            .every(type => discovered.find(m => m.className === type))
-        ) {
-          throw MetadataError.fromUnknownEntity(prop.type, `${meta.className}.${prop.name}`);
-        }
-        if (prop.pivotEntity) {
-          const props = pivotProps.get(name(prop.pivotEntity)) ?? [];
-          props.push({ meta, prop });
-          pivotProps.set(name(prop.pivotEntity), props);
-        }
-      }),
-    );
-    pivotProps.forEach(props => {
-      // if the pivot entity is used in more than one property, check if they are linked
-      if (props.length > 1 && props.every(p => !p.prop.mappedBy && !p.prop.inversedBy)) {
-        throw MetadataError.invalidManyToManyWithPivotEntity(
-          props[0].meta,
-          props[0].prop,
-          props[1].meta,
-          props[1].prop,
-        );
-      }
-    });
-  }
-  validateReference(meta, prop, options) {
-    // references do have types
-    if (!prop.type) {
-      throw MetadataError.fromWrongTypeDefinition(meta, prop);
-    }
-    // Polymorphic relations have multiple targets, validate PK compatibility
-    if (prop.polymorphic && prop.polymorphTargets) {
-      this.validatePolymorphicTargets(meta, prop);
-      return;
-    }
-    const targetMeta = prop.targetMeta;
-    // references do have type of known entity
-    if (!targetMeta) {
-      throw MetadataError.fromWrongTypeDefinition(meta, prop);
-    }
-    if (targetMeta.abstract && !targetMeta.root?.inheritanceType && !targetMeta.embeddable) {
-      throw MetadataError.targetIsAbstract(meta, prop);
-    }
-    if (
-      [ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind) &&
-      prop.persist === false &&
-      targetMeta.compositePK &&
-      options.checkNonPersistentCompositeProps
-    ) {
-      throw MetadataError.nonPersistentCompositeProp(meta, prop);
-    }
-    this.validateTargetKey(meta, prop, targetMeta);
-  }
-  validateTargetKey(meta, prop, targetMeta) {
-    if (!prop.targetKey) {
-      return;
-    }
-    // targetKey is not supported for ManyToMany relations
-    if (prop.kind === ReferenceKind.MANY_TO_MANY) {
-      throw MetadataError.targetKeyOnManyToMany(meta, prop);
-    }
-    // targetKey must point to an existing property
-    const targetProp = targetMeta.properties[prop.targetKey];
-    if (!targetProp) {
-      throw MetadataError.targetKeyNotFound(meta, prop);
-    }
-    // targetKey must point to a unique property (composite unique is not sufficient)
-    if (!this.isPropertyUnique(targetProp, targetMeta)) {
-      throw MetadataError.targetKeyNotUnique(meta, prop);
     }
-  }
-  /**
-   * Checks if a property has a unique constraint (either via `unique: true` or single-property `@Unique` decorator).
-   * Composite unique constraints are not sufficient for targetKey.
-   */
-  isPropertyUnique(prop, meta) {
-    if (prop.unique || (prop.primary && meta.primaryKeys.length === 1)) {
-      return true;
+    validateDiscovered(discovered, options) {
+        if (discovered.length === 0 && options.warnWhenNoEntities) {
+            throw MetadataError.noEntityDiscovered();
+        }
+        // Validate no mixing of STI and TPT in the same hierarchy
+        this.validateInheritanceStrategies(discovered);
+        const tableNames = discovered.filter(meta => !meta.abstract &&
+            !meta.embeddable &&
+            meta === meta.root &&
+            (meta.tableName || meta.collection) &&
+            meta.schema !== '*');
+        const duplicateTableNames = Utils.findDuplicates(tableNames.map(meta => {
+            const tableName = meta.tableName || meta.collection;
+            return (meta.schema ? '.' + meta.schema : '') + tableName;
+        }));
+        if (duplicateTableNames.length > 0 && options.checkDuplicateTableNames) {
+            throw MetadataError.duplicateEntityDiscovered(duplicateTableNames);
+        }
+        // validate we found at least one entity (not just abstract/base entities)
+        if (discovered.filter(meta => meta.name).length === 0 && options.warnWhenNoEntities) {
+            throw MetadataError.onlyAbstractEntitiesDiscovered();
+        }
+        const unwrap = (type) => type
+            .replace(/Array<(.*)>/, '$1') // unwrap array
+            .replace(/\[]$/, '') // remove array suffix
+            .replace(/\((.*)\)/, '$1'); // unwrap union types
+        const name = (p) => {
+            if (typeof p === 'function' && !p.prototype) {
+                return Utils.className(p());
+            }
+            return Utils.className(p);
+        };
+        const pivotProps = new Map();
+        // check for not discovered entities
+        discovered.forEach(meta => Object.values(meta.properties).forEach(prop => {
+            if (prop.kind !== ReferenceKind.SCALAR &&
+                !unwrap(prop.type)
+                    .split(/ ?\| ?/)
+                    .every(type => discovered.find(m => m.className === type))) {
+                throw MetadataError.fromUnknownEntity(prop.type, `${meta.className}.${prop.name}`);
+            }
+            if (prop.pivotEntity) {
+                const props = pivotProps.get(name(prop.pivotEntity)) ?? [];
+                props.push({ meta, prop });
+                pivotProps.set(name(prop.pivotEntity), props);
+            }
+        }));
+        pivotProps.forEach(props => {
+            // if the pivot entity is used in more than one property, check if they are linked
+            if (props.length > 1 && props.every(p => !p.prop.mappedBy && !p.prop.inversedBy)) {
+                throw MetadataError.invalidManyToManyWithPivotEntity(props[0].meta, props[0].prop, props[1].meta, props[1].prop);
+            }
+        });
+    }
+    validateReference(meta, prop, options) {
+        // references do have types
+        if (!prop.type) {
+            throw MetadataError.fromWrongTypeDefinition(meta, prop);
+        }
+        // Polymorphic relations have multiple targets, validate PK compatibility
+        if (prop.polymorphic && prop.polymorphTargets) {
+            this.validatePolymorphicTargets(meta, prop);
+            return;
+        }
+        const targetMeta = prop.targetMeta;
+        // references do have type of known entity
+        if (!targetMeta) {
+            throw MetadataError.fromWrongTypeDefinition(meta, prop);
+        }
+        if (targetMeta.abstract && !targetMeta.root?.inheritanceType && !targetMeta.embeddable) {
+            throw MetadataError.targetIsAbstract(meta, prop);
+        }
+        if ([ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind) &&
+            prop.persist === false &&
+            targetMeta.compositePK &&
+            options.checkNonPersistentCompositeProps) {
+            throw MetadataError.nonPersistentCompositeProp(meta, prop);
+        }
+        this.validateTargetKey(meta, prop, targetMeta);
     }
-    // Check for single-property unique constraint via @Unique decorator
-    return !!meta.uniques?.some(u => {
-      const props = Utils.asArray(u.properties);
-      return props.length === 1 && props[0] === prop.name && !u.options;
-    });
-  }
-  validatePolymorphicTargets(meta, prop) {
-    const targets = prop.polymorphTargets;
-    // Validate targetKey exists and is compatible across all targets
-    if (prop.targetKey) {
-      for (const target of targets) {
-        const targetProp = target.properties[prop.targetKey];
+    validateTargetKey(meta, prop, targetMeta) {
+        if (!prop.targetKey) {
+            return;
+        }
+        // targetKey is not supported for ManyToMany relations
+        if (prop.kind === ReferenceKind.MANY_TO_MANY) {
+            throw MetadataError.targetKeyOnManyToMany(meta, prop);
+        }
+        // targetKey must point to an existing property
+        const targetProp = targetMeta.properties[prop.targetKey];
         if (!targetProp) {
-          throw MetadataError.targetKeyNotFound(meta, prop, target);
+            throw MetadataError.targetKeyNotFound(meta, prop);
         }
         // targetKey must point to a unique property (composite unique is not sufficient)
-        if (!this.isPropertyUnique(targetProp, target)) {
-          throw MetadataError.targetKeyNotUnique(meta, prop, target);
+        if (!this.isPropertyUnique(targetProp, targetMeta)) {
+            throw MetadataError.targetKeyNotUnique(meta, prop);
         }
-      }
     }
-    const firstPKs = targets[0].getPrimaryProps();
-    for (let i = 1; i < targets.length; i++) {
-      const target = targets[i];
-      const targetPKs = target.getPrimaryProps();
-      if (targetPKs.length !== firstPKs.length) {
-        throw MetadataError.incompatiblePolymorphicTargets(
-          meta,
-          prop,
-          targets[0],
-          target,
-          'different number of primary keys',
-        );
-      }
-      for (let j = 0; j < firstPKs.length; j++) {
-        const firstPK = firstPKs[j];
-        const targetPK = targetPKs[j];
-        if (firstPK.runtimeType !== targetPK.runtimeType) {
-          throw MetadataError.incompatiblePolymorphicTargets(
-            meta,
-            prop,
-            targets[0],
-            target,
-            `incompatible primary key types: ${firstPK.name} (${firstPK.runtimeType}) vs ${targetPK.name} (${targetPK.runtimeType})`,
-          );
-        }
-      }
+    /**
+     * Checks if a property has a unique constraint (either via `unique: true` or single-property `@Unique` decorator).
+     * Composite unique constraints are not sufficient for targetKey.
+     */
+    isPropertyUnique(prop, meta) {
+        if (prop.unique || (prop.primary && meta.primaryKeys.length === 1)) {
+            return true;
+        }
+        // Check for single-property unique constraint via @Unique decorator
+        return !!meta.uniques?.some(u => {
+            const props = Utils.asArray(u.properties);
+            return props.length === 1 && props[0] === prop.name && !u.options;
+        });
+    }
+    validatePolymorphicTargets(meta, prop) {
+        const targets = prop.polymorphTargets;
+        // Validate targetKey exists and is compatible across all targets
+        if (prop.targetKey) {
+            for (const target of targets) {
+                const targetProp = target.properties[prop.targetKey];
+                if (!targetProp) {
+                    throw MetadataError.targetKeyNotFound(meta, prop, target);
+                }
+                // targetKey must point to a unique property (composite unique is not sufficient)
+                if (!this.isPropertyUnique(targetProp, target)) {
+                    throw MetadataError.targetKeyNotUnique(meta, prop, target);
+                }
+            }
+        }
+        const firstPKs = targets[0].getPrimaryProps();
+        for (let i = 1; i < targets.length; i++) {
+            const target = targets[i];
+            const targetPKs = target.getPrimaryProps();
+            if (targetPKs.length !== firstPKs.length) {
+                throw MetadataError.incompatiblePolymorphicTargets(meta, prop, targets[0], target, 'different number of primary keys');
+            }
+            for (let j = 0; j < firstPKs.length; j++) {
+                const firstPK = firstPKs[j];
+                const targetPK = targetPKs[j];
+                if (firstPK.runtimeType !== targetPK.runtimeType) {
+                    throw MetadataError.incompatiblePolymorphicTargets(meta, prop, targets[0], target, `incompatible primary key types: ${firstPK.name} (${firstPK.runtimeType}) vs ${targetPK.name} (${targetPK.runtimeType})`);
+                }
+            }
+        }
     }
-  }
-  validateBidirectional(meta, prop) {
-    if (prop.inversedBy) {
-      this.validateOwningSide(meta, prop);
-    } else if (prop.mappedBy) {
-      this.validateInverseSide(meta, prop);
-    } else if (prop.kind === ReferenceKind.ONE_TO_MANY && !prop.mappedBy) {
-      // 1:m property has `mappedBy`
-      throw MetadataError.fromMissingOption(meta, prop, 'mappedBy');
+    validateBidirectional(meta, prop) {
+        if (prop.inversedBy) {
+            this.validateOwningSide(meta, prop);
+        }
+        else if (prop.mappedBy) {
+            this.validateInverseSide(meta, prop);
+        }
+        else if (prop.kind === ReferenceKind.ONE_TO_MANY && !prop.mappedBy) {
+            // 1:m property has `mappedBy`
+            throw MetadataError.fromMissingOption(meta, prop, 'mappedBy');
+        }
     }
-  }
-  validateOwningSide(meta, prop) {
-    // For polymorphic relations, inversedBy may point to multiple entity types
-    if (prop.polymorphic && prop.polymorphTargets?.length) {
-      // For polymorphic relations, validate inversedBy against each target
-      // The inverse property should exist on the target entities and reference back to this property
-      for (const targetMeta of prop.polymorphTargets) {
-        const inverse = targetMeta.properties[prop.inversedBy];
-        // The inverse property is optional - some targets may not have it
+    validateOwningSide(meta, prop) {
+        // For polymorphic relations, inversedBy may point to multiple entity types
+        if (prop.polymorphic && prop.polymorphTargets?.length) {
+            // For polymorphic relations, validate inversedBy against each target
+            // The inverse property should exist on the target entities and reference back to this property
+            for (const targetMeta of prop.polymorphTargets) {
+                const inverse = targetMeta.properties[prop.inversedBy];
+                // The inverse property is optional - some targets may not have it
+                if (!inverse) {
+                    continue;
+                }
+                // Validate the inverse property
+                if (inverse.targetMeta?.root.class !== meta.root.class) {
+                    throw MetadataError.fromWrongReference(meta, prop, 'inversedBy', inverse);
+                }
+                // inverse side is not defined as owner
+                if (inverse.inversedBy || inverse.owner) {
+                    throw MetadataError.fromWrongOwnership(meta, prop, 'inversedBy');
+                }
+            }
+            return;
+        }
+        const inverse = prop.targetMeta.properties[prop.inversedBy];
+        // has correct `inversedBy` on owning side
         if (!inverse) {
-          continue;
+            throw MetadataError.fromWrongReference(meta, prop, 'inversedBy');
         }
-        // Validate the inverse property
-        if (inverse.targetMeta?.root.class !== meta.root.class) {
-          throw MetadataError.fromWrongReference(meta, prop, 'inversedBy', inverse);
+        const targetClass = inverse.targetMeta?.root.class;
+        // has correct `inversedBy` reference type
+        if (inverse.type !== meta.className && targetClass !== meta.root.class) {
+            throw MetadataError.fromWrongReference(meta, prop, 'inversedBy', inverse);
         }
         // inverse side is not defined as owner
         if (inverse.inversedBy || inverse.owner) {
-          throw MetadataError.fromWrongOwnership(meta, prop, 'inversedBy');
+            throw MetadataError.fromWrongOwnership(meta, prop, 'inversedBy');
         }
-      }
-      return;
-    }
-    const inverse = prop.targetMeta.properties[prop.inversedBy];
-    // has correct `inversedBy` on owning side
-    if (!inverse) {
-      throw MetadataError.fromWrongReference(meta, prop, 'inversedBy');
-    }
-    const targetClass = inverse.targetMeta?.root.class;
-    // has correct `inversedBy` reference type
-    if (inverse.type !== meta.className && targetClass !== meta.root.class) {
-      throw MetadataError.fromWrongReference(meta, prop, 'inversedBy', inverse);
-    }
-    // inverse side is not defined as owner
-    if (inverse.inversedBy || inverse.owner) {
-      throw MetadataError.fromWrongOwnership(meta, prop, 'inversedBy');
-    }
-  }
-  validateInverseSide(meta, prop) {
-    const owner = prop.targetMeta.properties[prop.mappedBy];
-    // has correct `mappedBy` on inverse side
-    if (prop.mappedBy && !owner) {
-      throw MetadataError.fromWrongReference(meta, prop, 'mappedBy');
-    }
-    // has correct `mappedBy` reference type
-    // For polymorphic relations, check if this entity is one of the polymorphic targets
-    const isValidPolymorphicInverse =
-      owner.polymorphic && owner.polymorphTargets?.some(target => target.class === meta.root.class);
-    if (
-      !isValidPolymorphicInverse &&
-      owner.type !== meta.className &&
-      owner.targetMeta?.root.class !== meta.root.class
-    ) {
-      throw MetadataError.fromWrongReference(meta, prop, 'mappedBy', owner);
-    }
-    // owning side is not defined as inverse
-    if (owner.mappedBy) {
-      throw MetadataError.fromWrongOwnership(meta, prop, 'mappedBy');
-    }
-    // owning side is not defined as inverse
-    const valid = [
-      { owner: ReferenceKind.MANY_TO_ONE, inverse: ReferenceKind.ONE_TO_MANY },
-      { owner: ReferenceKind.MANY_TO_MANY, inverse: ReferenceKind.MANY_TO_MANY },
-      { owner: ReferenceKind.ONE_TO_ONE, inverse: ReferenceKind.ONE_TO_ONE },
-    ];
-    if (!valid.find(spec => spec.owner === owner.kind && spec.inverse === prop.kind)) {
-      throw MetadataError.fromWrongReferenceKind(meta, owner, prop);
-    }
-    if (prop.primary) {
-      throw MetadataError.fromInversideSidePrimary(meta, owner, prop);
-    }
-  }
-  validateIndexes(meta, indexes, type) {
-    for (const index of indexes) {
-      for (const propName of Utils.asArray(index.properties)) {
-        const prop = meta.properties[propName] ?? meta.root.properties[propName];
-        if (
-          !prop &&
-          !Object.values(meta.properties).some(p => propName.startsWith(p.name + '.')) &&
-          !Object.values(meta.root.properties).some(p => propName.startsWith(p.name + '.'))
-        ) {
-          throw MetadataError.unknownIndexProperty(meta, propName, type);
-        }
-      }
-    }
-  }
-  validateDuplicateFieldNames(meta, options) {
-    const candidates = Object.values(meta.properties)
-      .filter(
-        prop =>
-          prop.persist !== false &&
-          !prop.inherited &&
-          prop.fieldNames?.length === 1 &&
-          (prop.kind !== ReferenceKind.EMBEDDED || prop.object),
-      )
-      .map(prop => prop.fieldNames[0]);
-    const duplicates = Utils.findDuplicates(candidates);
-    if (duplicates.length > 0 && options.checkDuplicateFieldNames) {
-      const pairs = duplicates.flatMap(name => {
-        return Object.values(meta.properties)
-          .filter(p => p.fieldNames?.[0] === name)
-          .map(prop => {
-            return [prop.embedded ? prop.embedded.join('.') : prop.name, prop.fieldNames[0]];
-          });
-      });
-      throw MetadataError.duplicateFieldName(meta.class, pairs);
     }
-  }
-  validateVersionField(meta) {
-    if (!meta.versionProperty) {
-      return;
+    validateInverseSide(meta, prop) {
+        const owner = prop.targetMeta.properties[prop.mappedBy];
+        // has correct `mappedBy` on inverse side
+        if (prop.mappedBy && !owner) {
+            throw MetadataError.fromWrongReference(meta, prop, 'mappedBy');
+        }
+        // has correct `mappedBy` reference type
+        // For polymorphic relations, check if this entity is one of the polymorphic targets
+        const isValidPolymorphicInverse = owner.polymorphic && owner.polymorphTargets?.some(target => target.class === meta.root.class);
+        if (!isValidPolymorphicInverse &&
+            owner.type !== meta.className &&
+            owner.targetMeta?.root.class !== meta.root.class) {
+            throw MetadataError.fromWrongReference(meta, prop, 'mappedBy', owner);
+        }
+        // owning side is not defined as inverse
+        if (owner.mappedBy) {
+            throw MetadataError.fromWrongOwnership(meta, prop, 'mappedBy');
+        }
+        // owning side is not defined as inverse
+        const valid = [
+            { owner: ReferenceKind.MANY_TO_ONE, inverse: ReferenceKind.ONE_TO_MANY },
+            { owner: ReferenceKind.MANY_TO_MANY, inverse: ReferenceKind.MANY_TO_MANY },
+            { owner: ReferenceKind.ONE_TO_ONE, inverse: ReferenceKind.ONE_TO_ONE },
+        ];
+        if (!valid.find(spec => spec.owner === owner.kind && spec.inverse === prop.kind)) {
+            throw MetadataError.fromWrongReferenceKind(meta, owner, prop);
+        }
+        if (prop.primary) {
+            throw MetadataError.fromInversideSidePrimary(meta, owner, prop);
+        }
     }
-    const props = Object.values(meta.properties).filter(p => p.version);
-    if (props.length > 1) {
-      throw MetadataError.multipleVersionFields(
-        meta,
-        props.map(p => p.name),
-      );
+    validateIndexes(meta, indexes, type) {
+        for (const index of indexes) {
+            for (const propName of Utils.asArray(index.properties)) {
+                const prop = meta.properties[propName] ?? meta.root.properties[propName];
+                if (!prop &&
+                    !Object.values(meta.properties).some(p => propName.startsWith(p.name + '.')) &&
+                    !Object.values(meta.root.properties).some(p => propName.startsWith(p.name + '.'))) {
+                    throw MetadataError.unknownIndexProperty(meta, propName, type);
+                }
+            }
+        }
     }
-    const prop = meta.properties[meta.versionProperty];
-    const type = prop.runtimeType ?? prop.columnTypes?.[0] ?? prop.type;
-    if (type !== 'number' && type !== 'Date' && !type.startsWith('timestamp') && !type.startsWith('datetime')) {
-      throw MetadataError.invalidVersionFieldType(meta);
+    validateDuplicateFieldNames(meta, options) {
+        const candidates = Object.values(meta.properties)
+            .filter(prop => prop.persist !== false &&
+            !prop.inherited &&
+            prop.fieldNames?.length === 1 &&
+            (prop.kind !== ReferenceKind.EMBEDDED || prop.object))
+            .map(prop => prop.fieldNames[0]);
+        const duplicates = Utils.findDuplicates(candidates);
+        if (duplicates.length > 0 && options.checkDuplicateFieldNames) {
+            const pairs = duplicates.flatMap(name => {
+                return Object.values(meta.properties)
+                    .filter(p => p.fieldNames?.[0] === name)
+                    .map(prop => {
+                    return [prop.embedded ? prop.embedded.join('.') : prop.name, prop.fieldNames[0]];
+                });
+            });
+            throw MetadataError.duplicateFieldName(meta.class, pairs);
+        }
     }
-  }
-  /**
-   * Validates that entity properties do not use dangerous names that could lead to
-   * prototype pollution vulnerabilities. This validation ensures that property names
-   * cannot be exploited to modify object prototypes when values are assigned during
-   * entity hydration or persistence operations.
-   *
-   * @internal
-   */
-  validatePropertyNames(meta) {
-    for (const prop of Utils.values(meta.properties)) {
-      if (DANGEROUS_PROPERTY_NAMES.includes(prop.name)) {
-        throw MetadataError.dangerousPropertyName(meta, prop);
-      }
+    validateVersionField(meta) {
+        if (!meta.versionProperty) {
+            return;
+        }
+        const props = Object.values(meta.properties).filter(p => p.version);
+        if (props.length > 1) {
+            throw MetadataError.multipleVersionFields(meta, props.map(p => p.name));
+        }
+        const prop = meta.properties[meta.versionProperty];
+        const type = prop.runtimeType ?? prop.columnTypes?.[0] ?? prop.type;
+        if (type !== 'number' && type !== 'Date' && !type.startsWith('timestamp') && !type.startsWith('datetime')) {
+            throw MetadataError.invalidVersionFieldType(meta);
+        }
     }
-  }
-  /**
-   * Validates view entity configuration.
-   * View entities must have an expression.
-   */
-  validateViewEntity(meta) {
-    // View entities must have an expression
-    if (!meta.expression) {
-      throw MetadataError.viewEntityWithoutExpression(meta);
+    /**
+     * Validates that entity properties do not use dangerous names that could lead to
+     * prototype pollution vulnerabilities. This validation ensures that property names
+     * cannot be exploited to modify object prototypes when values are assigned during
+     * entity hydration or persistence operations.
+     *
+     * @internal
+     */
+    validatePropertyNames(meta) {
+        for (const prop of Utils.values(meta.properties)) {
+            if (DANGEROUS_PROPERTY_NAMES.includes(prop.name)) {
+                throw MetadataError.dangerousPropertyName(meta, prop);
+            }
+        }
     }
-    // Validate indexes if present
-    this.validateIndexes(meta, meta.indexes ?? [], 'index');
-    this.validateIndexes(meta, meta.uniques ?? [], 'unique');
-    // Validate property names
-    this.validatePropertyNames(meta);
-  }
-  /**
-   * Validates that STI and TPT are not mixed in the same inheritance hierarchy.
-   * An entity hierarchy can use either STI (discriminatorColumn) or TPT (inheritance: 'tpt'),
-   * but not both.
-   *
-   * Note: This validation runs before `initTablePerTypeInheritance` sets `inheritanceType`,
-   * so we check the raw `inheritance` option from the decorator/schema.
-   */
-  validateInheritanceStrategies(discovered) {
-    const checkedRoots = new Set();
-    for (const meta of discovered) {
-      if (meta.embeddable) {
-        continue;
-      }
-      const root = meta.root;
-      if (checkedRoots.has(root)) {
-        continue;
-      }
-      checkedRoots.add(root);
-      const hasSTI = !!root.discriminatorColumn;
-      const hasTPT = root.inheritanceType === 'tpt' || root.inheritance === 'tpt';
-      if (hasSTI && hasTPT) {
-        throw MetadataError.mixedInheritanceStrategies(root, meta);
-      }
+    /**
+     * Validates view entity configuration.
+     * View entities must have an expression.
+     */
+    validateViewEntity(meta) {
+        // View entities must have an expression
+        if (!meta.expression) {
+            throw MetadataError.viewEntityWithoutExpression(meta);
+        }
+        // Validate indexes if present
+        this.validateIndexes(meta, meta.indexes ?? [], 'index');
+        this.validateIndexes(meta, meta.uniques ?? [], 'unique');
+        // Validate property names
+        this.validatePropertyNames(meta);
+    }
+    /**
+     * Validates that STI and TPT are not mixed in the same inheritance hierarchy.
+     * An entity hierarchy can use either STI (discriminatorColumn) or TPT (inheritance: 'tpt'),
+     * but not both.
+     *
+     * Note: This validation runs before `initTablePerTypeInheritance` sets `inheritanceType`,
+     * so we check the raw `inheritance` option from the decorator/schema.
+     */
+    validateInheritanceStrategies(discovered) {
+        const checkedRoots = new Set();
+        for (const meta of discovered) {
+            if (meta.embeddable) {
+                continue;
+            }
+            const root = meta.root;
+            if (checkedRoots.has(root)) {
+                continue;
+            }
+            checkedRoots.add(root);
+            const hasSTI = !!root.discriminatorColumn;
+            const hasTPT = root.inheritanceType === 'tpt' || root.inheritance === 'tpt';
+            if (hasSTI && hasTPT) {
+                throw MetadataError.mixedInheritanceStrategies(root, meta);
+            }
+        }
     }
-  }
 }