@contember/schema-migrations 0.12.0-beta.2 → 1.0.0-alpha.0

package/src/modifications/relations/MakeRelationNullableModification.ts

@@ -1,6 +1,5 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
-import { ContentEvent } from '@contember/engine-common'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
 import { ModificationHandlerStatic } from '../ModificationHandler'
 import { getEntity, tryGetColumnName } from '@contember/schema-utils'
@@ -38,10 +37,6 @@ export const MakeRelationNullableModification: ModificationHandlerStatic<MakeRel
 		)
 	}
 
-	public transformEvents(events: ContentEvent[]): ContentEvent[] {
-		return events
-	}
-
 	describe() {
 		return {
 			message: `Make relation ${this.data.entityName}.${this.data.fieldName} nullable`,

package/src/modifications/relations/UpdateRelationOnDeleteModification.ts

@@ -1,6 +1,5 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
-import { ContentEvent } from '@contember/engine-common'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
 import { ModificationHandlerStatic } from '../ModificationHandler'
 import { isIt } from '../../utils/isIt'
@@ -25,10 +24,6 @@ export const UpdateRelationOnDeleteModification: ModificationHandlerStatic<Updat
 		)
 	}
 
-	public transformEvents(events: ContentEvent[]): ContentEvent[] {
-		return events
-	}
-
 	describe() {
 		return { message: `Change on-delete policy of relation ${this.data.entityName}.${this.data.fieldName}` }
 	}

package/src/modifications/relations/UpdateRelationOrderByModification.ts

@@ -1,6 +1,5 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
-import { ContentEvent } from '@contember/engine-common'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
 import { ModificationHandlerStatic } from '../ModificationHandler'
 import deepEqual from 'fast-deep-equal'
@@ -25,10 +24,6 @@ export const UpdateRelationOrderByModification: ModificationHandlerStatic<Update
 		)
 	}
 
-	public transformEvents(events: ContentEvent[]): ContentEvent[] {
-		return events
-	}
-
 	describe() {
 		return { message: `Update order-by of relation ${this.data.entityName}.${this.data.fieldName}` }
 	}

package/src/modifications/relations/index.ts

@@ -1,3 +1,4 @@
+export * from './ConvertOneToManyRelationModification'
 export * from './CreateRelationInverseSideModification'
 export * from './CreateRelationModification'
 export * from './DisableOrphanRemovalModification'

package/src/modifications/utils/schemaUpdateUtils.ts

@@ -7,12 +7,12 @@ import { VERSION_ACL_PATCH, VERSION_REMOVE_RELATION_INVERSE_SIDE } from '../Modi
 export type SchemaUpdater = (args: { schema: Schema }) => Schema
 export type ModelUpdater = (args: { schema: Schema; model: Model.Schema }) => Model.Schema
 export type EntityUpdater = (args: { schema: Schema; model: Model.Schema; entity: Model.Entity }) => Model.Entity
-export type FieldUpdater<T extends Model.AnyField> = (args: {
+export type FieldUpdater<In extends Model.AnyField, Out extends Model.AnyField> = (args: {
 	schema: Schema
 	model: Model.Schema
 	entity: Model.Entity
-	field: T
-}) => T | undefined
+	field: In
+}) => Out | undefined
 export type AclUpdater = (args: { schema: Schema; acl: Acl.Schema }) => Acl.Schema
 export type AclRoleUpdater = (args: {
 	schema: Schema
@@ -183,13 +183,13 @@ export const updateEveryEntity =
 		})
 
 export const updateField =
-	<T extends Model.AnyField = Model.AnyField>(name: string, fieldUpdater: FieldUpdater<T>): EntityUpdater =>
+	<In extends Model.AnyField = Model.AnyField, Out extends Model.AnyField = In>(name: string, fieldUpdater: FieldUpdater<In, Out>): EntityUpdater =>
 		args => {
 			const { [name]: field, ...otherFields } = args.entity.fields
 			if (!args.entity.fields[name]) {
 				throw new SchemaUpdateError(`Field ${args.entity.name}::${name} not found`)
 			}
-			const updatedField = fieldUpdater({ ...args, field: args.entity.fields[name] as T })
+			const updatedField = fieldUpdater({ ...args, field: args.entity.fields[name] as In })
 
 			return {
 				...args.entity,
@@ -201,7 +201,7 @@ export const updateField =
 		}
 
 export const updateEveryField =
-	(fieldUpdater: FieldUpdater<Model.AnyField>): EntityUpdater =>
+	(fieldUpdater: FieldUpdater<Model.AnyField, Model.AnyField>): EntityUpdater =>
 		args => ({
 			...args.entity,
 			fields: Object.fromEntries(

package/src/modifications/validation/PatchValidationSchemaModification.ts

@@ -1,5 +1,4 @@
 import { MigrationBuilder } from '@contember/database-migrations'
-import { ContentEvent } from '@contember/engine-common'
 import { SchemaUpdater } from '../utils/schemaUpdateUtils'
 import { ModificationHandlerStatic } from '../ModificationHandler'
 import { applyPatch, Operation } from 'rfc6902'
@@ -26,10 +25,6 @@ export const PatchValidationSchemaModification: ModificationHandlerStatic<PatchV
 		}
 	}
 
-	public transformEvents(events: ContentEvent[]): ContentEvent[] {
-		return events
-	}
-
 	describe() {
 		return { message: 'Update validation schema' }
 	}

package/src/modifications/validation/UpdateValidationSchemaModification.ts

@@ -1,11 +1,9 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Schema, Validation } from '@contember/schema'
-import { ContentEvent } from '@contember/engine-common'
 import { SchemaUpdater } from '../utils/schemaUpdateUtils'
 import { ModificationHandlerStatic } from '../ModificationHandler'
 import deepEqual from 'fast-deep-equal'
 import { createPatch } from 'rfc6902'
-import { PatchAclSchemaModification } from '../acl'
 import { PatchValidationSchemaModification } from './PatchValidationSchemaModification'
 
 export const UpdateValidationSchemaModification: ModificationHandlerStatic<UpdateValidationSchemaModificationData> = class {
@@ -21,10 +19,6 @@ export const UpdateValidationSchemaModification: ModificationHandlerStatic<Updat
 		})
 	}
 
-	public transformEvents(events: ContentEvent[]): ContentEvent[] {
-		return events
-	}
-
 	describe() {
 		return { message: 'Update validation schema' }
 	}

package/tests/cases/integration/convertOneToManyRelation.test.ts

@@ -0,0 +1,59 @@
+import { testMigrations } from '../../src/tests'
+import { SchemaBuilder } from '@contember/schema-definition'
+import { Model } from '@contember/schema'
+import { SQL } from '../../src/tags'
+
+testMigrations('convert one has one to many has one relation without inverse side', {
+	originalSchema: new SchemaBuilder()
+		.entity('Post', entity =>
+			entity.oneHasOne('image', rel => rel.target('Image')),
+		)
+		.entity('Image', e => e.column('url', c => c.type(Model.ColumnType.String)))
+		.buildSchema(),
+	updatedSchema: new SchemaBuilder()
+		.entity('Post', entity =>
+			entity.manyHasOne('image', rel => rel.target('Image')),
+		)
+		.entity('Image', e => e.column('url', c => c.type(Model.ColumnType.String)))
+		.buildSchema(),
+	diff: [
+		{
+			modification: 'convertOneToManyRelation',
+			entityName: 'Post',
+			fieldName: 'image',
+		},
+	],
+	sql: SQL`
+		CREATE INDEX "post_image_id_index" ON "post" ("image_id");
+		ALTER TABLE "post" DROP CONSTRAINT "unique_Post_image_a85d6d";
+	`,
+})
+
+
+testMigrations('convert one has one to many has one relation with inverse side', {
+	originalSchema: new SchemaBuilder()
+		.entity('Post', entity =>
+			entity.oneHasOne('image', rel => rel.target('Image').inversedBy('post')),
+		)
+		.entity('Image', e => e.column('url', c => c.type(Model.ColumnType.String)))
+		.buildSchema(),
+	updatedSchema: new SchemaBuilder()
+		.entity('Post', entity =>
+			entity.manyHasOne('image', rel => rel.target('Image').inversedBy('posts')),
+		)
+		.entity('Image', e => e.column('url', c => c.type(Model.ColumnType.String)))
+		.buildSchema(),
+	diff: [
+		{
+			modification: 'convertOneToManyRelation',
+			entityName: 'Post',
+			fieldName: 'image',
+			newInverseSideFieldName: 'posts',
+		},
+	],
+	sql: SQL`
+		CREATE INDEX "post_image_id_index" ON "post" ("image_id");
+		ALTER TABLE "post" DROP CONSTRAINT "unique_Post_image_a85d6d";
+	`,
+})
+

package/tests/cases/integration/createOneHasOne.test.ts

@@ -132,7 +132,8 @@ testMigrations('create one has one relation (site with settings)', {
 	ALTER TABLE "site"
 		ADD "setting_id" uuid;
 	ALTER TABLE "site"
-		ADD CONSTRAINT "unique_Site_setting_8653a0" UNIQUE ("setting_id");
+		ADD CONSTRAINT "fk_site_setting_id_6a4aa6" FOREIGN KEY ("setting_id") REFERENCES "site_setting"("id") ON DELETE NO ACTION DEFERRABLE INITIALLY IMMEDIATE;
 	ALTER TABLE "site"
-		ADD CONSTRAINT "fk_site_setting_id_6a4aa6" FOREIGN KEY ("setting_id") REFERENCES "site_setting"("id") ON DELETE NO ACTION DEFERRABLE INITIALLY IMMEDIATE;`,
+		ADD CONSTRAINT "unique_Site_setting_8653a0" UNIQUE ("setting_id");
+		`,
 })

package/tests/cases/integration/removeEntity.test.ts

@@ -22,7 +22,7 @@ testMigrations('remove an entity', {
 			entityName: 'Author',
 		},
 	],
-	sql: SQL`DROP TABLE "author" CASCADE;`,
+	sql: SQL`ALTER TABLE "post" DROP "author_id"; DROP TABLE "author";`,
 })
 
 testMigrations('remove entity with acl', {
@@ -90,7 +90,7 @@ testMigrations('remove entity with acl', {
 			entityName: 'Site',
 		},
 	],
-	sql: SQL`DROP TABLE "site" CASCADE;`,
+	sql: SQL`ALTER TABLE "post" DROP "site_id"; DROP TABLE "site";`,
 })
 
 namespace ViewEntityOriginalSchema {

package/tests/cases/integration/updateView.test.ts

@@ -40,5 +40,5 @@ testMigrations('update a view', {
 			},
 		},
 	],
-	sql: SQL`CREATE OR REPLACE VIEW "author" AS SELECT null as id, 'John' AS name, 'root@localhost' as email;`,
+	sql: SQL`DROP VIEW IF EXISTS "author"; CREATE VIEW "author" AS SELECT null as id, 'John' AS name, 'root@localhost' as email;`,
 })

package/dist/src/modifications/CreateModificationFieldVisitor.d.ts

@@ -1,14 +0,0 @@
-import { Model } from '@contember/schema';
-import { CreateColumnModificationData } from './columns';
-import { CreateRelationInverseSideModificationData, CreateRelationModificationData } from './relations';
-import { Migration } from '../Migration';
-export default class CreateFieldVisitor implements Model.ColumnVisitor<Migration.Modification<any>>, Model.RelationByTypeVisitor<Migration.Modification<any>> {
-    visitColumn(entity: Model.Entity, updatedColumn: Model.AnyColumn): Migration.Modification<CreateColumnModificationData>;
-    visitManyHasOne(entity: Model.Entity, relation: Model.ManyHasOneRelation, {}: {}, targetRelation: Model.OneHasManyRelation | null): Migration.Modification<CreateRelationModificationData>;
-    visitOneHasMany(entity: Model.Entity, relation: Model.OneHasManyRelation): Migration.Modification<CreateRelationInverseSideModificationData>;
-    visitOneHasOneOwning(entity: Model.Entity, relation: Model.OneHasOneOwningRelation, {}: {}, targetRelation: Model.OneHasOneInverseRelation | null): Migration.Modification<CreateRelationModificationData>;
-    visitOneHasOneInverse(entity: Model.Entity, relation: Model.OneHasOneInverseRelation): Migration.Modification<CreateRelationInverseSideModificationData>;
-    visitManyHasManyOwning(entity: Model.Entity, relation: Model.ManyHasManyOwningRelation, {}: {}, targetRelation: Model.ManyHasManyInverseRelation | null): Migration.Modification<CreateRelationModificationData>;
-    visitManyHasManyInverse(entity: Model.Entity, relation: Model.ManyHasManyInverseRelation): Migration.Modification<CreateRelationInverseSideModificationData>;
-}
-//# sourceMappingURL=CreateModificationFieldVisitor.d.ts.map

package/dist/src/modifications/CreateModificationFieldVisitor.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"CreateModificationFieldVisitor.d.ts","sourceRoot":"","sources":["../../../src/modifications/CreateModificationFieldVisitor.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAA;AACzC,OAAO,EAA4B,4BAA4B,EAAE,MAAM,WAAW,CAAA;AAClF,OAAO,EAEN,yCAAyC,EAEzC,8BAA8B,EAC9B,MAAM,aAAa,CAAA;AACpB,OAAO,EAAE,SAAS,EAAE,MAAM,cAAc,CAAA;AAGxC,MAAM,CAAC,OAAO,OAAO,kBACrB,YAAW,KAAK,CAAC,aAAa,CAAC,SAAS,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,EAAE,KAAK,CAAC,qBAAqB,CAAC,SAAS,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC;IAC7G,WAAW,CACjB,MAAM,EAAE,KAAK,CAAC,MAAM,EACpB,aAAa,EAAE,KAAK,CAAC,SAAS,GAC5B,SAAS,CAAC,YAAY,CAAC,4BAA4B,CAAC;IAWhD,eAAe,CACrB,MAAM,EAAE,KAAK,CAAC,MAAM,EACpB,QAAQ,EAAE,KAAK,CAAC,kBAAkB,EAClC,EAAE,IAAA,EACF,cAAc,EAAE,KAAK,CAAC,kBAAkB,GAAG,IAAI,GAC7C,SAAS,CAAC,YAAY,CAAC,8BAA8B,CAAC;IAQlD,eAAe,CACrB,MAAM,EAAE,KAAK,CAAC,MAAM,EACpB,QAAQ,EAAE,KAAK,CAAC,kBAAkB,GAChC,SAAS,CAAC,YAAY,CAAC,yCAAyC,CAAC;IAO7D,oBAAoB,CAC1B,MAAM,EAAE,KAAK,CAAC,MAAM,EACpB,QAAQ,EAAE,KAAK,CAAC,uBAAuB,EACvC,EAAE,IAAA,EACF,cAAc,EAAE,KAAK,CAAC,wBAAwB,GAAG,IAAI,GACnD,SAAS,CAAC,YAAY,CAAC,8BAA8B,CAAC;IAQlD,qBAAqB,CAC3B,MAAM,EAAE,KAAK,CAAC,MAAM,EACpB,QAAQ,EAAE,KAAK,CAAC,wBAAwB,GACtC,SAAS,CAAC,YAAY,CAAC,yCAAyC,CAAC;IAO7D,sBAAsB,CAC5B,MAAM,EAAE,KAAK,CAAC,MAAM,EACpB,QAAQ,EAAE,KAAK,CAAC,yBAAyB,EACzC,EAAE,IAAA,EACF,cAAc,EAAE,KAAK,CAAC,0BAA0B,GAAG,IAAI,GACrD,SAAS,CAAC,YAAY,CAAC,8BAA8B,CAAC;IAQlD,uBAAuB,CAC7B,MAAM,EAAE,KAAK,CAAC,MAAM,EACpB,QAAQ,EAAE,KAAK,CAAC,0BAA0B,GACxC,SAAS,CAAC,YAAY,CAAC,yCAAyC,CAAC;CAMpE"}

package/dist/src/modifications/CreateModificationFieldVisitor.js

@@ -1,61 +0,0 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-const columns_1 = require("./columns");
-const relations_1 = require("./relations");
-const deepCopy_1 = __importDefault(require("../utils/deepCopy"));
-class CreateFieldVisitor {
-    visitColumn(entity, updatedColumn) {
-        const createColumn = columns_1.CreateColumnModification.createModification({
-            entityName: entity.name,
-            field: deepCopy_1.default(updatedColumn),
-        });
-        if (updatedColumn.default !== undefined) {
-            createColumn.fillValue = updatedColumn.default;
-        }
-        return createColumn;
-    }
-    visitManyHasOne(entity, relation, {}, targetRelation) {
-        return relations_1.CreateRelationModification.createModification({
-            entityName: entity.name,
-            owningSide: relation,
-            ...(targetRelation ? { inverseSide: targetRelation } : {}),
-        });
-    }
-    visitOneHasMany(entity, relation) {
-        return relations_1.CreateRelationInverseSideModification.createModification({
-            entityName: entity.name,
-            relation: relation,
-        });
-    }
-    visitOneHasOneOwning(entity, relation, {}, targetRelation) {
-        return relations_1.CreateRelationModification.createModification({
-            entityName: entity.name,
-            owningSide: relation,
-            ...(targetRelation ? { inverseSide: targetRelation } : {}),
-        });
-    }
-    visitOneHasOneInverse(entity, relation) {
-        return relations_1.CreateRelationInverseSideModification.createModification({
-            entityName: entity.name,
-            relation: relation,
-        });
-    }
-    visitManyHasManyOwning(entity, relation, {}, targetRelation) {
-        return relations_1.CreateRelationModification.createModification({
-            entityName: entity.name,
-            owningSide: relation,
-            ...(targetRelation ? { inverseSide: targetRelation } : {}),
-        });
-    }
-    visitManyHasManyInverse(entity, relation) {
-        return relations_1.CreateRelationInverseSideModification.createModification({
-            entityName: entity.name,
-            relation: relation,
-        });
-    }
-}
-exports.default = CreateFieldVisitor;
-//# sourceMappingURL=CreateModificationFieldVisitor.js.map

package/dist/src/modifications/CreateModificationFieldVisitor.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"CreateModificationFieldVisitor.js","sourceRoot":"","sources":["../../../src/modifications/CreateModificationFieldVisitor.ts"],"names":[],"mappings":";;;;;AACA,uCAAkF;AAClF,2CAKoB;AAEpB,iEAAwC;AAExC,MAAqB,kBAAkB;IAE/B,WAAW,CACjB,MAAoB,EACpB,aAA8B;QAE9B,MAAM,YAAY,GAAG,kCAAwB,CAAC,kBAAkB,CAAC;YAChE,UAAU,EAAE,MAAM,CAAC,IAAI;YACvB,KAAK,EAAE,kBAAQ,CAAC,aAAa,CAAC;SAC9B,CAAC,CAAA;QACF,IAAI,aAAa,CAAC,OAAO,KAAK,SAAS,EAAE;YACxC,YAAY,CAAC,SAAS,GAAG,aAAa,CAAC,OAAO,CAAA;SAC9C;QACD,OAAO,YAAY,CAAA;IACpB,CAAC;IAEM,eAAe,CACrB,MAAoB,EACpB,QAAkC,EAClC,EAAE,EACF,cAA+C;QAE/C,OAAO,sCAA0B,CAAC,kBAAkB,CAAC;YACpD,UAAU,EAAE,MAAM,CAAC,IAAI;YACvB,UAAU,EAAE,QAAQ;YACpB,GAAG,CAAC,cAAc,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,cAAc,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;SAC1D,CAAC,CAAA;IACH,CAAC;IAEM,eAAe,CACrB,MAAoB,EACpB,QAAkC;QAElC,OAAO,iDAAqC,CAAC,kBAAkB,CAAC;YAC/D,UAAU,EAAE,MAAM,CAAC,IAAI;YACvB,QAAQ,EAAE,QAAQ;SAClB,CAAC,CAAA;IACH,CAAC;IAEM,oBAAoB,CAC1B,MAAoB,EACpB,QAAuC,EACvC,EAAE,EACF,cAAqD;QAErD,OAAO,sCAA0B,CAAC,kBAAkB,CAAC;YACpD,UAAU,EAAE,MAAM,CAAC,IAAI;YACvB,UAAU,EAAE,QAAQ;YACpB,GAAG,CAAC,cAAc,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,cAAc,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;SAC1D,CAAC,CAAA;IACH,CAAC;IAEM,qBAAqB,CAC3B,MAAoB,EACpB,QAAwC;QAExC,OAAO,iDAAqC,CAAC,kBAAkB,CAAC;YAC/D,UAAU,EAAE,MAAM,CAAC,IAAI;YACvB,QAAQ,EAAE,QAAQ;SAClB,CAAC,CAAA;IACH,CAAC;IAEM,sBAAsB,CAC5B,MAAoB,EACpB,QAAyC,EACzC,EAAE,EACF,cAAuD;QAEvD,OAAO,sCAA0B,CAAC,kBAAkB,CAAC;YACpD,UAAU,EAAE,MAAM,CAAC,IAAI;YACvB,UAAU,EAAE,QAAQ;YACpB,GAAG,CAAC,cAAc,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,cAAc,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;SAC1D,CAAC,CAAA;IACH,CAAC;IAEM,uBAAuB,CAC7B,MAAoB,EACpB,QAA0C;QAE1C,OAAO,iDAAqC,CAAC,kBAAkB,CAAC;YAC/D,UAAU,EAAE,MAAM,CAAC,IAAI;YACvB,QAAQ,EAAE,QAAQ;SAClB,CAAC,CAAA;IACH,CAAC;CACD;AApFD,qCAoFC"}

package/src/modifications/CreateModificationFieldVisitor.ts

@@ -1,96 +0,0 @@
-import { Model } from '@contember/schema'
-import { CreateColumnModification, CreateColumnModificationData } from './columns'
-import {
-	CreateRelationInverseSideModification,
-	CreateRelationInverseSideModificationData,
-	CreateRelationModification,
-	CreateRelationModificationData,
-} from './relations'
-import { Migration } from '../Migration'
-import deepCopy from '../utils/deepCopy'
-
-export default class CreateFieldVisitor
-implements Model.ColumnVisitor<Migration.Modification<any>>, Model.RelationByTypeVisitor<Migration.Modification<any>> {
-	public visitColumn(
-		entity: Model.Entity,
-		updatedColumn: Model.AnyColumn,
-	): Migration.Modification<CreateColumnModificationData> {
-		const createColumn = CreateColumnModification.createModification({
-			entityName: entity.name,
-			field: deepCopy(updatedColumn),
-		})
-		if (updatedColumn.default !== undefined) {
-			createColumn.fillValue = updatedColumn.default
-		}
-		return createColumn
-	}
-
-	public visitManyHasOne(
-		entity: Model.Entity,
-		relation: Model.ManyHasOneRelation,
-		{},
-		targetRelation: Model.OneHasManyRelation | null,
-	): Migration.Modification<CreateRelationModificationData> {
-		return CreateRelationModification.createModification({
-			entityName: entity.name,
-			owningSide: relation,
-			...(targetRelation ? { inverseSide: targetRelation } : {}),
-		})
-	}
-
-	public visitOneHasMany(
-		entity: Model.Entity,
-		relation: Model.OneHasManyRelation,
-	): Migration.Modification<CreateRelationInverseSideModificationData> {
-		return CreateRelationInverseSideModification.createModification({
-			entityName: entity.name,
-			relation: relation,
-		})
-	}
-
-	public visitOneHasOneOwning(
-		entity: Model.Entity,
-		relation: Model.OneHasOneOwningRelation,
-		{},
-		targetRelation: Model.OneHasOneInverseRelation | null,
-	): Migration.Modification<CreateRelationModificationData> {
-		return CreateRelationModification.createModification({
-			entityName: entity.name,
-			owningSide: relation,
-			...(targetRelation ? { inverseSide: targetRelation } : {}),
-		})
-	}
-
-	public visitOneHasOneInverse(
-		entity: Model.Entity,
-		relation: Model.OneHasOneInverseRelation,
-	): Migration.Modification<CreateRelationInverseSideModificationData> {
-		return CreateRelationInverseSideModification.createModification({
-			entityName: entity.name,
-			relation: relation,
-		})
-	}
-
-	public visitManyHasManyOwning(
-		entity: Model.Entity,
-		relation: Model.ManyHasManyOwningRelation,
-		{},
-		targetRelation: Model.ManyHasManyInverseRelation | null,
-	): Migration.Modification<CreateRelationModificationData> {
-		return CreateRelationModification.createModification({
-			entityName: entity.name,
-			owningSide: relation,
-			...(targetRelation ? { inverseSide: targetRelation } : {}),
-		})
-	}
-
-	public visitManyHasManyInverse(
-		entity: Model.Entity,
-		relation: Model.ManyHasManyInverseRelation,
-	): Migration.Modification<CreateRelationInverseSideModificationData> {
-		return CreateRelationInverseSideModification.createModification({
-			entityName: entity.name,
-			relation: relation,
-		})
-	}
-}