fix: Increase Strictness of Rollback Check

Now that items don't receive a trade cooldown on reversal, their asset ID doesn't change either.

This change scopes the check to also ensure that the buyer/seller pair is aligned as well.

Ref CSF-1401

Author: Step7750

src/lib/alarms/rollback.ts

@@ -30,7 +30,10 @@ function findRollbackTrades(pendingTrades: SlimTrade[], tradeHistory: TradeHisto
 
         // Does it correspond to an active CSFloat sale?
         const csfloatTrade = pendingTrades.find(
-            (e) => e.state === TradeState.PENDING && all_ids.includes(e.contract.item.asset_id)
+            (e) =>
+                e.state === TradeState.PENDING &&
+                all_ids.includes(e.contract.item.asset_id) &&
+                (trade.other_party_id === e.seller_id || trade.other_party_id === e.buyer_id)
         );
         if (!csfloatTrade) {
             continue;

src/lib/alarms/trade_history.ts

@@ -147,6 +147,7 @@ export async function getTradeHistoryFromAPI(
         .map((e) => {
             return {
                 other_party_url: `https://steamcommunity.com/profiles/${e.steamid_other}`,
+                other_party_id: e.steamid_other,
                 received_assets: (e.assets_received || [])
                     .filter((e) => e.appid === AppId.CSGO)
                     .map((e) => {
@@ -193,6 +194,7 @@ function parseTradeHistoryHTML(body: string): TradeHistoryStatus[] {
     const statuses = [...links].map((e) => {
         return {
             other_party_url: `https://steamcommunity.com/${e[1]}`,
+            other_party_id: '',
             received_assets: [],
             given_assets: [],
             trade_id: '',

src/lib/bridge/handlers/trade_history_status.ts

@@ -11,6 +11,7 @@ export interface TradeHistoryStatus {
     trade_id: string;
     status: number;
     other_party_url: string;
+    other_party_id: string;
     received_assets: TradeHistoryAsset[];
     given_assets: TradeHistoryAsset[];
     time_init: number;