From 558a053e5477fe43a7641b467b13de6a6ac298a8 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:23:37 -0400
Subject: [PATCH 01/20] Audit Stimulus controllers for conventions and update
 AI docs

- Remove dead controllers (nav, search_box)
- Convert querySelector/getElementById to static targets
- Convert instance variables to static values
- Replace addEventListener with data-action attributes
- Replace style.display with hidden class toggling
- Fix CSS injection leak in remote_select
- Remove console.log statements
- Add comprehensive Stimulus conventions to AI instruction files
- Add AI Instruction Files reference table to CLAUDE.md
- Update PR instruction wording to "content" (covers checklists)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/copilot-instructions.md               | 231 ++++++------------
 AGENTS.md                                     | 156 +++++-------
 CLAUDE.md                                     | 138 +++++++----
 .../controllers/asset_picker_controller.js    |   4 +
 .../controllers/collection_controller.js      |  46 ++--
 .../controllers/column_toggle_controller.js   |   6 +-
 .../comment_edit_toggle_controller.js         |  29 +--
 .../controllers/dirty_form_controller.js      |  29 +--
 .../controllers/dropdown_controller.js        |  10 +-
 app/frontend/javascript/controllers/index.js  |   3 -
 .../javascript/controllers/nav_controller.js  |  24 --
 .../paginated_fields_controller.js            |  30 +--
 .../controllers/password_toggle_controller.js |  14 +-
 .../controllers/print_options_controller.js   |  24 +-
 .../controllers/remote_select_controller.js   |  54 ++--
 .../controllers/search_box_controller.js      |  14 --
 .../javascript/controllers/tabs_controller.js |  52 ++--
 .../tags_combination_highlight_controller.js  |  18 +-
 .../controllers/timeframe_controller.js       |  38 ++-
 .../controllers/toggle_lock_controller.js     |   4 +-
 .../toggle_user_icon_controller.js            |   4 +-
 app/helpers/analytics_helper.rb               |   8 +-
 app/views/comments/_comment_fields.html.erb   |   4 +-
 app/views/devise/passwords/edit.html.erb      |   4 +-
 app/views/devise/sessions/new.html.erb        |   2 +-
 app/views/event_registrations/_form.html.erb  |   2 +-
 app/views/events/_manage_results.html.erb     |   4 +-
 app/views/organizations/_form.html.erb        |   2 +-
 app/views/people/_form.html.erb               |   5 +-
 .../taggings/_explore_by_combination.html.erb |   4 +
 app/views/users/_form.html.erb                |  11 +-
 app/views/users/change_password.html.erb      |   6 +-
 app/views/users/edit.html.erb                 |   6 +-
 .../video_recordings/_search_boxes.html.erb   |   4 +-
 app/views/welcome/show.html.erb               |   4 +-
 35 files changed, 421 insertions(+), 573 deletions(-)
 delete mode 100644 app/frontend/javascript/controllers/nav_controller.js
 delete mode 100644 app/frontend/javascript/controllers/search_box_controller.js

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
index d9751921d..14aa7aefc 100644
--- a/.github/copilot-instructions.md
+++ b/.github/copilot-instructions.md
@@ -1,48 +1,31 @@
-# This is a Ruby on Rails application.
+# This project is a Ruby on Rails application.
 <!-- Keep code style rules in sync with CLAUDE.md -->
 
-For project overview, tech stack, architecture reference (models, controllers, services, testing), and more, read `AGENTS.md`.
-
-## Setup
-
-Full setup (bundle, npm, database create/migrate/seed):
-```
-bin/setup
-```
-
-If you just need frontend dependencies:
-```
-npm ci
-```
-
-## AI Instruction Files
-
-When the user says "AI files", "AI instructions", "tell AI to", or "remember to always", these are the files.
-If you notice the user repeatedly correcting the same pattern, suggest adding it to the AI files with a concrete proposal.
-
-| File | Purpose |
-|---|---|
-| `CLAUDE.md` | Coding rules and conventions (this file) |
-| `AGENTS.md` | Architecture reference + project details |
-| `.github/copilot-instructions.md` | Coding rules for Copilot (duplicated from CLAUDE.md — keep in sync) |
-| `ai/` | Shell script shortcuts for common dev tasks |
-
-## Related Files
-
-When changing a model or controller, check whether these related files need updates:
-
-| If you change... | Also check... |
-|---|---|
-| Model | Decorator, policy, factory, model spec |
-| Controller | Policy, request spec, routing spec, views |
-| View | System spec, Stimulus controller (if interactive) |
-| Service | Service spec |
-| Decorator | Decorator spec |
-| Mailer (add/remove) | Mailer spec, mailer preview (follow existing patterns) |
-| Add/remove model, concern, service, or gem | AGENTS.md |
-
-## Code Style
-
+# Frontend requirements:
+- Strongly prefer Stimulus for JavaScript behavior — do not write raw/inline JS or jQuery
+- Always use Tailwind CSS utility classes for styling — do not write custom CSS unless absolutely necessary
+- Prefer Turbo for navigation and form submissions before reaching for Stimulus
+- ES6+ syntax, ESM imports/exports
+- Stimulus controller naming: `[name]_controller.js`
+
+# Stimulus conventions:
+- Use `static targets` and `data-[controller]-target` — never `querySelector` or `getElementById` for elements that could be targets
+- Use `static values = { name: Type }` for state — not instance variables. Use `[name]ValueChanged()` for reactive updates
+- Use `data-action` attributes — not `addEventListener` in `connect()`. Omit default events (`click` for buttons, `input` for inputs, `submit` for forms, `change` for selects)
+- Use `@window`/`@document` suffixes for global events in data-action
+- Use action options (`:prevent`, `:stop`) instead of `event.preventDefault()` in methods
+- Use `static classes` when CSS classes should be configurable from HTML
+- Use `static outlets` for cross-controller communication instead of `getElementById`
+- Always clean up in `disconnect()` anything created in `connect()` (listeners, timers, observers)
+- Use `[name]TargetConnected`/`TargetDisconnected` for dynamic DOM (cocoon, Turbo)
+- Toggle `hidden` class instead of `style.display`. Use `class="hidden"` not `style="display:none"` in HTML
+
+# PRs
+- After completing work, create a pull request using `gh pr create`
+- Once the PR is created, prepend the PR number to the branch name (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
+- On every push, update the PR title and content to reflect the current diff
+
+# Code style requirements:
 - Use modern Ruby syntax
 - Prefer early returns and guard clauses
 - Avoid unnecessary and/or complex conditionals
@@ -51,59 +34,56 @@ When changing a model or controller, check whether these related files need upda
 - Use `presence` over blank checks
 - Use `Arel.sql` for raw SQL in order clauses
 - Avoid `update_all` unless explicitly intended
-- Prefer service objects under app/services/
+- Prefer service objects under app/services
 - Prefer POROs over concerns when possible
 - Use `after_commit` instead of `after_save` for side effects
 
-## RuboCop (rubocop-rails-omakase)
-
+# RuboCop (rubocop-rails-omakase)
 This project uses rubocop-rails-omakase. All code MUST follow these rules:
 
-### Strings
-- **Always use double quotes** for strings: `"foo"` not `'foo'`
-
-### Spacing
-- **Spaces inside array brackets:** `[ a, b, c ]` not `[a, b, c]` (empty arrays: `[]`)
-- **Spaces inside hash braces:** `{ a: 1, b: 2 }` not `{a: 1}` (empty hashes: `{}`)
-- **Spaces inside block braces:** `foo { bar }` not `foo {bar}` (empty blocks: `foo { }`)
-- **No spaces inside parens:** `foo(bar)` not `foo( bar )`
-- **No spaces inside reference brackets:** `hash[:key]` not `hash[ :key ]`
-- **Space before block braces:** `foo { }` not `foo{ }`
-
-### Commas
-- **No trailing commas** in arrays, hashes, or method arguments
-
-### Indentation
-- **2-space indentation**, no tabs
-- **Consistent indentation** at normal level — do NOT indent methods under `private`/`protected`
-- **Align `end` with the variable** in assignments:
-  ```ruby
-  result = if condition
-    value
-  end
-  ```
-- **Align `when` with `end`**, not with `case`
-
-### Whitespace
-- **No trailing whitespace** on any line
-- **No trailing blank lines** at end of file
-- **No empty lines** inside class, module, method, or block bodies
-
-### Syntax
-- **Use `%w[]` and `%i[]`** with square bracket delimiters (not parens)
-- **Use modern hash syntax:** `{ key: value }` not `{ :key => value }`
-- **No redundant returns** — omit `return` on last expression
-- **Use `flat_map`** instead of `.map { }.flatten`
-- **No redundant `.to_s`** inside string interpolation
-- **Use `Foo.method`** not `Foo::method` for method calls
-- **No parentheses around conditions:** `if foo` not `if (foo)`
-- **No semicolons** to separate statements
-
-## HTML/ERB Formatting
-
-### Tag Attributes
-- **Closing `>` on same line as last attribute** — do not put `>` on its own line
-- When attributes span multiple lines, keep the closing `>` with the last attribute
+## Strings
+- Always use double quotes: `"foo"` not `'foo'`
+
+## Spacing
+- Spaces inside array brackets: `[ a, b, c ]` not `[a, b, c]` (empty arrays: `[]`)
+- Spaces inside hash braces: `{ a: 1, b: 2 }` not `{a: 1}` (empty hashes: `{}`)
+- Spaces inside block braces: `foo { bar }` not `foo {bar}` (empty blocks: `foo { }`)
+- No spaces inside parens: `foo(bar)` not `foo( bar )`
+- No spaces inside reference brackets: `hash[:key]` not `hash[ :key ]`
+- Space before block braces: `foo { }` not `foo{ }`
+
+## Commas
+- No trailing commas in arrays, hashes, or method arguments
+
+## Indentation
+- 2-space indentation, no tabs
+- Consistent indentation at normal level — do NOT indent methods under `private`/`protected`
+- Align `end` with the variable in assignments
+- Align `when` with `end`, not with `case`
+
+## Whitespace
+- No trailing whitespace on any line
+- No trailing blank lines at end of file
+- No empty lines inside class, module, method, or block bodies
+
+## Syntax
+- Use `%w[]` and `%i[]` with square bracket delimiters (not parens)
+- Use modern hash syntax: `{ key: value }` not `{ :key => value }`
+- No redundant returns — omit `return` on last expression
+- Use `flat_map` instead of `.map { }.flatten`
+- No redundant `.to_s` inside string interpolation
+- Use `Foo.method` not `Foo::method` for method calls
+- No parentheses around conditions: `if foo` not `if (foo)`
+- No semicolons to separate statements
+
+# Git
+- When rebasing onto main, review incoming changes for their intent and flag any oversights — missing tests, incomplete migrations, broken assumptions, or conflicts between the two branches. Check both directions: schema/model changes on either branch that affect views, partials, or layouts on the other (e.g., main redesigned a table's CSS but your branch adds new columns to it, or vice versa)
+
+# HTML/ERB Formatting
+
+## Tag attributes
+- When a tag has long attributes, place the closing `>` on the same line as the last attribute
+- Do NOT put the closing `>` on its own line
 - Example (GOOD):
   ```erb
   <div class="relative z-10 w-full bg-white text-gray-800 py-2 px-4"
@@ -115,72 +95,3 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
        id="dropdown"
   >
   ```
-
-## JavaScript
-
-- ES6+ syntax, ESM imports/exports, `const`/`let` (no `var`)
-- Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
-- **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
-- **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
-- **Prefer Font Awesome (free)** icons over inline SVGs — use `icon("fa-solid fa-foo")` helper. Inline SVGs are acceptable when a specific icon design is preferred.
-- Prefer Turbo for navigation and form submissions before reaching for Stimulus
-- Controller naming: `[name]_controller.js`
-- Keep controllers focused and small
-
-### Stimulus Conventions
-
-Follow the [Stimulus Handbook](https://stimulus.hotwired.dev/handbook/introduction) and reference docs. Key rules:
-
-**Targets over querySelector** — declare `static targets = [...]` and use `data-[controller]-target` attributes in views. Never use `this.element.querySelector` or `document.getElementById` to find elements that could be targets. Exception: elements outside the controller's scope (e.g., in a parent view).
-
-**Values API for state** — use `static values = { name: Type }` for any state that persists or drives UI. Do not store state in instance variables when a value would work. Use `[name]ValueChanged()` callbacks for reactive updates instead of manual syncing.
-
-**Actions over manual listeners** — use `data-action` attributes instead of `addEventListener` in `connect()`. Omit the event when it's the default for the element (`click` for buttons/links, `input` for inputs/textareas, `submit` for forms, `change` for selects). Use `@window` or `@document` suffixes for global events when possible (e.g., `resize@window->controller#layout`). Use action options like `:prevent` and `:stop` instead of calling `event.preventDefault()` in methods.
-
-**Classes API for CSS** — use `static classes = [...]` when CSS classes need to be configurable from HTML. For standard Tailwind utilities used internally (e.g., `"hidden"`), hardcoding is acceptable.
-
-**Outlets for cross-controller communication** — use `static outlets = [...]` to reference other controllers instead of `document.getElementById` or custom events when the relationship is stable.
-
-**Lifecycle discipline** — every listener, timer, or observer created in `connect()` must be cleaned up in `disconnect()`. Store bound handler references so they can be removed. Use `initialize()` for one-time setup (e.g., binding functions).
-
-**Target lifecycle callbacks** — use `[name]TargetConnected(element)` and `[name]TargetDisconnected(element)` to respond to dynamically added/removed targets (e.g., cocoon nested fields, Turbo streams).
-
-**Visibility** — toggle the `hidden` class via `classList.toggle("hidden", condition)` instead of setting `style.display`. Use `class="hidden"` in HTML for initial hidden state, not `style="display:none"`.
-
-## Migrations
-
-- Name migration files using **UTC timestamps** (e.g., `20260228143000`), not sequential numbers (e.g., `20260228000007`)
-- Multiple branches adding migrations on the same date will collide if they use sequential numbering
-
-## Git
-
-- Default branch is `main`
-- Commit messages should explain why, not what
-- CI runs via GitHub Actions (`.github/workflows/`)
-- **When rebasing onto main**, review incoming changes for their intent and flag any oversights — missing tests, incomplete migrations, broken assumptions, or conflicts between the two branches. Check both directions: schema/model changes on either branch that affect views, partials, or layouts on the other (e.g., main redesigned a table's CSS but your branch adds new columns to it, or vice versa)
-
-## PRs
-
-- **Push to a draft PR early** — push commits and create a draft PR (`gh pr create --draft`) as soon as work begins, rather than keeping changes in a local branch. Push on every commit.
-- After completing work, **mark the PR ready** using `gh pr ready`
-- Once the PR is created, **prepend the PR number to the branch name** (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
-- Use `docs/pull_request_template.md` for PR description structure
-- Use bullet points, not paragraphs, when filling out each section
-- Description must explain why the change was made, not just what
-- Include screenshots for UI changes
-- **On every push**, update the PR title and content to reflect the current diff
-- **On every push**, update AI instruction files if the diff adds, removes, or renames anything tracked in AGENTS.md — specifically: Stimulus controllers, services, model/controller concerns, mailers, rake tasks, and directory file counts
-
-## Quick Commands
-
-See `ai/` directory for executable scripts:
-
-| Command | What it does |
-|---|---|
-| `ai/test [args]` | Run RSpec |
-| `ai/lint` | Rubocop on all files |
-| `ai/lint --fix` | Auto-fix lint issues |
-| `ai/server` | Start dev services (web + vite) |
-| `ai/console` | Rails console |
-| `ai/routes -g pattern` | Search Rails routes |
-| `ai/db-migrate` | Run database migrations |
diff --git a/AGENTS.md b/AGENTS.md
index c9f365a00..9607473a7 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -1,38 +1,15 @@
-# AGENTS.md
+# AGENTS.md — AWBW Portal
 
-Architecture reference for AI agents. For coding rules and conventions, see `CLAUDE.md` (single source of truth).
+Architecture reference for AI agents working on the AWBW Portal codebase. For coding rules and quick commands, see `CLAUDE.md`.
 
-## Project Overview
+## Project Summary
 
-AWBW Portal is a Rails 8.1 application (Ruby 4.0.1) for A Window Between Worlds — a platform where workshop leaders manage workshops, resources, community news, stories, and events.
-
-## Tech Stack
-
-- **Backend:** Rails 8.1, Ruby 4.0.1, MySQL (via Trilogy adapter)
-- **Frontend:** Vite, Tailwind CSS v4, Stimulus, Turbo Rails
-- **Auth:** Devise with JWT token support
-- **Authorization:** ActionPolicy (app/policies/)
-- **Rich text:** ActionText with Rhino editor (TipTap-based)
-- **File uploads:** ActiveStorage with DigitalOcean Spaces
-- **Background jobs:** SolidQueue
-- **Caching:** SolidCache
-
-## Setup
-
-Full setup (bundle, npm, database create/migrate/seed):
-```
-bin/setup
-```
-
-If you just need frontend dependencies:
-```
-npm ci
-```
+AWBW Portal is a Rails 8.1 application (Ruby 4.0.1) for A Window Between Worlds — a platform where workshop leaders manage workshops, resources, community news, stories, and events. It uses MySQL, Vite, Tailwind CSS v4, and the Hotwire stack (Stimulus + Turbo).
 
 ## Architecture Overview
 
 ```
-This codebase (Rails 8.1)
+AWBW Portal (Rails 8.1)
 ├── Main app (app/)              — Workshops, resources, stories, events, people, organizations
 ├── Frontend                     — Stimulus + Turbo + Tailwind CSS v4 (Vite bundler)
 ├── Background jobs              — SolidQueue
@@ -48,19 +25,19 @@ This codebase (Rails 8.1)
 
 | Directory | Purpose | Count |
 |---|---|---|
-| `app/models/` | ActiveRecord models | ~66 files |
-| `app/services/` | Service objects for complex logic | ~21 files |
-| `app/jobs/` | SolidQueue background jobs | 3 files |
-| `app/models/concerns/` | Shared model modules | 12 concerns |
+| `app/models/` | ActiveRecord models | ~78 files |
+| `app/services/` | Service objects for complex logic | ~15 files |
+| `app/jobs/` | SolidQueue background jobs | 2 files |
+| `app/models/concerns/` | Shared model modules | ~11 concerns |
 
 ### Presentation
 
 | Directory | Purpose | Count |
 |---|---|---|
-| `app/controllers/` | Rails controllers (admin/, events/) | ~63 files |
-| `app/views/` | ERB templates | ~465 files |
+| `app/controllers/` | Rails controllers (admin/, events/) | ~61 files |
+| `app/views/` | ERB templates | ~434 files |
 | `app/decorators/` | Draper decorators for view logic | ~36 files |
-| `app/policies/` | ActionPolicy authorization rules | ~44 files |
+| `app/policies/` | ActionPolicy authorization rules | ~43 files |
 | `app/presenters/` | Presentation objects | 1 file |
 | `app/helpers/` | View helpers | ~19 files |
 | `app/mailers/` | ActionMailer classes | 6 files |
@@ -71,7 +48,7 @@ This codebase (Rails 8.1)
 | Directory | Purpose |
 |---|---|
 | `app/frontend/entrypoints/` | Vite entry points (application.js, application.css) |
-| `app/frontend/javascript/controllers/` | Stimulus controllers (34) |
+| `app/frontend/javascript/controllers/` | Stimulus controllers (~32) |
 | `app/frontend/javascript/rhino/` | Rich text editor customizations (mentions, grid) |
 | `app/frontend/stylesheets/` | Tailwind CSS and component styles |
 
@@ -81,7 +58,7 @@ This codebase (Rails 8.1)
 |---|---|
 | `config/routes.rb` | All routes (single file) |
 | `config/database.yml` | MySQL via Trilogy adapter |
-| `config/initializers/` | ~28 initializer files |
+| `config/initializers/` | ~32 initializer files |
 | `.github/workflows/` | GitHub Actions CI |
 | `Procfile.dev` | Dev services: `vite` + `web` |
 | `ai/` | Shell script shortcuts for common dev tasks (see `ai/README.md`) |
@@ -118,24 +95,23 @@ This codebase (Rails 8.1)
 
 | Concern | Purpose |
 |---|---|
-| `AhoyTrackable` | Event tracking integration |
-| `AuthorCreditable` | Author attribution |
 | `Featureable` | `featured`, `publicly_featured` scopes |
-| `Mentioner` | ActionText @mention extraction and grouping |
-| `NameFilterable` | Name-based filtering |
 | `Publishable` | `published`, `publicly_visible` scopes |
-| `PunctuationStrippable` | Strips punctuation from strings |
-| `RemoteSearchable` | AJAX remote search by column |
-| `RichTextSearchable` | Full-text search on ActionText rich_text fields |
 | `TagFilterable` | Scope-based filtering by tag names |
 | `Trendable` | Trending metrics tracking |
 | `WindowsTypeFilterable` | Filter by WindowsType association |
+| `RemoteSearchable` | AJAX remote search by column |
+| `RichTextSearchable` | Full-text search on ActionText rich_text fields |
+| `Mentioner` | ActionText @mention extraction and grouping |
+| `NameFilterable` | Name-based filtering |
+| `PunctuationStrippable` | Strips punctuation from strings |
+| `AhoyTrackable` | Event tracking integration |
 
 ## Controllers
 
 ### Namespaces
 
-- **Root level** (~51 controllers): Workshops, stories, resources, events, people, organizations, etc.
+- **Root level** (~48 controllers): Workshops, stories, resources, events, people, organizations, etc.
 - **`admin/`**: HomeController, AnalyticsController, AhoyActivitiesController
 - **`events/`**: Registrations sub-resource (create/destroy + slug-based show at `/registration/:slug`)
 - **Devise overrides**: Registrations, Confirmations, Passwords
@@ -159,7 +135,6 @@ end
 - `AhoyTracking` — Event tracking integration
 - `Dedupable` — Data deduplication helpers
 - `ExternallyRedirectable` — External URL redirection
-- `TagAssignable` — Tag assignment helpers
 
 ## Services
 
@@ -178,27 +153,9 @@ end
 - `PersonFromUserService` — Create Person from User account
 - `BulkInviteService` — Bulk send welcome instructions and reset created_at for users
 - `ModelDeduper` — Deduplication logic
-- `RichTextMigrator` — Rich text migration utility
-- `DisplayImagePresenter` — Image display logic
-
-### Event Registrations
-
-- `EventRegistrationServices::ProcessConfirmation` — Registration confirmation flow
-- `EventRegistrationServices::PublicRegistration` — Public registration handling
-- `ExtendedEventRegistrationFormBuilder` — Extended registration form builder
-- `ShortEventRegistrationFormBuilder` — Short registration form builder
-- `ScholarshipApplicationFormBuilder` — Scholarship form builder
-
-### Notifications
-
 - `NotificationServices::CreateNotification` — Notification creation
 - `NotificationServices::PersistDeliveredEmail` — Email delivery tracking
 
-### User Management
-
-- `UserServices::ProcessEmailChange` — Email change processing
-- `UserServices::ProcessEmailManualConfirm` — Manual email confirmation
-
 ## Decorators (Draper)
 
 All inherit from `ApplicationDecorator` which provides:
@@ -239,7 +196,7 @@ end
 
 ## Mailers
 
-- `ApplicationMailer` — Base, from: `ENV["REPLY_TO_EMAIL"]`
+- `ApplicationMailer` — Base, from: `ENV["REPLY_TO_EMAIL"]` (programs@awbw.org)
 - `DeviseMailer` — Custom Devise emails
 - `EventMailer` — Event registration confirmations
 - `NotificationMailer` — Notification delivery
@@ -249,10 +206,15 @@ end
 
 ## Frontend
 
+### Preferences
+
+- **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
+- **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
+- Prefer Turbo for navigation and form submissions before reaching for Stimulus
+
 ### Stimulus Controllers
 
-- `affiliation_dates` — Recalculate affiliation date ranges
-- `anchor_highlight` — Highlight anchored elements
+Key controllers:
 - `asset_picker` — Asset selection UI
 - `autosave` — Auto-save form state
 - `carousel` — Swiper-based carousels
@@ -260,31 +222,19 @@ end
 - `collection` — Filter form auto-submit with debounce
 - `column_toggle` — Toggle table column visibility
 - `comment_edit_toggle` — Inline comment editing mode
-- `confirm_email` — Email confirmation UI
 - `dirty_form` — Unsaved changes detection
-- `dismiss` — Dismissable elements
 - `dropdown` — Dropdown menus with keyboard/click-outside handling
 - `file_preview` — File upload preview
 - `inactive_toggle` — Gray out expired affiliations
 - `optimistic_bookmark` — Instant bookmark UI feedback
-- `org_toggle` — Organization toggle UI
 - `paginated_fields` — Client-side pagination of nested fields
-- `password_toggle` — Show/hide password fields
-- `prefetch_lazy` — Prefetch lazy-loaded content
-- `print_options` — Print options toggle for analytics
 - `remote_select` — AJAX-powered select dropdown
-- `rhino_source` — Rich text editor integration
 - `searchable_checkbox` — TomSelect checkbox-style multi-select
 - `searchable_select` — Tom Select autocomplete
-- `share_url` — URL sharing/copying
 - `sortable` — Drag-drop sorting (SortableJS)
 - `tabs` — Tab panel navigation
-- `tag_link_loading` — Loading state for tag links
 - `tags_combination_highlight` — Highlight tags matching selected filters
-- `tags_sync_list_heights` — Sync tag list column heights
-- `timeframe` — Date range filtering
-- `toggle_lock` — Lock/unlock toggle UI
-- `toggle_user_icon` — User icon visibility toggle
+- `rhino_source` — Rich text editor integration
 
 ### JS Dependencies
 
@@ -310,17 +260,17 @@ Custom colors defined in `app/frontend/stylesheets/application.tailwind.css`:
 
 | Directory | Count | Purpose |
 |---|---|---|
-| `spec/models/` | ~58 | Model unit tests |
-| `spec/views/` | ~73 | View template tests |
-| `spec/requests/` | ~47 | HTTP request/integration tests |
-| `spec/system/` | ~25 | End-to-end browser tests (Capybara) |
+| `spec/models/` | ~54 | Model unit tests |
+| `spec/views/` | ~72 | View template tests |
+| `spec/requests/` | ~41 | HTTP request/integration tests |
+| `spec/system/` | ~30 | End-to-end browser tests (Capybara) |
 | `spec/routing/` | ~13 | Route definition tests |
 | `spec/policies/` | ~9 | Authorization policy tests |
-| `spec/decorators/` | ~10 | Decorator tests |
-| `spec/services/` | ~12 | Service object tests |
-| `spec/mailers/` | ~5 | Mailer tests |
-| `spec/helpers/` | ~1 | Helper tests |
-| `spec/factories/` | ~53 | FactoryBot factory definitions |
+| `spec/decorators/` | ~8 | Decorator tests |
+| `spec/services/` | ~4 | Service object tests |
+| `spec/mailers/` | ~3 | Mailer tests |
+| `spec/helpers/` | ~2 | Helper tests |
+| `spec/factories/` | ~52 | FactoryBot factory definitions |
 
 ### Configuration
 
@@ -344,15 +294,6 @@ Common factory traits across models:
 - `:admin` (User with super_user=true)
 - `:with_primary_asset`, `:with_gallery_assets`
 
-## Linting & Security
-
-```
-bundle exec rubocop        # lint
-bundle exec rubocop -a     # auto-fix
-bundle exec brakeman       # security scan
-bundle exec bundle-audit check --update
-```
-
 ## CI Pipeline (GitHub Actions)
 
 ### ci.yml
@@ -364,6 +305,11 @@ bundle exec bundle-audit check --update
 
 RuboCop linting on PRs and pushes to main.
 
+## PR Workflow
+
+- After completing work, create a pull request using `gh pr create`
+- Once the PR is created, prepend the PR number to the branch name (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
+
 ## Key Library Usage
 
 | Need | Library |
@@ -382,10 +328,18 @@ RuboCop linting on PRs and pushes to main.
 | Email styling | Premailer-rails (inline CSS) |
 | Positioning | Positioning gem for ordered records |
 
+## PRs
+
+- On every push, update the PR title and content to reflect the current diff
+
+## Git
+
+- When rebasing onto main, review incoming changes for their intent and flag any oversights — missing tests, incomplete migrations, broken assumptions, or conflicts between the two branches. Check both directions: schema/model changes on either branch that affect views, partials, or layouts on the other (e.g., main redesigned a table's CSS but your branch adds new columns to it, or vice versa)
+
 ## Rake Tasks
 
-Located in `lib/tasks/` (4 files):
+Located in `lib/tasks/` (~17 files). Notable:
 - `dev.rake` — Development database seeding from XML/CSV
+- `paperclip_to_active_storage.rake` — File upload migration
 - `rhino_migrator.rake` — Rich text editor migration
-- `attachment_report.rake` — Attachment reporting
-- `migrate_internal_id_to_filemaker_code.rake` — FileMaker code migration
+- `tag_deduping.rake` — Tag deduplication
diff --git a/CLAUDE.md b/CLAUDE.md
index 8108c0526..9cfe92577 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -1,7 +1,23 @@
-# This is a Ruby on Rails application.
-<!-- Keep code style rules in sync with .github/copilot-instructions.md -->
+# CLAUDE.md
 
-For project overview, tech stack, architecture reference (models, controllers, services, testing), and more, read `AGENTS.md`.
+## Architecture Reference
+
+For detailed architecture, models, controllers, services, and testing structure, read `AGENTS.md`.
+
+## Project Overview
+
+This is a Ruby on Rails 8.1 application (Ruby 4.0.1) — the Portal for A Window Between Worlds (AWBW). It manages workshops, resources, community news, stories, and events for workshop leaders.
+
+## Tech Stack
+
+- **Backend:** Rails 8.1, Ruby 4.0.1, MySQL (via Trilogy adapter)
+- **Frontend:** Vite, Tailwind CSS v4, Stimulus, Turbo Rails
+- **Auth:** Devise with JWT token support
+- **Authorization:** ActionPolicy (app/policies/)
+- **Rich text:** ActionText with Rhino editor (TipTap-based)
+- **File uploads:** ActiveStorage with DigitalOcean Spaces
+- **Background jobs:** SolidQueue
+- **Caching:** SolidCache
 
 ## Setup
 
@@ -15,32 +31,6 @@ If you just need frontend dependencies:
 npm ci
 ```
 
-## AI Instruction Files
-
-When the user says "AI files", "AI instructions", "tell AI to", or "remember to always", these are the files.
-If you notice the user repeatedly correcting the same pattern, suggest adding it to the AI files with a concrete proposal.
-
-| File | Purpose |
-|---|---|
-| `CLAUDE.md` | Coding rules and conventions (this file) |
-| `AGENTS.md` | Architecture reference + project details |
-| `.github/copilot-instructions.md` | Coding rules for Copilot (duplicated from CLAUDE.md — keep in sync) |
-| `ai/` | Shell script shortcuts for common dev tasks |
-
-## Related Files
-
-When changing a model or controller, check whether these related files need updates:
-
-| If you change... | Also check... |
-|---|---|
-| Model | Decorator, policy, factory, model spec |
-| Controller | Policy, request spec, routing spec, views |
-| View | System spec, Stimulus controller (if interactive) |
-| Service | Service spec |
-| Decorator | Decorator spec |
-| Mailer (add/remove) | Mailer spec, mailer preview (follow existing patterns) |
-| Add/remove model, concern, service, or gem | AGENTS.md |
-
 ## Code Style
 
 - Use modern Ruby syntax
@@ -104,25 +94,83 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 ### Tag Attributes
 - **Closing `>` on same line as last attribute** — do not put `>` on its own line
 - When attributes span multiple lines, keep the closing `>` with the last attribute
-- Example (GOOD):
-  ```erb
-  <div class="relative z-10 w-full bg-white text-gray-800 py-2 px-4"
-       id="dropdown">
-  ```
-- Example (BAD):
-  ```erb
-  <div class="relative z-10 w-full bg-white text-gray-800 py-2 px-4"
-       id="dropdown"
-  >
-  ```
+- Good: `<div class="..." id="...">` or `<div class="...\n     id="...">`
+- Bad: `<div class="...\n     id="..."\n  >`
+
+## AI Instruction Files
+
+When the user says "AI files" or "AI instructions", these are the files:
+
+| File | Purpose |
+|---|---|
+| `CLAUDE.md` | Primary rules for Claude Code (this file) |
+| `AGENTS.md` | Architecture reference (models, controllers, services) |
+| `.github/copilot-instructions.md` | Rules for GitHub Copilot (keep in sync with CLAUDE.md) |
+| `ai/` | Shell script shortcuts for common dev tasks |
+
+## Related Files
+
+When changing a model or controller, check whether these related files need updates:
+
+| If you change... | Also check... |
+|---|---|
+| Model | Decorator, policy, factory, model spec |
+| Controller | Policy, request spec, routing spec, views |
+| View | System spec, Stimulus controller (if interactive) |
+| Service | Service spec |
+| Decorator | Decorator spec |
+| Add/remove model, concern, service, or gem | AGENTS.md, `.github/copilot-instructions.md` |
+| Code style rules | `.github/copilot-instructions.md` (keep in sync) |
+
+## Key Directories
+
+- `app/services/` — Business logic service objects
+- `app/decorators/` — Draper decorators for view presentation
+- `app/policies/` — ActionPolicy authorization rules
+- `app/presenters/` — Presentation objects
+- `app/frontend/` — Vite/JS components (Stimulus controllers, etc.)
+
+## Testing
+
+- **Framework:** RSpec (`bundle exec rspec`)
+- **Factories:** FactoryBot (spec/factories/)
+- **Matchers:** Shoulda Matchers
+- **System tests:** Capybara with Selenium
+- **Coverage:** SimpleCov
+
+Run all tests:
+```
+bundle exec rspec
+```
+
+Run a single test file:
+```
+bundle exec rspec spec/models/some_model_spec.rb
+```
+
+## Linting
+
+```
+bundle exec rubocop
+```
+
+Auto-fix:
+```
+bundle exec rubocop -a
+```
+
+## Security Scanning
+
+```
+bundle exec brakeman
+bundle exec bundle-audit check --update
+```
 
 ## JavaScript
 
-- ES6+ syntax, ESM imports/exports, `const`/`let` (no `var`)
-- Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
+- ES6+ syntax, ESM imports/exports
 - **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
 - **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
-- **Prefer Font Awesome (free)** icons over inline SVGs — use `icon("fa-solid fa-foo")` helper. Inline SVGs are acceptable when a specific icon design is preferred.
 - Prefer Turbo for navigation and form submissions before reaching for Stimulus
 - Controller naming: `[name]_controller.js`
 - Keep controllers focused and small
@@ -161,15 +209,13 @@ Follow the [Stimulus Handbook](https://stimulus.hotwired.dev/handbook/introducti
 
 ## PRs
 
-- **Push to a draft PR early** — push commits and create a draft PR (`gh pr create --draft`) as soon as work begins, rather than keeping changes in a local branch. Push on every commit.
-- After completing work, **mark the PR ready** using `gh pr ready`
+- After completing work, **create a pull request** using `gh pr create`
 - Once the PR is created, **prepend the PR number to the branch name** (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
 - Use `docs/pull_request_template.md` for PR description structure
 - Use bullet points, not paragraphs, when filling out each section
 - Description must explain why the change was made, not just what
 - Include screenshots for UI changes
 - **On every push**, update the PR title and content to reflect the current diff
-- **On every push**, update AI instruction files if the diff adds, removes, or renames anything tracked in AGENTS.md — specifically: Stimulus controllers, services, model/controller concerns, mailers, rake tasks, and directory file counts
 
 ## Quick Commands
 
diff --git a/app/frontend/javascript/controllers/asset_picker_controller.js b/app/frontend/javascript/controllers/asset_picker_controller.js
index 99a6ab651..7d734ee4b 100644
--- a/app/frontend/javascript/controllers/asset_picker_controller.js
+++ b/app/frontend/javascript/controllers/asset_picker_controller.js
@@ -1,6 +1,10 @@
 import { Controller } from "@hotwired/stimulus";
 
 // Connects to data-controller="asset-picker"
+//
+// The heroImage, uploadProgress, and uploadProgressBar elements live outside
+// this controller's scope (in the parent view), so getElementById is used
+// intentionally rather than targets.
 export default class extends Controller {
   static targets = ["file"];
 
diff --git a/app/frontend/javascript/controllers/collection_controller.js b/app/frontend/javascript/controllers/collection_controller.js
index b9f56ce46..f15a7dcc2 100644
--- a/app/frontend/javascript/controllers/collection_controller.js
+++ b/app/frontend/javascript/controllers/collection_controller.js
@@ -3,28 +3,32 @@ import { Controller } from "@hotwired/stimulus";
 // Connects to data-controller="collection"
 export default class extends Controller {
   static classes = ["unselected", "selected"];
-  connect() {
-    this.element.addEventListener("change", (event) => {
-      const { type } = event.target;
 
-      if (type === "checkbox") {
-        this.toggleClass(event.target);
-      }
+  handleChange(event) {
+    const { type } = event.target;
 
-      if (
-        type === "checkbox" ||
-        type === "radio" ||
-        type === "select-one" ||
-        type === "select-multiple"
-      ) {
-        this.submitForm();
-      }
-    });
-    this.element.addEventListener("input", (event) => {
-      if (event.target.type === "text") {
-        this.debouncedSubmit();
-      }
-    });
+    if (type === "checkbox") {
+      this.toggleClass(event.target);
+    }
+
+    if (
+      type === "checkbox" ||
+      type === "radio" ||
+      type === "select-one" ||
+      type === "select-multiple"
+    ) {
+      this.submitForm();
+    }
+  }
+
+  handleInput(event) {
+    if (event.target.type === "text" || event.target.type === "search") {
+      this.debouncedSubmit();
+    }
+  }
+
+  disconnect() {
+    clearTimeout(this.timeout);
   }
 
   submitForm() {
@@ -43,12 +47,10 @@ export default class extends Controller {
     const button = el.closest("label");
     if (!button || !this.selectedClasses) return;
 
-    // Toggle selected classes
     this.selectedClasses.forEach((cls) => {
       button.classList.toggle(cls);
     });
 
-    // Toggle unselected classes
     this.unselectedClasses.forEach((cls) => {
       button.classList.toggle(cls);
     });
diff --git a/app/frontend/javascript/controllers/column_toggle_controller.js b/app/frontend/javascript/controllers/column_toggle_controller.js
index 7992e4da0..0b8362d2a 100644
--- a/app/frontend/javascript/controllers/column_toggle_controller.js
+++ b/app/frontend/javascript/controllers/column_toggle_controller.js
@@ -4,11 +4,11 @@ import { Controller } from "@hotwired/stimulus"
 // Toggles visibility of table columns marked with a data attribute.
 
 export default class extends Controller {
-  static targets = ["toggle", "track", "knob"]
+  static targets = ["toggle", "track", "knob", "col"]
 
   toggle() {
     const checked = this.toggleTarget.checked
-    this.element.querySelectorAll("[data-column-toggle-col]").forEach((el) => {
+    this.colTargets.forEach((el) => {
       el.classList.toggle("hidden", !checked)
     })
 
@@ -17,7 +17,7 @@ export default class extends Controller {
       this.trackTarget.classList.toggle("bg-blue-600", checked)
     }
     if (this.hasKnobTarget) {
-      this.knobTarget.style.transform = checked ? "translateX(16px)" : ""
+      this.knobTarget.classList.toggle("translate-x-4", checked)
     }
   }
 }
diff --git a/app/frontend/javascript/controllers/comment_edit_toggle_controller.js b/app/frontend/javascript/controllers/comment_edit_toggle_controller.js
index 108aa8abc..76757d459 100644
--- a/app/frontend/javascript/controllers/comment_edit_toggle_controller.js
+++ b/app/frontend/javascript/controllers/comment_edit_toggle_controller.js
@@ -6,20 +6,17 @@ import { Controller } from "@hotwired/stimulus";
 // When exiting edit mode, syncs textarea values back to the truncated display.
 //
 export default class extends Controller {
-  static targets = ["editLabel", "viewLabel"];
-
-  connect() {
-    this.editing = false;
-  }
+  static targets = ["editLabel", "viewLabel", "commentView", "commentEdit"];
+  static values = { editing: Boolean };
 
   toggle() {
-    this.editing = !this.editing;
+    this.editingValue = !this.editingValue;
 
     // When leaving edit mode, sync textarea values to view display
-    if (!this.editing) {
-      this.element.querySelectorAll(".nested-fields").forEach((item) => {
-        const textarea = item.querySelector(".comment-edit textarea");
-        const viewBody = item.querySelector(".comment-view .comment-body");
+    if (!this.editingValue) {
+      this.commentEditTargets.forEach((editDiv, i) => {
+        const textarea = editDiv.querySelector("textarea");
+        const viewBody = this.commentViewTargets[i]?.querySelector(".comment-body");
         if (textarea && viewBody) {
           const text = textarea.value;
           viewBody.textContent =
@@ -29,16 +26,12 @@ export default class extends Controller {
       });
     }
 
-    this.element
-      .querySelectorAll(".comment-view")
-      .forEach((el) => (el.style.display = this.editing ? "none" : ""));
-    this.element
-      .querySelectorAll(".comment-edit")
-      .forEach((el) => (el.style.display = this.editing ? "" : "none"));
+    this.commentViewTargets.forEach((el) => el.classList.toggle("hidden", this.editingValue));
+    this.commentEditTargets.forEach((el) => el.classList.toggle("hidden", !this.editingValue));
 
     if (this.hasEditLabelTarget && this.hasViewLabelTarget) {
-      this.editLabelTarget.style.display = this.editing ? "none" : "";
-      this.viewLabelTarget.style.display = this.editing ? "" : "none";
+      this.editLabelTarget.classList.toggle("hidden", this.editingValue);
+      this.viewLabelTarget.classList.toggle("hidden", !this.editingValue);
     }
   }
 }
diff --git a/app/frontend/javascript/controllers/dirty_form_controller.js b/app/frontend/javascript/controllers/dirty_form_controller.js
index 7d23cc11b..f72c8eb60 100644
--- a/app/frontend/javascript/controllers/dirty_form_controller.js
+++ b/app/frontend/javascript/controllers/dirty_form_controller.js
@@ -4,23 +4,13 @@ import { Controller } from "@hotwired/stimulus";
 //
 // Tracks whether a form has unsaved changes and warns before navigating away.
 // Uses three layers of protection:
-//   1. turbo:before-visit   – catches Turbo Drive link clicks (custom message)
-//   2. beforeunload         – catches non-Turbo navigations / tab close (browser message)
-//   3. confirmCancel action – explicit cancel button binding (custom message)
+//   1. turbo:before-visit   - catches Turbo Drive link clicks (custom message)
+//   2. beforeunload         - catches non-Turbo navigations / tab close (browser message)
+//   3. confirmCancel action - explicit cancel button binding (custom message)
 //
 export default class extends Controller {
   connect() {
     this.dirty = false;
-    this.handleChange = () => (this.dirty = true);
-
-    this.element.addEventListener("input", this.handleChange);
-    this.element.addEventListener("change", this.handleChange);
-    this.element.addEventListener("cocoon:after-insert", this.handleChange);
-    this.element.addEventListener("cocoon:after-remove", this.handleChange);
-
-    // Clear dirty flag on form submit so guards don't block saving
-    this.handleSubmit = () => (this.dirty = false);
-    this.element.addEventListener("submit", this.handleSubmit);
 
     // Guard Turbo Drive navigations (links without data-turbo="false")
     this.handleTurboVisit = (event) => {
@@ -40,6 +30,14 @@ export default class extends Controller {
     window.addEventListener("beforeunload", this.handleBeforeUnload);
   }
 
+  markDirty() {
+    this.dirty = true;
+  }
+
+  clearDirty() {
+    this.dirty = false;
+  }
+
   confirmCancel(event) {
     if (this.dirty) {
       if (!confirm("You have unsaved changes. Are you sure you want to leave?")) {
@@ -51,11 +49,6 @@ export default class extends Controller {
   }
 
   disconnect() {
-    this.element.removeEventListener("input", this.handleChange);
-    this.element.removeEventListener("change", this.handleChange);
-    this.element.removeEventListener("cocoon:after-insert", this.handleChange);
-    this.element.removeEventListener("cocoon:after-remove", this.handleChange);
-    this.element.removeEventListener("submit", this.handleSubmit);
     document.removeEventListener("turbo:before-visit", this.handleTurboVisit);
     window.removeEventListener("beforeunload", this.handleBeforeUnload);
   }
diff --git a/app/frontend/javascript/controllers/dropdown_controller.js b/app/frontend/javascript/controllers/dropdown_controller.js
index 7e9a1831f..30ffdc291 100644
--- a/app/frontend/javascript/controllers/dropdown_controller.js
+++ b/app/frontend/javascript/controllers/dropdown_controller.js
@@ -14,9 +14,9 @@ import { Controller } from "@hotwired/stimulus";
 export default class extends Controller {
   // add a content target if you want the dropdown to close with "escape button" or clicking outside of content
   static targets = ["content"];
+  static values = { open: Boolean };
 
   connect() {
-    this.open = false;
     this.handleOutsideClick = this.handleOutsideClick.bind(this);
     this.handleEscapeKey = this.handleEscapeKey.bind(this);
   }
@@ -24,7 +24,7 @@ export default class extends Controller {
   toggle(event) {
     this.processPayload(event.params.payload);
     this.manageEventListeners();
-    this.open = !this.open;
+    this.openValue = !this.openValue;
   }
 
   toggleClassesOnElement(element, classString) {
@@ -41,7 +41,7 @@ export default class extends Controller {
   }
 
   manageEventListeners() {
-    if (this.open) {
+    if (this.openValue) {
       this.removeEventListeners();
     } else {
       // Add delay to avoid immediate trigger
@@ -79,10 +79,10 @@ export default class extends Controller {
   }
 
   close() {
-    if (this.open) {
+    if (this.openValue) {
       this.processPayload(this.lastPayload);
       this.removeEventListeners();
-      this.open = false;
+      this.openValue = false;
     }
   }
 
diff --git a/app/frontend/javascript/controllers/index.js b/app/frontend/javascript/controllers/index.js
index 48d8f2f28..62e020ddd 100644
--- a/app/frontend/javascript/controllers/index.js
+++ b/app/frontend/javascript/controllers/index.js
@@ -66,9 +66,6 @@ application.register("print-options", PrintOptionsController)
 import RhinoSourceController from "./rhino_source_controller"
 application.register("rhino-source", RhinoSourceController)
 
-import SearchBoxController from "./search_box_controller"
-application.register("search-box", SearchBoxController)
-
 import SearchableCheckboxController from "./searchable_checkbox_controller"
 application.register("searchable-checkbox", SearchableCheckboxController)
 
diff --git a/app/frontend/javascript/controllers/nav_controller.js b/app/frontend/javascript/controllers/nav_controller.js
deleted file mode 100644
index 5d9af9832..000000000
--- a/app/frontend/javascript/controllers/nav_controller.js
+++ /dev/null
@@ -1,24 +0,0 @@
-import { Controller } from "@hotwired/stimulus"
-
-export default class extends Controller {
-    static targets = ["section", "caret"]
-
-    toggleSection(event) {
-        const sectionName = event.currentTarget.dataset.section
-
-        this.sectionTargets.forEach((section) => {
-            section.classList.toggle(
-                "hidden",
-                section.dataset.section !== sectionName || !section.classList.contains("hidden")
-            )
-        })
-
-        this.caretTargets.forEach((caret) => {
-            if (caret.dataset.section === sectionName) {
-                caret.classList.toggle("rotate-180")
-            } else {
-                caret.classList.remove("rotate-180")
-            }
-        })
-    }
-}
diff --git a/app/frontend/javascript/controllers/paginated_fields_controller.js b/app/frontend/javascript/controllers/paginated_fields_controller.js
index 5bb1b4674..8562903da 100644
--- a/app/frontend/javascript/controllers/paginated_fields_controller.js
+++ b/app/frontend/javascript/controllers/paginated_fields_controller.js
@@ -2,10 +2,12 @@ import { Controller } from "@hotwired/stimulus";
 
 export default class extends Controller {
   static targets = ["item", "nav"];
-  static values = { perPage: { type: Number, default: 10 } };
+  static values = {
+    perPage: { type: Number, default: 10 },
+    currentPage: { type: Number, default: 1 }
+  };
 
   connect() {
-    this.currentPage = 1;
     this.render();
   }
 
@@ -20,30 +22,30 @@ export default class extends Controller {
   }
 
   next() {
-    if (this.currentPage < this.totalPages) {
-      this.currentPage++;
+    if (this.currentPageValue < this.totalPages) {
+      this.currentPageValue++;
       this.render();
     }
   }
 
   previous() {
-    if (this.currentPage > 1) {
-      this.currentPage--;
+    if (this.currentPageValue > 1) {
+      this.currentPageValue--;
       this.render();
     }
   }
 
   render() {
     const items = this.visibleItems;
-    const start = (this.currentPage - 1) * this.perPageValue;
+    const start = (this.currentPageValue - 1) * this.perPageValue;
     const end = start + this.perPageValue;
 
     items.forEach((el, i) => {
       el.classList.toggle("hidden", i < start || i >= end);
     });
 
-    if (this.currentPage > this.totalPages) {
-      this.currentPage = this.totalPages;
+    if (this.currentPageValue > this.totalPages) {
+      this.currentPageValue = this.totalPages;
       this.render();
       return;
     }
@@ -63,16 +65,16 @@ export default class extends Controller {
     this.navTarget.innerHTML = `
       <div class="flex items-center justify-center gap-3 mt-4">
         <button type="button" data-action="paginated-fields#previous"
-                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPage <= 1 ? 'opacity-30 cursor-default' : ''}"
-                ${this.currentPage <= 1 ? 'disabled' : ''}>
+                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPageValue <= 1 ? 'opacity-30 cursor-default' : ''}"
+                ${this.currentPageValue <= 1 ? 'disabled' : ''}>
           &laquo;
         </button>
         <span class="text-sm text-gray-600">
-          ${this.currentPage} / ${this.totalPages}
+          ${this.currentPageValue} / ${this.totalPages}
         </span>
         <button type="button" data-action="paginated-fields#next"
-                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPage >= this.totalPages ? 'opacity-30 cursor-default' : ''}"
-                ${this.currentPage >= this.totalPages ? 'disabled' : ''}>
+                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPageValue >= this.totalPages ? 'opacity-30 cursor-default' : ''}"
+                ${this.currentPageValue >= this.totalPages ? 'disabled' : ''}>
           &raquo;
         </button>
       </div>
diff --git a/app/frontend/javascript/controllers/password_toggle_controller.js b/app/frontend/javascript/controllers/password_toggle_controller.js
index a95cf5925..1e89e06db 100644
--- a/app/frontend/javascript/controllers/password_toggle_controller.js
+++ b/app/frontend/javascript/controllers/password_toggle_controller.js
@@ -1,13 +1,13 @@
 import { Controller } from "@hotwired/stimulus"
 
 export default class extends Controller {
-    static targets = ["input", "eyeOpen", "eyeClosed"];
+  static targets = ["input", "eyeOpen", "eyeClosed"];
 
-    toggle() {
-        const isPassword = this.inputTarget.type === "password"
-        this.inputTarget.type = isPassword ? "text" : "password"
+  toggle() {
+    const isPassword = this.inputTarget.type === "password"
+    this.inputTarget.type = isPassword ? "text" : "password"
 
-        this.eyeOpenTarget.style.display = isPassword ? "none" : ""
-        this.eyeClosedTarget.style.display = isPassword ? "" : "none"
-    }
+    this.eyeOpenTarget.classList.toggle("hidden", isPassword)
+    this.eyeClosedTarget.classList.toggle("hidden", !isPassword)
+  }
 }
diff --git a/app/frontend/javascript/controllers/print_options_controller.js b/app/frontend/javascript/controllers/print_options_controller.js
index 87b60b9d1..34608df86 100644
--- a/app/frontend/javascript/controllers/print_options_controller.js
+++ b/app/frontend/javascript/controllers/print_options_controller.js
@@ -3,34 +3,24 @@ import { Controller } from "@hotwired/stimulus";
 // Connects to data-controller="print-options"
 
 export default class extends Controller {
+  static targets = ["toggle", "checkIcon", "xIcon"];
   static values = {
     images: Boolean,
   };
 
   connect() {
-    this.toggle = document.getElementById("print-images-toggle");
-    if (!this.toggle) return;
-
-    //ensure value matches icon
     this.updateIcons();
-
-    this.toggle.addEventListener("click", this.handleToggle);
-  }
-
-  disconnect() {
-    if (!this.toggle) return;
-    this.toggle.removeEventListener("click", this.handleToggle);
   }
 
-  handleToggle = (event) => {
+  handleToggle() {
     this.imagesValue = !this.imagesValue;
     this.updateIcons();
-  };
+  }
+
   updateIcons() {
-    const checkIcon = document.getElementById("image-check-icon");
-    const xIcon = document.getElementById("image-x-icon");
+    if (!this.hasCheckIconTarget || !this.hasXIconTarget) return;
 
-    checkIcon.classList.toggle("hidden", !this.imagesValue);
-    xIcon.classList.toggle("hidden", this.imagesValue);
+    this.checkIconTarget.classList.toggle("hidden", !this.imagesValue);
+    this.xIconTarget.classList.toggle("hidden", this.imagesValue);
   }
 }
diff --git a/app/frontend/javascript/controllers/remote_select_controller.js b/app/frontend/javascript/controllers/remote_select_controller.js
index 0c3d0b7d2..c291b63b8 100644
--- a/app/frontend/javascript/controllers/remote_select_controller.js
+++ b/app/frontend/javascript/controllers/remote_select_controller.js
@@ -1,6 +1,8 @@
 import { Controller } from "@hotwired/stimulus";
 import TomSelect from "tom-select";
 
+const STYLE_ID = "remote-select-overrides";
+
 export default class extends Controller {
   static values = { model: String, exclude: String };
 
@@ -26,31 +28,33 @@ export default class extends Controller {
           .catch(() => callback());
       },
     });
-    // Inject CSS to remove some default tom-select styles -might be a better way to do this.
-    const style = document.createElement("style");
-    style.textContent = `
-      /* Remove border and shadow */
-      .ts-control .ts-input {
-        border: none !important;
-        box-shadow: none !important;
-        outline: none !important;
-        background: transparent !important;
-        padding: 0 !important;           /* Remove padding from input */
-        margin: 0 !important;            /* Remove margin if any */
-      }
-      .ts-control {
-        border: none !important;
-        box-shadow: none !important;
-        padding: 0 !important;           /* Remove padding from container */
-        margin: 0 !important;
-        min-height: 0 !important;        /* Remove min-height TomSelect sets */
-      }
-      .ts-control .item {
-        margin: 0 !important;            /* Remove padding/margin from selected items */
-        padding: 0 !important;
-      }
-    `;
-    document.head.appendChild(style);
+
+    if (!document.getElementById(STYLE_ID)) {
+      const style = document.createElement("style");
+      style.id = STYLE_ID;
+      style.textContent = `
+        .ts-control .ts-input {
+          border: none !important;
+          box-shadow: none !important;
+          outline: none !important;
+          background: transparent !important;
+          padding: 0 !important;
+          margin: 0 !important;
+        }
+        .ts-control {
+          border: none !important;
+          box-shadow: none !important;
+          padding: 0 !important;
+          margin: 0 !important;
+          min-height: 0 !important;
+        }
+        .ts-control .item {
+          margin: 0 !important;
+          padding: 0 !important;
+        }
+      `;
+      document.head.appendChild(style);
+    }
   }
 
   disconnect() {
diff --git a/app/frontend/javascript/controllers/search_box_controller.js b/app/frontend/javascript/controllers/search_box_controller.js
deleted file mode 100644
index 5b9f30d52..000000000
--- a/app/frontend/javascript/controllers/search_box_controller.js
+++ /dev/null
@@ -1,14 +0,0 @@
-import { Controller } from "@hotwired/stimulus"
-
-console.log("💡 search_box_controller.js loaded by Vite")
-
-export default class extends Controller {
-    static targets = ["select"]
-
-    connect() {
-        console.log("✅ search-box connected")
-        this.selectTargets.forEach(select => {
-            select.addEventListener("change", () => this.element.requestSubmit())
-        })
-    }
-}
diff --git a/app/frontend/javascript/controllers/tabs_controller.js b/app/frontend/javascript/controllers/tabs_controller.js
index b692b88ea..e5a7b27ff 100644
--- a/app/frontend/javascript/controllers/tabs_controller.js
+++ b/app/frontend/javascript/controllers/tabs_controller.js
@@ -1,37 +1,33 @@
 import { Controller } from "@hotwired/stimulus"
 
 export default class extends Controller {
-    static targets = ["tabLink", "tabContent"]
+  static targets = ["tabLink", "tabContent"]
 
-    connect() {
-        // Hide all tab contents except the first one
-        this.tabContentTargets.forEach((c, i) => {
-            c.classList.toggle("hidden", i !== 0)
-        })
-        // Make the first tab link active
-        this.tabLinkTargets.forEach((l, i) => {
-            l.classList.toggle("is-active", i === 0)
-        })
-    }
-
-    showTab(link) {
-        // Remove active class from all links
-        this.tabLinkTargets.forEach(l => l.classList.remove("is-active"))
-        // Hide all tab contents
-        this.tabContentTargets.forEach(c => c.classList.add("hidden"))
+  connect() {
+    // Hide all tab contents except the first one
+    this.tabContentTargets.forEach((c, i) => {
+      c.classList.toggle("hidden", i !== 0)
+    })
+    // Make the first tab link active
+    this.tabLinkTargets.forEach((l, i) => {
+      l.classList.toggle("is-active", i === 0)
+    })
+  }
 
-        // Add active class to clicked link
-        link.classList.add("is-active")
+  switch(event) {
+    event.preventDefault()
+    const index = this.tabLinkTargets.indexOf(event.currentTarget)
+    if (index === -1) return
 
-        // Show the corresponding content
-        const contentId = link.dataset.tabContentId
-        if (!contentId) return
-        const content = document.getElementById(contentId)
-        if (content) content.classList.remove("hidden")
-    }
+    // Remove active class from all links
+    this.tabLinkTargets.forEach(l => l.classList.remove("is-active"))
+    // Hide all tab contents
+    this.tabContentTargets.forEach(c => c.classList.add("hidden"))
 
-    switch(event) {
-        event.preventDefault()
-        this.showTab(event.currentTarget)
+    // Activate clicked tab
+    event.currentTarget.classList.add("is-active")
+    if (this.tabContentTargets[index]) {
+      this.tabContentTargets[index].classList.remove("hidden")
     }
+  }
 }
diff --git a/app/frontend/javascript/controllers/tags_combination_highlight_controller.js b/app/frontend/javascript/controllers/tags_combination_highlight_controller.js
index 47896ef34..f08740c7d 100644
--- a/app/frontend/javascript/controllers/tags_combination_highlight_controller.js
+++ b/app/frontend/javascript/controllers/tags_combination_highlight_controller.js
@@ -5,25 +5,19 @@ import { Controller } from "@hotwired/stimulus"
 const HIGHLIGHT_CLASS = "tags-combination-selected"
 
 export default class extends Controller {
-  connect() {
-    this.boundSync = this.syncHighlights.bind(this)
-    this.sectorInputs = () => this.element.querySelectorAll('input[name="sector_names_all[]"]')
-    this.categoryInputs = () => this.element.querySelectorAll('input[name="category_names_all[]"]')
-
-    this.sectorInputs().forEach(input => input.addEventListener("change", this.boundSync))
-    this.categoryInputs().forEach(input => input.addEventListener("change", this.boundSync))
+  static targets = ["sectorInput", "categoryInput"]
 
+  connect() {
+    this.handleFrameLoad = this.handleFrameLoad.bind(this)
     document.addEventListener("turbo:frame-load", this.handleFrameLoad)
     this.syncHighlights()
   }
 
   disconnect() {
-    this.sectorInputs().forEach(input => input.removeEventListener("change", this.boundSync))
-    this.categoryInputs().forEach(input => input.removeEventListener("change", this.boundSync))
     document.removeEventListener("turbo:frame-load", this.handleFrameLoad)
   }
 
-  handleFrameLoad = (event) => {
+  handleFrameLoad(event) {
     const frame = event?.detail?.frameElement
     if (frame?.id === "sectors_tags" || frame?.id === "categories_tags") {
       this.syncHighlights()
@@ -36,7 +30,7 @@ export default class extends Controller {
   }
 
   syncSectorHighlights() {
-    const selected = Array.from(this.sectorInputs())
+    const selected = this.sectorInputTargets
       .filter(el => el.checked)
       .map(el => el.value.trim())
     const frame = document.getElementById("sectors_tags")
@@ -48,7 +42,7 @@ export default class extends Controller {
   }
 
   syncCategoryHighlights() {
-    const selected = Array.from(this.categoryInputs())
+    const selected = this.categoryInputTargets
       .filter(el => el.checked)
       .map(el => el.value.trim())
     const frame = document.getElementById("categories_tags")
diff --git a/app/frontend/javascript/controllers/timeframe_controller.js b/app/frontend/javascript/controllers/timeframe_controller.js
index cb4f45390..f215c9586 100644
--- a/app/frontend/javascript/controllers/timeframe_controller.js
+++ b/app/frontend/javascript/controllers/timeframe_controller.js
@@ -2,31 +2,27 @@ import { Controller } from "@hotwired/stimulus"
 
 // Connects to data-controller="timeframe"
 export default class extends Controller {
-    static targets = ["input", "output"]
+  static targets = ["input", "output"]
 
-    connect() {
-        console.log("✅ Timeframe controller connected")
-        this.update()
-    }
+  connect() {
+    this.update()
+  }
 
-    update() {
-        console.log("🔁 Updating timeframe...")
-        let totalMinutes = this.inputTargets
-            .map(el => parseInt(el.value || 0, 10))
-            .reduce((a, b) => a + b, 0)
+  update() {
+    let totalMinutes = this.inputTargets
+      .map(el => parseInt(el.value || 0, 10))
+      .reduce((a, b) => a + b, 0)
 
-        // ✅ Always round up to the next 15 minutes
-        totalMinutes = Math.ceil(totalMinutes / 15) * 15
+    totalMinutes = Math.ceil(totalMinutes / 15) * 15
 
-        const hours = Math.floor(totalMinutes / 60)
-        const minutes = totalMinutes % 60
+    const hours = Math.floor(totalMinutes / 60)
+    const minutes = totalMinutes % 60
 
-        const formatted =
-            hours > 0
-                ? `${hours}:${minutes.toString().padStart(2, "0")} hours`
-                : `${minutes} mins`
+    const formatted =
+      hours > 0
+        ? `${hours}:${minutes.toString().padStart(2, "0")} hours`
+        : `${minutes} mins`
 
-        console.log("→ totalMinutes:", totalMinutes, "formatted:", formatted)
-        this.outputTarget.textContent = formatted
-    }
+    this.outputTarget.textContent = formatted
+  }
 }
diff --git a/app/frontend/javascript/controllers/toggle_lock_controller.js b/app/frontend/javascript/controllers/toggle_lock_controller.js
index b64cb351d..dffed0430 100644
--- a/app/frontend/javascript/controllers/toggle_lock_controller.js
+++ b/app/frontend/javascript/controllers/toggle_lock_controller.js
@@ -5,7 +5,7 @@ import { Controller } from "@hotwired/stimulus";
  *
  * Targets:
  *   checkbox  - the <input type="checkbox">
- *   icon      - lock icon elements (shown when checked, removed when unchecked)
+ *   icon      - lock icon elements (shown when checked, hidden when unchecked)
  *   labelText - the text span toggled between "Lock" / "Locked"
  */
 export default class extends Controller {
@@ -19,7 +19,7 @@ export default class extends Controller {
     }
 
     this.iconTargets.forEach((el) => {
-      el.style.display = locked ? "" : "none";
+      el.classList.toggle("hidden", !locked);
     });
   }
 }
diff --git a/app/frontend/javascript/controllers/toggle_user_icon_controller.js b/app/frontend/javascript/controllers/toggle_user_icon_controller.js
index 34b7ea84f..1f929f12d 100644
--- a/app/frontend/javascript/controllers/toggle_user_icon_controller.js
+++ b/app/frontend/javascript/controllers/toggle_user_icon_controller.js
@@ -22,7 +22,7 @@ export default class extends Controller {
     }
 
     this.lockIconTargets.forEach((el) => {
-      el.style.display = locked ? "" : "none";
+      el.classList.toggle("hidden", !locked);
     });
   }
 
@@ -30,7 +30,7 @@ export default class extends Controller {
     const admin = this.adminCheckboxTarget.checked;
 
     this.adminIconTargets.forEach((el) => {
-      el.style.display = admin ? "" : "none";
+      el.classList.toggle("hidden", !admin);
     });
   }
 }
diff --git a/app/helpers/analytics_helper.rb b/app/helpers/analytics_helper.rb
index 25e40c7c1..16980e831 100644
--- a/app/helpers/analytics_helper.rb
+++ b/app/helpers/analytics_helper.rb
@@ -30,13 +30,13 @@ def print_button(record,
 
     toggle_div = if image_toggle
       tag.div(
-      id: "print-images-toggle",
       title: "With or without images",
-      class: "inline-flex items-center cursor-pointer select-none ml-2 border-l-2 pl-2"
+      class: "inline-flex items-center cursor-pointer select-none ml-2 border-l-2 pl-2",
+      data: { print_options_target: "toggle", action: "click->print-options#handleToggle" }
     ) do
       safe_join([
-        icon("image-check", class: "w-5 h-5 animate-fade", id: "image-check-icon"),
-        icon("image-x", class: "w-5 h-5 animate-fade hidden", id: "image-x-icon")
+        icon("image-check", class: "w-5 h-5 animate-fade", "data-print-options-target": "checkIcon"),
+        icon("image-x", class: "w-5 h-5 animate-fade hidden", "data-print-options-target": "xIcon")
       ])
     end
     end
diff --git a/app/views/comments/_comment_fields.html.erb b/app/views/comments/_comment_fields.html.erb
index c1047b7f2..5fbafe0d1 100644
--- a/app/views/comments/_comment_fields.html.erb
+++ b/app/views/comments/_comment_fields.html.erb
@@ -32,7 +32,7 @@
 <div class="nested-fields" data-paginated-fields-target="item">
   <%= f.hidden_field :id if f.object.persisted? %>
   <% if f.object.persisted? %>
-    <div class="comment-view flex items-center mb-3 pl-4 gap-x-1">
+    <div class="comment-view flex items-center mb-3 pl-4 gap-x-1" data-comment-edit-toggle-target="commentView">
       <span class="text-xs text-gray-400 whitespace-nowrap shrink-0">
         <%= f.object.created_at.strftime("%-m/%-d/%Y %-I:%M %p") %>
       </span>
@@ -48,7 +48,7 @@
         <%= truncate(f.object.body, length: 135) %>
       </span>
     </div>
-    <div class="comment-edit flex items-start mb-1 pl-4 gap-x-1" style="display:none">
+    <div class="comment-edit flex items-start mb-1 pl-4 gap-x-1 hidden" data-comment-edit-toggle-target="commentEdit">
       <span class="text-xs text-gray-400 whitespace-nowrap shrink-0 mt-2">
         <%= f.object.created_at.strftime("%-m/%-d/%Y %-I:%M %p") %>
       </span>
diff --git a/app/views/devise/passwords/edit.html.erb b/app/views/devise/passwords/edit.html.erb
index 37920a33e..172d907e6 100644
--- a/app/views/devise/passwords/edit.html.erb
+++ b/app/views/devise/passwords/edit.html.erb
@@ -27,7 +27,7 @@
                       data: { password_toggle_target: "input" } %>
           <button type="button" class="toggle-password" data-action="password-toggle#toggle" aria-label="Toggle password visibility">
             <i class="fa-solid fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
         <p class="text-sm text-gray-500 mt-1">Password must be at least 5 characters long</p>
@@ -49,7 +49,7 @@
                       data: { password_toggle_target: "input" } %>
           <button type="button" class="toggle-password" data-action="password-toggle#toggle" aria-label="Toggle password visibility">
             <i class="fa-solid fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
       </div>
diff --git a/app/views/devise/sessions/new.html.erb b/app/views/devise/sessions/new.html.erb
index ea5d1d650..a5d7af73a 100644
--- a/app/views/devise/sessions/new.html.erb
+++ b/app/views/devise/sessions/new.html.erb
@@ -35,7 +35,7 @@
             data-action="password-toggle#toggle"
             aria-label="Toggle password visibility">
             <i class="fa fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
       </div>
diff --git a/app/views/event_registrations/_form.html.erb b/app/views/event_registrations/_form.html.erb
index 62d26ba33..c98d04b02 100644
--- a/app/views/event_registrations/_form.html.erb
+++ b/app/views/event_registrations/_form.html.erb
@@ -137,7 +137,7 @@
                   style="float:right"
                   data-action="click->comment-edit-toggle#toggle">
             <span data-comment-edit-toggle-target="editLabel">Edit Comments</span>
-            <span data-comment-edit-toggle-target="viewLabel" style="display:none">Done Editing</span>
+            <span data-comment-edit-toggle-target="viewLabel" class="hidden">Done Editing</span>
           </button>
           <%= link_to_add_association "➕ Add Comment",
                                       f,
diff --git a/app/views/events/_manage_results.html.erb b/app/views/events/_manage_results.html.erb
index 3b4c0707f..39487a542 100644
--- a/app/views/events/_manage_results.html.erb
+++ b/app/views/events/_manage_results.html.erb
@@ -26,7 +26,7 @@
               <% if @event.cost_cents.to_i > 0 %>
                 <th class="px-4 py-2 text-left text-sm font-semibold text-gray-700">Payment status</th>
               <% end %>
-              <th class="px-4 py-2 text-center text-sm font-semibold text-gray-700 w-[60px] hidden" data-column-toggle-col>Confirmed</th>
+              <th class="px-4 py-2 text-center text-sm font-semibold text-gray-700 w-[60px] hidden" data-column-toggle-target="col">Confirmed</th>
               <th class="px-4 py-2 text-left text-sm font-semibold text-gray-700">Attendance status</th>
               <th class="px-4 py-2"></th>
             </tr>
@@ -112,7 +112,7 @@
                     <% end %>
                   </td>
                 <% end %>
-                <td class="px-4 py-2 text-center align-top hidden" data-column-toggle-col>
+                <td class="px-4 py-2 text-center align-top hidden" data-column-toggle-target="col">
                   <% if person.user.present? %>
                     <%= render "users/confirmed_status", user: person.user, invite_params: {} %>
                   <% else %>
diff --git a/app/views/organizations/_form.html.erb b/app/views/organizations/_form.html.erb
index 9747020e0..14101ae7d 100644
--- a/app/views/organizations/_form.html.erb
+++ b/app/views/organizations/_form.html.erb
@@ -373,7 +373,7 @@
             data-action="click->comment-edit-toggle#toggle"
           >
             <span data-comment-edit-toggle-target="editLabel">Edit Comments</span>
-            <span data-comment-edit-toggle-target="viewLabel" style="display:none">Done Editing</span>
+            <span data-comment-edit-toggle-target="viewLabel" class="hidden">Done Editing</span>
           </button>
 
           <%= link_to_add_association "➕ Add Comment",
diff --git a/app/views/people/_form.html.erb b/app/views/people/_form.html.erb
index 204639580..856e2d0fc 100644
--- a/app/views/people/_form.html.erb
+++ b/app/views/people/_form.html.erb
@@ -1,5 +1,6 @@
 <%= simple_form_for @person, html: { multipart: true, class: "space-y-8",
-                                          data: { turbo: false, controller: "dirty-form affiliation-dates" } } do |f| %>
+                                          data: { turbo: false, controller: "dirty-form affiliation-dates",
+                                                  action: "input->dirty-form#markDirty change->dirty-form#markDirty cocoon:after-insert->dirty-form#markDirty cocoon:after-remove->dirty-form#markDirty submit->dirty-form#clearDirty" } } do |f| %>
   <%= render 'shared/errors', resource: @person if @person.errors.any? %>
 
   <%= f.hidden_field :created_by_id, value: current_user&.id unless f.object.persisted? %>
@@ -401,7 +402,7 @@
                   style="float:right"
                   data-action="click->comment-edit-toggle#toggle">
             <span data-comment-edit-toggle-target="editLabel">Edit Comments</span>
-            <span data-comment-edit-toggle-target="viewLabel" style="display:none">Done Editing</span>
+            <span data-comment-edit-toggle-target="viewLabel" class="hidden">Done Editing</span>
           </button>
           <%= link_to_add_association "➕ Add Comment",
                                       f,
diff --git a/app/views/taggings/_explore_by_combination.html.erb b/app/views/taggings/_explore_by_combination.html.erb
index f167469bb..8a1b7aa72 100644
--- a/app/views/taggings/_explore_by_combination.html.erb
+++ b/app/views/taggings/_explore_by_combination.html.erb
@@ -30,6 +30,8 @@
                      name="sector_names_all[]"
                      value="<%= sector.name %>"
                      data-tags-sync-list-heights-target="sector"
+                     data-tags-combination-highlight-target="sectorInput"
+                     data-action="change->tags-combination-highlight#syncHighlights"
                      <%= "checked" if selected_sectors.include?(sector.name) %>
                      class="rounded border-gray-300 text-lime-600
                           focus:ring-lime-500" />
@@ -74,6 +76,8 @@
                      name="category_names_all[]"
                      value="<%= category.name %>"
                      data-tags-sync-list-heights-target="category"
+                     data-tags-combination-highlight-target="categoryInput"
+                     data-action="change->tags-combination-highlight#syncHighlights"
                      <%= "checked" if selected_categories.include?(category.name.downcase) %>
                      class="rounded border-gray-300 text-lime-600
                         focus:ring-lime-500" />
diff --git a/app/views/users/_form.html.erb b/app/views/users/_form.html.erb
index f4c1a93df..db949907b 100644
--- a/app/views/users/_form.html.erb
+++ b/app/views/users/_form.html.erb
@@ -1,5 +1,6 @@
 <%= simple_form_for @user, html: { multipart: true, class: "space-y-6",
-                                          data: { controller: "dirty-form" } } do |f| %>
+                                          data: { controller: "dirty-form",
+                                                  action: "input->dirty-form#markDirty change->dirty-form#markDirty submit->dirty-form#clearDirty" } } do |f| %>
   <%= render "shared/errors", resource: @user if @user.errors.any? %>
   <%= f.hidden_field :person_id, value: @person&.id || f.object.person_id || params[:person_id] %>
 
@@ -46,8 +47,7 @@
                                         "toggle-user-icon-target": "lockCheckbox" } },
                               "1", "0" %>
               <span data-toggle-user-icon-target="lockLabel"><%= f.object.locked_at.present? ? "Locked" : "Lock" %></span>
-              <i class="fa-solid fa-lock text-red-500" data-toggle-user-icon-target="lockIcon"
-                 style="<%= 'display:none' unless f.object.locked_at.present? %>"></i>
+              <i class="fa-solid fa-lock text-red-500 <%= 'hidden' unless f.object.locked_at.present? %>" data-toggle-user-icon-target="lockIcon"></i>
             </label>
             <label class="flex items-center gap-2 cursor-pointer">
               <%= f.hidden_field :super_user, value: "0" %>
@@ -57,8 +57,7 @@
                                         "toggle-user-icon-target": "adminCheckbox" } },
                               "1", "0" %>
               <span>Admin</span>
-              <i class="fa-solid fa-user-shield text-blue-500" data-toggle-user-icon-target="adminIcon"
-                 style="<%= 'display:none' unless f.object.super_user? %>"></i>
+              <i class="fa-solid fa-user-shield text-blue-500 <%= 'hidden' unless f.object.super_user? %>" data-toggle-user-icon-target="adminIcon"></i>
             </label>
           </div>
         </div>
@@ -132,7 +131,7 @@
                   class="text-sm text-gray-400 hover:text-blue-600 underline cursor-pointer whitespace-nowrap"
                   data-action="click->comment-edit-toggle#toggle">
             <span data-comment-edit-toggle-target="editLabel">Edit Comments</span>
-            <span data-comment-edit-toggle-target="viewLabel" style="display:none">Done Editing</span>
+            <span data-comment-edit-toggle-target="viewLabel" class="hidden">Done Editing</span>
           </button>
         </div>
       </div>
diff --git a/app/views/users/change_password.html.erb b/app/views/users/change_password.html.erb
index 9e08ff70e..ac02121d0 100644
--- a/app/views/users/change_password.html.erb
+++ b/app/views/users/change_password.html.erb
@@ -22,7 +22,7 @@
                                data: { password_toggle_target: "input" } %>
           <button type="button" class="toggle-password" data-action="password-toggle#toggle" aria-label="Toggle password visibility">
             <i class="fa fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
       </div>
@@ -53,7 +53,7 @@
                                data: { password_toggle_target: "input" } %>
           <button type="button" class="toggle-password" data-action="password-toggle#toggle" aria-label="Toggle password visibility">
             <i class="fa fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
         <p class="text-sm text-gray-500 mt-1">Password must be at least 5 characters long</p>
@@ -75,7 +75,7 @@
                                data: { password_toggle_target: "input" } %>
           <button type="button" class="toggle-password" data-action="password-toggle#toggle" aria-label="Toggle password visibility">
             <i class="fa fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
       </div>
diff --git a/app/views/users/edit.html.erb b/app/views/users/edit.html.erb
index 8fa1ccb2b..463278149 100644
--- a/app/views/users/edit.html.erb
+++ b/app/views/users/edit.html.erb
@@ -43,10 +43,8 @@
             <div class="flex-1 min-w-0">
               <h1 class="font-bold text-2xl text-gray-900">
                 <%= @user.email %>
-                <i class="fa-solid fa-user-shield text-blue-500 text-lg" data-toggle-user-icon-target="adminIcon"
-                   style="<%= 'display:none' unless @user.super_user? %>"></i>
-                <i class="fa-solid fa-lock text-red-500 text-lg" data-toggle-user-icon-target="lockIcon"
-                   style="<%= 'display:none' unless @user.locked_at.present? %>"></i>
+                <i class="fa-solid fa-user-shield text-blue-500 text-lg <%= 'hidden' unless @user.super_user? %>" data-toggle-user-icon-target="adminIcon"></i>
+                <i class="fa-solid fa-lock text-red-500 text-lg <%= 'hidden' unless @user.locked_at.present? %>" data-toggle-user-icon-target="lockIcon"></i>
               </h1>
               <% if @user.person %>
                 <p class="text-sm text-gray-500 mt-1">
diff --git a/app/views/video_recordings/_search_boxes.html.erb b/app/views/video_recordings/_search_boxes.html.erb
index 6118e0bbf..50eb218f1 100644
--- a/app/views/video_recordings/_search_boxes.html.erb
+++ b/app/views/video_recordings/_search_boxes.html.erb
@@ -1,5 +1,7 @@
 <!-- Filters -->
-<div class="mb-6 p-4 bg-white border border-gray-200 rounded-lg" data-controller="collection">
+<div class="mb-6 p-4 bg-white border border-gray-200 rounded-lg"
+     data-controller="collection"
+     data-action="change->collection#handleChange input->collection#handleInput">
   <%= form_with url: video_recordings_path,
                 method: :get,
                 data: { turbo_frame: "video_recordings_results", controller: "searchable-checkbox" },
diff --git a/app/views/welcome/show.html.erb b/app/views/welcome/show.html.erb
index b9e52ff7e..c129a4729 100644
--- a/app/views/welcome/show.html.erb
+++ b/app/views/welcome/show.html.erb
@@ -35,7 +35,7 @@
                                data: { password_toggle_target: "input" } %>
           <button type="button" class="toggle-password" data-action="password-toggle#toggle" aria-label="Toggle password visibility">
             <i class="fa fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
       </div>
@@ -57,7 +57,7 @@
                                data: { password_toggle_target: "input" } %>
           <button type="button" class="toggle-password" data-action="password-toggle#toggle" aria-label="Toggle password visibility">
             <i class="fa fa-eye" data-password-toggle-target="eyeOpen"></i>
-            <svg data-password-toggle-target="eyeClosed" style="display:none" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
+            <svg data-password-toggle-target="eyeClosed" class="hidden" width="1em" height="1em" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M2 10c3 4 6.5 6 10 6s7-2 10-6"/><path d="M7.5 14l-1.5 3"/><path d="M12 16v3"/><path d="M16.5 14l1.5 3"/></svg>
           </button>
         </div>
       </div>

From 049eb0e6f5f4a3877f91e27b922ec8dff1396e8b Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:30:23 -0400
Subject: [PATCH 02/20] Remove stale tag_deduping.rake reference, add AI files
 update rule

- tag_deduping.rake no longer exists, remove from AGENTS.md
- Add instruction to update AI files on every push when conventions,
  architecture, or file lists change

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/copilot-instructions.md | 1 +
 AGENTS.md                       | 2 +-
 CLAUDE.md                       | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
index 14aa7aefc..980f5610b 100644
--- a/.github/copilot-instructions.md
+++ b/.github/copilot-instructions.md
@@ -24,6 +24,7 @@
 - After completing work, create a pull request using `gh pr create`
 - Once the PR is created, prepend the PR number to the branch name (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
 - On every push, update the PR title and content to reflect the current diff
+- On every push, update AI instruction files if the diff introduces changes that affect conventions, architecture, or file lists documented there
 
 # Code style requirements:
 - Use modern Ruby syntax
diff --git a/AGENTS.md b/AGENTS.md
index 9607473a7..fb52f27d5 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -331,6 +331,7 @@ RuboCop linting on PRs and pushes to main.
 ## PRs
 
 - On every push, update the PR title and content to reflect the current diff
+- On every push, update AI instruction files if the diff introduces changes that affect conventions, architecture, or file lists documented there
 
 ## Git
 
@@ -342,4 +343,3 @@ Located in `lib/tasks/` (~17 files). Notable:
 - `dev.rake` — Development database seeding from XML/CSV
 - `paperclip_to_active_storage.rake` — File upload migration
 - `rhino_migrator.rake` — Rich text editor migration
-- `tag_deduping.rake` — Tag deduplication
diff --git a/CLAUDE.md b/CLAUDE.md
index 9cfe92577..dbd8f809b 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -216,6 +216,7 @@ Follow the [Stimulus Handbook](https://stimulus.hotwired.dev/handbook/introducti
 - Description must explain why the change was made, not just what
 - Include screenshots for UI changes
 - **On every push**, update the PR title and content to reflect the current diff
+- **On every push**, update AI instruction files if the diff introduces changes that affect conventions, architecture, or file lists documented there
 
 ## Quick Commands
 

From 72209b2ab745b81c5cba9e55b906c5691bbf75f0 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:33:31 -0400
Subject: [PATCH 03/20] =?UTF-8?q?Deduplicate=20AI=20instruction=20files=20?=
 =?UTF-8?q?=E2=80=94=20CLAUDE.md=20is=20single=20source=20of=20truth?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- copilot-instructions.md now just points to CLAUDE.md and AGENTS.md
- AGENTS.md removes duplicated PR, Git, and frontend preference sections
- CLAUDE.md updated to reflect the new structure

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/copilot-instructions.md | 99 +--------------------------------
 AGENTS.md                       | 22 +-------
 CLAUDE.md                       |  9 ++-
 3 files changed, 8 insertions(+), 122 deletions(-)

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
index 980f5610b..4278d3f73 100644
--- a/.github/copilot-instructions.md
+++ b/.github/copilot-instructions.md
@@ -1,98 +1,5 @@
-# This project is a Ruby on Rails application.
-<!-- Keep code style rules in sync with CLAUDE.md -->
+# Copilot Instructions
 
-# Frontend requirements:
-- Strongly prefer Stimulus for JavaScript behavior — do not write raw/inline JS or jQuery
-- Always use Tailwind CSS utility classes for styling — do not write custom CSS unless absolutely necessary
-- Prefer Turbo for navigation and form submissions before reaching for Stimulus
-- ES6+ syntax, ESM imports/exports
-- Stimulus controller naming: `[name]_controller.js`
+Read and follow all rules in `CLAUDE.md` at the project root — it is the single source of truth for coding conventions, style rules, Stimulus conventions, HTML/ERB formatting, Git workflow, and PR process.
 
-# Stimulus conventions:
-- Use `static targets` and `data-[controller]-target` — never `querySelector` or `getElementById` for elements that could be targets
-- Use `static values = { name: Type }` for state — not instance variables. Use `[name]ValueChanged()` for reactive updates
-- Use `data-action` attributes — not `addEventListener` in `connect()`. Omit default events (`click` for buttons, `input` for inputs, `submit` for forms, `change` for selects)
-- Use `@window`/`@document` suffixes for global events in data-action
-- Use action options (`:prevent`, `:stop`) instead of `event.preventDefault()` in methods
-- Use `static classes` when CSS classes should be configurable from HTML
-- Use `static outlets` for cross-controller communication instead of `getElementById`
-- Always clean up in `disconnect()` anything created in `connect()` (listeners, timers, observers)
-- Use `[name]TargetConnected`/`TargetDisconnected` for dynamic DOM (cocoon, Turbo)
-- Toggle `hidden` class instead of `style.display`. Use `class="hidden"` not `style="display:none"` in HTML
-
-# PRs
-- After completing work, create a pull request using `gh pr create`
-- Once the PR is created, prepend the PR number to the branch name (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
-- On every push, update the PR title and content to reflect the current diff
-- On every push, update AI instruction files if the diff introduces changes that affect conventions, architecture, or file lists documented there
-
-# Code style requirements:
-- Use modern Ruby syntax
-- Prefer early returns and guard clauses
-- Avoid unnecessary and/or complex conditionals
-- Prefer constants and scopes over magic strings
-- Use safe navigation (`&.`) where appropriate
-- Use `presence` over blank checks
-- Use `Arel.sql` for raw SQL in order clauses
-- Avoid `update_all` unless explicitly intended
-- Prefer service objects under app/services
-- Prefer POROs over concerns when possible
-- Use `after_commit` instead of `after_save` for side effects
-
-# RuboCop (rubocop-rails-omakase)
-This project uses rubocop-rails-omakase. All code MUST follow these rules:
-
-## Strings
-- Always use double quotes: `"foo"` not `'foo'`
-
-## Spacing
-- Spaces inside array brackets: `[ a, b, c ]` not `[a, b, c]` (empty arrays: `[]`)
-- Spaces inside hash braces: `{ a: 1, b: 2 }` not `{a: 1}` (empty hashes: `{}`)
-- Spaces inside block braces: `foo { bar }` not `foo {bar}` (empty blocks: `foo { }`)
-- No spaces inside parens: `foo(bar)` not `foo( bar )`
-- No spaces inside reference brackets: `hash[:key]` not `hash[ :key ]`
-- Space before block braces: `foo { }` not `foo{ }`
-
-## Commas
-- No trailing commas in arrays, hashes, or method arguments
-
-## Indentation
-- 2-space indentation, no tabs
-- Consistent indentation at normal level — do NOT indent methods under `private`/`protected`
-- Align `end` with the variable in assignments
-- Align `when` with `end`, not with `case`
-
-## Whitespace
-- No trailing whitespace on any line
-- No trailing blank lines at end of file
-- No empty lines inside class, module, method, or block bodies
-
-## Syntax
-- Use `%w[]` and `%i[]` with square bracket delimiters (not parens)
-- Use modern hash syntax: `{ key: value }` not `{ :key => value }`
-- No redundant returns — omit `return` on last expression
-- Use `flat_map` instead of `.map { }.flatten`
-- No redundant `.to_s` inside string interpolation
-- Use `Foo.method` not `Foo::method` for method calls
-- No parentheses around conditions: `if foo` not `if (foo)`
-- No semicolons to separate statements
-
-# Git
-- When rebasing onto main, review incoming changes for their intent and flag any oversights — missing tests, incomplete migrations, broken assumptions, or conflicts between the two branches. Check both directions: schema/model changes on either branch that affect views, partials, or layouts on the other (e.g., main redesigned a table's CSS but your branch adds new columns to it, or vice versa)
-
-# HTML/ERB Formatting
-
-## Tag attributes
-- When a tag has long attributes, place the closing `>` on the same line as the last attribute
-- Do NOT put the closing `>` on its own line
-- Example (GOOD):
-  ```erb
-  <div class="relative z-10 w-full bg-white text-gray-800 py-2 px-4"
-       id="dropdown">
-  ```
-- Example (BAD):
-  ```erb
-  <div class="relative z-10 w-full bg-white text-gray-800 py-2 px-4"
-       id="dropdown"
-  >
-  ```
+Read `AGENTS.md` for architecture reference (models, controllers, services, testing structure).
diff --git a/AGENTS.md b/AGENTS.md
index fb52f27d5..c54a0ac85 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -1,6 +1,6 @@
 # AGENTS.md — AWBW Portal
 
-Architecture reference for AI agents working on the AWBW Portal codebase. For coding rules and quick commands, see `CLAUDE.md`.
+Architecture reference for AI agents working on the AWBW Portal codebase. For coding rules, conventions, and quick commands, see `CLAUDE.md` (single source of truth).
 
 ## Project Summary
 
@@ -206,12 +206,6 @@ end
 
 ## Frontend
 
-### Preferences
-
-- **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
-- **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
-- Prefer Turbo for navigation and form submissions before reaching for Stimulus
-
 ### Stimulus Controllers
 
 Key controllers:
@@ -305,11 +299,6 @@ Common factory traits across models:
 
 RuboCop linting on PRs and pushes to main.
 
-## PR Workflow
-
-- After completing work, create a pull request using `gh pr create`
-- Once the PR is created, prepend the PR number to the branch name (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
-
 ## Key Library Usage
 
 | Need | Library |
@@ -328,15 +317,6 @@ RuboCop linting on PRs and pushes to main.
 | Email styling | Premailer-rails (inline CSS) |
 | Positioning | Positioning gem for ordered records |
 
-## PRs
-
-- On every push, update the PR title and content to reflect the current diff
-- On every push, update AI instruction files if the diff introduces changes that affect conventions, architecture, or file lists documented there
-
-## Git
-
-- When rebasing onto main, review incoming changes for their intent and flag any oversights — missing tests, incomplete migrations, broken assumptions, or conflicts between the two branches. Check both directions: schema/model changes on either branch that affect views, partials, or layouts on the other (e.g., main redesigned a table's CSS but your branch adds new columns to it, or vice versa)
-
 ## Rake Tasks
 
 Located in `lib/tasks/` (~17 files). Notable:
diff --git a/CLAUDE.md b/CLAUDE.md
index dbd8f809b..c001602bf 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -103,9 +103,9 @@ When the user says "AI files" or "AI instructions", these are the files:
 
 | File | Purpose |
 |---|---|
-| `CLAUDE.md` | Primary rules for Claude Code (this file) |
-| `AGENTS.md` | Architecture reference (models, controllers, services) |
-| `.github/copilot-instructions.md` | Rules for GitHub Copilot (keep in sync with CLAUDE.md) |
+| `CLAUDE.md` | Single source of truth for all coding rules and conventions (this file) |
+| `AGENTS.md` | Architecture reference (models, controllers, services) — defers to CLAUDE.md for rules |
+| `.github/copilot-instructions.md` | Points Copilot to CLAUDE.md and AGENTS.md |
 | `ai/` | Shell script shortcuts for common dev tasks |
 
 ## Related Files
@@ -119,8 +119,7 @@ When changing a model or controller, check whether these related files need upda
 | View | System spec, Stimulus controller (if interactive) |
 | Service | Service spec |
 | Decorator | Decorator spec |
-| Add/remove model, concern, service, or gem | AGENTS.md, `.github/copilot-instructions.md` |
-| Code style rules | `.github/copilot-instructions.md` (keep in sync) |
+| Add/remove model, concern, service, or gem | AGENTS.md |
 
 ## Key Directories
 

From 50755c60207af39a1ef87590130c555a6d91e032 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:36:54 -0400
Subject: [PATCH 04/20] Move reference material from CLAUDE.md to AGENTS.md
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

CLAUDE.md should be lean — only coding rules and conventions that
need to be in every conversation context. Project overview, tech stack,
setup, key directories, testing details, linting/security commands
all moved to AGENTS.md where agents can look them up on demand.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 AGENTS.md |  38 +++++++++++++++++--
 CLAUDE.md | 111 ++++++++++--------------------------------------------
 2 files changed, 54 insertions(+), 95 deletions(-)

diff --git a/AGENTS.md b/AGENTS.md
index c54a0ac85..038524dd0 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -1,10 +1,33 @@
 # AGENTS.md — AWBW Portal
 
-Architecture reference for AI agents working on the AWBW Portal codebase. For coding rules, conventions, and quick commands, see `CLAUDE.md` (single source of truth).
+Architecture reference for AI agents. For coding rules and conventions, see `CLAUDE.md` (single source of truth).
 
-## Project Summary
+## Project Overview
 
-AWBW Portal is a Rails 8.1 application (Ruby 4.0.1) for A Window Between Worlds — a platform where workshop leaders manage workshops, resources, community news, stories, and events. It uses MySQL, Vite, Tailwind CSS v4, and the Hotwire stack (Stimulus + Turbo).
+AWBW Portal is a Rails 8.1 application (Ruby 4.0.1) for A Window Between Worlds — a platform where workshop leaders manage workshops, resources, community news, stories, and events.
+
+## Tech Stack
+
+- **Backend:** Rails 8.1, Ruby 4.0.1, MySQL (via Trilogy adapter)
+- **Frontend:** Vite, Tailwind CSS v4, Stimulus, Turbo Rails
+- **Auth:** Devise with JWT token support
+- **Authorization:** ActionPolicy (app/policies/)
+- **Rich text:** ActionText with Rhino editor (TipTap-based)
+- **File uploads:** ActiveStorage with DigitalOcean Spaces
+- **Background jobs:** SolidQueue
+- **Caching:** SolidCache
+
+## Setup
+
+Full setup (bundle, npm, database create/migrate/seed):
+```
+bin/setup
+```
+
+If you just need frontend dependencies:
+```
+npm ci
+```
 
 ## Architecture Overview
 
@@ -288,6 +311,15 @@ Common factory traits across models:
 - `:admin` (User with super_user=true)
 - `:with_primary_asset`, `:with_gallery_assets`
 
+## Linting & Security
+
+```
+bundle exec rubocop        # lint
+bundle exec rubocop -a     # auto-fix
+bundle exec brakeman       # security scan
+bundle exec bundle-audit check --update
+```
+
 ## CI Pipeline (GitHub Actions)
 
 ### ci.yml
diff --git a/CLAUDE.md b/CLAUDE.md
index c001602bf..56d108c51 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -1,35 +1,30 @@
 # CLAUDE.md
 
-## Architecture Reference
+For architecture reference (models, controllers, services, testing), read `AGENTS.md`.
 
-For detailed architecture, models, controllers, services, and testing structure, read `AGENTS.md`.
-
-## Project Overview
-
-This is a Ruby on Rails 8.1 application (Ruby 4.0.1) — the Portal for A Window Between Worlds (AWBW). It manages workshops, resources, community news, stories, and events for workshop leaders.
+## AI Instruction Files
 
-## Tech Stack
+When the user says "AI files" or "AI instructions", these are the files:
 
-- **Backend:** Rails 8.1, Ruby 4.0.1, MySQL (via Trilogy adapter)
-- **Frontend:** Vite, Tailwind CSS v4, Stimulus, Turbo Rails
-- **Auth:** Devise with JWT token support
-- **Authorization:** ActionPolicy (app/policies/)
-- **Rich text:** ActionText with Rhino editor (TipTap-based)
-- **File uploads:** ActiveStorage with DigitalOcean Spaces
-- **Background jobs:** SolidQueue
-- **Caching:** SolidCache
+| File | Purpose |
+|---|---|
+| `CLAUDE.md` | Coding rules and conventions (this file) |
+| `AGENTS.md` | Architecture reference + project details |
+| `.github/copilot-instructions.md` | Points Copilot to CLAUDE.md and AGENTS.md |
+| `ai/` | Shell script shortcuts for common dev tasks |
 
-## Setup
+## Related Files
 
-Full setup (bundle, npm, database create/migrate/seed):
-```
-bin/setup
-```
+When changing a model or controller, check whether these related files need updates:
 
-If you just need frontend dependencies:
-```
-npm ci
-```
+| If you change... | Also check... |
+|---|---|
+| Model | Decorator, policy, factory, model spec |
+| Controller | Policy, request spec, routing spec, views |
+| View | System spec, Stimulus controller (if interactive) |
+| Service | Service spec |
+| Decorator | Decorator spec |
+| Add/remove model, concern, service, or gem | AGENTS.md |
 
 ## Code Style
 
@@ -97,74 +92,6 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 - Good: `<div class="..." id="...">` or `<div class="...\n     id="...">`
 - Bad: `<div class="...\n     id="..."\n  >`
 
-## AI Instruction Files
-
-When the user says "AI files" or "AI instructions", these are the files:
-
-| File | Purpose |
-|---|---|
-| `CLAUDE.md` | Single source of truth for all coding rules and conventions (this file) |
-| `AGENTS.md` | Architecture reference (models, controllers, services) — defers to CLAUDE.md for rules |
-| `.github/copilot-instructions.md` | Points Copilot to CLAUDE.md and AGENTS.md |
-| `ai/` | Shell script shortcuts for common dev tasks |
-
-## Related Files
-
-When changing a model or controller, check whether these related files need updates:
-
-| If you change... | Also check... |
-|---|---|
-| Model | Decorator, policy, factory, model spec |
-| Controller | Policy, request spec, routing spec, views |
-| View | System spec, Stimulus controller (if interactive) |
-| Service | Service spec |
-| Decorator | Decorator spec |
-| Add/remove model, concern, service, or gem | AGENTS.md |
-
-## Key Directories
-
-- `app/services/` — Business logic service objects
-- `app/decorators/` — Draper decorators for view presentation
-- `app/policies/` — ActionPolicy authorization rules
-- `app/presenters/` — Presentation objects
-- `app/frontend/` — Vite/JS components (Stimulus controllers, etc.)
-
-## Testing
-
-- **Framework:** RSpec (`bundle exec rspec`)
-- **Factories:** FactoryBot (spec/factories/)
-- **Matchers:** Shoulda Matchers
-- **System tests:** Capybara with Selenium
-- **Coverage:** SimpleCov
-
-Run all tests:
-```
-bundle exec rspec
-```
-
-Run a single test file:
-```
-bundle exec rspec spec/models/some_model_spec.rb
-```
-
-## Linting
-
-```
-bundle exec rubocop
-```
-
-Auto-fix:
-```
-bundle exec rubocop -a
-```
-
-## Security Scanning
-
-```
-bundle exec brakeman
-bundle exec bundle-audit check --update
-```
-
 ## JavaScript
 
 - ES6+ syntax, ESM imports/exports

From 5da8e14f3d9f66b20a1862d873a9a89a9f235864 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:38:21 -0400
Subject: [PATCH 05/20] =?UTF-8?q?Add=20draft=20PR=20workflow=20=E2=80=94?=
 =?UTF-8?q?=20push=20early,=20mark=20ready=20when=20done?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 CLAUDE.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index 56d108c51..717d0e4c7 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -135,7 +135,8 @@ Follow the [Stimulus Handbook](https://stimulus.hotwired.dev/handbook/introducti
 
 ## PRs
 
-- After completing work, **create a pull request** using `gh pr create`
+- **Push to a draft PR early** — push commits and create a draft PR (`gh pr create --draft`) as soon as work begins, rather than keeping changes in a local branch. Push on every commit.
+- After completing work, **mark the PR ready** using `gh pr ready`
 - Once the PR is created, **prepend the PR number to the branch name** (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
 - Use `docs/pull_request_template.md` for PR description structure
 - Use bullet points, not paragraphs, when filling out each section

From 380a6ba13918ab6e0d072dd65bb589c84c76e223 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:39:50 -0400
Subject: [PATCH 06/20] Restore coding rules in copilot-instructions.md

Copilot inline suggestions can't follow file references, so it needs
its own copy of code style, RuboCop, Stimulus, and HTML/ERB rules.
Without these, Copilot suggestions regularly fail RuboCop CI.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/copilot-instructions.md | 81 ++++++++++++++++++++++++++++++++-
 CLAUDE.md                       |  2 +-
 2 files changed, 80 insertions(+), 3 deletions(-)

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
index 4278d3f73..045a474b5 100644
--- a/.github/copilot-instructions.md
+++ b/.github/copilot-instructions.md
@@ -1,5 +1,82 @@
 # Copilot Instructions
 
-Read and follow all rules in `CLAUDE.md` at the project root — it is the single source of truth for coding conventions, style rules, Stimulus conventions, HTML/ERB formatting, Git workflow, and PR process.
-
 Read `AGENTS.md` for architecture reference (models, controllers, services, testing structure).
+
+# Code Style
+
+- Use modern Ruby syntax
+- Prefer early returns and guard clauses
+- Avoid unnecessary and/or complex conditionals
+- Prefer constants and scopes over magic strings
+- Use safe navigation (`&.`) where appropriate
+- Use `presence` over blank checks
+- Use `Arel.sql` for raw SQL in order clauses
+- Avoid `update_all` unless explicitly intended
+- Prefer service objects under app/services
+- Prefer POROs over concerns when possible
+- Use `after_commit` instead of `after_save` for side effects
+
+# RuboCop (rubocop-rails-omakase)
+
+This project uses rubocop-rails-omakase. All code MUST follow these rules:
+
+## Strings
+- Always use double quotes: `"foo"` not `'foo'`
+
+## Spacing
+- Spaces inside array brackets: `[ a, b, c ]` not `[a, b, c]` (empty arrays: `[]`)
+- Spaces inside hash braces: `{ a: 1, b: 2 }` not `{a: 1}` (empty hashes: `{}`)
+- Spaces inside block braces: `foo { bar }` not `foo {bar}` (empty blocks: `foo { }`)
+- No spaces inside parens: `foo(bar)` not `foo( bar )`
+- No spaces inside reference brackets: `hash[:key]` not `hash[ :key ]`
+- Space before block braces: `foo { }` not `foo{ }`
+
+## Commas
+- No trailing commas in arrays, hashes, or method arguments
+
+## Indentation
+- 2-space indentation, no tabs
+- Consistent indentation at normal level — do NOT indent methods under `private`/`protected`
+- Align `end` with the variable in assignments
+- Align `when` with `end`, not with `case`
+
+## Whitespace
+- No trailing whitespace on any line
+- No trailing blank lines at end of file
+- No empty lines inside class, module, method, or block bodies
+
+## Syntax
+- Use `%w[]` and `%i[]` with square bracket delimiters (not parens)
+- Use modern hash syntax: `{ key: value }` not `{ :key => value }`
+- No redundant returns — omit `return` on last expression
+- Use `flat_map` instead of `.map { }.flatten`
+- No redundant `.to_s` inside string interpolation
+- Use `Foo.method` not `Foo::method` for method calls
+- No parentheses around conditions: `if foo` not `if (foo)`
+- No semicolons to separate statements
+
+# HTML/ERB Formatting
+
+## Tag attributes
+- When a tag has long attributes, place the closing `>` on the same line as the last attribute
+- Do NOT put the closing `>` on its own line
+
+# JavaScript
+
+- ES6+ syntax, ESM imports/exports
+- Strongly prefer Stimulus for JavaScript behavior — do not write raw/inline JS or jQuery
+- Always use Tailwind CSS utility classes for styling — do not write custom CSS unless absolutely necessary
+- Prefer Turbo for navigation and form submissions before reaching for Stimulus
+- Stimulus controller naming: `[name]_controller.js`
+
+# Stimulus conventions
+- Use `static targets` and `data-[controller]-target` — never `querySelector` or `getElementById` for elements that could be targets
+- Use `static values = { name: Type }` for state — not instance variables. Use `[name]ValueChanged()` for reactive updates
+- Use `data-action` attributes — not `addEventListener` in `connect()`. Omit default events (`click` for buttons, `input` for inputs, `submit` for forms, `change` for selects)
+- Use `@window`/`@document` suffixes for global events in data-action
+- Use action options (`:prevent`, `:stop`) instead of `event.preventDefault()` in methods
+- Use `static classes` when CSS classes should be configurable from HTML
+- Use `static outlets` for cross-controller communication instead of `getElementById`
+- Always clean up in `disconnect()` anything created in `connect()` (listeners, timers, observers)
+- Use `[name]TargetConnected`/`TargetDisconnected` for dynamic DOM (cocoon, Turbo)
+- Toggle `hidden` class instead of `style.display`. Use `class="hidden"` not `style="display:none"` in HTML
diff --git a/CLAUDE.md b/CLAUDE.md
index 717d0e4c7..5ab6ddb92 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -10,7 +10,7 @@ When the user says "AI files" or "AI instructions", these are the files:
 |---|---|
 | `CLAUDE.md` | Coding rules and conventions (this file) |
 | `AGENTS.md` | Architecture reference + project details |
-| `.github/copilot-instructions.md` | Points Copilot to CLAUDE.md and AGENTS.md |
+| `.github/copilot-instructions.md` | Coding rules for Copilot (duplicated from CLAUDE.md — keep in sync) |
 | `ai/` | Shell script shortcuts for common dev tasks |
 
 ## Related Files

From 59a6e7d7e37a21d3971987c7a2f40a8c66c71d02 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:41:37 -0400
Subject: [PATCH 07/20] Expand AI files trigger phrases in CLAUDE.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 CLAUDE.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index 5ab6ddb92..3714f63c0 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -4,7 +4,7 @@ For architecture reference (models, controllers, services, testing), read `AGENT
 
 ## AI Instruction Files
 
-When the user says "AI files" or "AI instructions", these are the files:
+When the user says "AI files", "AI instructions", "tell AI to", or "remember to always", these are the files:
 
 | File | Purpose |
 |---|---|

From 76d8f515a7c0a29d7091abc64ac2865051ee1388 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:45:51 -0400
Subject: [PATCH 08/20] Fix stale counts and missing items in AGENTS.md
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Rake tasks: 17 → 4 (removed nonexistent paperclip_to_active_storage)
- Services: 15 → 21 (added event_registration, user, and other services)
- Stimulus controllers: 20 listed → all 34 listed
- Model concerns: added AuthorCreditable
- Controller concerns: added TagAssignable
- Updated all directory and spec counts to match actual codebase
- Jobs: 2 → 3

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 AGENTS.md | 92 +++++++++++++++++++++++++++++++++++++------------------
 1 file changed, 63 insertions(+), 29 deletions(-)

diff --git a/AGENTS.md b/AGENTS.md
index 038524dd0..d482d0da9 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -48,19 +48,19 @@ AWBW Portal (Rails 8.1)
 
 | Directory | Purpose | Count |
 |---|---|---|
-| `app/models/` | ActiveRecord models | ~78 files |
-| `app/services/` | Service objects for complex logic | ~15 files |
-| `app/jobs/` | SolidQueue background jobs | 2 files |
-| `app/models/concerns/` | Shared model modules | ~11 concerns |
+| `app/models/` | ActiveRecord models | ~66 files |
+| `app/services/` | Service objects for complex logic | ~21 files |
+| `app/jobs/` | SolidQueue background jobs | 3 files |
+| `app/models/concerns/` | Shared model modules | 12 concerns |
 
 ### Presentation
 
 | Directory | Purpose | Count |
 |---|---|---|
-| `app/controllers/` | Rails controllers (admin/, events/) | ~61 files |
-| `app/views/` | ERB templates | ~434 files |
+| `app/controllers/` | Rails controllers (admin/, events/) | ~63 files |
+| `app/views/` | ERB templates | ~465 files |
 | `app/decorators/` | Draper decorators for view logic | ~36 files |
-| `app/policies/` | ActionPolicy authorization rules | ~43 files |
+| `app/policies/` | ActionPolicy authorization rules | ~44 files |
 | `app/presenters/` | Presentation objects | 1 file |
 | `app/helpers/` | View helpers | ~19 files |
 | `app/mailers/` | ActionMailer classes | 6 files |
@@ -71,7 +71,7 @@ AWBW Portal (Rails 8.1)
 | Directory | Purpose |
 |---|---|
 | `app/frontend/entrypoints/` | Vite entry points (application.js, application.css) |
-| `app/frontend/javascript/controllers/` | Stimulus controllers (~32) |
+| `app/frontend/javascript/controllers/` | Stimulus controllers (34) |
 | `app/frontend/javascript/rhino/` | Rich text editor customizations (mentions, grid) |
 | `app/frontend/stylesheets/` | Tailwind CSS and component styles |
 
@@ -81,7 +81,7 @@ AWBW Portal (Rails 8.1)
 |---|---|
 | `config/routes.rb` | All routes (single file) |
 | `config/database.yml` | MySQL via Trilogy adapter |
-| `config/initializers/` | ~32 initializer files |
+| `config/initializers/` | ~28 initializer files |
 | `.github/workflows/` | GitHub Actions CI |
 | `Procfile.dev` | Dev services: `vite` + `web` |
 | `ai/` | Shell script shortcuts for common dev tasks (see `ai/README.md`) |
@@ -118,23 +118,24 @@ AWBW Portal (Rails 8.1)
 
 | Concern | Purpose |
 |---|---|
+| `AhoyTrackable` | Event tracking integration |
+| `AuthorCreditable` | Author attribution |
 | `Featureable` | `featured`, `publicly_featured` scopes |
+| `Mentioner` | ActionText @mention extraction and grouping |
+| `NameFilterable` | Name-based filtering |
 | `Publishable` | `published`, `publicly_visible` scopes |
+| `PunctuationStrippable` | Strips punctuation from strings |
+| `RemoteSearchable` | AJAX remote search by column |
+| `RichTextSearchable` | Full-text search on ActionText rich_text fields |
 | `TagFilterable` | Scope-based filtering by tag names |
 | `Trendable` | Trending metrics tracking |
 | `WindowsTypeFilterable` | Filter by WindowsType association |
-| `RemoteSearchable` | AJAX remote search by column |
-| `RichTextSearchable` | Full-text search on ActionText rich_text fields |
-| `Mentioner` | ActionText @mention extraction and grouping |
-| `NameFilterable` | Name-based filtering |
-| `PunctuationStrippable` | Strips punctuation from strings |
-| `AhoyTrackable` | Event tracking integration |
 
 ## Controllers
 
 ### Namespaces
 
-- **Root level** (~48 controllers): Workshops, stories, resources, events, people, organizations, etc.
+- **Root level** (~51 controllers): Workshops, stories, resources, events, people, organizations, etc.
 - **`admin/`**: HomeController, AnalyticsController, AhoyActivitiesController
 - **`events/`**: Registrations sub-resource (create/destroy + slug-based show at `/registration/:slug`)
 - **Devise overrides**: Registrations, Confirmations, Passwords
@@ -158,6 +159,7 @@ end
 - `AhoyTracking` — Event tracking integration
 - `Dedupable` — Data deduplication helpers
 - `ExternallyRedirectable` — External URL redirection
+- `TagAssignable` — Tag assignment helpers
 
 ## Services
 
@@ -176,9 +178,27 @@ end
 - `PersonFromUserService` — Create Person from User account
 - `BulkInviteService` — Bulk send welcome instructions and reset created_at for users
 - `ModelDeduper` — Deduplication logic
+- `RichTextMigrator` — Rich text migration utility
+- `DisplayImagePresenter` — Image display logic
+
+### Event Registrations
+
+- `EventRegistrationServices::ProcessConfirmation` — Registration confirmation flow
+- `EventRegistrationServices::PublicRegistration` — Public registration handling
+- `ExtendedEventRegistrationFormBuilder` — Extended registration form builder
+- `ShortEventRegistrationFormBuilder` — Short registration form builder
+- `ScholarshipApplicationFormBuilder` — Scholarship form builder
+
+### Notifications
+
 - `NotificationServices::CreateNotification` — Notification creation
 - `NotificationServices::PersistDeliveredEmail` — Email delivery tracking
 
+### User Management
+
+- `UserServices::ProcessEmailChange` — Email change processing
+- `UserServices::ProcessEmailManualConfirm` — Manual email confirmation
+
 ## Decorators (Draper)
 
 All inherit from `ApplicationDecorator` which provides:
@@ -231,7 +251,8 @@ end
 
 ### Stimulus Controllers
 
-Key controllers:
+- `affiliation_dates` — Recalculate affiliation date ranges
+- `anchor_highlight` — Highlight anchored elements
 - `asset_picker` — Asset selection UI
 - `autosave` — Auto-save form state
 - `carousel` — Swiper-based carousels
@@ -239,19 +260,31 @@ Key controllers:
 - `collection` — Filter form auto-submit with debounce
 - `column_toggle` — Toggle table column visibility
 - `comment_edit_toggle` — Inline comment editing mode
+- `confirm_email` — Email confirmation UI
 - `dirty_form` — Unsaved changes detection
+- `dismiss` — Dismissable elements
 - `dropdown` — Dropdown menus with keyboard/click-outside handling
 - `file_preview` — File upload preview
 - `inactive_toggle` — Gray out expired affiliations
 - `optimistic_bookmark` — Instant bookmark UI feedback
+- `org_toggle` — Organization toggle UI
 - `paginated_fields` — Client-side pagination of nested fields
+- `password_toggle` — Show/hide password fields
+- `prefetch_lazy` — Prefetch lazy-loaded content
+- `print_options` — Print options toggle for analytics
 - `remote_select` — AJAX-powered select dropdown
+- `rhino_source` — Rich text editor integration
 - `searchable_checkbox` — TomSelect checkbox-style multi-select
 - `searchable_select` — Tom Select autocomplete
+- `share_url` — URL sharing/copying
 - `sortable` — Drag-drop sorting (SortableJS)
 - `tabs` — Tab panel navigation
+- `tag_link_loading` — Loading state for tag links
 - `tags_combination_highlight` — Highlight tags matching selected filters
-- `rhino_source` — Rich text editor integration
+- `tags_sync_list_heights` — Sync tag list column heights
+- `timeframe` — Date range filtering
+- `toggle_lock` — Lock/unlock toggle UI
+- `toggle_user_icon` — User icon visibility toggle
 
 ### JS Dependencies
 
@@ -277,17 +310,17 @@ Custom colors defined in `app/frontend/stylesheets/application.tailwind.css`:
 
 | Directory | Count | Purpose |
 |---|---|---|
-| `spec/models/` | ~54 | Model unit tests |
-| `spec/views/` | ~72 | View template tests |
-| `spec/requests/` | ~41 | HTTP request/integration tests |
-| `spec/system/` | ~30 | End-to-end browser tests (Capybara) |
+| `spec/models/` | ~58 | Model unit tests |
+| `spec/views/` | ~73 | View template tests |
+| `spec/requests/` | ~47 | HTTP request/integration tests |
+| `spec/system/` | ~25 | End-to-end browser tests (Capybara) |
 | `spec/routing/` | ~13 | Route definition tests |
 | `spec/policies/` | ~9 | Authorization policy tests |
-| `spec/decorators/` | ~8 | Decorator tests |
-| `spec/services/` | ~4 | Service object tests |
-| `spec/mailers/` | ~3 | Mailer tests |
-| `spec/helpers/` | ~2 | Helper tests |
-| `spec/factories/` | ~52 | FactoryBot factory definitions |
+| `spec/decorators/` | ~10 | Decorator tests |
+| `spec/services/` | ~12 | Service object tests |
+| `spec/mailers/` | ~5 | Mailer tests |
+| `spec/helpers/` | ~1 | Helper tests |
+| `spec/factories/` | ~53 | FactoryBot factory definitions |
 
 ### Configuration
 
@@ -351,7 +384,8 @@ RuboCop linting on PRs and pushes to main.
 
 ## Rake Tasks
 
-Located in `lib/tasks/` (~17 files). Notable:
+Located in `lib/tasks/` (4 files):
 - `dev.rake` — Development database seeding from XML/CSV
-- `paperclip_to_active_storage.rake` — File upload migration
 - `rhino_migrator.rake` — Rich text editor migration
+- `attachment_report.rake` — Attachment reporting
+- `migrate_internal_id_to_filemaker_code.rake` — FileMaker code migration

From 72dd7ddef7caaed2e72c6711886e6bcb844eb214 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:46:22 -0400
Subject: [PATCH 09/20] Add mailer to Related Files checklist in CLAUDE.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 CLAUDE.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CLAUDE.md b/CLAUDE.md
index 3714f63c0..59c7d1a76 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -24,6 +24,7 @@ When changing a model or controller, check whether these related files need upda
 | View | System spec, Stimulus controller (if interactive) |
 | Service | Service spec |
 | Decorator | Decorator spec |
+| Mailer (add/remove) | Mailer spec, mailer preview (follow existing patterns) |
 | Add/remove model, concern, service, or gem | AGENTS.md |
 
 ## Code Style

From db0445e1cae06217c3fc417ff049070b6a42c244 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:48:20 -0400
Subject: [PATCH 10/20] Make AGENTS.md update instruction more specific

List the exact things to check: Stimulus controllers, services,
concerns, mailers, rake tasks, and file counts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 CLAUDE.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index 59c7d1a76..6701733d6 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -144,7 +144,7 @@ Follow the [Stimulus Handbook](https://stimulus.hotwired.dev/handbook/introducti
 - Description must explain why the change was made, not just what
 - Include screenshots for UI changes
 - **On every push**, update the PR title and content to reflect the current diff
-- **On every push**, update AI instruction files if the diff introduces changes that affect conventions, architecture, or file lists documented there
+- **On every push**, update AI instruction files if the diff adds, removes, or renames anything tracked in AGENTS.md — specifically: Stimulus controllers, services, model/controller concerns, mailers, rake tasks, and directory file counts
 
 ## Quick Commands
 

From fb904eef219147485d74001e2c96d86906fa8102 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:49:41 -0400
Subject: [PATCH 11/20] Prefer camelCase const over SCREAMING_SNAKE_CASE in JS
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Add convention to AI files and rename existing constants:
STYLE_ID → styleId, ACTIVE_CLASSES → activeClasses,
GRAY_CLASSES → grayClasses, HIGHLIGHT_CLASS → highlightClass,
FILE_TYPE_ICONS → fileTypeIcons

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/copilot-instructions.md                           | 3 ++-
 CLAUDE.md                                                 | 3 ++-
 .../javascript/controllers/file_preview_controller.js     | 4 ++--
 .../javascript/controllers/inactive_toggle_controller.js  | 8 ++++----
 .../javascript/controllers/remote_select_controller.js    | 6 +++---
 .../controllers/tags_combination_highlight_controller.js  | 6 +++---
 6 files changed, 16 insertions(+), 14 deletions(-)

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
index 045a474b5..a093d9d2d 100644
--- a/.github/copilot-instructions.md
+++ b/.github/copilot-instructions.md
@@ -63,7 +63,8 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 
 # JavaScript
 
-- ES6+ syntax, ESM imports/exports
+- ES6+ syntax, ESM imports/exports, `const`/`let` (no `var`)
+- Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
 - Strongly prefer Stimulus for JavaScript behavior — do not write raw/inline JS or jQuery
 - Always use Tailwind CSS utility classes for styling — do not write custom CSS unless absolutely necessary
 - Prefer Turbo for navigation and form submissions before reaching for Stimulus
diff --git a/CLAUDE.md b/CLAUDE.md
index 6701733d6..391e51a27 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -95,7 +95,8 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 
 ## JavaScript
 
-- ES6+ syntax, ESM imports/exports
+- ES6+ syntax, ESM imports/exports, `const`/`let` (no `var`)
+- Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
 - **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
 - **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
 - Prefer Turbo for navigation and form submissions before reaching for Stimulus
diff --git a/app/frontend/javascript/controllers/file_preview_controller.js b/app/frontend/javascript/controllers/file_preview_controller.js
index bcaf8f1e6..776040d0c 100644
--- a/app/frontend/javascript/controllers/file_preview_controller.js
+++ b/app/frontend/javascript/controllers/file_preview_controller.js
@@ -1,6 +1,6 @@
 import { Controller } from "@hotwired/stimulus"
 
-const FILE_TYPE_ICONS = {
+const fileTypeIcons = {
     "application/pdf": "fa-regular fa-file-pdf",
     "application/msword": "fa-regular fa-file-word",
     "application/vnd.openxmlformats-officedocument.wordprocessingml.document": "fa-regular fa-file-word",
@@ -72,7 +72,7 @@ export default class extends Controller {
 
     showFileIcon(mimeType) {
         const placeholder = this.ensurePlaceholder()
-        const iconClass = FILE_TYPE_ICONS[mimeType] || "fa-regular fa-file"
+        const iconClass = fileTypeIcons[mimeType] || "fa-regular fa-file"
         placeholder.innerHTML = `<i class="${iconClass} text-5xl"></i>`
         placeholder.classList.remove("hidden")
 
diff --git a/app/frontend/javascript/controllers/inactive_toggle_controller.js b/app/frontend/javascript/controllers/inactive_toggle_controller.js
index 29733c9f0..8d39f56f2 100644
--- a/app/frontend/javascript/controllers/inactive_toggle_controller.js
+++ b/app/frontend/javascript/controllers/inactive_toggle_controller.js
@@ -1,17 +1,17 @@
 import { Controller } from "@hotwired/stimulus";
 
 // Active themed classes used by person (sky) and organization (emerald) profile buttons
-const ACTIVE_CLASSES = [
+const activeClasses = [
   "bg-sky-50", "bg-sky-100", "bg-sky-200", "hover:bg-sky-100", "hover:bg-sky-200",
   "text-sky-700", "text-sky-800", "border-sky-200", "border-sky-300",
   "bg-emerald-50", "bg-emerald-100", "bg-emerald-200", "hover:bg-emerald-100", "hover:bg-emerald-200",
   "text-emerald-700", "text-emerald-800", "border-emerald-200", "border-emerald-300"
 ];
-const GRAY_CLASSES = ["bg-gray-100", "hover:bg-gray-200", "text-gray-400", "border-gray-300"];
+const grayClasses = ["bg-gray-100", "hover:bg-gray-200", "text-gray-400", "border-gray-300"];
 
 function grayOut(el) {
-  ACTIVE_CLASSES.forEach((cls) => el.classList.remove(cls));
-  GRAY_CLASSES.forEach((cls) => el.classList.add(cls));
+  activeClasses.forEach((cls) => el.classList.remove(cls));
+  grayClasses.forEach((cls) => el.classList.add(cls));
 }
 
 export default class extends Controller {
diff --git a/app/frontend/javascript/controllers/remote_select_controller.js b/app/frontend/javascript/controllers/remote_select_controller.js
index c291b63b8..19dc3fbd7 100644
--- a/app/frontend/javascript/controllers/remote_select_controller.js
+++ b/app/frontend/javascript/controllers/remote_select_controller.js
@@ -1,7 +1,7 @@
 import { Controller } from "@hotwired/stimulus";
 import TomSelect from "tom-select";
 
-const STYLE_ID = "remote-select-overrides";
+const styleId = "remote-select-overrides";
 
 export default class extends Controller {
   static values = { model: String, exclude: String };
@@ -29,9 +29,9 @@ export default class extends Controller {
       },
     });
 
-    if (!document.getElementById(STYLE_ID)) {
+    if (!document.getElementById(styleId)) {
       const style = document.createElement("style");
-      style.id = STYLE_ID;
+      style.id = styleId;
       style.textContent = `
         .ts-control .ts-input {
           border: none !important;
diff --git a/app/frontend/javascript/controllers/tags_combination_highlight_controller.js b/app/frontend/javascript/controllers/tags_combination_highlight_controller.js
index f08740c7d..ed930be2c 100644
--- a/app/frontend/javascript/controllers/tags_combination_highlight_controller.js
+++ b/app/frontend/javascript/controllers/tags_combination_highlight_controller.js
@@ -2,7 +2,7 @@ import { Controller } from "@hotwired/stimulus"
 
 // When sector/category checkboxes in "Explore by combination" are checked,
 // adds a highlight class to the corresponding tag block in the Sectors/Categories panels above.
-const HIGHLIGHT_CLASS = "tags-combination-selected"
+const highlightClass = "tags-combination-selected"
 
 export default class extends Controller {
   static targets = ["sectorInput", "categoryInput"]
@@ -37,7 +37,7 @@ export default class extends Controller {
     if (!frame) return
     frame.querySelectorAll("a[data-sector-name]").forEach(link => {
       const name = link.dataset.sectorName?.trim()
-      link.classList.toggle(HIGHLIGHT_CLASS, name && selected.includes(name))
+      link.classList.toggle(highlightClass, name && selected.includes(name))
     })
   }
 
@@ -49,7 +49,7 @@ export default class extends Controller {
     if (!frame) return
     frame.querySelectorAll("a[data-category-name]").forEach(link => {
       const name = link.dataset.categoryName?.trim()
-      link.classList.toggle(HIGHLIGHT_CLASS, name && selected.includes(name))
+      link.classList.toggle(highlightClass, name && selected.includes(name))
     })
   }
 }

From 91580259fa9e507a2f35519fb2eab16e3b3721a0 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:51:34 -0400
Subject: [PATCH 12/20] Add Font Awesome icon preference to AI files

Prefer Font Awesome free icons over inline SVGs by default.
Inline SVGs still allowed when a specific icon design is preferred.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/copilot-instructions.md | 1 +
 CLAUDE.md                       | 1 +
 2 files changed, 2 insertions(+)

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
index a093d9d2d..5c2408db7 100644
--- a/.github/copilot-instructions.md
+++ b/.github/copilot-instructions.md
@@ -67,6 +67,7 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 - Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
 - Strongly prefer Stimulus for JavaScript behavior — do not write raw/inline JS or jQuery
 - Always use Tailwind CSS utility classes for styling — do not write custom CSS unless absolutely necessary
+- Prefer Font Awesome (free) icons over inline SVGs — inline SVGs are acceptable when a specific icon design is preferred
 - Prefer Turbo for navigation and form submissions before reaching for Stimulus
 - Stimulus controller naming: `[name]_controller.js`
 
diff --git a/CLAUDE.md b/CLAUDE.md
index 391e51a27..d07526ccb 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -99,6 +99,7 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 - Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
 - **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
 - **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
+- **Prefer Font Awesome (free)** icons over inline SVGs — use `icon("fa-solid fa-foo")` helper. Inline SVGs are acceptable when a specific icon design is preferred.
 - Prefer Turbo for navigation and form submissions before reaching for Stimulus
 - Controller naming: `[name]_controller.js`
 - Keep controllers focused and small

From 37464b349e300e3ff60a9d8f5531422aeeb3617a Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:54:16 -0400
Subject: [PATCH 13/20] Replace project name with generic references in
 AGENTS.md

Keep the full name in the project overview, use "this codebase"
elsewhere to avoid hardcoding the name throughout.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 AGENTS.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/AGENTS.md b/AGENTS.md
index d482d0da9..c80194e9a 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -1,4 +1,4 @@
-# AGENTS.md — AWBW Portal
+# AGENTS.md
 
 Architecture reference for AI agents. For coding rules and conventions, see `CLAUDE.md` (single source of truth).
 
@@ -32,7 +32,7 @@ npm ci
 ## Architecture Overview
 
 ```
-AWBW Portal (Rails 8.1)
+This codebase (Rails 8.1)
 ├── Main app (app/)              — Workshops, resources, stories, events, people, organizations
 ├── Frontend                     — Stimulus + Turbo + Tailwind CSS v4 (Vite bundler)
 ├── Background jobs              — SolidQueue

From ca437c5ab934cd3f36f59c69415f939fecb3cfbc Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 06:59:56 -0400
Subject: [PATCH 14/20] Remove hardcoded email from mailer description in
 AGENTS.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 AGENTS.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/AGENTS.md b/AGENTS.md
index c80194e9a..c9f365a00 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -239,7 +239,7 @@ end
 
 ## Mailers
 
-- `ApplicationMailer` — Base, from: `ENV["REPLY_TO_EMAIL"]` (programs@awbw.org)
+- `ApplicationMailer` — Base, from: `ENV["REPLY_TO_EMAIL"]`
 - `DeviseMailer` — Custom Devise emails
 - `EventMailer` — Event registration confirmations
 - `NotificationMailer` — Notification delivery

From 4a1cd246caa5e04af2a62e5b085ea54370649ccc Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 07:01:59 -0400
Subject: [PATCH 15/20] Prompt AI to suggest AI file updates for repeated
 corrections

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 CLAUDE.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index d07526ccb..8f554c687 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -4,7 +4,8 @@ For architecture reference (models, controllers, services, testing), read `AGENT
 
 ## AI Instruction Files
 
-When the user says "AI files", "AI instructions", "tell AI to", or "remember to always", these are the files:
+When the user says "AI files", "AI instructions", "tell AI to", or "remember to always", these are the files.
+If you notice the user repeatedly correcting the same pattern, suggest adding it to the AI files with a concrete proposal.
 
 | File | Purpose |
 |---|---|

From dc9f60fbbcd6dc6288d91d896ff018e63ffcc52d Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 07:10:01 -0400
Subject: [PATCH 16/20] =?UTF-8?q?Remove=20AI=20file=20changes=20=E2=80=94?=
 =?UTF-8?q?=20moved=20to=20separate=20PR=20(#1390)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/copilot-instructions.md |  53 +++++-----
 AGENTS.md                       | 165 +++++++++++---------------------
 CLAUDE.md                       | 127 +++++++++++++++---------
 3 files changed, 166 insertions(+), 179 deletions(-)

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
index 5c2408db7..138c038e9 100644
--- a/.github/copilot-instructions.md
+++ b/.github/copilot-instructions.md
@@ -1,9 +1,19 @@
-# Copilot Instructions
+# This project is a Ruby on Rails application.
+<!-- Keep code style rules in sync with CLAUDE.md -->
 
-Read `AGENTS.md` for architecture reference (models, controllers, services, testing structure).
+# Frontend requirements:
+- Strongly prefer Stimulus for JavaScript behavior — do not write raw/inline JS or jQuery
+- Always use Tailwind CSS utility classes for styling — do not write custom CSS unless absolutely necessary
+- Prefer Turbo for navigation and form submissions before reaching for Stimulus
+- ES6+ syntax, ESM imports/exports
+- Stimulus controller naming: `[name]_controller.js`
 
-# Code Style
+# PRs
+- After completing work, create a pull request using `gh pr create`
+- Once the PR is created, prepend the PR number to the branch name (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
+- On every push, update the PR title and description to reflect the current diff
 
+# Code style requirements:
 - Use modern Ruby syntax
 - Prefer early returns and guard clauses
 - Avoid unnecessary and/or complex conditionals
@@ -17,7 +27,6 @@ Read `AGENTS.md` for architecture reference (models, controllers, services, test
 - Use `after_commit` instead of `after_save` for side effects
 
 # RuboCop (rubocop-rails-omakase)
-
 This project uses rubocop-rails-omakase. All code MUST follow these rules:
 
 ## Strings
@@ -55,30 +64,22 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 - No parentheses around conditions: `if foo` not `if (foo)`
 - No semicolons to separate statements
 
+# Git
+- When rebasing onto main, review incoming changes for their intent and flag any oversights — missing tests, incomplete migrations, broken assumptions, or conflicts between the two branches. Check both directions: schema/model changes on either branch that affect views, partials, or layouts on the other (e.g., main redesigned a table's CSS but your branch adds new columns to it, or vice versa)
+
 # HTML/ERB Formatting
 
 ## Tag attributes
 - When a tag has long attributes, place the closing `>` on the same line as the last attribute
 - Do NOT put the closing `>` on its own line
-
-# JavaScript
-
-- ES6+ syntax, ESM imports/exports, `const`/`let` (no `var`)
-- Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
-- Strongly prefer Stimulus for JavaScript behavior — do not write raw/inline JS or jQuery
-- Always use Tailwind CSS utility classes for styling — do not write custom CSS unless absolutely necessary
-- Prefer Font Awesome (free) icons over inline SVGs — inline SVGs are acceptable when a specific icon design is preferred
-- Prefer Turbo for navigation and form submissions before reaching for Stimulus
-- Stimulus controller naming: `[name]_controller.js`
-
-# Stimulus conventions
-- Use `static targets` and `data-[controller]-target` — never `querySelector` or `getElementById` for elements that could be targets
-- Use `static values = { name: Type }` for state — not instance variables. Use `[name]ValueChanged()` for reactive updates
-- Use `data-action` attributes — not `addEventListener` in `connect()`. Omit default events (`click` for buttons, `input` for inputs, `submit` for forms, `change` for selects)
-- Use `@window`/`@document` suffixes for global events in data-action
-- Use action options (`:prevent`, `:stop`) instead of `event.preventDefault()` in methods
-- Use `static classes` when CSS classes should be configurable from HTML
-- Use `static outlets` for cross-controller communication instead of `getElementById`
-- Always clean up in `disconnect()` anything created in `connect()` (listeners, timers, observers)
-- Use `[name]TargetConnected`/`TargetDisconnected` for dynamic DOM (cocoon, Turbo)
-- Toggle `hidden` class instead of `style.display`. Use `class="hidden"` not `style="display:none"` in HTML
+- Example (GOOD):
+  ```erb
+  <div class="relative z-10 w-full bg-white text-gray-800 py-2 px-4"
+       id="dropdown">
+  ```
+- Example (BAD):
+  ```erb
+  <div class="relative z-10 w-full bg-white text-gray-800 py-2 px-4"
+       id="dropdown"
+  >
+  ```
diff --git a/AGENTS.md b/AGENTS.md
index c9f365a00..4965990ed 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -1,38 +1,15 @@
-# AGENTS.md
+# AGENTS.md — AWBW Portal
 
-Architecture reference for AI agents. For coding rules and conventions, see `CLAUDE.md` (single source of truth).
+Architecture reference for AI agents working on the AWBW Portal codebase. For coding rules and quick commands, see `CLAUDE.md`.
 
-## Project Overview
+## Project Summary
 
-AWBW Portal is a Rails 8.1 application (Ruby 4.0.1) for A Window Between Worlds — a platform where workshop leaders manage workshops, resources, community news, stories, and events.
-
-## Tech Stack
-
-- **Backend:** Rails 8.1, Ruby 4.0.1, MySQL (via Trilogy adapter)
-- **Frontend:** Vite, Tailwind CSS v4, Stimulus, Turbo Rails
-- **Auth:** Devise with JWT token support
-- **Authorization:** ActionPolicy (app/policies/)
-- **Rich text:** ActionText with Rhino editor (TipTap-based)
-- **File uploads:** ActiveStorage with DigitalOcean Spaces
-- **Background jobs:** SolidQueue
-- **Caching:** SolidCache
-
-## Setup
-
-Full setup (bundle, npm, database create/migrate/seed):
-```
-bin/setup
-```
-
-If you just need frontend dependencies:
-```
-npm ci
-```
+AWBW Portal is a Rails 8.1 application (Ruby 4.0.1) for A Window Between Worlds — a platform where workshop leaders manage workshops, resources, community news, stories, and events. It uses MySQL, Vite, Tailwind CSS v4, and the Hotwire stack (Stimulus + Turbo).
 
 ## Architecture Overview
 
 ```
-This codebase (Rails 8.1)
+AWBW Portal (Rails 8.1)
 ├── Main app (app/)              — Workshops, resources, stories, events, people, organizations
 ├── Frontend                     — Stimulus + Turbo + Tailwind CSS v4 (Vite bundler)
 ├── Background jobs              — SolidQueue
@@ -48,19 +25,19 @@ This codebase (Rails 8.1)
 
 | Directory | Purpose | Count |
 |---|---|---|
-| `app/models/` | ActiveRecord models | ~66 files |
-| `app/services/` | Service objects for complex logic | ~21 files |
-| `app/jobs/` | SolidQueue background jobs | 3 files |
-| `app/models/concerns/` | Shared model modules | 12 concerns |
+| `app/models/` | ActiveRecord models | ~78 files |
+| `app/services/` | Service objects for complex logic | ~15 files |
+| `app/jobs/` | SolidQueue background jobs | 2 files |
+| `app/models/concerns/` | Shared model modules | ~11 concerns |
 
 ### Presentation
 
 | Directory | Purpose | Count |
 |---|---|---|
-| `app/controllers/` | Rails controllers (admin/, events/) | ~63 files |
-| `app/views/` | ERB templates | ~465 files |
+| `app/controllers/` | Rails controllers (admin/, events/) | ~61 files |
+| `app/views/` | ERB templates | ~434 files |
 | `app/decorators/` | Draper decorators for view logic | ~36 files |
-| `app/policies/` | ActionPolicy authorization rules | ~44 files |
+| `app/policies/` | ActionPolicy authorization rules | ~43 files |
 | `app/presenters/` | Presentation objects | 1 file |
 | `app/helpers/` | View helpers | ~19 files |
 | `app/mailers/` | ActionMailer classes | 6 files |
@@ -71,7 +48,7 @@ This codebase (Rails 8.1)
 | Directory | Purpose |
 |---|---|
 | `app/frontend/entrypoints/` | Vite entry points (application.js, application.css) |
-| `app/frontend/javascript/controllers/` | Stimulus controllers (34) |
+| `app/frontend/javascript/controllers/` | Stimulus controllers (~32) |
 | `app/frontend/javascript/rhino/` | Rich text editor customizations (mentions, grid) |
 | `app/frontend/stylesheets/` | Tailwind CSS and component styles |
 
@@ -81,7 +58,7 @@ This codebase (Rails 8.1)
 |---|---|
 | `config/routes.rb` | All routes (single file) |
 | `config/database.yml` | MySQL via Trilogy adapter |
-| `config/initializers/` | ~28 initializer files |
+| `config/initializers/` | ~32 initializer files |
 | `.github/workflows/` | GitHub Actions CI |
 | `Procfile.dev` | Dev services: `vite` + `web` |
 | `ai/` | Shell script shortcuts for common dev tasks (see `ai/README.md`) |
@@ -118,24 +95,23 @@ This codebase (Rails 8.1)
 
 | Concern | Purpose |
 |---|---|
-| `AhoyTrackable` | Event tracking integration |
-| `AuthorCreditable` | Author attribution |
 | `Featureable` | `featured`, `publicly_featured` scopes |
-| `Mentioner` | ActionText @mention extraction and grouping |
-| `NameFilterable` | Name-based filtering |
 | `Publishable` | `published`, `publicly_visible` scopes |
-| `PunctuationStrippable` | Strips punctuation from strings |
-| `RemoteSearchable` | AJAX remote search by column |
-| `RichTextSearchable` | Full-text search on ActionText rich_text fields |
 | `TagFilterable` | Scope-based filtering by tag names |
 | `Trendable` | Trending metrics tracking |
 | `WindowsTypeFilterable` | Filter by WindowsType association |
+| `RemoteSearchable` | AJAX remote search by column |
+| `RichTextSearchable` | Full-text search on ActionText rich_text fields |
+| `Mentioner` | ActionText @mention extraction and grouping |
+| `NameFilterable` | Name-based filtering |
+| `PunctuationStrippable` | Strips punctuation from strings |
+| `AhoyTrackable` | Event tracking integration |
 
 ## Controllers
 
 ### Namespaces
 
-- **Root level** (~51 controllers): Workshops, stories, resources, events, people, organizations, etc.
+- **Root level** (~48 controllers): Workshops, stories, resources, events, people, organizations, etc.
 - **`admin/`**: HomeController, AnalyticsController, AhoyActivitiesController
 - **`events/`**: Registrations sub-resource (create/destroy + slug-based show at `/registration/:slug`)
 - **Devise overrides**: Registrations, Confirmations, Passwords
@@ -159,7 +135,6 @@ end
 - `AhoyTracking` — Event tracking integration
 - `Dedupable` — Data deduplication helpers
 - `ExternallyRedirectable` — External URL redirection
-- `TagAssignable` — Tag assignment helpers
 
 ## Services
 
@@ -178,27 +153,9 @@ end
 - `PersonFromUserService` — Create Person from User account
 - `BulkInviteService` — Bulk send welcome instructions and reset created_at for users
 - `ModelDeduper` — Deduplication logic
-- `RichTextMigrator` — Rich text migration utility
-- `DisplayImagePresenter` — Image display logic
-
-### Event Registrations
-
-- `EventRegistrationServices::ProcessConfirmation` — Registration confirmation flow
-- `EventRegistrationServices::PublicRegistration` — Public registration handling
-- `ExtendedEventRegistrationFormBuilder` — Extended registration form builder
-- `ShortEventRegistrationFormBuilder` — Short registration form builder
-- `ScholarshipApplicationFormBuilder` — Scholarship form builder
-
-### Notifications
-
 - `NotificationServices::CreateNotification` — Notification creation
 - `NotificationServices::PersistDeliveredEmail` — Email delivery tracking
 
-### User Management
-
-- `UserServices::ProcessEmailChange` — Email change processing
-- `UserServices::ProcessEmailManualConfirm` — Manual email confirmation
-
 ## Decorators (Draper)
 
 All inherit from `ApplicationDecorator` which provides:
@@ -239,7 +196,7 @@ end
 
 ## Mailers
 
-- `ApplicationMailer` — Base, from: `ENV["REPLY_TO_EMAIL"]`
+- `ApplicationMailer` — Base, from: `ENV["REPLY_TO_EMAIL"]` (programs@awbw.org)
 - `DeviseMailer` — Custom Devise emails
 - `EventMailer` — Event registration confirmations
 - `NotificationMailer` — Notification delivery
@@ -249,42 +206,28 @@ end
 
 ## Frontend
 
-### Stimulus Controllers
+### Preferences
+
+- **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
+- **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
+- Prefer Turbo for navigation and form submissions before reaching for Stimulus
+
+### Stimulus Controllers (32)
 
-- `affiliation_dates` — Recalculate affiliation date ranges
-- `anchor_highlight` — Highlight anchored elements
+Key controllers:
 - `asset_picker` — Asset selection UI
 - `autosave` — Auto-save form state
 - `carousel` — Swiper-based carousels
 - `cocoon` — Nested form handling (cocoon gem)
-- `collection` — Filter form auto-submit with debounce
-- `column_toggle` — Toggle table column visibility
-- `comment_edit_toggle` — Inline comment editing mode
-- `confirm_email` — Email confirmation UI
 - `dirty_form` — Unsaved changes detection
-- `dismiss` — Dismissable elements
 - `dropdown` — Dropdown menus with keyboard/click-outside handling
 - `file_preview` — File upload preview
-- `inactive_toggle` — Gray out expired affiliations
 - `optimistic_bookmark` — Instant bookmark UI feedback
-- `org_toggle` — Organization toggle UI
-- `paginated_fields` — Client-side pagination of nested fields
-- `password_toggle` — Show/hide password fields
-- `prefetch_lazy` — Prefetch lazy-loaded content
-- `print_options` — Print options toggle for analytics
 - `remote_select` — AJAX-powered select dropdown
-- `rhino_source` — Rich text editor integration
-- `searchable_checkbox` — TomSelect checkbox-style multi-select
 - `searchable_select` — Tom Select autocomplete
-- `share_url` — URL sharing/copying
 - `sortable` — Drag-drop sorting (SortableJS)
 - `tabs` — Tab panel navigation
-- `tag_link_loading` — Loading state for tag links
-- `tags_combination_highlight` — Highlight tags matching selected filters
-- `tags_sync_list_heights` — Sync tag list column heights
-- `timeframe` — Date range filtering
-- `toggle_lock` — Lock/unlock toggle UI
-- `toggle_user_icon` — User icon visibility toggle
+- `rhino_source` — Rich text editor integration
 
 ### JS Dependencies
 
@@ -310,17 +253,17 @@ Custom colors defined in `app/frontend/stylesheets/application.tailwind.css`:
 
 | Directory | Count | Purpose |
 |---|---|---|
-| `spec/models/` | ~58 | Model unit tests |
-| `spec/views/` | ~73 | View template tests |
-| `spec/requests/` | ~47 | HTTP request/integration tests |
-| `spec/system/` | ~25 | End-to-end browser tests (Capybara) |
+| `spec/models/` | ~54 | Model unit tests |
+| `spec/views/` | ~72 | View template tests |
+| `spec/requests/` | ~41 | HTTP request/integration tests |
+| `spec/system/` | ~30 | End-to-end browser tests (Capybara) |
 | `spec/routing/` | ~13 | Route definition tests |
 | `spec/policies/` | ~9 | Authorization policy tests |
-| `spec/decorators/` | ~10 | Decorator tests |
-| `spec/services/` | ~12 | Service object tests |
-| `spec/mailers/` | ~5 | Mailer tests |
-| `spec/helpers/` | ~1 | Helper tests |
-| `spec/factories/` | ~53 | FactoryBot factory definitions |
+| `spec/decorators/` | ~8 | Decorator tests |
+| `spec/services/` | ~4 | Service object tests |
+| `spec/mailers/` | ~3 | Mailer tests |
+| `spec/helpers/` | ~2 | Helper tests |
+| `spec/factories/` | ~52 | FactoryBot factory definitions |
 
 ### Configuration
 
@@ -344,15 +287,6 @@ Common factory traits across models:
 - `:admin` (User with super_user=true)
 - `:with_primary_asset`, `:with_gallery_assets`
 
-## Linting & Security
-
-```
-bundle exec rubocop        # lint
-bundle exec rubocop -a     # auto-fix
-bundle exec brakeman       # security scan
-bundle exec bundle-audit check --update
-```
-
 ## CI Pipeline (GitHub Actions)
 
 ### ci.yml
@@ -364,6 +298,11 @@ bundle exec bundle-audit check --update
 
 RuboCop linting on PRs and pushes to main.
 
+## PR Workflow
+
+- After completing work, create a pull request using `gh pr create`
+- Once the PR is created, prepend the PR number to the branch name (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
+
 ## Key Library Usage
 
 | Need | Library |
@@ -382,10 +321,18 @@ RuboCop linting on PRs and pushes to main.
 | Email styling | Premailer-rails (inline CSS) |
 | Positioning | Positioning gem for ordered records |
 
+## PRs
+
+- On every push, update the PR title and description to reflect the current diff
+
+## Git
+
+- When rebasing onto main, review incoming changes for their intent and flag any oversights — missing tests, incomplete migrations, broken assumptions, or conflicts between the two branches. Check both directions: schema/model changes on either branch that affect views, partials, or layouts on the other (e.g., main redesigned a table's CSS but your branch adds new columns to it, or vice versa)
+
 ## Rake Tasks
 
-Located in `lib/tasks/` (4 files):
+Located in `lib/tasks/` (~17 files). Notable:
 - `dev.rake` — Development database seeding from XML/CSV
+- `paperclip_to_active_storage.rake` — File upload migration
 - `rhino_migrator.rake` — Rich text editor migration
-- `attachment_report.rake` — Attachment reporting
-- `migrate_internal_id_to_filemaker_code.rake` — FileMaker code migration
+- `tag_deduping.rake` — Tag deduplication
diff --git a/CLAUDE.md b/CLAUDE.md
index 8f554c687..5f5220e75 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -1,32 +1,35 @@
 # CLAUDE.md
 
-For architecture reference (models, controllers, services, testing), read `AGENTS.md`.
+## Architecture Reference
 
-## AI Instruction Files
+For detailed architecture, models, controllers, services, and testing structure, read `AGENTS.md`.
 
-When the user says "AI files", "AI instructions", "tell AI to", or "remember to always", these are the files.
-If you notice the user repeatedly correcting the same pattern, suggest adding it to the AI files with a concrete proposal.
+## Project Overview
 
-| File | Purpose |
-|---|---|
-| `CLAUDE.md` | Coding rules and conventions (this file) |
-| `AGENTS.md` | Architecture reference + project details |
-| `.github/copilot-instructions.md` | Coding rules for Copilot (duplicated from CLAUDE.md — keep in sync) |
-| `ai/` | Shell script shortcuts for common dev tasks |
+This is a Ruby on Rails 8.1 application (Ruby 4.0.1) — the Portal for A Window Between Worlds (AWBW). It manages workshops, resources, community news, stories, and events for workshop leaders.
 
-## Related Files
+## Tech Stack
 
-When changing a model or controller, check whether these related files need updates:
+- **Backend:** Rails 8.1, Ruby 4.0.1, MySQL (via Trilogy adapter)
+- **Frontend:** Vite, Tailwind CSS v4, Stimulus, Turbo Rails
+- **Auth:** Devise with JWT token support
+- **Authorization:** ActionPolicy (app/policies/)
+- **Rich text:** ActionText with Rhino editor (TipTap-based)
+- **File uploads:** ActiveStorage with DigitalOcean Spaces
+- **Background jobs:** SolidQueue
+- **Caching:** SolidCache
 
-| If you change... | Also check... |
-|---|---|
-| Model | Decorator, policy, factory, model spec |
-| Controller | Policy, request spec, routing spec, views |
-| View | System spec, Stimulus controller (if interactive) |
-| Service | Service spec |
-| Decorator | Decorator spec |
-| Mailer (add/remove) | Mailer spec, mailer preview (follow existing patterns) |
-| Add/remove model, concern, service, or gem | AGENTS.md |
+## Setup
+
+Full setup (bundle, npm, database create/migrate/seed):
+```
+bin/setup
+```
+
+If you just need frontend dependencies:
+```
+npm ci
+```
 
 ## Code Style
 
@@ -94,36 +97,74 @@ This project uses rubocop-rails-omakase. All code MUST follow these rules:
 - Good: `<div class="..." id="...">` or `<div class="...\n     id="...">`
 - Bad: `<div class="...\n     id="..."\n  >`
 
-## JavaScript
+## Related Files
 
-- ES6+ syntax, ESM imports/exports, `const`/`let` (no `var`)
-- Use `const` for fixed values — not `SCREAMING_SNAKE_CASE` constants (e.g., `const styleId = "foo"` not `const STYLE_ID = "foo"`)
-- **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
-- **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
-- **Prefer Font Awesome (free)** icons over inline SVGs — use `icon("fa-solid fa-foo")` helper. Inline SVGs are acceptable when a specific icon design is preferred.
-- Prefer Turbo for navigation and form submissions before reaching for Stimulus
-- Controller naming: `[name]_controller.js`
-- Keep controllers focused and small
+When changing a model or controller, check whether these related files need updates:
+
+| If you change... | Also check... |
+|---|---|
+| Model | Decorator, policy, factory, model spec |
+| Controller | Policy, request spec, routing spec, views |
+| View | System spec, Stimulus controller (if interactive) |
+| Service | Service spec |
+| Decorator | Decorator spec |
+| Add/remove model, concern, service, or gem | AGENTS.md, `.github/copilot-instructions.md` |
+| Code style rules | `.github/copilot-instructions.md` (keep in sync) |
 
-### Stimulus Conventions
+## Key Directories
 
-Follow the [Stimulus Handbook](https://stimulus.hotwired.dev/handbook/introduction) and reference docs. Key rules:
+- `app/services/` — Business logic service objects
+- `app/decorators/` — Draper decorators for view presentation
+- `app/policies/` — ActionPolicy authorization rules
+- `app/presenters/` — Presentation objects
+- `app/frontend/` — Vite/JS components (Stimulus controllers, etc.)
 
-**Targets over querySelector** — declare `static targets = [...]` and use `data-[controller]-target` attributes in views. Never use `this.element.querySelector` or `document.getElementById` to find elements that could be targets. Exception: elements outside the controller's scope (e.g., in a parent view).
+## Testing
 
-**Values API for state** — use `static values = { name: Type }` for any state that persists or drives UI. Do not store state in instance variables when a value would work. Use `[name]ValueChanged()` callbacks for reactive updates instead of manual syncing.
+- **Framework:** RSpec (`bundle exec rspec`)
+- **Factories:** FactoryBot (spec/factories/)
+- **Matchers:** Shoulda Matchers
+- **System tests:** Capybara with Selenium
+- **Coverage:** SimpleCov
 
-**Actions over manual listeners** — use `data-action` attributes instead of `addEventListener` in `connect()`. Omit the event when it's the default for the element (`click` for buttons/links, `input` for inputs/textareas, `submit` for forms, `change` for selects). Use `@window` or `@document` suffixes for global events when possible (e.g., `resize@window->controller#layout`). Use action options like `:prevent` and `:stop` instead of calling `event.preventDefault()` in methods.
+Run all tests:
+```
+bundle exec rspec
+```
 
-**Classes API for CSS** — use `static classes = [...]` when CSS classes need to be configurable from HTML. For standard Tailwind utilities used internally (e.g., `"hidden"`), hardcoding is acceptable.
+Run a single test file:
+```
+bundle exec rspec spec/models/some_model_spec.rb
+```
 
-**Outlets for cross-controller communication** — use `static outlets = [...]` to reference other controllers instead of `document.getElementById` or custom events when the relationship is stable.
+## Linting
 
-**Lifecycle discipline** — every listener, timer, or observer created in `connect()` must be cleaned up in `disconnect()`. Store bound handler references so they can be removed. Use `initialize()` for one-time setup (e.g., binding functions).
+```
+bundle exec rubocop
+```
 
-**Target lifecycle callbacks** — use `[name]TargetConnected(element)` and `[name]TargetDisconnected(element)` to respond to dynamically added/removed targets (e.g., cocoon nested fields, Turbo streams).
+Auto-fix:
+```
+bundle exec rubocop -a
+```
 
-**Visibility** — toggle the `hidden` class via `classList.toggle("hidden", condition)` instead of setting `style.display`. Use `class="hidden"` in HTML for initial hidden state, not `style="display:none"`.
+## Security Scanning
+
+```
+bundle exec brakeman
+bundle exec bundle-audit check --update
+```
+
+## JavaScript
+
+- ES6+ syntax, ESM imports/exports
+- **Strongly prefer Stimulus** for JavaScript behavior — do not write raw/inline JS or jQuery
+- **Always use Tailwind CSS** utility classes for styling — do not write custom CSS unless absolutely necessary
+- Prefer Turbo for navigation and form submissions before reaching for Stimulus
+- Controller naming: `[name]_controller.js`
+- Keep controllers focused and small
+- **Use Stimulus targets and data attributes** to reference DOM elements — avoid `this.element.querySelector` and direct DOM queries. Declare `static targets = [...]` and use `data-[controller]-target` attributes in views.
+- **Use Stimulus shorthand action descriptors and shorthand pairs** — omit the event when it's the default for that element (e.g., `input` for `<input>`/`<textarea>`, `click` for `<button>`/`<a>`, `submit` for `<form>`). Write `controller#action` not `input->controller#action` on an input element. Only specify the event when using a non-default (e.g., `change->controller#action` on an input). See [Stimulus Actions](https://stimulus.hotwired.dev/reference/actions#event-shorthand).
 
 ## Migrations
 
@@ -139,15 +180,13 @@ Follow the [Stimulus Handbook](https://stimulus.hotwired.dev/handbook/introducti
 
 ## PRs
 
-- **Push to a draft PR early** — push commits and create a draft PR (`gh pr create --draft`) as soon as work begins, rather than keeping changes in a local branch. Push on every commit.
-- After completing work, **mark the PR ready** using `gh pr ready`
+- After completing work, **create a pull request** using `gh pr create`
 - Once the PR is created, **prepend the PR number to the branch name** (e.g., rename `maebeale/fix-login` to `maebeale/1234-fix-login`) using `git branch -m` and `git push origin -u` with the new name, then delete the old remote branch
 - Use `docs/pull_request_template.md` for PR description structure
 - Use bullet points, not paragraphs, when filling out each section
 - Description must explain why the change was made, not just what
 - Include screenshots for UI changes
-- **On every push**, update the PR title and content to reflect the current diff
-- **On every push**, update AI instruction files if the diff adds, removes, or renames anything tracked in AGENTS.md — specifically: Stimulus controllers, services, model/controller concerns, mailers, rake tasks, and directory file counts
+- **On every push**, update the PR title and description to reflect the current diff
 
 ## Quick Commands
 

From 3d3ec1fe674b08d06fae5cf2b7372c7041837d1f Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 08:22:06 -0400
Subject: [PATCH 17/20] Fix specs broken by Stimulus audit changes

- events_spec: update data-column-toggle-col to data-column-toggle-target="col"
- password_field_toggle_spec: scope toggle click within controller wrapper
  and use eventually matcher for async type change

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 spec/requests/events_spec.rb              | 2 +-
 spec/system/password_field_toggle_spec.rb | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/spec/requests/events_spec.rb b/spec/requests/events_spec.rb
index cbd79016a..4ae1c2f04 100644
--- a/spec/requests/events_spec.rb
+++ b/spec/requests/events_spec.rb
@@ -292,7 +292,7 @@
       it "renders confirmed column header hidden by default" do
         get manage_event_path(event)
 
-        expect(response.body).to include("data-column-toggle-col")
+        expect(response.body).to include('data-column-toggle-target="col"')
       end
 
       context "when registrant has a confirmed user" do
diff --git a/spec/system/password_field_toggle_spec.rb b/spec/system/password_field_toggle_spec.rb
index c6cbcc18c..7f4ba075b 100644
--- a/spec/system/password_field_toggle_spec.rb
+++ b/spec/system/password_field_toggle_spec.rb
@@ -21,12 +21,13 @@
     expect(page).to have_css("[data-action='password-toggle#toggle']", wait: 5)
 
     %w[user_password user_password_confirmation].each do |field_id|
+      wrapper = find("input##{field_id}").ancestor("[data-controller='password-toggle']")
       password_field = find("input##{field_id}")
       password_field.set("my_secure_password")
       expect(password_field[:type]).to eq("password")
 
-      find(".toggle-password .fa-eye", match: :first).click
-      expect(password_field[:type]).to eq("text")
+      within(wrapper) { find("[data-action='password-toggle#toggle']").click }
+      expect { password_field[:type] }.to eventually(eq("text"))
       expect(password_field.value).to eq("my_secure_password")
     end
   end

From e1b91fb8c943bb87a9c43d8df451e14868721e40 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 08:24:39 -0400
Subject: [PATCH 18/20] Add data-action to all collection controller forms

The collection controller was refactored to use data-action instead
of addEventListener, but only video_recordings was updated. All 17
other forms using the collection controller were missing the action
attributes, breaking search/filter auto-submit.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 app/views/bookmarks/_search_boxes.html.erb                | 1 +
 app/views/bookmarks/tally.html.erb                        | 2 +-
 app/views/community_news/_search_boxes.html.erb           | 1 +
 app/views/events/_manage_search.html.erb                  | 2 +-
 app/views/faqs/_search_boxes.html.erb                     | 2 +-
 app/views/notifications/_search_boxes.html.erb            | 1 +
 app/views/organizations/_search_boxes.html.erb            | 2 +-
 app/views/people/_search_boxes.html.erb                   | 2 +-
 app/views/quotes/_search_boxes.html.erb                   | 3 ++-
 app/views/resources/_search_boxes.html.erb                | 1 +
 app/views/stories/_search_boxes.html.erb                  | 1 +
 app/views/story_ideas/_search_boxes.html.erb              | 2 +-
 app/views/users/_search_boxes.html.erb                    | 2 +-
 app/views/users/sections/_account_activity.html.erb       | 2 +-
 app/views/workshop_variation_ideas/_search_boxes.html.erb | 2 +-
 app/views/workshop_variations/_search_boxes.html.erb      | 2 +-
 app/views/workshops/index.html.erb                        | 3 ++-
 17 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/app/views/bookmarks/_search_boxes.html.erb b/app/views/bookmarks/_search_boxes.html.erb
index ff2c83a77..93fb00ff2 100644
--- a/app/views/bookmarks/_search_boxes.html.erb
+++ b/app/views/bookmarks/_search_boxes.html.erb
@@ -1,6 +1,7 @@
 <%= form_with url: url,
               method: :get,
               data: { controller: "collection",
+                      action: "change->collection#handleChange input->collection#handleInput",
                       turbo_frame: frame_id },
               autocomplete: "off",
               class: "space-y-4" do |f| %>
diff --git a/app/views/bookmarks/tally.html.erb b/app/views/bookmarks/tally.html.erb
index bf448efae..47b21b696 100644
--- a/app/views/bookmarks/tally.html.erb
+++ b/app/views/bookmarks/tally.html.erb
@@ -3,7 +3,7 @@
       <h1 class="text-2xl font-bold mb-4">Bookmarks tally</h1>
 
       <!-- Filter form -->
-      <%= form_tag('/bookmarks/tally', method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+      <%= form_tag('/bookmarks/tally', method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
         <%= render "search_fields" %>
       <% end %>
 
diff --git a/app/views/community_news/_search_boxes.html.erb b/app/views/community_news/_search_boxes.html.erb
index d851956d4..dc63cf31c 100644
--- a/app/views/community_news/_search_boxes.html.erb
+++ b/app/views/community_news/_search_boxes.html.erb
@@ -2,6 +2,7 @@
 <%= form_with url: community_news_index_path,
               method: :get,
               data: {controller: "collection",
+                    action: "change->collection#handleChange input->collection#handleInput",
                     turbo_frame: "community_news_results" },
                     autocomplete: "off",
               class: "space-y-2" do |f| %>
diff --git a/app/views/events/_manage_search.html.erb b/app/views/events/_manage_search.html.erb
index 51b5a0e41..799a2f743 100644
--- a/app/views/events/_manage_search.html.erb
+++ b/app/views/events/_manage_search.html.erb
@@ -1,5 +1,5 @@
 <%= form_with url: manage_event_path(@event), method: :get,
-  data: { controller: "collection", turbo_frame: "manage_results" },
+  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "manage_results" },
   autocomplete: "off",
   class: "flex flex-col md:flex-row md:flex-wrap md:items-end gap-4 mb-6" do %>
 
diff --git a/app/views/faqs/_search_boxes.html.erb b/app/views/faqs/_search_boxes.html.erb
index 40c69d4f5..0e5783c44 100644
--- a/app/views/faqs/_search_boxes.html.erb
+++ b/app/views/faqs/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do |f| %>
+<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do |f| %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/notifications/_search_boxes.html.erb b/app/views/notifications/_search_boxes.html.erb
index 2511c1c84..d18f3a817 100644
--- a/app/views/notifications/_search_boxes.html.erb
+++ b/app/views/notifications/_search_boxes.html.erb
@@ -3,6 +3,7 @@
   <%= form_with url: notifications_path,
                 method: :get,
                 data: {controller: "collection",
+                      action: "change->collection#handleChange input->collection#handleInput",
                       turbo_frame: "notifications_results" },
                       autocomplete: "off",
                 class: "flex flex-wrap gap-4" do |f| %>
diff --git a/app/views/organizations/_search_boxes.html.erb b/app/views/organizations/_search_boxes.html.erb
index c06da8533..65d65ea9c 100644
--- a/app/views/organizations/_search_boxes.html.erb
+++ b/app/views/organizations/_search_boxes.html.erb
@@ -1,6 +1,6 @@
 <div class="mb-4">
   <%= form_with url: organizations_path, method: :get,
-    data: { controller: "collection", turbo_frame: "organization_results" },
+    data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "organization_results" },
     autocomplete: "off",
     class: "flex flex-wrap items-end gap-3" do %>
     <div>
diff --git a/app/views/people/_search_boxes.html.erb b/app/views/people/_search_boxes.html.erb
index 98d76ab3b..a38165f89 100644
--- a/app/views/people/_search_boxes.html.erb
+++ b/app/views/people/_search_boxes.html.erb
@@ -1,5 +1,5 @@
 <%= form_tag(people_path, method: :get,
-  data: { controller: "collection", turbo_frame: "people_results" },
+  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "people_results" },
   autocomplete: "off",
   class: "space-y-4") do |f| %>
   <div class="flex flex-col md:flex-row md:flex-wrap md:items-end gap-4 mb-6">
diff --git a/app/views/quotes/_search_boxes.html.erb b/app/views/quotes/_search_boxes.html.erb
index 59c724a54..550ff12c4 100644
--- a/app/views/quotes/_search_boxes.html.erb
+++ b/app/views/quotes/_search_boxes.html.erb
@@ -2,7 +2,8 @@
 <div class="mb-6 p-4 bg-white border border-gray-200 rounded-lg">
   <%= form_with url: quotes_path,
                 method: :get,
-                data: {controller: "collection"},
+                data: {controller: "collection",
+                      action: "change->collection#handleChange input->collection#handleInput"},
                       autocomplete: "off",
                 class: "flex flex-wrap gap-4" do |f| %>
     <div class="flex-grow min-w-[200px]">
diff --git a/app/views/resources/_search_boxes.html.erb b/app/views/resources/_search_boxes.html.erb
index 2cedc9f1c..7e8d3cf0b 100644
--- a/app/views/resources/_search_boxes.html.erb
+++ b/app/views/resources/_search_boxes.html.erb
@@ -6,6 +6,7 @@
 
 <%= form_tag resources_path, method: :get,
   data: {controller: "collection share-url",
+         action: "change->collection#handleChange input->collection#handleInput",
          collection_unselected_class: default_btn,
          collection_selected_class: selected_btn,
          turbo_frame: "resource_results" },
diff --git a/app/views/stories/_search_boxes.html.erb b/app/views/stories/_search_boxes.html.erb
index de046f5ed..bab79473a 100644
--- a/app/views/stories/_search_boxes.html.erb
+++ b/app/views/stories/_search_boxes.html.erb
@@ -2,6 +2,7 @@
 <%= form_with url: stories_path,
               method: :get,
               data: {controller: "collection",
+                    action: "change->collection#handleChange input->collection#handleInput",
                     turbo_frame: "story_results" },
                     autocomplete: "off",
               class: "space-y-2" do |f| %>
diff --git a/app/views/story_ideas/_search_boxes.html.erb b/app/views/story_ideas/_search_boxes.html.erb
index bb78ff730..a7bc90a9f 100644
--- a/app/views/story_ideas/_search_boxes.html.erb
+++ b/app/views/story_ideas/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(story_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+<%= form_tag(story_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/users/_search_boxes.html.erb b/app/views/users/_search_boxes.html.erb
index 4d645291e..b0439fc0a 100644
--- a/app/views/users/_search_boxes.html.erb
+++ b/app/views/users/_search_boxes.html.erb
@@ -1,5 +1,5 @@
 <%= form_tag(users_path, method: :get,
-  data: { controller: "collection", turbo_frame: "users_results" },
+  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "users_results" },
   autocomplete: "off",
   class: "w-full") do %>
   <div class="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-4 gap-4 mb-6 items-end">
diff --git a/app/views/users/sections/_account_activity.html.erb b/app/views/users/sections/_account_activity.html.erb
index e60c38e2a..8647d15a7 100644
--- a/app/views/users/sections/_account_activity.html.erb
+++ b/app/views/users/sections/_account_activity.html.erb
@@ -9,7 +9,7 @@
     </div>
 
     <%= form_tag(user_path(user), method: :get,
-      data: { controller: "collection", turbo_frame: "account_activity_section" },
+      data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "account_activity_section" },
       autocomplete: "off",
       class: "mb-4") do %>
       <%= hidden_field_tag :section, "account_activity" %>
diff --git a/app/views/workshop_variation_ideas/_search_boxes.html.erb b/app/views/workshop_variation_ideas/_search_boxes.html.erb
index 06a3f3048..b03f0a3d7 100644
--- a/app/views/workshop_variation_ideas/_search_boxes.html.erb
+++ b/app/views/workshop_variation_ideas/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(workshop_variation_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+<%= form_tag(workshop_variation_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/workshop_variations/_search_boxes.html.erb b/app/views/workshop_variations/_search_boxes.html.erb
index b57a1de43..58b35e1da 100644
--- a/app/views/workshop_variations/_search_boxes.html.erb
+++ b/app/views/workshop_variations/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(workshop_variations_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+<%= form_tag(workshop_variations_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/workshops/index.html.erb b/app/views/workshops/index.html.erb
index b33d59c61..2917e361f 100644
--- a/app/views/workshops/index.html.erb
+++ b/app/views/workshops/index.html.erb
@@ -30,7 +30,8 @@
   <div class="w-full">
     <%= form_tag(workshops_path, method: :get,
                  data: {turbo_stream: true, turbo_frame: "workshop_results",
-                        controller: "collection share-url"}, autocomplete: "off", class: "space-y-6") do %>
+                        controller: "collection share-url",
+                        action: "change->collection#handleChange input->collection#handleInput"}, autocomplete: "off", class: "space-y-6") do %>
       <%= render "filters" %>
 
       <div><hr class="border-gray-300 my-6"></div>

From a5cad9ae3d18215fb08e546b288011eb04e9cbe4 Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 19:09:05 -0400
Subject: [PATCH 19/20] Revert collection, dropdown, remote_select per PR
 review
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- collection_controller: Keep addEventListener in connect() — idiomatic
  when listening on the controller's root element, avoids data-action
  attributes in 18 views
- dropdown_controller: Use this.open instance property instead of
  Stimulus value — values are for DOM-reflected state, not internal tracking
- remote_select_controller: Remove styleId guard — unnecessary abstraction
- paginated_fields_controller: Use this.currentPage instance property
- comment_edit_toggle_controller: Use this.editing instance property

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .../controllers/collection_controller.js      | 46 ++++++++--------
 .../comment_edit_toggle_controller.js         | 17 +++---
 .../controllers/dropdown_controller.js        | 10 ++--
 .../paginated_fields_controller.js            | 30 +++++------
 .../controllers/remote_select_controller.js   | 54 +++++++++----------
 app/views/bookmarks/_search_boxes.html.erb    |  1 -
 app/views/bookmarks/tally.html.erb            |  2 +-
 .../community_news/_search_boxes.html.erb     |  1 -
 app/views/events/_manage_search.html.erb      |  2 +-
 app/views/faqs/_search_boxes.html.erb         |  2 +-
 .../notifications/_search_boxes.html.erb      |  1 -
 .../organizations/_search_boxes.html.erb      |  2 +-
 app/views/people/_search_boxes.html.erb       |  2 +-
 app/views/quotes/_search_boxes.html.erb       |  3 +-
 app/views/resources/_search_boxes.html.erb    |  1 -
 app/views/stories/_search_boxes.html.erb      |  1 -
 app/views/story_ideas/_search_boxes.html.erb  |  2 +-
 app/views/users/_search_boxes.html.erb        |  2 +-
 .../users/sections/_account_activity.html.erb |  2 +-
 .../video_recordings/_search_boxes.html.erb   |  4 +-
 .../_search_boxes.html.erb                    |  2 +-
 .../_search_boxes.html.erb                    |  2 +-
 app/views/workshops/index.html.erb            |  3 +-
 23 files changed, 89 insertions(+), 103 deletions(-)

diff --git a/app/frontend/javascript/controllers/collection_controller.js b/app/frontend/javascript/controllers/collection_controller.js
index f15a7dcc2..b9f56ce46 100644
--- a/app/frontend/javascript/controllers/collection_controller.js
+++ b/app/frontend/javascript/controllers/collection_controller.js
@@ -3,32 +3,28 @@ import { Controller } from "@hotwired/stimulus";
 // Connects to data-controller="collection"
 export default class extends Controller {
   static classes = ["unselected", "selected"];
+  connect() {
+    this.element.addEventListener("change", (event) => {
+      const { type } = event.target;
 
-  handleChange(event) {
-    const { type } = event.target;
-
-    if (type === "checkbox") {
-      this.toggleClass(event.target);
-    }
-
-    if (
-      type === "checkbox" ||
-      type === "radio" ||
-      type === "select-one" ||
-      type === "select-multiple"
-    ) {
-      this.submitForm();
-    }
-  }
-
-  handleInput(event) {
-    if (event.target.type === "text" || event.target.type === "search") {
-      this.debouncedSubmit();
-    }
-  }
+      if (type === "checkbox") {
+        this.toggleClass(event.target);
+      }
 
-  disconnect() {
-    clearTimeout(this.timeout);
+      if (
+        type === "checkbox" ||
+        type === "radio" ||
+        type === "select-one" ||
+        type === "select-multiple"
+      ) {
+        this.submitForm();
+      }
+    });
+    this.element.addEventListener("input", (event) => {
+      if (event.target.type === "text") {
+        this.debouncedSubmit();
+      }
+    });
   }
 
   submitForm() {
@@ -47,10 +43,12 @@ export default class extends Controller {
     const button = el.closest("label");
     if (!button || !this.selectedClasses) return;
 
+    // Toggle selected classes
     this.selectedClasses.forEach((cls) => {
       button.classList.toggle(cls);
     });
 
+    // Toggle unselected classes
     this.unselectedClasses.forEach((cls) => {
       button.classList.toggle(cls);
     });
diff --git a/app/frontend/javascript/controllers/comment_edit_toggle_controller.js b/app/frontend/javascript/controllers/comment_edit_toggle_controller.js
index 76757d459..c5751531c 100644
--- a/app/frontend/javascript/controllers/comment_edit_toggle_controller.js
+++ b/app/frontend/javascript/controllers/comment_edit_toggle_controller.js
@@ -7,13 +7,16 @@ import { Controller } from "@hotwired/stimulus";
 //
 export default class extends Controller {
   static targets = ["editLabel", "viewLabel", "commentView", "commentEdit"];
-  static values = { editing: Boolean };
+
+  connect() {
+    this.editing = false;
+  }
 
   toggle() {
-    this.editingValue = !this.editingValue;
+    this.editing = !this.editing;
 
     // When leaving edit mode, sync textarea values to view display
-    if (!this.editingValue) {
+    if (!this.editing) {
       this.commentEditTargets.forEach((editDiv, i) => {
         const textarea = editDiv.querySelector("textarea");
         const viewBody = this.commentViewTargets[i]?.querySelector(".comment-body");
@@ -26,12 +29,12 @@ export default class extends Controller {
       });
     }
 
-    this.commentViewTargets.forEach((el) => el.classList.toggle("hidden", this.editingValue));
-    this.commentEditTargets.forEach((el) => el.classList.toggle("hidden", !this.editingValue));
+    this.commentViewTargets.forEach((el) => el.classList.toggle("hidden", this.editing));
+    this.commentEditTargets.forEach((el) => el.classList.toggle("hidden", !this.editing));
 
     if (this.hasEditLabelTarget && this.hasViewLabelTarget) {
-      this.editLabelTarget.classList.toggle("hidden", this.editingValue);
-      this.viewLabelTarget.classList.toggle("hidden", !this.editingValue);
+      this.editLabelTarget.classList.toggle("hidden", this.editing);
+      this.viewLabelTarget.classList.toggle("hidden", !this.editing);
     }
   }
 }
diff --git a/app/frontend/javascript/controllers/dropdown_controller.js b/app/frontend/javascript/controllers/dropdown_controller.js
index 30ffdc291..7e9a1831f 100644
--- a/app/frontend/javascript/controllers/dropdown_controller.js
+++ b/app/frontend/javascript/controllers/dropdown_controller.js
@@ -14,9 +14,9 @@ import { Controller } from "@hotwired/stimulus";
 export default class extends Controller {
   // add a content target if you want the dropdown to close with "escape button" or clicking outside of content
   static targets = ["content"];
-  static values = { open: Boolean };
 
   connect() {
+    this.open = false;
     this.handleOutsideClick = this.handleOutsideClick.bind(this);
     this.handleEscapeKey = this.handleEscapeKey.bind(this);
   }
@@ -24,7 +24,7 @@ export default class extends Controller {
   toggle(event) {
     this.processPayload(event.params.payload);
     this.manageEventListeners();
-    this.openValue = !this.openValue;
+    this.open = !this.open;
   }
 
   toggleClassesOnElement(element, classString) {
@@ -41,7 +41,7 @@ export default class extends Controller {
   }
 
   manageEventListeners() {
-    if (this.openValue) {
+    if (this.open) {
       this.removeEventListeners();
     } else {
       // Add delay to avoid immediate trigger
@@ -79,10 +79,10 @@ export default class extends Controller {
   }
 
   close() {
-    if (this.openValue) {
+    if (this.open) {
       this.processPayload(this.lastPayload);
       this.removeEventListeners();
-      this.openValue = false;
+      this.open = false;
     }
   }
 
diff --git a/app/frontend/javascript/controllers/paginated_fields_controller.js b/app/frontend/javascript/controllers/paginated_fields_controller.js
index 8562903da..5bb1b4674 100644
--- a/app/frontend/javascript/controllers/paginated_fields_controller.js
+++ b/app/frontend/javascript/controllers/paginated_fields_controller.js
@@ -2,12 +2,10 @@ import { Controller } from "@hotwired/stimulus";
 
 export default class extends Controller {
   static targets = ["item", "nav"];
-  static values = {
-    perPage: { type: Number, default: 10 },
-    currentPage: { type: Number, default: 1 }
-  };
+  static values = { perPage: { type: Number, default: 10 } };
 
   connect() {
+    this.currentPage = 1;
     this.render();
   }
 
@@ -22,30 +20,30 @@ export default class extends Controller {
   }
 
   next() {
-    if (this.currentPageValue < this.totalPages) {
-      this.currentPageValue++;
+    if (this.currentPage < this.totalPages) {
+      this.currentPage++;
       this.render();
     }
   }
 
   previous() {
-    if (this.currentPageValue > 1) {
-      this.currentPageValue--;
+    if (this.currentPage > 1) {
+      this.currentPage--;
       this.render();
     }
   }
 
   render() {
     const items = this.visibleItems;
-    const start = (this.currentPageValue - 1) * this.perPageValue;
+    const start = (this.currentPage - 1) * this.perPageValue;
     const end = start + this.perPageValue;
 
     items.forEach((el, i) => {
       el.classList.toggle("hidden", i < start || i >= end);
     });
 
-    if (this.currentPageValue > this.totalPages) {
-      this.currentPageValue = this.totalPages;
+    if (this.currentPage > this.totalPages) {
+      this.currentPage = this.totalPages;
       this.render();
       return;
     }
@@ -65,16 +63,16 @@ export default class extends Controller {
     this.navTarget.innerHTML = `
       <div class="flex items-center justify-center gap-3 mt-4">
         <button type="button" data-action="paginated-fields#previous"
-                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPageValue <= 1 ? 'opacity-30 cursor-default' : ''}"
-                ${this.currentPageValue <= 1 ? 'disabled' : ''}>
+                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPage <= 1 ? 'opacity-30 cursor-default' : ''}"
+                ${this.currentPage <= 1 ? 'disabled' : ''}>
           &laquo;
         </button>
         <span class="text-sm text-gray-600">
-          ${this.currentPageValue} / ${this.totalPages}
+          ${this.currentPage} / ${this.totalPages}
         </span>
         <button type="button" data-action="paginated-fields#next"
-                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPageValue >= this.totalPages ? 'opacity-30 cursor-default' : ''}"
-                ${this.currentPageValue >= this.totalPages ? 'disabled' : ''}>
+                class="px-3 py-1 border border-gray-200 rounded-md bg-white text-gray-500 hover:bg-gray-100 ${this.currentPage >= this.totalPages ? 'opacity-30 cursor-default' : ''}"
+                ${this.currentPage >= this.totalPages ? 'disabled' : ''}>
           &raquo;
         </button>
       </div>
diff --git a/app/frontend/javascript/controllers/remote_select_controller.js b/app/frontend/javascript/controllers/remote_select_controller.js
index 19dc3fbd7..0c3d0b7d2 100644
--- a/app/frontend/javascript/controllers/remote_select_controller.js
+++ b/app/frontend/javascript/controllers/remote_select_controller.js
@@ -1,8 +1,6 @@
 import { Controller } from "@hotwired/stimulus";
 import TomSelect from "tom-select";
 
-const styleId = "remote-select-overrides";
-
 export default class extends Controller {
   static values = { model: String, exclude: String };
 
@@ -28,33 +26,31 @@ export default class extends Controller {
           .catch(() => callback());
       },
     });
-
-    if (!document.getElementById(styleId)) {
-      const style = document.createElement("style");
-      style.id = styleId;
-      style.textContent = `
-        .ts-control .ts-input {
-          border: none !important;
-          box-shadow: none !important;
-          outline: none !important;
-          background: transparent !important;
-          padding: 0 !important;
-          margin: 0 !important;
-        }
-        .ts-control {
-          border: none !important;
-          box-shadow: none !important;
-          padding: 0 !important;
-          margin: 0 !important;
-          min-height: 0 !important;
-        }
-        .ts-control .item {
-          margin: 0 !important;
-          padding: 0 !important;
-        }
-      `;
-      document.head.appendChild(style);
-    }
+    // Inject CSS to remove some default tom-select styles -might be a better way to do this.
+    const style = document.createElement("style");
+    style.textContent = `
+      /* Remove border and shadow */
+      .ts-control .ts-input {
+        border: none !important;
+        box-shadow: none !important;
+        outline: none !important;
+        background: transparent !important;
+        padding: 0 !important;           /* Remove padding from input */
+        margin: 0 !important;            /* Remove margin if any */
+      }
+      .ts-control {
+        border: none !important;
+        box-shadow: none !important;
+        padding: 0 !important;           /* Remove padding from container */
+        margin: 0 !important;
+        min-height: 0 !important;        /* Remove min-height TomSelect sets */
+      }
+      .ts-control .item {
+        margin: 0 !important;            /* Remove padding/margin from selected items */
+        padding: 0 !important;
+      }
+    `;
+    document.head.appendChild(style);
   }
 
   disconnect() {
diff --git a/app/views/bookmarks/_search_boxes.html.erb b/app/views/bookmarks/_search_boxes.html.erb
index 93fb00ff2..ff2c83a77 100644
--- a/app/views/bookmarks/_search_boxes.html.erb
+++ b/app/views/bookmarks/_search_boxes.html.erb
@@ -1,7 +1,6 @@
 <%= form_with url: url,
               method: :get,
               data: { controller: "collection",
-                      action: "change->collection#handleChange input->collection#handleInput",
                       turbo_frame: frame_id },
               autocomplete: "off",
               class: "space-y-4" do |f| %>
diff --git a/app/views/bookmarks/tally.html.erb b/app/views/bookmarks/tally.html.erb
index 47b21b696..bf448efae 100644
--- a/app/views/bookmarks/tally.html.erb
+++ b/app/views/bookmarks/tally.html.erb
@@ -3,7 +3,7 @@
       <h1 class="text-2xl font-bold mb-4">Bookmarks tally</h1>
 
       <!-- Filter form -->
-      <%= form_tag('/bookmarks/tally', method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
+      <%= form_tag('/bookmarks/tally', method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
         <%= render "search_fields" %>
       <% end %>
 
diff --git a/app/views/community_news/_search_boxes.html.erb b/app/views/community_news/_search_boxes.html.erb
index dc63cf31c..d851956d4 100644
--- a/app/views/community_news/_search_boxes.html.erb
+++ b/app/views/community_news/_search_boxes.html.erb
@@ -2,7 +2,6 @@
 <%= form_with url: community_news_index_path,
               method: :get,
               data: {controller: "collection",
-                    action: "change->collection#handleChange input->collection#handleInput",
                     turbo_frame: "community_news_results" },
                     autocomplete: "off",
               class: "space-y-2" do |f| %>
diff --git a/app/views/events/_manage_search.html.erb b/app/views/events/_manage_search.html.erb
index 799a2f743..51b5a0e41 100644
--- a/app/views/events/_manage_search.html.erb
+++ b/app/views/events/_manage_search.html.erb
@@ -1,5 +1,5 @@
 <%= form_with url: manage_event_path(@event), method: :get,
-  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "manage_results" },
+  data: { controller: "collection", turbo_frame: "manage_results" },
   autocomplete: "off",
   class: "flex flex-col md:flex-row md:flex-wrap md:items-end gap-4 mb-6" do %>
 
diff --git a/app/views/faqs/_search_boxes.html.erb b/app/views/faqs/_search_boxes.html.erb
index 0e5783c44..40c69d4f5 100644
--- a/app/views/faqs/_search_boxes.html.erb
+++ b/app/views/faqs/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do |f| %>
+<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do |f| %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/notifications/_search_boxes.html.erb b/app/views/notifications/_search_boxes.html.erb
index d18f3a817..2511c1c84 100644
--- a/app/views/notifications/_search_boxes.html.erb
+++ b/app/views/notifications/_search_boxes.html.erb
@@ -3,7 +3,6 @@
   <%= form_with url: notifications_path,
                 method: :get,
                 data: {controller: "collection",
-                      action: "change->collection#handleChange input->collection#handleInput",
                       turbo_frame: "notifications_results" },
                       autocomplete: "off",
                 class: "flex flex-wrap gap-4" do |f| %>
diff --git a/app/views/organizations/_search_boxes.html.erb b/app/views/organizations/_search_boxes.html.erb
index 65d65ea9c..c06da8533 100644
--- a/app/views/organizations/_search_boxes.html.erb
+++ b/app/views/organizations/_search_boxes.html.erb
@@ -1,6 +1,6 @@
 <div class="mb-4">
   <%= form_with url: organizations_path, method: :get,
-    data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "organization_results" },
+    data: { controller: "collection", turbo_frame: "organization_results" },
     autocomplete: "off",
     class: "flex flex-wrap items-end gap-3" do %>
     <div>
diff --git a/app/views/people/_search_boxes.html.erb b/app/views/people/_search_boxes.html.erb
index a38165f89..98d76ab3b 100644
--- a/app/views/people/_search_boxes.html.erb
+++ b/app/views/people/_search_boxes.html.erb
@@ -1,5 +1,5 @@
 <%= form_tag(people_path, method: :get,
-  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "people_results" },
+  data: { controller: "collection", turbo_frame: "people_results" },
   autocomplete: "off",
   class: "space-y-4") do |f| %>
   <div class="flex flex-col md:flex-row md:flex-wrap md:items-end gap-4 mb-6">
diff --git a/app/views/quotes/_search_boxes.html.erb b/app/views/quotes/_search_boxes.html.erb
index 550ff12c4..59c724a54 100644
--- a/app/views/quotes/_search_boxes.html.erb
+++ b/app/views/quotes/_search_boxes.html.erb
@@ -2,8 +2,7 @@
 <div class="mb-6 p-4 bg-white border border-gray-200 rounded-lg">
   <%= form_with url: quotes_path,
                 method: :get,
-                data: {controller: "collection",
-                      action: "change->collection#handleChange input->collection#handleInput"},
+                data: {controller: "collection"},
                       autocomplete: "off",
                 class: "flex flex-wrap gap-4" do |f| %>
     <div class="flex-grow min-w-[200px]">
diff --git a/app/views/resources/_search_boxes.html.erb b/app/views/resources/_search_boxes.html.erb
index 7e8d3cf0b..2cedc9f1c 100644
--- a/app/views/resources/_search_boxes.html.erb
+++ b/app/views/resources/_search_boxes.html.erb
@@ -6,7 +6,6 @@
 
 <%= form_tag resources_path, method: :get,
   data: {controller: "collection share-url",
-         action: "change->collection#handleChange input->collection#handleInput",
          collection_unselected_class: default_btn,
          collection_selected_class: selected_btn,
          turbo_frame: "resource_results" },
diff --git a/app/views/stories/_search_boxes.html.erb b/app/views/stories/_search_boxes.html.erb
index bab79473a..de046f5ed 100644
--- a/app/views/stories/_search_boxes.html.erb
+++ b/app/views/stories/_search_boxes.html.erb
@@ -2,7 +2,6 @@
 <%= form_with url: stories_path,
               method: :get,
               data: {controller: "collection",
-                    action: "change->collection#handleChange input->collection#handleInput",
                     turbo_frame: "story_results" },
                     autocomplete: "off",
               class: "space-y-2" do |f| %>
diff --git a/app/views/story_ideas/_search_boxes.html.erb b/app/views/story_ideas/_search_boxes.html.erb
index a7bc90a9f..bb78ff730 100644
--- a/app/views/story_ideas/_search_boxes.html.erb
+++ b/app/views/story_ideas/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(story_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
+<%= form_tag(story_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/users/_search_boxes.html.erb b/app/views/users/_search_boxes.html.erb
index b0439fc0a..4d645291e 100644
--- a/app/views/users/_search_boxes.html.erb
+++ b/app/views/users/_search_boxes.html.erb
@@ -1,5 +1,5 @@
 <%= form_tag(users_path, method: :get,
-  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "users_results" },
+  data: { controller: "collection", turbo_frame: "users_results" },
   autocomplete: "off",
   class: "w-full") do %>
   <div class="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-4 gap-4 mb-6 items-end">
diff --git a/app/views/users/sections/_account_activity.html.erb b/app/views/users/sections/_account_activity.html.erb
index 8647d15a7..e60c38e2a 100644
--- a/app/views/users/sections/_account_activity.html.erb
+++ b/app/views/users/sections/_account_activity.html.erb
@@ -9,7 +9,7 @@
     </div>
 
     <%= form_tag(user_path(user), method: :get,
-      data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "account_activity_section" },
+      data: { controller: "collection", turbo_frame: "account_activity_section" },
       autocomplete: "off",
       class: "mb-4") do %>
       <%= hidden_field_tag :section, "account_activity" %>
diff --git a/app/views/video_recordings/_search_boxes.html.erb b/app/views/video_recordings/_search_boxes.html.erb
index 50eb218f1..6118e0bbf 100644
--- a/app/views/video_recordings/_search_boxes.html.erb
+++ b/app/views/video_recordings/_search_boxes.html.erb
@@ -1,7 +1,5 @@
 <!-- Filters -->
-<div class="mb-6 p-4 bg-white border border-gray-200 rounded-lg"
-     data-controller="collection"
-     data-action="change->collection#handleChange input->collection#handleInput">
+<div class="mb-6 p-4 bg-white border border-gray-200 rounded-lg" data-controller="collection">
   <%= form_with url: video_recordings_path,
                 method: :get,
                 data: { turbo_frame: "video_recordings_results", controller: "searchable-checkbox" },
diff --git a/app/views/workshop_variation_ideas/_search_boxes.html.erb b/app/views/workshop_variation_ideas/_search_boxes.html.erb
index b03f0a3d7..06a3f3048 100644
--- a/app/views/workshop_variation_ideas/_search_boxes.html.erb
+++ b/app/views/workshop_variation_ideas/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(workshop_variation_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
+<%= form_tag(workshop_variation_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/workshop_variations/_search_boxes.html.erb b/app/views/workshop_variations/_search_boxes.html.erb
index 58b35e1da..b57a1de43 100644
--- a/app/views/workshop_variations/_search_boxes.html.erb
+++ b/app/views/workshop_variations/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(workshop_variations_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
+<%= form_tag(workshop_variations_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/workshops/index.html.erb b/app/views/workshops/index.html.erb
index 2917e361f..b33d59c61 100644
--- a/app/views/workshops/index.html.erb
+++ b/app/views/workshops/index.html.erb
@@ -30,8 +30,7 @@
   <div class="w-full">
     <%= form_tag(workshops_path, method: :get,
                  data: {turbo_stream: true, turbo_frame: "workshop_results",
-                        controller: "collection share-url",
-                        action: "change->collection#handleChange input->collection#handleInput"}, autocomplete: "off", class: "space-y-6") do %>
+                        controller: "collection share-url"}, autocomplete: "off", class: "space-y-6") do %>
       <%= render "filters" %>
 
       <div><hr class="border-gray-300 my-6"></div>

From 254cd701e9b5ec79bae8b823a7c8cb3c3048e67c Mon Sep 17 00:00:00 2001
From: maebeale <maebeale@users.noreply.github.com>
Date: Tue, 10 Mar 2026 19:11:18 -0400
Subject: [PATCH 20/20] Add Stimulus conventions from PR review feedback

- addEventListener in connect() is fine for root-element events
- Use instance properties for internal state, not static values
- Don't add guards without evidence of a bug

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 CLAUDE.md | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/CLAUDE.md b/CLAUDE.md
index 5f5220e75..937320ff2 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -165,6 +165,9 @@ bundle exec bundle-audit check --update
 - Keep controllers focused and small
 - **Use Stimulus targets and data attributes** to reference DOM elements — avoid `this.element.querySelector` and direct DOM queries. Declare `static targets = [...]` and use `data-[controller]-target` attributes in views.
 - **Use Stimulus shorthand action descriptors and shorthand pairs** — omit the event when it's the default for that element (e.g., `input` for `<input>`/`<textarea>`, `click` for `<button>`/`<a>`, `submit` for `<form>`). Write `controller#action` not `input->controller#action` on an input element. Only specify the event when using a non-default (e.g., `change->controller#action` on an input). See [Stimulus Actions](https://stimulus.hotwired.dev/reference/actions#event-shorthand).
+- **`this.element.addEventListener` in `connect()` is fine** when listening on the controller's root element (e.g., a form). This is equivalent to `data-action` but avoids adding HTML attributes to every view that uses the controller. Prefer `connect()` listeners for root-element events; use `data-action` for events on specific child elements.
+- **Use instance properties for internal state** — `this.open`, `this.editing`, `this.currentPage`. Reserve `static values` for state that needs to be reflected in or read from the DOM (e.g., configuration passed via `data-[controller]-[name]-value` attributes in HTML).
+- **Don't add guards or abstractions without a clear bug** — e.g., don't add duplicate-injection guards for style elements unless there's evidence of the bug occurring.
 
 ## Migrations