bible270 1.4.0 → 1.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8975632aa3e90165634356f25fc0c8cff2909e84b3177a58e192b16ee325b745
-  data.tar.gz: 543aa029054018c26f87d2d1836302ab671de51b70caa7a531ed0011a405f5e6
+  metadata.gz: 8792007d09956f4b53cbe9842007baea209a5eb0010758ef35e7d6f1977928f0
+  data.tar.gz: 0c5d16299136af62a6e1466210e962ef6e8d48065d759d9b5b57450215471c31
 SHA512:
-  metadata.gz: c76e854eb6449addef5b1deb2066d70cc6c51e8650817efa7c4fef8efa0d847d0b1e85c75d9faa91dde497495a01c980c674342148326f8c43f146a3321175d4
-  data.tar.gz: 80761d784e504c167954827dbe3e3e8193e571e91eee67b8bd9aa12b7990b9845020f20399c3367f63b41d441107fa15748e6e4199d327ce2588d1976a6227ca
+  metadata.gz: 89cbba3ff6adfc107efacddb9b2d80903452488384983b488ecf733aa6e16d55451a0137626cda5a446591b2c28585201f49b612927a6ba80ad0b54a5889c377
+  data.tar.gz: d2d501c3f9935405bf0b850124f2329b3c5355ccfdf5d7403688329a65f1bfdc4a779fc3d65d957eba242363393e6a057519ef509b337986f6426bb2a3c6ffdc

data/CHANGELOG.md

@@ -6,15 +6,50 @@ All notable changes to bible270. Format follows [Keep a Changelog](https://keepa
 
 ### Added
 
+- let administrators disable the day-completion dove animation for individual readers
+
+### Changed
+
+- simplify day completion to one continuous server-confirmed dove flight from the lower center, eliminating the stagger between a pre-submit animation and the Turbo or HTML response
+- keep the dove bright in dark mode, grow it to an 80%-of-viewport fit, and finish with a quick fade instead of veering offscreen
+
+### Fixed
+
+- show the dove only once for each persisted day-completion event, even when Turbo replays a response or retries a stale-session request
+
+**[Full Changelog](https://github.com/avonderluft/bible270/compare/v1.4.0...main)**
+
+## [1.4.0] - 2026-09-02
+
+### Added
+
 - mark a newly completed day with a brief, translucent dove animation that begins immediately on the final checkoff in JavaScript-capable browsers and remains available after Turbo or HTML-fallback responses; it respects reduced-motion preferences, replaces the final generic success message, and never blocks interaction
 
+### Changed
+
+- refine the completion animation to 1.6 seconds with smoother easing, limit checkoff transitions to relevant visual properties, and make reduced-motion transitions effectively instantaneous
+
 ### Fixed
 
-- correct Day 27's New Testament reading to include Matthew 26:75
-- silently recover all Turbo interactions from stale sessions, retry once with a fresh CSRF token, and reload rather than submit with a known-stale token if refreshing fails
 - keep repeated or concurrent explicit checkoffs idempotent instead of raising a duplicate-validation Rails 422 error
 
-**[Full Changelog](https://github.com/avonderluft/bible270/compare/v1.3.0...main)**
+**[Full Changelog](https://github.com/avonderluft/bible270/compare/v1.3.2...v1.4.0)**
+
+## [1.3.2] - 2026-09-02
+
+### Fixed
+
+- silently recover all Turbo interactions from stale sessions, retry once with a fresh CSRF token, and reload rather than submit with a known-stale token if refreshing fails
+
+**[Full Changelog](https://github.com/avonderluft/bible270/compare/v1.3.1...v1.3.2)**
+
+## [1.3.1] - 2026-09-01
+
+### Fixed
+
+- correct Day 27's New Testament reading to include Matthew 26:75
+
+**[Full Changelog](https://github.com/avonderluft/bible270/compare/v1.3.0...v1.3.1)**
 
 ## [1.3.0] - 2026-08-29
 
@@ -596,6 +631,9 @@ This is the initial public release.
 
 * Email sign-in requires working Action Mailer delivery in the host application. Set `config.mailer_from`; delivery is inline by default, or set `email_sign_in_deliver_later` when a queue backend is available.
 
+[1.4.0]: https://github.com/avonderluft/bible270/releases/tag/v1.4.0
+[1.3.2]: https://github.com/avonderluft/bible270/releases/tag/v1.3.2
+[1.3.1]: https://github.com/avonderluft/bible270/releases/tag/v1.3.1
 [1.3.0]: https://github.com/avonderluft/bible270/releases/tag/v1.3.0
 [1.2.0]: https://github.com/avonderluft/bible270/releases/tag/v1.2.0
 [1.1.7]: https://github.com/avonderluft/bible270/releases/tag/v1.1.7

data/app/controllers/bible270/admin_controller.rb

@@ -13,19 +13,22 @@ module Bible270
 
     before_action :require_admin!
     before_action :load_reader,
-                  only: %i[show destroy update_start update_profile update_notifications remove_avatar complete_through
-                           toggle_day]
+                  only: %i[show destroy update_start update_profile update_notifications update_completion_animation
+                           remove_avatar complete_through toggle_day]
     before_action :load_comment, only: %i[hide_comment unhide_comment destroy_comment]
 
     # ---- enrolment --------------------------------------------------------
 
     def update_enrollment
-      if params[:state] == 'closed'
+      case params[:state]
+      when 'closed'
         Setting.close_enrollment!
         redirect_to admin_path, notice: 'Closed to new readers. Existing readers can still sign in.'
-      else
+      when 'open'
         Setting.open_enrollment!
         redirect_to admin_path, notice: 'Open to new readers.'
+      else
+        redirect_to admin_path, alert: 'Choose whether enrollment is open or closed. Nothing was changed.'
       end
     end
 
@@ -71,6 +74,7 @@ module Bible270
 
     def show
       @comment_notifications_available = comment_notifications_available_for?(@reader)
+      @completion_dove_preference_available = completion_dove_available_for(@reader)
       @days_completed = @reader.days_completed
       @comments = @reader.comments.order(created_at: :desc).limit(50)
     end
@@ -122,12 +126,23 @@ module Bible270
       recipients = Reader.where.not(email: [nil, '']).order(:id)
       return redirect_to(admin_path, alert: 'Nobody has an email address.') if recipients.empty?
 
+      recipient_count = recipients.count
       sent = deliver_broadcast(recipients, subject, body)
+      later = Bible270.config.registration_notice_deliver_later
+
+      if sent.zero?
+        action = later ? 'queued' : 'sent'
+        return redirect_to admin_path, alert: "The message could not be #{action} to any readers."
+      end
+
       Setting.write(LAST_BROADCAST_AT, Time.current.iso8601)
       Setting.write(LAST_BROADCAST_SUBJECT, subject)
 
-      redirect_to admin_path,
-                  notice: "Sent to #{sent} #{'reader'.pluralize(sent)}."
+      action = later ? 'Queued for' : 'Sent to'
+      failures = recipient_count - sent
+      notice = "#{action} #{sent} #{'reader'.pluralize(sent)}."
+      notice += " #{failures} #{'reader'.pluralize(failures)} could not be #{later ? 'queued' : 'sent'}." if failures.positive?
+      redirect_to admin_path, notice: notice
     end
 
     def update_bible_version
@@ -169,9 +184,14 @@ module Bible270
           redirect_to admin_reader_path(@reader),
                       alert: "Couldn't read #{params[:start_date].inspect} as a date."
         end
-      elsif params[:day].present? && @reader.restart_on!(day: params[:day])
-        redirect_to admin_reader_path(@reader),
-                    notice: "#{@reader.display_name} is now on day #{params[:day]}."
+      elsif params[:day].present?
+        day = strict_integer(params[:day], 1..Plan::DAYS)
+        if day && @reader.restart_on!(day: day)
+          redirect_to admin_reader_path(@reader),
+                      notice: "#{@reader.display_name} is now on day #{day}."
+        else
+          redirect_to admin_reader_path(@reader), alert: 'Give a start date or a day between 1 and 270.'
+        end
       else
         redirect_to admin_reader_path(@reader), alert: 'Give a start date or a day between 1 and 270.'
       end
@@ -181,12 +201,15 @@ module Bible270
     # themselves on their own profile.
     def update_profile
       problems = []
-      if (params[:first_name].present? || params[:last_name].present?) && !@reader.update_names(params[:first_name],
-                                                                                                params[:last_name])
-        problems << 'both a first and last name'
-      end
-      if params[:avatar].present? && !@reader.attach_avatar(params[:avatar])
-        problems << (@reader.errors[:avatar].first || 'a valid image')
+      @reader.transaction do
+        if (params[:first_name].present? || params[:last_name].present?) && !@reader.update_names(params[:first_name],
+                                                                                                  params[:last_name])
+          problems << 'both a first and last name'
+        end
+        if problems.empty? && params[:avatar].present? && !@reader.attach_avatar(params[:avatar])
+          problems << (@reader.errors[:avatar].first || 'a valid image')
+        end
+        raise ActiveRecord::Rollback if problems.any?
       end
 
       if problems.empty?
@@ -214,28 +237,40 @@ module Bible270
       redirect_to admin_reader_path(@reader), notice: 'Reflection email setting updated.'
     end
 
+    def update_completion_animation
+      unless completion_dove_available_for(@reader)
+        alert = 'Run the pending Bible270 database migration before changing the completion animation.'
+        return redirect_to(admin_reader_path(@reader), alert: alert)
+      end
+
+      @reader.update!(completion_dove_disabled: params[:completion_dove_disabled] == '1')
+      redirect_to admin_reader_path(@reader), notice: 'Completion animation setting updated.'
+    end
+
     def remove_avatar
       @reader.remove_avatar!
       redirect_to admin_reader_path(@reader), notice: "Removed #{@reader.display_name}'s picture."
     end
 
     def complete_through
-      day = params[:day].to_i
-      if @reader.mark_through!(day)
-        redirect_to admin_reader_path(@reader),
-                    notice: day.zero? ? 'All check-offs cleared.' : "Marked days 1-#{day} complete."
-      else
-        redirect_to admin_reader_path(@reader), alert: 'Give a day between 0 and 270.'
+      day = strict_integer(params[:day], 0..Plan::DAYS)
+      unless day
+        return redirect_to admin_reader_path(@reader), alert: 'Give a day between 0 and 270.'
       end
+
+      @reader.mark_through!(day)
+      redirect_to admin_reader_path(@reader),
+                  notice: day.zero? ? 'All check-offs cleared.' : "Marked days 1-#{day} complete."
     end
 
     def toggle_day
-      day = params[:day].to_i
-      if @reader.toggle_day!(day)
-        redirect_to admin_reader_path(@reader, anchor: "day-#{day}")
-      else
-        redirect_to admin_reader_path(@reader), alert: 'That day is outside the plan.'
+      day = strict_integer(params[:day], 1..Plan::DAYS)
+      unless day
+        return redirect_to admin_reader_path(@reader), alert: 'That day is outside the plan.'
       end
+
+      @reader.toggle_day!(day)
+      redirect_to admin_reader_path(@reader, anchor: "day-#{day}")
     end
 
     # Failures are counted rather than raised: one bad address must not stop the
@@ -274,6 +309,12 @@ module Bible270
       true
     end
 
+    def completion_dove_available_for(reader)
+      available = Reader.completion_dove_column?
+      reader.reload if available && !reader.has_attribute?('completion_dove_disabled')
+      available
+    end
+
     def require_admin!
       return if Bible270.config.admin_configured? && Bible270.config.admin?(current_reader)
 
@@ -282,16 +323,25 @@ module Bible270
 
     def load_comment
       @comment = Comment.find_by(id: params[:id])
-      redirect_to(admin_comments_path, alert: 'Reflection not found.') if @comment.nil?
+      return if @comment
+
+      redirect_to admin_comments_path, alert: 'Reflection not found.'
     end
 
     def redirect_back_to_comments(notice)
-      redirect_to(params[:return_to].presence || admin_comments_path, notice: notice)
+      redirect_to(url_from(params[:return_to]) || admin_comments_path, notice: notice)
     end
 
     def load_reader
       @reader = Reader.find_by(id: params[:id])
-      redirect_to(admin_path, alert: 'Reader not found.') if @reader.nil?
+      return if @reader
+
+      redirect_to admin_path, alert: 'Reader not found.'
+    end
+
+    def strict_integer(value, range)
+      integer = Integer(value.to_s, 10, exception: false)
+      integer if range.cover?(integer)
     end
   end
 end

data/app/controllers/bible270/checkoffs_controller.rb

@@ -18,25 +18,27 @@ module Bible270
       requested_state = params[:checked] if params.key?(:checked)
       head :bad_request and return if params.key?(:checked) && !%w[0 1].include?(requested_state)
 
-      marked_read, checkoff_changed = apply_checkoff(requested_state)
-
-      @reader = current_reader.reload.reload_progress
-      @day_just_completed = marked_read && checkoff_changed && @reader.day_complete?(@day)
+      # Serialize this reader's transitions so two different final portions
+      # submitted concurrently cannot both announce the same day completion.
+      marked_read = nil
+      current_reader.with_lock do
+        marked_read, @completion_event_id = apply_checkoff(requested_state)
+        @reader = current_reader.reload.reload_progress
+        @day_just_completed = marked_read && @completion_event_id && @reader.day_complete?(@day)
+      end
       @reader_tracks = @reader.read_tracks_for(@day)
       @readings = Plan.readings_for(@day)
 
-      required = Plan.total_parts(@day)
-      completer_ids = Checkoff.where(day: @day)
-        .group(:reader_id)
-        .having('COUNT(*) >= ?', required)
-        .pluck(:reader_id)
-      @completers = Reader.where(id: completer_ids).order(:display_name)
+      @completers = Reader.completers_for(@day)
 
       respond_to do |format|
         format.turbo_stream
         format.html do
           if @day_just_completed
-            redirect_to day_path(@day), flash: { b270_day_just_completed: @day }
+            redirect_to day_path(@day), flash: {
+              b270_day_just_completed: @day,
+              b270_completion_event_id: @completion_event_id
+            }
           else
             message = marked_read ? 'Reading marked read.' : 'Reading marked unread.'
             redirect_to day_path(@day), flash: { b270_interaction_status: message }
@@ -51,9 +53,10 @@ module Bible270
       existing = find_checkoff
       if requested_state == '1'
         current_reader.ensure_started!
-        [true, existing.nil? && create_checkoff]
+        [true, existing.nil? ? create_checkoff : nil]
       elsif requested_state == '0' || existing
-        [false, existing ? existing.destroy.present? : false]
+        existing&.destroy
+        [false, nil]
       else
         current_reader.ensure_started!
         [true, create_checkoff]
@@ -66,15 +69,16 @@ module Bible270
       end
     end
 
+    # The created id is stable if Turbo replays this response, while a later
+    # re-completion creates a new id and is correctly treated as a new event.
     def create_checkoff
-      Checkoff.create!(reader: current_reader, day: @day, track: @track, part: @part)
-      true
+      Checkoff.create!(reader: current_reader, day: @day, track: @track, part: @part).id
     rescue ActiveRecord::RecordInvalid, ActiveRecord::RecordNotUnique
       # A repeated or concurrent desired-state request is already successful. Do
       # not hide any other validation failure under the same recovery.
       raise unless find_checkoff
 
-      false
+      nil
     end
   end
 end

data/app/controllers/bible270/days_controller.rb

@@ -21,16 +21,9 @@ module Bible270
       @new_comment = Comment.new(day: @day, parent_id: reply_parent&.id)
       @reader_tracks = current_reader ? current_reader.read_tracks_for(@day) : []
       @day_just_completed = flash[:b270_day_just_completed].to_i == @day
+      @completion_event_id = flash[:b270_completion_event_id] if @day_just_completed
 
-      # Every box on the day, not every track: an Old Testament reading of three
-      # chapters is three rows, so counting tracks would call someone finished
-      # who had only read the Old Testament.
-      required = Plan.total_parts(@day)
-      completer_ids = Checkoff.where(day: @day)
-        .group(:reader_id)
-        .having('COUNT(*) >= ?', required)
-        .pluck(:reader_id)
-      @completers = Reader.where(id: completer_ids).order(:display_name)
+      @completers = Reader.completers_for(@day)
     end
 
   private

data/app/models/bible270/reader.rb

@@ -33,6 +33,7 @@ module Bible270
     REFLECTIONS_SEEN_COLUMN = 'reflections_seen_at'
     COMMENT_NOTIFICATION_LEVELS = %w[all personal none].freeze
     COMMENT_NOTIFICATION_COLUMNS = %w[notify_on_mention notify_on_all_comments].freeze
+    COMPLETION_DOVE_COLUMN = 'completion_dove_disabled'
     MENTION_SUGGESTION_LIMIT = 5
 
     validates :display_name, presence: true
@@ -64,6 +65,18 @@ module Bible270
       end
     end
 
+    # A day is completed when its final required checkoff is created, so MAX(created_at)
+    # is the completion moment. Keep reader_id as a deterministic tie-breaker.
+    def self.completers_for(day)
+      reader_ids = Checkoff.where(day: day)
+        .group(:reader_id)
+        .having('COUNT(*) >= ?', Plan.total_parts(day))
+        .order(Checkoff.arel_table[:created_at].maximum.asc, :reader_id)
+        .pluck(:reader_id)
+      readers_by_id = where(id: reader_ids).index_by(&:id)
+      reader_ids.filter_map { |reader_id| readers_by_id[reader_id] }
+    end
+
     def self.reflections_seen_column?
       column_names.include?(REFLECTIONS_SEEN_COLUMN)
     rescue ActiveRecord::StatementInvalid
@@ -289,6 +302,22 @@ module Bible270
       where(notify_on_all_comments: true).where.not(email: [nil, ''])
     end
 
+    def self.completion_dove_column?
+      return true if column_names.include?(COMPLETION_DOVE_COLUMN)
+
+      reset_column_information
+      column_names.include?(COMPLETION_DOVE_COLUMN)
+    rescue ActiveRecord::StatementInvalid
+      false
+    end
+
+    # Keep day completion usable while an upgraded host is waiting to run the
+    # preference migration. Historically the dove was enabled, so a missing
+    # column preserves that behavior.
+    def completion_dove_disabled?
+      has_attribute?(COMPLETION_DOVE_COLUMN) && self[COMPLETION_DOVE_COLUMN] == true
+    end
+
     # Small, deterministic suggestions for the reflection composer. Full-handle
     # collisions are omitted because the server must never guess which reader a
     # mention intended.

data/app/views/bible270/admin/show.html.erb

@@ -70,6 +70,29 @@
   <% end %>
 </div>
 
+<div class="b270-panel">
+  <h2 class="b270-subhead">Completion animation</h2>
+  <% unless @completion_dove_preference_available %>
+    <p class="b270-flash alert" style="margin:0">
+      Run the pending Bible270 database migration before changing the completion animation.
+    </p>
+  <% else %>
+    <%= form_with url: admin_reader_completion_animation_path(@reader), method: :patch,
+                  class: "b270-emailform" do %>
+      <div class="b270-field">
+        <label class="b270-reminder-optin">
+          <%= check_box_tag :completion_dove_disabled, "1", @reader.completion_dove_disabled? %>
+          <span>Turn off the dove animation for this reader</span>
+        </label>
+      </div>
+      <p class="b270-hint" style="margin:0">
+        Day completion and progress tracking still work normally when the animation is off.
+      </p>
+      <%= submit_tag 'Save', class: 'b270-btn' %>
+    <% end %>
+  <% end %>
+</div>
+
 <div class="b270-panel">
   <h2 class="b270-subhead">Where they are in the plan</h2>
   <%= form_with url: admin_reader_start_path(@reader), method: :patch, class: "b270-startform" do %>

data/app/views/bible270/checkoffs/toggle.turbo_stream.erb

@@ -5,7 +5,7 @@
 
 <%= turbo_stream.replace "day_progress_#{@day}" do %>
   <%= render "bible270/days/day_progress", day: @day, reader: @reader,
-        just_completed: @day_just_completed %>
+        just_completed: @day_just_completed, completion_event_id: @completion_event_id %>
 <% end %>
 
 <%= turbo_stream.replace "completers_#{@day}" do %>

data/app/views/bible270/days/_completers.html.erb

@@ -2,9 +2,8 @@
 <div class="b270-completers" id="completers_<%= day %>">
   <% if completers.any? %>
     <span>Finished this day:</span>
-    <% completers.first(12).each do |reader| %>
-      <%= link_to b270_avatar(reader), reader_path(reader), title: reader.display_name %>
+    <% completers.each do |reader| %>
+      <%= link_to b270_avatar(reader, size: 56), reader_path(reader), title: reader.display_name %>
     <% end %>
-    <% if completers.size > 12 %><span><%= completers.size - 12 %> more</span><% end %>
   <% end %>
 </div>

data/app/views/bible270/days/_day_progress.html.erb

@@ -1,4 +1,4 @@
-<%# locals: (day:, reader:, just_completed: false) %>
+<%# locals: (day:, reader:, just_completed: false, completion_event_id: nil) %>
 <%
   previous_day = day > 1 ? day - 1 : nil
   next_day = day < Bible270::Plan::DAYS ? day + 1 : nil
@@ -33,7 +33,9 @@
 
 
   <% if complete %>
-    <%= render "bible270/shared/day_completion_dove", day: day if just_completed %>
+    <% if just_completed && completion_event_id && !reader&.completion_dove_disabled? %>
+      <%= render "bible270/shared/day_completion_dove", day: day, completion_event_id: completion_event_id %>
+    <% end %>
     <div class="b270-day-complete" role="status" aria-live="polite"
          <%= tag.attributes(data: completion_data) %>>
       <div>

data/app/views/bible270/days/show.html.erb

@@ -2,7 +2,7 @@
 
 <div class="b270-day-page">
   <%= render "bible270/days/day_progress", day: @day, reader: current_reader,
-        just_completed: @day_just_completed %>
+        just_completed: @day_just_completed, completion_event_id: @completion_event_id %>
 
   <%= render "bible270/shared/interaction_status" %>
 
@@ -16,8 +16,6 @@
   <% end %>
 
   <div class="b270-day-community">
-    <%= render "bible270/days/completers", day: @day, completers: @completers %>
-
     <h2 class="b270-section-h">Reflections</h2>
     <div id="comments">
       <%= render partial: "bible270/comments/comment", collection: @comments, as: :comment %>
@@ -30,8 +28,9 @@
       <%= render "bible270/shared/comment_drafts" %>
       <%= render "bible270/shared/mention_typeahead" %>
     <% end %>
+
+    <%= render "bible270/days/completers", day: @day, completers: @completers %>
   </div>
 </div>
 
 <%= render "bible270/shared/interaction_ui" %>
-<%= render "bible270/shared/day_completion_effect", day: @day %>

data/app/views/bible270/shared/_day_completion_dove.html.erb

@@ -1,17 +1,19 @@
-<%# locals: (day:) %>
+<%# locals: (day:, completion_event_id:) %>
 <svg class="b270-completion-dove" data-b270-completion-dove data-b270-day="<%= day %>"
+     data-b270-completion-event="<%= completion_event_id %>" data-turbo-temporary
      viewBox="0 0 260 200" aria-hidden="true" focusable="false">
   <defs>
     <radialGradient id="b270-dove-light" cx="50%" cy="43%" r="68%">
-      <stop offset="0%" stop-color="#fff" />
-      <stop offset="58%" stop-color="#fffef8" />
-      <stop offset="100%" stop-color="#fff9e2" />
+      <stop class="b270-dove-light-center" offset="0%" stop-color="#fff" />
+      <stop class="b270-dove-light-middle" offset="58%" stop-color="#fffef8" />
+      <stop class="b270-dove-light-edge" offset="100%" stop-color="#fff9e2" />
     </radialGradient>
   </defs>
   <g class="b270-dove-silhouette">
     <path d="M122 94C103 82 87 69 72 53L39 17l11 28-23-23 15 32-23-20 19 30-24-16 22 26-25-12 27 23-24-7 30 19-22-3 31 17c20 11 42 16 59 8 5-9 8-18 10-25Z" />
     <path d="M138 94c19-12 35-25 50-41l33-36-11 28 23-23-15 32 23-20-19 30 24-16-22 26 25-12-27 23 24-7-30 19 22-3-31 17c-20 11-42 16-59 8-5-9-8-18-10-25Z" />
     <path d="M148 140c6 7 16 14 24 19 2 2 1 5-2 6-5 2-10-1-14-5 4 8 7 17 4 22-2 3-5 1-7-2l-6-11c3 10 2 19-3 24-2 3-5 1-6-3l-4-15c0 11-3 21-8 24-3 2-5-1-6-5l-1-19c-3 10-6 17-11 20-3 2-5-1-5-5l2-19c-4 8-9 13-14 14-3 1-5-2-4-6l6-18c-5 5-11 7-16 4-3-1-3-4 0-7 7-7 17-13 26-19Z" />
-    <path d="M108 116c5-10 9-17 14-17-1-11-2-23 4-29 5-4 12-3 15 1 2 3 2 6 1 9 3 .2 6 .8 8 1.5-2.5 1.3-5.5 2.1-8 2.5-1 5-3 10-5 15 6 3 11 9 15 17-1 15-6 28-16 39-3 4-7 7-10 9-4-2-8-5-11-9-9-11-14-25-11-37 1-1 2-2 4-2Z" />
+    <path class="b270-dove-beak" d="M140 80C144 80 148 80.8 151 82 152 82.4 152 83.2 151 83.8 148 85.1 144 85.8 140 85 138.5 83.5 138.5 81.2 140 80Z" />
+    <path d="M108 116c5-10 9-17 14-17-1-11-2-23 4-29 5-4 12-3 15 1 2 3 2 6 1 9 1 1 1 3 0 4-1 5-3 10-5 15 6 3 11 9 15 17-1 15-6 28-16 39-3 4-7 7-10 9-4-2-8-5-11-9-9-11-14-25-11-37 1-1 2-2 4-2Z" />
   </g>
 </svg>

data/app/views/bible270/shared/_interaction_ui.html.erb

@@ -18,6 +18,53 @@
       region.dataset.state = state;
     }
 
+    const shownCompletionDoves = new Set();
+
+    function claimCompletionDove(dove) {
+      const eventId = dove.dataset.b270CompletionEvent;
+      if (!eventId) return false;
+
+      const key = `b270-completion-dove:${eventId}`;
+      if (shownCompletionDoves.has(key)) return false;
+
+      try {
+        if (window.sessionStorage.getItem(key) === "shown") return false;
+        window.sessionStorage.setItem(key, "shown");
+      } catch (_error) {
+        // Some privacy modes disable session storage. The in-memory set still
+        // prevents duplicate Turbo rendering within the current page.
+      }
+      shownCompletionDoves.add(key);
+      return true;
+    }
+
+    function initializeCompletionDove(dove) {
+      if (dove.dataset.b270DoveInitialized === "true") return;
+      dove.dataset.b270DoveInitialized = "true";
+
+      if (window.matchMedia("(prefers-reduced-motion: reduce)").matches) {
+        dove.remove();
+        return;
+      }
+
+      if (!claimCompletionDove(dove)) {
+        dove.remove();
+        return;
+      }
+
+      // Keep an active flight outside the Turbo-replaced progress panel so a
+      // replayed response cannot restart or cut short the same completion event.
+      if (dove.parentElement !== document.body) document.body.appendChild(dove);
+      requestAnimationFrame(() => {
+        if (dove.isConnected) dove.classList.add("b270-completion-dove-active");
+      });
+    }
+
+    function initializeCompletionDoves(root = document) {
+      if (root.matches && root.matches("[data-b270-completion-dove]")) initializeCompletionDove(root);
+      root.querySelectorAll?.("[data-b270-completion-dove]").forEach(initializeCompletionDove);
+    }
+
     function resetForm(form) {
       form.removeAttribute("aria-busy");
       form.dataset.b270Submitting = "false";
@@ -29,8 +76,18 @@
 
     function reset() {
       document.querySelectorAll("form[data-b270-submit='true']").forEach(resetForm);
+      initializeCompletionDoves();
     }
 
+    const completionDoveObserver = new MutationObserver((mutations) => {
+      mutations.forEach((mutation) => {
+        mutation.addedNodes.forEach((node) => {
+          if (node.nodeType === Node.ELEMENT_NODE) initializeCompletionDoves(node);
+        });
+      });
+    });
+    completionDoveObserver.observe(document.documentElement, { childList: true, subtree: true });
+
     const SESSION_REFRESH_AFTER = 5 * 60 * 1000;
     let sessionRefreshedAt = Date.now();
     let sessionRefresh = null;
@@ -201,6 +258,13 @@
       }
     });
     document.addEventListener("turbo:load", reset);
+    document.addEventListener("turbo:before-cache", () => {
+      document.querySelectorAll("[data-b270-completion-dove]").forEach((dove) => dove.remove());
+    });
+    document.addEventListener("animationend", (event) => {
+      if (event.animationName === "b270-dove-flight" &&
+          event.target.matches("[data-b270-completion-dove]")) event.target.remove();
+    });
 
     window.Bible270InteractionUI = { reset };
     reset();

data/app/views/bible270/shared/_styles.html.erb

@@ -147,17 +147,21 @@
   .b270-day-complete strong{font-family:"Spectral",serif;font-size:16px}
   .b270-day-complete span{font-size:12px;color:var(--muted)}
   .b270 .b270-day-complete .b270-btn{flex:none;margin:0 0 0 auto}
-  .b270-completion-dove{position:fixed;z-index:1000;top:var(--b270-dove-origin-y,72vh);left:var(--b270-dove-origin-x,50vw);
-                        width:min(75vw,97.5vh);height:auto;overflow:visible;pointer-events:none;opacity:0;
+  .b270-completion-dove{position:fixed;z-index:1000;top:72vh;left:50vw;
+                        width:min(80vw,104vh);height:auto;overflow:visible;pointer-events:none;opacity:0;
+                        color-scheme:only light;forced-color-adjust:none;
                         filter:drop-shadow(0 0 8px rgba(255,255,255,1)) drop-shadow(0 0 24px rgba(255,252,220,1)) drop-shadow(0 0 52px rgba(225,191,92,.82));
-                        will-change:top,left,transform,opacity;animation:b270-dove-flight 1.6s cubic-bezier(.18,.72,.28,1) both}
-  .b270-dove-silhouette{fill:url(#b270-dove-light);stroke:none}
+                        will-change:top,left,transform,opacity}
+  .b270-completion-dove-active{animation:b270-dove-flight 1.7s cubic-bezier(.18,.72,.28,1) both}
+  .b270-dove-silhouette{fill:url(#b270-dove-light)!important;stroke:none}
+  .b270-dove-light-center{stop-color:#fff!important}
+  .b270-dove-light-middle{stop-color:#fffef8!important}
+  .b270-dove-light-edge{stop-color:#fff9e2!important}
   @keyframes b270-dove-flight{
-    0%{top:var(--b270-dove-origin-y,72vh);left:var(--b270-dove-origin-x,50vw);opacity:.22;visibility:visible;transform:translate(-50%,-50%) scale(.02) rotate(-8deg)}
-    8%{top:calc(var(--b270-dove-origin-y,72vh) - 10px);left:var(--b270-dove-origin-x,50vw);opacity:.78;transform:translate(-50%,-50%) scale(.08) rotate(-5deg)}
-    64%{top:38vh;left:52vw;opacity:.92;transform:translate(-50%,-50%) scale(1) rotate(4deg)}
-    99%{visibility:visible}
-    100%{top:-25vh;left:115vw;opacity:0;visibility:hidden;transform:translate(-50%,-50%) scale(1.55) rotate(12deg)}
+    0%{top:72vh;left:50vw;opacity:.22;visibility:visible;transform:translate(-50%,-50%) scale(.02) rotate(-8deg)}
+    8%{top:calc(72vh - 10px);left:50vw;opacity:.78;transform:translate(-50%,-50%) scale(.08) rotate(-5deg)}
+    33%{top:50vh;left:50vw;opacity:1;visibility:visible;transform:translate(-50%,-50%) scale(1) rotate(4deg)}
+    100%{top:50vh;left:50vw;opacity:0;visibility:hidden;transform:translate(-50%,-50%) scale(1) rotate(4deg)}
   }
 
   /* readings */

data/config/routes.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+# Rails routes are intentionally kept in one cohesive DSL block.
 Bible270::Engine.routes.draw do
   root to: 'days#index'
 
@@ -22,6 +23,8 @@ Bible270::Engine.routes.draw do
   delete 'admin/readers/:id',            to: 'admin#destroy'
   patch  'admin/readers/:id/profile',    to: 'admin#update_profile', as: :admin_reader_profile
   patch  'admin/readers/:id/notifications', to: 'admin#update_notifications', as: :admin_reader_notifications
+  patch  'admin/readers/:id/completion-animation', to: 'admin#update_completion_animation',
+                                                   as: :admin_reader_completion_animation
   delete 'admin/readers/:id/avatar',     to: 'admin#remove_avatar', as: :admin_reader_avatar
   patch  'admin/readers/:id/version',    to: 'admin#update_bible_version', as: :admin_reader_version
   patch  'admin/readers/:id/start',      to: 'admin#update_start',     as: :admin_reader_start

data/db/migrate/20260101000020_add_completion_dove_disabled_to_bible270_readers.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddCompletionDoveDisabledToBible270Readers < ActiveRecord::Migration[7.0]
+  def change
+    add_column :bible270_readers, :completion_dove_disabled, :boolean, null: false, default: false
+  end
+end

data/lib/bible270/migration_reconciler.rb

@@ -29,6 +29,7 @@ module Bible270
       'AddReflectionsSeenAtToBible270Readers' => :check_reflections_seen_at,
       'AddAllCommentNoticesToBible270Readers' => :check_all_comment_notices,
       'ChangePassageSourceDefaultToBibleCom' => :check_passage_source_default,
+      'AddCompletionDoveDisabledToBible270Readers' => :check_completion_dove_disabled,
       'CreateActiveStorageTables' => :check_active_storage
     }.freeze
 
@@ -204,6 +205,10 @@ module Bible270
       column&.default == 'bible_com' ? [] : ['column bible270_readers.passage_source default bible_com']
     end
 
+    def check_completion_dove_disabled
+      table_missing(:bible270_readers, columns: %i[completion_dove_disabled])
+    end
+
     # The Bible270 installer can install Active Storage for reader avatars. A
     # copied database can therefore have the same timestamp mismatch for Rails'
     # generated Active Storage migration as it has for the engine migrations.

data/lib/bible270/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bible270
-  VERSION = '1.4.0'
+  VERSION = '1.4.2'
 end

data/lib/generators/bible270/install/install_generator.rb

@@ -18,10 +18,12 @@ module Bible270
     #     --mailer-from=no-reply@example.org
     #
     class InstallGenerator < Rails::Generators::Base
-      MOUNT_LINE        = 'mount Bible270::Engine, at: Bible270.config.mount_at'
-      DEFAULT_MOUNT     = '/daily-bread'
-      DEFAULT_PROVIDERS = 'github'
-      KNOWN_PROVIDERS   = {
+      MOUNT_LINE          = 'mount Bible270::Engine, at: Bible270.config.mount_at'
+      MOUNT_LINE_PATTERN  = %r{^\s*mount\s*\(?\s*Bible270::Engine\b}
+      PROVIDER_IDENTIFIER = %r{\A[a-z][a-z0-9_]*\z}
+      DEFAULT_MOUNT       = '/daily-bread'
+      DEFAULT_PROVIDERS   = 'github'
+      KNOWN_PROVIDERS = {
         'github' => 'omniauth-github',
         'google_oauth2' => 'omniauth-google-oauth2',
         'facebook' => 'omniauth-facebook',
@@ -97,18 +99,23 @@ module Bible270
       def add_strategy_gems
         return if @providers.empty?
 
-        missing = @providers.filter_map { |p| KNOWN_PROVIDERS[p] }.reject { |g| gemfile_has?(g) }
-        unknown = @providers.reject { |p| KNOWN_PROVIDERS.key?(p) }
+        missing = @providers.filter_map { |provider| KNOWN_PROVIDERS[provider] }
+          .reject { |gem_name| gemfile_has?(gem_name) }
+        @unknown_providers = @providers.reject { |provider| KNOWN_PROVIDERS.key?(provider) }
+        @strategy_gems_added = missing.any?
 
         missing.each { |name| gem name }
 
-        return if unknown.empty?
+        return if @unknown_providers.empty?
 
-        say "  Unrecognised provider(s): #{unknown.join(', ')}. Add their strategy gems yourself.", :yellow
+        identifiers = @unknown_providers.map { |provider| provider.to_sym.inspect }.join(', ')
+        say "  Unrecognised provider(s): #{@unknown_providers.join(', ')}.", :yellow
+        say "  Add an OmniAuth strategy gem that registers #{identifiers} to the Gemfile, then run bundle install.",
+            :yellow
       end
 
       def install_bundle
-        return if @providers.empty?
+        return unless @strategy_gems_added
 
         @bundled = decide(:bundle, 'Run bundle install now (needed before the app can boot)?', true)
         run 'bundle install' if @bundled
@@ -121,7 +128,16 @@ module Bible270
       # rather than assuming a fresh app has it.
 
       def ensure_active_storage
-        return if skip_rails_commands?
+        if skip_rails_commands?
+          @active_storage = if !active_storage_enabled?
+                              :unavailable
+                            elsif active_storage_installed?
+                              :present
+                            else
+                              :pending
+                            end
+          return
+        end
 
         unless active_storage_enabled?
           @active_storage = :unavailable
@@ -153,11 +169,15 @@ module Bible270
       # Migrations run before the initializers are written, so a problem in the
       # generated config can never block the database work.
       def copy_migrations
-        return if skip_rails_commands?
+        if skip_rails_commands?
+          @migrations_copied = false
+          return
+        end
 
         say ''
         say 'Copying migrations into db/migrate', :yellow
         rails_command 'bible270:install:migrations'
+        @migrations_copied = true
       end
 
       def run_migrations
@@ -191,7 +211,11 @@ module Bible270
 
       def add_route
         if routes_mounted?
-          say "  config/routes.rb already mounts Bible270::Engine — left as is.#{cms_order_hint}", :yellow
+          if move_mount_above_catch_all
+            say '  config/routes.rb already mounted Bible270::Engine; moved it above the catch-all route.', :yellow
+          else
+            say "  config/routes.rb already mounts Bible270::Engine — left as is.#{cms_order_hint}", :yellow
+          end
           return
         end
 
@@ -202,7 +226,9 @@ module Bible270
         # across files, an unusual layout, a missing file). Verify rather than
         # assume, since a silent miss leaves the plan unreachable.
         if routes_mounted?
-          say "  Mounted in config/routes.rb.#{cms_order_hint}", :green
+          moved = move_mount_above_catch_all
+          detail = moved ? ' Mounted above the catch-all route.' : cms_order_hint
+          say "  Mounted in config/routes.rb.#{detail}", :green
         else
           @route_failed = true
           say '  Could not edit config/routes.rb automatically.', :red
@@ -220,7 +246,7 @@ module Bible270
         say "  Plan mounted at   #{@mount_at}"
         say "  Sign-in           #{auth_summary}"
         say "  Initializers      #{initializer_summary}"
-        say "  Migrations        #{@migrated ? 'copied and run' : 'copied (not yet run)'}"
+        say "  Migrations        #{migration_summary}"
         say "  Picture uploads   #{active_storage_summary}"
         say ''
 
@@ -241,16 +267,13 @@ module Bible270
       # bin/rails call fail ("Could not find ... in locally installed gems"), so
       # don't even try.
       def skip_rails_commands?
-        return false unless @providers.any? && @bundled == false
+        return false unless @strategy_gems_added && @bundled == false
         return true if @warned_unbundled
 
         @warned_unbundled = true
         say ''
         say '  Gemfile changed but not bundled, so bin/rails cannot run yet.', :yellow
-        say '  Skipping the migration steps — run these once you have bundled:', :yellow
-        say '    bundle install'
-        say '    bin/rails bible270:install:migrations'
-        say '    bin/rails db:migrate'
+        say '  Skipping Active Storage and migration commands until bundle install succeeds.', :yellow
         true
       end
 
@@ -320,10 +343,10 @@ module Bible270
 
       def ask_mailer_from
         default = "no-reply@#{app_domain}"
-        address = prompt('From: address for sign-in emails?', default)
+        address = options[:mailer_from] || prompt('From: address for sign-in emails?', default)
 
         until valid_email?(address)
-          say "  '#{address}' doesn't look like an email address.", :red
+          say "  #{address.inspect} doesn't look like an email address.", :red
           address = prompt('From: address for sign-in emails?', default)
         end
         address
@@ -352,7 +375,15 @@ module Bible270
       end
 
       def parse_providers(value)
-        value.to_s.split(',').map { |p| p.strip.downcase.tr('-', '_') }.reject(&:empty?).uniq
+        providers = value.to_s.split(',').map { |provider| provider.strip.downcase.tr('-', '_') }
+          .reject(&:empty?).uniq
+        invalid = providers.grep_v(PROVIDER_IDENTIFIER)
+        if invalid.any?
+          say "  Ignoring unsafe provider identifier(s): #{invalid.join(', ')}.", :red
+          say '  Provider names must start with a letter and contain only lowercase letters, numbers, and underscores.',
+              :red
+        end
+        providers - invalid
       end
 
       def app_domain
@@ -363,7 +394,11 @@ module Bible270
       end
 
       def providers_literal
-        @providers.map { |p| ":#{p}" }.join(', ')
+        @providers.map { |provider| provider.to_sym.inspect }.join(', ')
+      end
+
+      def ruby_literal(value)
+        value.to_s.dump
       end
 
       def auth_path_prefix
@@ -382,15 +417,40 @@ module Bible270
         path = File.join(destination_root, 'config', 'routes.rb')
         return '' unless File.exist?(path)
 
-        body = File.read(path)
-        return '' unless body.match?(%r{comfy_route|match\s+['"]\*|get\s+['"]\*})
+        lines = File.readlines(path)
+        mount_index = lines.index { |line| line.match?(MOUNT_LINE_PATTERN) }
+        catch_all_index = catch_all_route_index(lines)
+        return '' unless catch_all_index && (!mount_index || mount_index > catch_all_index)
 
-        ' Check the mount line sits ABOVE any catch-all route.'
+        ' Move the mount line ABOVE the catch-all route.'
+      end
+
+      def move_mount_above_catch_all
+        path = File.join(destination_root, 'config', 'routes.rb')
+        lines = File.readlines(path)
+        mount_index = lines.index { |line| line.match?(MOUNT_LINE_PATTERN) }
+        catch_all_index = catch_all_route_index(lines)
+        return false unless mount_index && catch_all_index && mount_index > catch_all_index
+
+        mount_line = lines.delete_at(mount_index)
+        lines.insert(catch_all_index, mount_line)
+        File.write(path, lines.join)
+        true
+      rescue StandardError
+        false
+      end
+
+      def catch_all_route_index(lines)
+        lines.index do |line|
+          code = line.sub(%r{#.*\z}, '')
+          code.match?(%r{\bcomfy_route\s+:cms\b.*\bpath:\s*['"]/['"]}) ||
+            code.match?(%r{\b(?:get|match)\s*\(?\s*['"]\*})
+        end
       end
 
       def routes_mounted?
         path = File.join(destination_root, 'config', 'routes.rb')
-        File.exist?(path) && File.read(path).include?('Bible270::Engine')
+        File.exist?(path) && File.foreach(path).any? { |line| line.match?(MOUNT_LINE_PATTERN) }
       end
 
       def report_auth_choice
@@ -434,17 +494,30 @@ module Bible270
         body.include?('path_prefix') && !body.include?('Bible270.config.auth_path_prefix')
       end
 
+      def migration_summary
+        if @migrations_copied
+          @migrated ? 'copied and run' : 'copied (not yet run)'
+        else
+          'not copied (not run)'
+        end
+      end
+
       def active_storage_summary
         case @active_storage
         when :installed then 'Active Storage installed'
         when :present then 'Active Storage already present'
         when :declined then 'off — Active Storage not installed'
         when :unavailable then 'off — Active Storage not enabled in this app'
+        when :pending then 'not installed — waiting for bundle install'
         else 'unchanged'
         end
       end
 
       def outstanding_steps
+        route_and_initializer_steps + installation_steps + provider_setup_steps + mailer_steps
+      end
+
+      def route_and_initializer_steps
         steps = []
         steps << "Add to config/routes.rb: #{MOUNT_LINE}" if @route_failed
         if @providers.any? && !initializer_present?('omniauth.rb')
@@ -453,22 +526,36 @@ module Bible270
         if stale_omniauth_prefix?
           steps << "config/initializers/omniauth.rb sets its own path_prefix — make it 'Bible270.config.auth_path_prefix'"
         end
-        steps << 'bundle install' if @providers.any? && !@bundled
-        steps << 'bin/rails bible270:install:migrations' if skip_rails_commands?
+        steps
+      end
+
+      def installation_steps
+        steps = []
+        steps << 'bundle install' if @strategy_gems_added && !@bundled
+        steps << unknown_provider_gem_step if @unknown_providers&.any?
+        steps << 'bin/rails active_storage:install' if @active_storage == :pending
+        steps << 'bin/rails bible270:install:migrations' unless @migrations_copied
         steps << 'bin/rails db:migrate' unless @migrated
         if @active_storage == :declined
           steps << 'bin/rails active_storage:install && bin/rails db:migrate — for picture uploads'
         end
+        steps
+      end
 
-        if @providers.any?
-          steps << 'Set each provider\'s client id/secret in credentials or ENV'
-          @providers.each { |p| steps << "Register callback URL: https://YOUR-HOST#{auth_path_prefix}/#{p}/callback" }
-        end
+      def unknown_provider_gem_step
+        providers = @unknown_providers.map { |provider| provider.to_sym.inspect }.join(', ')
+        "Add an OmniAuth strategy gem registering #{providers} to Gemfile, then run bundle install"
+      end
 
-        if @email
-          steps << 'Confirm Action Mailer can deliver mail in this environment'
-        end
-        steps
+      def provider_setup_steps
+        return [] if @providers.empty?
+
+        ['Set each provider\'s client id/secret in credentials or ENV'] +
+          @providers.map { |provider| "Register callback URL: https://YOUR-HOST#{auth_path_prefix}/#{provider}/callback" }
+      end
+
+      def mailer_steps
+        @email ? ['Confirm Action Mailer can deliver mail in this environment'] : []
       end
 
       # ---- file bodies ------------------------------------------------------
@@ -489,7 +576,7 @@ module Bible270
             # Change this one value to move the plan. config/routes.rb and
             # config/initializers/omniauth.rb both read it, so nothing else needs editing
             # (but do update the callback URLs registered with each OAuth provider).
-            config.mount_at = '#{@mount_at}'
+            config.mount_at = #{ruby_literal(@mount_at)}
 
             config.app_name = 'Daily Bread'
             config.tagline  = 'A 270-day journey through Scripture'
@@ -532,7 +619,7 @@ module Bible270
             # Passwordless email sign-in: a one-time link, no password, no third-party
             # account. This is what lets anyone take part. Needs Action Mailer delivery.
             config.email_sign_in = true
-            config.mailer_from   = '#{@mailer_from}'
+            config.mailer_from   = #{ruby_literal(@mailer_from)}
             config.email_sign_in_ask_name = true
             # config.email_sign_in_ttl = 20 * 60             # link lifetime, seconds
             # config.email_sign_in_max_per_window = 5        # per address, per window
@@ -607,11 +694,12 @@ module Bible270
       end
 
       def provider_line(provider)
+        provider_symbol = provider.to_sym.inspect
         env = provider.upcase
         <<~RUBY.chomp
-          provider :#{provider},
-                   Rails.application.credentials.dig(:#{provider}, :client_id) || ENV.fetch('#{env}_CLIENT_ID', nil),
-                   Rails.application.credentials.dig(:#{provider}, :client_secret) || ENV.fetch('#{env}_CLIENT_SECRET', nil)
+          provider #{provider_symbol},
+                   Rails.application.credentials.dig(#{provider_symbol}, :client_id) || ENV.fetch(#{ruby_literal("#{env}_CLIENT_ID")}, nil),
+                   Rails.application.credentials.dig(#{provider_symbol}, :client_secret) || ENV.fetch(#{ruby_literal("#{env}_CLIENT_SECRET")}, nil)
         RUBY
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bible270
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.4.2
 platform: ruby
 authors:
 - Andrew vonderLuft
@@ -139,7 +139,6 @@ files:
 - app/views/bible270/shared/_comment_drafts.html.erb
 - app/views/bible270/shared/_comment_notification_choices.html.erb
 - app/views/bible270/shared/_day_completion_dove.html.erb
-- app/views/bible270/shared/_day_completion_effect.html.erb
 - app/views/bible270/shared/_day_index.html.erb
 - app/views/bible270/shared/_footer.html.erb
 - app/views/bible270/shared/_grid_legend.html.erb
@@ -176,6 +175,7 @@ files:
 - db/migrate/20260101000017_add_reflections_seen_at_to_bible270_readers.rb
 - db/migrate/20260101000018_add_all_comment_notices_to_bible270_readers.rb
 - db/migrate/20260101000019_change_passage_source_default_to_bible_com.rb
+- db/migrate/20260101000020_add_completion_dove_disabled_to_bible270_readers.rb
 - lib/bible270.rb
 - lib/bible270/avatars.rb
 - lib/bible270/configuration.rb

data/app/views/bible270/shared/_day_completion_effect.html.erb

@@ -1,67 +0,0 @@
-<%# locals: (day:) %>
-<template data-b270-completion-dove-template data-b270-day="<%= day %>">
-  <%= render "bible270/shared/day_completion_dove", day: day %>
-</template>
-<script nonce="<%= content_security_policy_nonce %>">
-  (() => {
-    if (window.Bible270DayCompletionEffect) return;
-
-    function willCompleteDay(form) {
-      const desiredState = form.querySelector("input[name='checked']");
-      if (!desiredState || desiredState.value !== "1") return false;
-
-      const unfinished = Array.from(
-        document.querySelectorAll("form[data-b270-checkoff='true'] input[name='checked'][value='1']")
-      );
-      return unfinished.length === 1 && unfinished[0].form === form;
-    }
-
-    function startDove(day) {
-      if (document.querySelector("[data-b270-completion-dove][data-b270-immediate]")) return;
-
-      const template = document.querySelector("[data-b270-completion-dove-template]");
-      if (!template || template.dataset.b270Day !== day) return;
-
-      const dove = template.content.querySelector("[data-b270-completion-dove]").cloneNode(true);
-      dove.dataset.b270Immediate = "true";
-      document.body.append(dove);
-    }
-
-    function removeServerDuplicate() {
-      const immediate = document.querySelector("[data-b270-completion-dove][data-b270-immediate]");
-      if (!immediate) return;
-
-      document.querySelectorAll("[data-b270-completion-dove]").forEach((dove) => {
-        if (dove !== immediate) dove.remove();
-      });
-    }
-
-    document.addEventListener("submit", (event) => {
-      const form = event.target.closest("form[data-b270-checkoff='true']");
-      const check = form && form.querySelector(".b270-check");
-      if (!check) return;
-
-      const bounds = check.getBoundingClientRect();
-      document.documentElement.style.setProperty(
-        "--b270-dove-origin-x",
-        `${bounds.left + bounds.width / 2}px`
-      );
-      document.documentElement.style.setProperty(
-        "--b270-dove-origin-y",
-        `${bounds.top + bounds.height / 2}px`
-      );
-
-      if (willCompleteDay(form)) startDove(form.action.match(/\/day\/(\d+)/)?.[1] || "");
-    });
-
-    document.addEventListener("animationend", (event) => {
-      if (event.target.matches("[data-b270-completion-dove]")) event.target.remove();
-    });
-
-    new MutationObserver(removeServerDuplicate).observe(document.documentElement, {
-      childList: true,
-      subtree: true
-    });
-    window.Bible270DayCompletionEffect = true;
-  })();
-</script>