uql-orm 0.24.6 → 0.24.7

package/dist/dialect/pgLikeSqlDialect.js

@@ -1,4 +1,3 @@
-import { getMeta } from '../entity/index.js';
 import { QueryRaw, } from '../type/index.js';
 import { escapeSingleQuotes } from '../util/sqlLiteral.js';
 import { AbstractSqlDialect } from './abstractSqlDialect.js';
@@ -41,6 +40,8 @@ export class PgLikeSqlDialect extends AbstractSqlDialect {
     commitTransactionCommand = 'COMMIT';
     rollbackTransactionCommand = 'ROLLBACK';
     alterColumnStrategy = 'separate-clauses';
+    /** `$N` placeholders carry their own index, so the upsert's assignments need no scratch context. */
+    upsertUpdateBindsInPlace = true;
     insertIdSource = 'returning';
     maxBindValues = 65535;
     /**
@@ -115,19 +116,6 @@ export class PgLikeSqlDialect extends AbstractSqlDialect {
     placeholder(index) {
         return `$${index}`;
     }
-    /**
-     * The same statement as the base, binding the assignments into the main context instead of a second
-     * one: `$N` placeholders carry their own index, so the values need not be in statement order, and
-     * computing them first is what keeps `appendInsertValues`' `onInsert` fields out of the update set.
-     */
-    upsert(ctx, entity, conflictPaths, payload, extraReturning = '') {
-        const meta = getMeta(entity);
-        const update = this.getUpsertUpdateAssignments(ctx, meta, conflictPaths, payload, this.upsertExcluded);
-        const keys = this.getUpsertConflictPathsStr(meta, conflictPaths);
-        const onConflict = update ? `DO UPDATE SET ${update}` : 'DO NOTHING';
-        this.appendInsertValues(ctx, entity, payload);
-        ctx.append(` ON CONFLICT (${keys}) ${onConflict} ${this.returningId(entity)}${extraReturning}`);
-    }
     /**
      * `to_tsvector(...) @@ websearch_to_tsquery(...)`. `websearch_to_tsquery` takes free-form user input
      * (quoted phrases, `or`, `-negation`) and never raises a syntax error, unlike `to_tsquery`, which

package/dist/dialect/vectorCast.d.ts

@@ -30,3 +30,18 @@ export declare function resolveVectorCast(field: {
  * declaring `type: 'sparsevec'` still hands UQL the dense array its field type promises.
  */
 export declare function toSparsevecLiteral(values: readonly unknown[]): string;
+/**
+ * The inverse of the two literals above. pgvector hands a vector column back as **text**, so a read
+ * that did not parse it returned a string from a field whose declared type is `number[]`: invisible
+ * to the compiler, and invisible to any mocked test, because a mock returns the array the entity
+ * promises. It surfaces only as arithmetic quietly producing nonsense on real rows.
+ *
+ * Driven by `cast`, never by the shape of the text, so this is the exact mirror of the write side:
+ * a `sparsevec` column is read as `{1:1,3:2}/3` because that is what it was written as, and a dense
+ * one as `[1,2,3]`. Both return the dense array the field type promises, whichever width the column
+ * has. Sniffing the string instead would guess at a type the caller already knows.
+ *
+ * Returns `undefined` when the text does not match the column's own format, so a caller can keep the
+ * raw value rather than replace it with something invented.
+ */
+export declare function parseVectorLiteral(raw: string, cast: VectorCast): number[] | undefined;

package/dist/dialect/vectorCast.js

@@ -39,3 +39,61 @@ export function toSparsevecLiteral(values) {
         .join(',');
     return `{${pairs}}/${values.length}`;
 }
+/**
+ * The inverse of the two literals above. pgvector hands a vector column back as **text**, so a read
+ * that did not parse it returned a string from a field whose declared type is `number[]`: invisible
+ * to the compiler, and invisible to any mocked test, because a mock returns the array the entity
+ * promises. It surfaces only as arithmetic quietly producing nonsense on real rows.
+ *
+ * Driven by `cast`, never by the shape of the text, so this is the exact mirror of the write side:
+ * a `sparsevec` column is read as `{1:1,3:2}/3` because that is what it was written as, and a dense
+ * one as `[1,2,3]`. Both return the dense array the field type promises, whichever width the column
+ * has. Sniffing the string instead would guess at a type the caller already knows.
+ *
+ * Returns `undefined` when the text does not match the column's own format, so a caller can keep the
+ * raw value rather than replace it with something invented.
+ */
+export function parseVectorLiteral(raw, cast) {
+    const text = raw.trim();
+    return cast === 'sparsevec' ? parseSparse(text) : parseDense(text);
+}
+const SPARSE_LITERAL = /^\{(.*)\}\/(\d+)$/;
+/** `{1:1,3:2}/3` expanded to the dense array the field type promises, zeros included. */
+function parseSparse(text) {
+    const sparse = SPARSE_LITERAL.exec(text);
+    if (!sparse)
+        return undefined;
+    const dense = new Array(Number(sparse[2])).fill(0);
+    if (!sparse[1])
+        return dense;
+    for (const pair of sparse[1].split(',')) {
+        // Split into exactly two non-empty parts before converting: `Number('')` is 0, not NaN, so a
+        // truncated `{1:}/3` would otherwise decode to a confident zero instead of being refused.
+        const parts = pair.split(':');
+        if (parts.length !== 2 || !parts[0] || !parts[1])
+            return undefined;
+        const [index, value] = parts.map(Number);
+        if (!Number.isInteger(index) || index < 1 || index > dense.length || Number.isNaN(value))
+            return undefined;
+        dense[index - 1] = value;
+    }
+    return dense;
+}
+/**
+ * `[1,0,2]`, whatever width the column has.
+ *
+ * A dense literal is valid JSON by construction, so parsing it as JSON is both stricter and cheaper
+ * than splitting: `[1,,2]` throws here, where `split(',').map(Number)` would have turned the hole
+ * into a 0.
+ */
+function parseDense(text) {
+    if (!text.startsWith('[') || !text.endsWith(']'))
+        return undefined;
+    try {
+        const dense = JSON.parse(text);
+        return Array.isArray(dense) && dense.every((n) => typeof n === 'number') ? dense : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}

package/dist/entity/metadata/definition.d.ts

@@ -26,6 +26,5 @@ type MemberSpecs = {
 export declare function applyMembers<E>(entity: Type<E>, specs: MemberSpecs | undefined): void;
 export declare function defineEntity<E>(entity: Type<E>, opts?: EntityOptions<E>): EntityMeta<E>;
 export declare function getEntities(): Type<unknown>[];
-export declare function ensureMeta<E>(entity: Type<E>): EntityMeta<E>;
 export declare function getMeta<E>(entity: Type<E>): EntityMeta<E>;
 export {};

package/dist/entity/metadata/definition.js

@@ -163,7 +163,7 @@ export function getEntities() {
         return acc;
     }, []);
 }
-export function ensureMeta(entity) {
+function ensureMeta(entity) {
     let meta = metas.get(entity);
     if (meta) {
         return meta;

package/dist/maria/mariaDialect.d.ts

@@ -1,5 +1,5 @@
 import { MysqlLikeSqlDialect } from '../dialect/mysqlLikeSqlDialect.js';
-import type { DialectFeatures, FieldOptions, IndexFeature, IndexSchema, QueryConflictPaths, QueryContext, Type, VectorDistance } from '../type/index.js';
+import type { DialectFeatures, FieldOptions, IndexFeature, IndexSchema, QueryContext, Type, VectorDistance } from '../type/index.js';
 export declare class MariaDialect extends MysqlLikeSqlDialect {
     readonly dialectName = "mariadb";
     readonly insertIdSource = "returning";
@@ -11,7 +11,8 @@ export declare class MariaDialect extends MysqlLikeSqlDialect {
     protected readonly indexFeatures: Set<IndexFeature>;
     /** Unlike MySQL: `VECTOR(n)` takes its dimension, and its vector index is declared inline. */
     protected readonly featureOverrides: Partial<DialectFeatures>;
-    upsert<E>(ctx: QueryContext, entity: Type<E>, conflictPaths: QueryConflictPaths<E>, payload: E | E[]): void;
+    /** MariaDB 10.5+ supports `INSERT ... RETURNING`, so the ids are exact per row. */
+    protected upsertReturning<E>(entity: Type<E>): string;
     /**
      * MariaDB supports neither MySQL's `->`/`->>` shorthand nor the base's chained form. `JSON_VALUE`
      * reads a scalar and `JSON_EXTRACT` the subtree that the array operators need.

package/dist/maria/mariaDialect.js

@@ -1,7 +1,6 @@
 import { jsonPath } from '../dialect/jsonSql.js';
 import { MysqlLikeSqlDialect } from '../dialect/mysqlLikeSqlDialect.js';
 import { isVectorFieldType } from '../dialect/vectorCast.js';
-import { getMeta } from '../entity/index.js';
 export class MariaDialect extends MysqlLikeSqlDialect {
     dialectName = 'mariadb';
     // MariaDB 10.5+ supports `INSERT ... RETURNING` (see `insert` below), so IDs are exact per row.
@@ -17,23 +16,9 @@ export class MariaDialect extends MysqlLikeSqlDialect {
         vectorSupportsLength: true,
         inlineVectorIndex: true,
     };
-    upsert(ctx, entity, conflictPaths, payload) {
-        const meta = getMeta(entity);
-        const updateCtx = this.createContext();
-        const update = this.getUpsertUpdateAssignments(updateCtx, meta, conflictPaths, payload, (name) => `VALUES(${name})`);
-        const returning = this.returningId(entity);
-        if (update) {
-            this.appendInsertValues(ctx, entity, payload);
-            ctx.append(` ON DUPLICATE KEY UPDATE ${update} ${returning}`);
-            ctx.pushValue(...updateCtx.values);
-        }
-        else {
-            const insertCtx = this.createContext();
-            this.appendInsertValues(insertCtx, entity, payload);
-            ctx.append(insertCtx.sql.replace(/^INSERT/, 'INSERT IGNORE'));
-            ctx.append(' ' + returning);
-            ctx.pushValue(...insertCtx.values);
-        }
+    /** MariaDB 10.5+ supports `INSERT ... RETURNING`, so the ids are exact per row. */
+    upsertReturning(entity) {
+        return ` ${this.returningId(entity)}`;
     }
     /**
      * MariaDB supports neither MySQL's `->`/`->>` shorthand nor the base's chained form. `JSON_VALUE`

package/dist/maria/mariadbQuerierPool.js

@@ -7,7 +7,12 @@ export class MariadbQuerierPool extends AbstractSqlQuerierPool {
     pool;
     constructor(opts, extra) {
         super(new MariaDialect({ namingStrategy: extra?.namingStrategy }), extra);
-        this.pool = createPool(opts);
+        // `mariadb` defaults to handing BIGINT back as a BigInt, and uql maps `type: Number` to BIGINT
+        // (see `schema/canonicalType.ts`), so every auto-increment id reached a field declared `number`
+        // as `9n` without this. Same trade as the pg pools: exact to 2^53, and `...opts` wins for a
+        // caller who needs more. This belongs to the pool, not to the suites - it lived in
+        // `mariadbQuerier.test.ts`, which meant the tests passed on behaviour the library never shipped.
+        this.pool = createPool({ bigIntAsNumber: true, ...opts });
         // `mariadb`'s own `createPool` already attaches a silent no-op 'error'
         // listener (so a dropped connection can't crash the process), but its
         // `Pool` type only declares `on` for 'acquire' | 'connection' | 'enqueue'

package/dist/migrate/builder/migrationBuilder.d.ts

@@ -9,6 +9,15 @@ import type { ForeignKeyAction } from '../../schema/types.js';
 import type { IndexColumnInput, IndexOptions } from '../../type/index.js';
 import type { SqlQuerier } from '../../type/querier.js';
 import type { AnyMigrationOperation, IAlterTableBuilder, IColumnBuilder, IColumnFactory, IMigrationBuilder, ITableBuilder } from './types.js';
+type ForeignKeyTarget = {
+    table: string;
+    columns: string[];
+};
+type ForeignKeyOptions = {
+    name?: string;
+    onDelete?: ForeignKeyAction;
+    onUpdate?: ForeignKeyAction;
+};
 /**
  * Records migration operations without executing them.
  * Use for migration code generation and dry-run scenarios.
@@ -28,14 +37,7 @@ export declare class OperationRecorder implements IMigrationBuilder {
     renameColumn(tableName: string, oldName: string, newName: string): Promise<void>;
     createIndex(tableName: string, columns: readonly IndexColumnInput[], options?: IndexOptions): Promise<void>;
     dropIndex(tableName: string, indexName: string): Promise<void>;
-    addForeignKey(tableName: string, columns: string[], target: {
-        table: string;
-        columns: string[];
-    }, options?: {
-        name?: string;
-        onDelete?: ForeignKeyAction;
-        onUpdate?: ForeignKeyAction;
-    }): Promise<void>;
+    addForeignKey(tableName: string, columns: string[], target: ForeignKeyTarget, options?: ForeignKeyOptions): Promise<void>;
     dropForeignKey(tableName: string, constraintName: string): Promise<void>;
     raw(sql: string): Promise<void>;
     getOperations(): AnyMigrationOperation[];
@@ -74,16 +76,10 @@ export declare class MigrationBuilder extends OperationRecorder {
     renameColumn(tableName: string, oldName: string, newName: string): Promise<void>;
     createIndex(tableName: string, columns: readonly IndexColumnInput[], options?: IndexOptions): Promise<void>;
     dropIndex(tableName: string, indexName: string): Promise<void>;
-    addForeignKey(tableName: string, columns: string[], target: {
-        table: string;
-        columns: string[];
-    }, options?: {
-        name?: string;
-        onDelete?: ForeignKeyAction;
-        onUpdate?: ForeignKeyAction;
-    }): Promise<void>;
+    addForeignKey(tableName: string, columns: string[], target: ForeignKeyTarget, options?: ForeignKeyOptions): Promise<void>;
     dropForeignKey(tableName: string, constraintName: string): Promise<void>;
     private getCreateTableStatements;
     private execute;
     private operationToSql;
 }
+export {};

package/dist/migrate/builder/migrationBuilder.js

@@ -32,6 +32,27 @@ function createIndexOperation(tableName, columns, options = {}) {
         },
     };
 }
+/**
+ * The one shape of an `addForeignKey` operation, `NO ACTION` defaults included.
+ *
+ * Three callers build it and differ only in what they do with the result: the table builder records
+ * it through its parent, the recorder records it directly, and the executing builder also runs it.
+ * Spelled out three times, a changed default would have had to be found in all three.
+ */
+function addForeignKeyOperation(tableName, columns, target, options = {}) {
+    return {
+        type: 'addForeignKey',
+        tableName,
+        foreignKey: {
+            name: options.name,
+            columns,
+            referencesTable: target.table,
+            referencesColumns: target.columns,
+            onDelete: options.onDelete ?? 'NO ACTION',
+            onUpdate: options.onUpdate ?? 'NO ACTION',
+        },
+    };
+}
 /**
  * Declare one column through the same vocabulary `createTable` uses. A throwaway {@link TableBuilder}
  * is that vocabulary: `addColumn`/`alterColumn` used to take a bare {@link IColumnBuilder}, which
@@ -95,18 +116,7 @@ class AlterTableBuilder {
         return this;
     }
     addForeignKey(columns, target, options) {
-        this.parentBuilder.recordOperationSync({
-            type: 'addForeignKey',
-            tableName: this.tableName,
-            foreignKey: {
-                name: options?.name,
-                columns,
-                referencesTable: target.table,
-                referencesColumns: target.columns,
-                onDelete: options?.onDelete ?? 'NO ACTION',
-                onUpdate: options?.onUpdate ?? 'NO ACTION',
-            },
-        });
+        this.parentBuilder.recordOperationSync(addForeignKeyOperation(this.tableName, columns, target, options));
         return this;
     }
     dropForeignKey(name) {
@@ -124,9 +134,6 @@ class AlterTableBuilder {
  */
 export class OperationRecorder {
     operations = [];
-    // ============================================================================
-    // Table Operations
-    // ============================================================================
     async createTable(name, callback) {
         const builder = new TableBuilder(name);
         callback(builder);
@@ -154,9 +161,6 @@ export class OperationRecorder {
         const builder = new AlterTableBuilder(name, this);
         callback(builder);
     }
-    // ============================================================================
-    // Column Operations
-    // ============================================================================
     async addColumn(tableName, callback) {
         this.recordOperationSync({
             type: 'addColumn',
@@ -188,9 +192,6 @@ export class OperationRecorder {
             newName,
         });
     }
-    // ============================================================================
-    // Index Operations
-    // ============================================================================
     async createIndex(tableName, columns, options) {
         this.recordOperationSync(createIndexOperation(tableName, columns, options));
     }
@@ -201,22 +202,8 @@ export class OperationRecorder {
             indexName,
         });
     }
-    // ============================================================================
-    // Foreign Key Operations
-    // ============================================================================
     async addForeignKey(tableName, columns, target, options = {}) {
-        this.recordOperationSync({
-            type: 'addForeignKey',
-            tableName,
-            foreignKey: {
-                name: options.name,
-                columns,
-                referencesTable: target.table,
-                referencesColumns: target.columns,
-                onDelete: options.onDelete ?? 'NO ACTION',
-                onUpdate: options.onUpdate ?? 'NO ACTION',
-            },
-        });
+        this.recordOperationSync(addForeignKeyOperation(tableName, columns, target, options));
     }
     async dropForeignKey(tableName, constraintName) {
         this.recordOperationSync({
@@ -225,18 +212,12 @@ export class OperationRecorder {
             constraintName,
         });
     }
-    // ============================================================================
-    // Raw SQL
-    // ============================================================================
     async raw(sql) {
         this.recordOperationSync({
             type: 'raw',
             sql,
         });
     }
-    // ============================================================================
-    // Operation Access
-    // ============================================================================
     getOperations() {
         return [...this.operations];
     }
@@ -284,9 +265,7 @@ export class MigrationBuilder extends OperationRecorder {
         this.operations.push(operation);
         await this.querier.run(sql);
     }
-    // ============================================================================
     // Override async methods to execute immediately
-    // ============================================================================
     async createTable(name, callback) {
         const builder = new TableBuilder(name);
         callback(builder);
@@ -370,18 +349,7 @@ export class MigrationBuilder extends OperationRecorder {
         await this.execute(operation);
     }
     async addForeignKey(tableName, columns, target, options = {}) {
-        const operation = {
-            type: 'addForeignKey',
-            tableName,
-            foreignKey: {
-                name: options.name,
-                columns,
-                referencesTable: target.table,
-                referencesColumns: target.columns,
-                onDelete: options.onDelete ?? 'NO ACTION',
-                onUpdate: options.onUpdate ?? 'NO ACTION',
-            },
-        };
+        const operation = addForeignKeyOperation(tableName, columns, target, options);
         this.operations.push(operation);
         await this.execute(operation);
     }
@@ -394,9 +362,6 @@ export class MigrationBuilder extends OperationRecorder {
         this.operations.push(operation);
         await this.execute(operation);
     }
-    // ============================================================================
-    // Private Methods
-    // ============================================================================
     getCreateTableStatements(operation) {
         return this.sqlGenerator.generateCreateTableFromDefinition(operation.table);
     }

package/dist/migrate/builder/tableBuilder.js

@@ -65,9 +65,6 @@ export class TableBuilder {
     constructor(name) {
         this._name = name;
     }
-    // ============================================================================
-    // Numeric Types
-    // ============================================================================
     id(name = 'id', options = {}) {
         return this.add(name, { category: 'integer' }, { ...options, primaryKey: true, autoIncrement: true });
     }
@@ -138,9 +135,6 @@ export class TableBuilder {
         this._columnBuilders.push(column);
         return column;
     }
-    // ============================================================================
-    // Convenience Methods
-    // ============================================================================
     createdAt() {
         return this.add('createdAt', { category: 'timestamp' }, { defaultValue: t.now() });
     }
@@ -151,9 +145,6 @@ export class TableBuilder {
         this.createdAt();
         this.updatedAt();
     }
-    // ============================================================================
-    // Indexes & Constraints
-    // ============================================================================
     primaryKey(columns) {
         this._primaryKey = columns;
         return this;
@@ -186,9 +177,6 @@ export class TableBuilder {
         this._foreignKeyBuilders.push(fk);
         return fk;
     }
-    // ============================================================================
-    // Utilities
-    // ============================================================================
     comment(text) {
         this._comment = text;
         return this;

package/dist/migrate/cli.d.ts

@@ -17,4 +17,3 @@ export declare function runGenerateFromEntities(migrator: Migrator, args: string
 export declare function runSync(migrator: Migrator, args: string[], config: Partial<Config>): Promise<void>;
 export declare function runGenerateFromDb(migrator: Migrator, args: string[], config: Partial<Config>): Promise<void>;
 export declare function runDriftCheck(migrator: Migrator, config: Partial<Config>): Promise<void>;
-export declare function printHelp(): void;

package/dist/migrate/cli.js

@@ -309,7 +309,7 @@ function printDriftGroup(title, drifts, icon, showSuggestion) {
         console.log('');
     }
 }
-export function printHelp() {
+function printHelp() {
     console.log(`
 uql-orm/migrate - Database migration tool for uql ORM
 

package/dist/migrate/codegen/entityCodeGenerator.js

@@ -268,9 +268,6 @@ export class EntityCodeGenerator {
         }
         return lines.join('\n');
     }
-    // ============================================================================
-    // Helper Methods
-    // ============================================================================
     /**
      * Get decorator name for relation type.
      */

package/dist/migrate/drift/driftDetector.js

@@ -110,7 +110,11 @@ export class DriftDetector {
      * Add drifts for column alterations (type/nullable mismatches).
      */
     addAlterColumnDrifts(colDiff, drifts) {
-        if (this.options.checkTypes && colDiff.expected && colDiff.actual) {
+        // Every check below compares the two sides, so there is nothing to report without both.
+        if (!colDiff.expected || !colDiff.actual) {
+            return;
+        }
+        if (this.options.checkTypes) {
             const expectedType = this.formatType(colDiff.expected.type);
             const actualType = this.formatType(colDiff.actual.type);
             if (expectedType !== actualType) {
@@ -128,21 +132,19 @@ export class DriftDetector {
                 });
             }
         }
-        if (this.options.checkNullable && colDiff.expected && colDiff.actual) {
-            if (colDiff.expected.nullable !== colDiff.actual.nullable) {
-                drifts.push({
-                    type: 'constraint_mismatch',
-                    severity: 'warning',
-                    table: colDiff.table,
-                    column: colDiff.column,
-                    expected: colDiff.expected.nullable ? 'NULLABLE' : 'NOT NULL',
-                    actual: colDiff.actual.nullable ? 'NULLABLE' : 'NOT NULL',
-                    details: `Nullable mismatch for "${colDiff.column}"`,
-                    suggestion: 'Align nullable setting in entity or database',
-                });
-            }
+        if (this.options.checkNullable && colDiff.expected.nullable !== colDiff.actual.nullable) {
+            drifts.push({
+                type: 'constraint_mismatch',
+                severity: 'warning',
+                table: colDiff.table,
+                column: colDiff.column,
+                expected: colDiff.expected.nullable ? 'NULLABLE' : 'NOT NULL',
+                actual: colDiff.actual.nullable ? 'NULLABLE' : 'NOT NULL',
+                details: `Nullable mismatch for "${colDiff.column}"`,
+                suggestion: 'Align nullable setting in entity or database',
+            });
         }
-        if (this.options.checkDefaults && colDiff.expected && colDiff.actual) {
+        if (this.options.checkDefaults) {
             const expected = String(colDiff.expected.defaultValue ?? 'NULL');
             const actual = String(colDiff.actual.defaultValue ?? 'NULL');
             if (expected !== actual) {

package/dist/migrate/introspection/abstractSqlSchemaIntrospector.d.ts

@@ -77,8 +77,14 @@ export declare abstract class AbstractSqlSchemaIntrospector extends BaseSqlIntro
     protected abstract getForeignKeysQuery(tableName: string): string;
     /** SQL query to get primary key columns. Parameter: tableName (for PRAGMA-style). */
     protected abstract getPrimaryKeyQuery(tableName: string): string;
-    /** Extract table name from a row returned by getTableNamesQuery. */
-    protected abstract mapTableNameRow(row: RawRow): string;
+    /**
+     * Extract table name from a row returned by getTableNamesQuery.
+     *
+     * Defaults to `information_schema`'s own column, which is what every engine with an
+     * `information_schema` returns and what Postgres and MySQL both restated identically. SQLite reads
+     * `sqlite_master` instead and overrides.
+     */
+    protected mapTableNameRow(row: RawRow): string;
     /** Map column query results to ColumnSchema array. Allows async for SQLite's unique column check. */
     protected abstract mapColumnsResult(read: TableRowReader, tableName: string, results: RawRow[]): Promise<ColumnSchema[]>;
     /** Map index query results to IndexSchema array. Allows async for SQLite's index_info calls. */

package/dist/migrate/introspection/abstractSqlSchemaIntrospector.js

@@ -25,9 +25,6 @@ export class AbstractSqlSchemaIntrospector extends BaseSqlIntrospector {
         super(pool.dialect);
         this.pool = pool;
     }
-    // ============================================================================
-    // Template Methods (shared control flow)
-    // ============================================================================
     async getTableSchema(tableName) {
         return this.withSqlQuerier(async (querier) => {
             const read = createTableRowReader(querier);
@@ -91,9 +88,6 @@ export class AbstractSqlSchemaIntrospector extends BaseSqlIntrospector {
         const results = await read(this.getPrimaryKeyQuery(tableName), this.getPrimaryKeyParams(tableName));
         return this.mapPrimaryKeyResult(results);
     }
-    // ============================================================================
-    // Parameter Methods (can be overridden by dialects)
-    // ============================================================================
     tableExistsParams(tableName) {
         return [tableName];
     }
@@ -109,9 +103,6 @@ export class AbstractSqlSchemaIntrospector extends BaseSqlIntrospector {
     getPrimaryKeyParams(tableName) {
         return [tableName];
     }
-    // ============================================================================
-    // Shared Utilities
-    // ============================================================================
     /**
      * Normalize referential action string to standard type.
      */
@@ -138,6 +129,16 @@ export class AbstractSqlSchemaIntrospector extends BaseSqlIntrospector {
         }
         return Number(value);
     }
+    /**
+     * Extract table name from a row returned by getTableNamesQuery.
+     *
+     * Defaults to `information_schema`'s own column, which is what every engine with an
+     * `information_schema` returns and what Postgres and MySQL both restated identically. SQLite reads
+     * `sqlite_master` instead and overrides.
+     */
+    mapTableNameRow(row) {
+        return row['table_name'];
+    }
     /**
      * Map primary key query results to column names, in key order. `information_schema` gives every SQL
      * engine here a `column_name` per row; SQLite reads its key off `PRAGMA table_info` instead and

package/dist/migrate/introspection/mysqlIntrospector.d.ts

@@ -14,9 +14,6 @@ export declare class MysqlSchemaIntrospector extends AbstractSqlSchemaIntrospect
     protected getIndexesQuery(_tableName: string): string;
     protected getForeignKeysQuery(_tableName: string): string;
     protected getPrimaryKeyQuery(_tableName: string): string;
-    protected mapTableNameRow(row: {
-        table_name: string;
-    }): string;
     protected mapColumnsResult(_read: TableRowReader, _tableName: string, results: MysqlColumnRow[]): Promise<ColumnSchema[]>;
     protected mapIndexesResult(_read: TableRowReader, _tableName: string, results: {
         index_name: string;

package/dist/migrate/introspection/mysqlIntrospector.js

@@ -4,9 +4,6 @@ import { AbstractSqlSchemaIntrospector } from './abstractSqlSchemaIntrospector.j
  * Works with both MySQL and MariaDB as they share the same information_schema structure.
  */
 export class MysqlSchemaIntrospector extends AbstractSqlSchemaIntrospector {
-    // ============================================================================
-    // SQL Queries (dialect-specific)
-    // ============================================================================
     getTableNamesQuery() {
         return /*sql*/ `
       SELECT TABLE_NAME as table_name
@@ -95,12 +92,6 @@ export class MysqlSchemaIntrospector extends AbstractSqlSchemaIntrospector {
       ORDER BY ORDINAL_POSITION
     `;
     }
-    // ============================================================================
-    // Internal Types
-    // ============================================================================
-    mapTableNameRow(row) {
-        return row.table_name;
-    }
     async mapColumnsResult(_read, _tableName, results) {
         return results.map((row) => ({
             name: row.column_name,

package/dist/migrate/introspection/postgresIntrospector.d.ts

@@ -11,9 +11,6 @@ export declare class PostgresSchemaIntrospector extends AbstractSqlSchemaIntrosp
     protected getIndexesQuery(_tableName: string): string;
     protected getForeignKeysQuery(_tableName: string): string;
     protected getPrimaryKeyQuery(_tableName: string): string;
-    protected mapTableNameRow(row: {
-        table_name: string;
-    }): string;
     protected mapColumnsResult(_read: TableRowReader, _tableName: string, results: PostgresColumnRow[]): Promise<ColumnSchema[]>;
     protected mapIndexesResult(_read: TableRowReader, _tableName: string, results: {
         index_name: string;