activerecord-pg-format-db-structure 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 476baa5e7f08de7b010ef98240bf4dc9d5f92d8c381759682dfdcbdd33ba5723
-  data.tar.gz: 8a9337bff151ccadbfc77cb638aca3b1097831979206cf971d4e3af4b3258c7c
+  metadata.gz: a18ecc82be3d91f7163c58a3e585c2fa4792767cf07467d3bee7e741fac2281b
+  data.tar.gz: 33def417b36f7f11eba8fce6d57f270aaf29b1320e6cf75b3a183719493bb2b5
 SHA512:
-  metadata.gz: a6c728fe8aba0d6c943ff8ac5cf16d386099ea85f4a35c101eed701b899ab3ec266b9004313007bba2d5e2567273f5aedfff857567743ea64c1a2d5bdaa1961c
-  data.tar.gz: 17b9b5722ac383a92969640cf30a6a68aaee073eec784cbe8545eae6f85d983a5fc7a5b2bebd66755bcf097340a8e931a4e831cc298647759f039078b51663f4
+  metadata.gz: a9a16f498480f55da503970918cf3a6df4e1899dd0c9331faa684b12ddbb32f1bde3935d7becae37f3e7f9e15190e0939f9587348c5786031c1c1edbf7cd7cfb
+  data.tar.gz: a1cead05ba2829d0fee3364e097495f7c100cc84cee09bfe2ae08e8b8a23d99ec5a16500af9d0611f5bf4f9db2b3fcc70ed492e1acc508aa2aa038e5db55de8e

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 ## [Unreleased]
 
+## [0.3.0] - 2025-03-15
+
+- Introduce StatementAppender for better spacing between statements
+
 ## [0.2.1] - 2025-02-15
 
 - Add transform to remove SET commands with default values

data/README.md

@@ -5,7 +5,7 @@
 
 Automatically cleans up your PostgreSQL `structure.sql` file after each rails migration.
 
-Say good-bye to small those small diffs you get between coworkers!
+Say good-bye to those pesky diffs you get between coworkers!
 
 By default, it will:
 
@@ -191,8 +191,6 @@ INSERT INTO "schema_migrations" (version) VALUES
 into this normalize (and much more compatch & readable) version:
 
 ```sql
-
-
 CREATE EXTENSION IF NOT EXISTS pgcrypto SCHEMA public;
 
 

data/lib/activerecord-pg-format-db-structure/deparser.rb

@@ -32,42 +32,42 @@ module ActiveRecordPgFormatDbStructure
     private
 
     def deparse_stmt_generic(stmt)
-      generic_str = +"\n\n"
+      generic_str = +""
       generic_str << deparse_stmt_and_indent(stmt)
       generic_str << ";"
       generic_str
     end
 
     def deparse_stmt_compact(stmt)
-      compact_str = +"\n"
+      compact_str = +""
       compact_str << deparse_stmt(stmt)
       compact_str << ";"
       compact_str
     end
 
     def deparse_insert_stmt(stmt)
-      insert_str = +"\n\n\n"
+      insert_str = +""
       insert_str << deparse_stmt_and_indent(stmt)
       insert_str << "\n;"
       insert_str
     end
 
     def deparse_create_stmt(stmt)
-      table_str = "\n\n\n-- Name: #{stmt.relation.relname}; Type: TABLE;\n\n"
+      table_str = "-- Name: #{stmt.relation.relname}; Type: TABLE;\n\n"
       table_str << deparse_stmt_and_indent(stmt)
       table_str << ";"
       table_str
     end
 
     def deparse_view_stmt(stmt)
-      table_str = "\n\n\n-- Name: #{stmt.view.relname}; Type: VIEW;\n\n"
+      table_str = "-- Name: #{stmt.view.relname}; Type: VIEW;\n\n"
       table_str << deparse_stmt_and_indent(stmt)
       table_str << ";"
       table_str
     end
 
     def deparse_create_table_as_stmt(stmt)
-      table_str = "\n\n\n-- Name: #{stmt.into.rel.relname}; Type: MATERIALIZED VIEW;\n\n"
+      table_str = "-- Name: #{stmt.into.rel.relname}; Type: MATERIALIZED VIEW;\n\n"
       table_str << deparse_stmt_and_indent(stmt)
 
       # couldn't find a better solution for this, but probably an OK workaround?

data/lib/activerecord-pg-format-db-structure/formatter.rb

@@ -6,14 +6,16 @@ require_relative "../activerecord-pg-format-db-structure"
 module ActiveRecordPgFormatDbStructure
   # Formats & normalizes in place the given SQL string
   class Formatter
-    attr_reader :transforms, :deparser
+    attr_reader :transforms, :deparser, :statement_appender
 
     def initialize(
       transforms: DEFAULT_TRANSFORMS,
-      deparser: DEFAULT_DEPARSER
+      deparser: DEFAULT_DEPARSER,
+      statement_appender: DEFAULT_STATEMENT_APPENDER
     )
       @transforms = transforms
       @deparser = deparser
+      @statement_appender = statement_appender
     end
 
     def format(source)
@@ -23,9 +25,17 @@ module ActiveRecordPgFormatDbStructure
         transform.new(raw_statements).transform!
       end
 
-      raw_statements.map do |raw_statement|
-        deparser.new(source).deparse_raw_statement(raw_statement)
-      end.compact.join
+      appender = statement_appender.new
+      raw_statements.each do |raw_statement|
+        statement = deparser.new(source).deparse_raw_statement(raw_statement)
+        appender.append_statement!(
+          statement,
+          statement_kind: PgQuery::Node.inner_class_to_name(
+            raw_statement.stmt.inner.class
+          )
+        )
+      end
+      appender.output
     end
   end
 end

data/lib/activerecord-pg-format-db-structure/railtie.rb

@@ -6,6 +6,7 @@ module ActiveRecordPgFormatDbStructure
     config.activerecord_pg_format_db_structure = ActiveSupport::OrderedOptions.new
     config.activerecord_pg_format_db_structure.transforms = DEFAULT_TRANSFORMS.dup
     config.activerecord_pg_format_db_structure.deparser = DEFAULT_DEPARSER
+    config.activerecord_pg_format_db_structure.statement_appender = DEFAULT_STATEMENT_APPENDER
 
     rake_tasks do
       load "activerecord-pg-format-db-structure/tasks/clean_db_structure.rake"

data/lib/activerecord-pg-format-db-structure/statement_appender.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+require "pg_query"
+require_relative "indenter"
+
+module ActiveRecordPgFormatDbStructure
+  # Appends statements with reasonable spacing in-between
+  class StatementAppender
+    attr_reader :output
+
+    def initialize(output = +"")
+      @output = output
+      @previous_statement_kind = nil
+    end
+
+    def append_statement!(statement, statement_kind:)
+      output.chomp!
+      output << newlines_separator(
+        previous_kind: @previous_statement_kind,
+        current_kind: statement_kind
+      )
+      @previous_statement_kind = statement_kind
+      output << statement
+      output << "\n"
+    end
+
+    private
+
+    def newlines_separator(previous_kind:, current_kind:)
+      case [
+        previous_kind,
+        current_kind
+      ]
+      in [
+        nil,
+        _
+      ]
+        ""
+      in [
+        _,
+        :insert_stmt | :create_stmt | :view_stmt | :create_table_as_stmt
+      ]
+        "\n\n\n"
+      in [
+        :create_stmt | :view_stmt | :create_table_as_stmt | :index_stmt,
+        :index_stmt
+      ] | [
+        :variable_set_stmt,
+        :variable_set_stmt
+      ]
+        "\n"
+      else
+        "\n\n"
+      end
+    end
+  end
+end

data/lib/activerecord-pg-format-db-structure/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveRecordPgFormatDbStructure
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end

data/lib/activerecord-pg-format-db-structure.rb

@@ -3,6 +3,7 @@
 require_relative "activerecord-pg-format-db-structure/version"
 
 require_relative "activerecord-pg-format-db-structure/deparser"
+require_relative "activerecord-pg-format-db-structure/statement_appender"
 require_relative "activerecord-pg-format-db-structure/transforms/remove_comments_on_extensions"
 require_relative "activerecord-pg-format-db-structure/transforms/inline_serials"
 require_relative "activerecord-pg-format-db-structure/transforms/inline_primary_keys"
@@ -29,6 +30,7 @@ module ActiveRecordPgFormatDbStructure
   ].freeze
 
   DEFAULT_DEPARSER = Deparser
+  DEFAULT_STATEMENT_APPENDER = StatementAppender
 end
 
 # :nocov:

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-pg-format-db-structure
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Jell
 bindir: exe
 cert_chain: []
-date: 2025-02-15 00:00:00.000000000 Z
+date: 2025-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg_query
@@ -43,6 +43,7 @@ files:
 - lib/activerecord-pg-format-db-structure/formatter.rb
 - lib/activerecord-pg-format-db-structure/indenter.rb
 - lib/activerecord-pg-format-db-structure/railtie.rb
+- lib/activerecord-pg-format-db-structure/statement_appender.rb
 - lib/activerecord-pg-format-db-structure/tasks/clean_db_structure.rake
 - lib/activerecord-pg-format-db-structure/transforms/base.rb
 - lib/activerecord-pg-format-db-structure/transforms/group_alter_table_statements.rb