🐛 [BUG] - P2025 Prisma renvoie une 500 au lieu d'une 404 (filtre d'exception global manquant)

[KepoParis]

Description

Côté server-nestjs, aucun filtre d'exception global n'est enregistré (pas de @Catch, pas de useGlobalFilters, pas d'APP_FILTER). Par conséquent, lorsqu'une requête Prisma lève une PrismaClientKnownRequestError de code P2025 (« An operation failed because it depends on one or more records that were required but not found »), l'erreur n'est pas un HttpException et le filtre par défaut de NestJS renvoie une 500 Internal Server Error au lieu d'une 404 Not Found.

C'est notamment le cas des appels findUniqueOrThrow / findFirstOrThrow / update / delete sur un id inexistant. Exemple concret : DeploymentDatastoreService.getDeploymentById utilise findUniqueOrThrow, donc PUT d'un déploiement avec un deploymentId inconnu retourne actuellement 500 au lieu de 404.

Le reste du code contourne le problème au cas par cas (findUnique + if (!x) throw new NotFoundException(...) dans project, environment, project-loader, etc.), ce qui est incohérent et facile à oublier.

Proposition : ajouter un filtre d'exception global (@Catch(PrismaClientKnownRequestError), enregistré via APP_FILTER) qui mappe une fois pour toute l'application :

• P2025 → 404 Not Found
• (à considérer : P2002 conflit d'unicité → 409 Conflict, P2003 violation de clé étrangère → 400/409)

Cela permettra ensuite de supprimer les gardes if (!x) throw new NotFoundException redondantes et de laisser les *OrThrow remonter naturellement le bon code HTTP.

Etapes de reproduction

1. Se placer sur un projet existant
2. Envoyer PUT /api/v1/projects/:projectId/deployments/:deploymentId avec un :deploymentId inexistant (mais un UUID valide)
3. Observer la réponse HTTP
4. La réponse est 500 Internal Server Error, alors qu'on attend 404 Not Found

Logs

PrismaClientKnownRequestError:
Invalid `prisma.deployment.findUniqueOrThrow()` invocation
An operation failed because it depends on one or more records that were required but not found. No record was found for a query.
  code: 'P2025'

Version de la console impactée

(à compléter — présent sur la branche feat/deployment-value-sources-api / main selon avancement de la pile)

Définition du fini

• Le correctif est terminé
• Les tests liés à ce correctif ont été ajoutés
• La communication avec les autres équipes impliquées par ce correctif a été faite