bulltrackers-module 1.0.547 → 1.0.548

package/functions/computation-system/persistence/ResultCommitter.js

@@ -237,7 +237,6 @@ async function writeSingleResult(result, docRef, name, dateContext, logger, conf
     const opCounts = { writes: 0, deletes: 0 };
     
     // [FIX 1] Always check for shards if we might compress, not just on 'isInitialWrite'. 
-    // This prevents zombie shards from persisting if a previous run failed.
     let wasSharded = false;
     try {
         const currentSnap = await docRef.get();
@@ -245,9 +244,7 @@ async function writeSingleResult(result, docRef, name, dateContext, logger, conf
             const d = currentSnap.data();
             wasSharded = (d._sharded === true);
         }
-    } catch (e) {
-        // Ignore read errors, proceed with write attempts
-    }
+    } catch (e) {}
 
     // --- COMPRESSION STRATEGY ---
     try {
@@ -259,24 +256,25 @@ async function writeSingleResult(result, docRef, name, dateContext, logger, conf
             if (compressedBuffer.length < 900 * 1024) {
                 logger.log('INFO', `[Compression] ${name}: Compressed ${(rawBuffer.length/1024).toFixed(0)}KB -> ${(compressedBuffer.length/1024).toFixed(0)}KB.`);
                 
+                // [FIX 2] Explicitly wrap buffer to ensure it saves as a Blob, NOT a map of index->byte
+                const payloadBuffer = Buffer.from(compressedBuffer); 
+
                 const compressedPayload = { 
                     _compressed: true, 
                     _completed: true, 
                     _lastUpdated: new Date().toISOString(), 
-                    payload: compressedBuffer 
+                    payload: payloadBuffer 
                 };
                 
-                // [FIX 2] Self-Healing: If we are writing compressed, we MUST ensure shards are gone.
-                // We use a batch to atomically write the blob and delete the shards.
+                // [FIX 3] Self-Healing: If we are writing compressed, we MUST ensure shards are gone.
                 if (wasSharded) {
                     const updates = [];
                     const shardCol = docRef.collection('_shards');
                     const shardDocs = await shardCol.listDocuments();
                     
-                    // Add delete ops for all shards
                     shardDocs.forEach(d => updates.push({ type: 'DELETE', ref: d }));
                     
-                    // [FIX 3] Use merge: false (overwrite) to clear old indexes and ensure we don't inherit "too many indexes" error.
+                    // Use merge: false (overwrite)
                     updates.push({ ref: docRef, data: compressedPayload, options: { merge: false } });
                     
                     opCounts.deletes += shardDocs.length;
@@ -284,7 +282,6 @@ async function writeSingleResult(result, docRef, name, dateContext, logger, conf
 
                     await commitBatchInChunks(config, deps, updates, `${name}::Cleanup+Compress`);
                 } else {
-                    // [FIX 3] Use merge: false here as well for consistency and safety.
                     await docRef.set(compressedPayload, { merge: false });
                     opCounts.writes += 1;
                 }
@@ -293,8 +290,6 @@ async function writeSingleResult(result, docRef, name, dateContext, logger, conf
             }
         }
     } catch (compErr) {
-        // [FIX 4] Log the error but allow fallthrough to sharding (Self-Healing).
-        // This ensures that if compression fails (e.g., unexpected encoding issue), we still try to save the data via sharding.
         logger.log('WARN', `[SelfHealing] Compression failed for ${name}, reverting to sharding. Error: ${compErr.message}`);
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bulltrackers-module",
-  "version": "1.0.547",
+  "version": "1.0.548",
   "description": "Helper Functions for Bulltrackers.",
   "main": "index.js",
   "files": [