critical priority low complexity backend pending backend specialist Tier 0

Acceptance Criteria

ProxyActivityPayload Dart class exists and extends or composes ActivityRegistration without breaking existing usages
Class includes required fields: recorded_by_user_id (String, coordinator's user ID) and peer_mentor_id (String, attributed peer mentor's user ID)
toJson() and fromJson() methods handle both fields correctly, with null safety enforced
JSON keys match the Supabase column names exactly (snake_case): recorded_by_user_id, peer_mentor_id
dart analyze reports zero errors or warnings on the new model file
Existing ActivityRegistration model and all its usages remain unmodified
A unit test verifies round-trip serialization: Dart object → JSON → Dart object produces identical values for all fields
Code review confirms no duplicate field definitions between ProxyActivityPayload and ActivityRegistration

Technical Requirements

frameworks
Flutter
Dart
apis
Supabase REST API (activities table insert)
data models
ActivityRegistration
ProxyActivityPayload
security requirements
recorded_by_user_id must be derived from the authenticated session server-side — never trust client-supplied coordinator ID without validation
peer_mentor_id must reference a valid user in the same organization — validate foreign key at the Supabase RLS level

Execution Context

Execution Tier
Tier 0

Tier 0 - 440 tasks

Implementation Notes

Prefer composition over inheritance: hold an ActivityRegistration instance inside ProxyActivityPayload rather than extending it, to avoid tight coupling to the base class constructor. Use Dart's `copyWith` pattern if the base class supports it. For JSON serialization, manually implement toJson/fromJson rather than introducing a code generation dependency (json_serializable) unless the project already uses it. Align field naming strictly with the Supabase activities table schema — verify column names in the database migration files before finalizing.

Place the new model in the same directory as ActivityRegistration (e.g., `lib/features/activities/models/`) to keep domain models co-located.

Testing Requirements

Write unit tests using flutter_test covering: (1) ProxyActivityPayload.toJson() includes all ActivityRegistration fields plus the two new proxy fields, (2) fromJson() correctly parses a JSON map with proxy fields, (3) fromJson() handles missing optional fields gracefully without throwing, (4) equality operator returns true for two instances with identical fields. No widget or integration tests required for this task.

Component
Proxy Activity Wizard
ui medium
Epic Risks (3)
medium impact medium prob scope

If the batch insert RPC returns a mix of successes and failures (e.g., 3 of 10 mentors fail due to constraint violations that slipped through application-level duplicate detection), the confirmation screen result state becomes ambiguous. A coordinator who sees '7 of 10 succeeded' may not know whether to manually register the 3 failures, retry, or escalate — leading to either duplicate registrations or silent underreporting.

Mitigation & Contingency

Mitigation: Design the Bulk Registration Service to return a strongly typed BulkRegistrationResult with per-mentor RegistrationOutcome (success | duplicate_detected | constraint_violation | permission_denied). Design the result screen to list each failed mentor with a specific, plain-language reason and a one-tap 'Retry for this mentor' action that pre-fills the activity wizard with the batch template for that individual.

Contingency: If per-mentor retry UI is too complex to deliver within the epic scope, fall back to displaying failed mentors with their error codes and instructing coordinators to use single-proxy mode for the failures. Document this as a known limitation in release notes and create a follow-up ticket for per-mentor retry in the next sprint.

medium impact medium prob dependency

The Proxy Activity Wizard must reuse the existing activity wizard step widgets (type, date, duration, notes) while injecting a proxy attribution banner and a different submission payload builder. If the existing wizard is not designed for composability, the proxy variant may require forking the widget tree, creating two maintenance-diverging codebases that will drift out of sync when the base wizard is updated (e.g., new activity types added, new mandatory fields).

Mitigation & Contingency

Mitigation: Before implementing the Proxy Activity Wizard, audit the existing activity wizard's architecture. If steps are already extracted as independent StatelessWidget/ConsumerWidget classes, compose them directly with a wrapping Column that injects the attribution banner. If they are tightly coupled inside a parent widget, refactor the existing wizard to accept a nullable ProxyContext parameter before starting the proxy variant — this refactor should be a prerequisite task in this epic.

Contingency: If refactoring the base wizard is blocked by unrelated in-flight work on that component, implement the proxy wizard as a full fork but create a shared StepWidgets library file that both the base wizard and proxy wizard import. Schedule a deduplication refactor as a tech-debt ticket in the next planning cycle.

medium impact medium prob technical

The bulk registration flow spans three sequential screens (multi-select → activity form → confirmation → result) with shared mutable state: the selected mentor list, the activity template, the per-mentor duplicate warnings, and the final submission result. Managing this state across screens without a well-designed Bloc risks state leaks, stale duplicate warning data after mentor removal, and confirmation screen inconsistencies if the user navigates back and changes the mentor selection.

Mitigation & Contingency

Mitigation: Define a single BulkRegistrationBloc (or Cubit) with explicit state transitions covering: MentorsSelected → ActivityTemplateCompleted → DuplicatesChecked → ConfirmationReady → Submitting → SubmissionResult. Each backward navigation event (e.g., 'Back' from confirmation to mentor selection) dispatches a ResetToMentorSelection event that clears downstream state. Unit test every state transition with edge cases including empty mentor list, all mentors having duplicates, and network failure during submission.

Contingency: If state management complexity causes persistent bugs in testing, simplify by passing state explicitly through Navigator arguments (immutable snapshots per screen) rather than a shared Bloc. This reduces flexibility but eliminates cross-screen state mutation bugs.