bible270 0.19.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4e5f2002e8da19945110a09f27e658b7de650e8d63e4d0cc429e0c9c74bf6bf6
-  data.tar.gz: '09e8a0851405b78ccb8bd39c2938546afe3bf2cf47cfe8e693d000f60713a16f'
+  metadata.gz: 48e80937debff292b2c896ae9eaba55fe5eae8624ef28c6eb9274236fed4fb2c
+  data.tar.gz: 72f8fbd17194fd00bbc8602fd69180cb91565d54f995868dacf8edfb4419657f
 SHA512:
-  metadata.gz: 516d821a92220f758959fe337d4cb2591ffe7785102f1c3e6d230ae31d801175b563d2b004dad1f77829d7fbc19cce1d5f87cb1f13a96405d77b81c0d2de7e1a
-  data.tar.gz: 27ba2c3d4f110bd64e8fc26a40b03e98e7b2f512cffd13f79faf267d35ba32f69e03834ec1dd678696941adc5c35ad11e804b7a6547bbeddac7561ed02e8a47f
+  metadata.gz: b493f52caadc3fa2bbb7e3ef4dd59c9822d0045b9fb6f42151f4fad0c9057d017b7604064188e35a3dce03c8c6c3c02cabce8a95a342e84e6fbf5a2682a46309
+  data.tar.gz: a38f6e3b979b76ccaf5c666e3698d29302ac97b82692f7c2a33ac1ef0d28f139b28ddaf89705e4a13c132713533edd87c8e804eff06974c089ad2bb8b1322c0b

data/CHANGELOG.md

@@ -4,9 +4,71 @@ All notable changes to bible270. Format follows [Keep a Changelog](https://keepa
 
 ## Unreleased Changes
 
-See the changes since the last release:
+### Removed
+
+- remove the Compact reading preference so daily reading pages have one simple, predictable presentation
+- replace the homepage reader preview with one clear link to the Community page
+- remove the redundant schedule-restart shortcut from progress recovery choices
+- remove reader-facing start-date status, controls, and self-service endpoints while retaining automatic and admin-managed personal calendars
+- remove weekly progress, its unused schedule helper, milestones, and explanatory schedule-recovery copy from My Progress
+- replace the Reflections page’s separate day grid with the standard “View all 270 days” control
+
+### Changed
+
+- notify reflection authors directly about replies, share one opt-out with explicit mentions, and stop inserting handles into reply text
+- consolidate duplicate visitor sign-in prompts on day pages into one invitation
+- show “Finish what you started” on My Progress only when partially read days need attention
+- clarify the global day-index control as “View all 270 days” and use that standard collapsed control on My Progress
+
+### Fixed
+
+- make the mobile navigation menu reliably open and close when its button is tapped
+
+**[Full Changelog](https://github.com/avonderluft/bible270/compare/v1.0.0...main)**
+
+## [1.0.0] - 2026-08-20
+
+### Added
+
+- optional daily reading reminders with per-reader opt-in and delivery time, reminder emails, and a scheduler-friendly Rake task
+- immediate day-completion feedback with a preserved next-day or Progress action
+- actionable Progress sections for unfinished portions, the current calendar week, milestones, and schedule recovery without losing reading or reflection history
+- browser-local reflection and reply drafts, day and author filters, pagination, and new-activity markers on the Reflections page
+- a remembered compact reading mode that keeps passages, checkoffs, navigation, and completion next steps in focus
+- a two-worker default `test` task with real-time colorized progress, one aggregate result and duration, merged coverage, a browsable project-local `coverage/` report, and per-file runtime recording, plus `test:serial` when a serial run is needed
+
+### Changed
+
+- replace progress-ranked community lists with transparent recent-participation and alphabetical views that emphasize reading together
+- show why each reader appears in Reading together, including their latest reading, reflection, or join activity
+- add consistent pending, success, and retry feedback to reading, like, reflection, and reply actions, with repeat-safe checkoff and like requests
+- improve accessibility with AA text contrast, consistent focus indicators, descriptive day-grid labels, semantic boundary navigation, touch-friendly tooltips, responsive forms, and reduced-motion support
+- replace the reminder time field with aligned hour and minute selectors and an accessible time-zone tooltip
+- hide personal start-date controls and calendar prompts when the community configuration does not allow readers to change dates
+
+### Fixed
+
+- keep profiles and scheduled reminder jobs available while a newly installed daily-reminder database migration is pending
+
+**[Full Changelog](https://github.com/avonderluft/bible270/compare/v0.19.0...v1.0.0)**
+
+## [0.19.0] - 2026-08-17
+
+### Added
+
+- shared responsive navigation with clearer Community, Reflections, My Progress, and Profile labels plus a mobile menu
+
+### Changed
+
+- align page headings with navigation destinations and clarify the Start or Continue reading and Go to today actions
+- always email explicitly mentioned readers, including the original reflection author when a reply prefills their mention
+- normalize packaged file permissions before gem builds
+
+### Removed
+
+- the per-reader mention-email opt-out; writing a reader's name explicitly now always sends the mention notice
 
-**[Full Changelog](https://github.com/avonderluft/bible270/compare/v0.18.2...main)**
+**[Full Changelog](https://github.com/avonderluft/bible270/compare/v0.18.2...v0.19.0)**
 
 ## [0.18.2] - 2026-08-15
 
@@ -426,6 +488,8 @@ 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.0.0]: https://github.com/avonderluft/bible270/releases/tag/v1.0.0
+[0.19.0]: https://github.com/avonderluft/bible270/releases/tag/v0.19.0
 [0.18.2]: https://github.com/avonderluft/bible270/releases/tag/v0.18.2
 [0.18.1]: https://github.com/avonderluft/bible270/releases/tag/v0.18.1
 [0.18.0]: https://github.com/avonderluft/bible270/releases/tag/v0.18.0

data/README.md

@@ -24,6 +24,10 @@ The schedule is pure deterministic Ruby (`Bible270::Plan`) — none of it is sto
 - **Rails** >= 7.0. On Ruby 4.0 your host app is the real constraint, not this engine — check your lockfile with RailsBump first.
 - A host app, and Turbo if you want check-offs without a page reload.
 
+Mutation controls expose pending, success, and retry feedback when JavaScript runs, while ordinary HTML
+forms remain the fallback. Repeated checkoff and like requests converge on the intended state rather
+than accidentally reversing it.
+
 ## Install
 
 ```ruby
@@ -272,6 +276,19 @@ Admins get an extra "Admin" link in the header. From a reader's page you can:
   written; hiding one takes it off the day and community pages without deleting it, and is
   reversible. Delete is there too, for anything that shouldn't be kept.
 
+The Reflections page lists conversations by their latest approved reply, supports day and author
+filters, and paginates older threads. Signed-in readers see which conversations became active since
+their previous visit. Reflection and reply drafts are retained for up to 30 days in that browser and
+cleared after a successful post. On a shared device, clearing browser storage also removes saved drafts.
+
+After upgrading, copy and run migration `20260101000017`, which stores each reader's last visit to the
+Reflections page:
+
+```bash
+bin/rails bible270:install:migrations
+bin/rails db:migrate
+```
+
 Break points can also live in your app rather than the gem, and be changed without a restart:
 
 ```ruby
@@ -291,17 +308,17 @@ The plan runs **undated** (day numbers only) or **dated** (days mapped onto a ca
 
 ```ruby
 config.start_date = Date.new(2026, 9, 6)   # Date, Time, or "YYYY-MM-DD". nil = undated
-config.allow_reader_start_date = true      # can readers set their own?
+config.allow_reader_start_date = true      # allow personal per-reader calendars
 ```
 
 | `start_date` | `allow_reader_start_date` | What happens |
 |---|---|---|
-| nil | true *(default)* | Each reader gets stamped on first check-off, and can change it |
-| a date | true | Community date is the default; anyone may override |
-| a date | false | Everyone pinned to the community date, form hidden |
+| nil | true *(default)* | Each reader gets stamped on their first check-off; an admin may adjust the date |
+| a date | true | The community date is the default; an admin may assign an individual date |
+| a date | false | Everyone is pinned to the community date; stored individual dates are ignored |
 | nil | false | Fully undated — no calendar anywhere |
 
-Dated plans show each day's date with a **Today** badge, a "Go to today" link, and whether you're ahead or behind. Readers set their own from the overview. Changing a start date only re-maps the calendar — check-offs and reflections are keyed to day numbers, so nothing moves.
+Dated plans show each day's date with a **Today** badge and a "Go to today" link. Administrators may re-map an individual reader's calendar from the admin panel; check-offs and reflections remain keyed to day numbers, so nothing in their history moves.
 
 Date helpers are pure functions, usable anywhere:
 
@@ -313,6 +330,38 @@ Bible270::Plan.day_for(date, start, clamp: false) # => -3 or 271, to spot out-of
 Bible270::Plan.before_start?(date, start)
 ```
 
+## Daily reading reminders
+
+Daily reminders are optional at both levels: the host enables the feature, then each reader chooses
+whether to receive it and at what local time from their profile. The feature and reader opt-in both
+default to off; each reader's time defaults to `08:00`.
+
+```ruby
+# config/initializers/bible270.rb
+Bible270.configure do |config|
+  config.daily_reminders = true
+  config.mailer_from = "no-reply@yourdomain.com"
+  config.mailer_host = "yourdomain.com" # builds links back to the day and profile
+end
+```
+
+After upgrading, copy and run migration `20260101000016`, which adds the reader opt-in, preferred
+time, and last-sent date. Bible270 does not install a scheduler. The host application should run this
+task periodically—every 15 minutes is recommended—using its existing cron, platform scheduler, or
+recurring-job system:
+
+```bash
+bin/rails bible270:reminders:send
+```
+
+Each run converts `Time.now` through `Bible270.local_time`, waits until each reader's chosen `HH:MM`
+time, sends inline with `deliver_now`, and prints the number delivered. Set `config.time_zone` when the
+plan should not follow the server's local clock. Running periodically is safe because delivery is
+idempotent once per local date: readers already marked for that date are skipped. Undated readers,
+calendar dates before or after their plan, completed days, blank email addresses, and readers who have
+not opted in are also skipped. A failure for one reader is logged and does not stop the rest. The sent
+date is recorded only after delivery succeeds.
+
 ## Configuration reference
 
 ```ruby
@@ -326,6 +375,10 @@ Bible270.configure do |config|
   # config.start_date = Date.new(2026, 9, 6)  
   config.allow_reader_start_date = true
 
+  config.daily_reminders = false             # schedule reminders:send every 15 minutes
+  config.mention_notifications = true         # replies and @mentions; readers may opt out
+  config.mailer_host = "example.com"          # links in reminder/notice email
+
   config.parent_controller = "ActionController::Base"   # or "::ApplicationController"
   config.layout            = "bible270/application"
 
@@ -397,7 +450,7 @@ The CMS serves your content; this is a separate mounted app. Either **just link
 
 | Table | Holds |
 |-------|-------|
-| `bible270_readers` | identity, avatar, provider/uid or polymorphic `owner`, start date, translation and passage-reader preferences |
+| `bible270_readers` | identity, avatar, provider/uid or polymorphic `owner`, start date, translation, passage-reader, and reminder preferences |
 | `bible270_checkoffs` | one row per reader per day per track (`ot`/`nt`/`pp`), unique-indexed |
 | `bible270_comments` | a reflection on a day, optionally scoped to a track, public to all |
 | `bible270_sign_in_tokens` | magic-link tokens: email, **digest only**, expiry, consumed-at |
@@ -413,8 +466,6 @@ GET    /day/:day                 days#show       readings, who's finished, refle
 POST   /day/:day/toggle/:track   checkoffs#toggle
 POST   /day/:day/comments        comments#create
 DELETE /comments/:id             comments#destroy
-PATCH  /start-date               readers#update_start_date
-DELETE /start-date               readers#clear_start_date
 GET    /community                readers#index   leaderboard
 GET    /readers/:id              readers#show
 GET    /sign_in                  sessions#new    email form + provider list
@@ -471,7 +522,7 @@ All computed at load and memoized — no generated schedule, so nothing to migra
 
 ## Scaling
 
-The leaderboard counts completed days in Ruby from grouped check-offs — fine for hundreds of readers. Past a few thousand, cache a `days_completed` counter on `Reader` via a `Checkoff` `after_commit` and sort in SQL.
+Community pages count completed days in Ruby from grouped check-offs — fine for hundreds of readers. Past a few thousand, cache a `days_completed` counter on `Reader` via a `Checkoff` `after_commit` and read it directly in SQL.
 
 ## Troubleshooting
 
@@ -499,9 +550,18 @@ The engine's CSS rides in a partial its own layout renders. Using your layout? A
 
 ```bash
 bundle install
-rake test
+bundle exec rake test
+bundle exec rubocop --cache false
 ```
 
+`rake test` runs two isolated processes, streams colorized progress in real time, then prints one
+consolidated result and duration. It merges their SimpleCov results, writes a browsable report to
+`coverage/index.html`, and
+records per-file runtimes in `tmp/parallel_runtime_test.log` to improve
+subsequent balancing. Set `PARALLEL_WORKERS` to tune the worker count. Use
+`bundle exec rake test:serial` when a serial coverage run is specifically needed; it writes to the same
+project-local `coverage/` directory.
+
 ### Get an email link for local testing
 
 in the rails console: `rails c`
@@ -518,7 +578,7 @@ Bug reports and pull requests are welcome at <https://github.com/avonderluft/bib
 
 1. Fork it and clone your fork.
 2. Branch off `main`: `git checkout -b my-feature`.
-3. Make your change, and add tests — `rake test` should stay green.
+3. Make your change, and add tests — `bundle exec rake test` and `bundle exec rubocop --cache false` should stay green.
 4. Commit with a clear message: `git commit -am "Add my feature"`.
 5. Push: `git push origin my-feature`.
 6. Open a pull request against `main`, saying what changed and why.

data/Rakefile

@@ -1,7 +1,138 @@
 # frozen_string_literal: true
 
 require 'bundler/gem_tasks' # provides build / install / release
+require 'open3'
 require 'rake/testtask'
+require 'stringio'
+
+module Bible270TestOutput
+  SUMMARY = %r{(?<assertions>\d+) assertions, (?<errors>\d+) errors, (?<failures>\d+) failures, (?<skips>\d+) skips?, (?<tests>\d+) tests}
+  COLORS = { green: 32, yellow: 33, red: 31, cyan: 36 }.freeze
+
+  class Progress
+    def initialize(io)
+      @io = io
+      @line = +''
+      @pending = +''
+      @awaiting_blank = false
+      @expecting_progress = false
+      @printing_progress = false
+    end
+
+    def feed(chunk)
+      chunk.each_char { |character| consume(character) }
+      flush
+    end
+
+    def finish
+      flush
+      @io.puts if @printing_progress
+    end
+
+  private
+
+    def consume(character)
+      return if character == "\r"
+      return finish_line if character == "\n"
+
+      if @expecting_progress && character.match?(%r{[.EFS]})
+        @printing_progress = true
+        @pending << character
+      elsif @printing_progress
+        flush
+        @io.puts
+        @printing_progress = false
+        @expecting_progress = false
+        @line = +character
+      else
+        @expecting_progress = false if @expecting_progress
+        @line << character
+      end
+    end
+
+    def finish_line
+      flush
+      if @printing_progress
+        @io.puts
+        @expecting_progress = false
+      elsif @line.start_with?('# Running tests')
+        @awaiting_blank = true
+      elsif @awaiting_blank && @line.empty?
+        @expecting_progress = true
+        @awaiting_blank = false
+      end
+      @line.clear
+      @printing_progress = false
+    end
+
+    def flush
+      return if @pending.empty?
+
+      @io.print Bible270TestOutput.progress_color(@pending)
+      @io.flush
+      @pending.clear
+    end
+  end
+
+module_function
+
+  def capture(env, command)
+    output = +''
+    errors = +''
+    status = nil
+    started_at = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+    progress = Progress.new($stdout)
+
+    Open3.popen3(env, *command) do |stdin, stdout, stderr, wait_thread|
+      stdin.close
+      error_reader = Thread.new { stderr.read }
+      loop do
+        chunk = stdout.readpartial(4096)
+        output << chunk
+        progress.feed(chunk)
+      end
+    rescue EOFError
+      errors = error_reader.value
+      status = wait_thread.value
+    ensure
+      progress.finish
+    end
+
+    elapsed = Process.clock_gettime(Process::CLOCK_MONOTONIC) - started_at
+    [output, errors, status, elapsed]
+  end
+
+  def color(text, name)
+    return text unless $stdout.tty? && !ENV.key?('NO_COLOR')
+
+    "\e[#{COLORS.fetch(name)}m#{text}\e[0m"
+  end
+
+  def progress_color(text)
+    return text unless $stdout.tty? && !ENV.key?('NO_COLOR')
+
+    highlighted = text.gsub('S', "\e[33mS\e[32m")
+      .gsub(%r{[EF]}) { |character| "\e[31m#{character}\e[32m" }
+    "\e[32m#{highlighted}\e[0m"
+  end
+
+  def summary(output)
+    counts = output.match(SUMMARY)
+    raise 'Could not read the parallel test summary' unless counts
+
+    "#{counts[:tests]} tests, #{counts[:assertions]} assertions, " \
+      "#{counts[:failures]} failures, #{counts[:errors]} errors, " \
+      "#{counts[:skips]} #{counts[:skips] == '1' ? 'skip' : 'skips'}"
+  end
+
+  def quietly
+    previous = $stdout
+    $stdout = StringIO.new
+    yield
+  ensure
+    $stdout = previous
+  end
+end
 
 desc 'Normalize permissions for files included in the gem'
 task :normalize_gem_permissions do
@@ -13,10 +144,53 @@ end
 
 Rake::Task[:build].enhance([:normalize_gem_permissions])
 
-Rake::TestTask.new(:test) do |t|
-  t.libs << 'test' << 'lib'
-  t.pattern = 'test/**/*_test.rb'
-  t.warning = false
+desc 'Run tests in isolated processes (default: 2 workers)'
+namespace :test do
+  task :prepare_serial do
+    FileUtils.rm_rf('coverage')
+    ENV['TMPDIR'] = ENV.fetch('BIBLE270_TEST_TMPDIR', '/tmp')
+  end
+
+  desc 'Run tests serially with coverage'
+  Rake::TestTask.new(:serial) do |t|
+    t.libs << 'test' << 'lib'
+    t.pattern = 'test/**/*_test.rb'
+    t.warning = false
+  end
+  Rake::Task['test:serial'].enhance(['test:prepare_serial'])
+
+  task :parallel do
+    workers = ENV.fetch('PARALLEL_WORKERS', '2')
+    FileUtils.rm_rf('coverage')
+    env = {
+      'PARALLEL_COVERAGE' => 'true',
+      'RECORD_RUNTIME' => 'true',
+      'SKIP_COV' => nil,
+      'NO_COLOR' => '1',
+      'TMPDIR' => ENV.fetch('BIBLE270_TEST_TMPDIR', '/tmp')
+    }
+    command = ['bundle', 'exec', 'parallel_test', 'test', '--type', 'test', '-n', workers]
+    puts Bible270TestOutput.color("Running tests (#{workers} workers)…", :cyan)
+    stdout, stderr, status, elapsed = Bible270TestOutput.capture(env, command)
+    unless status.success?
+      $stdout.write(stdout)
+      $stderr.write(stderr)
+      raise "Parallel tests failed with status #{status.exitstatus}"
+    end
+    $stderr.write(stderr) unless stderr.empty?
+    puts Bible270TestOutput.color(Bible270TestOutput.summary(stdout), :green)
+    puts Bible270TestOutput.color(format('Completed in %.1f seconds', elapsed), :cyan)
+
+    require 'simplecov'
+    SimpleCov.coverage_dir File.expand_path('coverage', __dir__)
+    result = SimpleCov::ResultMerger.merged_result
+    Bible270TestOutput.quietly { SimpleCov::Formatter::HTMLFormatter.new.format(result) }
+    coverage = format('Coverage: %<percent>.2f%% (%<covered>d/%<total>d) — coverage/index.html',
+                      percent: result.covered_percent, covered: result.covered_lines, total: result.total_lines)
+    puts Bible270TestOutput.color(coverage, :green)
+  end
 end
 
+desc 'Run tests in parallel'
+task test: 'test:parallel'
 task default: :test

data/app/controllers/bible270/admin_controller.rb

@@ -130,9 +130,8 @@ module Bible270
 
     def update_start
       if params[:start_date].present?
-        # set_start_date!, not update_start_date!: an admin is not subject to
-        # config.allow_reader_start_date, which governs what readers may do to
-        # their own dates.
+        # The administrator may store an individual date even when personal
+        # calendars are currently disabled for this installation.
         if @reader.set_start_date!(params[:start_date])
           redirect_to admin_reader_path(@reader),
                       notice: "Start date set to #{@reader.started_on.strftime('%B %-d, %Y')}."

data/app/controllers/bible270/checkoffs_controller.rb

@@ -15,21 +15,46 @@ module Bible270
                                           Plan.present_tracks(@day).include?(@track) &&
                                           Plan.valid_part?(@day, @track, @part)
 
-      existing = current_reader.checkoffs.find_by(day: @day, track: @track, part: @part)
-      if existing
-        existing.destroy
-      else
-        current_reader.ensure_started!
-        current_reader.checkoffs.create(day: @day, track: @track, part: @part)
-      end
+      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 = apply_checkoff(requested_state)
 
-      @reader = current_reader.reload
+      @reader = current_reader.reload.reload_progress
       @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)
+
       respond_to do |format|
         format.turbo_stream
-        format.html { redirect_to day_path(@day) }
+        format.html do
+          message = marked_read ? 'Reading marked read.' : 'Reading marked unread.'
+          redirect_to day_path(@day), flash: { b270_interaction_status: message }
+        end
+      end
+    end
+
+  private
+
+    def apply_checkoff(requested_state)
+      existing = current_reader.checkoffs.find_by(day: @day, track: @track, part: @part)
+      if requested_state == '1'
+        current_reader.ensure_started!
+        current_reader.checkoffs.create_or_find_by!(day: @day, track: @track, part: @part)
+        true
+      elsif requested_state == '0' || existing
+        existing&.destroy
+        false
+      else
+        current_reader.ensure_started!
+        current_reader.checkoffs.create(day: @day, track: @track, part: @part)
+        true
       end
     end
   end

data/app/controllers/bible270/comments_controller.rb

@@ -3,8 +3,12 @@
 module Bible270
   class CommentsController < ApplicationController
     def index
+      visited_at = Time.current
       @days_with_reflections = Comment.approved.distinct.pluck(:day).sort
-      @threads = recent_threads
+      load_reflection_filters
+      load_reflection_page
+      load_new_reflections
+      current_reader&.mark_reflections_seen!(visited_at)
     end
 
     def create
@@ -19,13 +23,18 @@ module Bible270
       if @comment.save
         respond_to do |format|
           format.turbo_stream
-          format.html { redirect_to day_path(@day, anchor: "comment-#{@comment.id}") }
+          format.html do
+            message = @comment.reply? ? 'Reply posted.' : 'Reflection posted.'
+            redirect_to day_path(@day, anchor: "comment-#{@comment.id}"),
+                        flash: { b270_interaction_status: message }
+          end
         end
       else
         respond_to do |format|
           format.turbo_stream do
             render turbo_stream: turbo_stream.replace('new_comment_form', partial: 'bible270/comments/form',
-                                                                          locals: { day: @day, comment: @comment })
+                                                                          locals: { day: @day, comment: @comment }),
+                   status: :unprocessable_entity
           end
           format.html { redirect_to day_path(@day), alert: @comment.errors.full_messages.to_sentence }
         end
@@ -42,14 +51,18 @@ module Bible270
       if @comment.update(comment_params.except(:parent_id))
         respond_to do |format|
           format.turbo_stream
-          format.html { redirect_to day_path(@day, anchor: "comment-#{@comment.id}") }
+          format.html do
+            redirect_to day_path(@day, anchor: "comment-#{@comment.id}"),
+                        flash: { b270_interaction_status: 'Changes saved.' }
+          end
         end
       else
         respond_to do |format|
           format.turbo_stream do
             render turbo_stream: turbo_stream.replace("comment-#{@comment.id}",
                                                       partial: 'bible270/comments/edit_form',
-                                                      locals: { comment: @comment })
+                                                      locals: { comment: @comment }),
+                   status: :unprocessable_entity
           end
           format.html { redirect_to day_path(@day), alert: @comment.errors.full_messages.to_sentence }
         end
@@ -72,30 +85,52 @@ module Bible270
 
   private
 
-    # The most recently active conversations, newest first. A reply counts as
-    # activity, so answering an old reflection brings the whole thread back up —
-    # otherwise a lively discussion on day 3 would stay buried while quieter, newer
-    # reflections sat above it.
-    def recent_threads
-      wanted = Bible270.config.reflections_page_size.to_i
-      wanted = 10 unless wanted.positive?
-
-      # Look at rather more messages than threads, since several may belong to one.
-      recent = Comment.approved.order(created_at: :desc).limit(wanted * 5)
-        .pluck(:id, :parent_id, :created_at)
-
-      roots = {}
-      recent.each do |id, parent_id, created_at|
-        root = parent_id || id
-        roots[root] ||= created_at
-        roots[root] = created_at if created_at > roots[root]
-      end
+    def load_reflection_filters
+      root_reflections = Comment.approved.where(parent_id: nil)
+      reader_ids = root_reflections.distinct.pluck(:reader_id)
+      @reflection_readers = Reader.where(id: reader_ids).to_a.sort_by(&:sort_name)
+
+      requested_day = params[:day].presence&.to_i
+      @reflection_day = requested_day if @days_with_reflections.include?(requested_day)
+
+      requested_reader = params[:reader_id].presence&.to_i
+      @reflection_reader_id = requested_reader if reader_ids.include?(requested_reader)
+    end
+
+    def load_reflection_page
+      result = Comment.thread_page(
+        day: @reflection_day,
+        reader_id: @reflection_reader_id,
+        page: params[:page],
+        per_page: Bible270.config.reflections_page_size
+      )
+      @threads = result.threads
+      @thread_activity = result.activity_by_id
+      @reflection_page = result.page
+      @reflection_pages = result.pages
+      @reflection_total = result.total
+    end
+
+    def load_new_reflections
+      @reflections_since = reflections_since
+      @reflections_since_param = @reflections_since&.iso8601(6) || 'none'
+      @new_thread_ids = if @reflections_since
+                          @thread_activity.filter_map do |id, activity_at|
+                            id if activity_at > @reflections_since
+                          end
+                        else
+                          []
+                        end
+    end
+
+    def reflections_since
+      return nil unless current_reader&.class&.reflections_seen_column?
+      return current_reader.reflections_seen_at unless params.key?(:since)
+      return nil if params[:since] == 'none'
 
-      ids = roots.sort_by { |_root, at| -at.to_i }.first(wanted).map(&:first)
-      threads = Comment.approved.where(id: ids)
-        .includes(:reader, { likes: :reader }, { replies: [:reader, { likes: :reader }] })
-        .index_by(&:id)
-      ids.filter_map { |id| threads[id] }
+      Time.iso8601(params[:since].to_s)
+    rescue ArgumentError
+      current_reader.reflections_seen_at
     end
 
     def comment_params