🐛 [BUG] server-nestjs: zone.upsert/zone.delete listeners have no emitter (AppEventsService missing emitZoneEvent) — cutover blocker

[shikanime]

Contexte / Context

La migration apps/server → apps/server-nestjs mappe les syncs de plugins via des événements :

• apps/server (legacy) : fire hook.zone.upsert / hook.zone.delete directement dans business.ts sur les routes zone.
• apps/server-nestjs : émet via AppEventsService.emit*(...) → EventEmitter2 → consommateurs @OnEvent.

Problème / Problem

VaultService écoute sur @OnEvent('zone.upsert') et @OnEvent('zone.delete') (vault.service.ts:87, 101), mais :

1. AppEventsService n'expose AUCUNE méthode d'émission pour zone.* — ni emitZoneEvent, ni de type ZoneEventName. La seule API publique est emitProjectEvent / emitProjectMemberEvent / emitRepositoryEvent (app-events.service.ts:72-109).
2. Aucun service nestjs n'émet zone.upsert / zone.delete — le grep emitProjectEvent|emitProjectMemberEvent|emitRepositoryEvent|eventEmitter.emit ne trouve aucune référence zone (hors logs).
3. Le module zone n'existe pas encore dans server-nestjs — la route zone est toujours propriétaire de apps/server (legacy).

Impact

Ce n'est pas un bug LIVE aujourd'hui (la route zone reste sur legacy, qui fire les hooks directement). C'est un régression silencieuse au cutover : dès que la route zone migrera vers server-nestjs sans brancher un emitZoneEvent, les listeners @OnEvent('zone.*') de Vault deviendront morts — le mount/policy/approle Vault des zones ne seront plus provisionnés/suprimés.

Cf. migration-parity-checklist : « if the migrated service emits eventEmitter.emitAsync('<entity>.<verb>', ...) but NO @OnEvent('<entity>.<verb>') handler exists anywhere » — ici c'est l'inverse (listener sans émetteur), même classe de régression silencieuse.

Preuves / Evidence

• Baseline unitaire : 578 passé / 59 skipped (2026-08-25, Node 24).
• Seed unitaire validé : app-events.service.spec.ts ajoute une assertion de parité zone.* (listener sans émetteur) — 12/12 vert.
• vault.service.spec.ts : 4 specs nouveaux scellent la surface externe upsertZone/deleteZone (mount + policy + approle + tech-readonly) — 8/8 vert.
• E2E vault.e2e-spec.ts : case zone ajoutée (gated E2E=1), émet zone.upsert/zone.delete directement via EventEmitter2 pour couvrir le contrat externe Vault (runnable sur host connecté / CI).

Definition du fini / Done

• (cutover-blocker) Ajouter un module zone dans server-nestjs avec un émetteur zone.upsert/zone.delete via AppEventsService — OU garder la route sur legacy jusqu'à ce que l'émetteur existe.
• Le seed unitaire + e2e ci-dessus restentverts (ils valident le contrat).