Skip to content

fix: preserve Twitch state on API failure - #110

Merged
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/preserve-twitch-state-on-api-failure
Aug 10, 2026
Merged

fix: preserve Twitch state on API failure#110
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/preserve-twitch-state-on-api-failure

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Distinguish Twitch API polling failures from a successful empty live-stream result.
  • Preserve existing subscription live state when Twitch status cannot be fetched, avoiding false offline transitions.
  • Pass the Quartz cancellation token through the live-status request.

Verification

  • dotnet restore Morpheus.sln — passed with existing NU1903 SQLite vulnerability warning.
  • dotnet build Morpheus.sln --no-restore -v minimal — passed with the same NU1903 warning.
  • dotnet test Morpheus.Tests/Morpheus.Tests.csproj --no-build --no-restore --filter FullyQualifiedName~TwitchServiceTests -v minimal — passed (7 tests).
  • Full dotnet test — 298 passed, 2 failed; the same two invariant-globalization failures reproduce on unchanged upstream/develop (baseline: 297 passed, 2 failed).
  • dotnet format Morpheus.sln --no-restore --verify-no-changes — skipped as a publish gate: the unchanged repository has broad pre-existing CRLF/import/final-newline violations.

Risk

  • Low: failed Twitch polling now leaves state untouched for the next scheduled poll; successful responses retain existing behavior.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev
vycdev merged commit 137a125 into vycdev:develop Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants