rake-db 2.33.11 → 2.34.0

package/dist/index.mjs

@@ -518,14 +518,34 @@ const resetChangeTableData = () => {
 	changeTableData = newChangeTableData();
 };
 const addOrDropChanges = [];
+const standaloneCheckChanges = [];
 function add(item, options) {
 	consumeColumnName();
 	setName(this, item);
-	if (isNoForeignKeyChangeInput(item)) throw new Error("t.noForeignKey() is only supported in t.change(...)");
 	if (item instanceof Column) {
 		const result = addOrDrop("add", item, options);
-		if (result.type === "change") return result;
+		if (result.type === "change") {
+			result.name ??= getName(this);
+			return result;
+		}
+		addOrDropChanges.push(result);
+		return addOrDropChanges.length - 1;
+	}
+	if (isStandaloneAddOrDropInput(item)) {
+		const result = standaloneAddOrDropToChange("add", item);
+		result.name ??= getName(this);
+		addOrDropChanges.push(result);
+		return addOrDropChanges.length - 1;
+	}
+	if (isStandaloneCheckAddOrDropInput(item)) {
+		const result = standaloneAddOrDropToChange("add", item);
+		result.name ??= getName(this);
 		addOrDropChanges.push(result);
+		standaloneCheckChanges.push({
+			index: addOrDropChanges.length - 1,
+			type: "add",
+			item
+		});
 		return addOrDropChanges.length - 1;
 	}
 	for (const key in item) {
@@ -545,11 +565,30 @@ function add(item, options) {
 const drop = function(item, options) {
 	consumeColumnName();
 	setName(this, item);
-	if (isNoForeignKeyChangeInput(item)) throw new Error("t.noForeignKey() is only supported in t.change(...)");
 	if (item instanceof Column) {
 		const result = addOrDrop("drop", item, options);
-		if (result.type === "change") return result;
+		if (result.type === "change") {
+			result.name ??= getName(this);
+			return result;
+		}
+		addOrDropChanges.push(result);
+		return addOrDropChanges.length - 1;
+	}
+	if (isStandaloneAddOrDropInput(item)) {
+		const result = standaloneAddOrDropToChange("drop", item);
+		result.name ??= getName(this);
+		addOrDropChanges.push(result);
+		return addOrDropChanges.length - 1;
+	}
+	if (isStandaloneCheckAddOrDropInput(item)) {
+		const result = standaloneAddOrDropToChange("drop", item);
+		result.name ??= getName(this);
 		addOrDropChanges.push(result);
+		standaloneCheckChanges.push({
+			index: addOrDropChanges.length - 1,
+			type: "drop",
+			item
+		});
 		return addOrDropChanges.length - 1;
 	}
 	for (const key in item) {
@@ -589,22 +628,48 @@ const addOrDrop = (type, item, options) => {
 		dropMode: options?.dropMode
 	};
 };
+const standaloneAddOrDropToChange = (type, item) => {
+	const empty = columnTypeToColumnChange({
+		type: "change",
+		to: {}
+	});
+	const change = changeInputToColumnChange(item);
+	return {
+		type: "change",
+		from: type === "add" ? empty : change,
+		to: type === "add" ? change : empty
+	};
+};
 const isColumnForeignKeyChangeInput = (item) => {
 	return "columnForeignKey" in item;
 };
-const isNoForeignKeyChangeInput = (item) => {
-	return "noForeignKey" in item;
+const isColumnPrimaryKeyChangeInput = (item) => {
+	return "columnPrimaryKey" in item;
+};
+const isColumnIndexChangeInput = (item) => {
+	return "columnIndex" in item;
+};
+const isColumnExcludeChangeInput = (item) => {
+	return "columnExclude" in item;
+};
+const isStandaloneAddOrDropInput = (item) => {
+	return isColumnForeignKeyChangeInput(item) || isColumnPrimaryKeyChangeInput(item) || isColumnIndexChangeInput(item) || isColumnExcludeChangeInput(item);
+};
+const isStandaloneCheckAddOrDropInput = (item) => {
+	return isCheckConstraintItem(item);
 };
 const isCheckConstraintItem = (item) => {
 	return !!item.constraint?.check;
 };
-const changeInputToColumnChange = (item, opposite) => {
+const changeInputToColumnChange = (item) => {
 	if (item instanceof Column || "type" in item) return columnTypeToColumnChange(item);
+	if (isColumnPrimaryKeyChangeInput(item)) return {
+		primaryKey: true,
+		primaryKeyName: item.columnPrimaryKey.name
+	};
+	if (isColumnIndexChangeInput(item)) return { indexes: [item.columnIndex] };
+	if (isColumnExcludeChangeInput(item)) return { excludes: [item.columnExclude] };
 	if (isColumnForeignKeyChangeInput(item)) return { foreignKeys: [item.columnForeignKey] };
-	if (isNoForeignKeyChangeInput(item)) {
-		if (!isColumnForeignKeyChangeInput(opposite)) throw new Error("t.noForeignKey() in t.change(...) must be paired with t.foreignKey(...)");
-		return { foreignKeys: [] };
-	}
 	if (isCheckConstraintItem(item)) return { checks: [{
 		sql: item.constraint.check,
 		name: item.constraint.name
@@ -635,6 +700,9 @@ const columnTypeToColumnChange = (item, name) => {
 	return item.to;
 };
 const nameKey = Symbol("name");
+const getName = (self) => {
+	return self[nameKey];
+};
 const setName = (self, item) => {
 	const name = self[nameKey];
 	if (!name) return;
@@ -654,6 +722,47 @@ function foreignKey(...args) {
 		options
 	} };
 }
+function primaryKey(...args) {
+	if (Array.isArray(args[0])) {
+		const [columns, name] = args;
+		return tableDataMethods.primaryKey(columns, name);
+	}
+	const [name] = args;
+	return { columnPrimaryKey: { name } };
+}
+function index(...args) {
+	if (Array.isArray(args[0])) {
+		const [columns, first, second] = args;
+		if (typeof first === "string") return tableDataMethods.index(columns, first, second);
+		return tableDataMethods.index(columns, first);
+	}
+	const [options] = args;
+	return { columnIndex: { options: { ...options } } };
+}
+function unique(...args) {
+	if (Array.isArray(args[0])) {
+		const [columns, first, second] = args;
+		if (typeof first === "string") return tableDataMethods.unique(columns, first, second);
+		return tableDataMethods.unique(columns, first);
+	}
+	const [options] = args;
+	return { columnIndex: { options: {
+		...options,
+		unique: true
+	} } };
+}
+function exclude(...args) {
+	if (Array.isArray(args[0])) {
+		const [columns, first, second] = args;
+		if (typeof first === "string") return tableDataMethods.exclude(columns, first, second);
+		return tableDataMethods.exclude(columns, first);
+	}
+	const [with_, options] = args;
+	return { columnExclude: {
+		with: with_,
+		options: { ...options }
+	} };
+}
 const tableChangeMethods = {
 	...tableMethods,
 	...tableDataMethods,
@@ -665,14 +774,15 @@ const tableChangeMethods = {
 	},
 	add,
 	drop,
+	primaryKey,
+	index,
+	unique,
+	exclude,
 	foreignKey,
-	noForeignKey() {
-		return { noForeignKey: true };
-	},
 	change(from, to, using) {
 		consumeColumnName();
-		const f = changeInputToColumnChange(from, to);
-		const t = changeInputToColumnChange(to, from);
+		const f = changeInputToColumnChange(from);
+		const t = changeInputToColumnChange(to);
 		setName(this, f);
 		setName(this, t);
 		return {
@@ -722,6 +832,7 @@ const changeTable = async (migration, up, tableName, options, fn) => {
 	const tableChanger = Object.create(migration.columnTypes);
 	Object.assign(tableChanger, tableChangeMethods);
 	addOrDropChanges.length = 0;
+	standaloneCheckChanges.length = 0;
 	const changeData = fn?.(tableChanger) || {};
 	const schema = migration.adapter.getSchema();
 	const queries = astToQueries(schema, makeAst$3(schema, up, tableName, changeData, changeTableData, options), snakeCase, language);
@@ -736,6 +847,7 @@ const makeAst$3 = (schema, up, name, changeData, changeTableData, options) => {
 	const consumedChanges = {};
 	for (const key in changeData) {
 		let item = changeData[key];
+		if (item === void 0) continue;
 		if (typeof item === "number") {
 			consumedChanges[item] = true;
 			item = addOrDropChanges[item];
@@ -761,9 +873,15 @@ const makeAst$3 = (schema, up, name, changeData, changeTableData, options) => {
 			}
 		} : item;
 	}
+	for (const checkChange of standaloneCheckChanges) {
+		if (consumedChanges[checkChange.index]) continue;
+		parseTableDataInput(changeTableData[checkChange.type], checkChange.item);
+		consumedChanges[checkChange.index] = true;
+	}
 	for (let i = 0; i < addOrDropChanges.length; i++) {
 		if (consumedChanges[i]) continue;
 		const change = addOrDropChanges[i];
+		if (change.type === "change") throw new Error("Standalone helper add/drop changes must be assigned to a column key");
 		const name = change.item.data.name;
 		if (!name) throw new Error(`Column in ...t.${change.type}() must have a name`);
 		const arr = shape[name] ? toArray(shape[name]) : [];
@@ -859,6 +977,11 @@ const alterTableSql = (tableName, lines, values) => ({
   ${lines.join(",\n  ")}`,
 	values
 });
+const setPrimaryKeyName = (key, primaryKey, name) => {
+	if (!name) return;
+	if (primaryKey.name && primaryKey.name !== name) throw new Error(`Cannot use different primary key names in standalone changes for column ${key}`);
+	primaryKey.name = name;
+};
 const handlePrerequisitesForTableItem = (schema, key, item, queries, addPrimaryKeys, dropPrimaryKeys, snakeCase) => {
 	if ("item" in item) {
 		const { item: column } = item;
@@ -872,10 +995,12 @@ const handlePrerequisitesForTableItem = (schema, key, item, queries, addPrimaryK
 		if (item.from.column instanceof EnumColumn) queries.push(makePopulateEnumQuery(schema, item.from.column));
 		if (item.to.column instanceof EnumColumn) queries.push(makePopulateEnumQuery(schema, item.to.column));
 		if (item.from.primaryKey) {
+			setPrimaryKeyName(key, dropPrimaryKeys, item.from.primaryKeyName);
 			dropPrimaryKeys.columns.push(item.from.column ? getColumnName(item.from.column, key, snakeCase) : snakeCase ? toSnakeCase(key) : key);
 			dropPrimaryKeys.change = true;
 		}
 		if (item.to.primaryKey) {
+			setPrimaryKeyName(key, addPrimaryKeys, item.to.primaryKeyName);
 			addPrimaryKeys.columns.push(item.to.column ? getColumnName(item.to.column, key, snakeCase) : snakeCase ? toSnakeCase(key) : key);
 			addPrimaryKeys.change = true;
 		}
@@ -2135,7 +2260,7 @@ const wrapWithEnhancingError = async (text, values, promise) => {
 	try {
 		return await promise;
 	} catch (err) {
-		if (err && typeof err === "object" && "message" in err && typeof err.message === "string" && err.constructor.name === "PostgresError") {
+		if (err && typeof err === "object" && "message" in err && typeof err.message === "string" && (err.constructor.name === "PostgresError" || err.constructor.name === "DatabaseError")) {
 			err.message += `\nSQL: ${text}${values ? `\nVariables: ${JSON.stringify(values)}` : ""}`;
 			throw new err.constructor(err);
 		}
@@ -2609,13 +2734,11 @@ const getMaybeTransactionAdapter = (db) => "$getAdapter" in db ? db.$getAdapter(
 const runSqlInSavePoint = async (db, sql, code) => {
 	const adapter = getMaybeTransactionAdapter(db);
 	try {
-		await adapter.query(adapter.isInTransaction() ? `SAVEPOINT s; ${sql}; RELEASE SAVEPOINT s` : sql);
+		const query = () => adapter.query(sql);
+		await (adapter.isInTransaction() ? adapter.savepoint("s", query) : query());
 		return "done";
 	} catch (err) {
-		if (err.code === code) {
-			if (adapter.isInTransaction()) await adapter.query(`ROLLBACK TO SAVEPOINT s`);
-			return "already";
-		}
+		if (err.code === code) return "already";
 		throw err;
 	}
 };
@@ -2745,17 +2868,14 @@ const deleteMigratedVersion = async (adapter, version, name, config) => {
 var NoMigrationsTableError = class extends Error {};
 const getMigratedVersionsMap = async (ctx, adapter, config, renameTo) => {
 	const table = migrationsSchemaTableSql(adapter, config);
-	const inTransaction = "isInTransaction" in adapter && adapter.isInTransaction();
+	const queryVersion = () => adapter.arrays(`SELECT * FROM ${table} ORDER BY version`);
 	let result;
 	try {
-		if (inTransaction) await adapter.query(`SAVEPOINT check_migrations_table`);
-		result = await adapter.arrays(`SELECT * FROM ${table} ORDER BY version`);
-		if (inTransaction) await adapter.query(`RELEASE SAVEPOINT check_migrations_table`);
+		if (adapter.isInTransaction()) result = await adapter.savepoint("check_migrations_table", queryVersion);
+		else result = await queryVersion();
 	} catch (err) {
-		if (err.code === "42P01") {
-			if (inTransaction) await adapter.query(`ROLLBACK TO SAVEPOINT check_migrations_table`);
-			throw new NoMigrationsTableError();
-		} else throw err;
+		if (err.code === "42P01") throw new NoMigrationsTableError();
+		else throw err;
 	}
 	if (!result.fields[1]) {
 		const { migrations } = await getMigrations(ctx, config, true);