Émettre les événements projet avec l'identifiant du projet plutôt que l'objet complet

[shikanime]

Contexte

AppEventsService.emitProjectEvent accepte aujourd'hui string | ProjectWithDetails (apps/server-nestjs/src/modules/events/app-events.service.ts:72) : le service charge le projet puis émet l'objet complet. Tous les listeners @OnEvent('project.upsert' | 'project.delete') reçoivent donc un instantané partagé qu'ils n'ont pas sélectionné eux-mêmes.

Conséquences :

• Le projectSelect est mutualisé : un plugin ne peut pas restreindre ou étendre les champs dont il a besoin.
• Un appelant peut passer un objet pré-transaction (cas de l'archivage), ce qui rend l'état vu par les listeners implicite et dépendant du site d'appel.
• Le payload du log admin (data.args) embarque tout l'objet projet.

Le wrapper @OnEvent prend projectId: string, charge le projet. Les méthodes privées syncProject / cleanupProject gardent leur signature ProjectWithDetails :

@OnEvent('project.upsert')
async handleUpsert(projectId: string): Promise<RequiredPluginResult<'argocd'>> {
  return capturePluginResult('argocd', async () => {
    const project = await this.datastore.getProject(projectId)
    if (!project) return `Project not found (projectId=${projectId}), skipped`
    await this.syncProject(project)
  })
}

Modules concernés : argocd, gitlab, keycloak, nexus, observability, registry, sonarqube, vault.

Points à arbitrer

• Payload du log admin : data.args devient un UUID au lieu de l'objet projet (peut etre chiant a debug).

Références

• Émetteur : apps/server-nestjs/src/modules/events/app-events.service.ts
• Modèle de wrapper plugin : apps/server-nestjs/src/modules/plugin/plugin.utils.ts (capturePluginResult)
• Modèle de datastore : apps/server-nestjs/src/modules/registry/registry-datastore.service.ts

[shikanime]

La PR #2448 (correctif de #2445, fix: missing stage in observability) est une conséquence directe de ce design. Le plugin observability avait besoin de environment.stage.name ; faute de pouvoir choisir son propre select, le champ a été ajouté au projectSelect partagé :

   autosync: true,
   clusterId: true,
   stageId: true,
+  stage: {
+    select: {
+      name: true,
+    },
+  },

Le coût est donc payé par les 8 listeners et par toutes les requêtes projet, pour le besoin d'un seul plugin. Chaque nouveau besoin d'un plugin élargira le même select partagé.

Une fois les événements réduits à un identifiant, chaque module charge le projet avec le select dont il a besoin, et ce type de correctif reste local au plugin concerné.

[shikanime]

cc @cloud-pi-native/socle

[StephaneTrebel]

Oui ça me paraît pertinent d'alléger le contrat entre les briques et de ne faire transiter qu'un ID. Ça peut augmenter les requêtes sous-jacentes, mais je trouve ça bien mieux que d'avoir du state qui se balade…et qui pourrait être périmé ou incomplet !