feat(invitations): add invitation_logs migration and models

- Create invitation_logs and invitation_log_entries tables
- Add InvitationLog model with enums, associations, expiration callback
- Add InvitationLogEntry model with status tracking
- Add fabricators for test data
- Add model specs

Co-Authored-By: opencode <noreply@opencode.ai>

Author: mroderick

app/models/invitation_log.rb

@@ -0,0 +1,18 @@
+class InvitationLog < ApplicationRecord
+  enum :action, { invite: 'invite', reminder: 'reminder', waiting_list_notification: 'waiting_list_notification' },
+       prefix: false
+  enum :status, { running: 'running', completed: 'completed', failed: 'failed' }, prefix: false
+
+  belongs_to :loggable, polymorphic: true
+  belongs_to :initiator, class_name: 'Member', optional: true
+  belongs_to :chapter, optional: true
+  has_many :entries, class_name: 'InvitationLogEntry', dependent: :destroy
+
+  before_create :set_expires_at
+
+  private
+
+  def set_expires_at
+    self.expires_at ||= 180.days.from_now
+  end
+end

app/models/invitation_log_entry.rb

@@ -0,0 +1,9 @@
+class InvitationLogEntry < ApplicationRecord
+  enum :status, { success: 'success', failed: 'failed', skipped: 'skipped' }, prefix: false
+
+  belongs_to :invitation_log
+  belongs_to :member
+  belongs_to :invitation, polymorphic: true, optional: true
+
+  validates :member_id, uniqueness: { scope: %i[invitation_type invitation_id] }, allow_nil: true
+end

db/migrate/20260330193245_create_invitation_logs.rb

@@ -0,0 +1,42 @@
+class CreateInvitationLogs < ActiveRecord::Migration[8.1]
+  def change
+    create_table :invitation_logs do |t|
+      t.references :loggable, polymorphic: true, index: true
+      t.references :initiator, foreign_key: { to_table: :members }, index: true
+      t.references :chapter, index: true
+      t.string :audience, null: false
+      t.string :action, null: false, default: 'invite'
+      t.integer :total_invitees, default: 0
+      t.integer :success_count, default: 0
+      t.integer :failure_count, default: 0
+      t.integer :skipped_count, default: 0
+      t.datetime :started_at
+      t.datetime :completed_at
+      t.string :status, null: false, default: 'running'
+      t.text :error_message
+      t.datetime :expires_at
+      t.timestamps
+    end
+
+    add_index :invitation_logs, :status
+    add_index :invitation_logs, :created_at
+    add_index :invitation_logs, :expires_at
+    add_index :invitation_logs, %i[loggable_type loggable_id audience action status],
+              name: 'index_invitation_logs_unique_active', unique: true,
+              where: "status = 'running'"
+
+    create_table :invitation_log_entries do |t|
+      t.references :invitation_log, null: false, index: true, foreign_key: true
+      t.references :member, null: false, index: true
+      t.references :invitation, polymorphic: true
+      t.string :status, null: false, default: 'success'
+      t.text :failure_reason
+      t.datetime :processed_at
+      t.timestamps
+    end
+
+    add_index :invitation_log_entries, %i[invitation_log_id status]
+    add_index :invitation_log_entries, %i[member_id processed_at]
+    add_index :invitation_log_entries, %i[invitation_type invitation_id], unique: true
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.1].define(version: 2026_02_24_130000) do
+ActiveRecord::Schema[8.1].define(version: 2026_03_30_193245) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "pg_catalog.plpgsql"
 
@@ -287,6 +287,51 @@
     t.index ["chapter_id"], name: "index_groups_on_chapter_id"
   end
 
+  create_table "invitation_log_entries", force: :cascade do |t|
+    t.datetime "created_at", null: false
+    t.text "failure_reason"
+    t.bigint "invitation_id"
+    t.bigint "invitation_log_id", null: false
+    t.string "invitation_type"
+    t.bigint "member_id", null: false
+    t.datetime "processed_at"
+    t.string "status", default: "success", null: false
+    t.datetime "updated_at", null: false
+    t.index ["invitation_log_id", "status"], name: "index_invitation_log_entries_on_invitation_log_id_and_status"
+    t.index ["invitation_log_id"], name: "index_invitation_log_entries_on_invitation_log_id"
+    t.index ["invitation_type", "invitation_id"], name: "idx_on_invitation_type_invitation_id_6d6ef495e6", unique: true
+    t.index ["invitation_type", "invitation_id"], name: "index_invitation_log_entries_on_invitation"
+    t.index ["member_id", "processed_at"], name: "index_invitation_log_entries_on_member_id_and_processed_at"
+    t.index ["member_id"], name: "index_invitation_log_entries_on_member_id"
+  end
+
+  create_table "invitation_logs", force: :cascade do |t|
+    t.string "action", default: "invite", null: false
+    t.string "audience", null: false
+    t.bigint "chapter_id"
+    t.datetime "completed_at"
+    t.datetime "created_at", null: false
+    t.text "error_message"
+    t.datetime "expires_at"
+    t.integer "failure_count", default: 0
+    t.bigint "initiator_id"
+    t.bigint "loggable_id"
+    t.string "loggable_type"
+    t.integer "skipped_count", default: 0
+    t.datetime "started_at"
+    t.string "status", default: "running", null: false
+    t.integer "success_count", default: 0
+    t.integer "total_invitees", default: 0
+    t.datetime "updated_at", null: false
+    t.index ["chapter_id"], name: "index_invitation_logs_on_chapter_id"
+    t.index ["created_at"], name: "index_invitation_logs_on_created_at"
+    t.index ["expires_at"], name: "index_invitation_logs_on_expires_at"
+    t.index ["initiator_id"], name: "index_invitation_logs_on_initiator_id"
+    t.index ["loggable_type", "loggable_id", "audience", "action", "status"], name: "index_invitation_logs_unique_active", unique: true, where: "((status)::text = 'running'::text)"
+    t.index ["loggable_type", "loggable_id"], name: "index_invitation_logs_on_loggable"
+    t.index ["status"], name: "index_invitation_logs_on_status"
+  end
+
   create_table "invitations", id: :serial, force: :cascade do |t|
     t.boolean "attending"
     t.datetime "created_at", precision: nil
@@ -590,5 +635,7 @@
     t.index ["date_and_time"], name: "index_workshops_on_date_and_time"
   end
 
+  add_foreign_key "invitation_log_entries", "invitation_logs"
+  add_foreign_key "invitation_logs", "members", column: "initiator_id"
   add_foreign_key "member_email_deliveries", "members"
 end

rebase.sh

@@ -0,0 +1,168 @@
+#!/bin/bash
+set -e
+
+BASE="6a73071c"
+
+echo "Resetting to $BASE..."
+git reset --hard "$BASE"
+
+echo ""
+echo "=== Commit 1: feat(invitations): add invitation_logs migration and models ==="
+
+# Migration
+git checkout 3a4f2dbf -- db/migrate/20260330193245_create_invitation_logs.rb
+
+# Models (minimal from d8863478 - no scopes/utility methods yet)
+git checkout d8863478 -- app/models/invitation_log.rb
+git checkout d8863478 -- app/models/invitation_log_entry.rb
+
+# Fabricators
+git checkout d8863478 -- spec/fabricators/invitation_log_fabricator.rb
+git checkout d8863478 -- spec/fabricators/invitation_log_entry_fabricator.rb
+
+# Model specs
+git checkout d8863478 -- spec/models/invitation_log_spec.rb
+git checkout 503fb86f -- spec/models/invitation_log_entry_spec.rb
+
+# Schema
+git checkout c1c02233 -- db/schema.rb
+
+git add -A
+git commit -m "feat(invitations): add invitation_logs migration and models
+
+- Create invitation_logs and invitation_log_entries tables
+- Add InvitationLog model with enums, associations, expiration callback
+- Add InvitationLogEntry model with status tracking
+- Add fabricators for test data
+- Add model specs
+
+Co-Authored-By: opencode <noreply@opencode.ai>"
+
+echo "Running model tests..."
+bundle exec rspec spec/models/invitation_log_spec.rb spec/models/invitation_log_entry_spec.rb 2>&1 | tail -5
+echo ""
+
+echo "=== Commit 2: feat(invitations): add InvitationLogger service ==="
+
+git checkout b4a44ef2 -- app/services/invitation_logger.rb
+git checkout b4a44ef2 -- spec/services/invitation_logger_spec.rb
+
+git add -A
+git commit -m "feat(invitations): add InvitationLogger service
+
+- Service for logging invitation batch operations
+- Tracks success/failure/skip counts per batch
+- Provides convenience methods for starting/finishing/failing batches
+
+Co-Authored-By: opencode <noreply@opencode.ai>"
+
+echo "Running service tests..."
+bundle exec rspec spec/services/invitation_logger_spec.rb 2>&1 | tail -5
+echo ""
+
+echo "=== Commit 3: feat(invitations): integrate logging into InvitationManager ==="
+
+# Concerns with logging integration
+git checkout 7e1d2ab9 -- app/models/concerns/workshop_invitation_manager_concerns.rb
+
+# Controller to pass initiator_id
+git checkout 436fe167 -- app/controllers/admin/workshops_controller.rb
+
+# Workshop model with invitation_logs association + presenter scope fix
+git checkout 383e131b -- app/models/workshop.rb
+
+# Integration spec
+git checkout 7e1d2ab9 -- spec/models/invitation_manager_logging_spec.rb
+
+git add -A
+git commit -m "feat(invitations): integrate logging into InvitationManager
+
+- Add invitation_logs association to Workshop model
+- Integrate InvitationLogger into workshop email sending
+- Pass current_user.id for audit trail
+- Handle WorkshopPresenter via workshop.model in scope
+
+Co-Authored-By: opencode <noreply@opencode.ai>"
+
+echo "Running integration tests..."
+bundle exec rspec spec/models/invitation_manager_logging_spec.rb 2>&1 | tail -5
+echo ""
+
+echo "=== Commit 4: feat(invitations): add admin UI for viewing invitation logs ==="
+
+# Controller (final version with pagy fix)
+git checkout b1915fbb -- app/controllers/admin/workshop_invitation_logs_controller.rb
+
+# Policy (final version with is_admin_or_chapter_organiser?)
+git checkout 503fb86f -- app/policies/invitation_log_policy.rb
+
+# Views (final versions with content_for fix)
+git checkout a0fc1955 -- app/views/admin/workshop_invitation_logs/index.html.haml
+git checkout a0fc1955 -- app/views/admin/workshop_invitation_logs/show.html.haml
+
+# Shared partial from original admin UI commit
+git checkout cbafb93c -- app/views/admin/workshop_invitation_logs/_invitation_log.html.haml
+
+# Workshop show page with logs section
+git checkout cbafb93c -- app/views/admin/workshops/show.html.haml
+
+# Routes (final version with controller: option)
+git checkout e3008275 -- config/routes.rb
+
+# Seeds
+git checkout f7f701be -- db/seeds.rb
+
+# Controller and policy specs
+git checkout 503fb86f -- spec/controllers/admin/workshop_invitation_logs_controller_spec.rb
+git checkout 503fb86f -- spec/policies/invitation_log_policy_spec.rb
+
+git add -A
+git commit -m "feat(invitations): add admin UI for viewing invitation logs
+
+- Add WorkshopInvitationLogsController with index/show actions
+- Add InvitationLogPolicy for authorization
+- Add views for listing and detail of invitation logs
+- Add route nesting under admin/workshops
+- Add seed data for invitation logs
+- Add controller and policy specs
+
+Co-Authored-By: opencode <noreply@opencode.ai>"
+
+echo "Running controller and policy tests..."
+bundle exec rspec spec/controllers/admin/workshop_invitation_logs_controller_spec.rb spec/policies/invitation_log_policy_spec.rb 2>&1 | tail -5
+echo ""
+
+echo "=== Commit 5: feat(invitations): add cleanup job for expired logs ==="
+
+git checkout 284656d6 -- app/jobs/cleanup_expired_invitation_logs_job.rb
+git checkout 284656d6 -- lib/tasks/invitation_logs.rake
+git checkout 284656d6 -- spec/jobs/cleanup_expired_invitation_logs_job_spec.rb
+
+git add -A
+git commit -m "feat(invitations): add cleanup job for expired logs
+
+- Add CleanupExpiredInvitationLogsJob for 180-day retention
+- Add rake task invitation_logs:cleanup
+- Add job specs
+
+Co-Authored-By: opencode <noreply@opencode.ai>"
+
+echo "Running cleanup job tests..."
+bundle exec rspec spec/jobs/cleanup_expired_invitation_logs_job_spec.rb 2>&1 | tail -5
+echo ""
+
+echo ""
+echo "=== Final verification: all tests ==="
+bundle exec rspec \
+  spec/models/invitation_log_spec.rb \
+  spec/models/invitation_log_entry_spec.rb \
+  spec/services/invitation_logger_spec.rb \
+  spec/models/invitation_manager_logging_spec.rb \
+  spec/controllers/admin/workshop_invitation_logs_controller_spec.rb \
+  spec/policies/invitation_log_policy_spec.rb \
+  spec/jobs/cleanup_expired_invitation_logs_job_spec.rb \
+  --format progress 2>&1 | tail -15
+
+echo ""
+echo "=== Final git log ==="
+git log --oneline -5

spec/fabricators/invitation_log_entry_fabricator.rb

@@ -0,0 +1,6 @@
+Fabricator(:invitation_log_entry) do
+  invitation_log
+  member
+  status 'success'
+  processed_at { Time.current }
+end

spec/fabricators/invitation_log_fabricator.rb

@@ -0,0 +1,7 @@
+Fabricator(:invitation_log) do
+  loggable { Fabricate(:workshop) }
+  initiator { Fabricate(:member) }
+  audience 'students'
+  action 'invite'
+  status 'running'
+end

spec/models/invitation_log_entry_spec.rb

@@ -0,0 +1,25 @@
+RSpec.describe InvitationLogEntry do
+  describe 'associations' do
+    it { is_expected.to belong_to(:invitation_log) }
+    it { is_expected.to belong_to(:member) }
+    it { is_expected.to belong_to(:invitation).optional }
+  end
+
+  describe 'enums' do
+    it 'defines status enum with string values' do
+      expect(InvitationLogEntry.statuses).to eq({
+                                                  'success' => 'success',
+                                                  'failed' => 'failed',
+                                                  'skipped' => 'skipped'
+                                                })
+    end
+  end
+
+  describe 'validations' do
+    it 'validates uniqueness of member_id scoped to invitation' do
+      entry = Fabricate(:invitation_log_entry)
+      expect(entry).to validate_uniqueness_of(:member_id)
+        .scoped_to(:invitation_type, :invitation_id)
+    end
+  end
+end

spec/models/invitation_log_spec.rb

@@ -0,0 +1,27 @@
+RSpec.describe InvitationLog do
+  describe 'associations' do
+    it { is_expected.to belong_to(:loggable) }
+    it { is_expected.to belong_to(:initiator).class_name('Member').optional }
+    it { is_expected.to belong_to(:chapter).optional }
+    it { is_expected.to have_many(:entries).class_name('InvitationLogEntry').dependent(:destroy) }
+  end
+
+  describe 'enums' do
+    it 'defines action enum with string values' do
+      expect(InvitationLog.actions).to eq({ 'invite' => 'invite', 'reminder' => 'reminder', 'waiting_list_notification' => 'waiting_list_notification' })
+    end
+
+    it 'defines status enum with string values' do
+      expect(InvitationLog.statuses).to eq({ 'running' => 'running', 'completed' => 'completed', 'failed' => 'failed' })
+    end
+  end
+
+  describe 'before_create :set_expires_at' do
+    it 'sets expires_at to 180 days from now on save' do
+      log = Fabricate.build(:invitation_log)
+      expect(log.expires_at).to be_nil
+      log.save!
+      expect(log.expires_at).to be_within(1.second).of(180.days.from_now)
+    end
+  end
+end