Add Prisma middleware for automatic soft-delete filtering #10

New Issue

kyle · 2026-02-08T06:44:19-08:00

kyle commented

2026-02-08 06:44:19 -08:00

Context: where: { deletedAt: null } is manually added to 50+ Prisma queries across all CRUD services. This is error-prone and verbose.

Files affected: All service files (vips, drivers, vehicles, events, users, flights)

Suggested fix:

Add Prisma middleware in prisma.service.ts that automatically appends deletedAt: null to all findMany, findFirst, findUnique, count, and aggregate queries
Add an escape hatch like { includeDeleted: true } for queries that need to see soft-deleted records
This eliminates ~50 manual filter additions

Effort: Medium

**Context:** `where: { deletedAt: null }` is manually added to 50+ Prisma queries across all CRUD services. This is error-prone and verbose. **Files affected:** All service files (vips, drivers, vehicles, events, users, flights) **Suggested fix:** - Add Prisma middleware in `prisma.service.ts` that automatically appends `deletedAt: null` to all `findMany`, `findFirst`, `findUnique`, `count`, and `aggregate` queries - Add an escape hatch like `{ includeDeleted: true }` for queries that need to see soft-deleted records - This eliminates ~50 manual filter additions **Effort:** Medium

kyle referenced this issue

2026-02-08 06:44:30 -08:00

Code Efficiency #8

kyle referenced this issue from a commit

2026-02-08 07:34:25 -08:00

refactor: complete code efficiency pass (Issues #10, #14, #16)

kyle commented

2026-02-08 07:34:35 -08:00

Added Prisma middleware in prisma.service.ts that auto-appends deletedAt: null to all queries on soft-deletable models. Removed manual filters from 50+ queries across all services.

kyle closed this issue

2026-02-08 07:34:35 -08:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kyle/vip-coordinator#10