Add feature controllers, models, repos & health updates

Adds a large set of feature-focused controllers, data models, and repositories plus related view/route and pubspec updates. Key additions:
- New controllers/providers: gear_reviews, knowledge_base, pet_breed (breed identifier + history), pet_events, pet_insurance, pet_memorial, pet_nutrition, pet_sitter, pet_training and related providers.
- New models: GearReview, KnowledgeArticle, PetEvent, PetFriendlyPlace, PetMemorialEntry and others used by feature repos.
- New repositories: adoption, community_group, and a consolidated feature_repositories (training, nutrition, gear reviews, memorials, etc.) backed by Supabase utilities.
- HealthController enhancements: include upcomingAppointments in state, overdue appointment counting, and an idempotent 30-day medication dose generation flow (times/frequency helpers and repo call). Also triggers dose regeneration when medications change.
- PetController updates: photo removal method (storage cleanup + DB clear) and a breed suggestions provider for autocomplete.
- Added docs: remaining_issues_plan.md and analysis_results.txt.
- Misc: multiple view files, routes, and package files updated to wire in new features.

These changes implement many previously-mocked/stubbed features (events, groups, adoption, nutrition, training, gear reviews, memorials, breed ID) and improve health/medication handling and pet media management.

Author: syed-reza98

analysis_results.txt

@@ -0,0 +1,110 @@
+# PetSphere – Remaining Issues & Implementation Plan
+
+> **Generated:** May 7, 2026 | Based on GitHub issues + docs/codebase cross-check
+
+## ✅ Already Closed (18 issues)
+
+| # | Title | Epic |
+|---|-------|------|
+| #23 | Cart persistence (SharedPreferences) | #57 |
+| #24 | FCM deep-link navigation | #59 |
+| #25 | Appointment sync (HealthController → PetCare) | #54 |
+| #26 | Server-side stock validation | #57 |
+| #27 | Feed repo compile error | #58 |
+| #28 | Auth: profile upsert fatal | #58 |
+| #30 | Testing foundation | #58 |
+| #31 | Supabase migrations workflow | #58 |
+| #32 | Stripe payment integration | #57 |
+| #35 | Offline support baseline | #55 |
+| #38 | Route hardening | #58 |
+| #40 | Stories expiry filter | #60 |
+| #42 | Comment author pet details | #60 |
+| #54 | EPIC: Care & Health Improvements | — |
+| #57 | EPIC: Marketplace Reliability | — |
+| #58 | EPIC: Foundation & Release Blockers | — |
+| #59 | EPIC: Notifications & FCM | — |
+
+---
+
+## 🔴 Open Issues — 29 remaining
+
+### Group A: Close-ready (implemented but not closed)
+These were implemented in the codebase but GitHub issues left open.
+
+| # | Title | Action |
+|---|-------|--------|
+| #29 | Follower count N+1 → batch `fetchPetFollowerCounts` done | Close |
+| #39 | Care cache flicker → payload dedup done | Close |
+| #41 | Realtime channel cleanup → feed + chat lifecycle done | Close |
+| #48 | Supabase debug fallback config → `SUPABASE_ALLOW_EMBEDDED_DEBUG_FALLBACK` done | Close |
+
+### Group B: Health sub-issues (P1)
+
+| # | Title | Status |
+|---|-------|--------|
+| #43 | Appointment overdue alerts in UI state | ❌ Not implemented |
+| #44 | Auto-generate medication doses (30-day schedule) | ❌ Not implemented |
+| #47 | Vaccination recurrence via reference schedule table | ❌ Not implemented |
+
+### Group C: Pet management (P2)
+
+| # | Title | Status |
+|---|-------|--------|
+| #45 | Pet photo delete → storage cleanup | ❌ Not implemented |
+| #46 | Breed autocomplete + validation | ❌ Not implemented |
+
+### Group D: All stub screens (Epic #56 — P1-P2)
+
+| # | Screen | Status |
+|---|--------|--------|
+| #33 | Vet Booking (reuse `pet_vet_appointments` table) | ❌ Mock only |
+| #34 | Lost & Found (new tables + workflow) | ❌ Mock only |
+| #36 | Community Groups (groups + membership) | ❌ Mock only |
+| #37 | Adoption Center (listings + applications) | ❌ Mock only |
+| #49 | Meta: complete all remaining mock screens | ❌ Open |
+| #50 | Pet Training (programs + progress per pet) | ❌ Mock only |
+| #51 | Pet Sitter Dashboard (profiles + booking) | ❌ Mock only |
+| #52 | Pet Insurance Hub (claims + status) | ❌ Mock only |
+| #62 | Pet Friendly Places (data-driven list/map) | ❌ Mock only |
+| #63 | Nutrition Planner (personalized + persistent) | ❌ Mock only |
+| #64 | Knowledge Base (articles + search) | ❌ Mock only |
+| #65 | Gear Reviews (CRUD + ratings) | ❌ Mock only |
+| #66 | Breed Identifier (real ML/API, not fake scan) | ❌ Fake scan |
+| #67 | Pet Memorial (archival + entries) | ❌ Mock only |
+| #68 | Pet Events Discovery (events + RSVP) | ❌ Mock only |
+
+### Open Epics (need sub-issues done first)
+
+| # | Title | Blocking Sub-issues |
+|---|-------|---------------------|
+| #55 | EPIC: Offline, Performance & Query Efficiency | #29 done → close |
+| #60 | EPIC: Social Feed, Stories & Realtime Quality | #40,#42 done; #41 close |
+| #61 | EPIC: Security & Config Hygiene | #48 close |
+| #69 | EPIC: Pet Profile Quality | #45, #46 |
+| #56 | EPIC: Complete Stub Screens | All #33–68 sub-issues |
+
+---
+
+## 📋 Sequential Implementation Order
+
+```
+Phase 1 (immediate): Close already-done issues #29, #39, #41, #48
+Phase 2: Health issues #43, #44, #47
+Phase 3: Pet mgmt #45, #46
+Phase 4 (stub screens):
+  4a. #33 Vet Booking
+  4b. #34 Lost & Found
+  4c. #36 Community Groups
+  4d. #37 Adoption Center
+  4e. #50 Pet Training
+  4f. #52 Pet Insurance Hub
+  4g. #51 Pet Sitter Dashboard
+  4h. #63 Nutrition Planner
+  4i. #62 Pet Friendly Places
+  4j. #64 Knowledge Base
+  4k. #65 Gear Reviews
+  4l. #67 Pet Memorial
+  4m. #68 Pet Events Discovery
+  4n. #66 Breed Identifier (API-based)
+Phase 5: Close epics #55, #60, #61, #69, #56
+```

docs/plan/remaining_issues_plan.md

@@ -0,0 +1,28 @@
+import 'package:flutter_riverpod/flutter_riverpod.dart';
+import '../models/gear_review_models.dart';
+import '../repositories/feature_repositories.dart';
+
+final gearReviewsRepositoryProvider = Provider<GearReviewsRepository>((ref) {
+  return GearReviewsRepository();
+});
+
+final gearReviewsProvider = FutureProvider.family<List<GearReview>, String?>((ref, category) async {
+  final repository = ref.watch(gearReviewsRepositoryProvider);
+  return repository.fetchReviews(category: category);
+});
+
+class SelectedGearCategoryNotifier extends Notifier<String?> {
+  @override
+  String? build() => null;
+  void set(String? category) => state = category;
+}
+
+final selectedGearCategoryProvider =
+    NotifierProvider<SelectedGearCategoryNotifier, String?>(() {
+  return SelectedGearCategoryNotifier();
+});
+
+final filteredGearReviewsProvider = FutureProvider<List<GearReview>>((ref) async {
+  final category = ref.watch(selectedGearCategoryProvider);
+  return ref.watch(gearReviewsProvider(category).future);
+});

lib/controllers/gear_reviews_controller.dart

@@ -1,3 +1,5 @@
+import 'dart:developer';
+
 import 'package:flutter/foundation.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
 
@@ -18,6 +20,8 @@ class HealthState {
   final List<PetAllergy> allergies;
   final List<ParasitePrevention> parasitePrevention;
   final List<DentalLog> dentalLogs;
+  /// Upcoming/overdue vet appointments (scheduled status only).
+  final List<PetVetAppointment> upcomingAppointments;
   final bool isLoading;
   final String? error;
   final String? activePetId;
@@ -28,6 +32,7 @@ class HealthState {
     this.allergies = const [],
     this.parasitePrevention = const [],
     this.dentalLogs = const [],
+    this.upcomingAppointments = const [],
     this.isLoading = false,
     this.error,
     this.activePetId,
@@ -71,11 +76,17 @@ class HealthState {
     return matches.first;
   }
 
-  /// Number of active health alerts (overdue medications, parasite, etc.)
+  /// Appointments past their scheduled_at that are still 'scheduled' → overdue.
+  List<PetVetAppointment> get overdueAppointments => upcomingAppointments
+      .where((a) => a.status == 'scheduled' && a.scheduledAt.isBefore(DateTime.now()))
+      .toList();
+
+  /// Number of active health alerts (overdue medications, parasite, overdue appts).
   int get alertCount {
     int count = 0;
     count += overdueParasite.length;
     count += todayDoses.where((d) => d.isOverdue).length;
+    count += overdueAppointments.length;
     return count;
   }
 
@@ -94,6 +105,7 @@ class HealthState {
     List<PetAllergy>? allergies,
     List<ParasitePrevention>? parasitePrevention,
     List<DentalLog>? dentalLogs,
+    List<PetVetAppointment>? upcomingAppointments,
     bool? isLoading,
     String? error,
     bool clearError = false,
@@ -105,6 +117,7 @@ class HealthState {
         allergies: allergies ?? this.allergies,
         parasitePrevention: parasitePrevention ?? this.parasitePrevention,
         dentalLogs: dentalLogs ?? this.dentalLogs,
+        upcomingAppointments: upcomingAppointments ?? this.upcomingAppointments,
         isLoading: isLoading ?? this.isLoading,
         error: clearError ? null : (error ?? this.error),
         activePetId: activePetId ?? this.activePetId,
@@ -142,6 +155,7 @@ class HealthNotifier extends Notifier<HealthState> {
         _repo.fetchAllergies(petId),
         _repo.fetchParasitePrevention(petId),
         _repo.fetchDentalLogs(petId),
+        _repo.fetchUpcomingAppointments(petId),
       ]);
       if (!ref.mounted) return;
       state = state.copyWith(
@@ -150,6 +164,7 @@ class HealthNotifier extends Notifier<HealthState> {
         allergies: results[2] as List<PetAllergy>,
         parasitePrevention: results[3] as List<ParasitePrevention>,
         dentalLogs: results[4] as List<DentalLog>,
+        upcomingAppointments: results[5] as List<PetVetAppointment>,
         isLoading: false,
       );
     } catch (e) {
@@ -171,6 +186,8 @@ class HealthNotifier extends Notifier<HealthState> {
       state = state.copyWith(
         medications: [saved, ...state.medications],
       );
+      // Generate dose schedule for the next 30 days (#44).
+      await _generateUpcomingDoses(saved);
     } catch (e) {
       state = state.copyWith(error: e.toString());
     }
@@ -183,6 +200,8 @@ class HealthNotifier extends Notifier<HealthState> {
         medications:
             state.medications.map((m) => m.id == saved.id ? saved : m).toList(),
       );
+      // Regenerate future doses when frequency/schedule changes (#44).
+      await _generateUpcomingDoses(saved);
     } catch (e) {
       state = state.copyWith(error: e.toString());
     }
@@ -350,6 +369,78 @@ class HealthNotifier extends Notifier<HealthState> {
       state = state.copyWith(error: e.toString());
     }
   }
+
+  // ── Dose generation (#44) ────────────────────────────────────────────────
+
+  /// Generates scheduled dose rows for [med] for the next 30 days.
+  /// Idempotent: existing doses for a time slot are not duplicated.
+  Future<void> _generateUpcomingDoses(PetMedication med) async {
+    if (!med.isActive) return;
+    if (med.frequency == 'as_needed') return;
+
+    final now = DateTime.now();
+    final end = now.add(const Duration(days: 30));
+    final doses = <MedicationDose>[];
+
+    DateTime cursor = med.startDate.isAfter(now) ? med.startDate : now;
+    cursor = DateTime(cursor.year, cursor.month, cursor.day);
+
+    while (cursor.isBefore(end)) {
+      if (med.endDate != null && cursor.isAfter(med.endDate!)) break;
+
+      final timesForDay = _timesForFrequency(med);
+      for (final hour in timesForDay) {
+        final scheduled = cursor.add(Duration(hours: hour));
+        if (scheduled.isBefore(now.subtract(const Duration(hours: 1)))) continue;
+        doses.add(MedicationDose(
+          id: '',
+          medicationId: med.id,
+          petId: med.petId,
+          scheduledFor: scheduled,
+          skipped: false,
+        ));
+      }
+      cursor = _nextCursor(med.frequency, cursor);
+    }
+
+    if (doses.isEmpty) return;
+    try {
+      await _repo.generateDosesIdempotent(doses);
+      // Refresh today's doses so UI stays in sync.
+      if (!ref.mounted) return;
+      final today = await _repo.fetchTodayDoses(med.petId);
+      state = state.copyWith(todayDoses: today);
+    } catch (e) {
+      log('Dose generation failed: $e', name: 'HealthNotifier');
+    }
+  }
+
+  List<int> _timesForFrequency(PetMedication med) {
+    if (med.timesOfDay.isNotEmpty) {
+      return med.timesOfDay.map((t) {
+        switch (t) {
+          case 'morning': return 8;
+          case 'noon': return 12;
+          case 'evening': return 18;
+          case 'night': return 21;
+          default: return 8;
+        }
+      }).toList();
+    }
+    switch (med.frequency) {
+      case 'twice_daily': return [8, 20];
+      case 'three_times_daily': return [8, 14, 20];
+      default: return [8];
+    }
+  }
+
+  DateTime _nextCursor(String frequency, DateTime from) {
+    switch (frequency) {
+      case 'weekly': return from.add(const Duration(days: 7));
+      case 'monthly': return DateTime(from.year, from.month + 1, from.day);
+      default: return from.add(const Duration(days: 1)); // daily variants
+    }
+  }
 }
 
 // ─────────────────────────────────────────────────────────────────────────────

lib/controllers/health_controller.dart

@@ -0,0 +1,40 @@
+import 'package:flutter_riverpod/flutter_riverpod.dart';
+import '../models/knowledge_base_models.dart';
+import '../repositories/feature_repositories.dart';
+
+class KnowledgeBaseCategoryNotifier extends Notifier<String> {
+  @override
+  String build() => 'All Topics';
+  void set(String category) => state = category;
+}
+
+final knowledgeBaseCategoryProvider =
+    NotifierProvider<KnowledgeBaseCategoryNotifier, String>(() {
+  return KnowledgeBaseCategoryNotifier();
+});
+
+class KnowledgeBaseSearchQueryNotifier extends Notifier<String> {
+  @override
+  String build() => '';
+  void set(String query) => state = query;
+}
+
+final knowledgeBaseSearchQueryProvider =
+    NotifierProvider<KnowledgeBaseSearchQueryNotifier, String>(() {
+  return KnowledgeBaseSearchQueryNotifier();
+});
+
+final featuredArticlesProvider = FutureProvider<List<KnowledgeArticle>>((ref) async {
+  return ref.watch(knowledgeBaseRepositoryProvider).fetchFeaturedArticles();
+});
+
+final knowledgeBaseArticlesProvider = FutureProvider<List<KnowledgeArticle>>((ref) async {
+  final category = ref.watch(knowledgeBaseCategoryProvider);
+  final query = ref.watch(knowledgeBaseSearchQueryProvider);
+  return ref.watch(knowledgeBaseRepositoryProvider).fetchArticles(
+    category: category,
+    query: query,
+  );
+});
+
+final knowledgeBaseRepositoryProvider = Provider((ref) => knowledgeBaseRepository);

lib/controllers/knowledge_base_controller.dart

@@ -0,0 +1,42 @@
+import 'package:flutter_riverpod/flutter_riverpod.dart';
+import '../repositories/feature_repositories.dart';
+
+class PetBreedController extends Notifier<AsyncValue<BreedScan?>> {
+  @override
+  AsyncValue<BreedScan?> build() {
+    return const AsyncValue.data(null);
+  }
+
+  Future<void> identifyBreed(String imagePath) async {
+    state = const AsyncValue.loading();
+    try {
+      final repository = ref.read(breedIdentifierRepositoryProvider);
+      final result = await repository.identifyBreed(imagePath);
+      state = AsyncValue.data(result);
+      
+      // Save to history
+      await repository.saveScan(result);
+      
+      // Refresh history provider
+      ref.invalidate(breedScanHistoryProvider);
+    } catch (e, st) {
+      state = AsyncValue.error(e, st);
+    }
+  }
+
+  void reset() {
+    state = const AsyncValue.data(null);
+  }
+}
+
+final breedIdentifierRepositoryProvider = Provider((ref) => breedIdentifierRepository);
+
+final breedIdentifierControllerProvider =
+    NotifierProvider<PetBreedController, AsyncValue<BreedScan?>>(() {
+  return PetBreedController();
+});
+
+final breedScanHistoryProvider = FutureProvider<List<BreedScan>>((ref) async {
+  final repository = ref.watch(breedIdentifierRepositoryProvider);
+  return repository.fetchScanHistory();
+});