Skip to content

Display Trade Reversal Status on Steam Profile#371

Open
hayesZach wants to merge 15 commits intomasterfrom
feature/reversal-status
Open

Display Trade Reversal Status on Steam Profile#371
hayesZach wants to merge 15 commits intomasterfrom
feature/reversal-status

Conversation

@hayesZach
Copy link

@hayesZach hayesZach commented Mar 12, 2026

This PR introduces reversal status fetching when viewing a player's steam profile and displays a warning if they have reversed.

image

Closes CSF-774


Note

Medium Risk
Introduces a new external API integration (reverse.watch) and injects new UI into Steam profile pages, which could affect page rendering and error handling if the third-party service is unavailable or changes.

Overview
Adds a new FETCH_REVERSAL_STATUS bridge handler that calls the reverse.watch API (new reverse_watch_base_api_url environment config) to retrieve trade reversal status for a Steam ID.

Injects a new ReversalStatus component on Steam profile pages that fetches/caches reversal status (30-min TTL) and shows a warning banner with “days since last trade reversal” plus an info link when the user has reversals.

Written by Cursor Bugbot for commit b93001c. This will update automatically on new commits. Configure here.

@hayesZach hayesZach self-assigned this Mar 12, 2026
@hayesZach hayesZach changed the title Display Trade Reversal Status Display Trade Reversal Status on Steam Profile Mar 12, 2026
@hayesZach hayesZach requested review from GODrums and Step7750 March 12, 2026 22:46
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Autofix Details

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Non-reactive property used in render output
    • I marked reversalStatus with @state() so updates to it now trigger Lit re-renders for daysSinceLastReversal output.

Create PR

Or push these changes by commenting:

@cursor push 6055f782aa
Preview (6055f782aa)
diff --git a/src/lib/components/profile/reversal_status.ts b/src/lib/components/profile/reversal_status.ts
--- a/src/lib/components/profile/reversal_status.ts
+++ b/src/lib/components/profile/reversal_status.ts
@@ -14,6 +14,7 @@
     @state()
     show: boolean = false;
 
+    @state()
     reversalStatus: FetchReversalStatusResponse | undefined = undefined;
 
     static styles = [

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

@hayesZach hayesZach requested a review from Step7750 March 13, 2026 19:13
@hayesZach hayesZach requested a review from Step7750 March 14, 2026 01:25
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Copy link
Member

@Step7750 Step7750 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great overall! Just a few small comments.

We should wait until the service is up and running before merging. I'll defer to @GODrums for final approval.

Copy link
Collaborator

@GODrums GODrums left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for the late review. Pretty cool addition!


.powered-by-link {
text-decoration: none;
color: #ebebeb;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if the info-link gets a hover-effect, this one should probably get the same one to stay consistent

.warning {
display: inline;
align-items: flex-start;
justify-content: start;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

justify-content has no effect on inline display


.warning {
display: inline;
align-items: flex-start;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the default value, can be omitted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants