Skip to content

Show a frozen hosted session as paused before it is cleared - #1106

Merged
rowkav09 merged 1 commit into
mainfrom
fix/hosted-stalled-playing
Oct 2, 2026
Merged

rowkav09 merged 1 commit into
mainfrom
fix/hosted-stalled-playing

Conversation

@rowkav09

@rowkav09 rowkav09 commented Oct 2, 2026

Copy link
Copy Markdown
Member

Fixes #1100. Mirrors the Discord change in #1104: a stallAfterMs window (default 60s, always below stuckAfterMs) uploads a frozen playing session as paused at its position; the existing idle upload at 300s and the return to playing on movement are unchanged. Two existing hosted-loop expectations that assumed 'playing' while frozen were updated. New test fails without the change; full suite 1568/1568, npm run check ok.

@codecov

codecov Bot commented Oct 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

The hosted loop only reacted to a frozen position after stuckAfterMs (300s), so until then every poll uploaded the session as playing at a fixed position and the card kept showing a live track after the player closed without a clean stop. A stallAfterMs window (60s, below stuckAfterMs, same as the Discord loop in #1099) now uploads the item as paused at that position; the 300s idle upload is unchanged and moving playback goes back to playing. Fixes #1100.
@rowkav09
rowkav09 force-pushed the fix/hosted-stalled-playing branch from 919a760 to f45834a Compare October 2, 2026 16:44
@github-actions github-actions Bot added size:M and removed size:S labels Oct 2, 2026
@rowkav09
rowkav09 merged commit 6ed67ae into main Oct 2, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Hosted card keeps showing "playing" for up to 5 minutes after the player closes

1 participant