@mikro-orm/sql 7.1.0-dev.34 → 7.1.0-dev.36

package/SqlEntityManager.d.ts

@@ -31,7 +31,28 @@ export declare class SqlEntityManager<Driver extends AbstractSqlDriver = Abstrac
      */
     qb<Entity extends object, RootAlias extends string = never>(entityName: EntityName<Entity>, alias?: RootAlias, type?: ConnectionType, loggerContext?: LoggingOptions): QueryBuilder<Entity, RootAlias>;
     /**
-     * Returns configured Kysely instance.
+     * Returns a configured Kysely instance bound to this EntityManager.
+     *
+     * When the EntityManager is inside a transaction (e.g. within `em.transactional(...)`, or after
+     * `em.begin()`), the returned Kysely instance automatically uses the transaction context, so any
+     * queries executed via Kysely's own `.execute()` / `.executeTakeFirst*()` participate in the
+     * current transaction.
+     *
+     * If you need a Kysely instance that is **not** bound to the current transaction (e.g. to perform
+     * a side query against the pool while inside a transactional block), fork the EntityManager first:
+     *
+     * ```ts
+     * await em.transactional(async em => {
+     *   // bound to the current transaction
+     *   await em.getKysely().selectFrom('user').selectAll().execute();
+     *
+     *   // bound to the pool, runs outside the transaction
+     *   await em.fork().getKysely().selectFrom('audit_log').selectAll().execute();
+     * });
+     * ```
+     *
+     * The `options.type` (`'read'` / `'write'`) is only honored outside a transaction — inside a
+     * transaction the connection is already pinned.
      */
     getKysely<TDB = undefined, TOptions extends GetKyselyOptions = GetKyselyOptions>(options?: TOptions): Kysely<TDB extends undefined ? InferKyselyDB<EntitiesFromManager<this>, TOptions> & InferClassEntityDB<AllEntitiesFromManager<this>, TOptions> : TDB>;
     /**

package/SqlEntityManager.js

@@ -20,10 +20,33 @@ export class SqlEntityManager extends EntityManager {
         return this.createQueryBuilder(entityName, alias, type, loggerContext);
     }
     /**
-     * Returns configured Kysely instance.
+     * Returns a configured Kysely instance bound to this EntityManager.
+     *
+     * When the EntityManager is inside a transaction (e.g. within `em.transactional(...)`, or after
+     * `em.begin()`), the returned Kysely instance automatically uses the transaction context, so any
+     * queries executed via Kysely's own `.execute()` / `.executeTakeFirst*()` participate in the
+     * current transaction.
+     *
+     * If you need a Kysely instance that is **not** bound to the current transaction (e.g. to perform
+     * a side query against the pool while inside a transactional block), fork the EntityManager first:
+     *
+     * ```ts
+     * await em.transactional(async em => {
+     *   // bound to the current transaction
+     *   await em.getKysely().selectFrom('user').selectAll().execute();
+     *
+     *   // bound to the pool, runs outside the transaction
+     *   await em.fork().getKysely().selectFrom('audit_log').selectAll().execute();
+     * });
+     * ```
+     *
+     * The `options.type` (`'read'` / `'write'`) is only honored outside a transaction — inside a
+     * transaction the connection is already pinned.
      */
     getKysely(options = {}) {
-        let kysely = this.getConnection(options.type).getClient();
+        const context = this.getContext(false);
+        const ctx = context.getTransactionContext();
+        let kysely = ctx ?? this.getConnection(options.type).getClient();
         if (options.columnNamingStrategy != null ||
             options.tableNamingStrategy != null ||
             options.processOnCreateHooks != null ||

package/dialects/mysql/BaseMySqlPlatform.d.ts

@@ -35,11 +35,9 @@ export declare class BaseMySqlPlatform extends AbstractSqlPlatform {
     getDefaultMappedType(type: string): Type<unknown>;
     isNumericColumn(mappedType: Type<unknown>): boolean;
     supportsUnsigned(): boolean;
-    /**
-     * Returns the default name of index for the given columns
-     * cannot go past 64 character length for identifiers in MySQL
-     */
-    getIndexName(tableName: string, columns: string[], type: 'index' | 'unique' | 'foreign' | 'primary' | 'sequence'): string;
+    /** MySQL/MariaDB identifier limit. */
+    getMaxIdentifierLength(): number;
+    getIndexName(tableName: string, columns: string[], type: 'index' | 'unique' | 'foreign' | 'primary' | 'sequence' | 'check'): string;
     getDefaultPrimaryName(tableName: string, columns: string[]): string;
     supportsCreatingFullTextIndex(): boolean;
     getFullTextWhereClause(): string;

package/dialects/mysql/BaseMySqlPlatform.js

@@ -1,4 +1,4 @@
-import { Utils, QueryOrder, DecimalType, DoubleType, } from '@mikro-orm/core';
+import { QueryOrder, DecimalType, DoubleType, } from '@mikro-orm/core';
 import { MySqlSchemaHelper } from './MySqlSchemaHelper.js';
 import { MySqlExceptionConverter } from './MySqlExceptionConverter.js';
 import { AbstractSqlPlatform } from '../../AbstractSqlPlatform.js';
@@ -86,19 +86,15 @@ export class BaseMySqlPlatform extends AbstractSqlPlatform {
     supportsUnsigned() {
         return true;
     }
-    /**
-     * Returns the default name of index for the given columns
-     * cannot go past 64 character length for identifiers in MySQL
-     */
+    /** MySQL/MariaDB identifier limit. */
+    getMaxIdentifierLength() {
+        return 64;
+    }
     getIndexName(tableName, columns, type) {
         if (type === 'primary') {
             return this.getDefaultPrimaryName(tableName, columns);
         }
-        const indexName = super.getIndexName(tableName, columns, type);
-        if (indexName.length > 64) {
-            return `${indexName.substring(0, 56 - type.length)}_${Utils.hash(indexName, 5)}_${type}`;
-        }
-        return indexName;
+        return super.getIndexName(tableName, columns, type);
     }
     getDefaultPrimaryName(tableName, columns) {
         return 'PRIMARY'; // https://dev.mysql.com/doc/refman/8.0/en/create-table.html#create-table-indexes-keys

package/dialects/postgresql/BasePostgreSqlPlatform.d.ts

@@ -100,11 +100,8 @@ export declare class BasePostgreSqlPlatform extends AbstractSqlPlatform {
     getDefaultMappedType(type: string): Type<unknown>;
     supportsSchemas(): boolean;
     getDefaultSchemaName(): string | undefined;
-    /**
-     * Returns the default name of index for the given columns
-     * cannot go past 63 character length for identifiers in MySQL
-     */
-    getIndexName(tableName: string, columns: string[], type: 'index' | 'unique' | 'foreign' | 'primary' | 'sequence'): string;
+    /** Postgres identifier limit (NAMEDATALEN - 1). */
+    getMaxIdentifierLength(): number;
     getDefaultPrimaryName(tableName: string, columns: string[]): string;
     /**
      * @inheritDoc

package/dialects/postgresql/BasePostgreSqlPlatform.js

@@ -383,17 +383,9 @@ export class BasePostgreSqlPlatform extends AbstractSqlPlatform {
     getDefaultSchemaName() {
         return 'public';
     }
-    /**
-     * Returns the default name of index for the given columns
-     * cannot go past 63 character length for identifiers in MySQL
-     */
-    getIndexName(tableName, columns, type) {
-        const indexName = super.getIndexName(tableName, columns, type);
-        if (indexName.length > 63) {
-            const suffix = type === 'primary' ? 'pkey' : type;
-            return `${indexName.substring(0, 55 - type.length)}_${Utils.hash(indexName, 5)}_${suffix}`;
-        }
-        return indexName;
+    /** Postgres identifier limit (NAMEDATALEN - 1). */
+    getMaxIdentifierLength() {
+        return 63;
     }
     getDefaultPrimaryName(tableName, columns) {
         const indexName = `${tableName}_pkey`;

package/dialects/sqlite/SqlitePlatform.d.ts

@@ -63,7 +63,7 @@ export declare class SqlitePlatform extends AbstractSqlPlatform {
      * including all Date properties, as we would be comparing Date object with timestamp.
      */
     processDateProperty(value: unknown): string | number | Date;
-    getIndexName(tableName: string, columns: string[], type: 'index' | 'unique' | 'foreign' | 'primary' | 'sequence'): string;
+    getIndexName(tableName: string, columns: string[], type: 'index' | 'unique' | 'foreign' | 'primary' | 'sequence' | 'check'): string;
     supportsDeferredUniqueConstraints(): boolean;
     /**
      * SQLite supports schemas via ATTACH DATABASE. Returns true when there are

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/sql",
-  "version": "7.1.0-dev.34",
+  "version": "7.1.0-dev.36",
   "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",
@@ -53,7 +53,7 @@
     "@mikro-orm/core": "^7.0.15"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.1.0-dev.34"
+    "@mikro-orm/core": "7.1.0-dev.36"
   },
   "engines": {
     "node": ">= 22.17.0"