admindb 2.1.0 → 2.2.1

package/dist/db/manager.js

@@ -11,25 +11,17 @@ const postgres_1 = require("./postgres");
 const logger_1 = require("../utils/logger");
 const common_1 = require("../utils/common");
 const DB_EXTENSIONS = ['.db', '.sqlite', '.sqlite3'];
-/**
- * Manages multiple SQLite database files and PostgreSQL database connections.
- *
- * Databases come from three sources:
- * - a `connections` map — named PostgreSQL or SQLite connection strings (e.g. from JSON config),
- * - a `dir` — every SQLite file with a known extension is scanned in,
- * - an explicit `files[]` array — specific file locations or connection strings.
- */
 class DbManager {
     logger;
     dir;
     createDir;
     readonlyMode;
-    files = []; // targets (paths or URIs), registry order
-    idByPath = new Map(); // target -> id
-    pathById = new Map(); // id -> target
-    nameById = new Map(); // id -> display name
-    readonlyById = new Map(); // id -> readonly override
-    openDbs = new Map(); // id -> db
+    files = [];
+    idByPath = new Map();
+    pathById = new Map();
+    nameById = new Map();
+    readonlyById = new Map();
+    openDbs = new Map();
     constructor(options, logger) {
         const opts = typeof options === 'string' ? { dir: options } : (options ?? {});
         this.logger = logger ?? (0, logger_1.createLogger)('info', 'admindb');
@@ -47,7 +39,6 @@ class DbManager {
     get directory() {
         return this.dir ?? this.createDir;
     }
-    /** True when this manager opens every database read-only. */
     get isReadOnly() {
         return this.readonlyMode;
     }
@@ -85,8 +76,7 @@ class DbManager {
             const raw = String(f ?? '').trim();
             if (!raw)
                 continue;
-            const isPg = raw.startsWith('postgres://') || raw.startsWith('postgresql://');
-            if (isPg) {
+            if ((0, common_1.isPostgresConnectionString)(raw)) {
                 let name = 'postgres';
                 try {
                     const u = new URL(raw);
@@ -135,7 +125,6 @@ class DbManager {
         this.pathById.set(id, abs);
         this.nameById.set(id, base);
     }
-    /** List all managed databases (metadata only — does not open them). */
     list() {
         const entries = [];
         const seen = new Set();
@@ -143,22 +132,11 @@ class DbManager {
             if (seen.has(id))
                 continue;
             seen.add(id);
-            const isPg = target.startsWith('postgres://') || target.startsWith('postgresql://');
-            if (isPg) {
-                let sanitized = target;
-                try {
-                    const u = new URL(target);
-                    if (u.password)
-                        u.password = '****';
-                    sanitized = u.toString();
-                }
-                catch {
-                    sanitized = target.replace(/:([^:@]+)@/, ':****@');
-                }
+            if ((0, common_1.isPostgresConnectionString)(target)) {
                 entries.push({
                     id,
                     name: this.nameById.get(id) ?? id,
-                    path: sanitized,
+                    path: (0, common_1.sanitizeConnectionString)(target),
                     dialect: 'postgres',
                     size: 0,
                     modified: 'Connected',
@@ -174,7 +152,6 @@ class DbManager {
                     modified = st.mtime.toISOString();
                 }
                 catch {
-                    /* ignore */
                 }
                 entries.push({
                     id,
@@ -192,11 +169,9 @@ class DbManager {
     has(id) {
         return this.pathById.has(id);
     }
-    /** True if global manager is in readonly mode or this database is explicitly marked read-only. */
     isDbReadOnly(id) {
         return this.readonlyMode || (this.readonlyById.get(id) ?? false);
     }
-    /** Set or toggle read-only mode for a specific database id. */
     setReadonly(id, readonly) {
         if (this.readonlyMode && !readonly) {
             throw new Error('Manager is in global read-only mode — databases cannot be made writable.');
@@ -205,19 +180,16 @@ class DbManager {
         if (current === readonly)
             return;
         this.readonlyById.set(id, readonly);
-        // Close existing connection instance so it is re-opened with new readonly mode on next use
         const db = this.openDbs.get(id);
         if (db) {
             try {
                 db.close();
             }
             catch {
-                /* ignore */
             }
             this.openDbs.delete(id);
         }
     }
-    /** Open (and cache) a database by id; creates the file if it is new SQLite database. */
     open(id, readonlyOverride) {
         const target = this.pathById.get(id);
         if (!target)
@@ -229,14 +201,12 @@ class DbManager {
                 db.close();
             }
             catch {
-                /* ignore */
             }
             this.openDbs.delete(id);
             db = undefined;
         }
         if (!db) {
-            const isPg = target.startsWith('postgres://') || target.startsWith('postgresql://');
-            if (isPg) {
+            if ((0, common_1.isPostgresConnectionString)(target)) {
                 db = new postgres_1.PostgresDatabase(target, this.logger.child(`db:${id}`), { readonly: isRo });
             }
             else {
@@ -246,11 +216,6 @@ class DbManager {
         }
         return db;
     }
-    /**
-     * Register and open an existing database file by path, returning its id.
-     * Used by the "Open an existing database" file browser. Validates that the
-     * file is a readable SQLite database before keeping it registered.
-     */
     openFile(absPath, readonly) {
         const abs = node_path_1.default.resolve(String(absPath ?? '').trim());
         if (!abs)
@@ -271,10 +236,9 @@ class DbManager {
         if (readonly !== undefined)
             this.readonlyById.set(id, readonly);
         try {
-            this.open(id); // throws if the file is not a readable SQLite database
+            this.open(id);
         }
         catch (err) {
-            // Roll the registration back so a bad file doesn't linger in the list.
             this.pathById.delete(id);
             this.idByPath.delete(abs);
             this.nameById.delete(id);
@@ -287,7 +251,6 @@ class DbManager {
     get(id) {
         return this.openDbs.get(id);
     }
-    /** Count the tables in a database (opens it if necessary). */
     async countTables(id) {
         try {
             const db = this.open(id);
@@ -298,7 +261,6 @@ class DbManager {
             return 0;
         }
     }
-    /** Create a new (empty) SQLite database. Returns its id. */
     create(name) {
         if (this.readonlyMode)
             throw new Error('Read-only mode — creating databases is disabled.');
@@ -310,10 +272,9 @@ class DbManager {
         const id = this.idByPath.get(abs);
         if (!id)
             throw new Error('Failed to register new database.');
-        this.open(id); // better-sqlite3 creates the file on first open
+        this.open(id);
         return id;
     }
-    /** Dynamically register a named PostgreSQL or SQLite connection string. Returns its id. */
     addConnection(name, connectionString, readonly) {
         if (this.readonlyMode)
             throw new Error('Read-only mode — adding connections is disabled.');
@@ -338,7 +299,6 @@ class DbManager {
             this.readonlyById.set(id, readonly);
         return id;
     }
-    /** Close and delete a database file (including WAL/SHM sidecars). */
     remove(id) {
         if (this.readonlyMode)
             throw new Error('Read-only mode — deleting databases is disabled.');
@@ -351,7 +311,6 @@ class DbManager {
                 db.close();
             }
             catch {
-                /* ignore */
             }
             this.openDbs.delete(id);
         }
@@ -360,14 +319,12 @@ class DbManager {
         this.nameById.delete(id);
         this.readonlyById.delete(id);
         this.files = this.files.filter((p) => p !== target);
-        const isPg = target.startsWith('postgres://') || target.startsWith('postgresql://');
-        if (!isPg) {
+        if (!(0, common_1.isPostgresConnectionString)(target)) {
             for (const suffix of ['', '-wal', '-shm']) {
                 try {
                     (0, node_fs_1.unlinkSync)(target + suffix);
                 }
                 catch {
-                    /* ignore */
                 }
             }
         }
@@ -378,14 +335,12 @@ class DbManager {
                 db.close();
             }
             catch {
-                /* ignore */
             }
         }
         this.openDbs.clear();
     }
 }
 exports.DbManager = DbManager;
-/** Turn a friendly name into a safe database file name (always ends in `.db`). */
 function sanitizeFileName(name) {
     let n = String(name ?? '').trim().replace(/[\\/]/g, '');
     if (!n)
@@ -394,4 +349,3 @@ function sanitizeFileName(name) {
         n += '.db';
     return n;
 }
-//# sourceMappingURL=manager.js.map

package/dist/db/migrations.d.ts

@@ -1,5 +1 @@
-import type Database from 'better-sqlite3';
-import { type ColumnDef } from '../sql/generator';
-export declare function modifyTableStructureSync(db: Database.Database, table: string, transformColumns: (columns: ColumnDef[]) => ColumnDef[], renamedOldCol?: string, renamedNewCol?: string): {
-    changes: number;
-};
+import type Database from 'better-sqlite3';import{type ColumnDef} from '../sql/generator';export declare function modifyTableStructureSync(db:Database.Database,table:string,transformColumns:(columns:ColumnDef[])=>ColumnDef[],renamedOldCol?:string,renamedNewCol?:string):{changes:number;};

package/dist/db/migrations.js

@@ -8,7 +8,6 @@ function modifyTableStructureSync(db, table, transformColumns, renamedOldCol, re
         throw new Error(`Table "${table}" does not exist.`);
     const fks = db.prepare(`PRAGMA foreign_key_list(${(0, generator_1.quoteIdentifier)(table)})`).all();
     const indexRows = db.prepare(`PRAGMA index_list(${(0, generator_1.quoteIdentifier)(table)})`).all();
-    // Discover which single columns are UNIQUE across the whole table (so existing uniqueness is preserved)
     const uniqueCols = new Set();
     for (const ix of indexRows) {
         if (ix.unique) {
@@ -82,7 +81,6 @@ function modifyTableStructureSync(db, table, transformColumns, renamedOldCol, re
                 db.exec(`CREATE ${uq}INDEX IF NOT EXISTS ${(0, generator_1.quoteIdentifier)(ix.name)} ON ${(0, generator_1.quoteIdentifier)(table)} (${ixColsStr});`);
             }
             catch {
-                /* ignore index recreation error */
             }
         }
         const fkCheck = db.prepare('PRAGMA foreign_key_check;').all();
@@ -96,15 +94,14 @@ function modifyTableStructureSync(db, table, transformColumns, renamedOldCol, re
         try {
             db.exec('ROLLBACK;');
         }
-        catch { /* ignore */ }
+        catch { }
         try {
             db.exec(`DROP TABLE IF EXISTS ${(0, generator_1.quoteIdentifier)(tempTable)};`);
         }
-        catch { /* ignore */ }
+        catch { }
         throw err;
     }
     finally {
         db.exec('PRAGMA foreign_keys = ON;');
     }
 }
-//# sourceMappingURL=migrations.js.map

package/dist/db/postgres.d.ts

@@ -1,85 +1 @@
-import type { Logger } from '../utils/logger';
-import { type ColumnDef, type IndexDef } from '../sql/generator';
-import type { DatabaseDialect, IDatabase, MutationResult, PostgresOptions, QueryOptions, ReferencingTableInfo, Result, RowFilters, SavedQuery, SchemaInfo, SQLInputValue, TableInfoData, TableListItem, WhereClause } from './types';
-export declare class PostgresDatabase implements IDatabase {
-    private pool;
-    private logger;
-    readonly path: string;
-    readonly isReadOnly: boolean;
-    readonly dialect: DatabaseDialect;
-    readonly schema: string;
-    constructor(connection: string | PostgresOptions, logger: Logger, options?: PostgresOptions);
-    private sanitizeConnectionString;
-    private readonlyBlocked;
-    close(): Promise<void>;
-    private tryRun;
-    /** Run a callback inside an atomic transaction on a dedicated client. */
-    private transaction;
-    private initSchema;
-    hasMultipleStatements(sql: string): boolean;
-    all(sql: string, params?: SQLInputValue[]): Promise<Result<unknown[]>>;
-    run(sql: string, params?: SQLInputValue[]): Promise<Result<MutationResult>>;
-    execResult(sql: string): Promise<Result<{
-        changes?: number;
-    }>>;
-    runWrite(sql: string): Promise<Result<{
-        changes?: number;
-    }>>;
-    listTables(): Promise<Result<TableListItem[]>>;
-    getTableInfo(table: string): Promise<Result<TableInfoData>>;
-    getRowCount(table: string, filters?: RowFilters): Promise<Result<number>>;
-    getRows(table: string, opts?: QueryOptions): Promise<Result<Record<string, unknown>[]>>;
-    getAllRows(table: string): Promise<Result<Record<string, unknown>[]>>;
-    getRow(table: string, where: WhereClause[]): Promise<Result<Record<string, unknown> | null>>;
-    insertRow(table: string, fields: WhereClause[]): Promise<Result<MutationResult>>;
-    insertRows(table: string, rows: WhereClause[][]): Promise<Result<{
-        inserted: number;
-        skipped: number;
-    }>>;
-    updateRow(table: string, fields: WhereClause[], where: WhereClause[]): Promise<Result<MutationResult>>;
-    updateRows(table: string, rows: {
-        fields: WhereClause[];
-        where: WhereClause[];
-    }[]): Promise<Result<{
-        updated: number;
-    }>>;
-    deleteRow(table: string, where: WhereClause[]): Promise<Result<MutationResult>>;
-    deleteRows(table: string, rows: WhereClause[][]): Promise<Result<{
-        deleted: number;
-    }>>;
-    getRowsByPks(table: string, pks: WhereClause[][]): Promise<Result<Record<string, unknown>[]>>;
-    getRowsByFk(table: string, column: string, value: unknown, limit?: number): Promise<Result<{
-        rows: Record<string, unknown>[];
-        total: number;
-    }>>;
-    getCreateStatement(table: string): Promise<Result<string | null>>;
-    listSavedQueries(): Promise<Result<SavedQuery[]>>;
-    saveQuery(name: string, sql: string): Promise<Result<MutationResult>>;
-    deleteSavedQuery(id: string | number): Promise<Result<MutationResult>>;
-    getSchema(table: string): Promise<Result<SchemaInfo>>;
-    getReferencingTables(table: string): Promise<Result<ReferencingTableInfo[]>>;
-    renameTable(oldName: string, newName: string): Promise<Result<{
-        changes?: number;
-    }>>;
-    addColumn(table: string, col: ColumnDef): Promise<Result<{
-        changes?: number;
-    }>>;
-    modifyColumn(table: string, oldColName: string, newColDef: ColumnDef): Promise<Result<{
-        changes?: number;
-    }>>;
-    renameColumn(table: string, oldName: string, newName: string): Promise<Result<{
-        changes?: number;
-    }>>;
-    dropColumn(table: string, column: string): Promise<Result<{
-        changes?: number;
-    }>>;
-    dropTable(table: string): Promise<Result<{
-        changes?: number;
-    }>>;
-    createIndex(table: string, index: IndexDef): Promise<Result<{
-        changes?: number;
-    }>>;
-    dropIndex(indexName: string): Promise<Result<{
-        changes?: number;
-    }>>;
-}
+import type {Logger} from '../utils/logger';import{type ColumnDef,type IndexDef} from '../sql/generator';import type {DatabaseDialect,IDatabase,MutationResult,PostgresOptions,QueryOptions,ReferencingTableInfo,Result,RowFilters,SavedQuery,SchemaInfo,SQLInputValue,TableInfoData,TableListItem,WhereClause} from './types';export declare class PostgresDatabase implements IDatabase{private pool;private logger;readonly path:string;readonly isReadOnly:boolean;readonly dialect:DatabaseDialect;readonly schema:string;constructor(connection:string|PostgresOptions,logger:Logger,options?:PostgresOptions);private readonlyBlocked;close():Promise<void>;private tryRun;private transaction;private initSchema;hasMultipleStatements(sql:string):boolean;all(sql:string,params?:SQLInputValue[]):Promise<Result<unknown[]>>;run(sql:string,params?:SQLInputValue[]):Promise<Result<MutationResult>>;execResult(sql:string):Promise<Result<{changes?:number;}>>;runWrite(sql:string):Promise<Result<{changes?:number;}>>;listTables():Promise<Result<TableListItem[]>>;getTableInfo(table:string):Promise<Result<TableInfoData>>;getRowCount(table:string,filters?:RowFilters):Promise<Result<number>>;getRows(table:string,opts?:QueryOptions):Promise<Result<Record<string,unknown>[]>>;getAllRows(table:string):Promise<Result<Record<string,unknown>[]>>;getRow(table:string,where:WhereClause[]):Promise<Result<Record<string,unknown>|null>>;insertRow(table:string,fields:WhereClause[]):Promise<Result<MutationResult>>;insertRows(table:string,rows:WhereClause[][]):Promise<Result<{inserted:number;skipped:number;}>>;updateRow(table:string,fields:WhereClause[],where:WhereClause[]):Promise<Result<MutationResult>>;updateRows(table:string,rows:{fields:WhereClause[];where:WhereClause[];}[]):Promise<Result<{updated:number;}>>;deleteRow(table:string,where:WhereClause[]):Promise<Result<MutationResult>>;deleteRows(table:string,rows:WhereClause[][]):Promise<Result<{deleted:number;}>>;getRowsByPks(table:string,pks:WhereClause[][]):Promise<Result<Record<string,unknown>[]>>;getRowsByFk(table:string,column:string,value:unknown,limit?:number):Promise<Result<{rows:Record<string,unknown>[];total:number;}>>;getCreateStatement(table:string):Promise<Result<string|null>>;listSavedQueries():Promise<Result<SavedQuery[]>>;saveQuery(name:string,sql:string):Promise<Result<MutationResult>>;deleteSavedQuery(id:string|number):Promise<Result<MutationResult>>;getSettings():Promise<Result<Record<string,string>>>;getSchema(table:string):Promise<Result<SchemaInfo>>;getReferencingTables(table:string):Promise<Result<ReferencingTableInfo[]>>;renameTable(oldName:string,newName:string):Promise<Result<{changes?:number;}>>;addColumn(table:string,col:ColumnDef):Promise<Result<{changes?:number;}>>;modifyColumn(table:string,oldColName:string,newColDef:ColumnDef):Promise<Result<{changes?:number;}>>;renameColumn(table:string,oldName:string,newName:string):Promise<Result<{changes?:number;}>>;dropColumn(table:string,column:string):Promise<Result<{changes?:number;}>>;dropTable(table:string):Promise<Result<{changes?:number;}>>;createIndex(table:string,index:IndexDef):Promise<Result<{changes?:number;}>>;dropIndex(indexName:string):Promise<Result<{changes?:number;}>>;}

package/dist/db/postgres.js

@@ -31,15 +31,12 @@ class PostgresDatabase {
                     sslConfig = false;
                 }
                 else if (sslConfig === undefined) {
-                    // Auto-enable SSL for remote/cloud hosts (e.g. Neon, Supabase, Render, Railway, AWS RDS, etc.)
-                    // or whenever sslmode is specified or not explicitly local
                     if (sslMode === 'require' || sslMode === 'prefer' || sslMode === 'no-verify' || !isLocal) {
                         sslConfig = { rejectUnauthorized: false };
                     }
                 }
             }
             catch {
-                /* ignore URL parse error */
             }
         }
         else if (opts.host && sslConfig === undefined) {
@@ -63,7 +60,7 @@ class PostgresDatabase {
             keepAlive: true,
         };
         this.path = opts.connectionString
-            ? this.sanitizeConnectionString(opts.connectionString)
+            ? (0, common_1.sanitizeConnectionString)(opts.connectionString)
             : `${opts.user || 'postgres'}@${opts.host || 'localhost'}:${opts.port || 5432}/${opts.database || 'postgres'}`;
         this.pool = new pg_1.Pool(poolConfig);
         this.pool.on('error', (err) => {
@@ -76,17 +73,6 @@ class PostgresDatabase {
         }
         this.logger.info(`Initialized PostgreSQL pool for ${this.path}${this.isReadOnly ? ' (read-only)' : ''}`);
     }
-    sanitizeConnectionString(conn) {
-        try {
-            const url = new URL(conn);
-            if (url.password)
-                url.password = '****';
-            return url.toString();
-        }
-        catch {
-            return conn.replace(/:([^@]+)@/, ':****@');
-        }
-    }
     readonlyBlocked() {
         return { success: false, error: 'Database is open in read-only mode — write operations are disabled.' };
     }
@@ -109,7 +95,6 @@ class PostgresDatabase {
             return { success: false, error: msg };
         }
     }
-    /** Run a callback inside an atomic transaction on a dedicated client. */
     async transaction(fn) {
         if (this.isReadOnly)
             return this.readonlyBlocked();
@@ -125,7 +110,6 @@ class PostgresDatabase {
                 await client.query('ROLLBACK');
             }
             catch {
-                /* ignore */
             }
             const msg = (0, common_1.errorMessage)(err);
             this.logger.error(msg);
@@ -162,7 +146,6 @@ class PostgresDatabase {
                 inString = !inString;
             }
             else if (ch === ';' && !inString) {
-                // Only count if there are non-whitespace characters following
                 const rest = s.slice(i + 1).trim();
                 if (rest.length > 0)
                     count++;
@@ -222,9 +205,7 @@ class PostgresDatabase {
     }
     async getTableInfo(table) {
         return this.tryRun(async () => {
-            // Execute all 4 metadata queries concurrently in parallel
             const [colRes, pkRes, fkRes, idxRes] = await Promise.all([
-                // 1. Columns
                 this.pool.query(`SELECT
              ordinal_position AS cid,
              column_name AS name,
@@ -235,7 +216,6 @@ class PostgresDatabase {
            FROM information_schema.columns
            WHERE table_schema = $1 AND table_name = $2
            ORDER BY ordinal_position;`, [this.schema, table]),
-                // 2. Primary Keys
                 this.pool.query(`SELECT kcu.column_name
            FROM information_schema.table_constraints tc
            JOIN information_schema.key_column_usage kcu
@@ -245,7 +225,6 @@ class PostgresDatabase {
              AND tc.table_schema = $1
              AND tc.table_name = $2
            ORDER BY kcu.ordinal_position;`, [this.schema, table]),
-                // 3. Foreign Keys
                 this.pool.query(`SELECT
              row_number() OVER () AS id,
              kcu.position_in_unique_constraint AS seq,
@@ -267,7 +246,6 @@ class PostgresDatabase {
            WHERE tc.constraint_type = 'FOREIGN KEY'
              AND tc.table_schema = $1
              AND tc.table_name = $2;`, [this.schema, table]),
-                // 4. Indexes
                 this.pool.query(`SELECT
              i.relname AS name,
              ix.indisunique AS unique,
@@ -304,7 +282,6 @@ class PostgresDatabase {
                 columns: Array.isArray(ix.columns) ? ix.columns : [],
                 sql: ix.sql,
             }));
-            // Approximate DDL for display
             const colDefs = columns.map((c) => {
                 let def = `  ${(0, generator_1.quoteIdentifier)(c.name)} ${c.type}`;
                 if (c.notnull)
@@ -532,13 +509,32 @@ class PostgresDatabase {
             };
         });
     }
+    async getSettings() {
+        return this.tryRun(async () => {
+            const keys = ['server_version', 'max_connections', 'port', 'timezone', 'shared_buffers', 'work_mem'];
+            const settings = {};
+            try {
+                const res = await this.pool.query(`SELECT name, setting FROM pg_settings WHERE name = ANY($1::text[])`, [keys]);
+                for (const row of res.rows) {
+                    settings[row.name] = String(row.setting);
+                }
+            }
+            catch (err) {
+                try {
+                    const ver = await this.pool.query('SHOW server_version');
+                    settings['server_version'] = String(ver.rows[0]?.server_version);
+                }
+                catch { }
+            }
+            return settings;
+        });
+    }
     async getSchema(table) {
         return this.tryRun(async () => {
             const infoR = await this.getTableInfo(table);
             if (!infoR.success || !infoR.data)
                 throw new Error(infoR.error || `Table ${table} not found.`);
             const { columns, foreignKeys, indexes, primaryKey } = infoR.data;
-            // Find references in other tables that point to this table
             const refRes = await this.pool.query(`SELECT
            tc.table_name AS table,
            kcu.column_name AS "from",
@@ -661,21 +657,17 @@ class PostgresDatabase {
             return this.readonlyBlocked();
         return this.tryRun(async () => {
             const newName = String(newColDef.name || oldColName).trim();
-            // Rename column first if name changed
             if (oldColName !== newName) {
                 await this.pool.query(`ALTER TABLE ${(0, generator_1.quoteIdentifier)(table)} RENAME COLUMN ${(0, generator_1.quoteIdentifier)(oldColName)} TO ${(0, generator_1.quoteIdentifier)(newName)};`);
             }
             const colToModify = newName;
-            // Alter Type if provided
             if (newColDef.type) {
                 await this.pool.query(`ALTER TABLE ${(0, generator_1.quoteIdentifier)(table)} ALTER COLUMN ${(0, generator_1.quoteIdentifier)(colToModify)} TYPE ${newColDef.type} USING ${(0, generator_1.quoteIdentifier)(colToModify)}::${newColDef.type};`);
             }
-            // Alter NOT NULL
             if (newColDef.notNull !== undefined) {
                 const action = newColDef.notNull ? 'SET NOT NULL' : 'DROP NOT NULL';
                 await this.pool.query(`ALTER TABLE ${(0, generator_1.quoteIdentifier)(table)} ALTER COLUMN ${(0, generator_1.quoteIdentifier)(colToModify)} ${action};`);
             }
-            // Alter DEFAULT
             if (newColDef.defaultValue !== undefined) {
                 if (newColDef.defaultValue === null || newColDef.defaultValue === '') {
                     await this.pool.query(`ALTER TABLE ${(0, generator_1.quoteIdentifier)(table)} ALTER COLUMN ${(0, generator_1.quoteIdentifier)(colToModify)} DROP DEFAULT;`);
@@ -684,7 +676,6 @@ class PostgresDatabase {
                     await this.pool.query(`ALTER TABLE ${(0, generator_1.quoteIdentifier)(table)} ALTER COLUMN ${(0, generator_1.quoteIdentifier)(colToModify)} SET DEFAULT ${newColDef.defaultValue};`);
                 }
             }
-            // Handle unique index
             if (newColDef.unique !== undefined) {
                 const idxName = `idx_${table}_${colToModify}_unique`;
                 if (newColDef.unique) {
@@ -742,4 +733,3 @@ class PostgresDatabase {
     }
 }
 exports.PostgresDatabase = PostgresDatabase;
-//# sourceMappingURL=postgres.js.map