From 6f4affcd66226ad740255503a49daa3039a69727 Mon Sep 17 00:00:00 2001
From: Mathieu <mathieu.fort@gmail.com>
Date: Thu, 11 Jun 2026 23:09:08 +0200
Subject: [PATCH] fix: DiffTable colonnes visibles + filtrage lignes parasites
 dans Sheets
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- table-fixed avec colgroup pour éviter que la colonne Ref écrase les autres
- overflow-x-auto pour permettre le scroll si nécessaire
- parseSheetRows : ignore les refs invalides (< 2 chars, caractères spéciaux, mots réservés)
- parseSheetRows : ignore les lignes sans prix valide (séparateurs, notes)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 src/components/sync/DiffTable.tsx | 21 +++++++++++++--------
 src/lib/googleSheets.ts           | 16 ++++++++++++++--
 2 files changed, 27 insertions(+), 10 deletions(-)

diff --git a/src/components/sync/DiffTable.tsx b/src/components/sync/DiffTable.tsx
index c6d57ce..1d61d1f 100644
--- a/src/components/sync/DiffTable.tsx
+++ b/src/components/sync/DiffTable.tsx
@@ -56,28 +56,33 @@ export function DiffTable({ diff }: DiffTableProps) {
       {paginated.length === 0 ? (
         <p className="text-slate-600 text-sm py-4 text-center">Aucun changement dans cette catégorie.</p>
       ) : (
-        <div className="border border-slate-700 rounded-xl overflow-hidden">
-          <table className="w-full text-xs">
+        <div className="border border-slate-700 rounded-xl overflow-x-auto">
+          <table className="w-full text-xs table-fixed">
+            <colgroup>
+              <col className="w-36" />
+              <col className="w-32" />
+              <col />
+            </colgroup>
             <thead className="bg-slate-800 text-slate-400">
               <tr>
                 <th className="px-3 py-2 text-left">Ref</th>
                 <th className="px-3 py-2 text-left">Type</th>
-                <th className="px-3 py-2 text-left">Champs modifiés</th>
+                <th className="px-3 py-2 text-left">Détail</th>
               </tr>
             </thead>
             <tbody>
               {paginated.map((change) => (
                 <tr key={change.ref} className={`border-t border-slate-800 ${TYPE_CONFIG[change.type].bg}`}>
-                  <td className="px-3 py-2 font-mono text-slate-300">{change.ref}</td>
-                  <td className={`px-3 py-2 font-semibold ${TYPE_CONFIG[change.type].color}`}>
+                  <td className="px-3 py-2 font-mono text-slate-300 truncate max-w-0" title={change.ref}>{change.ref}</td>
+                  <td className={`px-3 py-2 font-semibold whitespace-nowrap ${TYPE_CONFIG[change.type].color}`}>
                     {TYPE_CONFIG[change.type].label}
                   </td>
                   <td className="px-3 py-2 text-slate-400">
                     {change.type === 'update' && change.changedFields?.map((f) => (
-                      <span key={f.field} className="inline-block mr-2">
+                      <span key={f.field} className="inline-block mr-3">
                         <span className="text-slate-500">{f.field} : </span>
-                        <span className="line-through text-red-400">{f.from}</span>
-                        <span className="text-slate-400"> → </span>
+                        <span className="line-through text-red-400 mr-1">{f.from}</span>
+                        <span className="text-slate-500 mr-1">→</span>
                         <span className="text-green-400">{f.to}</span>
                       </span>
                     ))}
diff --git a/src/lib/googleSheets.ts b/src/lib/googleSheets.ts
index 57879e1..25bc8fa 100644
--- a/src/lib/googleSheets.ts
+++ b/src/lib/googleSheets.ts
@@ -12,21 +12,33 @@ function normalizeHandle(ref: string): string {
  * Colonnes : A=Référence, B=Nom, C=Description, D=Prix, E=Stock, F=Statut
  * Exporté pour les tests unitaires.
  */
+const HEADER_WORDS = new Set(['id', 'ref', 'sku', 'nom', 'titre', 'reference', 'prix', 'statut'])
+
+function isValidRef(ref: string): boolean {
+  if (ref.length < 2) return false
+  if (/^[^a-zA-Z0-9]+$/.test(ref)) return false  // que des caractères spéciaux
+  if (HEADER_WORDS.has(ref.toLowerCase())) return false
+  return true
+}
+
 export function parseSheetRows(rows: string[][]): SyncProduct[] {
   return rows
     .map((row): SyncProduct | null => {
       const ref = row[0]?.trim()
-      if (!ref) return null
+      if (!ref || !isValidRef(ref)) return null
 
       const rawPrice = (row[3] ?? '').replace(',', '.').trim()
       const price = parseFloat(rawPrice || '0')
 
+      // Ligne sans prix valide = séparateur ou note, pas un produit
+      if (isNaN(price) || price <= 0) return null
+
       return {
         ref,
         handle: normalizeHandle(ref),
         title: row[1]?.trim() || ref,
         description: row[2]?.trim() || '',
-        price: isNaN(price) ? '0.00' : price.toFixed(2),
+        price: price.toFixed(2),
         stock: parseInt(row[4] ?? '0', 10) || 0,
         status: (row[5] ?? '').toLowerCase().includes('inac') ? 'draft' : 'active',
       }