diff --git a/script/migrations/split-sibling-paragraphs-with-p-br.rb b/script/migrations/split-sibling-paragraphs-with-p-br.rb new file mode 100644 index 000000000..f3e8954d6 --- /dev/null +++ b/script/migrations/split-sibling-paragraphs-with-p-br.rb @@ -0,0 +1,124 @@ +#!/usr/bin/env ruby + +BACKFILL_TIMESTAMP = Time.parse("2025-12-18 00:00:00 UTC") +ACCOUNT_ID = nil # restrict to an account_id + +# Split sibling
tags with content by inserting
s in %.2f seconds." % seconds
+ end
+
+ private
+ def suppressing_turbo_broadcasts
+ Board.suppressing_turbo_broadcasts do
+ Card.suppressing_turbo_broadcasts do
+ yield
+ end
+ end
+ end
+
+ def separate_nonblank_paragraphs
+ scanned = 0
+ fixed = 0
+ insertions = 0
+
+ action_texts_scope.find_each(**batch_options) do |rich_text|
+ next if account_id && rich_text.record.account.external_account_id != account_id
+
+ scanned += 1
+ edited = false
+
+ rich_text.body&.fragment.tap do |fragment|
+ next unless fragment
+
+ fragment.find_all("p + p").each do |node|
+ unless ineligible?(node) || ineligible?(node.previous_sibling)
+ node.add_previous_sibling empty_node_markup
+ edited = true
+ insertions += 1
+ end
+ end
+
+ if edited
+ puts " - modifying #{rich_text.record.class.name} #{rich_text.record.to_param} (account: #{rich_text.record.account.external_account_id})" unless demo_card?(rich_text.record)
+ # allow implicit touching to invalidate caches
+ rich_text.update! body: fragment.to_html
+ fixed +=1
+ end
+ end
+ end
+
+ puts "\n\Separation complete!"
+ puts " Rich texts examined: #{scanned}"
+ puts " Rich texts modified: #{fixed}"
+ puts " Paragraphs inserted: #{insertions}"
+ fixed
+ end
+
+ def action_texts_scope
+ ActionText::RichText.where(updated_at: updated_at)
+ end
+
+ def batch_options
+ { batch_size: 20, order: :desc }
+ end
+
+ def ineligible?(node)
+ empty_node?(node) || wrapping_attachment?(node)
+ end
+
+ def empty_node?(node)
+ node.to_html == empty_node_markup
+ end
+
+ def empty_node_markup
+ "
to replicaate previous view.
+# Run for the time range before paragraphs were not spaced
+# See https://app.fizzy.do/5986089/cards/3472
+# and https://github.com/basecamp/fizzy/pull/2107
+#
+# MUST BE RUN AFTER `decrypt!` when using ActiveRecord Encryption
+#
+# Run locally:
+# bin/rails runner script/migrations/split-sibling-paragraphs-with-p-br.rb
+#
+# Run via Kamal:
+# kamal app exec -d