aiden-shared-calculations-unified 1.0.107 → 1.0.108

package/calculations/core/asset-pnl-status.js

@@ -4,7 +4,9 @@
  */
 class AssetPnlStatus {
     constructor() {
-        this.userStats = new Map();
+        // [FIX 1] Use 'this.results' directly. 
+        // The StandardExecutor monitors this property to perform Batch Flushing.
+        this.results = {}; 
     }
 
     static getMetadata() {
@@ -22,29 +24,26 @@ class AssetPnlStatus {
     static getSchema() {
         return {
             "type": "object",
-            "properties": {
-                "by_user": {
+            // [FIX 2] Flattened Schema (removed "by_user" wrapper)
+            "patternProperties": {
+                "^[0-9]+$": {
                     "type": "object",
-                    "patternProperties": {
-                        "^[0-9]+$": {
-                            "type": "object",
-                            "properties": {
-                                "winners": { "type": "array", "items": { "type": "string" } },
-                                "losers": { "type": "array", "items": { "type": "string" } }
-                            },
-                            "required": ["winners", "losers"]
-                        }
-                    }
+                    "properties": {
+                        "winners": { "type": "array", "items": { "type": "string" } },
+                        "losers": { "type": "array", "items": { "type": "string" } }
+                    },
+                    "required": ["winners", "losers"]
                 }
             }
         };
     }
     
     _initUser(userId) {
-        if (!this.userStats.has(userId)) {
-            this.userStats.set(userId, { winners: new Set(), losers: new Set() });
+        // [FIX 3] Direct access to this.results
+        if (!this.results[userId]) {
+            this.results[userId] = { winners: [], losers: [] };
         }
-        return this.userStats.get(userId);
+        return this.results[userId];
     }
 
     process(context) {
@@ -55,7 +54,9 @@ class AssetPnlStatus {
         const positions = extract.getPositions(user.portfolio.today, user.type);
         if (!positions || positions.length === 0) return;
 
-        let stats = null;
+        // Use temporary sets for processing to keep logic clean, then convert to array for storage
+        const winners = new Set();
+        const losers = new Set();
 
         for (const pos of positions) {
             const instId = extract.getInstrumentId(pos);
@@ -68,35 +69,26 @@ class AssetPnlStatus {
             const sector = mappings.instrumentToSector[instId];
             const isWinner = pnl > 0;
 
-            if (ticker || sector) {
-                if (!stats) stats = this._initUser(userId);
-                if (ticker) {
-                    if (isWinner) stats.winners.add(ticker); else stats.losers.add(ticker);
-                }
-                if (sector) {
-                    if (isWinner) stats.winners.add(sector); else stats.losers.add(sector);
-                }
-            }
+            if (ticker) { isWinner ? winners.add(ticker) : losers.add(ticker); }
+            if (sector) { isWinner ? winners.add(sector) : losers.add(sector); }
         }
-    }
 
-    async getResult() {
-        const byUser = {};
-
-        for (const [userId, stats] of this.userStats) {
-            if (stats.winners.size > 0 || stats.losers.size > 0) {
-                byUser[userId] = {
-                    winners: Array.from(stats.winners),
-                    losers: Array.from(stats.losers)
-                };
-            }
+        if (winners.size > 0 || losers.size > 0) {
+            this.results[userId] = {
+                winners: Array.from(winners),
+                losers: Array.from(losers)
+            };
         }
+    }
 
-        // Return flat object. Orchestrator handles 1MB limit.
-        return { by_user: byUser };
+    async getResult() {
+        // [FIX 4] Return 'this.results' directly.
+        // This is a FLAT object { "user1": {...}, "user2": {...} }
+        // The ResultCommitter can now shard this into 50-user chunks if needed.
+        return this.results;
     }
     
-    reset() { this.userStats.clear(); }
+    // [FIX 5] No manual reset needed, StandardExecutor does it.
 }
 
 module.exports = AssetPnlStatus;

package/calculations/core/user-history-reconstructor.js

@@ -82,9 +82,7 @@ class UserHistoryReconstructor {
             events.push({ time: openTime, type: 'OPEN', trade, ticker });
 
             // Add CLOSE event (if closed)
-            if (closeTime) {
-                events.push({ time: closeTime, type: 'CLOSE', trade, ticker });
-            }
+            if (closeTime) { events.push({ time: closeTime, type: 'CLOSE', trade, ticker }); }
         }
 
         // Sort events by time

package/calculations/pyro/squeeze-potential.js

@@ -49,8 +49,8 @@ class SqueezePotential {
 
         for (const ticker of tickers) {
             const shortSlAvg = signals.getMetric(computed, 'speculator-stop-loss-distance-by-ticker-short-long-breakdown', ticker, 'avg_short_sl_distance_pct');
-            const userCount = signals.getMetric(computed, 'short-position-per-stock', ticker, 'short_count');
-            const weight = signals.getMetric(computed, 'short-position-per-stock', ticker, 'total_short_exposure_weight');
+            const userCount  = signals.getMetric(computed, 'short-position-per-stock', ticker, 'short_count');
+            const weight     = signals.getMetric(computed, 'short-position-per-stock', ticker, 'total_short_exposure_weight');
 
             if (userCount > 0) {
                 result[ticker] = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aiden-shared-calculations-unified",
-  "version": "1.0.107",
+  "version": "1.0.108",
   "description": "Shared calculation modules for the BullTrackers Computation System.",
   "main": "index.js",
   "files": [