dynamic_migrations 3.0.0 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 389dc33f29bc39c47961125ad36f5faa377d8609d6d64c2f008517c81bbeb44c
-  data.tar.gz: 27bb1d8d55727f8467b52b7240373bfaf397262d88adfa07db2a04a5d3b52e65
+  metadata.gz: 971a68fa07394767b18c8781df8be28eada7a1024256698f63bc4c0b0c06a560
+  data.tar.gz: 266f91ca87e1da40a0fbd13a9cde183e167c2950f1d0718169fdb8e4ff4eec52
 SHA512:
-  metadata.gz: bf62099155ca8274632d52fbe7328e66eed962a73cbafd6c53cd056ca8fcea00851ddec52263f47d82ea8486756ea89bf79d03d8b2d2fa489399c7f7e60fb3c0
-  data.tar.gz: 0ea7b26061538ae65d1b1c9cddf91dd8afa3a8c2ac3ff41416926235ccb5e0b2ad6cad304129b038d39939cab32492d00c289a85331cc7724a24678b0e202821
+  metadata.gz: f4f5ca51ab18b44f7a832d9442c7412e725c6b4d688fd481d45752168a6fb6665b15cf288934399d0eb7ef64e8a78a44730fa138cb0f1ff0e1435c135a8b627e
+  data.tar.gz: 223b0432f2198a98f33dfeba24f156938b47d286908a62571f93b7acfebeef6fc5594f0eac8c47805e01c1038c25af6af68321e0e829d4f673b5a5292587eede

data/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## [3.1.1](https://github.com/craigulliott/dynamic_migrations/compare/v3.1.0...v3.1.1) (2023-08-16)
+
+
+### Bug Fixes
+
+* refactoring the migration generator so that it handles dependencies between migrations ([8d0f8d8](https://github.com/craigulliott/dynamic_migrations/commit/8d0f8d8cd4a22b974b6449c62ad2a2d74aeffb2e))
+
+## [3.1.0](https://github.com/craigulliott/dynamic_migrations/compare/v3.0.0...v3.1.0) (2023-08-14)
+
+
+### Features
+
+* adding a convenience method `to_migrations` on the differences class ([efe94be](https://github.com/craigulliott/dynamic_migrations/commit/efe94be9c620fc44d1cdba54f6cd365292ae0f34))
+
+
+### Bug Fixes
+
+* updating to_migrations method signature in differences class ([e4e502b](https://github.com/craigulliott/dynamic_migrations/commit/e4e502b7ec44add3d2b6727869a4ad90bc7531b7))
+
 ## [3.0.0](https://github.com/craigulliott/dynamic_migrations/compare/v2.2.0...v3.0.0) (2023-08-11)
 
 
@@ -13,8 +32,6 @@
 * functions and triggers, removed unusable index_type from primary key and unique constraints, added active record migrations and some general refactoring ([45fcb7c](https://github.com/craigulliott/dynamic_migrations/commit/45fcb7ca3b6724625a4868198a9e75aefb1ea964))
 * now generating migrations from all database structure object types ([871ab04](https://github.com/craigulliott/dynamic_migrations/commit/871ab048efb6247bc7fadb6e49bb15f6933c5586))
 * various improvements to triggers and functions, and other general improvements ([e7dd9ab](https://github.com/craigulliott/dynamic_migrations/commit/e7dd9abeee736ea2791c192d3f797497f54773b4))
-* work in progress on the migrations generator ([3e8deb9](https://github.com/craigulliott/dynamic_migrations/commit/3e8deb954913be291ad8e6e7d1c09af619378e4d))
-* work in progress on the migrations generator ([7a999ee](https://github.com/craigulliott/dynamic_migrations/commit/7a999ee8467527164e79f2c6da704245e78cf0cd))
 
 ## [2.2.0](https://github.com/craigulliott/dynamic_migrations/compare/v2.1.0...v2.2.0) (2023-07-31)
 
@@ -35,11 +52,11 @@
 
 ### ⚠ BREAKING CHANGES
 
-* changing all name related methods from %object%_name to just name (i.e. `table.table_name` is now just `table.name`)
+* changing all name related methods from `%object%_name` to just `name` (i.e. `table.table_name` is now just `table.name`)
 
 ### Features
 
-* changing all name related methods from %object%_name to just name (i.e. `table.table_name` is now just `table.name`) ([77f18ae](https://github.com/craigulliott/dynamic_migrations/commit/77f18ae168c2449fa437fa7692ff9339931f9076))
+* changing all name related methods from `%object%_name` to just `name` (i.e. `table.table_name` is now just `table.name`) ([77f18ae](https://github.com/craigulliott/dynamic_migrations/commit/77f18ae168c2449fa437fa7692ff9339931f9076))
 
 ## [1.1.1](https://github.com/craigulliott/dynamic_migrations/compare/v1.1.0...v1.1.1) (2023-07-17)
 
@@ -47,8 +64,6 @@
 ### Bug Fixes
 
 * validating that new databases don't already exist and removing a debug print statement ([73bed2d](https://github.com/craigulliott/dynamic_migrations/commit/73bed2d6ab928c7a8f53b2aa17a188a77ed369e9))
-* validating that new databases don't already exist and removing a debug print statement ([73bed2d](https://github.com/craigulliott/dynamic_migrations/commit/73bed2d6ab928c7a8f53b2aa17a188a77ed369e9))
-* validating that new databases don't already exist and removing a debug print statement ([f4cfb1e](https://github.com/craigulliott/dynamic_migrations/commit/f4cfb1e81252ed967ad1a6d864d495c182ca908f))
 
 ## [1.1.0](https://github.com/craigulliott/dynamic_migrations/compare/v1.0.0...v1.1.0) (2023-07-10)
 

data/lib/dynamic_migrations/postgres/generator/column.rb

@@ -28,11 +28,16 @@ module DynamicMigrations
             data_type = "\"#{data_type}\""
           end
 
-          add_migration column.table.schema.name, column.table.name, :add_column, column.name, code_comment, <<~RUBY
-            add_column :#{column.table.name}, :#{column.name}, :#{data_type}, #{options_syntax}, comment: <<~COMMENT
-              #{indent column.description}
-            COMMENT
-          RUBY
+          add_fragment schema: column.table.schema,
+            table: column.table,
+            migration_method: :add_column,
+            object: column,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              add_column :#{column.table.name}, :#{column.name}, :#{data_type}, #{options_syntax}, comment: <<~COMMENT
+                #{indent column.description}
+              COMMENT
+            RUBY
         end
 
         def change_column column, code_comment = nil
@@ -54,15 +59,25 @@ module DynamicMigrations
             data_type = ":\"#{data_type}\""
           end
 
-          add_migration column.table.schema.name, column.table.name, :change_column, column.name, code_comment, <<~RUBY
-            change_column :#{column.table.name}, :#{column.name}, :#{data_type}, #{options_syntax}
-          RUBY
+          add_fragment schema: column.table.schema,
+            table: column.table,
+            migration_method: :change_column,
+            object: column,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              change_column :#{column.table.name}, :#{column.name}, :#{data_type}, #{options_syntax}
+            RUBY
         end
 
         def remove_column column, code_comment = nil
-          add_migration column.table.schema.name, column.table.name, :remove_column, column.name, code_comment, <<~RUBY
-            remove_column :#{column.table.name}, :#{column.name}
-          RUBY
+          add_fragment schema: column.table.schema,
+            table: column.table,
+            migration_method: :remove_column,
+            object: column,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              remove_column :#{column.table.name}, :#{column.name}
+            RUBY
         end
 
         # add a comment to a column
@@ -73,18 +88,28 @@ module DynamicMigrations
             raise MissingDescriptionError
           end
 
-          add_migration column.table.schema.name, column.table.name, :set_column_comment, column.name, code_comment, <<~RUBY
-            set_column_comment :#{column.table.name}, :#{column.name}, <<~COMMENT
-              #{indent description}
-            COMMENT
-          RUBY
+          add_fragment schema: column.table.schema,
+            table: column.table,
+            migration_method: :set_column_comment,
+            object: column,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              set_column_comment :#{column.table.name}, :#{column.name}, <<~COMMENT
+                #{indent description}
+              COMMENT
+            RUBY
         end
 
         # remove the comment from a column
         def remove_column_comment column, code_comment = nil
-          add_migration column.table.schema.name, column.table.name, :remove_column_comment, column.name, code_comment, <<~RUBY
-            remove_column_comment :#{column.table.name}, :#{column.name}
-          RUBY
+          add_fragment schema: column.table.schema,
+            table: column.table,
+            migration_method: :remove_column_comment,
+            object: column,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              remove_column_comment :#{column.table.name}, :#{column.name}
+            RUBY
         end
       end
     end

data/lib/dynamic_migrations/postgres/generator/foreign_key_constraint.rb

@@ -29,15 +29,26 @@ module DynamicMigrations
 
           options_syntax = options.map { |k, v| "#{k}: #{v}" }.join(", ")
 
-          add_migration foreign_key_constraint.table.schema.name, foreign_key_constraint.table.name, :add_foreign_key, foreign_key_constraint.name, code_comment, <<~RUBY
-            add_foreign_key :#{foreign_key_constraint.table.name}, #{column_names}, :#{foreign_key_constraint.foreign_table.name}, #{foreign_column_names}, #{options_syntax}
-          RUBY
+          add_fragment schema: foreign_key_constraint.table.schema,
+            table: foreign_key_constraint.table,
+            migration_method: :add_foreign_key,
+            object: foreign_key_constraint,
+            code_comment: code_comment,
+            dependent_table: foreign_key_constraint.foreign_table,
+            migration: <<~RUBY
+              add_foreign_key :#{foreign_key_constraint.table.name}, #{column_names}, :#{foreign_key_constraint.foreign_table.name}, #{foreign_column_names}, #{options_syntax}
+            RUBY
         end
 
         def remove_foreign_key_constraint foreign_key_constraint, code_comment = nil
-          add_migration foreign_key_constraint.table.schema.name, foreign_key_constraint.table.name, :remove_foreign_key, foreign_key_constraint.name, code_comment, <<~RUBY
-            remove_foreign_key :#{foreign_key_constraint.table.name}, :#{foreign_key_constraint.name}
-          RUBY
+          add_fragment schema: foreign_key_constraint.table.schema,
+            table: foreign_key_constraint.table,
+            migration_method: :remove_foreign_key,
+            object: foreign_key_constraint,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              remove_foreign_key :#{foreign_key_constraint.table.name}, :#{foreign_key_constraint.name}
+            RUBY
         end
 
         def recreate_foreign_key_constraint original_foreign_key_constraint, updated_foreign_key_constraint
@@ -65,18 +76,28 @@ module DynamicMigrations
             raise MissingDescriptionError
           end
 
-          add_migration foreign_key_constraint.table.schema.name, foreign_key_constraint.table.name, :set_foreign_key_constraint_comment, foreign_key_constraint.name, code_comment, <<~RUBY
-            set_foreign_key_comment :#{foreign_key_constraint.table.name}, :#{foreign_key_constraint.name}, <<~COMMENT
-              #{indent description}
-            COMMENT
-          RUBY
+          add_fragment schema: foreign_key_constraint.table.schema,
+            table: foreign_key_constraint.table,
+            migration_method: :set_foreign_key_constraint_comment,
+            object: foreign_key_constraint,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              set_foreign_key_comment :#{foreign_key_constraint.table.name}, :#{foreign_key_constraint.name}, <<~COMMENT
+                #{indent description}
+              COMMENT
+            RUBY
         end
 
         # remove the comment from a foreign_key_constraint
         def remove_foreign_key_constraint_comment foreign_key_constraint, code_comment = nil
-          add_migration foreign_key_constraint.table.schema.name, foreign_key_constraint.table.name, :remove_foreign_key_constraint_comment, foreign_key_constraint.name, code_comment, <<~RUBY
-            remove_foreign_key_comment :#{foreign_key_constraint.table.name}, :#{foreign_key_constraint.name}
-          RUBY
+          add_fragment schema: foreign_key_constraint.table.schema,
+            table: foreign_key_constraint.table,
+            migration_method: :remove_foreign_key_constraint_comment,
+            object: foreign_key_constraint,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              remove_foreign_key_comment :#{foreign_key_constraint.table.name}, :#{foreign_key_constraint.name}
+            RUBY
         end
       end
     end

data/lib/dynamic_migrations/postgres/generator/fragment.rb

@@ -2,15 +2,25 @@ module DynamicMigrations
   module Postgres
     class Generator
       class Fragment
+        attr_reader :schema_name
+        attr_reader :table_name
+        attr_reader :migration_method
         attr_reader :object_name
-        attr_reader :code_comment
+        attr_reader :dependency_schema_name
+        attr_reader :dependency_table_name
 
-        def initialize object_name, code_comment, content
+        def initialize schema_name, table_name, migration_method, object_name, code_comment, content
+          @schema_name = schema_name
+          @table_name = table_name
+          @migration_method = migration_method
           @object_name = object_name
           @code_comment = code_comment
           @content = content
         end
 
+        # Returns a string representation of the fragment for use in the final
+        # migration. This final string is a combination of the code_comment (if present)
+        # and the content of the fragment.
         def to_s
           strings = []
           comment = @code_comment
@@ -21,9 +31,34 @@ module DynamicMigrations
           strings.join("\n").strip
         end
 
+        # Returns true if the fragment has a code comment, otherwise false.
         def has_code_comment?
           !@code_comment.nil?
         end
+
+        # If a table dependency has been set, then returns a hash with the schema_name
+        # and table_name, otherwise returns nil.
+        def dependency
+          if dependency_schema_name && dependency_table_name
+            {
+              schema_name: dependency_schema_name,
+              table_name: dependency_table_name
+            }
+          end
+        end
+
+        # returns true if the fragment has a table dependency, and the dependency matches
+        # the provided schema_name and table_name, otherwise returns false.
+        def is_dependent_on? schema_name, table_name
+          dependency_schema_name && schema_name == dependency_schema_name && table_name == dependency_table_name || false
+        end
+
+        # Set the table table dependency for this fragment. Takes a schema name and
+        # table name
+        def set_dependent_table schema_name, table_name
+          @dependency_schema_name = schema_name
+          @dependency_table_name = table_name
+        end
       end
     end
   end

data/lib/dynamic_migrations/postgres/generator/function.rb

@@ -25,13 +25,18 @@ module DynamicMigrations
             fn_sql << ";"
           end
 
-          add_migration function.schema.name, function.triggers.first&.table&.name, :create_function, function.name, code_comment, (comment_sql + <<~RUBY)
-            create_function :#{function.name}#{optional_options_syntax} do
-              <<~SQL
-                #{indent fn_sql}
-              SQL
-            end
-          RUBY
+          add_fragment schema: function.schema,
+            table: function.triggers.first&.table,
+            migration_method: :create_function,
+            object: function,
+            code_comment: code_comment,
+            migration: comment_sql + <<~RUBY
+              create_function :#{function.name}#{optional_options_syntax} do
+                <<~SQL
+                  #{indent fn_sql}
+                SQL
+              end
+            RUBY
         end
 
         def update_function function, code_comment = nil
@@ -41,35 +46,55 @@ module DynamicMigrations
             fn_sql << ";"
           end
 
-          add_migration function.schema.name, function.triggers.first&.table&.name, :update_function, function.name, code_comment, <<~RUBY
-            update_function :#{function.name} do
-              <<~SQL
-                #{indent fn_sql}
-              SQL
-            end
-          RUBY
+          add_fragment schema: function.schema,
+            table: function.triggers.first&.table,
+            migration_method: :update_function,
+            object: function,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              update_function :#{function.name} do
+                <<~SQL
+                  #{indent fn_sql}
+                SQL
+              end
+            RUBY
         end
 
         def drop_function function, code_comment = nil
-          add_migration function.schema.name, function.triggers.first&.table&.name, :drop_function, function.name, code_comment, <<~RUBY
-            drop_function :#{function.name}
-          RUBY
+          add_fragment schema: function.schema,
+            table: function.triggers.first&.table,
+            migration_method: :drop_function,
+            object: function,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              drop_function :#{function.name}
+            RUBY
         end
 
         # add a comment to a function
         def set_function_comment function, code_comment = nil
-          add_migration function.schema.name, function.triggers.first&.table&.name, :set_function_comment, function.name, code_comment, <<~RUBY
-            set_function_comment :#{function.name}, <<~COMMENT
-              #{indent function.description || ""}
-            COMMENT
-          RUBY
+          add_fragment schema: function.schema,
+            table: function.triggers.first&.table,
+            migration_method: :set_function_comment,
+            object: function,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              set_function_comment :#{function.name}, <<~COMMENT
+                #{indent function.description || ""}
+              COMMENT
+            RUBY
         end
 
         # remove the comment from a function
         def remove_function_comment function, code_comment = nil
-          add_migration function.schema.name, function.triggers.first&.table&.name, :remove_function_comment, function.name, code_comment, <<~RUBY
-            remove_function_comment :#{function.name}
-          RUBY
+          add_fragment schema: function.schema,
+            table: function.triggers.first&.table,
+            migration_method: :remove_function_comment,
+            object: function,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              remove_function_comment :#{function.name}
+            RUBY
         end
       end
     end

data/lib/dynamic_migrations/postgres/generator/index.rb

@@ -46,15 +46,25 @@ module DynamicMigrations
 
           options_syntax = options.map { |k, v| "#{k}: #{v}" }.join(", ")
 
-          add_migration index.table.schema.name, index.table.name, :add_index, index.name, code_comment, (where_sql + <<~RUBY)
-            add_index :#{index.table.name}, #{column_names}, #{options_syntax}
-          RUBY
+          add_fragment schema: index.table.schema,
+            table: index.table,
+            migration_method: :add_index,
+            object: index,
+            code_comment: code_comment,
+            migration: where_sql + <<~RUBY
+              add_index :#{index.table.name}, #{column_names}, #{options_syntax}
+            RUBY
         end
 
         def remove_index index, code_comment = nil
-          add_migration index.table.schema.name, index.table.name, :remove_index, index.name, code_comment, <<~RUBY
-            remove_index :#{index.table.name}, :#{index.name}
-          RUBY
+          add_fragment schema: index.table.schema,
+            table: index.table,
+            migration_method: :remove_index,
+            object: index,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              remove_index :#{index.table.name}, :#{index.name}
+            RUBY
         end
 
         def recreate_index original_index, updated_index
@@ -82,18 +92,28 @@ module DynamicMigrations
             raise MissingDescriptionError
           end
 
-          add_migration index.table.schema.name, index.table.name, :set_index_comment, index.name, code_comment, <<~RUBY
-            set_index_comment :#{index.table.name}, :#{index.name}, <<~COMMENT
-              #{indent description}
-            COMMENT
-          RUBY
+          add_fragment schema: index.table.schema,
+            table: index.table,
+            migration_method: :set_index_comment,
+            object: index,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              set_index_comment :#{index.table.name}, :#{index.name}, <<~COMMENT
+                #{indent description}
+              COMMENT
+            RUBY
         end
 
         # remove the comment from a index
         def remove_index_comment index, code_comment = nil
-          add_migration index.table.schema.name, index.table.name, :remove_index_comment, index.name, code_comment, <<~RUBY
-            remove_index_comment :#{index.table.name}, :#{index.name}
-          RUBY
+          add_fragment schema: index.table.schema,
+            table: index.table,
+            migration_method: :remove_index_comment,
+            object: index,
+            code_comment: code_comment,
+            migration: <<~RUBY
+              remove_index_comment :#{index.table.name}, :#{index.name}
+            RUBY
         end
       end
     end

data/lib/dynamic_migrations/postgres/generator/migration.rb

@@ -0,0 +1,175 @@
+module DynamicMigrations
+  module Postgres
+    class Generator
+      class Migration
+        class UnexpectedSchemaError < StandardError
+        end
+
+        class SectionNotFoundError < StandardError
+        end
+
+        class UnexpectedMigrationMethodNameError < StandardError
+        end
+
+        class DuplicateStructureTemplateError < StandardError
+        end
+
+        class NoFragmentsError < StandardError
+        end
+
+        # Defines a new section in the migration file, this is used to group
+        # migration fragments of the provided method names together under the
+        # provided header
+        def self.add_structure_template method_names, header
+          @structure_templates ||= []
+
+          if (@structure_templates.map { |s| s[:methods] }.flatten & method_names).any?
+            raise DuplicateStructureTemplateError, "Duplicate structure template for methods `#{method_names}`"
+          end
+
+          @structure_templates << {
+            methods: method_names,
+            header_comment: <<~COMMENT.strip
+              #
+              # #{header.strip}
+              #
+            COMMENT
+          }
+        end
+
+        # return the list of structure templates for use in this migration
+        def self.structure_templates
+          @structure_templates || []
+        end
+
+        # return the list of structure templates for use in this migration
+        def self.clear_structure_templates
+          @structure_templates = []
+        end
+
+        attr_reader :schema_name
+        attr_reader :fragments
+
+        def initialize schema_name
+          @schema_name = schema_name
+          @fragments = []
+        end
+
+        # Add a migration fragment to this migration, if the migration is not
+        # configured (via a structure template) to handle the method_name of the
+        # fragment, then am error is raised. An error will also be raised if the
+        # migration belongs to a different schema than the provided fragment.
+        def add_fragment fragment
+          raise UnexpectedSchemaError unless @schema_name == fragment.schema_name
+
+          unless supported_migration_method? fragment.migration_method
+            raise UnexpectedMigrationMethodNameError, "Expected method to be a valid migrator method, got `#{fragment.migration_method}`"
+          end
+
+          @fragments << fragment
+
+          fragment
+        end
+
+        # Return an array of table dependencies for this migration, this array comes from
+        # combining any table dependencies from each fragment.
+        # Will raise an error if no fragments have been provided.
+        def dependencies
+          raise NoFragmentsError if fragments.empty?
+          @fragments.map(&:dependency).compact
+        end
+
+        # removes and returns any fragments which have a dependency on the table with the
+        # provided schema_name and table_name, this is used for extracting fragments which
+        # cause circular dependencies so they can be placed into their own migrations
+        def extract_fragments_with_dependency schema_name, table_name
+          results = @fragments.filter { |f| f.is_dependent_on? schema_name, table_name }
+          # remove any of these from the internal array of fragments
+          @fragments.filter! { |f| !f.is_dependent_on?(schema_name, table_name) }
+          # return the results
+          results
+        end
+
+        # Combine the fragments, and build a string representation of the migration
+        # using the structure templates defined in this class.
+        # Will raise an error if no fragments have been provided.
+        def content
+          raise NoFragmentsError if fragments.empty?
+          sections = []
+          self.class.structure_templates.each do |section|
+            # add the header comment if we have a migration which matches one of the
+            # methods in this section
+            if (section[:methods] & @fragments.map(&:migration_method)).any?
+              sections << section[:header_comment].strip
+            end
+
+            # iterate through this sections methods in order and look
+            # for any that match the migrations we have
+            section[:methods].each do |migration_method|
+              # if we have any migration fragments for this method then add them
+              @fragments.filter { |f| f.migration_method == migration_method }.each do |fragment|
+                sections << fragment.to_s
+                sections << ""
+              end
+            end
+          end
+          sections.join("\n").strip
+        end
+
+        # Using the migration fragments, generate a friendly/descriptive name for the migration.
+        # Will raise an error if no fragments have been provided.
+        def name
+          raise NoFragmentsError if fragments.empty?
+
+          if fragments_for_method? :create_schema
+            "create_#{first_fragment_using_migration_method(:create_schema).schema_name}_schema".to_sym
+
+          elsif fragments_for_method? :drop_schema
+            "drop_#{first_fragment_using_migration_method(:drop_schema).schema_name}_schema".to_sym
+
+          elsif fragments_for_method? :create_table
+            "create_#{first_fragment_using_migration_method(:create_table).table_name}".to_sym
+
+          elsif fragments_for_method? :drop_table
+            "drop_#{first_fragment_using_migration_method(:drop_table).table_name}".to_sym
+
+          elsif all_fragments_for_method? [:create_function]
+            "create_function_#{@fragments.find { |s| s.migration_method == :create_function }&.object_name}".to_sym
+
+          elsif all_fragments_for_method? [:create_function, :update_function, :drop_function, :set_function_comment, :remove_function_comment]
+            :schema_functions
+
+          elsif @fragments.first&.table_name
+            "changes_for_#{@fragments.first&.table_name}".to_sym
+
+          else
+            :changes
+          end
+        end
+
+        private
+
+        def supported_migration_method? method_name
+          self.class.structure_templates.map { |s| s[:methods] }.flatten.include? method_name
+        end
+
+        def fragments_for_method? migration_method
+          @fragments.map(&:migration_method).include? migration_method
+        end
+
+        def first_fragment_using_migration_method migration_method
+          fragment = @fragments.find(&:migration_method)
+          if fragment.nil?
+            raise SectionNotFoundError, "No fragment of type #{migration_method} found"
+          end
+          fragment
+        end
+
+        # return true if the current migration only has the provided content types and comments
+        def all_fragments_for_method? migration_methods
+          (@fragments.map(&:migration_method) - migration_methods - [:comment]).empty?
+        end
+      end
+    end
+  end
+end

data/lib/dynamic_migrations/postgres/generator/migration_dependency_sorter.rb

@@ -0,0 +1,21 @@
+# TSort is a module included in the Ruby standard library for
+# executing topological sorts. We use it here to sort the migration
+# fragments so that they are executed in the correct order (i.e. tables
+# which have foreign keys are created after the tables they point to).
+require "tsort"
+
+module DynamicMigrations
+  module Postgres
+    class Generator
+      class MigrationDependencySorter < Hash
+        include TSort
+
+        alias_method :tsort_each_node, :each_key
+
+        def tsort_each_child(node, &block)
+          fetch(node).each(&block)
+        end
+      end
+    end
+  end
+end