bulltrackers-module 1.0.573 → 1.0.574

package/functions/alert-system/helpers/alert_helpers.js

@@ -212,14 +212,14 @@ async function findSubscriptionsForPI(db, logger, piCid, alertTypeId, computatio
         return subscriptions;
       }
     } else {
-      logger.log('WARN', `[findSubscriptionsForPI] WatchlistMembershipData/${computationDate} not found, falling back to scanning all watchlists`);
-      // Fallback to legacy method if membership data doesn't exist
-      return await findSubscriptionsForPILegacy(db, logger, piCid, alertTypeId, configKey);
+      logger.log('WARN', `[findSubscriptionsForPI] WatchlistMembershipData/${computationDate} not found, returning existing subscriptions (dev overrides only)`);
+      // Return subscriptions array which may contain dev overrides
+      return subscriptions;
     }
   } catch (error) {
     logger.log('ERROR', `[findSubscriptionsForPI] Error loading WatchlistMembershipData: ${error.message}`);
-    // Fallback to legacy method on error
-    return await findSubscriptionsForPILegacy(db, logger, piCid, alertTypeId, configKey);
+    // Return subscriptions array which may contain dev overrides
+    return subscriptions;
   }
   
   // Step 2: For each user, read their watchlists from SignedInUsers/{cid}/watchlists
@@ -298,51 +298,6 @@ async function findSubscriptionsForPI(db, logger, piCid, alertTypeId, computatio
   return subscriptions;
 }
 
-/**
- * Legacy fallback method - scans all watchlists (used if WatchlistMembershipData is not available)
- */
-async function findSubscriptionsForPILegacy(db, logger, piCid, alertTypeId, configKey) {
-  const subscriptions = [];
-  
-  logger.log('INFO', `[findSubscriptionsForPILegacy] Using legacy method to find subscriptions for PI ${piCid}`);
-  
-  // Get all watchlists from legacy path
-  const watchlistsCollection = db.collection('watchlists');
-  const watchlistsSnapshot = await watchlistsCollection.get();
-  
-  for (const userDoc of watchlistsSnapshot.docs) {
-    const userCid = Number(userDoc.id);
-    const userListsSnapshot = await userDoc.ref.collection('lists').get();
-    
-    for (const listDoc of userListsSnapshot.docs) {
-      const listData = listDoc.data();
-      
-      // Check static watchlists
-      if (listData.type === 'static' && listData.items && Array.isArray(listData.items)) {
-        for (const item of listData.items) {
-          if (Number(item.cid) === Number(piCid)) {
-            const isTestProbe = alertTypeId === 'TestSystemProbe';
-            const isEnabled = item.alertConfig && item.alertConfig[configKey] === true;
-            
-            if (isTestProbe || isEnabled) {
-              subscriptions.push({
-                userCid: userCid,
-                piCid: piCid,
-                piUsername: item.username || `PI-${piCid}`,
-                watchlistId: listDoc.id,
-                watchlistName: listData.name || 'Unnamed Watchlist',
-                alertConfig: item.alertConfig
-              });
-              break;
-            }
-          }
-        }
-      }
-    }
-  }
-  
-  return subscriptions;
-}
 
 /**
  * Check if alert should trigger based on thresholds

package/package.json

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