From 1617267128769376e79bc77b0874f860ad30a1ba Mon Sep 17 00:00:00 2001
From: Manus <dev-agent@manus.ai>
Date: Fri, 2 Oct 2026 21:10:42 +0000
Subject: [PATCH] =?UTF-8?q?fix:=20expliciter=20le=20r=C3=A9sultat=20des=20?=
 =?UTF-8?q?imports=20manuels?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 client/src/pages/Upload.tsx        | 35 ++++++++++++++++++++++++------
 server/importResultSummary.test.ts | 23 ++++++++++++++++++++
 shared/importResultSummary.ts      | 17 +++++++++++++++
 3 files changed, 68 insertions(+), 7 deletions(-)
 create mode 100644 server/importResultSummary.test.ts
 create mode 100644 shared/importResultSummary.ts

diff --git a/client/src/pages/Upload.tsx b/client/src/pages/Upload.tsx
index 4be9ee1..c1b5ad8 100644
--- a/client/src/pages/Upload.tsx
+++ b/client/src/pages/Upload.tsx
@@ -17,6 +17,7 @@ import {
 } from "lucide-react";
 import { toast } from "sonner";
 import { useLocation } from "wouter";
+import { parseImportResultSummary } from "@shared/importResultSummary";
 
 interface UploadedFile {
   name: string;
@@ -24,6 +25,9 @@ interface UploadedFile {
   status: "pending" | "processing" | "completed" | "error";
   progress?: string;
   invoicesDetected?: number;
+  importedCount?: number;
+  duplicateCount?: number;
+  errorCount?: number;
 }
 
 export default function Upload() {
@@ -66,11 +70,15 @@ export default function Upload() {
         prev.map((f) => {
           if (f.sourceFileId !== sf.id) return f;
           if (f.status === sf.processingStatus) return f;
+          const summary = parseImportResultSummary(sf.processingProgress);
           return {
             ...f,
             status: (sf.processingStatus ?? "pending") as UploadedFile["status"],
             progress: sf.processingProgress ?? undefined,
             invoicesDetected: sf.totalInvoicesDetected ?? undefined,
+            importedCount: summary?.importedCount,
+            duplicateCount: summary?.duplicateCount,
+            errorCount: summary?.errorCount,
           };
         })
       );
@@ -165,6 +173,10 @@ export default function Upload() {
   const processingCount = uploadedFiles.filter(
     (f) => f.status === "processing" || f.status === "pending"
   ).length;
+  const importedInvoiceCount = uploadedFiles.reduce(
+    (total, file) => total + (file.importedCount ?? 0),
+    0,
+  );
 
   return (
     <DashboardLayout>
@@ -282,8 +294,11 @@ export default function Upload() {
                 <div>
                   <CardTitle>Fichiers importés</CardTitle>
                   <CardDescription>
-                    {completedCount > 0 && (
-                      <span className="text-green-600 font-medium">{completedCount} terminé(s) </span>
+                    {completedCount > 0 && importedInvoiceCount > 0 && (
+                      <span className="text-green-600 font-medium">{importedInvoiceCount} facture(s) créée(s) </span>
+                    )}
+                    {completedCount > 0 && importedInvoiceCount === 0 && (
+                      <span className="text-amber-700 font-medium">{completedCount} terminé(s), aucune facture créée </span>
                     )}
                     {processingCount > 0 && (
                       <span className="text-blue-600 font-medium">{processingCount} en cours </span>
@@ -294,7 +309,7 @@ export default function Upload() {
                   </CardDescription>
                 </div>
                 <div className="flex gap-2">
-                  {completedCount > 0 && (
+                  {importedInvoiceCount > 0 && (
                     <Button
                       size="sm"
                       onClick={() => setLocation("/invoices")}
@@ -332,10 +347,16 @@ export default function Upload() {
                       {f.status === "processing" && (
                         <p className="text-xs text-muted-foreground">{f.progress || "Traitement en cours..."}</p>
                       )}
-                      {f.status === "completed" && f.invoicesDetected !== undefined && (
-                        <p className="text-xs text-green-600">
-                          {f.invoicesDetected} facture(s) détectée(s)
-                        </p>
+                      {f.status === "completed" && f.importedCount !== undefined && (
+                        f.importedCount > 0 ? (
+                          <p className="text-xs text-green-600">
+                            {f.importedCount} facture(s) créée(s)
+                          </p>
+                        ) : (
+                          <p className="text-xs text-amber-700">
+                            Aucune facture créée{f.duplicateCount ? ` : ${f.duplicateCount} doublon(s) ou facture(s) supprimée(s)` : ""}
+                          </p>
+                        )
                       )}
                       {f.status === "error" && (
                         <p className="text-xs text-red-500">{f.progress || "Erreur de traitement"}</p>
diff --git a/server/importResultSummary.test.ts b/server/importResultSummary.test.ts
new file mode 100644
index 0000000..47a1830
--- /dev/null
+++ b/server/importResultSummary.test.ts
@@ -0,0 +1,23 @@
+import { describe, expect, it } from "vitest";
+import { parseImportResultSummary } from "@shared/importResultSummary";
+
+describe("parseImportResultSummary", () => {
+  it("identifie un traitement terminé sans facture créée", () => {
+    expect(parseImportResultSummary("Terminé: 0 importée(s), 1 doublon(s)"))
+      .toEqual({ importedCount: 0, duplicateCount: 1, errorCount: 0 });
+  });
+
+  it("identifie une facture réellement créée", () => {
+    expect(parseImportResultSummary("Terminé: 1 importée(s), 0 doublon(s)"))
+      .toEqual({ importedCount: 1, duplicateCount: 0, errorCount: 0 });
+  });
+
+  it("conserve les erreurs éventuelles", () => {
+    expect(parseImportResultSummary("Terminé: 0 importée(s), 0 doublon(s), 1 erreur(s)"))
+      .toEqual({ importedCount: 0, duplicateCount: 0, errorCount: 1 });
+  });
+
+  it("retourne null pour un statut intermédiaire", () => {
+    expect(parseImportResultSummary("Extraction 1/1 factures...")).toBeNull();
+  });
+});
diff --git a/shared/importResultSummary.ts b/shared/importResultSummary.ts
new file mode 100644
index 0000000..b1fed96
--- /dev/null
+++ b/shared/importResultSummary.ts
@@ -0,0 +1,17 @@
+export type ImportResultSummary = {
+  importedCount: number;
+  duplicateCount: number;
+  errorCount: number;
+};
+
+/** Extrait le résultat final affiché par les services d'import. */
+export function parseImportResultSummary(progress: string | null | undefined): ImportResultSummary | null {
+  if (!progress) return null;
+  const match = progress.match(/Termin[ée]\s*:\s*(\d+)\s+importée\(s\),\s*(\d+)\s+doublon\(s\)(?:,\s*(\d+)\s+erreur\(s\))?/i);
+  if (!match) return null;
+  return {
+    importedCount: Number(match[1]),
+    duplicateCount: Number(match[2]),
+    errorCount: Number(match[3] ?? 0),
+  };
+}