Fix modal: zalo-sync products (newest from dataset) always prepend on top, not deduped against base catalog

src/vaix-rag.js

@@ -1128,7 +1128,7 @@ async function load() {
             specs: (p.specs && typeof p.specs === "object") ? p.specs : {},
             video: p.vid || "", image: p.i || p.image || "",
             images: Array.isArray(p.imgs) ? p.imgs : (Array.isArray(p.g) ? p.g : []), link: p.l || "",
-            _addedBy: "products_url_added"
+            _source: p._source || "", _addedBy: "products_url_added"
           }));
           // Dedup added products against base catalog AND against each other
           // (by sku, slug, link, or normalized name) so a URL added twice never
@@ -1144,9 +1144,17 @@ async function load() {
             const sl = String(p.slug || "").toLowerCase();
             const lk = String(p.link || "").toLowerCase();
             const nm = String(p.title_clean || p.name || "").toLowerCase().replace(/\s+/g, " ").trim();
-            // Collision with a base catalog product (same sku or slug) → drop.
-            if (sk && baseSkus.has(sk)) return false;
-            if (sl && baseSlugs.has(sl)) return false;
+            // Zalo-sync rows (written by auto_sync_worker with _source
+            // "zalo-products-all-autosync") carry the freshest dataset image +
+            // newest _added_at — they must NOT be dropped just because the base
+            // catalog already has the same SKU. Only dedup them against other
+            // rows in this same batch. Manual URL-added rows keep the old
+            // collision behavior (drop if base already has the sku/slug).
+            const isZaloSync = !!(p._source === "zalo-products-all-autosync" || p._addedBy === "zalo-products-all-autosync");
+            if (!isZaloSync) {
+              if (sk && baseSkus.has(sk)) return false;
+              if (sl && baseSlugs.has(sl)) return false;
+            }
             // Two URL-added records can share a name but have different SKUs
             // (e.g. Hitachi BD-1054HVOW vs 357869). Keep the FIRST record for
             // that name — file order puts the record with a link first.
@@ -1193,10 +1201,16 @@ async function load() {
             specs: (p.specs && typeof p.specs === "object") ? p.specs : {},
             video: p.vid || "", image: p.i || p.image || "",
             images: Array.isArray(p.imgs) ? p.imgs : (Array.isArray(p.g) ? p.g : []), link: p.l || "",
-            _addedBy: "zalo-products-all-autosync"
+            _source: p._source || "", _addedBy: "zalo-products-all-autosync"
           }));
           const baseSkus2 = new Set(allProducts.map((p) => String(p.sku).toLowerCase()).filter(Boolean));
           const baseSlugs2 = new Set(allProducts.map((p) => String(p.slug || "").toLowerCase()).filter(Boolean));
+          // IMPORTANT: do NOT dedup zalo-sync products against the base catalog.
+          // The base catalog (products_index.json, 11k rows) already contains
+          // many of these SKUs (MH700GT, EVF015M, ...) with STALE/empty images
+          // and OLD positions. The zalo-sync rows carry the fresh dataset image
+          // + newest _added_at, so they must always be prepended on top of the
+          // modal list. We only dedup against OTHER rows in this same batch.
           const seenKeys2 = new Set();
           const seenNames2 = new Set();
           const uniqueLocal = localAdded.filter((p) => {
@@ -1204,8 +1218,6 @@ async function load() {
             const sl = String(p.slug || "").toLowerCase();
             const lk = String(p.link || "").toLowerCase();
             const nm = String(p.title_clean || p.name || "").toLowerCase().replace(/\s+/g, " ").trim();
-            if (sk && baseSkus2.has(sk)) return false;
-            if (sl && baseSlugs2.has(sl)) return false;
             if (nm && seenNames2.has(nm)) return false;
             if (nm) seenNames2.add(nm);
             let key = "";