@fachkraftfreund/n8n-nodes-supabase 1.3.8 → 1.3.10

package/dist/nodes/Supabase/operations/database/index.js

@@ -62,7 +62,16 @@ function deduplicateByConflictKeys(rows, conflictColumns) {
     const seen = new Map();
     for (let i = 0; i < rows.length; i++) {
         const row = rows[i];
-        const compositeKey = keys.map((k) => { var _a; return String((_a = row[k]) !== null && _a !== void 0 ? _a : ''); }).join('\0');
+        const compositeKey = keys.map((k) => {
+            const val = row[k];
+            if (val === null || val === undefined)
+                return '\x00null';
+            const str = String(val).trim();
+            const num = Number(str);
+            if (str !== '' && !isNaN(num))
+                return String(num);
+            return str.toLowerCase();
+        }).join('\0');
         seen.set(compositeKey, i);
     }
     return Array.from(seen.values()).sort((a, b) => a - b).map((i) => rows[i]);
@@ -206,38 +215,27 @@ async function handleBulkUpsert(supabase, itemCount) {
     const deduplicate = this.getNodeParameter('deduplicateByConflict', 0, false);
     (0, supabaseClient_1.validateTableName)(table);
     let rows = collectRowData(this, itemCount);
-    console.log(`[Supabase UPSERT ${table}] rows=${rows.length}, onConflict="${onConflict}", deduplicate=${deduplicate}`);
     const options = {};
     if (onConflict)
         options.onConflict = onConflict;
     if (deduplicate && onConflict) {
         const before = rows.length;
         rows = deduplicateByConflictKeys(rows, onConflict);
-        console.log(`[Supabase UPSERT ${table}] dedup by "${onConflict}": ${before} → ${rows.length} rows`);
+        if (rows.length < before) {
+            console.log(`[Supabase UPSERT ${table}] dedup by "${onConflict}": ${before} → ${rows.length} rows`);
+        }
     }
     const returnData = [];
-    const duplicateError = 'cannot affect row a second time';
     for (let offset = 0; offset < rows.length; offset += BULK_BATCH_SIZE) {
-        let batch = rows.slice(offset, offset + BULK_BATCH_SIZE);
+        const batch = rows.slice(offset, offset + BULK_BATCH_SIZE);
         const batchLabel = `UPSERT ${table} batch ${Math.floor(offset / BULK_BATCH_SIZE) + 1}`;
         const data = await withRetry(async () => {
             const { data, error } = await supabase
                 .from(table)
                 .upsert(batch, options)
                 .select();
-            if (error) {
-                const msg = (0, supabaseClient_1.formatSupabaseError)(error);
-                if (msg.toLowerCase().includes(duplicateError) && onConflict) {
-                    const before = batch.length;
-                    batch = deduplicateByConflictKeys(batch, onConflict);
-                    console.log(`[Supabase ${batchLabel}] duplicate-row error caught, dedup fallback: ${before} → ${batch.length} rows`);
-                    const retry = await supabase.from(table).upsert(batch, options).select();
-                    if (retry.error)
-                        throw new Error((0, supabaseClient_1.formatSupabaseError)(retry.error));
-                    return retry.data;
-                }
-                throw new Error(msg);
-            }
+            if (error)
+                throw new Error((0, supabaseClient_1.formatSupabaseError)(error));
             return data;
         }, batchLabel);
         if (Array.isArray(data)) {
@@ -258,7 +256,6 @@ async function handleBulkUpdate(supabase, itemCount) {
     }
     const deduplicate = this.getNodeParameter('deduplicateByConflict', 0, false);
     let rows = collectRowData(this, itemCount);
-    console.log(`[Supabase UPDATE ${table}] rows=${rows.length}, matchColumn="${matchColumn}", deduplicate=${deduplicate}`);
     for (let i = 0; i < rows.length; i++) {
         const row = rows[i];
         if (!row || row[matchColumn] === undefined) {
@@ -271,28 +268,16 @@ async function handleBulkUpdate(supabase, itemCount) {
         console.log(`[Supabase UPDATE ${table}] dedup by "${matchColumn}": ${before} → ${rows.length} rows`);
     }
     const returnData = [];
-    const duplicateError = 'cannot affect row a second time';
     for (let offset = 0; offset < rows.length; offset += BULK_BATCH_SIZE) {
-        let batch = rows.slice(offset, offset + BULK_BATCH_SIZE);
+        const batch = rows.slice(offset, offset + BULK_BATCH_SIZE);
         const batchLabel = `UPDATE ${table} batch ${Math.floor(offset / BULK_BATCH_SIZE) + 1}`;
         const data = await withRetry(async () => {
             const { data, error } = await supabase
                 .from(table)
                 .upsert(batch, { onConflict: matchColumn })
                 .select();
-            if (error) {
-                const msg = (0, supabaseClient_1.formatSupabaseError)(error);
-                if (msg.toLowerCase().includes(duplicateError)) {
-                    const before = batch.length;
-                    batch = deduplicateByConflictKeys(batch, matchColumn);
-                    console.log(`[Supabase ${batchLabel}] duplicate-row error caught, dedup fallback: ${before} → ${batch.length} rows`);
-                    const retry = await supabase.from(table).upsert(batch, { onConflict: matchColumn }).select();
-                    if (retry.error)
-                        throw new Error((0, supabaseClient_1.formatSupabaseError)(retry.error));
-                    return retry.data;
-                }
-                throw new Error(msg);
-            }
+            if (error)
+                throw new Error((0, supabaseClient_1.formatSupabaseError)(error));
             return data;
         }, batchLabel);
         if (Array.isArray(data)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fachkraftfreund/n8n-nodes-supabase",
-  "version": "1.3.8",
+  "version": "1.3.10",
   "description": "Comprehensive n8n community node for Supabase with database and storage operations",
   "keywords": [
     "n8n-community-node-package",