Address code review feedback

- Remove dotenv import attempt (not needed, DATABASE_URL set via env)
- Remove @ts-ignore directive
- Replace dangerous 'prisma migrate reset' with safer 'prisma migrate resolve' in rollback docs
- Verified Prisma generation still works without dotenv import

Co-authored-by: johndoe6345789 <224850594+johndoe6345789@users.noreply.github.com>

.github/workflows/gated-deployment.yml

@@ -580,9 +580,10 @@ jobs:
             
             1. **Re-run workflow** with previous stable commit SHA
             2. **OR use manual rollback:**
-               - Revert database migrations: \`npx prisma migrate reset\`
+               - Rollback specific migration: \`npx prisma migrate resolve --rolled-back <migration_name>\`
                - Deploy previous Docker image/build
-               - Restore from pre-deployment backup
+               - Restore from pre-deployment backup if needed
+               - ⚠️ Avoid \`prisma migrate reset\` in production (causes data loss)
             3. **Notify:** Update team and status page
             4. **Document:** Create post-mortem issue
             

frontends/nextjs/prisma.config.ts

@@ -4,14 +4,6 @@
  * This file replaces the deprecated package.json#prisma configuration.
  * See: https://www.prisma.io/docs/orm/reference/prisma-config-reference
  */
-// Try to load dotenv, but don't fail if it's not available (e.g., in CI)
-try {
-  // @ts-ignore - dotenv might not be available
-  await import('dotenv/config')
-} catch {
-  // Dotenv not available or failed - that's okay, DATABASE_URL might be set directly
-}
-
 import { defineConfig } from '@prisma/config'
 
 export default defineConfig({