Fix project warnings and errors

.codex/MENTOLOOP_PROMPT.md

@@ -0,0 +1,288 @@
+# MentoLoop — Nurse practitioner preceptor–student mentorship system with AI-powered matching — Codex Project Context
+
+This file provides Codex with project-wide context, standards, and patterns to ensure output aligns with MentoLoop’s architecture, compliance, and quality requirements.
+
+## Project Overview
+- Platform: Nurse practitioner preceptor–student mentorship system with AI-powered matching
+- URL: sandboxmentoloop.online
+- Repository: https://github.com/Apex-ai-net/MentoLoop
+- Version: 0.9.7
+- Primary Users: Nursing students, preceptors, healthcare institutions
+
+## Tech Stack
+
+### Frontend
+- Framework: Next.js 15 with App Router (Turbopack)
+- Styling: TailwindCSS v4, shadcn/ui components
+- UI Libraries: Radix UI, Framer Motion, Motion Primitives
+- Icons: Lucide React, Tabler Icons
+- Charts: Recharts for data visualization
+- State Management: React hooks with Convex real-time sync
+
+### Backend & Database
+- Database: Convex (real-time serverless)
+- Authentication: Clerk (with JWT templates)
+- File Storage: Convex file storage
+- API Pattern: Convex mutations/actions/queries
+
+### AI & Integrations
+- AI Providers: OpenAI GPT-4, Google Gemini Pro
+- Email: SendGrid (internal action pattern)
+- SMS: Twilio
+- Payments: Stripe (subscription-based)
+- Webhooks: Svix for validation
+
+## Project Structure
+```
+├── app/                    # Next.js App Router
+│   ├── (landing)/         # Public landing pages
+│   ├── dashboard/         # Protected dashboard routes
+│   ├── admin/             # Admin panel
+│   ├── student-intake/    # Student onboarding
+│   ├── preceptor-intake/  # Preceptor onboarding
+│   └── api/               # API routes
+├── convex/                # Backend functions
+│   ├── schema.ts          # Database schema
+│   ├── users.ts           # User management
+│   ├── matches.ts         # Matching logic
+│   ├── aiMatching.ts      # AI-enhanced matching
+│   ├── messages.ts        # HIPAA-compliant messaging
+│   ├── payments.ts        # Stripe integration
+│   ├── emails.ts          # SendGrid templates
+│   └── sms.ts             # Twilio notifications
+├── components/            # React components
+│   ├── ui/                # shadcn/ui components
+│   ├── dashboard/         # Dashboard components
+│   └── shared/            # Shared components
+├── lib/                   # Utilities
+└── hooks/                 # Custom React hooks
+```
+
+## Key Coding Patterns
+
+### Convex Database Operations
+```ts
+import { v } from "convex/values";
+import { mutation, query, action } from "./_generated/server";
+
+// Queries for read operations
+export const getStudents = query({
+  args: { schoolId: v.optional(v.id("schools")) },
+  handler: async (ctx, args) => {
+    const identity = await ctx.auth.getUserIdentity();
+    if (!identity) throw new Error("Unauthorized");
+    return await ctx.db.query("students").collect();
+  },
+});
+
+// Mutations for write operations
+export const updateStudent = mutation({
+  args: { studentId: v.id("students"), data: v.object({ /* ... */ }) },
+  handler: async (ctx, args) => {
+    // Validate and update
+  },
+});
+
+// Actions for external API calls
+export const sendEmail = action({
+  args: { /* SendGrid params */ },
+  handler: async (ctx, args) => {
+    // External API calls go in actions
+  },
+});
+```
+
+### Component Structure
+```ts
+interface ComponentProps {
+  data: Student;
+  onUpdate: (id: Id<"students">, data: Partial<Student>) => void;
+}
+
+export function StudentCard({ data, onUpdate }: ComponentProps) {
+  const students = useQuery(api.students.list);
+  if (students === undefined) return <Skeleton />;
+  if (students === null) return <ErrorState />;
+  return <Card className="p-4">{/* ... */}</Card>;
+}
+```
+
+## Environment Variables (required)
+- Convex: `CONVEX_DEPLOYMENT`, `NEXT_PUBLIC_CONVEX_URL`
+- Clerk: `NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY`, `CLERK_SECRET_KEY`, `CLERK_JWT_ISSUER_DOMAIN`, `CLERK_WEBHOOK_SECRET`
+- AI: `OPENAI_API_KEY`, `GEMINI_API_KEY`
+- Communications: `SENDGRID_API_KEY`, `SENDGRID_FROM_EMAIL`, `TWILIO_ACCOUNT_SID`, `TWILIO_AUTH_TOKEN`, `TWILIO_PHONE_NUMBER`
+- Stripe: `STRIPE_SECRET_KEY`, `STRIPE_WEBHOOK_SECRET`, `NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY`, `STRIPE_PRICE_ID_CORE`, `STRIPE_PRICE_ID_PRO`, `STRIPE_PRICE_ID_PREMIUM`
+- App Settings: `NEXT_PUBLIC_APP_URL`, `EMAIL_DOMAIN`, `NODE_ENV=production`
+- Feature Flags: `ENABLE_AI_MATCHING`, `ENABLE_EMAIL_NOTIFICATIONS`, `ENABLE_SMS_NOTIFICATIONS`, `ENABLE_PAYMENT_PROCESSING`
+
+## Healthcare Compliance Requirements
+
+### HIPAA Compliance
+- Encrypt all patient data in transit and at rest
+- Implement audit logging for all data access
+- Use secure messaging for all communications
+- Never log PHI in console or error messages
+
+### FERPA Compliance
+- Protect student educational records
+- Implement proper access controls
+- Maintain data retention policies
+
+## Development Guidelines
+
+### Code Quality Standards
+- TypeScript strict mode
+- Comprehensive error handling
+- JSDoc comments for complex functions
+- Follow ESLint and Prettier
+- Unit tests for critical functions
+
+### Git Workflow
+- Feature branches: `feature/description`
+- Bug fixes: `fix/description`
+- Commit format: `type(scope): message`
+- Run tests before pushing
+
+### Performance Optimization
+- Use `React.memo` for expensive components
+- Virtual scrolling for large lists
+- Optimize images via `next/image`
+- Dynamic imports for code splitting
+- Cache Convex queries when appropriate
+
+### Security Best Practices
+- Validate all user inputs
+- Use parameterized queries
+- Implement rate limiting
+- Sanitize data before display
+- Use environment variables for secrets
+- Never expose sensitive data in client code
+
+## Common Tasks & Solutions
+- Add feature: define schema in `convex/schema.ts`, create functions in `convex/`, build UI in `components/`, add routes in `app/`, wire real-time via Convex hooks, add tests/docs.
+- Debug Convex: check dashboard logs, verify env vars, ensure auth flow, review schema migrations.
+- Optimize AI matching: use streamlined algorithm in `aiMatching.ts`, cache responses, implement fallbacks, monitor usage/costs.
+
+## Testing Requirements
+- Unit tests with Vitest for utilities
+- Integration tests for API endpoints
+- E2E tests with Playwright for critical flows
+- Component tests for complex UI logic
+
+## Deployment Notes
+- Primary: Netlify (automatic deployments)
+- Alternative: Vercel
+- CI/CD: GitHub Actions
+- Environments: development, staging, production
+
+## Important Patterns
+- Convex: use actions for external APIs, mutations for writes, queries for reads
+- Real-time: prefer `useQuery`, implement optimistic updates, handle connection states
+- Compliance: audit all data access, encrypt sensitive data, enforce access controls
+
+## Cursor Rules
+A stricter ruleset exists in `.cursor/rules/mentoloop.mdc` and is always applied.
+
+
+### Dashboard Stabilization Execution Plan (2025-09-18)
+
+Prepared for Codex hand-off; keep this section current as phases close.
+
+Goal
+- Stabilize dashboard features, fix dead buttons/flows, squash bugs, and get lint/types/tests clean so Codex can implement confidently.
+
+Scope snapshot
+- Dashboard routes under `app/dashboard/*` (student, admin, preceptor) have partially wired actions.
+- Payments/discounts verification: `NP12345` = 100%, `MENTO12345` = 99.9%.
+- Idempotency and checkout robustness need recheck after latest edits.
+- Lint/type/test hygiene before hand-off.
+
+Phase 0 — Baseline health (same-day)
+- Build, type-check, lint, unit + e2e smoke:
+  - `npm run type-check`, `npm run lint`, `npm run test:unit:run`, `npm run test:e2e`
+- MCP checks:
+  - Stripe: list coupons/prices, recent intents; verify `NP12345` (100%), `MENTO12345` (99.9%).
+  - Netlify: last deploy status and logs.
+  - Sentry: recent errors for dashboard routes.
+- Output: short report with failures, stack traces, and owners.
+
+Phase 1 — Inventory dead features (Day 1)
+- Crawl dashboard UI and log “no-op” UI:
+  - Buttons/menus that don’t navigate, dispatch, or call Convex.
+  - Modals/forms missing submit handlers or success toasts.
+- Prioritize by user impact (Blockers → Core UX → Nice-to-have).
+- Output: checklist per route: `action → expected → current → fix candidate`.
+
+Phase 2 — Payments and intake gating (Day 1–2)
+- Ensure discount behavior:
+  - `NP12345`: zero-total end-to-end (no charge), immediate access.
+  - `MENTO12345`: 99.9% off applied via promotion code; UI shows correct final total.
+- Verify idempotency:
+  - Customer create/update keys hash-based and vary with params; no `idempotency_error`.
+- Confirm webhook/audit:
+  - `payments`, `paymentAttempts`, `paymentsAudit` updated; receipt URL stored.
+
+Phase 3 — Wire dead features (Day 2–3)
+- Student dashboard: Messages (send, mark read), Billing (open portal, history).
+- Admin dashboard: Finance filters/CSV, matches actions, discount setup status.
+- Preceptor dashboard: Matches list/actions and navigation from notifications.
+- Each wire-up: optimistic UI, success/error toast, and e2e happy-path.
+
+Phase 4 — Bug fixes + polish (Day 3–4)
+- Fix navigation loops and guards (RoleGuard/intake step protection).
+- Loading/error states, skeletons, a11y labels on forms/buttons.
+- Stabilize flaky tests with data-testids and explicit waits.
+
+Phase 5 — Hygiene (Day 4)
+- Lint/types clean; remove unused/any; minimal typings where needed.
+- Unit coverage for critical UI logic (messages, payments summary).
+- E2E coverage:
+  - Intake basic → checkout with `NP12345` and `MENTO12345`
+  - Dashboard message send/read
+  - Admin discount init (NP + MENTO) smoke
+
+Phase 6 — Observability + compliance (Day 4–5)
+- Sentry: add breadcrumbs around checkout, intake transitions, and dashboard actions.
+- Logs: ensure no PHI/PII; audit trails for payments/matches.
+- Netlify headers/security: verify security headers applied site-wide.
+
+Handover to Codex
+- Open a single tracking issue with:
+  - The prioritized checklist (Phase 1 output).
+  - Repro steps for each bug (short).
+  - Acceptance criteria per feature.
+  - Commands to run locally and in CI.
+
+MCP-assisted tasks to automate during execution
+- Stripe: verify coupons, prices, intents, and fetch receipt URLs by email/session_id.
+- Netlify: poll last deploy status and recent logs on push.
+- Sentry: list recent issues filtered by dashboard paths.
+- GitHub: list/open issues for each item in the checklist.
+
+Definition of done
+- Dashboard actions wired with feedback and no console errors.
+- `NP12345` 100% and `MENTO12345` 99.9% pass e2e; idempotency errors eliminated.
+- `npm run type-check` and `npm run lint` clean; tests green in CI.
+- Sentry quiet for common flows; Netlify deploy green.
+
+### 2025-09-19 Update — Dark Mode + Phase 0 Results
+
+- Dark mode only: enforced via `html.dark`, dark palette in `app/globals.css`, Tailwind `darkMode: 'class'`, Sonner `theme="dark"`, charts theme mapping extended, Clerk UI set to dark.
+- Baseline checks:
+  - Type-check: clean
+  - Lint: clean
+  - Unit tests: 80 passed, 3 skipped
+  - E2E live smoke: passed via external Playwright config against `https://sandboxmentoloop.online` (artifacts in `tmp/browser-inspect/`)
+- Stripe MCP verification:
+  - Coupons: `NP12345` 100% (once), `MENTO12345` 99.9% (forever)
+  - Prices include $0.01 test price; recent PaymentIntents list empty (OK for idle)
+- Notes:
+  - Local e2e using dev server requires `NEXT_PUBLIC_CONVEX_URL`; live-run external config avoids env deps.
+  - Sentry SDK warns about `onRequestError` hook in instrumentation; add `Sentry.captureRequestError` in a follow-up.
+
+Next steps
+- Phase 1 inventory of dead features in dashboard routes and prioritize (Blockers → Core UX → Nice-to-have).
+- Phase 2 payments gating/idempotency recheck, webhook audit receipts.
+- Add Sentry breadcrumbs across checkout/intake/dashboard actions.
+- Netlify deploy/logs monitor on push.

.codex/PLAN.md

@@ -0,0 +1,47 @@
+Phased launch plan (live tracking)
+
+Phase 1 – Intake + Payments polish (start now)
+- Student intake step 1: hide full name/email/phone; prefill from Clerk; keep DOB only
+- Student intake step 2: ensure NP track includes “Other”; require university/school fields explicitly
+- Membership/pricing: verify 60h block and 30h a la carte are present (done); add one‑cent discount path (choose: test price or amount_off coupon)
+- UI: make dashboard navbar solid white; replace landing footer in dashboard with a simple footer; hide “More” menus where unused
+
+Phase 1a – Discount system GA (in flight)
+- Owner (Payments Eng): Run `api.payments.syncDiscountCouponsToStudentProducts` in Stripe test/live; capture recreated coupon IDs for finance log. Target: week 1.
+- Owner (Data Eng): Backfill existing `discountUsage` rows with new `stripePriceId`/`membershipPlan` columns; verify analytics dashboards still reconcile. Target: week 1.
+- Owner (QA): Build automated coverage (unit+integration) for discount validation helpers and intake logging; wire into CI. Target: week 2.
+- Owner (QA): Complete end-to-end QA: Core/Pro/Premium checkout (full + installments), NP12345 zero-cost flow, admin finance views. Target: week 2.
+- Owner (Ops): Document ops rollback + verification steps in Stripe Ops runbook; ensure permissions allow metadata spot-check. Target: week 1.
+- Dependencies: confirm STRIPE_SECRET_KEY + price IDs available in both test/live; ensure Convex access for backfill mutation.
+- QA exit criteria: automated tests green + manual matrix signed off + finance dashboard sanity check screenshot archived.
+
+Phase 2 – Messaging, Documents, Hours policy
+- Enforce preceptor‑first messaging in convex/messages.sendMessage
+- Wire documents UI (upload/view/delete) to convex/documents.ts and remove dead buttons
+- Model hour credits with issuedAt/expiresAt (1 year) and enforce no rollover for a la carte; surface expiration in student dashboard
+- Pre-work: audit existing messaging/document handlers to list missing validations, storage calls, and UI gaps.
+
+Phase 3 – Preceptor onboarding + payouts
+- Preceptor intake: add licenseNumber, telehealth willingness toggle; display verified/unverified on dashboard
+- Stripe Connect for preceptor payouts (connected account onboarding, destination charges/transfers); add payout summary UI
+- Loop Exchange: add privacy opt‑out in intake/profile; respect in directory
+
+Phase 4 – Enterprise refinements
+- Remove test analytics; enforce student capacity (e.g., 20 concurrent)
+- Add Calendly CTA on enterprise landing/dashboard
+
+Stripe envs
+- Netlify: NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY (set), OPENAI_API_KEY (set)
+- Convex: STRIPE_SECRET_KEY, STRIPE_WEBHOOK_SECRET, STRIPE_PRICE_ID_* (set in Convex dashboard)
+
+Toward v1.0 (new)
+- Harden Stripe webhooks: leverage enriched discount metadata, emit structured logs/alerts when coupons lack `audience=student` or sync fails.
+- Expand audit logging: record who runs discount sync, expose finance export summarizing discount usage by product.
+- Final regression matrix for 1.0: student/preceptor intake, payments, admin RBAC, matching basics; capture sign-off before release cut.
+- Prep release operations: draft 1.0 notes, update ROADMAP milestones, tag branch after CI (lint, type-check, tests, E2E) is green.
+- Release owner: assemble regression matrix + schedule; due week prior to 1.0 cut.
+- Product/Comms: draft release notes + changelog once Phase 1a + Phase 2 complete.
+
+Build stability
+- Short‑term: if needed, deploy with `npx convex deploy --typecheck=disable && next build`
+- Long‑term: refactor any Convex actions using ctx.db directly into internalQuery/internalMutation wrappers