chore: bump version to v0.1.135

config/version.yaml

@@ -1,2 +1,2 @@
 parameters:
-    app.version: '0.1.134'
+    app.version: '0.1.135'

frontend/i18n/locales/fr.json

@@ -528,7 +528,8 @@
                 "exportError": "L'export du répertoire transporteurs a échoué. Réessayez.",
                 "createSuccess": "Transporteur créé avec succès",
                 "integrateSuccess": "Transporteur QUALIMAT intégré",
-                "addressSaved": "Adresse enregistrée"
+                "addressSaved": "Adresse enregistrée",
+                "contactSaved": "Contact enregistré"
             },
             "containerType": {
                 "BENNE": "Benne",
@@ -591,6 +592,17 @@
                     "remove": "Supprimer l'adresse",
                     "degraded": "Service d'adresse indisponible : saisie de la ville et de l'adresse en mode libre."
                 },
+                "contact": {
+                    "lastName": "Nom",
+                    "firstName": "Prénom",
+                    "jobTitle": "Fonction",
+                    "phonePrimary": "Téléphone",
+                    "phoneSecondary": "Téléphone (2)",
+                    "addPhone": "Ajouter un numéro",
+                    "email": "Email",
+                    "add": "Nouveau contact",
+                    "remove": "Supprimer le contact"
+                },
                 "confirmDelete": {
                     "title": "Supprimer ce bloc",
                     "message": "Cette suppression est définitive. Confirmer ?",

frontend/modules/transport/components/CarrierContactBlock.vue

@@ -0,0 +1,108 @@
+<template>
+    <div class="relative grid grid-cols-4 gap-x-[44px] gap-y-4 bg-white py-4 pl-[28px] pr-[60px] shadow-[0_4px_4px_0_rgba(0,0,0,0.25)]">
+        <!-- Suppression : ouvre une modal de confirmation côté parent. Masquée si
+             non supprimable (1er bloc) ou en lecture seule. -->
+        <MalioButtonIcon
+            v-if="removable && !readonly"
+            icon="mdi:delete-outline"
+            variant="ghost"
+            button-class="absolute top-3 right-3"
+            v-bind="{ ariaLabel: t('transport.carriers.form.contact.remove') }"
+            @click="$emit('remove')"
+        />
+
+        <MalioInputText
+            :model-value="model.lastName"
+            :label="t('transport.carriers.form.contact.lastName')"
+            :readonly="readonly"
+            :error="errors?.lastName"
+            @update:model-value="(v: string) => update('lastName', v)"
+        />
+        <MalioInputText
+            :model-value="model.firstName"
+            :label="t('transport.carriers.form.contact.firstName')"
+            :readonly="readonly"
+            :error="errors?.firstName"
+            @update:model-value="(v: string) => update('firstName', v)"
+        />
+        <!-- Fonction sur 2 colonnes : on wrappe car MalioInputText (inheritAttrs:false)
+             renvoie `class` sur l'input interne, pas sur la cellule de grille. -->
+        <div class="col-span-2">
+            <MalioInputText
+                :model-value="model.jobTitle"
+                :label="t('transport.carriers.form.contact.jobTitle')"
+                :readonly="readonly"
+                :error="errors?.jobTitle"
+                @update:model-value="(v: string) => update('jobTitle', v)"
+            />
+        </div>
+        <MalioInputEmail
+            :model-value="model.email"
+            :label="t('transport.carriers.form.contact.email')"
+            :readonly="readonly"
+            :lowercase="true"
+            :error="errors?.email"
+            @update:model-value="(v: string) => update('email', v)"
+        />
+        <!-- Téléphone principal + bouton « + » révélant le 2e numéro (max 2). -->
+        <MalioInputPhone
+            :model-value="model.phonePrimary"
+            :label="t('transport.carriers.form.contact.phonePrimary')"
+            :mask="PHONE_MASK"
+            :readonly="readonly"
+            :error="errors?.phonePrimary"
+            :addable="!model.hasSecondaryPhone && !readonly"
+            :add-button-label="t('transport.carriers.form.contact.addPhone')"
+            @update:model-value="(v: string) => update('phonePrimary', v)"
+            @add="revealSecondaryPhone"
+        />
+        <!-- 2e numéro : révélé à la demande (max 2 téléphones — RG-4.08). -->
+        <MalioInputPhone
+            v-if="model.hasSecondaryPhone"
+            :model-value="model.phoneSecondary"
+            :label="t('transport.carriers.form.contact.phoneSecondary')"
+            :mask="PHONE_MASK"
+            :readonly="readonly"
+            :error="errors?.phoneSecondary"
+            @update:model-value="(v: string) => update('phoneSecondary', v)"
+        />
+    </div>
+</template>
+
+<script setup lang="ts">
+import type { CarrierContactFormDraft } from '~/modules/transport/types/carrierForm'
+
+// Masque téléphone FR : 5 groupes de 2 chiffres (la normalisation finale reste serveur).
+const PHONE_MASK = '## ## ## ## ##'
+
+const props = defineProps<{
+    /** Brouillon du contact (v-model). */
+    modelValue: CarrierContactFormDraft
+    /** Affiche l'icône de suppression (1er bloc non supprimable). */
+    removable?: boolean
+    /** Bloc en lecture seule (onglet validé). */
+    readonly?: boolean
+    /** Erreurs serveur 422 de cette ligne, indexées par champ (ERP-101). */
+    errors?: Record<string, string>
+}>()
+
+const emit = defineEmits<{
+    'update:modelValue': [value: CarrierContactFormDraft]
+    'remove': []
+}>()
+
+const { t } = useI18n()
+
+// Alias local pour la lisibilité du template.
+const model = computed(() => props.modelValue)
+
+/** Émet un nouveau brouillon avec le champ modifié (immutabilité). */
+function update<K extends keyof CarrierContactFormDraft>(field: K, value: CarrierContactFormDraft[K]): void {
+    emit('update:modelValue', { ...props.modelValue, [field]: value })
+}
+
+/** Révèle le 2e numéro (max 1 secondaire, le « + » disparaît). */
+function revealSecondaryPhone(): void {
+    emit('update:modelValue', { ...props.modelValue, hasSecondaryPhone: true })
+}
+</script>

frontend/modules/transport/composables/__tests__/useCarrierForm.test.ts

@@ -37,6 +37,8 @@ vi.stubGlobal('useToast', () => ({
 }))
 
 const { useCarrierForm, CARRIER_TAB_KEYS } = await import('../useCarrierForm')
+const { buildCarrierContactPayload, isCarrierContactBlank, isCarrierContactNamed } = await import('../../utils/forms/carrierContact')
+const { emptyCarrierContact } = await import('../../types/carrierForm')
 
 describe('useCarrierForm', () => {
     beforeEach(() => {
@@ -555,3 +557,145 @@ describe('useCarrierForm — onglet Adresses (ERP-167)', () => {
         expect(form.addresses.value).toHaveLength(1)
     })
 })
+
+describe('carrierContact (util) — validité alignée M1/M2/M3 + max 2 téléphones', () => {
+    it('isCarrierContactBlank : vrai si vide, faux dès un champ comptant rempli (phoneSecondary exclu)', () => {
+        expect(isCarrierContactBlank(emptyCarrierContact())).toBe(true)
+        expect(isCarrierContactBlank({ ...emptyCarrierContact(), jobTitle: 'Acheteur' })).toBe(false)
+        expect(isCarrierContactBlank({ ...emptyCarrierContact(), phonePrimary: '0102030405' })).toBe(false)
+        // phoneSecondary seul ne compte pas (aligné M1/M2/M3).
+        expect(isCarrierContactBlank({ ...emptyCarrierContact(), phoneSecondary: '0605040302', hasSecondaryPhone: true })).toBe(true)
+    })
+
+    it('isCarrierContactNamed : nommé seulement avec un prénom OU un nom', () => {
+        expect(isCarrierContactNamed(emptyCarrierContact())).toBe(false)
+        expect(isCarrierContactNamed({ ...emptyCarrierContact(), firstName: 'Jean' })).toBe(true)
+        expect(isCarrierContactNamed({ ...emptyCarrierContact(), lastName: 'Doe' })).toBe(true)
+        // Fonction / téléphone / email seuls ne « nomment » pas (≠ RG-4.08 large).
+        expect(isCarrierContactNamed({ ...emptyCarrierContact(), jobTitle: 'Acheteur' })).toBe(false)
+        expect(isCarrierContactNamed({ ...emptyCarrierContact(), email: 'a@b.fr' })).toBe(false)
+    })
+
+    it('buildCarrierContactPayload : phones = 1 numéro sans secondaire', () => {
+        const body = buildCarrierContactPayload({ ...emptyCarrierContact(), phonePrimary: '0102030405' })
+        expect(body.phones).toEqual(['0102030405'])
+    })
+
+    it('buildCarrierContactPayload : phones = 2 numéros si secondaire révélé', () => {
+        const body = buildCarrierContactPayload({
+            ...emptyCarrierContact(),
+            phonePrimary: '0102030405',
+            phoneSecondary: '0605040302',
+            hasSecondaryPhone: true,
+        })
+        expect(body.phones).toEqual(['0102030405', '0605040302'])
+    })
+
+    it('buildCarrierContactPayload : 2e numéro ignoré tant que non révélé', () => {
+        const body = buildCarrierContactPayload({
+            ...emptyCarrierContact(),
+            phonePrimary: '0102030405',
+            phoneSecondary: '0605040302',
+            hasSecondaryPhone: false,
+        })
+        expect(body.phones).toEqual(['0102030405'])
+    })
+})
+
+describe('useCarrierForm — onglet Contacts (ERP-168)', () => {
+    beforeEach(() => {
+        mockPost.mockReset()
+        mockPatch.mockReset()
+        mockDelete.mockReset()
+    })
+
+    /** Transporteur créé, onglet Contacts accessible. */
+    function createdForm() {
+        const form = useCarrierForm()
+        form.carrierId.value = 7
+        return form
+    }
+
+    it('« + Nouveau contact » désactivé tant que le bloc n\'est pas nommé (prénom OU nom, aligné M1/M2/M3)', () => {
+        const form = createdForm()
+        expect(form.canAddContact.value).toBe(false)
+
+        // addContact est un no-op tant que le bloc n'est pas nommé.
+        form.addContact()
+        expect(form.contacts.value).toHaveLength(1)
+
+        // Fonction seule ne suffit PAS à ajouter un nouveau bloc (≠ RG-4.08 large).
+        const first = form.contacts.value[0]
+        if (first) first.jobTitle = 'Acheteur'
+        expect(form.canAddContact.value).toBe(false)
+        form.addContact()
+        expect(form.contacts.value).toHaveLength(1)
+
+        // Un nom (ou prénom) débloque l'ajout.
+        if (first) first.lastName = 'Doe'
+        expect(form.canAddContact.value).toBe(true)
+        form.addContact()
+        expect(form.contacts.value).toHaveLength(2)
+    })
+
+    it('submitContacts : POST des nouveaux contacts (phones tableau), capture id, finalise', async () => {
+        mockPost.mockResolvedValueOnce({ id: 55 })
+        const form = createdForm()
+        const c = form.contacts.value[0]
+        if (c) { c.firstName = 'Jean'; c.phonePrimary = '0102030405' }
+
+        const ok = await form.submitContacts(vi.fn())
+
+        expect(ok).toBe(true)
+        const [url, body, opts] = mockPost.mock.calls[0] ?? []
+        expect(url).toBe('/carriers/7/contacts')
+        expect(body).toMatchObject({ firstName: 'Jean', phones: ['0102030405'] })
+        expect(opts).toMatchObject({ toast: false, headers: { Accept: 'application/ld+json' } })
+        expect(form.contacts.value[0]?.id).toBe(55)
+        expect(form.isValidated('contacts')).toBe(true)
+    })
+
+    it('submitContacts : PATCH des contacts existants sur /carrier_contacts/{id}', async () => {
+        mockPatch.mockResolvedValueOnce({})
+        const form = createdForm()
+        const c = form.contacts.value[0]
+        if (c) { c.id = 55; c.lastName = 'Doe' }
+
+        await form.submitContacts(vi.fn())
+
+        expect(mockPost).not.toHaveBeenCalled()
+        expect(mockPatch).toHaveBeenCalledWith('/carrier_contacts/55', expect.objectContaining({ lastName: 'Doe' }), { toast: false })
+    })
+
+    it('RG-4.08 : onglet vide → soumet l\'amorce pour déclencher la 422 inline', async () => {
+        mockPost.mockRejectedValueOnce({
+            response: {
+                status: 422,
+                _data: { violations: [{ propertyPath: 'firstName', message: 'Au moins un champ du contact est obligatoire.' }] },
+            },
+        })
+        const form = createdForm()
+
+        const ok = await form.submitContacts(vi.fn())
+
+        expect(ok).toBe(false)
+        expect(mockPost).toHaveBeenCalledTimes(1)
+        expect(form.contactErrors.value[0]?.firstName).toBe('Au moins un champ du contact est obligatoire.')
+        expect(form.isValidated('contacts')).toBe(false)
+    })
+
+    it('removeContact : DELETE /carrier_contacts/{id} puis retrait du bloc', async () => {
+        mockDelete.mockResolvedValueOnce({})
+        const form = createdForm()
+        const c = form.contacts.value[0]
+        if (c) { c.id = 90; c.lastName = 'Doe' }
+        form.addContact()
+        const c2 = form.contacts.value[1]
+        if (c2) c2.firstName = 'Jean'
+
+        await form.removeContact(0)
+
+        expect(mockDelete).toHaveBeenCalledWith('/carrier_contacts/90', {}, { toast: false })
+        expect(form.contacts.value).toHaveLength(1)
+    })
+})

frontend/modules/transport/composables/useCarrierForm.ts

@@ -5,13 +5,16 @@ import { removeCollectionRow } from '~/shared/utils/collectionRow'
 import {
     emptyCarrierAddress,
     emptyCarrierAddressCopy,
+    emptyCarrierContact,
     emptyCarrierMain,
     type CarrierAddressCopy,
     type CarrierAddressFormDraft,
+    type CarrierContactFormDraft,
     type CarrierMainDraft,
     type CarrierMainResponse,
 } from '~/modules/transport/types/carrierForm'
 import { buildCarrierAddressPayload, isCarrierAddressValid } from '~/modules/transport/utils/forms/carrierAddress'
+import { buildCarrierContactPayload, isCarrierContactBlank, isCarrierContactNamed } from '~/modules/transport/utils/forms/carrierContact'
 import type { QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
 
 /** Nom du cas spécial « compte-propre » LIOT (comparaison insensible à la casse, RG-4.01). */
@@ -382,6 +385,86 @@ export function useCarrierForm() {
         }
     }
 
+    // ── Onglet Contacts (ERP-168) ─────────────────────────────────────────────
+    const contacts = ref<CarrierContactFormDraft[]>([emptyCarrierContact()])
+    // Erreurs 422 par ligne (alignées sur l'index du v-for), peuplées par submitRows.
+    const contactErrors = ref<Record<string, string>[]>([])
+
+    // « + Nouveau contact » désactivé tant que le DERNIER bloc n'est pas « nommé »
+    // (prénom OU nom) — aligné sur M1/M2/M3 (fonction / téléphone / email seuls ne
+    // suffisent pas à ajouter un nouveau bloc).
+    const canAddContact = computed(() => {
+        const last = contacts.value[contacts.value.length - 1]
+        return last !== undefined && isCarrierContactNamed(last)
+    })
+
+    function addContact(): void {
+        if (canAddContact.value) {
+            contacts.value.push(emptyCarrierContact())
+        }
+    }
+
+    /** Suppression immédiate d'un contact existant (DELETE /carrier_contacts/{id}). */
+    async function removeContact(index: number): Promise<void> {
+        await removeCollectionRow({
+            rows: contacts.value,
+            errors: contactErrors.value,
+            index,
+            endpoint: '/carrier_contacts',
+            deleteRow: url => api.delete(url, {}, { toast: false }),
+            makeEmpty: emptyCarrierContact,
+            onError: notifyRemovalError,
+        })
+    }
+
+    /**
+     * Valide l'onglet Contacts : POST des nouveaux contacts sur
+     * /carriers/{id}/contacts, PATCH des existants sur /carrier_contacts/{id}
+     * (groupe carrier:write:contacts). RG-4.08 (≥ 1 champ rempli, max 2 téléphones)
+     * re-validée back → 422 par ligne. Si l'onglet ne contient QUE des amorces
+     * vides, on soumet la 1re pour déclencher la 422 RG-4.08 inline plutôt que de
+     * finaliser un onglet vide. Retourne true si l'onglet a été validé.
+     */
+    async function submitContacts(onError: (error: unknown) => void): Promise<boolean> {
+        if (carrierId.value === null || tabSubmitting.value) {
+            return false
+        }
+        tabSubmitting.value = true
+        try {
+            const hasSubmittable = contacts.value.some(c => c.id !== null || !isCarrierContactBlank(c))
+            const hasError = await submitRows(
+                contacts.value,
+                contactErrors,
+                async (contact) => {
+                    const body = buildCarrierContactPayload(contact)
+                    if (contact.id === null) {
+                        const created = await api.post<{ id: number }>(
+                            `/carriers/${carrierId.value}/contacts`,
+                            body,
+                            { headers: { Accept: 'application/ld+json' }, toast: false },
+                        )
+                        contact.id = created.id
+                    }
+                    else {
+                        await api.patch(`/carrier_contacts/${contact.id}`, body, { toast: false })
+                    }
+                },
+                onError,
+                // Amorce vide neuve ignorée s'il reste un autre bloc soumettable ;
+                // sinon on la soumet pour déclencher la 422 RG-4.08 (sur firstName).
+                contact => hasSubmittable && contact.id === null && isCarrierContactBlank(contact),
+            )
+            if (hasError) {
+                return false
+            }
+            completeTab('contacts')
+            return true
+        }
+        finally {
+            tabSubmitting.value = false
+        }
+    }
+
     /**
      * Intègre une ligne QUALIMAT sélectionnée dans l'onglet Qualimat (RG-4.01 /
      * § 2.5) : copie le nom, force la certification à « QUALIMAT » (lecture seule),
@@ -481,6 +564,13 @@ export function useCarrierForm() {
         addAddress,
         removeAddress,
         submitAddresses,
+        // contacts
+        contacts,
+        contactErrors,
+        canAddContact,
+        addContact,
+        removeContact,
+        submitContacts,
         // actions
         validateMainFront,
         buildMainPayload,

frontend/modules/transport/pages/carriers/new.vue

@@ -207,7 +207,40 @@
                 </div>
             </template>
 
-            <!-- Contacts / Prix : contenu aux tickets suivants. -->
+            <!-- Onglet Contacts (ERP-168) : un bloc par contact (RG-4.08 ≥ 1 champ,
+                 max 2 téléphones). Erreurs 422 par ligne. -->
+            <template #contacts>
+                <div class="mt-12 flex flex-col gap-6">
+                    <CarrierContactBlock
+                        v-for="(contact, index) in contacts"
+                        :key="index"
+                        :model-value="contact"
+                        :removable="isRowRemovable(contacts, index)"
+                        :readonly="isValidated('contacts')"
+                        :errors="contactErrors[index]"
+                        @update:model-value="(v) => contacts[index] = v"
+                        @remove="askRemoveContact(index)"
+                    />
+                    <div v-if="!isValidated('contacts')" class="flex justify-center gap-6">
+                        <MalioButton
+                            variant="secondary"
+                            icon-name="mdi:add-bold"
+                            icon-position="left"
+                            :label="t('transport.carriers.form.contact.add')"
+                            :disabled="!canAddContact"
+                            @click="addContact"
+                        />
+                        <MalioButton
+                            variant="primary"
+                            :label="t('transport.carriers.form.submit')"
+                            :disabled="tabSubmitting || carrierId === null"
+                            @click="onSubmitContacts"
+                        />
+                    </div>
+                </div>
+            </template>
+
+            <!-- Prix : contenu au ticket suivant. -->
             <template
                 v-for="key in placeholderTabs"
                 :key="key"
@@ -271,6 +304,7 @@ import { debounce } from '~/shared/utils/debounce'
 import { extractApiErrorMessage } from '~/shared/utils/api'
 import { isRowRemovable } from '~/shared/utils/collectionRow'
 import CarrierAddressBlock from '~/modules/transport/components/CarrierAddressBlock.vue'
+import CarrierContactBlock from '~/modules/transport/components/CarrierContactBlock.vue'
 import { useCarrierForm } from '~/modules/transport/composables/useCarrierForm'
 import { useQualimatSearch, type QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
 
@@ -315,6 +349,12 @@ const {
     addAddress,
     removeAddress,
     submitAddresses,
+    contacts,
+    contactErrors,
+    canAddContact,
+    addContact,
+    removeContact,
+    submitContacts,
     submitMain,
     applyQualimatSelection,
 } = useCarrierForm()
@@ -408,8 +448,10 @@ const tabs = computed(() => tabKeys.value.map((key, index) => ({
     disabled: index > unlockedIndex.value,
 })))
 
-// Onglets dont le contenu arrive aux tickets suivants (Contacts / Prix).
-const placeholderTabs = computed(() => tabKeys.value.filter(key => key !== 'qualimat' && key !== 'addresses'))
+// Onglets dont le contenu arrive aux tickets suivants (Prix).
+const placeholderTabs = computed(() => tabKeys.value.filter(
+    key => key !== 'qualimat' && key !== 'addresses' && key !== 'contacts',
+))
 
 // ── Onglet Adresses (ERP-167) ────────────────────────────────────────────────
 // Pays : France garantie en tete meme si /countries echoue (resilience), pour
@@ -469,7 +511,7 @@ async function onSubmitAddresses(): Promise<void> {
     }
 }
 
-// Modal de confirmation de suppression (bloc adresse).
+// Modal de confirmation de suppression (générique : bloc adresse OU contact).
 const deleteConfirm = reactive({ open: false, action: null as null | (() => void) })
 
 function askRemoveAddress(index: number): void {
@@ -477,6 +519,22 @@ function askRemoveAddress(index: number): void {
     deleteConfirm.open = true
 }
 
+/** Valide l'onglet Contacts (POST/PATCH par ligne ; avance gérée par le composable). */
+async function onSubmitContacts(): Promise<void> {
+    const ok = await submitContacts(error => toast.error({
+        title: t('transport.carriers.toast.error'),
+        message: apiErrorMessage(error),
+    }))
+    if (ok) {
+        toast.success({ title: t('transport.carriers.toast.contactSaved') })
+    }
+}
+
+function askRemoveContact(index: number): void {
+    deleteConfirm.action = () => { void removeContact(index) }
+    deleteConfirm.open = true
+}
+
 function runDeleteConfirm(): void {
     deleteConfirm.action?.()
     deleteConfirm.action = null

frontend/modules/transport/types/carrierForm.ts

@@ -94,6 +94,41 @@ export function emptyCarrierAddress(): CarrierAddressFormDraft {
     }
 }
 
+/**
+ * Brouillon d'un bloc Contact (onglet Contacts, ERP-168) — sous-ressource
+ * `CarrierContact` (groupe `carrier:write:contacts`). Les téléphones sont saisis
+ * en `phonePrimary` / `phoneSecondary` côté UI, puis envoyés au back sous forme du
+ * tableau `phones` (max 2 — RG-4.08). `hasSecondaryPhone` pilote l'affichage du 2e
+ * numéro (révélé via le bouton « + »). Pas d'`iri` : aucune relation M2M depuis
+ * l'adresse au M4 (≠ M3).
+ */
+export interface CarrierContactFormDraft {
+    /** Id serveur une fois le contact créé (null tant que non persisté). */
+    id: number | null
+    firstName: string | null
+    lastName: string | null
+    jobTitle: string | null
+    phonePrimary: string | null
+    phoneSecondary: string | null
+    email: string | null
+    /** UI : le 2e numéro a été révélé via le bouton « + » (max 2 téléphones). */
+    hasSecondaryPhone: boolean
+}
+
+/** Brouillon de contact vide (état initial d'un bloc Contact). */
+export function emptyCarrierContact(): CarrierContactFormDraft {
+    return {
+        id: null,
+        firstName: null,
+        lastName: null,
+        jobTitle: null,
+        phonePrimary: null,
+        phoneSecondary: null,
+        email: null,
+        hasSecondaryPhone: false,
+    }
+}
+
 /**
  * Réponse du POST / PATCH principal (groupe `carrier:read`). Le serveur renvoie
  * le nom normalisé (UPPERCASE, RG-4.13) que l'UI réaffiche tel quel.

frontend/modules/transport/utils/forms/carrierContact.ts

@@ -0,0 +1,61 @@
+/**
+ * Helpers purs de l'onglet Contact transporteur (M4 Transport, ERP-168) — ALIGNÉ
+ * sur `providerContact.ts` (M3) / les autres modules : mêmes règles de validité et
+ * de gating « + Nouveau contact » (un contact est « nommé » dès qu'il porte un
+ * prénom OU un nom). Seule spécificité M4 conservée : les téléphones partent au back
+ * dans le tableau virtuel `phones` (max 2), mappés par le CarrierContactProcessor.
+ * Testables sans Vue ni API.
+ */
+
+import type { CarrierContactFormDraft } from '~/modules/transport/types/carrierForm'
+
+/** Vrai si une chaîne porte au moins un caractère non-espace. */
+function isFilled(value: string | null | undefined): boolean {
+    return value !== null && value !== undefined && value.trim() !== ''
+}
+
+/**
+ * Un bloc Contact est VIDE tant qu'aucun champ comptant pour la validité n'est
+ * rempli — prénom / nom / fonction / téléphone principal / email. `phoneSecondary`
+ * est EXCLU (aligné M1/M2/M3 : un bloc ne portant qu'un 2e numéro reste vide). Sert
+ * le filtrage des amorces vides à la soumission.
+ */
+export function isCarrierContactBlank(contact: CarrierContactFormDraft): boolean {
+    return ![
+        contact.firstName,
+        contact.lastName,
+        contact.jobTitle,
+        contact.phonePrimary,
+        contact.email,
+    ].some(isFilled)
+}
+
+/**
+ * Un contact est « nommé » (valide) dès qu'il porte un prénom OU un nom — aligné
+ * sur M1/M2/M3. Pilote le gating « + Nouveau contact » : la fonction / le téléphone
+ * / l'email seuls ne suffisent pas pour ajouter un nouveau bloc.
+ */
+export function isCarrierContactNamed(contact: CarrierContactFormDraft): boolean {
+    return isFilled(contact.firstName) || isFilled(contact.lastName)
+}
+
+/**
+ * Payload de la sous-ressource contacts (groupe `carrier:write:contacts`). Les
+ * chaînes vides partent à null (le serveur normalise/trim). Les téléphones sont
+ * regroupés dans le tableau `phones` (numéros non vides, max 2 — RG-4.08) ; le 2e
+ * numéro n'est inclus que s'il a été révélé (`hasSecondaryPhone`).
+ */
+export function buildCarrierContactPayload(contact: CarrierContactFormDraft): Record<string, unknown> {
+    const phones = [
+        contact.phonePrimary,
+        contact.hasSecondaryPhone ? contact.phoneSecondary : null,
+    ].filter((phone): phone is string => isFilled(phone))
+
+    return {
+        firstName: contact.firstName || null,
+        lastName: contact.lastName || null,
+        jobTitle: contact.jobTitle || null,
+        email: contact.email || null,
+        phones,
+    }
+}

migrations/Version20260617120000.php

@@ -0,0 +1,49 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DoctrineMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * RG-4.08 (correctif) — aligne la regle de validite d'un contact transporteur sur
+ * le M1/M2/M3 : au moins le PRENOM OU le NOM (et non plus « un champ quelconque
+ * parmi prenom/nom/fonction/telephone/email »). Remplace le CHECK
+ * chk_carrier_contact_filled par chk_carrier_contact_name et met a jour les
+ * commentaires de colonnes. La garde applicative (CarrierContactProcessor::validateName)
+ * est alignee dans le meme commit ; le catalogue ColumnCommentsCatalog aussi.
+ *
+ * Placee au namespace racine DoctrineMigrations (et non en modulaire Transport) :
+ * elle ALTERE une table creee par une migration racine (Version20260615150000) ;
+ * le tri par version au sein du meme namespace garantit qu'elle joue APRES l'init
+ * (cf. CLAUDE.md regle 11 — le tri cross-namespace casserait l'ordre sur base vide).
+ */
+final class Version20260617120000 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'RG-4.08 : contact transporteur valide si prenom OU nom (alignement M1/M2/M3) — CHECK chk_carrier_contact_name.';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE carrier_contact DROP CONSTRAINT chk_carrier_contact_filled');
+        $this->addSql('ALTER TABLE carrier_contact ADD CONSTRAINT chk_carrier_contact_name CHECK (first_name IS NOT NULL OR last_name IS NOT NULL)');
+
+        $this->addSql('COMMENT ON TABLE carrier_contact IS $_$Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins le prenom OU le nom rempli (RG-4.08, chk_carrier_contact_name), max 2 telephones.$_$');
+        $this->addSql('COMMENT ON COLUMN carrier_contact.first_name IS $_$Prenom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).$_$');
+        $this->addSql('COMMENT ON COLUMN carrier_contact.last_name IS $_$Nom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).$_$');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE carrier_contact DROP CONSTRAINT chk_carrier_contact_name');
+        $this->addSql('ALTER TABLE carrier_contact ADD CONSTRAINT chk_carrier_contact_filled CHECK (first_name IS NOT NULL OR last_name IS NOT NULL OR job_title IS NOT NULL OR phone_primary IS NOT NULL OR email IS NOT NULL)');
+
+        $this->addSql('COMMENT ON TABLE carrier_contact IS $_$Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins un champ rempli (RG-4.08, chk_carrier_contact_filled), max 2 telephones.$_$');
+        $this->addSql('COMMENT ON COLUMN carrier_contact.first_name IS $_$Prenom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).$_$');
+        $this->addSql('COMMENT ON COLUMN carrier_contact.last_name IS $_$Nom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).$_$');
+    }
+}

src/Module/Transport/Domain/Entity/CarrierContact.php

@@ -21,8 +21,8 @@ use Symfony\Component\Validator\Constraints as Assert;
 
 /**
  * Contact d'un transporteur (1:n) — onglet Contact (M4). Jumeau de
- * SupplierContact (M2) : au moins un champ rempli (RG-4.08, garanti par le
- * CHECK chk_carrier_contact_filled + le Processor), max 2 telephones.
+ * SupplierContact (M2) : au moins le prenom OU le nom (RG-4.08, garanti par le
+ * CHECK chk_carrier_contact_name + le Processor), max 2 telephones.
  *
  * Lecture : proprietes en `carrier:item:read` (embarquees au detail du
  * transporteur). Ecriture : groupe `carrier:write:contacts`.

src/Module/Transport/Infrastructure/ApiPlatform/State/Processor/CarrierContactProcessor.php

@@ -23,21 +23,21 @@ use function is_string;
 /**
  * Processor d'ecriture de la sous-ressource Contact d'un transporteur (M4,
  * spec-back § 4.5). Jumeau du SupplierContactProcessor (M2), recentre sur le
- * perimetre ERP-160, AVEC deux specificites M4 : RG-4.08 (≥ 1 champ rempli, max
- * 2 telephones) portee a la fois par le CHECK BDD chk_carrier_contact_filled et
- * par ce Processor.
+ * perimetre ERP-160. RG-4.08 (correctif, alignement M1/M2/M3) : un contact exige
+ * au moins le PRENOM OU le NOM (la fonction / le telephone / l'email seuls ne
+ * suffisent pas), porte a la fois par le CHECK BDD chk_carrier_contact_name et par
+ * ce Processor ; le « max 2 telephones » reste une specificite M4.
  *
  * Sequence :
  *  - POST / PATCH : rattachement au transporteur parent (linkParent),
  *    normalisation serveur RG-4.13 (prenom/nom Title Case, email lowercase),
  *    mapping du tableau d'ecriture `phones` -> phonePrimary/phoneSecondary
- *    (max 2, chiffres uniquement), puis garde RG-4.08 (≥ 1 champ) avant
- *    persistance.
+ *    (max 2, chiffres uniquement), puis garde « prenom OU nom » avant persistance.
  *  - DELETE : aucune regle metier specifique (suppression physique directe).
  *
- * RG-4.08 vit ICI (double du CHECK BDD) pour transformer une violation SQL (500
- * generique) en 422 propre rattachee au champ `firstName` (mapping inline
- * ERP-101). Le « max 2 telephones » est rattache au champ `phones` : seul
+ * La garde « prenom OU nom » vit ICI (double du CHECK BDD) pour transformer une
+ * violation SQL (500 generique) en 422 propre rattachee au champ `firstName`
+ * (mapping inline ERP-101). Le « max 2 telephones » est rattache au champ `phones` : seul
  * point de saisie des numeros (les colonnes phonePrimary/phoneSecondary sont en
  * lecture seule).
  *
@@ -77,7 +77,7 @@ final class CarrierContactProcessor implements ProcessorInterface
         $this->linkParent($data, $uriVariables);
         $this->normalize($data);
         $this->applyPhones($data);
-        $this->validateAtLeastOneField($data);
+        $this->validateName($data);
 
         return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
     }
@@ -187,25 +187,18 @@ final class CarrierContactProcessor implements ProcessorInterface
     }
 
     /**
-     * RG-4.08 : un bloc Contact est valide des qu'au moins 1 champ est rempli
-     * (firstName, lastName, jobTitle, phonePrimary ou email — meme perimetre que
-     * le CHECK BDD chk_carrier_contact_filled, qui exclut phoneSecondary). Double
-     * garde : leve une 422 propre rattachee a `firstName` plutot qu'une 500 SQL.
-     * Joue apres normalisation + mapping telephones, donc les chaines vides sont
-     * deja ramenees a null.
+     * RG-4.08 (alignement M1/M2/M3) : un bloc Contact exige au moins le PRENOM OU le
+     * NOM — un contact se materialise par son nom ; fonction / telephone / email
+     * seuls ne suffisent pas. Double garde avec le CHECK BDD chk_carrier_contact_name
+     * — leve une 422 propre rattachee a `firstName` plutot qu'une 500 SQL. Joue apres
+     * normalisation + mapping telephones, donc les chaines vides sont deja null.
      */
-    private function validateAtLeastOneField(CarrierContact $contact): void
+    private function validateName(CarrierContact $contact): void
     {
-        if (
-            null === $contact->getFirstName()
-            && null === $contact->getLastName()
-            && null === $contact->getJobTitle()
-            && null === $contact->getPhonePrimary()
-            && null === $contact->getEmail()
-        ) {
+        if (null === $contact->getFirstName() && null === $contact->getLastName()) {
             $violations = new ConstraintViolationList();
             $violations->add(new ConstraintViolation(
-                'Renseignez au moins un champ pour le contact.',
+                'Le prénom ou le nom du contact est obligatoire.',
                 null,
                 [],
                 $contact,
@@ -219,8 +212,8 @@ final class CarrierContactProcessor implements ProcessorInterface
 
     /**
      * Trim + chaine vide -> null (la fonction n'est pas normalisee en casse,
-     * contrairement aux noms de personne). Garantit que RG-4.08 detecte un champ
-     * « non rempli » meme si le client envoie une chaine vide.
+     * contrairement aux noms de personne). Evite de persister une chaine vide
+     * (« » devient null) cote fonction du contact.
      */
     private function blankToNull(?string $value): ?string
     {

src/Module/Transport/Infrastructure/DataFixtures/CarrierFixtures.php

@@ -194,7 +194,7 @@ class CarrierFixtures extends Fixture implements DependentFixtureInterface
 
     /**
      * Ajoute un contact normalise au transporteur (cascade persist via
-     * Carrier.contacts). Au moins un champ est toujours fourni (RG-4.08).
+     * Carrier.contacts). Prenom OU nom toujours fourni (RG-4.08, chk_carrier_contact_name).
      */
     private function addContact(
         Carrier $carrier,

src/Shared/Infrastructure/Database/ColumnCommentsCatalog.php

@@ -509,11 +509,11 @@ final class ColumnCommentsCatalog
             ] + self::timestampableBlamableComments(),
 
             'carrier_contact' => [
-                '_table'          => 'Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins un champ rempli (RG-4.08, chk_carrier_contact_filled), max 2 telephones.',
+                '_table'          => 'Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins le prenom OU le nom rempli (RG-4.08, chk_carrier_contact_name), max 2 telephones.',
                 'id'              => 'Identifiant interne auto-incremente.',
                 'carrier_id'      => 'FK -> carrier.id, ON DELETE CASCADE — transporteur proprietaire du contact.',
-                'first_name'      => 'Prenom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).',
-                'last_name'       => 'Nom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).',
+                'first_name'      => 'Prenom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).',
+                'last_name'       => 'Nom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).',
                 'job_title'       => 'Fonction / intitule de poste du contact (≤ 120 caracteres).',
                 'phone_primary'   => 'Telephone principal — chiffres uniquement (normalisation serveur).',
                 'phone_secondary' => 'Telephone secondaire — chiffres uniquement (max 2 telephones, RG-4.08).',

tests/Module/Transport/Api/CarrierContactApiTest.php

@@ -15,8 +15,8 @@ use Symfony\Component\Console\Output\NullOutput;
  * POST /api/carriers/{id}/contacts, PATCH/DELETE /api/carrier_contacts/{id}.
  *
  * Contrat verifie :
- *  - RG-4.08 : contact totalement vide -> 422 (au moins 1 champ requis) ;
- *  - RG-4.08 : 1 seul champ rempli -> 201 ;
+ *  - RG-4.08 : contact sans prenom ni nom -> 422 (alignement M1/M2/M3) ;
+ *  - RG-4.08 : un nom (ou prenom) suffit -> 201 ;
  *  - RG-4.08 : 3 telephones (tableau `phones`) -> 422 (max 2) ;
  *  - mapping `phones[]` -> phonePrimary / phoneSecondary + normalisation (RG-4.13) ;
  *  - PATCH / DELETE OK avec transport.carriers.manage, 403 sans (view seul).
@@ -51,7 +51,8 @@ final class CarrierContactApiTest extends AbstractCarrierApiTestCase
 
     public function testEmptyContactReturns422(): void
     {
-        // RG-4.08 : aucun champ rempli -> 422 (garde Processor, double du CHECK BDD).
+        // RG-4.08 (alignement M1/M2/M3) : sans prenom ni nom -> 422 (garde Processor,
+        // double du CHECK BDD chk_carrier_contact_name).
         $carrier = $this->seedCarrier('Contact Vide');
         $client  = $this->createAdminClient();
 
@@ -60,13 +61,13 @@ final class CarrierContactApiTest extends AbstractCarrierApiTestCase
             'json'    => [],
         ]);
         self::assertResponseStatusCodeSame(422);
-        // RG-4.08 : la violation est rattachee a `firstName` (mapping inline ERP-101).
+        // La violation est rattachee a `firstName` (mapping inline ERP-101).
         self::assertViolationOnPath($response, 'firstName');
     }
 
     public function testSingleFieldContactIsCreated(): void
     {
-        // RG-4.08 : un seul champ suffit a valider le bloc.
+        // RG-4.08 : un nom (ou prenom) suffit a valider le bloc.
         $carrier = $this->seedCarrier('Contact Mono');
         $client  = $this->createAdminClient();