bulltrackers-module 1.0.559 → 1.0.560

package/functions/generic-api/user-api/helpers/metrics/personalized_metrics_helpers.js

@@ -98,18 +98,60 @@ async function getSignedInUserPIPersonalizedMetrics(req, res, dependencies, conf
     try {
         const devOverride = await getDevOverride(db, userCid, config, logger);
         
+        // Also read raw Firestore data to check pretendToBePI value directly (in case of normalization issues)
+        let rawPretendToBePI = null;
+        if (devOverride && devOverride.enabled) {
+            try {
+                const devOverridesCollection = config.devOverridesCollection || 'dev_overrides';
+                const overrideDoc = await db.collection(devOverridesCollection).doc(String(userCid)).get();
+                if (overrideDoc.exists) {
+                    const rawData = overrideDoc.data();
+                    rawPretendToBePI = rawData.pretendToBePI; // Get raw value (could be false, true, undefined, etc.)
+                }
+            } catch (err) {
+                logger.log('WARN', `[getSignedInUserPIPersonalizedMetrics] Error reading raw dev override: ${err.message}`);
+            }
+        }
+        
         // For PI-related checks, respect pretendToBePI flag
-        // If pretendToBePI is false, use actual userCid even if impersonateCid is set
+        // If pretendToBePI is false (either normalized or raw), use actual userCid even if impersonateCid is set
         let effectiveCid;
-        if (devOverride && devOverride.enabled && devOverride.pretendToBePI === false) {
-            // User explicitly doesn't want to pretend to be a PI, use actual CID
+        let shouldCheckPI = true;
+        
+        // Check if user explicitly doesn't want to pretend to be a PI
+        // Check both normalized value and raw value to be safe
+        const pretendToBePIValue = rawPretendToBePI !== undefined ? rawPretendToBePI : (devOverride ? devOverride.pretendToBePI : false);
+        const explicitlyNotPretending = pretendToBePIValue === false;
+        
+        if (devOverride && devOverride.enabled && explicitlyNotPretending) {
+            // User explicitly doesn't want to pretend to be a PI
+            // Use actual CID and don't check PI status (user doesn't want to be treated as PI)
             effectiveCid = Number(userCid);
+            shouldCheckPI = false; // Skip PI check since user doesn't want to pretend to be PI
+            
+            logger.log('INFO', `[getSignedInUserPIPersonalizedMetrics] DEV OVERRIDE: User ${userCid} has pretendToBePI=false (raw: ${rawPretendToBePI}, normalized: ${devOverride.pretendToBePI}), skipping PI check and returning 404`);
         } else {
             // Use normal effective CID logic (may use impersonateCid if set)
             effectiveCid = await getEffectiveCid(db, userCid, config, logger);
         }
         
-        const isImpersonating = devOverride && devOverride.enabled && devOverride.impersonateCid && effectiveCid !== Number(userCid);
+        // Calculate isImpersonating based on whether we're actually using impersonation
+        // If pretendToBePI is false, we're not impersonating for PI purposes
+        // Only consider it impersonating if pretendToBePI is not explicitly false AND effectiveCid differs from userCid
+        const isImpersonating = shouldCheckPI && 
+                                devOverride && devOverride.enabled && devOverride.impersonateCid && 
+                                !explicitlyNotPretending && effectiveCid !== Number(userCid);
+        
+        // If user explicitly doesn't want to pretend to be a PI, return 404 immediately
+        if (!shouldCheckPI) {
+            return res.status(404).json({
+                error: "Not a Popular Investor",
+                message: "This endpoint is only available for users who are Popular Investors",
+                effectiveCid: effectiveCid,
+                isImpersonating: false, // Not impersonating if pretendToBePI is false
+                actualCid: Number(userCid)
+            });
+        }
         
         // Check if user is a PI
         const rankEntry = await checkIfUserIsPI(db, effectiveCid, config, logger);
@@ -118,7 +160,8 @@ async function getSignedInUserPIPersonalizedMetrics(req, res, dependencies, conf
                 error: "Not a Popular Investor",
                 message: "This endpoint is only available for users who are Popular Investors",
                 effectiveCid: effectiveCid,
-                isImpersonating: isImpersonating || false
+                isImpersonating: isImpersonating || false,
+                actualCid: Number(userCid)
             });
         }
         
@@ -355,7 +398,8 @@ async function getSignedInUserPIPersonalizedMetrics(req, res, dependencies, conf
             isFallback: foundDate !== today,
             daysBackFromLatest: checkedDates.indexOf(foundDate),
             isImpersonating: isImpersonating || false,
-            actualCid: Number(userCid)
+            actualCid: Number(userCid),
+            pretendToBePI: devOverride && devOverride.enabled ? (devOverride.pretendToBePI || false) : false
         });
         
     } catch (error) {

package/package.json

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