@mikro-orm/core 7.0.0-dev.300 → 7.0.0-dev.301

package/EntityManager.d.ts

@@ -594,7 +594,7 @@ export interface MergeOptions {
     schema?: string;
     disableContextResolution?: boolean;
     validate?: boolean;
-    cascade?: boolean; /** @default true */
+    cascade?: boolean /** @default true */;
 }
 export interface ForkOptions {
     /** do we want a clear identity map? defaults to true */

package/EntityManager.js

@@ -123,7 +123,7 @@ export class EntityManager {
         else {
             options.orderBy ??= {};
         }
-        options.populate = await em.preparePopulate(entityName, options);
+        options.populate = (await em.preparePopulate(entityName, options));
         const populate = options.populate;
         const cacheKey = em.cacheKey(entityName, options, 'em.find', where);
         const cached = await em.tryCache(entityName, options.cache, cacheKey, options.refresh, true);
@@ -195,10 +195,10 @@ export class EntityManager {
         em.prepareOptions(options);
         options.strategy = 'joined';
         await em.tryFlush(entityName, options);
-        const where = await em.processWhere(entityName, options.where ?? {}, options, 'read');
+        const where = (await em.processWhere(entityName, options.where ?? {}, options, 'read'));
         validateParams(where);
         options.orderBy = options.orderBy || {};
-        options.populate = await em.preparePopulate(entityName, options);
+        options.populate = (await em.preparePopulate(entityName, options));
         const meta = this.metadata.get(entityName);
         options = { ...options };
         // save the original hint value so we know it was infer/all
@@ -304,7 +304,7 @@ export class EntityManager {
             if (!this.driver.getPlatform().supportsUnionWhere()) {
                 throw new Error(`unionWhere is only supported on SQL drivers`);
             }
-            options.unionWhere = await Promise.all(options.unionWhere.map(branch => this.processWhere(entityName, branch, options, type)));
+            options.unionWhere = (await Promise.all(options.unionWhere.map(branch => this.processWhere(entityName, branch, options, type))));
         }
     }
     // this method only handles the problem for mongo driver, SQL drivers have their implementation inside QueryBuilder
@@ -323,7 +323,10 @@ export class EntityManager {
         };
         lookUpChildren(children, meta.class);
         /* v8 ignore next */
-        where[meta.root.discriminatorColumn] = children.length > 0 ? { $in: [meta.discriminatorValue, ...children.map(c => c.discriminatorValue)] } : meta.discriminatorValue;
+        where[meta.root.discriminatorColumn] =
+            children.length > 0
+                ? { $in: [meta.discriminatorValue, ...children.map(c => c.discriminatorValue)] }
+                : meta.discriminatorValue;
         return where;
     }
     createPopulateWhere(cond, options) {
@@ -390,10 +393,11 @@ export class EntityManager {
         }
         const ret = options.populate;
         for (const prop of meta.relations) {
-            if (prop.object
-                || ![ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind)
-                || !((options.fields?.length ?? 0) === 0 || options.fields?.some(f => prop.name === f || prop.name.startsWith(`${String(f)}.`)))
-                || (parent?.class === prop.targetMeta.root.class && parent.propName === prop.inversedBy)) {
+            if (prop.object ||
+                ![ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind) ||
+                !((options.fields?.length ?? 0) === 0 ||
+                    options.fields?.some(f => prop.name === f || prop.name.startsWith(`${String(f)}.`))) ||
+                (parent?.class === prop.targetMeta.root.class && parent.propName === prop.inversedBy)) {
                 continue;
             }
             options = { ...options, filters: QueryHelper.mergePropertyFilters(prop.filters, options.filters) };
@@ -433,9 +437,7 @@ export class EntityManager {
         const ret = [];
         const active = new Set();
         const push = (source) => {
-            const activeFilters = QueryHelper
-                .getActiveFilters(meta, options, source)
-                .filter(f => !active.has(f.name));
+            const activeFilters = QueryHelper.getActiveFilters(meta, options, source).filter(f => !active.has(f.name));
             filters.push(...activeFilters);
             activeFilters.forEach(f => active.add(f.name));
         };
@@ -449,6 +451,7 @@ export class EntityManager {
             let cond;
             if (filter.cond instanceof Function) {
                 // @ts-ignore
+                // oxfmt-ignore
                 const args = Utils.isPlainObject(options?.[filter.name]) ? options[filter.name] : this.getContext().filterParams[filter.name];
                 if (!args && filter.cond.length > 0 && filter.args !== false) {
                     throw new Error(`No arguments provided for filter '${filter.name}'`);
@@ -591,12 +594,18 @@ export class EntityManager {
         for (const e of fork.unitOfWork.getIdentityMap()) {
             const ref = em.getReference(e.constructor, helper(e).getPrimaryKey());
             const data = helper(e).serialize({ ignoreSerializers: true, includeHidden: true, convertCustomTypes: false });
-            em.config.getHydrator(this.metadata).hydrate(ref, helper(ref).__meta, data, em.entityFactory, 'full', false, false);
+            em.config
+                .getHydrator(this.metadata)
+                .hydrate(ref, helper(ref).__meta, data, em.entityFactory, 'full', false, false);
             Utils.merge(helper(ref).__originalEntityData, this.comparator.prepareEntity(e));
             found ||= ref === entity;
         }
         if (!found) {
-            const data = helper(reloaded).serialize({ ignoreSerializers: true, includeHidden: true, convertCustomTypes: true });
+            const data = helper(reloaded).serialize({
+                ignoreSerializers: true,
+                includeHidden: true,
+                convertCustomTypes: true,
+            });
             em.config.getHydrator(this.metadata).hydrate(entity, wrapped.__meta, data, em.entityFactory, 'full', false, true);
             Utils.merge(wrapped.__originalEntityData, this.comparator.prepareEntity(reloaded));
         }
@@ -632,7 +641,7 @@ export class EntityManager {
             return em.lockAndPopulate(meta, entity, where, options);
         }
         validateParams(where);
-        options.populate = await em.preparePopulate(entityName, options);
+        options.populate = (await em.preparePopulate(entityName, options));
         const cacheKey = em.cacheKey(entityName, options, 'em.findOne', where);
         const cached = await em.tryCache(entityName, options.cache, cacheKey, options.refresh, true);
         if (cached?.data !== undefined) {
@@ -780,9 +789,12 @@ export class EntityManager {
             initialized: true,
             schema: options.schema,
         });
-        const uniqueFields = options.onConflictFields ?? (Utils.isPlainObject(where) ? Object.keys(where) : meta.primaryKeys);
+        const uniqueFields = options.onConflictFields ??
+            (Utils.isPlainObject(where) ? Object.keys(where) : meta.primaryKeys);
         const returning = getOnConflictReturningFields(meta, data, uniqueFields, options);
-        if (options.onConflictAction === 'ignore' || !helper(entity).hasPrimaryKey() || (returning.length > 0 && !(this.getPlatform().usesReturningStatement() && ret.row))) {
+        if (options.onConflictAction === 'ignore' ||
+            !helper(entity).hasPrimaryKey() ||
+            (returning.length > 0 && !(this.getPlatform().usesReturningStatement() && ret.row))) {
             const where = {};
             if (Array.isArray(uniqueFields)) {
                 for (const prop of uniqueFields) {
@@ -906,7 +918,9 @@ export class EntityManager {
                 }
             }
             const unique = options.onConflictFields ?? meta.props.filter(p => p.unique).map(p => p.name);
-            propIndex = !isRaw(unique) && unique.findIndex(p => data[p] ?? data[p.substring(0, p.indexOf('.'))] != null);
+            propIndex =
+                !isRaw(unique) &&
+                    unique.findIndex(p => data[p] ?? data[p.substring(0, p.indexOf('.'))] != null);
             const tmp = getWhereCondition(meta, options.onConflictFields, row, where);
             propIndex = tmp.propIndex;
             where = QueryHelper.processWhere({
@@ -939,12 +953,16 @@ export class EntityManager {
         entitiesByData.clear();
         const loadPK = new Map();
         allData.forEach((row, i) => {
-            em.unitOfWork.getChangeSetPersister().mapReturnedValues(Utils.isEntity(data[i]) ? data[i] : null, Utils.isEntity(data[i]) ? {} : data[i], res.rows?.[i], meta, true);
-            const entity = Utils.isEntity(data[i]) ? data[i] : em.entityFactory.create(entityName, row, {
-                refresh: true,
-                initialized: true,
-                schema: options.schema,
-            });
+            em.unitOfWork
+                .getChangeSetPersister()
+                .mapReturnedValues(Utils.isEntity(data[i]) ? data[i] : null, Utils.isEntity(data[i]) ? {} : data[i], res.rows?.[i], meta, true);
+            const entity = Utils.isEntity(data[i])
+                ? data[i]
+                : em.entityFactory.create(entityName, row, {
+                    refresh: true,
+                    initialized: true,
+                    schema: options.schema,
+                });
             if (!helper(entity).hasPrimaryKey()) {
                 loadPK.set(entity, allWhere[i]);
             }
@@ -952,12 +970,15 @@ export class EntityManager {
             entitiesByData.set(row, entity);
         });
         // skip if we got the PKs via returning statement (`rows`)
+        // oxfmt-ignore
         const uniqueFields = options.onConflictFields ?? (Utils.isPlainObject(allWhere[0]) ? Object.keys(allWhere[0]).flatMap(key => Utils.splitPrimaryKeys(key)) : meta.primaryKeys);
         const returning = getOnConflictReturningFields(meta, data[0], uniqueFields, options);
         const reloadFields = returning.length > 0 && !(this.getPlatform().usesReturningStatement() && res.rows?.length);
         if (options.onConflictAction === 'ignore' || (!res.rows?.length && loadPK.size > 0) || reloadFields) {
             const unique = meta.hydrateProps.filter(p => !p.lazy).map(p => p.name);
-            const add = new Set(propIndex !== false && propIndex >= 0 ? [unique[propIndex]] : []);
+            const add = new Set(propIndex !== false && propIndex >= 0
+                ? [unique[propIndex]]
+                : []);
             for (const cond of loadPK.values()) {
                 Utils.keys(cond).forEach(key => add.add(key));
             }
@@ -970,7 +991,9 @@ export class EntityManager {
                 });
             });
             const data2 = await this.driver.find(meta.class, where, {
-                fields: returning.concat(...add).concat(...(Array.isArray(uniqueFields) ? uniqueFields : [])),
+                fields: returning
+                    .concat(...add)
+                    .concat(...(Array.isArray(uniqueFields) ? uniqueFields : [])),
                 ctx: em.transactionContext,
                 convertCustomTypes: true,
                 connectionType: 'write',
@@ -1146,7 +1169,10 @@ export class EntityManager {
         }
         data = QueryHelper.processObjectParams(data);
         validateParams(data, 'insert data');
-        const res = await em.driver.nativeInsert(entityName, data, { ctx: em.transactionContext, ...options });
+        const res = await em.driver.nativeInsert(entityName, data, {
+            ctx: em.transactionContext,
+            ...options,
+        });
         return res.insertId;
     }
     /**
@@ -1186,7 +1212,10 @@ export class EntityManager {
         }
         data = data.map(row => QueryHelper.processObjectParams(row));
         data.forEach(row => validateParams(row, 'insert data'));
-        const res = await em.driver.nativeInsertMany(entityName, data, { ctx: em.transactionContext, ...options });
+        const res = await em.driver.nativeInsertMany(entityName, data, {
+            ctx: em.transactionContext,
+            ...options,
+        });
         if (res.insertedIds) {
             return res.insertedIds;
         }
@@ -1203,7 +1232,11 @@ export class EntityManager {
         where = await em.processWhere(entityName, where, { ...options, convertCustomTypes: false }, 'update');
         validateParams(data, 'update data');
         validateParams(where, 'update condition');
-        const res = await em.driver.nativeUpdate(entityName, where, data, { ctx: em.transactionContext, em, ...options });
+        const res = await em.driver.nativeUpdate(entityName, where, data, {
+            ctx: em.transactionContext,
+            em,
+            ...options,
+        });
         return res.affectedRows;
     }
     /**
@@ -1213,9 +1246,13 @@ export class EntityManager {
         const em = this.getContext(false);
         em.prepareOptions(options);
         await em.processUnionWhere(entityName, options, 'delete');
-        where = await em.processWhere(entityName, where, options, 'delete');
+        where = (await em.processWhere(entityName, where, options, 'delete'));
         validateParams(where, 'delete condition');
-        const res = await em.driver.nativeDelete(entityName, where, { ctx: em.transactionContext, em, ...options });
+        const res = await em.driver.nativeDelete(entityName, where, {
+            ctx: em.transactionContext,
+            em,
+            ...options,
+        });
         return res.affectedRows;
     }
     /**
@@ -1226,7 +1263,10 @@ export class EntityManager {
         const data = this.driver.mapResult(result, meta);
         for (const k of Object.keys(data)) {
             const prop = meta.properties[k];
-            if (prop?.kind === ReferenceKind.SCALAR && SCALAR_TYPES.has(prop.runtimeType) && !prop.customType && (prop.setter || !prop.getter)) {
+            if (prop?.kind === ReferenceKind.SCALAR &&
+                SCALAR_TYPES.has(prop.runtimeType) &&
+                !prop.customType &&
+                (prop.setter || !prop.getter)) {
                 validateProperty(prop, data[k], data);
             }
         }
@@ -1255,7 +1295,9 @@ export class EntityManager {
             return entity;
         }
         const dataIsEntity = Utils.isEntity(data);
-        entity = dataIsEntity ? data : em.entityFactory.create(entityName, data, { merge: true, ...options });
+        entity = dataIsEntity
+            ? data
+            : em.entityFactory.create(entityName, data, { merge: true, ...options });
         const visited = options.cascade ? undefined : new Set([entity]);
         em.unitOfWork.merge(entity, visited);
         return entity;
@@ -1325,7 +1367,7 @@ export class EntityManager {
         em.prepareOptions(options);
         await em.tryFlush(entityName, options);
         where = await em.processWhere(entityName, where, options, 'read');
-        options.populate = await em.preparePopulate(entityName, options);
+        options.populate = (await em.preparePopulate(entityName, options));
         options = { ...options };
         // save the original hint value so we know it was infer/all
         const meta = em.metadata.find(entityName);
@@ -1472,7 +1514,8 @@ export class EntityManager {
         em.config.set('allowGlobalContext', true);
         const fork = new em.constructor(em.config, em.driver, em.metadata, options.useContext, eventManager);
         fork.setFlushMode(options.flushMode ?? em.flushMode);
-        fork.disableTransactions = options.disableTransactions ?? this.disableTransactions ?? this.config.get('disableTransactions');
+        fork.disableTransactions =
+            options.disableTransactions ?? this.disableTransactions ?? this.config.get('disableTransactions');
         em.config.set('allowGlobalContext', allowGlobalContext);
         if (options.keepTransactionContext) {
             fork.transactionContext = em.transactionContext;
@@ -1639,11 +1682,14 @@ export class EntityManager {
                 const ret = [];
                 for (let field of fields) {
                     if (field === PopulatePath.ALL || field.startsWith(`${PopulatePath.ALL}.`)) {
-                        ret.push(...meta.props.filter(prop => prop.lazy || [ReferenceKind.SCALAR, ReferenceKind.EMBEDDED].includes(prop.kind)).map(prop => prop.name));
+                        ret.push(...meta.props
+                            .filter(prop => prop.lazy || [ReferenceKind.SCALAR, ReferenceKind.EMBEDDED].includes(prop.kind))
+                            .map(prop => prop.name));
                         continue;
                     }
                     field = field.split(':')[0];
-                    if (!field.includes('.') && ![ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(meta.properties[field].kind)) {
+                    if (!field.includes('.') &&
+                        ![ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(meta.properties[field].kind)) {
                         ret.push(field);
                         continue;
                     }
@@ -1672,7 +1718,8 @@ export class EntityManager {
             return populate;
         }
         if (typeof options.populate !== 'boolean') {
-            options.populate = Utils.asArray(options.populate).map(field => {
+            options.populate = Utils.asArray(options.populate)
+                .map(field => {
                 /* v8 ignore next */
                 if (typeof field === 'boolean' || field === PopulatePath.ALL) {
                     return [{ field: meta.primaryKeys[0], strategy: options.strategy, all: !!field }]; //
@@ -1687,7 +1734,8 @@ export class EntityManager {
                     return [{ field, strategy: options.strategy }];
                 }
                 return [field];
-            }).flat();
+            })
+                .flat();
         }
         const populate = this.entityLoader.normalizePopulate(entityName, options.populate, options.strategy, true, options.exclude);
         const invalid = populate.find(({ field }) => !this.canPopulate(entityName, field));
@@ -1789,7 +1837,7 @@ export class EntityManager {
         config ??= this.config.get('resultCache').global;
         if (config) {
             const em = this.getContext();
-            const expiration = Array.isArray(config) ? config[1] : (typeof config === 'number' ? config : undefined);
+            const expiration = Array.isArray(config) ? config[1] : typeof config === 'number' ? config : undefined;
             await em.resultCache.set(key.key, data instanceof Function ? data() : data, '', expiration);
         }
     }
@@ -1831,9 +1879,12 @@ export class EntityManager {
         const { DataloaderUtils } = await import('@mikro-orm/core/dataloader');
         const DataLoader = await DataloaderUtils.getDataLoader();
         switch (type) {
-            case 'ref': return (em.loaders[type] ??= new DataLoader(DataloaderUtils.getRefBatchLoadFn(em)));
-            case '1:m': return (em.loaders[type] ??= new DataLoader(DataloaderUtils.getColBatchLoadFn(em)));
-            case 'm:n': return (em.loaders[type] ??= new DataLoader(DataloaderUtils.getManyToManyColBatchLoadFn(em)));
+            case 'ref':
+                return (em.loaders[type] ??= new DataLoader(DataloaderUtils.getRefBatchLoadFn(em)));
+            case '1:m':
+                return (em.loaders[type] ??= new DataLoader(DataloaderUtils.getColBatchLoadFn(em)));
+            case 'm:n':
+                return (em.loaders[type] ??= new DataLoader(DataloaderUtils.getManyToManyColBatchLoadFn(em)));
         }
     }
     /**

package/MikroORM.js

@@ -104,9 +104,7 @@ export class MikroORM {
      */
     constructor(options) {
         const env = loadEnvironmentVars();
-        options = options.preferEnvVars
-            ? Utils.merge(options, env)
-            : Utils.merge(env, options);
+        options = options.preferEnvVars ? Utils.merge(options, env) : Utils.merge(env, options);
         this.config = new Configuration(options);
         const discovery = this.config.get('discovery');
         this.driver = this.config.getDriver();
@@ -213,6 +211,8 @@ export class MikroORM {
      * Gets the EntityGenerator.
      */
     get entityGenerator() {
-        return this.driver.getPlatform().getExtension('EntityGenerator', '@mikro-orm/entity-generator', '@mikro-orm/entity-generator', this.em);
+        return this.driver
+            .getPlatform()
+            .getExtension('EntityGenerator', '@mikro-orm/entity-generator', '@mikro-orm/entity-generator', this.em);
     }
 }

package/cache/FileCacheAdapter.js

@@ -68,9 +68,7 @@ export class FileCacheAdapter {
         if (!this.options.combined) {
             return;
         }
-        let path = typeof this.options.combined === 'string'
-            ? this.options.combined
-            : './metadata.json';
+        let path = typeof this.options.combined === 'string' ? this.options.combined : './metadata.json';
         path = fs.normalizePath(this.options.cacheDir, path);
         this.options.combined = path; // override in the options, so we can log it from the CLI in `cache:generate` command
         writeFileSync(path, JSON.stringify(this.cache, null, this.pretty ? 2 : undefined));

package/connections/Connection.js

@@ -16,7 +16,18 @@ export class Connection {
             this.options = Utils.copy(options);
         }
         else {
-            const props = ['dbName', 'clientUrl', 'host', 'port', 'user', 'password', 'multipleStatements', 'pool', 'schema', 'driverOptions'];
+            const props = [
+                'dbName',
+                'clientUrl',
+                'host',
+                'port',
+                'user',
+                'password',
+                'multipleStatements',
+                'pool',
+                'schema',
+                'driverOptions',
+            ];
             this.options = props.reduce((o, i) => {
                 o[i] = this.config.get(i);
                 return o;
@@ -89,8 +100,10 @@ export class Connection {
             this.options.host = ret.host = this.options.host ?? this.config.get('host', decodeURIComponent(url.hostname));
             this.options.port = ret.port = this.options.port ?? this.config.get('port', +url.port);
             this.options.user = ret.user = this.options.user ?? this.config.get('user', decodeURIComponent(url.username));
-            this.options.password = ret.password = this.options.password ?? this.config.get('password', decodeURIComponent(url.password));
-            this.options.dbName = ret.database = this.options.dbName ?? this.config.get('dbName', decodeURIComponent(url.pathname).replace(/^\//, ''));
+            this.options.password = ret.password =
+                this.options.password ?? this.config.get('password', decodeURIComponent(url.password));
+            this.options.dbName = ret.database =
+                this.options.dbName ?? this.config.get('dbName', decodeURIComponent(url.pathname).replace(/^\//, ''));
         }
         return ret;
     }

package/drivers/DatabaseDriver.js

@@ -157,11 +157,11 @@ export class DatabaseDriver {
                     Object.assign(o, createOrderBy(key, direction[key]));
                     return o;
                 }, {});
-                return ({ [prop]: value });
+                return { [prop]: value };
             }
             const desc = direction === QueryOrderNumeric.DESC || direction.toString().toLowerCase() === 'desc';
             const dir = Utils.xor(desc, isLast) ? 'desc' : 'asc';
-            return ({ [prop]: dir });
+            return { [prop]: dir };
         };
         return {
             orderBy: definition.map(([prop, direction]) => createOrderBy(prop, direction)),
@@ -301,16 +301,23 @@ export class DatabaseDriver {
             if (prop.joinColumns && Array.isArray(data[k])) {
                 const copy = Utils.flatten(data[k]);
                 delete data[k];
-                prop.joinColumns.forEach((joinColumn, idx) => data[joinColumn] = copy[idx]);
+                prop.joinColumns.forEach((joinColumn, idx) => (data[joinColumn] = copy[idx]));
                 return;
             }
             if (prop.joinColumns?.length > 1 && data[k] == null) {
                 delete data[k];
-                prop.ownColumns.forEach(joinColumn => data[joinColumn] = null);
+                prop.ownColumns.forEach(joinColumn => (data[joinColumn] = null));
                 return;
             }
-            if (prop.customType && convertCustomTypes && !(prop.customType instanceof JsonType && object) && !isRaw(data[k])) {
-                data[k] = prop.customType.convertToDatabaseValue(data[k], this.platform, { fromQuery: true, key: k, mode: 'query-data' });
+            if (prop.customType &&
+                convertCustomTypes &&
+                !(prop.customType instanceof JsonType && object) &&
+                !isRaw(data[k])) {
+                data[k] = prop.customType.convertToDatabaseValue(data[k], this.platform, {
+                    fromQuery: true,
+                    key: k,
+                    mode: 'query-data',
+                });
             }
             if (prop.hasConvertToDatabaseValueSQL && !prop.object && !isRaw(data[k])) {
                 const quoted = this.platform.quoteValue(data[k]);
@@ -390,7 +397,18 @@ export class DatabaseDriver {
     createReplicas(cb) {
         const replicas = this.config.get('replicas', []);
         const ret = [];
-        const props = ['dbName', 'clientUrl', 'host', 'port', 'user', 'password', 'multipleStatements', 'pool', 'name', 'driverOptions'];
+        const props = [
+            'dbName',
+            'clientUrl',
+            'host',
+            'port',
+            'user',
+            'password',
+            'multipleStatements',
+            'pool',
+            'name',
+            'driverOptions',
+        ];
         for (const conf of replicas) {
             const replicaConfig = Utils.copy(conf);
             for (const prop of props) {

package/entity/Collection.js

@@ -22,7 +22,7 @@ export class Collection {
         if (items) {
             let i = 0;
             this.items = new Set(items);
-            this.items.forEach(item => this[i++] = item);
+            this.items.forEach(item => (this[i++] = item));
         }
         this.initialized = !!items || initialized;
     }
@@ -58,7 +58,7 @@ export class Collection {
         helper(this.owner).setSerializationContext({
             populate: Array.isArray(options.populate)
                 ? options.populate.map(hint => `${this.property.name}.${hint}`)
-                : options.populate ?? [this.property.name],
+                : (options.populate ?? [this.property.name]),
         });
     }
     /**
@@ -79,8 +79,10 @@ export class Collection {
             return this._count;
         }
         const em = this.getEntityManager();
-        if (!em.getPlatform().usesPivotTable() && this.property.kind === ReferenceKind.MANY_TO_MANY && this.property.owner) {
-            return this._count = this.length;
+        if (!em.getPlatform().usesPivotTable() &&
+            this.property.kind === ReferenceKind.MANY_TO_MANY &&
+            this.property.owner) {
+            return (this._count = this.length);
         }
         const cond = this.createLoadCountCondition(where ?? {});
         const count = await em.count(this.property.targetMeta.class, cond, countOptions);
@@ -96,16 +98,18 @@ export class Collection {
         if (this.property.kind === ReferenceKind.MANY_TO_MANY && em.getPlatform().usesPivotTable()) {
             // M:N via pivot table bypasses em.find(), so merge all 3 levels here
             opts.orderBy = QueryHelper.mergeOrderBy(opts.orderBy, this.property.orderBy, this.property.targetMeta?.orderBy);
-            options.populate = await em.preparePopulate(this.property.targetMeta.class, options);
-            const cond = await em.applyFilters(this.property.targetMeta.class, where, options.filters ?? {}, 'read');
-            const map = await em.getDriver().loadFromPivotTable(this.property, [helper(this.owner).__primaryKeys], cond, opts.orderBy, ctx, options);
+            options.populate = (await em.preparePopulate(this.property.targetMeta.class, options));
+            const cond = (await em.applyFilters(this.property.targetMeta.class, where, options.filters ?? {}, 'read'));
+            const map = await em
+                .getDriver()
+                .loadFromPivotTable(this.property, [helper(this.owner).__primaryKeys], cond, opts.orderBy, ctx, options);
             items = map[helper(this.owner).getSerializedPrimaryKey()].map((item) => em.merge(this.property.targetMeta.class, item, { convertCustomTypes: true }));
             await em.populate(items, options.populate, options);
         }
         else {
             // em.find() merges entity-level orderBy, so only merge runtime + relation here
             opts.orderBy = QueryHelper.mergeOrderBy(opts.orderBy, this.property.orderBy);
-            items = await em.find(this.property.targetMeta.class, this.createCondition(where), opts);
+            items = (await em.find(this.property.targetMeta.class, this.createCondition(where), opts));
         }
         if (options.store) {
             this.hydrate(items, true);
@@ -262,11 +266,9 @@ export class Collection {
             return this;
         }
         const populate = Array.isArray(options.populate)
-            ? options.populate.map(f => f === '*' ? f : `${this.property.name}.${f}`)
+            ? options.populate.map(f => (f === '*' ? f : `${this.property.name}.${f}`))
             : [`${this.property.name}${options.ref ? ':ref' : ''}`];
-        const schema = this.property.targetMeta.schema === '*'
-            ? helper(this.owner).__schema
-            : undefined;
+        const schema = this.property.targetMeta.schema === '*' ? helper(this.owner).__schema : undefined;
         await em.populate(this.owner, populate, {
             refresh: true,
             ...options,
@@ -297,7 +299,8 @@ export class Collection {
         if (this.property.kind === ReferenceKind.ONE_TO_MANY) {
             cond[this.property.mappedBy] = helper(this.owner).getPrimaryKey();
         }
-        else { // MANY_TO_MANY
+        else {
+            // MANY_TO_MANY
             this.createManyToManyCondition(cond);
         }
         return cond;
@@ -386,7 +389,9 @@ export class Collection {
         if (items.length === 0) {
             return [];
         }
-        field ??= targetMeta.compositePK ? targetMeta.primaryKeys : (targetMeta.serializedPrimaryKey ?? targetMeta.primaryKeys[0]);
+        field ??= targetMeta.compositePK
+            ? targetMeta.primaryKeys
+            : (targetMeta.serializedPrimaryKey ?? targetMeta.primaryKeys[0]);
         const cb = (i, f) => {
             if (Utils.isEntity(i[f], true)) {
                 return wrap(i[f], true).getPrimaryKey();
@@ -616,6 +621,7 @@ export class Collection {
      * @internal
      */
     get property() {
+        // cannot be typed to `EntityProperty<O, T>` as it causes issues in assignability of `Loaded` type
         if (!this._property) {
             const meta = wrap(this.owner, true).__meta;
             /* v8 ignore next */
@@ -630,6 +636,7 @@ export class Collection {
      * @internal
      */
     set property(prop) {
+        // cannot be typed to `EntityProperty<O, T>` as it causes issues in assignability of `Loaded` type
         this._property = prop;
     }
     propagate(item, method) {
@@ -696,7 +703,18 @@ export class Collection {
     /** @ignore */
     [Symbol.for('nodejs.util.inspect.custom')](depth = 2) {
         const object = { ...this };
-        const hidden = ['items', 'owner', '_property', '_count', 'snapshot', '_populated', '_lazyInitialized', '_em', 'readonly', 'partial'];
+        const hidden = [
+            'items',
+            'owner',
+            '_property',
+            '_count',
+            'snapshot',
+            '_populated',
+            '_lazyInitialized',
+            '_em',
+            'readonly',
+            'partial',
+        ];
         hidden.forEach(k => delete object[k]);
         const ret = inspect(object, { depth });
         const name = `${this.constructor.name}<${this.property?.type ?? 'unknown'}>`;
@@ -704,7 +722,15 @@ export class Collection {
     }
 }
 Object.defineProperties(Collection.prototype, {
-    $: { get() { return this; } },
-    get: { get() { return () => this; } },
+    $: {
+        get() {
+            return this;
+        },
+    },
+    get: {
+        get() {
+            return () => this;
+        },
+    },
     __collection: { value: true, enumerable: false, writable: false },
 });