bulltrackers-module 1.0.610 → 1.0.611

package/functions/api-v2/helpers/data-fetchers/firestore.js

@@ -2212,6 +2212,50 @@ const checkDataStatus = async (db, userId) => {
     };
 };
 
+/**
+ * Check Popular Investor data status - finds the latest computation date for a PI
+ * Similar to checkDataStatus but specifically for PopularInvestorProfileMetrics
+ */
+const checkPopularInvestorDataStatus = async (db, piId) => {
+    const lookbackDays = 7;
+    const today = new Date();
+    let computationDate = null;
+    
+    const computationName = 'PopularInvestorProfileMetrics';
+    
+    // Check for computation results in the last 7 days
+    for (let i = 0; i < lookbackDays; i++) {
+        const checkDate = new Date(today);
+        checkDate.setDate(checkDate.getDate() - i);
+        const dateStr = checkDate.toISOString().split('T')[0];
+        
+        try {
+            const pageRef = db.collection('unified_insights')
+                .doc(dateStr)
+                .collection('results')
+                .doc('popular-investor')
+                .collection('computations')
+                .doc(computationName)
+                .collection('pages')
+                .doc(String(piId));
+            
+            const pageSnap = await pageRef.get();
+            if (pageSnap.exists) {
+                computationDate = dateStr;
+                break;
+            }
+        } catch (error) {
+            // Continue checking other dates
+            console.error(`Error checking computation ${computationName} for ${dateStr}:`, error);
+        }
+    }
+    
+    return {
+        computationDate: computationDate,
+        available: computationDate !== null
+    };
+};
+
 const sendTestAlert = async (db, userId, payload) => {
     // Create a fake notification
     const id = `test_${Date.now()}`;
@@ -2539,6 +2583,7 @@ module.exports = {
     getSyncStatus,
     autoGenerateWatchlist,
     checkDataStatus,
+    checkPopularInvestorDataStatus,
     sendTestAlert,
     isSignedInUser,
     getUserUsername,

package/functions/api-v2/routes/popular_investors.js

@@ -147,11 +147,16 @@ router.get('/:piId/profile', async (req, res) => {
         const computationName = 'PopularInvestorProfileMetrics';
         const profileData = await pageCollection(db, targetDate, computationName, piId, parseInt(lookback));
         
+        // Extract computationDate from the latest data entry (first item in array, sorted by date descending)
+        // The profileData array contains { date, data } objects, sorted with latest first
+        const computationDate = profileData && profileData.length > 0 ? profileData[0].date : null;
+        
         res.json({ 
             success: true, 
             computation: computationName,
             piId: piId,
             data: profileData,
+            computationDate: computationDate, // Latest computation date for up-to-date validation
             profileType: 'public' // Indicates this is a public PI profile
         });
     } catch (error) {

package/functions/api-v2/routes/sync.js

@@ -6,7 +6,9 @@ const {
     isDeveloper,
     isSignedInUser,
     fetchPopularInvestorMasterList,
-    getUserUsername
+    getUserUsername,
+    checkDataStatus,
+    checkPopularInvestorDataStatus
 } = require('../helpers/data-fetchers/firestore.js');
 
 const router = express.Router();
@@ -25,20 +27,56 @@ const handleSyncRequest = async (req, res) => {
         const limit = await checkSyncRateLimits(db, targetId, req.targetUserId, isDev);
         if (!limit.allowed) return res.status(429).json({ error: limit.message });
 
-        // 2. Detect User Types
+        // 2. Detect User Types (needed for validation)
         const [isSignedIn, isPI] = await Promise.all([
             isSignedInUser(db, targetId),
             fetchPopularInvestorMasterList(db, String(targetId)).then(() => true).catch(() => false)
         ]);
 
+        // 3. Check if data is already up-to-date (prevent unnecessary syncs)
+        // Developers can bypass this check
+        if (!isDev) {
+            try {
+                let computationDate = null;
+                
+                // Check Popular Investor data status if target is a PI (anyone can sync a PI, so always check)
+                if (isPI) {
+                    const piDataStatus = await checkPopularInvestorDataStatus(db, String(targetId));
+                    computationDate = piDataStatus.computationDate;
+                }
+                
+                // Check signed-in user data status if target is a signed-in user syncing themselves
+                // (Only check if not already found from PI check, and only if syncing themselves)
+                if (isSignedIn && targetId === req.targetUserId && !computationDate) {
+                    const userDataStatus = await checkDataStatus(db, String(targetId));
+                    computationDate = userDataStatus.computationDate;
+                }
+                
+                // If we found a computation date and it's today, block the sync
+                if (computationDate) {
+                    const today = new Date().toISOString().split('T')[0]; // YYYY-MM-DD format
+                    if (computationDate === today) {
+                        return res.status(400).json({ 
+                            error: "Data is already up-to-date for today. Sync is not needed.",
+                            code: "DATA_UP_TO_DATE",
+                            computationDate: computationDate
+                        });
+                    }
+                }
+            } catch (error) {
+                // If data status check fails, log but don't block sync (fail open for reliability)
+                console.warn(`[handleSyncRequest] Failed to check data status for ${targetId}:`, error.message);
+            }
+        }
+
         if (!isSignedIn && !isPI) {
             return res.status(404).json({ error: "User not found in SignedInUsers or Popular Investor master list" });
         }
 
-        // 3. Get username
+        // 4. Get username
         const username = await getUserUsername(db, targetId) || String(targetId);
 
-        // 4. Create request IDs and dispatch tasks
+        // 5. Create request IDs and dispatch tasks
         const requestIds = [];
         const tasks = [];
 

package/package.json

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