Wire RecurringTemplateSelector to repository layer
epic-coordinator-proxy-registration-core-task-005 — Connect the RecurringTemplateSelector widget to the recurring_activity_templates Supabase table via Riverpod provider. Implement RLS-filtered query returning only templates owned by the current coordinator's organisation unit. Support optimistic cache so templates load instantly on repeated opens.
Acceptance Criteria
Technical Requirements
Execution Context
Tier 2 - 518 tasks
Can start after Tier 1 completes
Implementation Notes
Create `RecurringTemplateRepository` with a single method `Future> fetchForCurrentCoordinator()`. Inject `SupabaseClient` via Riverpod provider. Use Riverpod's `keepAlive: true` on the `AsyncNotifierProvider` to maintain cache across widget rebuilds — but invalidate via `ref.invalidate()` on mutation events. The Supabase query: `supabase.from('recurring_activity_templates').select('id, activity_type_id, activity_types(name), default_duration_minutes, notes_template').order('created_at', ascending: false)`.
Map the joined `activity_types(name)` into `activityTypeName` on the model. Handle the case where `activity_type_id` references a deleted activity type (return the template with a fallback name 'Unknown type'). Do not use `.single()` — use `.select()` which returns a list and handles zero rows gracefully.
Testing Requirements
Integration tests using flutter_test with a mocked Supabase client (use `mocktail` to mock `SupabaseClient`). Test cases: (1) successful fetch maps JSON correctly to List
The 2-hour window duplicate detection logic requires querying existing proxy records with compound key matching (mentor + date + activity type within time range). If the query is too broad it produces false positives that frustrate coordinators; if too narrow it misses genuine duplicates that corrupt Bufdir data.
Mitigation & Contingency
Mitigation: Define the duplicate detection window as a configurable parameter from the start. Prototype the Supabase query with representative data covering edge cases (midnight boundaries, different activity types same day, same activity type different mentors) before finalising the implementation.
Contingency: If the detection produces excessive false positives in UAT, allow coordinators to explicitly acknowledge and bypass the duplicate warning with a reason field, preserving safety while reducing friction.
If the proxy registration form does not clearly distinguish between the acting coordinator and the attributed mentor, coordinators may submit records attributing activities to themselves, causing inaccurate Bufdir reporting and potential funding issues.
Mitigation & Contingency
Mitigation: Conduct UAT with at least one real coordinator via TestFlight before release. Use distinct visual treatment (different card colours, explicit 'Registering on behalf of:' label) and require the confirmation screen to show both identities prominently.
Contingency: Add a mandatory confirmation checkbox on the confirmation screen that explicitly names the attributed mentor, preventing accidental self-attribution from slipping through.
Coordinators with multi-chapter access must select an active chapter context before the mentor list is filtered correctly. If chapter scope resolution fails or is bypassed, cross-org proxy registrations could occur, violating data isolation between chapters.
Mitigation & Contingency
Mitigation: Reuse the existing active-chapter-state and hierarchy-service components established by the org hierarchy feature. Add a guard that blocks entry to the proxy flow if no chapter context is active, prompting chapter selection first.
Contingency: If the chapter resolution service is unavailable, default to the most restrictive scope (no mentors visible) and surface a clear error message rather than showing an unfiltered mentor list.