fix(front) : toast de succes a la suppression d'un bloc (contact / adresse / RIB / prix) sur les 4 modules (ERP-193)

removeCollectionRow expose un callback onSuccess declenche uniquement apres une
suppression serveur confirmee (pas sur le retrait d'un brouillon local). Cable
sur Client / Fournisseur / Prestataire / Transporteur via notifyRemovalSuccess,
avec un message i18n generique success.deleted.

frontend/shared/utils/__tests__/collectionRow.test.ts

@@ -22,11 +22,12 @@ describe('removeCollectionRow', () => {
         const errors: Record<string, string>[] = [{}, {}]
         const deleteRow = vi.fn().mockResolvedValue(undefined)
         const onError = vi.fn()
+        const onSuccess = vi.fn()
 
         const removed = await removeCollectionRow({
             rows, errors, index: 0,
             endpoint: '/client_contacts',
-            deleteRow, makeEmpty, onError,
+            deleteRow, makeEmpty, onError, onSuccess,
         })
 
         expect(deleteRow).toHaveBeenCalledOnce()
@@ -35,6 +36,8 @@ describe('removeCollectionRow', () => {
         expect(rows).toEqual([{ id: 11, label: 'B' }])
         expect(errors).toHaveLength(1)
         expect(onError).not.toHaveBeenCalled()
+        // Toast de succes uniquement sur suppression serveur confirmee.
+        expect(onSuccess).toHaveBeenCalledOnce()
     })
 
     it('ne fait AUCUN appel reseau pour un bloc jamais persiste (id null) — retrait local', async () => {
@@ -42,16 +45,19 @@ describe('removeCollectionRow', () => {
         const errors: Record<string, string>[] = [{}, {}]
         const deleteRow = vi.fn().mockResolvedValue(undefined)
         const onError = vi.fn()
+        const onSuccess = vi.fn()
 
         const removed = await removeCollectionRow({
             rows, errors, index: 1,
             endpoint: '/client_contacts',
-            deleteRow, makeEmpty, onError,
+            deleteRow, makeEmpty, onError, onSuccess,
         })
 
         expect(deleteRow).not.toHaveBeenCalled()
         expect(removed).toBe(true)
         expect(rows).toEqual([{ id: 10, label: 'A' }])
+        // Retrait d'un simple brouillon local : pas de toast « supprime ».
+        expect(onSuccess).not.toHaveBeenCalled()
     })
 
     it('conserve le bloc et remonte l\'erreur si le DELETE serveur echoue (ex. 409 dernier RIB LCR)', async () => {

frontend/shared/utils/collectionRow.ts

@@ -33,6 +33,12 @@ export interface RemoveCollectionRowOptions<T extends DeletableRow> {
     makeEmpty: () => T
     /** Remontee d'erreur 409/422 mappee proprement (message back, pas de toast fourre-tout). */
     onError: (error: unknown) => void
+    /**
+     * Callback de succes (toast) appele UNIQUEMENT apres une suppression serveur
+     * confirmee d'un bloc persiste (`id` non null). Pas appele sur le simple retrait
+     * d'un brouillon local non enregistre (aucune suppression reelle).
+     */
+    onSuccess?: () => void
 }
 
 /**
@@ -55,8 +61,9 @@ export interface RemoveCollectionRowOptions<T extends DeletableRow> {
 export async function removeCollectionRow<T extends DeletableRow>(
     options: RemoveCollectionRowOptions<T>,
 ): Promise<boolean> {
-    const { rows, errors, index, endpoint, deleteRow, makeEmpty, onError } = options
+    const { rows, errors, index, endpoint, deleteRow, makeEmpty, onError, onSuccess } = options
     const removed = rows[index]
+    let serverDeleted = false
 
     // Bloc existant : suppression serveur d'abord, retrait local seulement si OK.
     if (removed?.id != null) {
@@ -67,6 +74,7 @@ export async function removeCollectionRow<T extends DeletableRow>(
             onError(error)
             return false
         }
+        serverDeleted = true
     }
 
     rows.splice(index, 1)
@@ -75,5 +83,9 @@ export async function removeCollectionRow<T extends DeletableRow>(
     if (rows.length === 0) {
         rows.push(makeEmpty())
     }
+    // Toast de succes uniquement quand le serveur a confirme une vraie suppression.
+    if (serverDeleted) {
+        onSuccess?.()
+    }
     return true
 }