fix: 1101 correctifs etape 3 (formattage adresse)

[olivier-rabot]

Ticket(s) lié(s)

https://jira-mcas.atlassian.net/jira/software/c/projects/VAO/boards/336?selectedIssue=VAO-1194

Description

Correction du problème lié à l'ajout d'un séjour sur l'etape 3. Ajout d'une fonction de formattage de l'adresse.
Activation conditionnelle de certains champs (type deficiences, periode) > non required quand l'agrement est en statut brouillon

Screenshot / liens loom

Check-list

• Ma branche est rebase sur main
• Des tests ont été écrits pour tous les endpoints créés ou modifiés
• Refacto "à la volée" des parties sur lesquelles j'ai codée
• Plus de console.log
• J'ai ajouté une validation de schéma sur la route que j'ai ajouté ou modifié
• J'ai converti les fichiers vue en <script lang="ts">
• Mon code est en Typescript (autant que possible)

Testing instructions

[revu-bot]

PR Review: fix/1101 — Address formatting & step 3 corrections

This PR extracts a normalizeAdresse utility into shared-bridge, fixes a bug where allResults was populated even on failed validations, and adds TypeScript types to several Vue components. The overall direction is good, but there are a few issues worth addressing.

File	Lines	Severity	Issue
shared-bridge/src/utils/normalizeAdresse.ts	3	CRITICAL	Null-input type mismatch — function signature says AdresseDto but passes null/undefined through
shared-bridge/src/utils/normalizeAdresse.ts	13–17	IMPORTANT	Redundant null-checks inside already-guarded branch
frontend-usagers/…/listeSejours.vue	175–180	IMPORTANT	values cast to Record<string, any> defeats the purpose of the typed useForm<SejourFormValues>
frontend-usagers/…/typeDeficiences.vue	51–64	IMPORTANT	validationObligatoire prop declared but never used — dead code
frontend-usagers/…/tranchesAge.vue	49–62	IMPORTANT	Same requiredUnlessBrouillon wrapper is kept for non-BROUILLON statuses, making the BROUILLON branch redundant with the existing helper

[sonarqubecloud]

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[tokenbureau]

🎉 Deployment for commit 1c47fe8 :

Ingresses

• 🚀 https://api-vao-fix-1101-bilan-annees-etape-trois-jcdvcik6.ovh.fabrique.social.gouv.fr/
• 🚀 https://bo-vao-fix-1101-bilan-annees-etape-trois-jcdvcik6.ovh.fabrique.social.gouv.fr/
• 🚀 https://cron-vao-fix-1101-bilan-annees-etape-trois-jcdvcik6.ovh.fabrique.social.gouv.fr/
• 🚀 https://external-api-vao-fix-1101-bilan-annees-etape-trois-jcdvcik6.ovh.fabrique.social.gouv.fr/
• 🚀 https://maildev-vao-fix-1101-bilan-annees-etape-trois-jcdvcik6.ovh.fabrique.social.gouv.fr/
• 🚀 https://migrate-s3-vao-fix-1101-bilan-annees-etape-trois-jcdvcik6.ovh.fabrique.social.gouv.fr/
• 🚀 https://vao-fix-1101-bilan-annees-etape-trois-jcdvcik6.ovh.fabrique.social.gouv.fr/

Docker images

• 📦 docker pull harbor.fabrique.social.gouv.fr/vao/vao/backend:sha-1c47fe8d3ddba47e1eff81978ad7bc90e7ea157a
• 📦 docker pull harbor.fabrique.social.gouv.fr/vao/vao/cron:sha-1c47fe8d3ddba47e1eff81978ad7bc90e7ea157a
• 📦 docker pull harbor.fabrique.social.gouv.fr/vao/vao/external-api:sha-1c47fe8d3ddba47e1eff81978ad7bc90e7ea157a
• 📦 docker pull harbor.fabrique.social.gouv.fr/vao/vao/frontend-bo:sha-1c47fe8d3ddba47e1eff81978ad7bc90e7ea157a
• 📦 docker pull harbor.fabrique.social.gouv.fr/vao/vao/frontend-usagers:sha-1c47fe8d3ddba47e1eff81978ad7bc90e7ea157a
• 📦 docker pull harbor.fabrique.social.gouv.fr/vao/vao/migrations:sha-1c47fe8d3ddba47e1eff81978ad7bc90e7ea157a
• 📦 docker pull maildev/maildev:2.1.0

Debug

• 📕 Loki logs for namespace vao-fix-1101-bilan-annees-etape-trois-jcdvcik6
• 📈 Pods monitoring for namespace vao-fix-1101-bilan-annees-etape-trois-jcdvcik6
• 📈 Workloads monitoring for namespace vao-fix-1101-bilan-annees-etape-trois-jcdvcik6
• 🐘 CNPG pg

[l-scherer]

RAS globalement
Par contre il te faut ajouter des tests sur normalizeAdresse.ts
Créer au moins un normalizeAdresse.spec.ts au même niveau

[l-scherer]

adresse.codeInsee,
adresse.codePostal,
adresse.departement,

sont redondants par rapport à ...adresse,
Tu peux te contenter de retourner ça non ?

  return {
    ...adresse,
    coordinates: coords ?? null,
    lat,
    long,
  };